How to get the handness of  hand landmarks

Plugin Version or Commit ID
v0.10.1
Unity Version
2021.3.3f1
Your Host OS
MacOS BigSur 11.4
Target Platform
UnityEditor
Description
With the help of Wiki, I have successfully used this plugin to get the hand landmarks. Thank you very much for the great plugin.
Now, I would like to know if the acquired landmark is a right or left hand.
However, I don’t t know how to get the handness information of the acquired landmark from the outputstream. How can I do this?
Config Asset uses the following
mediapipe/graphs/hand_tracking/hand_tracking_desktop_live.pbtxt
Thank you !!
Code to Reproduce the issue
No response
Additional Context
No response

You can get the handedness by reading a value from the handedness stream with the same timestamp as the landmark.
I think you're initializing multiFaceLandmarksStream like this:
var multiFaceLandmarksStream = new OutputStream<NormalizedLandmarkListVectorPacket, List<NormalizedLandmarkList>>(_graph, "multi_face_landmarks");
multiFaceLandmarksStream.StartPolling().AssertOk();

In addition to that, you need to initialize handednessStream like this:
var handednessStream = new OutputStream<ClassificationListVectorPacket,
List<ClassificationList>>(_graph, "handedness");
handednessStream.StartPolling().AssertOk();

Note that the type of the value in the handedness stream is std::vector<ClassificationList> in C++ and the corresponding class in C# is List<ClassificationList>.
See also https://github.com/homuler/MediaPipeUnityPlugin/wiki/API-Overview#initialization-2.


hpi install/update command

Had an idea to make the install process perhaps easier, or at least automate some things - thought I'd leave it here -- personally I feel the most convenient way to use is through local editable installs, especially if someone is installing multiple (e.g. yours and mine) but I can imagine setting that up and managing updating it by git pulling can perhaps feel tedious for some users
So, this could add (one or two commands, or perhaps behind a click.group) to help manage that
Of course this isn't required, but for people who aren't as familiar with pip, it would automate some of the process.
Would define a directory like ~/.local/share/HPI where the repos sit and then running something like:
hpi repo install karlicoss/HPI clones your repo and pip install -es it
hpi repo install seanbreckenridge/HPI installs mine into another directory there (could manage name conflicts and the like)
hpi repo install could also probably just default to karlicoss/HPI
this would just for the most part subprocessing the pip/git commands.
And then, running hpi repo update would cd to those and git pull, printing updates to your console with git commit info
It does seem a bit like a monorepo command/plugin system like oh-my-zsh or doom something -- Not advocating for that, this is just to manage the editable installs. I guess they have the update commands for a reason, because would decrease the barrier to entry for someone not having to manage the editable installs or git
This would also be accessible by just doing pip install HPI, (which then would be uninstalled and the editable version would be reinstalled, which is fine if we're just execing a command like pip uninstall -y HPI && pip install -e /path/to/local/clone), and then let the user maintain local directories instead of the pypi installed HPI, so the instructions for an editable install could just be:
pip install HPI
hpi repo install

and then periodically
hpi repo update

which could loop over my.__path__, checking if a .git directory exists relative to it to update by git pulling
In [1]: import my

In [2]: my.__path__
Out[2]: _NamespacePath(['/home/sean/Repos/HPI/my', '/home/sean/Repos/HPI-fork/my'])

Can work on this if its something youre interested in adding

Hmm - it does add a bit of state, (probably just a JSON list in your data dir) but maybe even supporting
hpi repo install /path/to/local/clone
would be nice, allows you to place it wherever you want and is more flexible than just forcing a specific directory

I guess related to https://github.com/karlicoss/HPI/issues/12
Yeah, good idea!
I guess my only worry is that implementing it properly is going to be tricky, essentially it's a bit part of the plugin system. But if you wanna give it a go, would be cool! I'd try searching for something existing first, although last time I did, wasn't very fruitful sadly.
Btw, one way to get a lot of functionality for free is maybe using git submodules? E.g.  ~/.local/share/HPI could be a git repository, and karlicoss/HPI, seanbreckenridge/HPI would be submodules. Then it would be possible to do bulk operations, atomic updates/rollbacks etc. Not sure if there is some prior art, so just throwing in the idea, maybe it doesn't work well.

tried to implement some sort of this but ended up just spinning off into a a separate, generic project: clone-repos
Seems as if there are just too many edge cases and probably isnt worth the maintenance cost here
I have two clone-repos config files which let me manage hpi installs/updates:
https://github.com/seanbreckenridge/dotfiles/blob/master/.config/clone-repos/hpi-clone-repos.yaml
https://github.com/seanbreckenridge/dotfiles/blob/master/.config/clone-repos/update-forks.yaml
one that installs stuff and another that git pull upstream masters:
$ clone-repos ~/.config/clone-repos/update-forks.yaml
HPI-karlicoss: target /home/sean/Repos/HPI-karlicoss already exists
HPI-karlicoss: running preinstall 'git pull'
Already up to date.
HPI-karlicoss: running postinstall 'git pull upstream master'
From https://github.com/karlicoss/HPI
 * branch            master     -> FETCH_HEAD
Already up to date.
HPI-karlicoss: running postinstall 'git push'
Everything up-to-date
promnesia-fork: target /home/sean/Repos/promnesia-fork already exists
promnesia-fork: running preinstall 'git pull'
Already up to date.
promnesia-fork: running postinstall 'git pull upstream master'
From https://github.com/karlicoss/promnesia
 * branch            master     -> FETCH_HEAD
Already up to date.
promnesia-fork: running postinstall 'git push'
Everything up-to-date


Flutter Web: Resizing the crop grip does not work on web

When using Crop with a mouse on a Windows browser the Package works perfectly.
When using Crop on a device, Pixel One, WEB BROWSER, I can't resize the crop grid.  Moving the crop grid is very difficult.
Is there a trick to resizing the crop grid?
Is there a trick to moving the crop grid easily?
Thank you!

I was using full grid width, when I changed it to cropController.crop = const Rect.fromLTRB(0.1, 0.1, 0.9, 0.9); the resizing and moving worked perfectly.


Binding in iteration failing

Hi!
I had a Query and a Pojo.
I did the following
Query x = conn.createQuery(RECORD_TO_PROCESS);
for (MyPojo p : pojoList){
x.bind(p).executeUpdate();
}
This only did the update on the first pojo, because the binding didn't go away. I found no "clearParameters" method which would unbind.
I suspect the problem is in the Query class, on line 283:
if (addedParameters.contains( property.name )) continue;

Actually, bind() is now unusable except in the simplest one-off query cases. For instance, if you use bind() in the context of a batch query, it will only bind the first object (all others get skipped due to the addedParameters.contains() check).
Outside of a complete separation of the Query population phase from the Query execution phase, I think you've only got a few options.

Create a method that will clear the addedParameters set and either make it public or call it after each method that completes a unit of work (executeXXX and addToBatch). The inclusion of addToBatch is important, as it completes the unit of work (setting parameters) for a single item within a batch.

Get rid of the addedParameters concept completely and just document that a bind() acts like any other addParameter call and will overwrite parameters that already exist. If there is a parameter in the POJO that you want to overwrite/skip, simply bind() first and follow with the override.

I'm having to completely rewrite a batch loader due to this bug. I now have to use addParameter for every field in my POJOs. It's kind of negates the "sql2o handles POJOs" aspect of sql2o.

For anyone who needs to work around the bind bug, here's an alternate bind method that does the same thing as bind() but does not check for overwriting previous addParameters calls - i.e., it's behaviour is exactly the same as addParameter. As it's not part of the class, it takes Query as its first parameter.
private static Query bind(Query query, final Object pojo) {
    Class clazz = pojo.getClass();
    Map<String, PojoIntrospector.ReadableProperty> propertyMap = PojoIntrospector.readableProperties(clazz);
    for (PojoIntrospector.ReadableProperty property : propertyMap.values()) {
        try {
            if( query.getParamNameToIdxMap().containsKey(property.name)) {
                @SuppressWarnings("unchecked")
                final Class<Object> type = (Class<Object>) property.type;
                query.addParameter(property.name, type, property.get(pojo));
            }
        }
        catch(IllegalArgumentException ex) {
            logger.debug("Ignoring Illegal Arguments", ex);
        }
        catch(IllegalAccessException | InvocationTargetException ex) {
            throw new RuntimeException(ex);
        }
    }
    return query;
}

I think you are right, that the bind() method should act as any of the other addParameter() methods. This is also how it worked originally. The behavior was changed when issue #160 was fixed. I realize that this change wasn't thought through.

For all who working with column Mappings
private static Query bind(Query query, final Object pojo) {
    Class clazz = pojo.getClass();
    Map<String, PojoIntrospector.ReadableProperty> propertyMap = PojoIntrospector.readableProperties(clazz);
    for (PojoIntrospector.ReadableProperty property : propertyMap.values()) {
        try {
            final String name = query.getColumnMappings()
                .entrySet()
                .stream()
                .filter(e -> {
                    if (query.isCaseSensitive()) {
                        return e.getValue().contentEquals(property.name);
                    } else {
                        return e.getValue().contentEquals(property.name.toLowerCase());
                    }
                })
                .findFirst()
                .map(Map.Entry::getKey)
                .orElse(property.name);
            if (query.getParamNameToIdxMap().containsKey(name)) {
                @SuppressWarnings("unchecked")
                final Class<Object> type = (Class<Object>) property.type;
                query.addParameter(name, type, property.get(pojo));
            }
        } catch (IllegalArgumentException | IllegalAccessException | InvocationTargetException ex) {
            throw new RuntimeException(ex);
        }
    }
    return query;
}

For all who working with column Mappings
private static Query bind(Query query, final Object pojo) {
    Class clazz = pojo.getClass();
    Map<String, PojoIntrospector.ReadableProperty> propertyMap = PojoIntrospector.readableProperties(clazz);
    for (PojoIntrospector.ReadableProperty property : propertyMap.values()) {
        try {
            final String name = query.getColumnMappings()
                .entrySet()
                .stream()
                .filter(e -> {
                    if (query.isCaseSensitive()) {
                        return e.getValue().contentEquals(property.name);
                    } else {
                        return e.getValue().contentEquals(property.name.toLowerCase());
                    }
                })
                .findFirst()
                .map(Map.Entry::getKey)
                .orElse(property.name);
            if (query.getParamNameToIdxMap().containsKey(name)) {
                @SuppressWarnings("unchecked")
                final Class<Object> type = (Class<Object>) property.type;
                query.addParameter(name, type, property.get(pojo));
            }
        } catch (IllegalArgumentException | IllegalAccessException | InvocationTargetException ex) {
            throw new RuntimeException(ex);
        }
    }
    return query;
}

It works. Thanks!

For all who working with column Mappings
private static Query bind(Query query, final Object pojo) {
    Class clazz = pojo.getClass();
    Map<String, PojoIntrospector.ReadableProperty> propertyMap = PojoIntrospector.readableProperties(clazz);
    for (PojoIntrospector.ReadableProperty property : propertyMap.values()) {
        try {
            final String name = query.getColumnMappings()
                .entrySet()
                .stream()
                .filter(e -> {
                    if (query.isCaseSensitive()) {
                        return e.getValue().contentEquals(property.name);
                    } else {
                        return e.getValue().contentEquals(property.name.toLowerCase());
                    }
                })
                .findFirst()
                .map(Map.Entry::getKey)
                .orElse(property.name);
            if (query.getParamNameToIdxMap().containsKey(name)) {
                @SuppressWarnings("unchecked")
                final Class<Object> type = (Class<Object>) property.type;
                query.addParameter(name, type, property.get(pojo));
            }
        } catch (IllegalArgumentException | IllegalAccessException | InvocationTargetException ex) {
            throw new RuntimeException(ex);
        }
    }
    return query;
}

It works. Thanks!


Use refile with polymorphic relation

I'm not sure is it a bug or not, but it seems like refile can't handle polymorphic relations.
as example, I have 2 models:
class Picture < ActiveRecord::Base
  attachment :file, type: :image

  belongs_to :pictured, polymorphic: true

and
class Cleaning < ActiveRecord::Base
  has_many :pictures, as: :pictured
  accepts_attachments_for :pictures, append: true, attachment: :file

so now when I send request with "cleaning"=>{"pictures_files"=>["[{\"id\":\"192538c7bbcb0d983c97e2d1bae2cc6c0cdbeaaf8f60fae56399ca54e0bf\",\"filename\":\"38.jpg\",\"content_type\":\"image/jpeg\",\"size\":299980}]"]}
I get error No relation cleaning on Picture
but of course, everything works when I replace belongs_to :pictured, polymorphic: true to belongs_to :cleaning but it's not what I want :)
Thanks for any your help and suggestions.

Why did you close this issue? Did you resolve the problem?

yes, probably the problem was with counter_culture and after_save callback


Adding tutorial video and updating several pages to fix grammar and spelling

Motivate of the pull request

[ ] To address an existing issue. If so, please provide a link to the issue:
[ ] Breaking new feature. If so, please describe details in the description part.

Describe the technical details
What is currently done? (Provide issue link if applicable)

What does this pull request change?

This pull request adds the tutorial video for user_defined_template.v, and fixes grammar and spelling on the first few pages of the documentation page.

Which part of the code base require a change

[ ] VPR
[ ] Tileable routing architecture generator
[ ] OpenFPGA libraries
[ ] FPGA-Verilog
[ ] FPGA-Bitstream
[ ] FPGA-SDC
[ ] FPGA-SPICE
[ ] Flow scripts
[ ] Architecture library
[ ] Cell library
[x ] Documentation
[ ] Regression tests
[ ] Continous Integration (CI) scripts

Impact of the pull request

[ ] Require a change on Quality of Results (QoR)
[ ] Break back-compatibility. If so, please list who may be influenced.

@bbleaptrot Looks good to me but you need to resolve conflicts between merging.


Running 'Jaguar: Flash' action while having an open 'Jaguar: Monitor' terminal misbehaves

Looks like we just try to issue the command 'jag flash ...' in the same terminal without stopping the running program. I would expect this to run in another terminal and fail instead.

It is exactly what we do. You can only have one thing running on each port at a time which is why we reuse the terminal. If we run it in a new terminal, then you can end up with many terminals and will have to manually close all the terminals. I am not convinced that is better.

We can send a Ctrl-C to make sure that everything is actually sent. Would that be better?

Do we know that we're reusing the terminal (if it is already running)? If so, we could also just abandon the operation because it is not going to work and tell the user.

We could know whether the terminal is already running (right now we don't), but we do not know whether the previous command has exited. I think there are two way to do this but neither is optimal.
Using VS Code output
We could use an output instead of a terminal. In this case, we would use the node child_process lib to execute the commands and just forward the output to the appropriate output window.
The advantage would be that we can use the child process to keep track of the state of the command. In other words, we could know whether the previous command is still running.
The disadvantage here is that there is no good way to cancel a running command. We cannot extend the output view with buttons for interrupting the child process which I assume is to prevent extension developers to use outputs for other things than output. This issue extends to commands that require user interaction where we have no way of doing the interaction (we try to not use commands that require user interaction, but we have had changes to the CTL that would have been breaking for the VS Code extension if we were using output). If a command that uses a port doesn't terminate, then we have blocked that port (until it is reconnected) which is not great.
Using VS Code terminal
This is what we use. We open a terminal and send commands to it.
The advantage is that it is simple and a user can always interact with the terminal. The disadvantage is that it is simple, so we cannot know whether commands are running.


Corrected a typo in the bcm283x PWM implementation.

Computation of PWM resolution had desired and base frequencies reversed.

I've signed the CLA

Thanks!


on Mac, homebrew boost not found

(on a Mac M2)
When building outside of meson-python, meson build happily finds homebrew boost:
Checking potential boost root /Library/Developer/CommandLineTools/SDKs/MacOSX13.3.sdk/usr/include/c++
Checking potential boost root /Library/Developer/CommandLineTools/usr/lib/clang/14.0.3
Checking potential boost root /Library/Developer/CommandLineTools/SDKs/MacOSX13.3.sdk/usr
Checking potential boost root /Library/Developer/CommandLineTools/usr
Checking potential boost root /opt/homebrew/Cellar/boost/1.82.0_1
  - potential library dirs: ['/opt/homebrew/Cellar/boost/1.82.0_1/lib']
  - potential include dirs: ['/opt/homebrew/Cellar/boost/1.82.0_1/include']
  - found boost library dir: /opt/homebrew/Cellar/boost/1.82.0_1/lib
  - found boost 1.82.0 include dir: /opt/homebrew/Cellar/boost/1.82.0_1/include

However, inside meson-python, it is not found:
Checking potential boost root /Library/Developer/CommandLineTools/SDKs/MacOSX13.3.sdk/usr/include/c++
Checking potential boost root /Library/Developer/CommandLineTools/usr/lib/clang/14.0.3
Checking potential boost root /Library/Developer/CommandLineTools/SDKs/MacOSX13.3.sdk/usr
Checking potential boost root /Library/Developer/CommandLineTools/usr
Checking potential boost root /usr/local
Checking potential boost root /usr
Run-time dependency Boost found: NO (tried system)

Any ideas what could be causing this?

I don't think that meson-python is doing anything that could affect the boost dependency lookup. Are the meson version used in the two cases exactly the same? If you build with the default pip or build settings, the build happens in an isolated environment where the most recent version of meson (unless pinned by other components) is installed. This may be a different version that the one installed on the system.
To further debug this we would need to look at the meson detailed log for the working and not working case. Meson prints the location of the log file after the error message. To access the log file when using meson-python and building with pip or build you need to explicitly specify a build directory adding -Cbuilddri=something to the command line.

Also, is this a public project that I can use to try to replicate the issue?

yep, same version of meson. However it seems homebrew tweaks boost.py for meson:
https://github.com/Homebrew/homebrew-core/blob/9f3bb46766f10551a1e2c8b47af16c3cb91a7c95/Formula/m/meson.rb#L30
Not sure what a good fix for this is :/

You can disable isolated builds, passing the --no-build-isolation command line option to pip. In this way the system meson is used, but you need to install all other build dependencies for the system python. There may be other tricks you can play with environment variables to have meson look for boost in the right place. The only proper fix is to have Homebrew submit their patches upstream.

On m1/2 brew installs in /opt/homebrew
I’m prepping a patch for meson to also search here if HOMEBREW_PREFIX is set
On Tue, 29 Aug 2023 at 16:04, Daniele Nicolodi @.***>
wrote:

Looking at this a bit more in detail, it seems that the issue should
manifest only with an non-standard HOMEBREW_PREFIX, ie with boost
installed somewhere else than in /usr/local/. Can you confirm?
—
Reply to this email directly, view it on GitHub
https://github.com/mesonbuild/meson-python/issues/462#issuecomment-1697623783,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB4O4F7SIFYQZDB2V5QQX3XXYAGRANCNFSM6AAAAAA4CZ6YTQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

On m1/2 brew installs in /opt/homebrew

Interesting. Did the default change for the arm macs or on more recent macOS releases? On the intel macs the default location is /usr/local.

I’m prepping a patch for meson to also search here if HOMEBREW_PREFIX is set

This is most likely the best solution.

It came in for M1s, not sure if its now default for all archs.


CentOs 7 - Cannot Start Service /usr/bin/env: ruby: No such file or directory

Installing Dashing on CentOS 7 (I'm using CentOS 7.6.1810) is a bit of a chore thanks to CentOS no longer having an update ruby vesion in the general repositories. So I had to install Ruby this way - I installed rh-ruby25-ruby-devel but the process is the same.
I followed all of the instructions and made sure the bundle installed all of the gems. I amended the paths for the dashing-icinga2.service from /usr/local/bin/dashing to /opt/rh/rh-ruby25/root/usr/bin/dashing. Running /opt/rh/rh-ruby25/root/usr/bin/dashing -p 8005 gives me an error every 5 seconds or so (but I think this may be a red herring as it appears to keep running):
================================================================================
scheduler caught exception:
401 Unauthorized
/opt/rh/rh-ruby25/root/usr/share/gems/gems/rest-client-2.0.2/lib/restclient/abstract_response.rb:223:in `exception_with_response'
/opt/rh/rh-ruby25/root/usr/share/gems/gems/rest-client-2.0.2/lib/restclient/abstract_response.rb:103:in `return!'
/opt/rh/rh-ruby25/root/usr/share/gems/gems/rest-client-2.0.2/lib/restclient/request.rb:809:in `process_result'
/opt/rh/rh-ruby25/root/usr/share/gems/gems/rest-client-2.0.2/lib/restclient/request.rb:725:in `block in transmit'
/opt/rh/rh-ruby25/root/usr/share/ruby/net/http.rb:910:in `start'
/opt/rh/rh-ruby25/root/usr/share/gems/gems/rest-client-2.0.2/lib/restclient/request.rb:715:in `transmit'
/opt/rh/rh-ruby25/root/usr/share/gems/gems/rest-client-2.0.2/lib/restclient/request.rb:145:in `execute'
/opt/rh/rh-ruby25/root/usr/share/gems/gems/rest-client-2.0.2/lib/restclient/request.rb:52:in `execute'
/opt/rh/rh-ruby25/root/usr/share/gems/gems/rest-client-2.0.2/lib/restclient/resource.rb:51:in `get'
/usr/share/dashing-icinga2/lib/icinga2.rb:236:in `getApiData'
/usr/share/dashing-icinga2/lib/icinga2.rb:258:in `getIcingaApplicationData'
/usr/share/dashing-icinga2/lib/icinga2.rb:662:in `run'
/usr/share/dashing-icinga2/jobs/icinga2.rb:27:in `block in <top (required)>'
/opt/rh/rh-ruby25/root/usr/share/gems/gems/rufus-scheduler-2.0.24/lib/rufus/sc/jobs.rb:230:in `trigger_block'
/opt/rh/rh-ruby25/root/usr/share/gems/gems/rufus-scheduler-2.0.24/lib/rufus/sc/jobs.rb:204:in `block in trigger'
/opt/rh/rh-ruby25/root/usr/share/gems/gems/rufus-scheduler-2.0.24/lib/rufus/sc/scheduler.rb:430:in `block in trigger_job'
================================================================================

After copying the dashing-icinga2.service to the correct systemd folder and running sudo systemctl start dashing-icinga2 it immediately returns an error message and if I look at journalctl -xe I see:
-- Unit dashing-icinga2.service has begun starting up.
Jul 05 21:15:03 ip-172-30-0-241.ec2.internal dashing[6588]: /usr/bin/env: ruby: No such file or directory
Jul 05 21:15:03 ip-172-30-0-241.ec2.internal systemd[1]: dashing-icinga2.service: main process exited, code
Jul 05 21:15:03 ip-172-30-0-241.ec2.internal systemd[1]: Failed to start Dashing-Icinga2 host/service/netwo
-- Subject: Unit dashing-icinga2.service has failed
-- Defined-By: systemd
-- Support: http://lists.freedesktop.org/mailman/listinfo/systemd-devel
--
-- Unit dashing-icinga2.service has failed.

If I run /usr/bin/env ruby it just sits there until I press Ctrl + C. If I run which ruby I get /opt/rh/rh-ruby25/root/usr/bin/ruby.
Unfortunately the instructions don't take into account how things have changed with CentOS. Which is fine. If you're not using it you'll not know until somebody reports it. However I'm a bit lost now on where to do.
Any help appreciated.

CentOS no longer having an update ruby vesion in the general repositories.

Not sure why you would need to update the Ruby version. The Vagrant boxes use CentOS 7 as base, and Dashing runs inside like a charm. Without any SCL packages.

scheduler caught exception:
401 Unauthorized

The error is that the provided API credentials are wrong, you can modify such in the configuration file for dashing-icinga2.

/usr/bin/env: ruby: No such file or directory

Maybe Systemd needs more specific environment variables being set, when you change from native ruby to SCL.

Not sure why you would need to update the Ruby version. The Vagrant boxes use CentOS 7 as base, and Dashing runs inside like a charm. Without any SCL packages.

The verion of Ruby that was installed out of the box is 2.0.0. When I went to run the bundle step:
cd /usr/share/dashing-icinga2
bundle

I got an error that bundle required a minimum of Ruby 2.3.0. That's why I went to update, found out 2.0.0 was way out of date and searched for how to update it and came across SCL.

The error is that the provided API credentials are wrong, you can modify such in the configuration file for dashing-icinga2.

Hmmm. The credentials I made sure matched. I'll take a look.

Maybe Systemd needs more specific environment variables being set, when you change from native ruby to SCL.

That's where my knowledge will therefore stop, sadly 😢 SCL, from my limited understanding, is supposed to be a replacement for the native Ruby so all the environment variables and config should be updated by it.

The Vagrant boxes have EPEL enabled by default and then install bundler via packages. This is abstracted in this Puppet profile: https://github.com/Icinga/icinga-vagrant/blob/master/.puppet/modules/profiles/manifests/dashing/icinga2.pp
When you're going the gem install bundler route, I could imagine that bundler doesn't like the old Ruby version. Still, with the packaged variant, without updating bundler when asking from its run, it should work. Otherwise the Vagrant runs would fail.


cellxgene-schema must warn when a Seurat conversion cannot be performed

Context
See conversations on ux and ingestion.
Also - Seurat Failure Investigations.

Design
Note: Updated to reflect the comment thread below.
When present, both raw and final matrices must be examined - adata.X and adata.raw.X. See matrix layers for background.
If the matrix includes more than 2^32-1 non-zero elements, it exceeds the constraints of the R (dgCMatrix). Subsequent seurat conversion on the portal will fail.
The cellxgene-schema CLI MUST display this warning:
This dataset cannot be converted to the .rds (Seurat v3) format. fill-in-the-blank nonzero elements in fill-in-the-blank-matrix-name exceed the limitations in the R dgCMatrix sparse matrix class<PHONE_NUMBER>  nonzero elements).
and then continue processing

Just to clarify, this should be done for both the raw and the final matrices.

So do this check for adata.X and adata.raw.X (if available)

Just to be precise here, the following message does not accurately state the possible reasons for failing seurat convertibility due to size limitations:

This dataset cannot be converted to the .rds (Seurat v3) format. fill-in-the-blank nonzero elements in fill-in-the-blank-matrix-name exceed the limitations in the R dgCMatrix sparse matrix class<PHONE_NUMBER> nonzero elements).

This is only true for sparse matrices. For dense matrices, failure will occur if any of the dimensions exceed 2 ^ 31 - 1 irrespective of whether those elements are nonzero.

Added dependency since this change to cellxgene-schema needs to be released to PyPI before the corresponding change to Portal in:

https://app.zenhub.com/workspaces/single-cell-5e2a191dad828d52cc78b028/issues/chanzuckerberg/single-cell-data-portal/1579

(otherwise the Portal unit tests in CI will fail)

Deployed to prod yesterday.


Update the Timeline of the ACF page

Describe the bug
The timeline of the ACF page is not up-to-date
Expected behavior
The passed dates should be highlighted in green colour(Success)
Screenshots
If applicable, add screenshots to help explain your problem.

Are you working on it?
No

Shall I work on this?


Fixed getFormParams() to work with DB prefixes

I found one additional spot that should use getTableName() so it will work with prefixed databases.

Good find, thanks


ExcelPainter - table_properties argument & XLSX_Formats class.

The new table_properties argument allows the user to control the format of the tables by passing a dict of the properties to amend, to the ExcelPainter. The new XLSX_Formats class will be update these from their default.

This PR resolves #183.


feat(apps/whale-api): surface "paybackburn" type as string[]

What this PR does / why we need it:
Which issue(s) does this PR fixes?:

Fixes #1600
Additional comments?:

ohh.. damn!!
it get closed after dropped the first commit


How to snapshot view with brightness(_ amount: Double) modifier?

SnapshotTesting uses GitHub issues for bugs. For more general discussion and help, please use GitHub Discussions.
I'm trying to snapshot a view with brightness(_ amount: Double) modifier. Not sure if it is a bug or i do something wrong. My view looks like this:
let view = Color.red
      .frame(width: 50, height: 50)
      .padding()
      .background(Circle().fill(Color.red).brightness(0.5))

It is wrapped inside UIHostingViewController for snapshot test.
The recorded snapshot looks like this (like the brightness is not applied at all):

When view is rendered in app correctly (screenshot from simulator):

Looks like a bug to me, but I'm not sure so I want to ask a question if I miss something?

+1


Current status of Zinc, or what are the "must have's" before stable release

I've had a discussion with @eed3si9n about what's currently missing in Zinc. I want to share this information here so that interested contributors can have a view of the current status of the project.

[ ] Fix handling of anonymous and local classes defined in Java. #86
[ ] Honor as seen from correctly for inherited classes (current Zinc is undercompiling). @gkossakowski has talked about this here, here and here.
[ ] Create a Java binary compatible API that build tools can use (Gradle, Pants, SBT, Bazel). https://github.com/sbt/zinc/blob/1.0/internal/compiler-interface/src/main/java/xsbti/compile/IncrementalCompiler.java

After finishing up the work on class-deps and the name-hashed algorithm, there would be some benchmarking involved to assess the real speedup on medium-to-large codebases.
Eugene has said that he could work on item 3 while others tackle 1 and 2, which require some knowledge on scalac internals.
After these items are finished, there will be a beta version that people can depend on and try. Given the modularity among Zinc and the build tools, it is possible for Scala teams to assess the benefits of the algorithm by:

Cloning latest master
In your checkout, running sbt publishLocal
In your sbt project changing version to 0.13.12-SNAPSHOT and running sbt normally.

It would be great if @gkossakowski can confirm this list of missing tasks.

Honor as seen from correctly for inherited classes (current Zinc is undercompiling).

This issue has always been present in sbt incremental compiler and is still present in the 0.13 branch and in the 1.0 branch (except if you use dotty, which has its own fixed compiler bridge), so it should not be a blocker for a stable release.

Create a Java binary compatible API that build tools can use (Gradle, Pants, SBT, Bazel).

Are you also going to patch sbt 0.13 so that it can optionally use zinc? (Note that sbt 1.0 already depends on zinc, and that the only reason these changes were not backported to the sbt 0.13 incremental compiler is that they would break binary compatibility)

This issue has always been present in sbt incremental compiler and is still present in the 0.13 branch and in the 1.0 branch (except if you use dotty, which has its own fixed compiler bridge), so it should not be a blocker for a stable release.

When Zinc is released, we'd like to promise 2 years of binary compatibility at least. Even though this is an issue present in previous Zinc versions, it has a great effect in the correctness of the incremental compiler and we should aim at 100% correctness if possible before a stable release.

Are you also going to patch sbt 0.13 so that it can optionally use zinc? (Note that sbt 1.0 already depends on zinc, and that the only reason these changes were not backported to the sbt 0.13 incremental compiler is that they would break binary compatibility)

Don't know yet :smile:. I'll see whether this is a feasible option in the future.

When Zinc is released, we'd like to promise 2 years of binary compatibility at least. Even though this is an issue present in previous Zinc versions, it has a great effect in the correctness of the incremental compiler and we should aim at 100% correctness if possible before a stable release.

I don't think fixing this issue will have any consequence on binary compatibility.

Don't know yet :smile:. I'll see whether this is a feasible option in the future.

It seems to me your time might be better used in trying to get sbt 1.0 ready (which means creating some sort of compatibility layer for 0.13 code to work with sbt 1.0) instead, since it already has everything you need.

I don't think fixing this issue will have any consequence on binary compatibility.
I don't know, but I wouldn't like to hypothesize. @eed3si9n and I agree that these are blockers for a stable release.

It seems to me your time might be better spent in trying to get sbt 1.0 ready (which means creating some sort of compatibility layer for 0.13 code to work with sbt 1.0) instead, since it already has everything you need.

Good suggestion, but the Scala Center has already agreed to have a look at Zinc and fix these issues as soon as possible so that not only sbt, but all the other build tools available in our community, can benefit from it.

I don't know, but I wouldn't like to hypothesize. @eed3si9n and I agree that these are blockers for a stable release.

Are other undercompilation issues considered blockers or just this one? I'm thinking of https://github.com/sbt/zinc/issues/101 for example, which affects the API of zinc, but it might not affect the "Java binary compatible API that build tools can use" you plan to build if it's high-level enough.

In fact, I would very much like you to expand on what exactly will be part of the the stable API you mentioned. I was planning to work with a student on fixing the various issues in zinc I reported last year (https://github.com/sbt/zinc/issues/created_by/smarter), this cannot be done without changing the API of zinc, so we need to make sure that these APIs are considered internal/unstable or that the issues get fixed first (but bear in mind that once zinc 1.0 actually gets used, many more issues might be reported and fixing them might affect the API too)

What will be consider API of zinc? Interfaces for end users? Analysis formats? Behavior of incremental compilation?
Internally we've got a lot of work on zinc and I want to know deadline for PRs :)

@eed3si9n @dwijnand I we might use zinc gitter channel (especially if zinc will become something more then internal sbt dependency).

In fact, I would very much like you to expand on what exactly will be part of the the stable API you mentioned.
I'll work on that with @eed3si9n when the correctness of the compiler is up to a level that deserves a stable release. I don't know details of how this API will look like yet.

Are other undercompilation issues considered blockers or just this one?
Ideally, all of them. I'll try to work on them so that Zinc can have a beta out before 28th February.

What will be consider API of zinc? Interfaces for end users? Analysis formats? Behavior of incremental compilation?
Yet to be decided, but I'll be making PRs soon -- I've just started to hack on this. I agree that it's good that we all decide how that API should look like, so I'm happy to chat about it either here or in Gitter. However, I'll leave the API design until the end since my priority is to complete Grzgorz's algorithm.

Hi,
The list of remaining tasks for zinc's 1.0 release looks complete to me. I thought I'd give you guys estimates of how much work will it take to fix them:

Fix handling of anonymous and local classes defined in Java. #86 -- 60 hours
Honor as seen from correctly for inherited classes (current Zinc is undercompiling). -- 200 hours
_Create a Java binary compatible API that build tools can use (Gradle, Pants, SBT, Bazel). _ -- 160 hours

Total: 420 hours ~ 2.5 months of full time work of a single person. My estimates are for a person already familiar with zinc/incremental compilation.
It's true that undercompiling in case of inheritance is already an issue but class-based deps make the issue more pressing. More granular dependency tracking creates more opportunities for undercompilation to pop up in real life scenarios.

I might be able to help with bazel integration because we use bazel at Stripe. I will have time to look at it only in March, though.

And the pantsbuild/pants project is happy to contribute as well.

I would probably contact you when I'm defining the public API of the compiler. We'll need the help of other build tools to spot "hot" use cases. @stuhood @gkossakowski

@jvican I think I can help with gradle and intellij

As a general note, I'm open to discuss that the must-have lists are.
@smarter  wrote:

In fact, I would very much like you to expand on what exactly will be part of the the stable API you mentioned.

For the Zinc API, sbt would need things like compilation, docs, and REPL.
I think it would be nice if we can get abstraction of the REPL so it can be driven programmatically. See https://github.com/sbt/zinc/issues/170
What I would not promise as the Zinc API point would be any implementation detail of how we keep track of incrementality, including Analysis and its text representation.
Also, now that @romanowski has sent the cached compilation PR, Zinc might as well embrace Bazel/Pants caching and try to make Analysis machine independent - https://github.com/sbt/zinc/issues/218

Older Java API discussion: https://github.com/sbt/zinc/issues/79

and try to make Analysis machine independent

This would be handy, as currently we do our own rewriting of the analysis file for portability.

We're done with this. The Scala Center will continue to pour some resources into Zinc, so we will follow up on some improvements to the Zinc algorithm and the binary API.


Cannot find module socket.io-client

Hi,
Version 1.2.1
We're using socket.io-client via browserify and we're getting the following error
cannot find module socket-io-client
when using
require('socket.io-client)
I'm assuming this is because we don't have the main entry in the bower.json file. I created a fork of the project and changed the filename to socket.io-client.js and created a new release to be able to use it in our project which ultimately resolved the issue.
Is this valid or if something I'm doing wrong? Thanks in advance.

I think your confused. You mention bower.json and browserify in the same issue. Browserify doesn't use bower.json it uses package.json.

I've just updated the description above. Thanks!

This should not be an issue anymore since we have the main entry now. Feel free to reopen.


merge package from arlac77/npm-package-template

package.json

chore(devDependencies): update documentation:^7.0.0 from template
chore(devDependencies): add rollup-plugin-babel:^3.0.4 from template
chore(devDependencies): add babel-preset-env:^1.6.1 from template

.travis.yml

chore(travis): remove node versions 10.1
chore(travis): add node versions 10.2

:tada: This PR is included in version 6.0.6 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Handle file open exceptions in the Exceptions module

In the filter_nolint method of the Exceptions module there is a call to open(). That call can raise exceptions that we should handle. The exceptions I am most interested in handling are FileNotFoundError, PermissionError, and OSError. The difficulty in handling the exceptions is adding unit tests. I tried using mock to trigger those exceptions on the open call. The tests would hang though, likely due to using with open() in the __init__ method. I tried getting an Exceptions object, then setting the mock side effect before calling filter_nolint, but the test would still hang. (Note that type of pattern is used by tests in the Config module.)
An example of a unit test I tried adding to tests/exceptions/test_exceptions.py:
import mock

@mock.patch("statick_tool.exceptions.open")
  def test_filter_noissues_permission_error(mock_open):
      """
      Test that issues are found even when a PermissionError is raised.

      Expected result: one issue found
      """
      exceptions = Exceptions(
          os.path.join(os.path.dirname(__file__), "package_exceptions.yaml")
      )

      filename = "/root.py"
      line_number = "3"
      tool = "pylint"
      issue_type = "missing-docstring"
      severity = "3"
      message = "C0111: Missing module docstring"
      tool_issue = Issue(filename, line_number, tool, issue_type, severity, message, None)
      issues = {}
      issues["pylint"] = [tool_issue]

      mock_open.side_effect = PermissionError("mocked error")
      issues = exceptions.filter_nolint(issues)
      assert len(issues["pylint"]) == 1

This originated with #344.

Do you want to add handling of PermissionError and OSError now that we have a unit test pattern to copy?


Eclipse plugin ignores force on dependency with relocation

Expected Behavior
In our build.gradle we reference a compile dependency, which has a relocation in its pom.xml file (e.g. see http://central.maven.org/maven2/com/lowagie/itext/4.2.0/itext-4.2.0.pom). Because of licensing issues we have to use this exact version of the itext dependency and we do not want to follow the relocation.
For this reason we added the following block to our build.gradle file:
configurations.all {
	resolutionStrategy {
		force group: 'com.lowagie', name: 'itext', version: '4.2.0'
		all*.exclude group: 'com.itextpdf', module: 'itextpdf'
	}
}
The expected output of gradle dependencies is now:

compile - Dependencies for source set 'main' (deprecated, use 'implementation ' instead).
--- com.lowagie:itext:4.2.0

compileClasspath - Compile classpath for source set 'main'.
--- com.lowagie:itext:4.2.0
[...]

This works fine with all Gradle versions. The task gradle eclipse generates the .classpath file for Eclipse. The result for Gradle versions 2.14, 3.5 and 4.1 is (expected and correct):
<classpathentry path="C:/Daten-lokal/Benutzer/ml79/gradle/caches/modules-2/files-2.1/com.lowagie/itext/4.2.0/77e08389e3fa7b0212b67702ba6e4dbbbff68ae5/itext-4.2.0.jar" kind="lib" sourcepath="C:/Daten-lokal/Benutzer/ml79/gradle/caches/modules-2/files-2.1/com.lowagie/itext/4.2.0/e975b1e422df52672e57ca3da650ea27038c266/itext-4.2.0-sources.jar"/>
With Gradle versions 4.2 and 4.2.1 the itext dependency is missing completely in the .classpath file, what is wrong. I found no way to get itext 4.2.0 into the .classpath file, neither with dependency substitution, nor with any force and excude combinations.
We are not able to use versions 4.2 and 4.2.1 because of this issue. It makes us too much effort to manipulate the .classpath file for round about one hundred projects manually.
Our build.gradle file:
apply plugin: 'java'
apply plugin: 'eclipse'
allprojects {
repositories {
mavenCentral()
}
}
dependencies {
compile group: 'com.lowagie', name: 'itext', version: '4.2.0'
}
configurations.all {
resolutionStrategy {
force group: 'com.lowagie', name: 'itext', version: '4.2.0'
all*.exclude group: 'com.itextpdf', module: 'itextpdf'
}
}
We are using Gradle Wrapper for execution.

Can you please attach a reproducible project as a zip?

Eclipse project to reproduce the issue.
lib-test.zip

Potentially related to #2166 . I'll take a look.

@gradlifier Here's what's happening.
In Maven, an artifact can be relocated via relocation tags in pom.xml (see here). Before Gradle 4.2, if A is relocated to B, A.jar and B.jar will both be downloaded, which causes potential issues (e.g. #1789 ), so we changed this behaviour in 4.2 and avoid downloading A.jar in this situation.
In most cases it's correct, but your case looks slightly different. In my understanding, if A is relocated to B, they're basically the same artifact (e.g org.apache.commons:commons-io:1.3.2 and commons-io:commons-io:1.3.2). However, your com.lowagie:itext:4.2.0 is relocated to com.itextpdf:itextpdf:5.5.6, but they're totally different. Could you explain a bit why is it?
IMHO, relocation element shouldn't be used here and what Gradle 4.2 does is the right thing. @bigdaz , WDYT?
$ tree -d itext-4.2.0
itext-4.2.0
├── META-INF
│   └── maven
│       └── com.lowagie
│           └── itext
└── com
    └── lowagie
        ├── rups
        │   ├── controller
        │   ├── io
        │   │   └── filters
        │   ├── model
        │   └── view
        │       ├── icons
        │       ├── itext
        │       │   └── treenodes
        │       └── models
        ├── text
        │   ├── error_messages
        │   ├── exceptions
        │   ├── factories
        │   ├── html
        │   │   └── simpleparser
        │   ├── pdf
        │   │   ├── codec
        │   │   │   └── wmf
        │   │   ├── collection
        │   │   ├── crypto
        │   │   ├── draw
        │   │   ├── events
        │   │   ├── fonts
        │   │   │   └── cmaps
        │   │   ├── hyphenation
        │   │   ├── interfaces
        │   │   ├── internal
        │   │   └── parser
        │   ├── rtf
        │   │   ├── direct
        │   │   ├── document
        │   │   │   └── output
        │   │   ├── field
        │   │   ├── graphic
        │   │   ├── headerfooter
        │   │   ├── list
        │   │   ├── parser
        │   │   │   ├── ctrlwords
        │   │   │   ├── destinations
        │   │   │   ├── enumerations
        │   │   │   ├── exceptions
        │   │   │   └── properties
        │   │   ├── style
        │   │   ├── table
        │   │   └── text
        │   └── xml
        │       ├── simpleparser
        │       └── xmp
        ├── toolbox
        │   ├── arguments
        │   │   └── filters
        │   ├── plugins
        │   └── swing
        └── tools
$ tree -d itextpdf-5.5.6
itextpdf-5.5.6
├── META-INF
│   └── maven
│       └── com.itextpdf
│           └── itextpdf
└── com
    └── itextpdf
        ├── awt
        │   └── geom
        │       ├── gl
        │       └── misc
        ├── testutils
        ├── text
        │   ├── api
        │   ├── error_messages
        │   ├── exceptions
        │   ├── factories
        │   ├── html
        │   │   └── simpleparser
        │   ├── io
        │   ├── l10n
        │   │   └── error
        │   ├── log
        │   ├── pdf
        │   │   ├── codec
        │   │   │   └── wmf
        │   │   ├── collection
        │   │   ├── crypto
        │   │   ├── draw
        │   │   ├── events
        │   │   ├── fonts
        │   │   │   ├── cmaps
        │   │   │   └── otf
        │   │   ├── hyphenation
        │   │   │   └── hyph
        │   │   ├── interfaces
        │   │   ├── internal
        │   │   ├── languages
        │   │   ├── parser
        │   │   │   └── clipper
        │   │   ├── qrcode
        │   │   └── security
        │   └── xml
        │       ├── simpleparser
        │       │   └── handler
        │       └── xmp
        └── xmp
            ├── impl
            │   └── xpath
            ├── options
            └── properties

@blindpirate I agree that we should only be including the artifacts and dependencies for the target of the relocation. There's no way to tell Gradle (or Maven I think) to ignore the relocation, and the issue has nothing to do with the use of force in the build.
As far as I'm concerned com.itext:itext:4.2.0 doesn't exist as a real module: you'll see that the POM file has no dependencies in it, and I have no idea why there is a itext-4.2.0.jar file present in the repository.
Including both itext-4.2.0.jar and itext-pdf-5.5.6.jar in the resolution was a bug in earlier versions of Gradle. Now we (correctly) only include itext-pdf-5.5.6.jar, and the user is explicitly excluding this jar.
Our handling of relocation is pretty broken in general. Instead of adding a dependency on the relocated module and then ignoring artifacts of the original module, we should probably treat a relocation like a dependency substitution, replacing the original dependency with the relocated one.
So instead of:
+--- com.lowagie:itext:4.2.0
|    +--- com.itextpdf:itextpdf:5.5.6

we should have:
+--- com.lowagie:itext:4.2.0 -> com.itextpdf:itextpdf:5.5.6 (POM relocation)

It might be possible to use a dependency like com.lowagie:itext:4.2.0@jar to get the artifact from the original location.

Hi together, many thanks for your help an the workaround with com.lowagie:itext:4.2.0@jar. That seems to work well for us now!
And yes, the version 4.2.0 of this artifact seems not to be an official one as stated in   https://stackoverflow.com/questions/37740725/how-to-migrate-com-lowagie-itext-form-2-1-7-to-4-2-0
So in a longer term we will switch to a newer version or a fork of iText.
Regarding the relocation you may be right. It does not make sense to "suppress" relocation, so I am closing the issue now.


Visual studio brace completion is a bit overzealous

First world problems. :smiley:
TypeScript Version:  <IP_ADDRESS>
Visual Studio Version: Community 2015, 14.0.25431.01 Update 3
Code
for ( let i = 0; i < arr.length; i++ )

Expected behavior:
You can just type the whole line in one breath.
Actual behavior:
When typing this line the brace completion feature also tries to complete the angle bracket, resulting at one point in code like this:
for ( let i = 0; i <>

This is a bit annoying, since the other bracket now needs to be deleted. The same thing happens when trying to type anything with a "less than" comparison involved.
I don't know if proper context awareness is feasible for bracket completion, but perhaps the auto completion can be skipped for angle brackets? Or maybe it can be a separate setting for people with different tastes. I find that in my code I use comparisons more often that casts or generics, so the auto completion for angle brackets is doing more harm than good. But I like it for other kinds of brackets and strings, so the net gain is positive.

I'm facing the same problem. Just commenting so the issue gets more attention. :)

In VS 2019 with TS 3.6, I'm not seeing brace completion on <, even in JSX/TSX contexts.  Please let us know if you're still having issues.


Adds function relpath

This is needed later to calculate the relative path for the Index Page
to the different API pages.  Issue #43
I thought maybe smaller PR are easier to merge.

I had to redo it: the last one had some problems in certain situations.
I wrote an test case using results generated with python's relpath (expecting that this should work ok)

helper function: findlast: I know one could write it a bit shorter with a for loop - but in my tests such while loops are about 10% faster and I keep it for other things too.
If you prefer it different no problem if you want to rewrite it.
I'm not sure if this Tests will work on Windows - maybe I need to change the input files to use different pathsep.

Will use Base.path_separator cause Windows tests seem not ok.


Copy the InvocationHandler if list changed not when invoked

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: FlorianGrimm sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

sorry was my first rebase

i don't know what happend. Git strikes me down. should i close this?

@FlorianGrimm yes, close and reopen a PR on top of dev.

Closing this out


Initialize Array::m_executeOnGPU

Summary
Lack of Array::m_executeOnGPU initialization played a role in causing host-initialization of device-allocated memory in certain situations. I was not able to create a reproducer on the develop branch but these changes eliminate warnings about uninitialized memory in axom::Array under Valgrind and fix the crashes on my MIR branch when using CUDA-allocated memory.

This PR is a bugfix and closes #1432.
It adds some initialization for Array::m_executeOnGPU, a bool that was causing warnings about uninitialized memory.

@rhornung67: I think we'd need to test with asan/ubsan enabled in order to reliably capture a problem like this. Alternatively, -Weffc++ on Clang might be able to statically warn about uninitialized variables, but it might also spit out a bunch of other warnings that we'd have to work through.

I added a test that approximates what I was doing in my MIR branch. There was some weirdness with RelWithDebInfo but not a crash. The test calls a new function axom::isDeviceAllocator() to exercise it.


cf-auth-proxy should return a more specific error when the request is unauthorized

Currently, cf-auth-proxy returns a 404 for all source IDs that either do not exist or are not accessible to the user. This is ambiguous, as misconfigured endpoints, incorrect API paths and other problems give 404 errors, and confusing to the user, who may simply have an issue with their token.
The cf-auth-proxy should return a more specific error to non-admin users for nonexistent/unauthorized source IDs, such as 401 Not Authorized.

This is fairly deeply tied into our API contract; the work to change this is larger than the benefits.


FEATURE: Custom tags

In order to be able to use the tool in cases when we have the same product layout but on different platforms or product versions it would be useful to add custom tags. Currently, now it is possible to add os, browser, device. Would be better to support user custom tags e.g productVersion=1.0 or env=production

@suratdas thank you for contribution!


Collecting licenses for monorepo packages when license is only in root level

Hi,
we are using 'swr' package  and lately we upgraded to the latest version (1.3.0) and got a new error during the build:
WebpackLicensePlugin: Could not find license info for<EMAIL_ADDRESS>It seems the repo changed to a mono-repo causing an issue to collect the license for the sub-packages (since he only exists at the root level).
Are you going to detect cases like that or expect each package to have a license?

It would be nice if license was a mandatory field in package.json, but it isn’t.
We could implement workarounds in this package, but they would be brittle and error-prone. In addition, we would be implying an intention here (all sub-packages are supposed to be licensed like the monorepo root) that is not necessarily true.
Licenses have to be explicitly declared in packages. If they're not, they're basically UNLICENSED and cannot be used by any person or organization that depends on open source license compliance.
Please open pull requests in the respective repositories that copy the monorepo's license info to the individual packages.

Thanks for the fast response I have already opened a ticket for them.
https://github.com/vercel/swr/issues/2081

Chimed in with some basic advice. Hope that gets resolved soon, as it’s a very easy fix!


Khushi's and Team Work Final Review Ticket

[x] Create Task Punnu and Khushi Wiki Page
Wiki Page
[x] Punnu's Explanation on Math Quiz Wiki Page
Wiki Page 2
[x] Khushi's 1 min Video
Khushi's Video
[x] Mathapi Create Task Code
Mathapi
[x] Math Quiz Create Task Code
Math Quiz
[x] Team Turtles Video
Team's Video
[x] Khushi's Written Response
Khushi's Written Response
[x] Deployed Website
Deployed Website

Overview: Khushi's and Punnu's Create Task project consists of the Mathapi and Math Quiz, where both assignments had been completed as a team.

Overall Crossover Team Score: 11/15
Individual/Pair Evaluation on Create Task Project - 4/5 (All requirements for this category were met and their Wiki page was very well written. However, they talked in their video and the video is over 1 minute)

[x] Wiki/Jekyll page with design ideas and usage
[x] Create Performance Task PBL feature
[x] Final committed program code showing algorithms written in JS/Python
[x] Written responses to all the prompts in the performance task
[ ] 1min video that meets CB create task requirements

Team Final Grading Considerations - 5 pts
PBL Github pages. Jekyll, WIKI, live review focus - (they have a consistent theme, used bootstrap in volunteer page a lot and other pages, used blueprints, had CRUD, and had MVC features throughout code)

[x] PBL theme consistent through the project and described in Github Pages
[x] Sassy / Bootstrap
[x] Blueprint illustration/demonstration
[x] CRUD Async Operation - runtime and understanding of JavaScript and Fetch
[x] Individual PBL MVC feature/commits showing algorithms written in JS/Python

Runtime Features shown in 1 min video (had great features, went over scrum board, and had a useful CRUD database that was relevant to their project theme (volunteer contacts for student services page). However, they did not have a Google Search/Translate implemented into their project nor went over their code in their video.)

[x] Highlight of best features

[x] CRUD/Search on project database tables, easy to identify from UI

[ ] MVC - a code overview showing usage of MVC (HTML, Routes/Render Template functions, Model functions)

[ ] Google Search or Google translate implemented

[ ] Scrum Board - Overview of completed tasks and Github tangibles

[x] Clarity and quality of finding information (very organized and neat website!)

[x] Deployed Project 24/7 - Deployment Overview and Policies in Github Pages (doesn't work on school wifi)

[x] [N@tM] Participation and readiness for event (ready for event with deployed website!)

Total Review Score: 4/5 (individual) +  7/10 (team) = 11/15


Include the pagination control for when the records need to be broken up over multiple pages.

@corypride I removed this as an "issue" from the original ticket since we talked about it getting created on accident.


Bump require-dbt-version to 0.16.0

Per the release notes, the 0.3.0 release requires dbt version >= 0.16.0
We should update require-dbt-version here accordingly

So uh, it turns out that the release notes are incorrect, the require-dbt-version parameter is correct.
This version increase was required since we added the warn exception to this package, which was added in 0.15.1 (changelog).
I've opted to update the release notes, rather than reduce the dbt version range. Do you buy that? (If not, feel free to re-open) I could definitely be convinced that increasing to 0.16.0 is just good practice


911

Using VDK call? Seems like the appropriate config request to add support to, otherwise, global message i would assume.


Need to override NER identifications

We often know how a given entity should be labeled. Assignments from knowledge sources should be able to override the NER's default classifications.
For example: the CRF seems to be responsible for identifying 'H-RAS' and 'K-RAS' (but not 'HRAS' or 'KRAS') as protein families even though our knowledge sources list these exclusively as proteins.

Attached is a candidate sample file which could be used to implement a NER override and auxiliary grounding capability. It is based on the NMZ auxiliary spreadsheet of 1/11/16 merged and updated with the latest (6/9/16) NMZ spreadsheet model, after conversion of adhoc NMZ types to Reach types.
Note: the file extension should be .tsv but GitHub doesn't support uploads with this extension.
NMZ-NER-aux_160624.txt

See additional examples in issue #60.

@hickst: I added override capability to the bio NER in processors in the branch "ner-override". I also added your NMZ-NER-aux_160624.tsv.gz to bioresources. Can you please do the following:

We should merge the two files you created: NMZ-NER... and NMZ-merged into a single one. There is no reason for two. Having both there is a recipe for disaster, when we update one but forget the other. I currently assume that column 0 is the entities names (if multiple words, separated by space), and column 3 is the NE label. Columns should be separated by TAB. All this is trivial to change. Let me know if you change the format.

Find a better name. NMZ... is not descriptive. Also, having the date in there, is not needed given version control. I suggest something descriptive such as "NER-Grounding-Override.tsv".

Add all the entities that we consistently label incorrectly according to MITRE. More on this soon.

Let me know when the first 2 are done, so we can release bioresources and processors.


[mitmweb] asyncio has no attribute call_soon

Steps to reproduce the problem:
1.Launch mitmweb
2.Save some flow
3.Load those flows

Any other comments? What have you tried so far?
https://github.com/mitmproxy/mitmproxy/blob/0fa1280daa94729defa8411d86266bd2b52ad0b6/mitmproxy/tools/web/app.py#L238-L239
I replaced this line with asyncio.ensure_future(self.master.load_flows(i)) to make the loading works.
System information

Want to submit a PR for this? Your solution is correct.


[BUG] Space after Team name

Describe the bug
In a placeholder after the team name, there is a a space
To Reproduce
Steps to reproduce the behaviour:

Create Team
use the placeholder %betterTeams_name% but make it like "[%betterTeams_name%] and see there is a space after the name and the ]

Expected behaviour
Should ne no space
Screenshots

Details
Please provide the following details about your server set-up:

Spigot version (found by running /version):

- BetterTeams version (found by running /version BetterTeams):

Additional context

IM very dumb, If you just edit the placeholder config you can do something like [%s] instead of doing the [] on the other plugin


feat: Add owner file

OWNERS is missing for the project, this PR is to add it.
/assign @jlewi @carmark
Signed-off-by: Ce Gao<EMAIL_ADDRESS>

Can I merge it manually? @jlewi


How to modify "eclipseMirror"

The default eclipseMirror is http://mirror.netcologne.de, I can't access the network address. I need to modify the eclipseMirror.

You can specify arbitrary eclipse mirror by adding the following to your build.gradle:

wuff {

  eclipseVersion('4.4.2') {
    eclipseMirror = 'http://mirror.onet.pl/pub/mirrors'
  }
}


Potential hardcoding of cluster name

In this file:
kubeinit/kubeinit/roles/kubeinit_bind/templates/create-external-ingress.sh.j2
The name of the cluster and domain name is hardcoded:
podman pod create --name kubeinit-ingress-pod --dns ${KUBEINIT_INGRESS_IP} --dns <IP_ADDRESS> --dns-search okdcluster.kubeinit.local

Fixed with #324


feat: enable two way prop bindings

This PR enables Svelte-like 2 way prop bindings:
component App() {
  val inputValue = "Hello";
} => (
   <div>
      <Input value:={inputValue} />
    </div>
)

component Input(value) => (
   <input type="text" value:={value} />
)

The rationale for supporting this feature is to make it easier to build page editor UIs for the end-user in Craft.js - where it's much easier for the user to bind a prop value to a variable than to have them write a callback function.
https://github.com/prevwong/reka.js/assets/16416929/47839164-974e-475c-bc62-6aebacfae60d

Just curious but what software did you use for that video? It looks interesting :)


[Feature request]zip obb import

Thank you for wanting to share an idea! But before starting, ensure to check if this feature request respects the following requirements:

[x] It is written in English (I can translate what you say, but issues written in English are easier to read for the other users).
[x] There is not already a similar feature request among the open or closed issues.

Is your feature request related to a problem? Please describe.
GrapheneOS system cannot copy to "Android/data/nie.translator.rtranslator/files".

Please provide zip obb import function

Additional context
Add any other context or screenshots about the feature request here.

Hi @Zig-Rust-Odin, you can see this folder only if you connect the phone to a PC and you access that folder from the PC, please follow this guide precisely to do sideloading.

Ok, the normal download doesn't work?


When using temp as destination, have option to not start a new plot until archiver completes

When configured without a destination the software leaves it in the temp directory it created it in. In my case this is 1 of 24x 10 SAS hard drives. Then the archiver comes along and transfers it to the remote system. While the archiver is trying to do its thing plotman says hey I got a free temp directory and starts up a new plot on that drive slamming the transfer rate of the archiver.
This slows things WAY down.  Maybe an option could be added that just checks if a .plot file exists in the temp directory
consider it not ready.

You could limit the amount of plots with the phase options, no?
       # Optional: Allows the overriding of some scheduling characteristics of the         # tmp directories specified here.         # This contains a map of tmp directory names to attributes. If a tmp directory         # and attribute is not listed here, the default attribute setting from the main         # configuration will be used         #         # Currently support override parameters:         #     - tmpdir_stagger_phase_major (requires tmpdir_stagger_phase_minor)         #     - tmpdir_stagger_phase_minor (requires tmpdir_stagger_phase_major)         #     - tmpdir_stagger_phase_limit         #     - tmpdir_max_jobs         tmp_overrides:                 # In this example, /mnt/tmp/00 is larger and faster than the                 # other tmp dirs and it can hold more plots than the default,                 # allowing more simultaneous plots, so they are being started                 # earlier than the global setting above.                 #"/mnt/tmp/00":                 #        tmpdir_stagger_phase_major: 1                 #        tmpdir_stagger_phase_minor: 5                 #        tmpdir_max_jobs: 5                 # Here, /mnt/tmp/03 is smaller, so a different config might be                 # to space the phase stagger further apart and only allow 2 jobs                 # to run concurrently in it                 # QUESTION HOW TO PLAY WITH THESE PHASES?? :(                 #"/mnt/tmp/03":                 #        tmpdir_stagger_phase_major: 3                 #        tmpdir_stagger_phase_minor: 1                 #        tmpdir_max_jobs: 2


[BUG] mpegts no PTS found at end of file, duration not set

Describe the bug
Some (not all) .ts files created by egress have no Presentation timestamp (PTS) set.
Egress Version
We are use Livekit Cloud. This bug was observed with a recording of 2023-11-07 15:44:21:624 UTC.
Behavior
I'm attaching 2 .ts files from the same egress recording.

testbed_20231107154421624.ts -> no PTS found at end of file, duration not set
testbed_20231107154427624.ts -> no errors, everything in order

pts-missing.zip

I would expect all files to be timestamped in the same way. If this is expected, could you maybe explain why we're seeing this?
Additional context
This recording was made in cloud project p_2u70izoj75h egresses EG_hzVc5XuBuwp7.

Looks like there was some packet loss surrounding that segment. Does this break the playlist?

Yes, this breaks playback using video.js and hls.js. The video gets stuck into an endless buffer state.

After some more testing it seems that this specific error does not stop video.js from replaying the video.


formula lost

i have posted this issue on [bitbucket/openpyxl] (https://bitbucket.org/openpyxl/openpyxl/issue/413/formula-lost-in-cell) but the post was deemed invalid.
In pyexcel, there seems no data_only option in load_book(), so I can't read cell value with formula in some cells(not all the cells.)  In my case. the cell in first row with formula can be read but following rows are not.

Hi, I managed to reproduce the bug and am able to get you the value by passing data_only=True. Please get it from github and verify before I will release it to pypi:
pip install git+https://github.com/chfw/pyexcel-xlsx.git

However, passing data_only = False, openpyxl shall return all formula as text but as you have seen, it is not happening.  So, that's outside the scope of pyexcel.

Thank you chfw. I tested your code, Now with or without data_only option, all the values with formula can be returned correctly(as value, not formula string). I think it's ok, but it's your call to determine.

pyexcel-xlsx plugin has been released to pypi. Hence, I am closing this issue.


Fix compile bug on macOS

I got a compiling error (no member named '__cxx11') in make process after cmake . on macOS.
I found that IF(MACOS) in CMakeLists.txt is not working and should use IF(APPLE) instead.

Thanks for this PR.


Mnp and mku universities

Adds Mount Kenya University and Meru National Polytechnic Domains

@victormwenda lib/domains/ke/ac/mku/mylife: this request has an incorrect file path format, please ensure that you are familiar with     this repository rules located at the bottom of this page and change     your request accordingly. If you don't know how to change request, close it and open a new one.     If you are not able to make the pull request according the instructions, please send us an email to<EMAIL_ADDRESS>Please mention this request URL in the message.

@victormwenda Thank you for your patience while waiting for a review and processing your request. I corrected and merged the pull request.


feat: add support for JenkinsX

Signed-off-by: ankitm123<EMAIL_ADDRESS>

HI @ankitm123 did you say these changes support PR builds only? Or both "pull_request" and "push" builds? A full coveralls integration would cover both builds, since "push" builds on the default branch are required to provide PR comments on "pull_request" builds.

Were you able to get this working for a Coveralls repo? If so, can you share the Coveralls URL so I can review further?

yes, but it's a private repo. I can share it via email.

Did you say these changes support PR builds only

Yes, but I have it working for both pull request and master builds.
I will change the PR to set values specific to PR and push builds. Do you have an example from another CI providet that I can look at? I will make the changes ...


Removed UEK5 Preview - Now GA

UEK5 Kernel for Oracle Linux 7 is now Generally Available.

I've just removed references related to OL7-Preview.


pudl_datastore --list-partitions

Add capability to list available partition keys and known values.

This fixes #915

Will definitely improve usability, and efficiency too it looks like. Should we try and get some unit tests for this functionality too?


Services Enumeration Error

Test-HostServices throws an error each time it fails to query a service name which does not exist. It should suppress the error display and handle it as either an exception or shown in the testdata output.

Fixed in 0.3.0


[stdlib] Fix documentation for Int and UInt sizes

Discussion on swift-dev: https://lists.swift.org/pipermail/swift-dev/Week-of-Mon-20160208/001049.html
Add a note in the doc comments for Int and UInt that these types have
platform-dependent sizes.
Before this change, the documentation for Int is:
/// A 64-bit signed integer value type.
public struct Int

With this change, the documentation says:
/// A signed integer value type that has
/// the same size as the current platform's native word size.
///
/// On a 32-bit platform, `Int` is the same size
/// as `Int32`.
/// On a 64-bit platform, `Int` is the same size
/// as `Int64`.
public struct Int

(And analogously for UInt.)
The documentation for Int8, UInt8, Int16, etc. is unchanged.

@ole I'd like to make progress on this pull request.  I can't quite tell where you and @gribozavr ended up with your discussion, and what's the next step.  What do you want to do?

@tkremenek Thanks for reviving this, I apologize for not following up earlier. This thread is the entirety of my discussion with @gribozavr, i.e. we haven’t reached a definitive decision how to proceed.
As I said, I'm fine with hardcoding 32 and 64 and it would definitely make the code easier, though it clashes a bit with the existing logic in SwiftIntTypes.py. Can you make a decision which variant you prefer? I would then update the PR and rebase it on the current master.

@gribozavr: Great, thanks.


Update CI to Ubuntu 22.04

This adds support for compilation with, for example, libdecor which is not available in any earlier version: https://packages.ubuntu.com/search?keywords=libdecor-0-dev.
Thus it might be helpful for #57025.

One downside of doing this is that the Linux builds will no longer run on Linux distributions older than 22.04 (such as Debian stable).

One downside of doing this is that the Linux builds will no longer run on Linux distributions older than Ubuntu 22.04 (such as Ubuntu 20.04 or Debian stable).

Yes indeed, we might not want to update because of that. IIRC, official binaries are built with an even older version because of that, so it makes sense to me to keep being a few versions behind to make sure no commit is too destructive.

It looks like the implicit update to GCC has introduced new warnings thus breaking the CI build due to -Werror, for example:
In file included from ./core/io/packet_peer.h:36,
                 from core/io/packet_peer_udp.h:36,
                 from core/io/packet_peer_udp.cpp:31:
In member function 'int RingBuffer<T>::read(T*, int, bool) [with T = unsigned char]',
    inlined from 'virtual Error PacketPeerUDP::get_packet(const uint8_t**, int&)' at core/io/packet_peer_udp.cpp:112:9,
    inlined from 'virtual Error PacketPeerUDP::get_packet(const uint8_t**, int&)' at core/io/packet_peer_udp.cpp:99:7:
Error: ./core/templates/ring_buffer.h:68:46: error: writing 1 byte into a region of size 0 [-Werror=stringop-overflow=]
   68 |                                 p_buf[dst++] = read[pos + i];
      |                                 ~~~~~~~~~~~~~^~~~~~~

To be fair I have previously seen spurious occurrences of stringop-overflow where the code is well-behaved.

That makes sense, thanks. Keeping it at 20.04 seems a lot simpler then.

Does anyone know if the CI failure mentioned earlier is something that should be fixed or is it a -Wstringop-overflow=4 case of "warnings for benign code"? (Also at the bottom of https://github.com/godotengine/godot/actions/runs/4640866483)

I've tried gcc-13 and g++-13 and it gives me different warnings, so that is at least a reasonable sign that something like this could work in principle. Of course, those still need to be fixed: https://github.com/ndren/godot/actions/runs/5575011120/

As pointed out by @Calinou, it's better to stick to ubuntu-20.04 for Linux build as long as it's supported by GHA, as this improves the portability of the produced artifacts.
I wouldn't install gcc 13 as it adds significant setup time to our already congested CI. Most of us actually compile Godot with recent GCC anyway so we catch relevant warnings, and it's useful that CI catches warnings (sometimes false positives) that may be raised by earlier versions.
For the Android and Web CI builds, we could switch to ubuntu:latest though, there's no reason to stay on an older one there.

@akien-mga that's true, I did not actually want to merge this with gcc 13. I wanted to get some idea as to which compiler warnings are bugs in gcc/g++ and which are bugs in Godot (for example, warnings that disappear between versions seem to be false positives).
Please consider reopening as draft.

You can experiment in your fork before opening a PR. This would keep our CI free to build other contributions.


Create SocketlabsAdapter.php

What does this PR do?
This PR introduces a new messaging adapter to enable email notifications through the Socketlabs API within the Utopia PHP messaging library.
##Changes Made:

Created a new directory 'Socketlabs' within the 'adapters' folder.
Implemented the SocketlabsAdapter.php class, which adheres to the MessagingAdapterInterface interface.
Added methods for setting configuration variables, such as API keys and API endpoints.
Implemented the sendMessage() method to send email notifications using the Socketlabs API.
Wrote documentation for configuring and using the Socketlabs adapter within the Utopia PHP messaging library.

Test Plan
To verify the functionality of this PR, follow these steps:

Configure the Socketlabs adapter with valid API credentials.
Create a message with the Socketlabs adapter.
Send the message using the sendMessage() method.
Verify that the message is successfully delivered via Socketlabs.
Run unit tests to ensure the adapter's correctness.

Related PRs and Issues
(If this PR is related to any other PR or resolves any issue or related to any issue link all related PR and issues here.)
This PR is not related to any other PRs or open issues in the Utopia PHP messaging library.
Have you read the Contributing Guidelines on issues?
(Write your answer here.)
Yes, I have read and followed the Contributing Guidelines for issues in the Utopia PHP messaging library.

Hi, thank you for your interest in participating in Hacktoberfest!
We really appreciate your enthusiasm.
So this year, we’re focused on quality contributions. This doesn’t just mean we want quality code from contributors; we also want to foster a quality relationship with the contributors through extensive reviews and support so they can tackle more meaningful and difficult projects.
With that said, we have a small team with limited capacity. So we’re careful about how many active issues we post for Hacktoberfest, so we can review and support each contributor adequately.
That said, if you find a reasonably sized issue on the Appwrite repo that you’d like to tackle, you can pitch your solution to the problem, and we will review and label the issue Hacktoberfest if it makes sense.
We also have community rewards every month for everything from cool projects, helpful community members, and even just sick memes. If you’re interested in contributing, being an active community member is also a good way to go.


Flat component tree

Hello, recently I've found this lib https://github.com/neoziro/recompact and I started to wonder what are benefits for keeping nested component tree while using multiple HOC's?
You can checkout https://github.com/neoziro/recompact#recompact-vs-recompose specific example. Basically all HOC's used on component are merged into single outer component.
Some HOC's require for component to be nested inside cause they operate on lifecycle but I think that's not the case for many recompose HOC's and lifecycle itself is considered for deprecation. So why not use HOC's as pipe for props that will be passed to last component? I found it especially annoying for testing components build up with recompose HOC's.

This topic has been brought up several times. You might get our point about it from https://github.com/acdlite/recompose/pull/182#issuecomment-315214044.

I'm going to close this issue now, because there isn't too much we can do here. Please feel free to reopen it if you have further thought.


Feature request : Beatstep like beat repeat in fills

Hello Claudio, really nice work you've done so far on the performer!
Just proposing some new functions : a fill mode in the style of the beatstep pro beatreapeat ribbon.
As the other fill modes, you can choose wich track will be affected.
Then when fill is pressed, it will beatrepeat till released.
To choose the size of the beatrepeat buffer, while fill is on you can press S9 to S16, it will choose respectively these buffer size:
1/4 1/8 1/16 1/32 1/3 1/6 1/12 1/24
When the fill is releaed, the preformer kept the last buffer size so if the fill function is activated again, it will use the last used buffer size.
You can also control the buffer length size with the routing section.
You might maybe be able to choose different buffer length for différent tracks so you can create some kind of beatrepeat polyrythm?
Cheers

on version 0.2.0 with launchpad will be possible. see issue #57


ref(gunicorn): use os.cpu_count instead of multiprocessing for simplification

Use os.cpu_count instead of multiprocessing (they are pretty much the same) - Got added in 3.4: https://docs.python.org/3/library/os.html#os.cpu_count
https://bugs.python.org/issue17914
The default change may seem deceiving but the move from 8 to 4 actually ends up as 9 if CPUs can't be found, due to the math we do
Depends on #758

[Current coverage][cc-pull] is 85.93%

Merging [#760][cc-pull] into [master][cc-base-branch] will increase coverage by 0.09%

@@             master       #760   diff @@
==========================================
  Files            30         30
  Lines          2672       2672
  Methods           0          0
  Messages          0          0
  Branches        429        429
==========================================
+ Hits           2293       2296     +3
+ Misses          257        256     -1
+ Partials        122        120     -2

Powered by Codecov. Last updated by [6ac4d18...30e2492][cc-compare]
[cc-base-branch]: https://codecov.io/gh/deis/controller/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/deis/controller/compare/6ac4d1863dd1f29ee161682e8982126d9e7d9fa3...30e2492a8b27370710c0ec6d97c01d8e33deb381
[cc-pull]: https://codecov.io/gh/deis/controller/pull/760?src=pr


Fix os type response isuserdefined

Description
This PR resolves issue 8125. Changing the datatype of isUserDefined of class GuestOSResponse from String to Boolean.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[x] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

How did you try to break this feature and the system with this change?

@isupersky can you please properly rebase your PR branch with 4.18 branch?

@blueorangutan package

@shwstppr a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: :heavy_check_mark: el7 :heavy_check_mark: el8 :heavy_check_mark: el9 :heavy_check_mark: debian :heavy_check_mark: suse15. SL-JID 7458

@blueorangutan test

@blueorangutan test

@weizhouapache a [SL] Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

[SF] Trillian test result (tid-8050)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 43972 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr8128-t8050-kvm-centos7.zip
Smoke tests completed. 113 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File

@weizhouapache @DaanHoogland @harikrishna-patnala @rohityadavcloud should this also go into 4.18?

@weizhouapache @DaanHoogland @harikrishna-patnala @rohityadavcloud should this also go into 4.18?

this seems a regression @shwstppr , on 4.18 the field is already a bool.

my bad @DaanHoogland. Probably referred wrong original PR

@weizhouapache @DaanHoogland @harikrishna-patnala @rohityadavcloud should this also go into 4.18?

this seems a regression @shwstppr , on 4.18 the field is already a bool.

ooo
It has already been fixed in #2632, and reverted in #6979

yes @weizhouapache ,
this asks for some kind of test (beit unit or integration)

tested:
without this PR
    {
      "description": "Windows XP SP3 (32-bit)",
      "fordisplay": true,
      "id": "e6561287-6db0-11ee-9ca5-1e003a000ac4",
      "isuserdefined": "false",
      "name": "Windows XP SP3 (32-bit)",
      "oscategoryid": "e64d1a63-6db0-11ee-9ca5-1e003a000ac4",
      "oscategoryname": "Windows"
    }

with
    {
      "description": "Windows XP SP3 (32-bit)",
      "fordisplay": true,
      "id": "246184aa-727c-11ee-9c7e-1e00ac0002c4",
      "isuserdefined": false,
      "name": "Windows XP SP3 (32-bit)",
      "oscategoryid": "2452d76f-727c-11ee-9c7e-1e00ac0002c4",
      "oscategoryname": "Windows"
    }

merging


Internal deployer: Add/edit social media links

When deploying a store for a merchant, we need to be able to add their social media links and also edit them if needed.

I should be able to choose which social media links I want to add (Facebook, Twitter, YouTube, Instagram, Medium)
The social media icons should show up below the sidebar on the left, and take me to the respective pages when I click on them
I should be able to add/remove social media platforms I want to link to, and also edit the links

Done by @shahthepro


Server Crash

Have you read the Important information text above

[X] Yes i did

Current behavior
About every 30 minutes 1 hour the server crashes
Desired behavior
docker version 2.24.0
Links to screenshots

To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Software setup

OS:Debian12
Docker:2.24.0

Hardware setup

vCPU:
RAM: 16
Disk:258go SSD

Additional context
No response

"Ran out of memory" 👍

This would mean that my server doesn't have enough RAM?

Palworld has GIANT memory leaks on the client and server side. You have to calculate for that. If your Docker-Node runs other services you might have not enough RAM. I would suggest at least 8GB for 4-6 players.

Is this issue resolved?

I used 16GB Memory machine for 10 people. Our friends have trouble with losing connection after 1-2 hours, so when I monitored memory it, the server was shut down and restarted due to memory leak. The memory share decreased to 1GB and then started to increase again.

Im closing this issue as resolved, feel free to reach out again if you need help.
If you like this project, please consider giving this repo and the docker-hub-repo a Star.


Some points unclear and references missing

Hey,
There are a few points which seem to be unclear for understanding. Maybe you could explain a few things briefly or add some references to the README.md file. That would be helpful, thanks.
Cheers.

Readme updated with deepdream explanation https://github.com/InFoCusp/tf_cnnvis/commit/9adae3d8fa47cba9d3f62a7f35566bb27c41b144


cleanup OSAuthEnabled/OSAUTH_ENABLED due to idpv3

https://github.ibm.com/IBMPrivateCloud/roadmap/issues/57463

/lgtm


App crash-when generatePDF

'NSMutableRLEArray objectAtIndex:effectiveRange:: Out of bounds'

Hi Narendra, could you provide us more details about the crash?


Version 1.0.2 RPM package is built as 1.0.1

I have built the 1.0.2 RPM package, but it comes out as 1.0.1:
$ cmake3 -DSLURM_PREFIX=/usr -DCMAKE_BUILD_TYPE=Release -DAUTO_TMPDIR_DEFAULT_LOCAL_PREFIX=/scratch/slurm- ..
-- The C compiler identification is GNU 4.8.5
-- Check for working C compiler: /bin/cc
-- Check for working C compiler: /bin/cc - works
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Detecting C compile features
-- Detecting C compile features - done
-- Found SLURM: /usr/lib64/libslurm.so
-- SLURM modules directory: /usr/lib64/slurm
-- SLURM version detected as 21.08.8, 21.08
-- Configuring done
-- Generating done
-- Build files have been written to: /home/niflheim/ohni/auto_tmpdir/builddir

$ make package
Scanning dependencies of target auto_tmpdir
[ 33%] Building C object CMakeFiles/auto_tmpdir.dir/fs-utils.c.o
[ 66%] Building C object CMakeFiles/auto_tmpdir.dir/auto_tmpdir.c.o
[100%] Linking C shared module auto_tmpdir.so
[100%] Built target auto_tmpdir
Run CPack packaging tool...
CPack3: Create package using RPM
CPack3: Install projects
CPack3: - Run preinstall target for: auto_tmpdir
CPack3: - Install project: auto_tmpdir []
CPack3: Create package
CPackRPM: Will use USER specified spec file: /home/niflheim/ohni/auto_tmpdir/builddir/auto_tmpdir.spec
CPack3: - package: /home/niflheim/ohni/auto_tmpdir/builddir/auto_tmpdir-1.0.1-21.08.8.el7.x86_64.rpm generated.

The generated SPEC file starts with:
$ head auto_tmpdir.spec
Buildroot:      /home/niflheim/ohni/auto_tmpdir/builddir/_CPack_Packages/Linux/RPM/
Summary:        Slurm auto_tmpdir SPANK plugin
Name:           auto_tmpdir
Version:        1.0.1
Release:        21.08.8%{?dist}
License:        BSD
Group:          Unspecified
Prefix:         /usr/lib64/slurm
Requires:       slurm == 21.08.8

Maybe this line needs to be updated in CMakeLists.txt?
PROJECT (auto_tmpdir VERSION 1.0.1 LANGUAGES C)

Generated new v1.0.2 release with the change to version in CMakeLists.txt.


discourse marker lexicons to be added

https://github.com/TScheffler/UK_DiMLex
Hindi Discourse Relation Bank (Oza et al. 2009)
Italian PDTB (LUNA corpus, Tonelli et al. 2010)
https://cs.famaf.unc.edu.ar/~laura/shallowdisc4summ/discor/index.html (DiscMar, corpus!)
https://www.idiap.ch/dataset/Disco-Annotation (https://www.idiap.ch/webarchives/sites/www.idiap.ch/project/comtis/resources.html)
Chinese DTB (LDC) [done]
Chinese TED [done]

done: CDTB, C-TED (Chinese)

accessibility of most data sources updated in the original message.


Credentials (and Co) shouldn't be stored in environment variables

Cases of undesirable information gain:

Someone can access the variables
Someone can access it when the environment variables are displayed in plain text

The password should be directly fed into chpasswd within chroot.

features related to scripting are no longer important (as switching distros is plausible), so close #11 as well.


[$250] Critical: LHN says unavailable workspace

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number:
Reproducible in staging?: Needs Reproducing
Reproducible in production?: Needs Reproducing
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @heyjennahay
Slack conversation (hyperlinked to channel name): ts_external_expensify_migrate
Action Performed:
Prerequisite: Login as a expelled member of workspace and paymant received for submitted expense

Go to staging.new.expensify.com
Observe the LHN
Open the workspace to view the expenses

Expected Result:
The workspace chat is displayed as archived
Actual Result:
The workspace displayed as unavailable in LHN and refreshing the page resolves the issue
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Standalone
[ ] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos

Add any screenshot/video evidence

{
    "chatType": "policyExpenseChat",
    "currency": "USD",
    "description": "",
    "errorFields": {},
    "hasOutstandingChildRequest": false,
    "hasOutstandingChildTask": false,
    "isCancelledIOU": false,
    "isOwnPolicyExpenseChat": true,
    "isPinned": true,
    "isWaitingOnBankAccount": false,
    "lastActionType": "REPORTPREVIEW",
    "lastActorAccountID": "2012843",
    "lastMessageHtml": "paid £199.04",
    "lastMessageText": "paid £199.04",
    "lastReadSequenceNumber": 0,
    "lastReadTime": "2024-11-11 22:07:48.215",
    "lastVisibleActionCreated": "2024-11-11 21:52:20.625",
    "lastVisibleActionLastModified": "2024-11-11 21:52:20.625",
    "managerID": 0,
    "nonReimbursableTotal": 0,
    "notificationPreference": "always",
    "oldPolicyName": "",
    "ownerAccountID": 2840332,
    "participants": {
        "2840332": {
            "hidden": false,
            "notificationPreference": "always"
        },
        "9645353": {
            "hidden": false,
            "notificationPreference": "always"
        },
        "10903701": {
            "hidden": false,
            "notificationPreference": "always"
        }
    },
    "permissions": [],
    "policyAvatar": "",
    "policyID": "A5F944A401046336",
    "policyName": "Expenses - Expensify UK",
    "private_isArchived": "",
    "reportID": "7246397494396164",
    "reportName": "Chat Report",
    "stateNum": 0,
    "statusNum": 0,
    "total": 0,
    "type": "chat",
    "unheldNonReimbursableTotal": 0,
    "unheldTotal": 0,
    "welcomeMessage": "",
    "writeCapability": "all"
}

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021856829046221721338
Upwork Job ID:<PHONE_NUMBER>221721338
Last Price Increase: 2024-11-20

Issue OwnerCurrent Issue Owner: @dominictb

This has been labelled "Needs Reproduction". Follow the steps here: https://stackoverflowteams.com/c/expensify/questions/16989

@puneetlath I'll keep you assigned here as the BZ assignee but will look internally for someone to work on the backend part of this.

@puneetlath Can you also share the json data of policy that is related to the policy expense chat and the transaction data with receipt 404? It's helpful to find the RCA.

Hi! I’m Pedro Guerreiro from Callstack - expert contributor group. I’d like to work on this task!

Great. @pac-guerreiro you're going to specifically look into the "unavailable workspace" in the LHN, right? It's weird because in the report data shared, there is a policy name.

@puneetlath exactly! This one is interesting indeed 😅
My wild guess is that somewhere we're passing a null report or an object which is missing the policyName.
I'll need more data to find out the culprit 😄
Tomorow I'll let you know which data I need!

Today I did some blind investigation on the code to find where the problem might happen, and I think it must be happening in getReportPreviewMessage  and the whole message should be Unavailable workspace paid $99,95.
@m-natarajan could you share your onyx state with me? There must be some linked report or report action with some missing data which will require to go through different data

Today I received the necessary data from @heyjennahay and was able to apply a possible fix for this issue.
Here is a draft PR with the fix - https://github.com/Expensify/App/pull/53094

@puneetlath Of course! I saw that on the chat header we we're showing the right workspace name on the subtitle, so I took a look at the function that determines what to display on the subtitle:

By looking at this function, I saw that we were using the parent report to obtain the reportName and workspaceName.
So, I made a modification to use the parent report if it exists, otherwise fallback to the report.
I'm not sure if this can introduce regressions or additional bugs! All I know is that it fixes this issue in particular 😄

I added the remaining screenshots except the ones for iOS because I'm getting an issue importing the onyx state on the simulator.
I'll be off next week, but someone from my team will address any feedback reported here 😄

This is what Pedro sent me:

Root cause

If a user is removed from a workspace, he’ll still have access to the workspace chat history but some expense reports will show Unavailable workspace .

Solution

I already created a PR with a possible fix for this by getting the workspace name from the parent report of the expense report.

Hope this answers your question, let me know if you need any additional details!
CC: @dominictb

@heyjennahay could you confirm that this has been fixed for you?

Hey @dominictb - please complete the BZ checklist here and then I'll process the payment here

Thanks! Payment complete.