(7.0) Compare hostnames when searching for default hub credentials

Description

Fixes an issue with tele not finding credentials for default hub if token isn't provided:
➜  e git:(version/7.0.x) GRAVITY_K8S_VERSION=1.17.9 /code/go/src/github.com/gravitational/gravity/e//build/7.0.14-dev.24/tele build \
        /code/go/src/github.com/gravitational/gravity//assets/telekube/resources/app.yaml -f \
        --version=7.0.14-dev.24 \
        --state-dir=/code/go/src/github.com/gravitational/gravity/e//build/7.0.14-dev.24/packages \
        --skip-version-check \
        --hub=https://get.gravitational.io  \
        -o /code/go/src/github.com/gravitational/gravity/e//build/7.0.14-dev.24/telekube.tar
Mon Aug 17 22:39:23 UTC	Building cluster image telekube 7.0.14-dev.24
Mon Aug 17 22:39:23 UTC	Selecting base image version
	Will use base image version 7.0.14-dev.24
Mon Aug 17 22:39:23 UTC	Build finished in now
[ERROR]: no credentials for https://get.gravitational.io:443

Type of change

Bug fix (non-breaking change which fixes an issue)
Regression fix (non-breaking change which fixes a regression)

Linked tickets and other PRs

Requires https://github.com/gravitational/gravity.e/pull/4330.

TODOs

[x] Self-review the change
[x] Perform manual testing
[ ] Address review feedback

Testing done

make telekube works fine now:
GRAVITY_K8S_VERSION=1.17.9 /code/go/src/github.com/gravitational/gravity/e//build/7.0.14-dev.24/tele build \
	/code/go/src/github.com/gravitational/gravity//assets/telekube/resources/app.yaml -f \
	--version=7.0.14-dev.24 \
	--state-dir=/code/go/src/github.com/gravitational/gravity/e//build/7.0.14-dev.24/packages \
	--skip-version-check \
	--hub https://get.gravitational.io \
	-o /code/go/src/github.com/gravitational/gravity/e//build/7.0.14-dev.24/telekube.tar
Mon Aug 17 23:43:06 UTC	Building cluster image telekube 7.0.14-dev.24
Mon Aug 17 23:43:06 UTC	Selecting base image version
	Will use base image version 7.0.14-dev.24
Mon Aug 17 23:43:06 UTC	Syncing packages for 7.0.14-dev.24
Mon Aug 17 23:43:06 UTC	Local package cache is up-to-date
Mon Aug 17 23:43:06 UTC	Embedding application container images
	Using local image quay.io/gravitational/debian-tall:buster
	Using local image quay.io/gravitational/debian-tall:buster
	Using local image quay.io/gravitational/provisioner:ci.82
	Vendored image gravitational/debian-tall:buster
	Still embedding application container images (10 seconds elapsed)
	Vendored image gravitational/provisioner:ci.82
Mon Aug 17 23:43:22 UTC	Creating application
Mon Aug 17 23:43:29 UTC	Generating the cluster image
	Still generating the cluster image (10 seconds elapsed)
Mon Aug 17 23:43:42 UTC	Saving the image as /code/go/src/github.com/gravitational/gravity/e//build/7.0.14-dev.24/telekube.tar
Mon Aug 17 23:43:52 UTC	Build finished in 46 seconds
make[1]: Leaving directory '/code/go/src/github.com/gravitational/gravity'

On second thought, this needs to pick up https://github.com/gravitational/gravity/pull/1982/files#diff-b67911656ef5d18c4ae36cb6741b7965R529 to avoid breaking next time stolon-app CI runs.

@wadells As discussed offline, that change is already present in version/7.0.x branch: https://github.com/gravitational/gravity/blob/version/7.0.x/Makefile#L529.


Add AssemblyVersion property to DialogHost.Avalonia.csproj

Without this property, check on AssemblyName.Version shows <IP_ADDRESS>

Hello, @Gigas002
May be will be better if we add Version instead of separated PackageVersion and AssemblyVersion?

@SKProCH Good point. I've checked this out, works fine, replacing the PackageVersion and PackageVersion

Thanks for contribution! ❤


 If you have a Save button it should also have to be used. If i want to undo my changes, how do i do then?
I noticed that if i don't press save and leave the page and when i get back it will still show the slider but 'Keep slider always visible' is still set to 'off'.
I believe it was the same behavior on the notification tab in #673.
v18.40

Yes, it's only active until the first full page reload. Right?

Maybe it is, but it shouldn't be active before i press the Save button. It was the same behavior for the notification tab but you changed that so now i have to press Save for it to get away.
I can see now that you didn't changed it for the notification tab either.
Why do you activate something before the user press the Save button? It will confuse me if something gets activated before i press the Save button. It is a very strange behavior. I don't know why you have made it like this but maybe you have your reasons?

Well, I actually did it on purpose. I find this user friendly to make things displayed upon toggling options, when possible.

Ok, I have made few tweaks to make sure that the user will be aware that the the settings should be saved.
Other than than, reverting to previous settings, unsaved could be possible but would take much more work and bring extra complexity. I realize that it would be slightly better but I don't think it's gonna worth it especially after this patch.


Renamed to ASTSize, changed to Maybe CoverageIndex

My initial intentions was to stop using ast size in Code.sizePlc because it may be confusing,
but use flat byte size instead.
Instead I renamed Size related functions and modules to ASTSize to make it more clear what this size is supposed to be.
I will do the real switch to flat/cbor size in another PR.
Also CoverageIndex had 2 problems:

It was not wrapped in Maybe (when the coverageindex is missing) . So there was ambiguity when the coverageindex was empty: it could either mean the coverage yielded empty results or the coverage was never run.
It was not serialised in case of SerialisedCode

Pre-submit checklist:

Branch

[ ] Tests are provided (if possible)
[ ] Commit sequence broadly makes sense
[ ] Key commits have useful messages
[ ] Changelog fragments have been written (if appropriate)
[ ] Relevant tickets are mentioned in commit messages
[ ] Formatting, PNG optimization, etc. are updated

PR

[ ] (For external contributions) Corresponding issue exists and is linked in the description
[ ] Targeting master unless this is a cherry-pick backport
[ ] Self-reviewed the diff
[ ] Useful pull request description
[ ] Reviewer requested

/benchmark nofib


Update Bundle

Fixes CVE-2024-39908
Fixes GHSA-4xqq-m2hx-25v8

Dependabot was faster with #86 and #87.


Merge data sources

This allows admin users and developers to merge data sources together, updating any references to them in the process.
https://trello.com/c/1ASGr7JV/1718-4-build-admin-data-source-merge-page-5

@samuelhwilliams this is all working now - however I think ideally we'd refactor the Merge data sources form object to include the hidden fields. I couldn't work out how to do that though (there's a HiddenField built into to wtf forms but that's not expecting to get a list).


feat(cart): add Magento GraphQL shipping methods driver

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows our guidelines: https://github.com/graycoreio/daffodil/blob/master/CONTRIBUTING.md#commit
[x] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

What is the current behavior?

Fixes: N/A
What is the new behavior?
Adds a Magento GraphQL implementation for DaffCartShippingMethodsServiceInterface.
Does this PR introduce a breaking change?
[ ] Yes
[ ] No

Other information

lgtm


Things to fix

[x] #6
[ ] Add library requirements to module setup file
[ ] Write comments on functions in data processing and pca files
[ ] Fix Module use wiki page and add descriptions for data file type (oscilloscope whatever)
[ ] Upload datafiles to git hub so people can actually use them

Fixed isaac's conda by cooking up our own

I tested the installation on a windows computer and it worked (except it does not have any data so might fail there)

uploaded the datafiles in a zip so that it could be easily used on windows

Added requirements so that now pip can get them if need be. unfortunately only specific versions of libraries are given. there was a problem with numpy 1.24 because of np.bool. another issue was with python 3.10+, can't remember what


workaround: disable streaming when downloading codeql bundle

Description
This PR disables streaming the extraction of the CodeQL bundle as a temporary workaround to the issue described in https://github.com/github/codeql-action/issues/2593.
A proper fix will be implemented later so that we can stream while respecting proxy settings.
Fixes #2593
Merge / deployment checklist

[ ] Confirm this change is backwards compatible with existing workflows.
[ ] Confirm the readme has been updated if necessary.
[ ] Confirm the changelog has been updated if necessary.

@aeisenberg @henrymercer , thank you both for your reviews. I've implemented Henry's request which I found to be in the intersection of what both of you suggested. Would it be possible to have another look at this?

@henrymercer @aeisenberg sorry for this, can you please re-approve, because I've added the link that Andrew asked for and it automatically dismissed the reviews.


placing lightwood planks cause client lag

Version of Advanced Rocketry
1.12.2-1.5.0-131-UNSTABLE build -131
Version of LibVulpes
1.12.2-0.4.0-52-UNSTABLE build -52
Version of Minecraft
1.12.2
Description of the problem
if I place lightwood planks, my FPS drops from ~100 to ~5 for half a second.

Where did you place it?

Does placing a torch cause lag too?


Why ?
Thanks to require here  we basically got rid off requiring of ApplicationRecord.
and then good_migration started to report it.
But why only for bundle exec rake test:vmdb:setup it is throwing error ?
Because there is enviroment test and in development enviroment ApplicationRecord is also loaded https://github.com/ManageIQ/manageiq/blob/master/lib/vmdb/settings/database_source.rb#L74 by ::SettingsChange.table_exists?
(database_connectivity? is true )
But in test enviroment for task bundle exec rake test:vmdb:setup)
database_connectivity? is false because there is non_existing_db used
https://github.com/ManageIQ/manageiq/blob/master/lib/tasks/evm_rake_helper.rb#L5 and then it will not load  ApplicationRecord and then good_migrations will detect it.
Solution

Replace all occurences of ApplicationRecord in our migrations and in https://github.com/ManageIQ/manageiq/blob/master/lib/extensions/ar_migration.rb#L7(it is also part of migrations)
by anonymous class with included ArRegion module.
It is done by method on ArRegion.
All calls on ApplicationRecord were only class methods from ArRegion.

Benefit is that when somebody will use ApplicationRecord in migrations then good_migration gem will report it.
(we can do also do same thing for migration specs in follow up PR but it is not failling now.)

Second issue is that we have UUIDmixin in app/model . It was also revealed by https://github.com/ManageIQ/manageiq/pull/13355

So I moved it to lib directory as we did it before with reserved_mixin.

I found is that good_migration is reporting ApplicationRecord thanks to this place. It is in task :resync_excludes but it happens only during bundle exec rake test:vmdb:setup.
So I just required it in the:resync_excludes rake task

cc @jrafanie @Fryguy

Not a fan of the wrapper being a concrete object because its purpose is only for the migration and not to be used anywhere else.
I see a few options that seems cleaner:

move rails_sequence_start to a module method on ArRegion, then have an instance method that just calls the class name (if that's even necessary...we should check what actually uses this method).  The create_table can just call the module method: ArRegion.rails_sequence_start
Instead of using ApplicationRecord, create a new class in the fly just for that code snippet: Class.new.tap { |c| c.include ArRegion }.rails_sequence_start

I'm fine with the UUIDMixin moving.  We may also want to look at what about that mixin we are actually using, and see if it makes sense to split that mixin into 2 parts; one module for the part that needs to mix into a model and another module or class for the part that is needed elsewhere like in migrations (I'm being intentionally vague here as I don't really know what out of the UUIDMixin was needed)

Nice pr description @lpichler !

@Fryguy @jrafanie

move rails_sequence_start to a module method on ArRegion, then have an instance method that just calls the module method (if that's even necessary...we should check what actually uses this method). The create_table can just call the module method: ArRegion.rails_sequence_start

I think that I cannot do it exactly. Because  method .rails_sequence_start is depending on other ClassMethods and class variables defined in ArRegion and also on ActiveRecord::Base and I think I cannot call such methods from ArRegion defined in ClassMethods(or I don't if it is possible) and as I figured out  I will need to call more those class methods in other places.

Instead of using ApplicationRecord, create a new class in the fly just for that code snippet: Class.new.tap { |c| c.include ArRegion }.rails_sequence_start

This is working but I need to use it on more places.
So I did mix of these options.
Last thing what I found is that good_migration is reporting ApplicationRecord thanks to this place. It is in task :resync_excludes but it happens only during bundle exec rake test:vmdb:setup.
So I just required it in the:resync_excludes rake task

@lpichler Looks better, had a minor comment... can you update the title (remove the wip) and description with your new solution?

@lpichler Looks better, had a minor comment... can you update the title (remove the wip) and description with your new solution?

@jrafanie Updated. thanks

LGTM, very minor things remain... waiting on travis unless you have further changes @lpichler or if @Fryguy has further comments.

@jrafanie  done.
note: I saw here failure https://travis-ci.org/ManageIQ/manageiq/jobs/191346067 before - but it is not related to this PR.

@lpichler is this needed on euwe/darga?  I believe it's only euwe.  I don't remember when good_migrations was added.  Can you verify if the other branches have this problem?

@Fryguy Looks good to me, since it's migration code I'll let you merge.

@lpichler is this needed on euwe/darga? I believe it's only euwe. I don't remember when good_migrations was added. Can you verify if the other branches have this problem?

yes, you are right. good_migrations are only on euwe,
but this issue was introduced in https://github.com/ManageIQ/manageiq/pull/13355  which is not going to euwe and I verified that issue is not on euwe


[DEVOPS-754] AppVeyor: use cache-s3 --max-size=1600MB

The .stack-work cache was expanding with stale data.
The cache-s3 --max-size option prevents build failures when upload volumes exceed AppVeyor limits.

AppVeyor build still works, and I can see on the S3 bucket that it has successfully uploaded a .stack-work cache for this branch.


feat: Usage reporter

Describe your changes

Usage reporter for periodically publishing metrics to billing service

How best to test these changes

Unit tests
Running tigris locally with proper metronome config

Prerequisites

[ ] https://github.com/tigrisdata/tigris-infra/pull/831

Issue ticket number and link
TIG-1204

Have some minor comments/questions, overall it looks great!


Disable reverseGeo and poiTapping in route_preview mode.

Closing, wanted this to be in the mapzen/eraser-map repo.


Guys, add Seed please. This is not serious to make tests with random seeds

def initialize_model_cached(variant, seed):
key = variant
torch.manual_seed(seed)
# If you are using GPUs, set the seed for all GPUs
if torch.cuda.is_available():
torch.cuda.manual_seed(seed)
torch.cuda.manual_seed_all(seed)  # for multi-GPU setups
# Check if the model is already in the cache

Thank you for the contribution! We will try to add the seed and other functionalities (e.g. text-to-image generation) in the new Gradio demo.


Allow option to simulate time running faster

This would be useful for demo purposes because we could, say, make a day pass in 24 seconds rather than 24 hours, so someone could see the entire cycle quickly.

One possibility is spinning off a thread that actually adds an hour to the system time every second or something, but that would ruin the system time until we manually fix it, which is not desirable.  A more complex but more robust method would be adding an hour to an "offset" variable every second and then modifying the rule system to add the offset to the system time before using it for calculations.  This way, the system time would never be modified and the time will be back to normal after restarting the server.


System tests in CI for testnet build

It's really burdensome to run the system tests locally. We can't run them in CI for branches that aren't based off develop. Add support for running the system tests against the testnet branch as well.

Actually, this already works out of the box. If you open a PR based off a different branch and call bors there, it runs the tests against the base branch. See #2355 for an example.


KPConv parameters for large outdoor scene with low point density

First of all, thanks a lot for this amazing open-sourced code. I have played around with your code a little bit using the provided dataset. Now I really want to train a model using my own data sets. My data sets were created using aerial images and photogrammetry. I have 24 of these kinds of datasets that were annotated for only three classes (ground, buildings, and vegetation). The difference between my datasets and these existing ones is that my data has low point density (0.3-meter point spacing), each data covers a larger area of interest (around 1 square km for each area I have collected). Could you please help me on defining the KPConv parameters that you think maybe reasonable to produce some good results? If I understand correctly, I may need to modify all of the following parameters.
in_radius = 6.0
num_kernel_points = 15
conv_radius = 5.0
deform_radius = 6.0
KP_extent = 1.2
Thank you in advance for your help!! And thanks again for this amazing work!

Hi @meidachen,
When changing the dataset there are only two parameters that really count: in_radius and first_subsampling_dl. The first parameter that you want to set is first_subsampling_dl. If you want good results, you should have it set the lowest possible, but it is useless for it to be lower than the minimum point spacing in your dataset. So in your case, set it to 0.3 meters. Then the rule of thumb is to have in_radius approximatively 50 times bigger than first_subsampling_dl. In your case, it would thus be 15 meters. You can play with in_radius, a lower value means smaller input and thus faster computing time or/and bigger batch size. A bigger value means the network has more contextual information in the input spheres. Depending on the nature of your dataset, the ideal value varies.
I advise you to keep the default value for the following parameters, which define the core architecture, independently from the data:

num_kernel_points = 15
conv_radius = 2.5
deform_radius = 6.0
KP_extent = 1.2

Other parameters that you may want to modify are:

batch_num = 6     # If your GPU can handle it, make the batch size bigger for better convergence.

lr_decays = {i: 0.1 ** (1 / 150) for i in range(1, max_epoch)}    # (150 is the number of epoch for a learning rate decay of 0.1, reduce it for faster convergence but be careful, if it is to low, your network convergence will stop too early)

Eventually, you need to handle the input features:

in_features_dim = 1 + the number of features available for your data points. (for example, colors=3, intensity=1)

Then go to:
https://github.com/HuguesTHOMAS/KPConv-PyTorch/blob/7fefb6a8d38fd304775199777ad01d9f1546e2ff/datasets/S3DIS.py#L402-L411
And implement the features you want to have there

Really appreciate your suggestion and detailed explanation! I have trained a model with the following parameters and was able to get a great result (miou over 85% for all three classes, again, amazing!!) I was doing this in a very brute force way for training and testing, I basically prepared all my point clouds in the .ply format and put them into the Data\S3DIS\original_ply folder and used your S3DIS code to prepare/train/test. I'm wondering do you think if this is ok or there are any issues you may think could potentially affect the results I currently have?
in_radius = 15.0
num_kernel_points = 15
first_subsampling_dl = 0.3
conv_radius = 2.5
deform_radius = 6.0
KP_extent = 1.2

max_epoch = 300
learning_rate = 1e-2
momentum = 0.98
lr_decays = {i: 0.1 ** (1 / 100) for i in range(1, max_epoch)}
grad_clip_norm = 100.0

batch_num = 6
epoch_steps = 200

I would really like to include your kpconv into our workflow (generating game environment/simulation using UAV images), and have a few questions which I hope you could help me with.

I saw your original implementation performed better on Semantic3D by not using the deformable layer, should I follow that and use the 'resnetb_strided' instead of 'resnetb_deformable' and 'resnetb_deformable_strided' in your pytorch implementation?
In your original paper, under the "Pipeline for real scene segmentation" section, you mentioned "At testing, we pick spheres regularly in the point clouds but ensure each point is tested multiple times by different sphere locations." but when I try to test the model I didn't find how to "pick spheres regularly" (I guess the spheres are the "potentials" here, and they are still randomly selected in the validation data set). And I saw that config.validation_size is a hardcoded number, in my case, if I'm using a small number it may not cover the entire area and it will run forever for a large dataset, and if I'm using a large number it may also run for a very long time for a small data set. Could you please help me understand how should I make this work as you mentioned in the paper?
Also I saw in test_models.py
elif config.dataset == 'S3DIS':
test_dataset = S3DISDataset(config, set='validation', use_potentials=True)
test_sampler = S3DISSampler(test_dataset)
collate_fn = S3DISCollate
is this (set='validation') related to this problem I'm having?
I'm wondering if I don't want to use the color as the feature and just want to use the point location as the feature, should I put 1 for every point? or should I do something else?
Could you please point me to which part of the code I should change when I try to segment an unlabeled dataset? I'm confused a little bit here because originally I thought the data will be prepared during testing, but it seems like all the data were prepared during training, and when testing, it just goes into the validation set that was already prepared. I could be very wrong here, so please help.

Thanks again for your help! And looking forward to hearing back from you.

Hi @meidachen,
Thanks for your kind words, I think what you did is quite good actually. For your type of data (large point clouds covering areas), the S3DIS pipeline is the most adapted. When you start getting good results like you are, the margin of improvement becomes smaller. I noticed that you can gain 1 or 2 mIoU points by optimizing the parameters to the best possible values. But at this point, getting 85% or 87 % won't change much for you. The improvement could be more significant if you add a new feature to the input. For example, in your type of data, adding the height above ground could be a game-changer (if you don't do it already).
Now concerning your questions:

I would advise keeping the rigid convolution. It is much more simple, faster, and the results are not always worse. See that answer I gave previously.

The input pipeline using potentials is explained in my PhD thesis section IV.2.8.c (p127-128). The validation is only here to keep track of the performances during convergence. You should adapt the validation size to the size of your dataset, do not make it too big, you don't want to lose too much time (10 times smaller than epoch_steps is generally a good option). You can always use the script test_models.py afterward to test on the whole dataset. In test_models.py, we use  set='validation' because there is no test set for S3DIS

You should always put 1 for every point, even when you use colors. The rule is to have the constant features equal to one, and then more features, in your case I would advise using height above ground. Colors could also be helpful (especially for vegetation), so that that would make 5 features in total (or 2 if you don't use colors). Compare both and see what is the best in your case.

As I said, S3DIS does not have an unlabeled test set, so it is not implemented in this dataset. Here you can see where the pointclouds are chosen depending on the set: https://github.com/HuguesTHOMAS/KPConv-PyTorch/blob/7fefb6a8d38fd304775199777ad01d9f1546e2ff/datasets/S3DIS.py#L112-L149
Change the code from line 139 to 149 to choose the unlabeled files for set=='test'

Then in test_models.py change set=='validation' to set=='test':
https://github.com/HuguesTHOMAS/KPConv-PyTorch/blob/7fefb6a8d38fd304775199777ad01d9f1546e2ff/test_models.py#L169-L172
Eventually you may have to check if everything goes well in the test function :
https://github.com/HuguesTHOMAS/KPConv-PyTorch/blob/7fefb6a8d38fd304775199777ad01d9f1546e2ff/utils/tester.py#L176

Thank you for your reply! I'm wondering when you say "height above ground" do you actually mean segment the ground first and then get the actual height above the ground? or do you mean using the following feature which already implemented in your code?
Get original height as additional feature
input_features = np.hstack((input_colors, input_points[:, 2:] + center_point[:, 2:])).astype(np.float32)
I also tried to experiment with the S3DIS data and wanted to reproduce your result >74 MIOU but was not successful. Here are the parameters I have used, I tried to use the same as yours but my GPU doesn't have enough memory so I reduced the batch size and didn't use the deformable layer. I was getting an MIOU around 67. Could you please help to see what may be the reason for this low performance?
#########################
# Architecture definition
#########################

# Define layers
architecture = ['simple',
                'resnetb',
                'resnetb_strided',
                'resnetb',
                'resnetb',
                'resnetb_strided',
                'resnetb',
                'resnetb',
                'resnetb_strided',
                'resnetb',
                'resnetb',
                'resnetb_strided',
                'resnetb',
                'resnetb',
                'nearest_upsample',
                'unary',
                'nearest_upsample',
                'unary',
                'nearest_upsample',
                'unary',
                'nearest_upsample',
                'unary']

###################
# KPConv parameters
###################

# Radius of the input sphere
in_radius = 1.5

# Number of kernel points
num_kernel_points = 15

# Size of the first subsampling grid in meter
first_subsampling_dl = 0.03

# Radius of convolution in "number grid cell". (2.5 is the standard value)
conv_radius = 2.5

# Radius of deformable convolution in "number grid cell". Larger so that deformed kernel can spread out
deform_radius = 6.0

# Radius of the area of influence of each kernel point in "number grid cell". (1.0 is the standard value)
KP_extent = 1.2

# Behavior of convolutions in ('constant', 'linear', 'gaussian')
KP_influence = 'linear'

# Aggregation function of KPConv in ('closest', 'sum')
aggregation_mode = 'sum'

# Choice of input features
first_features_dim = 128
in_features_dim = 5

# Can the network learn modulations
modulated = False

# Batch normalization parameters
use_batch_norm = True
batch_norm_momentum = 0.02

# Deformable offset loss
# 'point2point' fitting geometry by penalizing distance from deform point to input points
# 'point2plane' fitting geometry by penalizing distance from deform point to input point triplet (not implemented)
deform_fitting_mode = 'point2point'
deform_fitting_power = 1.0              # Multiplier for the fitting/repulsive loss
deform_lr_factor = 0.1                  # Multiplier for learning rate applied to the deformations
repulse_extent = 1.2                    # Distance of repulsion for deformed kernel points

#####################
# Training parameters
#####################

# Maximal number of epochs
max_epoch = 500

# Learning rate management
learning_rate = 1e-2
momentum = 0.98
lr_decays = {i: 0.1 ** (1 / 150) for i in range(1, max_epoch)}
grad_clip_norm = 100.0

# Number of batch
batch_num = 4

# Number of steps per epochs
epoch_steps = 500

# Number of validation examples per epoch
validation_size = 50

# Number of epoch between each checkpoint
checkpoint_gap = 50

# Augmentations
augment_scale_anisotropic = True
augment_symmetries = [True, False, False]
augment_rotation = 'vertical'
augment_scale_min = 0.8
augment_scale_max = 1.2
augment_noise = 0.001
augment_color = 0.8

# The way we balance segmentation loss
#   > 'none': Each point in the whole batch has the same contribution.
#   > 'class': Each class has the same contribution (points are weighted according to class balance)
#   > 'batch': Each cloud in the batch has the same contribution (points are weighted according cloud sizes)
segloss_balance = 'none'

# Do we nee to save convergence
saving = True
saving_path = None

Thank you again for your help!

I'm wondering when you say "height above ground" do you actually mean segment the ground first and then get the actual height above the ground? or do you mean using the following feature which already implemented in your code?

Whatever you feel is the best for your dataset. Sometimes assuming the ground is the lowest height value is enough. Sometimes you need a more robust ground extraction. You can first try with the z coordinates of your points (as I did) to see if that helps.

I also tried to experiment with the S3DIS data and wanted to reproduce your result >74 MIOU but was not successful. Here are the parameters I have used, I tried to use the same as yours but my GPU doesn't have enough memory so I reduced the batch size and didn't use the deformable layer. I was getting an MIOU around 67. Could you please help to see what may be the reason for this low performance?

Have another look at the paper, 74% mIoU is for Semantic3D. Actually, the best score I got for S3DIS was 67% mIoU, with deformable convolutions. So you perf is pretty high.

I think the main issue was that my data have multiple floors and not every floor has the ground at z = 0.

Indeed, that causes trouble exactly for the reason you explained. As S3DIS is always aligned at a single level, using the height feature made sense, but in your case, you should just remove it. Simply change in_features_dim = 4 in your configuration and that should be a lot better. As you said, this height feature should only be used in specific cases where we don't have slopes or levels.

are there anything in the network that was trained to look into the location feature of the point cloud (ps. I didn't find anything doing this in your paper)?

No, the network only looks at the features you give it. So if you don't give any location feature, the network will only focus on the geometric shapes and the color patterns (if you have colors).
Best,
Hugues


Julia 1.7: Issue with/after BLAS.get_num_threads()

Julia 1.7.2
julia> using LinearAlgebra

julia> using BLISBLAS

julia> BLISBLAS.get_num_threads()
1

julia> BLAS.get_num_threads()
16790712

julia> BLISBLAS.get_num_threads()
-1359210104

Note that using only BLISBLAS.*_num_threads works just fine:
julia> BLISBLAS.set_num_threads(5)

julia> BLISBLAS.get_num_threads()
5

Julia 1.8.5 & Julia 1.10.3
julia> BLISBLAS.get_num_threads()
1

julia> BLAS.get_num_threads()
64

julia> BLISBLAS.get_num_threads()
1

For the Julia 1.7.2 case, the fixes for get/set_num_threads() were added recently, see: https://github.com/JuliaLinearAlgebra/BLIS.jl/issues/3#issuecomment-1772131793
but I think the compat bump might mean that it's still using an earlier blis_jll version 0.9? Or there is something happen with LBT in Julia 1.7?
Also, doesn't BLISBLAS adds to OpenBLAS in LBT by default?:
julia> BLAS.lbt_get_config()
LinearAlgebra.BLAS.LBTConfig
Libraries:
├ [ILP64] libopenblas64_.dll
└ [ILP64] libblis.dll


No windows or files opening

What steps will reproduce the problem?
Opening MacVim, either through application icon or mvim command.

What is the expected output? What do you see instead?
Expect a window to open with the desired file, or even a blank new file.
Nothing happens, the app will start (as indicated by MacVim options in toolbar)
but no windows will open, even when explicitly told to open a valid file
through File->Open...

What version of MacVim and OS X are you using (see "MacVim->About MacVim"
and  "Apple Menu->About This Mac" menu items, e.g. "Snapshot 40, 10.5.6
Intel")?
MacVim Snapshot 70
OS X 10.10 Yosemite Intel

Please provide any additional information below.
Again, the app seems to be opening and running but I cannot open any files or
windows. All I get is the toolbar and the icon in the Dock.

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 21 Oct 2014 at 3:09

This issue was moved to macvim-dev/macvim#16


Improve Zibo 737 support

fixes detection of Zibo 737
Fix autopilot altitude up / dn
Add MCP selector (Zibo)
Add SPD selector (Zibo)
Fixes laminar command schedule not called from Zibo
Add Efis zoom in / out support
Add Map mode selector
Add CRS (OBS) Selector

Does this require to be merged with the original plugin by jmpEA31 of is it a self contained plugin? Would love to have my Vrinsight working with zibo


Datafusion Upgrade from v12 to v13 is failing with errors

Hello, I have tried to the update current version of datafusion used by columnq from v12 to v13 but i was unsuccessful to do so .Here is the error that I'm was facing while upgrading:
error[E0277]: the trait bound `datafusion::arrow::datatypes::DataType: Deserialize<'_>` is not satisfied
    --> columnq/src/table/mod.rs:31:5
     |
31   |     pub data_type: arrow::datatypes::DataType,
     |     ^^^ the trait `Deserialize<'_>` is not implemented for `datafusion::arrow::datatypes::DataType`
     |
     = help: the following other types implement trait `Deserialize<'de>`:
               <&'a [u8] as Deserialize<'de>>
               <&'a bstr::bstr::BStr as Deserialize<'de>>
               <&'a serde_json::value::RawValue as Deserialize<'de>>
               <&'a std::path::Path as Deserialize<'de>>
               <&'a str as Deserialize<'de>>
               <() as Deserialize<'de>>
               <(T0, T1) as Deserialize<'de>>
               <(T0, T1, T2) as Deserialize<'de>>
             and 342 others
note: required by a bound in `next_value`
    --> /home/elliot/.cargo/registry/src/github.com-1ecc6299db9ec823/serde-1.0.145/src/de/mod.rs:1869:12
     |
1869 |         V: Deserialize<'de>,
     |            ^^^^^^^^^^^^^^^^ required by this bound in `next_value`

I am a bit new to both Rust and ROAPI. I have tried to implement deserialization as suggest by the compiler but I was running into more errors while doing so. I may be missing something simple, any help or suggestions on how to go about this is appreciated!

I have researched a bit on how to implement the Deserialization and tried the following approach:
#[derive(Deserialize, Clone, Debug, Eq, PartialEq)]
#[serde(deny_unknown_fields)]
pub struct TableColumn {
    pub name: String,
    #[serde(serialize_with = "serialize_datatype")]
    pub data_type: arrow::datatypes::DataType,
    #[serde(default)]
    pub nullable: bool,
}

fn serialize_datatype<S>(x: &arrow::datatypes::DataType, s:S) -> Result<S::Ok, S::Error>
where S: Serializer
 {
    match x {
        arrow::datatypes::DataType::Binary => s.serialize_str("binary"),
        _ => s.serialize_str("")
    }
 }

Am I in the right path or am I missing anything?

good news, Datafusion is upgraded to 14 at #242 . Shall we close this?

fyi

# pulling arrow-schema manually to enable the serde feature.
 # TODO: add serde feature in datafusion to avoid this workaround
 arrow-schema = { version ="26", features = ["serde"] }

Apologize @elliot14A , I totally missed your question here. Still trying to catch up with all my github notifications :D


Release 2.2.0

cc @mastahyeti

:+1:

2.2.0 released https://rubygems.org/gems/html-pipeline/versions/2.2.0

Thanks!

:+1: thanks

@mastahyeti : Sorry for telling it here. But can you check you e‑mails replies ?


Player spins when sprinting

Describe the bug
When "Not Enough Animations" is installed, the player does a 360 every time you start and stop sprinting.
Reproduce

go in 3rd Person(not vanilla 3rd person)
start running using Ctrl while walking
some times it only triggers if your also turning using the mouse.

Expected behavior
When I don't use this the "Not Enough Animations" Mod, the player does not react visually from me starting/stopping sprinting.
Additional context
I am using a rather big mod pack but I tested a lot of combinations and it has only happend with "Not Enough Animations".

Can't reproduce in 1.20.1  v2.0.8-beta.5
Could you provide the game log?

latest.log


Duplication

Under Configure the local intranet settings for device registration, is there a reason the device login URL is listed twice.
https://device.login.microsoftonline.com
https://device.login.microsoftonline.com

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

ID: b9c51404-0a1b-c22e-e220-ce4126ff88d0
Version Independent ID: bd3aab43-4202-bb69-bd1a-580e0464f357
Content: How to configure hybrid Azure Active Directory joined devices
Content Source: articles/active-directory/devices/hybrid-azuread-join-federated-domains.md
Service: active-directory
GitHub Login: @MarkusVi
Microsoft Alias: markvi

@win2000b
Thanks for your feedback! We will investigate and update as appropriate.

@win2000b can you screenshot where you see this? I don't see it in the document in that section.

It looks like this got resolved so I will proceed to close this thread. If you have further questions or are still seeing this issue, please tag me in the comments and leave a screenshot and we will update the document.


changing config.txt messes up Hassio

EDIT: this is with regards to the DECONZ add-on
I have a raspbee and am trying to get this working. As per guide i try to add the following to the end of config.txt in the root folder:
enable_uart=1
dtoverlay=pi3-miniuart-bt
However this messes up my system, and home assistant will no longer start. Als not after deleting it again. I have to do a full reinstall to get HA back.
I'm I placing the text in the wrong location?

Doublicate: https://github.com/home-assistant/hassos/issues/480


[Question] What is the point of calling themr twice on a single component?

During reading through source code to find some interesting patterns and good practices I have found that you themr is called twice. First only with identifier and only in index file with additional default theme. I have read themr docs already but I can't figure out the point of this.
Would you please enlighten me on this topic?

Actually it is not applied twice but only once. In component definition file we write a factory that is called with the identifier and without styles. The purpose is to be able to import the component/factory without any css requires. It is intended for people that want to mock CSS modules or just avoid them.
In the index file we do require the CSS file and call themr providing the module by default but it should be called over a factory invoke. The returning value of a factory should be a component without themr, otherwise it's a bug.
With this approach you can import from the component if you don't want to use CSS modules directly or even import the factory to inject custom dependencies. If you are using CSS modules maybe it's more convenient to import from index and to forget about anything else. :)


Refactor demo with a new folder structure

Actually the demo folder structure it's a little bit tricky. So I open this task to perform a refactor of the demo in order to be more easy to understand.

what exactly do you mean by easier to understand? What do you plan to change?

I will organize it in the same way I have organized the demo in Material-UI, look:

Do you agree?

Yes it looks good

I was going to make some changes in the demo structure, that's why I came here to see this thread. I'm leaving things as they are now then, as you're looking into this already. Thanks.

Today at night I will do this refactor so you can keep working properly with it :)

@andrerpena this is done! :smile: You can check it!


Netty Alpha 5 - SingleThreadEventLoop Unusable by default

Expected behavior
SingleThreadEventLoop is currently unusable because all of the IoHandler type constructors are private.
Actual behavior
IoHandler types (NioHandler, KQueueHandler, EpollHandler) constructors are private.
Steps to reproduce
Attempt to make a handler instance... new NioHandler()
Workaround...
    private EventLoopGroup makeBossGroup() {
        Class<?> c = ENABLE_KQ ? KQueueHandler.class : NioHandler.class;
        IoHandler handler;
        try {
            Constructor<?> con = c.getDeclaredConstructor();
            con.setAccessible(true);
            handler = (IoHandler) con.newInstance();
        } catch(Exception e) {
            throw new RuntimeException(e);
        }

        return new SingleThreadEventLoop(new ThreadFactory() {

            private final AtomicLong nextId = new AtomicLong();
            @Override
            public Thread newThread(Runnable r) {
                Thread t = new Thread(r);
                t.setUncaughtExceptionHandler(Thread.getDefaultUncaughtExceptionHandler());
                t.setPriority(BOSS_THREAD_PRIORITY);
                t.setName("netty-boss-" + nextId.incrementAndGet()); // should always be 1 unless failure from an executor
                return t;
            }
        }, handler);
    }

Netty version
Netty 5 Alpha 5
JVM version
JDK 20-ea
OS version
Mac OSX AARCH64

@normanmaurer where? All I see is Handler.newFactory()

@NotArchon it is like this because usually you create a group which will create multiple executors.

@normanmaurer thank you
Quick question, should boss and worker groups share the same IoHandlerFactory?
        return new SingleThreadEventLoop(new ThreadFactory() {

            private final AtomicLong nextId = new AtomicLong();
            @Override
            public Thread newThread(@NonNull Runnable r) {
                Thread t = new Thread(r);
                t.setUncaughtExceptionHandler(Thread.getDefaultUncaughtExceptionHandler());
                t.setPriority(BOSS_THREAD_PRIORITY);
                t.setName("netty-boss-" + nextId.incrementAndGet()); // should always be 1 unless failure from an executor
                return t;
            }
        }, NioHandler.newFactory().newHandler()); <-- this line

Should the same instance of the boss's factory be used for the worker EventLoopGroup ?

Same factory yes... but not same handler.


add lowang-bh as member

github-orgs git:(lowang) ✗ pytest test_org_yaml.py
============================================================ test session starts ============================================================
platform darwin -- Python 3.10.11, pytest-7.3.1, pluggy-1.0.0
rootdir: /Users/wlh/go/src/github.com/kubeflow/internal-acls/github-orgs
plugins: anyio-3.6.2
collected 1 item

test_org_yaml.py .                                                                                [100%]

============================================================= 1 passed in 0.08s =============================================================
➜  github-orgs git:(lowang)

Hi, members of Kubflow community. I am apply to be a member of training-operator and mpi-operator org.
Currently, I am a member of Volcano and I can help to maintain the volcano gang-schedule feature.
Here are my PRs:
training-operator
fix comment word spelling error #1826
update volcano apis to v1.7.0 #1828
support make test on darwin #1829
e2e testing support volcano gang-scheduler #1831
remove duplicate code of add task spec annotation #1839
fetch volcano log when e2e failed #1837
mpi-operator
add volcano gang-scheduler pg min resource calculation #566
add volcano gang-schedule integration and e2e test #569
merge kubeflow/common.v1 to mpi-operator #571
In process
update volcano scheduler to 1.8.0 #1894
update full change list in changelog #1895
update volcano scheduler to 1.8.0; ignore vendor #589

+1
Thank you for your contributions!
/ok-to-test
/lgtm
/assign @zijianjoy

/assign @johnugeorge

/assign @terrytangyuan

/lgtm

/lgtm

/approve


about block not in canonical chain

Please help to see, whether it is before the update, or after the block not in canonical chain  probability is  high。I don't think it's normal

@MisterBigfeng what are the blocks that didn't get included? I'm curious to see who else has produced them.

@MisterBigfeng what are the blocks that didn't get included? I'm curious to see who else has produced them.

Thank you for reporting that.
There could be multiple reasons for blocks not being included in the canonical chain, for example,

The block producer is not synced when producing the block, which means the block is not produced on top of the canonical chain;
Due to network issue, the produced block may not be gossiped to the entire network. So other nodes doesn't know about it (even though they are all honest);
At any given slot, there could be multiple block producers that win that slot. Those block producers would all produce blocks at the same time, and only one of those blocks has the chance to stay in the canonical chain. As to which block to pick, the vrf evaluation determines the final winner.

For reason 1 and 2, we are working hard to improve the stability and reliability of the system. I hope those would happen less in the future.
For reason 3, it is just how the protocol is designed.
I will close this issue for now. If your block is dropped for any other reason, please re-open this issue and add corresponding details to help us debug it.
Thank you.

Thank you for reporting that.
There could be multiple reasons for blocks not being included in the canonical chain, for example,

The block producer is not synced when producing the block, which means the block is not produced on top of the canonical chain;
Due to network issue, the produced block may not be gossiped to the entire network. So other nodes doesn't know about it (even though they are all honest);
At any given slot, there could be multiple block producers that win that slot. Those block producers would all produce blocks at the same time, and only one of those blocks has the chance to stay in the canonical chain. As to which block to pick, the vrf evaluation determines the final winner.

For reason 1 and 2, we are working hard to improve the stability and reliability of the system. I hope those would happen less in the future.
For reason 3, it is just how the protocol is designed.
I will close this issue for now. If your block is dropped for any other reason, please re-open this issue and add corresponding details to help us debug it.
Thank you.


RSA.generateKey gone?

Looks like this is no longer present in v2? Looked through most of the source and couldn't find a replacement. Is RSA key generation no longer supported?

Yes, it's gone. I don't understand the random generators and they are hard to implement right I decided to drop. Also the same time NodeJS v10.12.0 landed RSA key generation ( and Web Cryptography API has it too. I believe it's better to use it natively in every environment.

Yes, I agree that randomness should be sourced from native primitives when possible but I think there would still be value in including key generation using a PNRG provided by the user. Maybe something like
(params: RSAKeyParams, pnrg: (len: number) => Uint8Array) => {publicKey: Uint8Array, privateKey: Uint8Array}

For me, the point of using this library is to have a fallback for platforms that don't provide advanced crypto like RSA. Any platform that has support for RSA key generation will also have implementations for encryption, signing etc so in those cases there is really no point in using this library at all. PNRGs on the other hand are more widely supported natively and as a last resort there are still pure js implementations provided by other libraries.
My use case specifically is for browsers that do have crypto.getRandomValues() but don't or only only partially support the WebCrypto standard. asmcrypto checks all the boxes except RSA key generation. I really like the quality and completeness of this library and I'd prefer if I didn't have to pull in another dependency just for key generation, so I'd be super greatful if you'd consider adding a bring-your-own-pnrg RSA key generation option.
Awesome work on this library btw. I think the move to typescript was a great decision and the code quality is way beyond most other js crypto libraries that I've seen. There is almost no documentation on how to use it but I've had zero problems inferring the api from the source which says a lot imo. Thanks and keep up the good work!


Consider update CMake to 3.8.2

Latest version of CMake 3.8.2 has been released.
Update would allow to link arm .obj files in static libraries. In previous version it wasn't implemented.

Should I make PR for this?

I wonder what's the official update strategy version-wise. It looks like VS 2017 has CMake 3.8 included, CMake 3.9 release is supposedly just around the corner.

Superseded by #1526 @ras0219-msft


Fix time/date-based output tests to prevent inconsistent test results

Looking at https://travis-ci.org/nodecraft/spawnpoint/builds/584216421, tests sometimes fail inconsistently due to how we're testing console output with variable times/dates.
As an example at https://travis-ci.org/nodecraft/spawnpoint/jobs/584216430, you can see we expected [16:05] [WARN]: Test, but instead saw [16:06] [WARN]: Test as the test took slightly longer to run.
The tests that need to be tweaked to allow dates/times in the future slightly (or ignore the time prefix) are at https://github.com/nodecraft/spawnpoint/blob/master/test/output.js.
This should close #19 and close #18 and close #16 and close #13 and close #12.

I made a very simple bandaid solution (it basically replaces the time from the data being tested with the time the test sees). I wanted to go a step further and have it make sure the time from the data is only a minute behind, but it'd be a lot more code added to the test file and I'm wondering if that would be a problem, I'm not incredibly familiar with testing in Node.

Awesome job @mhoobler! More code being added to the test file is totally fine - the aim is to test the framework. Just submit a PR when you're ready. 😃

I made the pull request. If there's any changes that you need please let me know. Also, thanks for the opportunity it was a fun first issue! 😄

Congrats on your first contribution @mhoobler, that's awesome! I hope you reach the 4 PRs for Hacktoberfest.
I've left a few comments over on #22 - let me know if you have any questions.

Closed by #22


implement basic functions

[x] dotset!: mutate nested Dict using dotted keys
[ ] dotget: get nested value in a Dict using dotted keys
[ ] dotpush!: mutate a Vector inside a Dict by pushing new element to it

@JuliaRegistrator register

Registration pull request created: JuliaRegistries/General/15880
After the above pull request is merged, it is recommended that a tag is created on this repository for the registered package version.
This will be done automatically if the Julia TagBot GitHub Action is installed, or can be done manually through the github interface, or via:
git tag -a v0.1.0 -m "<description of version>" 5da1f0e265cdfc27ae63e425fd1ef678ca4bdcbe
git push origin v0.1.0


Add clarification about the current Add > Controller

Added clarification about the current Add > Controller process, which takes two steps and may create confusion about the outcome of the first step, and makes the reader question why the controller itself is not created.
When creating a new PR, please do the following and delete this template text:

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot

@mihemihe  thanks, I cleaned it up a bit.


Colours in the semantic segmentation masks

I had ran inference using the 'swin_large_IN21k_384_bs16_100ep' weight, and obtained the semantic segmenation masks. But, the colours in the mask do not match with the colours in the stuff_colours in the metadata of 'coco_2017_val_panoptic'. Is there something I'm doing wrong?

Hello, I want to customize a semantic segmentation data set, should I register this data set, do I need to define the color palette in advance for different categories


refactor schema definitions

schema definitions have some issues;

handlers;

func allows to validate input, and acts as a before_update handler
it lacks an after_update handler (see https://github.com/Mashape/kong/pull/800#discussion_r47604401)
the design forces a validation call (to func), and checks of its existence for each field. As it is generally only 1 or 2 fields that require validation, it seems more efficient to have one handler (one for before_update and one for after_update) on the schema level, instead of a per field handler

validation is done through a complex validation function. I think it would be more efficient to 'compile' a schema; create a closure that most efficiently validates that one field. Fields without validation just get an empty closure that only returns true (the 'validation ok signal'). At runtime just execute each closure per field instead of running the complex schema validation function (you can even skip the check for its existence as every field will have one).
enum arrays should get a reverse entry so we can skip the horrible table_contains iterating the table over and over again

Thoughts?

+1 to everything

Should the schema become an object? instead of just a table?

In terms of external interface, no I don't think so. For example it is already annoying that people have to create an object handler (and since every plugin is always copy pasted from other, all plugins have this caveat). A table is always more simple and can simply be returned by a Module, and does not prevent Kong core from creating an object out of the returned table.
Internally, I actually don't think so either because we would have to instantiate such an object every-time we want to perform validation, which is how it used to be and it is very verbose. A table is fine, maybe with less fields (some of them are used only once or twice) and more logic into the before and after methods of some entities.

One other reason why the current model is far from ideal is that the iteration over the fields cannot be determined in advance. Sometimes, fieldA depends on fieldB filling a certain condition. Some other entities might have fieldC depending on fieldA, etc... This leads to ugly workaround and spaghetti code.
Another undetermined side-effect is sometimes pairs iterating in a different order on some platforms, hence causing inter-dependency checks to fail. I don't recall seeing this but it could happen.

Another undetermined side-effect is sometimes  pairs  iterating in a different order on some platforms, hence causing inter-dependency checks to fail. I don't recall seeing this but it could happen.

This has nothing to do with platform. But with the internal representation of the table data (array part vs the hash part of a table) depending on how the table was created/filled/modified the order of pairs can indeed be different. Even on the same platform.

btw. I've started working on an after_update handler that gets "compiled" as a PoC.

Back in the day we had an issue with travis not iterating in the same order as our MacBooks, hence why I was saying this. They were created the same way.

On Dec 23, 2015, at 12:28 PM, Thijs Schreijer<EMAIL_ADDRESS>wrote:
Another undetermined side-effect is sometimes pairs iterating in a different order on some platforms, hence causing inter-dependency checks to fail. I don't recall seeing this but it could happen.
This has nothing to do with platform. But with the internal representation of the table data (array part vs the hash part of a table) depending on how the table was created/filled/modified the order of pairs can indeed be different. Even on the same platform.
—
Reply to this email directly or view it on GitHub.

I guess the real issue I was trying to point out here is that when the validation of one fled depends on that of others (inter-dependent validation), the current model becomes very messy. Like the API request_host/request_path validation.

On Dec 23, 2015, at 12:30 PM, Thijs Schreijer<EMAIL_ADDRESS>wrote:
btw. I've started working on an after_update handler that gets "compiled" as a PoC.
—
Reply to this email directly or view it on GitHub.

replacing the per-field-validation (the func parameter) with a generic before and after handler should resolve that order issue

Yes.

Closing this now since a new schema definition exists with 0.13.0.


Can I have WebRTC web apps embeded in Microsfot Teams?

I want to add a calling web app in Microsoft Teams, in order to make calls on a 3rd party platform.
Can I integrate a web app to MSFT Teams, that would have WebRTC capabilities, and access to the microphone and camera of the PC?
For example, using these methods here: https://learn.microsoft.com/en-us/microsoftteams/platform/overview , Personal Apps or Tabs (and iframe in a Tab, that can do WebRTC?)

Hello @kklitest - Thanks for raising your query.
Yes, you can integrate any web application inside a Teams tab application.
You can create a person or configurable tab:
https://learn.microsoft.com/en-us/microsoftteams/platform/tabs/what-are-tabs?tabs=desktop%2Cdesktop1%2Cpersonal
You can refer below documentation for device permissions:
https://learn.microsoft.com/en-us/microsoftteams/platform/concepts/device-capabilities/browser-device-permissions
If you have implemented any authentication, it won't work due to IFrame, so you can implement Tab SSO:
https://learn.microsoft.com/en-us/microsoftteams/platform/tabs/how-to/authentication/tab-sso-overview
Also, please add all the validDomains properly in Teams application manifest.json.


Search clipboard entries case insensitively

I find myself more and more often trying to search among clipboard entries in a case insensitive manner.
I couldn't find a setting for that (fixme), so I wondered if this would be considered as a feature? And if yes, how it would be best to implement?
Since each currently supported backend seems to be capable of it, I could think of at least two approaches so far, which are not necessarily excluding each other:

use a CM_CASE_INSENSITIVE toggle (might be useful for clipdel too)
use a generic CM_LAUNCHER_ARGS option to allow users to specify arbitrary additional launcher options they might need

Oh, nevermind, I just found out by reading the help text that any arguments passed to clipmenu will be passed through to the backend, so running e.g. clipmenu -i works already. Sorry for the noise :facepalm:
Hopefully this issue will still be helpful for others searching for "case insensitive" :upside_down_face:


Updates to instructions due to portal changes and for clarification

•	Module 1 - Lab 1 - Exercise 1:
o	You must be in Settings>Endpoints to access Device management
o	You must be in Settings>Endpoints to access Roles.
o	You must be in Settings>Endpoints to access Device groups.
•	Module 5 - Lab 1 - Exercise 1
o	Create a new workspace in Azure Sentinel uses the + Create button, not + Add
Change_log_SC200_July2021.docx

Accepting reviewer changes.


Add abstraction over scroll wheel

The current way to use the scroll wheel looks like this:
int scrollDelta = InputHelper.NewMouse.ScrollWheelValue - InputHelper.OldMouse.ScrollWheelValue;
if (scrollDelta != 0) {
    // Do something with the delta.
}

It would be nice to provide an abstraction over this pattern. It would also be nice to be able to use this in the context of the tracking system.
Possible API:
if (MouseCondition.Scrolled()) {
    // MouseCondition.ScrollDelta
}

Closed in 68af16f.

Closed in 68af16f.


Checkmark off with Bootstrap

First of all thank you for a very nice component!
I'm using it in a Laravel application and everything works nicely, but the checkmark is off by a few pixels (see screenshot).

The issue is with the Bootstrap CSS. See the following pen: https://codepen.io/paulheijman/pen/LOVzjP
Can you please have a look at it?
Also, what is the best way to override the default CSS used by treeselect?
Thanks in advance!

Thanks for your feedback! I saw the same on Windows.

It's got something todo with the following declaration in bootstrap.css:
*:after {
  -webkit-box-sizing: border-box;
     -moz-box-sizing: border-box;
          box-sizing: border-box;
}

Fixed it by changing style.css:
diff --git a/src/style.less b/src/style.less
index e4ca506..cd20824 100644
--- a/src/style.less
+++ b/src/style.less
@@ -714,6 +714,7 @@
     opacity: 0;
     display: table;
     transition: @treeselect-transition-duration all ease (@treeselect-transition-duration / 3);
+    box-sizing: content-box;
   }

   // minus mark

Yeah, exactly.

As for style overriding, unfortunately it's not implemented yet. At present the only way to modify the style is add your own CSS after vue-treeselect.min.css.
The initial planning is expose source LESS files and then you can modify the variable values. However I have not made that clear. Do you have any suggestions?


OVN network custom DNS support

We're migrating from isolated LXD hosts running groups of instances (our product's cluster, but I'll keep calling them group of instances to differentiate from an LXD cluster) to a cluster of LXD hosts.
We decided to use OVN for networking because FAN imposes too many restrictions on IPs, and other solutions required configuring physical switches to dynamically create VLANs, which is another can of worms we didn't want to open.
With LXD bridges we were able to do the following:

Have a single LXD bridge network for provisioning all groups of instances and provide DNS and default gateway. This network is created once when the host is configured for serving LXD and never ever touched again.
This meant that the host was able to reach the instances via SSH, and we use Ansible on top of that.
It also meant that the instances could reach the wider internet through this network.
We also create one or more LXD bridge networks per group, and one of them was running another dnsmasq instance serving 'private DNS' with several arbitrary DNS names pointing to the instances. For that we used the raw.dnsmasq option. This is because our product requires that the instances must have multiple network interfaces that are used for at least two logically different networks, backend and frontend. There's an extra interface connected to the LXD bridge from point 1.

As you can see in https://discuss.linuxcontainers.org/t/serving-dns-over-ovn-networks-and-accessing-the-instances-from-the-hosts/13900/46 I'm trying to figure out how to do points 1-3 without hacking workarounds to issues I might be introducing myself, but the DNS part will require some extra work to have dnsmasq in there.
If I manage to get points 1-3 natively (meaning no workarounds), the only missing piece is the DNS support. @tomponline suggested to replace this bridge with a small instance running dnsmasq, but I rather prefer for LXD to do something for me. Maybe it's just simpler to run dnsmasq in the same network namespace as the rest of the instances, but I still don't know how OVN connects to the real world.

This is basically adding custom DNS record support, as well as the ability to specify custom DNS servers for DHCP/IPv6 RA.

Can be achieved by using a managed LXD bridge as an uplink for OVN networks. Then the OVN networks provide DHCP, DNS and default gateway services for the instances connected to that network.
Can be achieved using either non-NAT OVN networks and static routes on the LXD host to allow direct network connectivity into the OVN network instances. Or potentially connecting the Ansible container instance to each of the OVN networks so Ansible can reach them directly.
Is handled already because OVN by default SNATs to the OVN router's address on the uplink network.

Adding support for custom DNS records would avoid the need to run a custom dnsmasq instance.

I wonder if we could also add custom DNS record support for bridge networks without having to resort to raw.dnsmasq

I was thinking of that, but it would likely need to be tied to network zones in some way to not have too many different solutions.

2. Or potentially connecting the Ansible container instance to each of the OVN networks so Ansible can reach them directly.

In our case, Ansible runs on the host, not in a container. Hence the need for the host to access the instances.


CLI crash when run on empty schema.prisma

npm init -y
npm install prisma@dev
Then:

C:\Users\Jan\Documents\throwaway\envRepro<EMAIL_ADDRESS>λ npx prisma db push
Prisma schema loaded from schema.prisma
Error: Get config thread 'main' panicked at 'index out of bounds: the len is 0 but the index is 0', query-engine\query-engine\src\opt.rs:148:21
stack backtrace:
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

C:\Users\Jan\Documents\throwaway\envRepro<EMAIL_ADDRESS>λ npx prisma migrate dev
Prisma schema loaded from schema.prisma
Error: Get config thread 'main' panicked at 'index out of bounds: the len is 0 but the index is 0', query-engine\query-engine\src\opt.rs:148:21
stack backtrace:
note: Some details are omitted, run with `RUST_BACKTRACE=full` for a verbose backtrace.

vs. handled error in Introspection:
C:\Users\Jan\Documents\throwaway\envRepro<EMAIL_ADDRESS>λ npx prisma introspect
Prisma schema loaded from schema.prisma

Introspecting based on datasource defined in schema.prisma …
Error: There is no datasource in the schema.

Pushing this to team/client because the crash is in the query engine (most likely getConfig).

We took this anyway since it's introspection-relevant and we know how to fix it.

Thanks! I've moved this along.

We currently don't have bandwidth for that, assigning the client team in addition to us, since it's a crash in the QE, so the issue doesn't get lost.

Error message is now different in 3.6.0:
C:\Users\Jan\Documents\throwaway\7186<EMAIL_ADDRESS>λ npx prisma db push
Prisma schema loaded from schema.prisma
Error: Cannot read property 'url' of undefined

C:\Users\Jan\Documents\throwaway\7186<EMAIL_ADDRESS>λ npx prisma migrate dev
Prisma schema loaded from schema.prisma
Error: Cannot read property 'url' of undefined

Current behavior
prisma validate

prisma db push

prisma db pull

prisma format

We should:

change "datamodel path" to "schema path" or something that users could understand,
add a snapshot test for all these commands in its current state,
then decide if there is a better way to handle that (in the CLI for example where we can shot a unified and nicer error message instead of letting getdmmf and others handle this problem.

Side question: Why are db push and db pull different than the others?

The Couldn't find a datasource in the prisma.schema file message appearing in prisma db push comes from the ensureDatabaseExists function of ensureDatabaseExists.ts
The There is no datasource in the schema. message appearing in prisma db pull comes from the query engines, e.g.,  from here.

What is the connection between db pull and Query Engine?

probably getConfig or getDmmf for generate?

After adding snapshots for commands referencing empty schemas in this PR, no other actionable task seems available for this issue.
Future steps might be:

[ ] Agree on a single, shared error message to display in case of "missing required blocks from the schema" (which is a generalization of having an empty schema)
[ ] This "missing required blocks" validation will most likely require access to the AST of the schema, so it should probably be performed by the Rust Prisma Engines and forwarded to the TS side

Note: We should consider that this issue doesn't seem to fall in the "broken promises" bucket, so we could postpone additional work on this until after our joint effort with the client team is completed.

So current state from the PR snapshots
PRISMA_CLI_QUERY_ENGINE_TYPE === 'library'

validate

format

migrate dev
Get DMMF: Error while trying to read datamodel path
Details: The "path" argument must be of type string or an instance of Buffer or URL. Received undefined
Prisma CLI Version : 0.0.0

db push

migrate reset
Couldn't find a datasource in the schema.prisma file`,

db pull

db execute
There is no datasource in the schema.

For PRISMA_CLI_QUERY_ENGINE_TYPE === 'binary' there are some differences to note:
The following are valid for the binary

validate
The schema at (.*) is valid 🚀
format
Formatted (.*) in \d+ms 🚀

And a different error:

migrate dev
Couldn't find a datasource in the schema.prisma file

The current situation (as per main) is:
PRISMA_CLI_QUERY_ENGINE_TYPE === 'library'

validate
migrate dev

      Get DMMF: Error while trying to read datamodel path
      Details: The "path" argument must be of type string or an instance of Buffer or URL. Received undefined
      Prisma CLI Version : 0.0.0

db push
migrate reset

      Couldn't find a datasource in the schema.prisma file`,

db pull
db execute

      There is no datasource in the schema.

format

     Formatted (.*) in \d+ms 🚀

For PRISMA_CLI_QUERY_ENGINE_TYPE === 'binary' there are some differences to note:
Different error:

migrate dev
migrate reset

   Couldn't find a datasource in the schema.prisma file

The following are valid for the binary but not for the library

validate

     The schema at (.*) is valid 🚀

Next step:

[ ] Figure out why validate and migrate dev fail with such an error when using the library. Can we change figure out that undefined parameter?
[ ] We need to identify the differences between getDmmf in the Rust CLI and the Node API
[ ] Refactor empty-schema tests to incomplete-schema tests:

You have a datasource URL, but the env variable is not set
You don't have a datasource block
You have a datasource block, but no URL

Note: this task requires the Wasm-ification of getConfig to be handled first (as getConfig can heavily influence the incomplete-schema tests linked to this issue).


Inline Documentation

Write understandable inline documentation for the application.

Hey! I would love to work on this issue

Go ahead


Neo4j feature

Add neo4j docker component in the docker-compose between profiler and networkbuilder
Modified the 'networkbuildercoordinator' script, adding export to neo4j function
Change the Docker-Networkerbuilder CMD to Entrypoint to be able to have an option if the networkx should be exported to neo4j

PS: I will modify the 'REAME' asap

Hi Raul
Thanks for the reply.
At first i thought it similar as well, cause i think the output function
should be after network building in a separate function. As you proposed,
if we make a new module for reading pkl and writing to neo4j, then where do
you think we should put it.
PS: I will put neo4j after network building docker in docker-compose
(currently I put before networkbulder, because the networkbuilder will
write to neo4j).
regards
Nati
On Sat, Jun 23, 2018 at 10:49 PM Raul<EMAIL_ADDRESS>wrote:

@raulcf commented on this pull request.
This looks great. I left a couple of comments. The main idea being that we
could push all the new functionality into a new, self-contained script that
reads a network (given via input parameter) and performs the serialization
to neo4j. This would then serve as a template for others who may want to
serialize to a different backend.
In networkbuildercoordinator.py
https://github.com/mitdbg/aurum-datadiscovery/pull/124#discussion_r197622170
:

@@ -8,7 +8,7 @@
import time

-def main(output_path=None):
+def main(output_path=None,neodb=False):
Instead of adding this new variable here (and the corresponding check
further down in the script), it'd be cleaner if there's a separate script
that reads the network in pkl from a given directory and transforms it to
neo4. There are 2 advantages of that approach: i) we do not need to include
a variable for each new backend here; ii) each new backend has its own
self-contained script.
In knowledgerepr/fieldnetwork.py
https://github.com/mitdbg/aurum-datadiscovery/pull/124#discussion_r197622187
:

@@ -439,6 +442,31 @@ def serialize_network_to_csv(network, path):
s = str(n) + "," + "node\n"
f.write(s)

+def serialize_network_to_neo4j(network,server="http://neo4j",port=7474,user="neo4j",pwd="aurum"):
Following up on my previous comment, this function to serialize the
network into neo4j could be part of that independent module as well.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mitdbg/aurum-datadiscovery/pull/124#pullrequestreview-131405250,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARfozLvybDih3j48bxFSwiyrB-Dmw_dWks5t_qm9gaJpZM4UplwI
.

Great. I think a good candidate to place the new neo4j code would be:
https://github.com/mitdbg/aurum-datadiscovery/tree/master/knowledgerepr/ekgstore
The naming of the folders is a bit outdated and will need to change, but 'ekgstore' refers to the different backends used to store the graph/network/ekg.
I'd propose to start from there. Does that sound good?
Raul

Good!
Another question, how do we call the export function( in which steps).
Raul<EMAIL_ADDRESS>schrieb am So. 24. Juni 2018 um 15:02:

Great. I think a good candidate to place the new neo4j code would be:
https://github.com/mitdbg/aurum-datadiscovery/tree/master/knowledgerepr/ekgstore
The naming of the folders is a bit outdated and will need to change, but
'ekgstore' refers to the different backends used to store the
graph/network/ekg.
I'd propose to start from there. Does that sound good?
Raul
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/mitdbg/aurum-datadiscovery/pull/124#issuecomment-399755043,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ARfozFAlPBald3p7xnM6yiAluPWUb_olks5t_43pgaJpZM4UplwI
.

One may want to directly store the network in neo4j, but for now, I'd make the assumption that the default is to create the pkl, and then there's an option to export to neo4j. Because it's an option, the export function does not need to be in the pipeline, along with the others.
For that reason, I'd just make sure we add a couple of lines in the README under a new section explaining that neo4j is an option and how to use the feature.
Unless you guys have a very specific pipeline in mind right now, in which case we may as well try to satisfy that one instead. Let me know ;)
Raul

Hi Raul
Following changes have been made,

docker-compose.yml revert back to orignal looks
Docker-networkbuilder revert back to orignal looks
networkbuildercoordinator.py & fieldnetwork.py revert back to orignal looks
add a export (export_network_2_neo4j.py) function, add create a docker image specific for running this function.

Please take a look.

Awesome, thanks!


5.10.0-1

Coverage remained the same at 62.077% when pulling 5a2d41b8199959bab10a7169bfeddb2a24e4393b on 5.10.0-1 into 16574ef2d5c5b2eddeff1418213b3a939611db2b on master.


host update email

Email sent out once a day (at 7pm GMT?) to hosts of active future events.
A separate email should go out an hour before the event is scheduled to begin (or at 9am if the event is "all day"?) with a list of attendees.
Email should include:

link to event dashboard
number of new bookings
total number of bookings
total amount raises
link they can use to promote the event
any other blurb you want about sending out updates, how to promote the event.
what about some true/false variables week_before, two_days_before which you could use to customise the content of specific emails?

@azadiproductle @roselukas thoughts? anything else?

Also need to do event_host_final_update which needs to include details on all guests including booking reference.


Showing text without scrolling

Is there currently a way to show some string centered (without scrolling) if it fits on the displays? It might be useful for showing things like the current time or temperature where you don't want the text to scroll.
If not, I could send you a Pull Request if you like.

Which type of display are you talking about? The 8x8 matrixes or the 7-segment displays?
If you mean the 7-segment displays, then you could use the letter function to position specific letters at different digit places (and it would be quite easy to wrap that up into a write_text function that manages left/right/centre justification). From memory, I think the position is counted from the right most digit.
The matrix version would be a tiny bit more complicated as you would need to take account of the font widths.
I'm away for a few days, so by all means go for a PR (try and stick to the PEP8 style guide and please add a docstring to any public methods).

Which type of display are you talking about? The 8x8 matrixes or the 7-segment displays?

Yes, I am talking about the 8x8 matrices. I'll send you a PR implementing it :+1:


WriteSharp.Check() ignore the passed Whitelist

WriteSharp.Check is ignoring the whitelist passed in writeSharpOptions and use the default whitelist

Fixed in commit #890e46

Fixed in commit #890e46d


interesting posts on simulation

https://stats.stackexchange.com/questions/105881/how-to-simulate-survival-times-using-true-base-line-hazard-function

https://github.com/faithghlee/SurvivalDataSimulation/blob/master/survival-data-simulation.pdf

I wrote some code last week do simulate some survival time data. It could probably be improved but it's something
`import numpy as np
import pandas as pd
def data_generator(n=10000,seed=101):
np.random.seed(seed=seed) #Setting seed
df = pd.DataFrame(index=range(n)) #generating empty data frame with n rows
df['eb'] =<PHONE_NUMBER> #Unix date time for start
df['de'] = np.random.uniform(size=len(df))450000000 #adding a random normal distribution
df['dob'] = pd.to_datetime(df['eb'] + df['de'],unit='s').dt.date #converting to datetime
df['z0'] = np.random.binomial(n=1,p=0.7,size=len(df)) #Simulating baseline confounder
df['pa0'] = 1 / (1 + np.exp( -(np.log(1) - 40.7 + 4df['z0']))) #Simulating probability for exposure
df['a0'] = np.random.binomial(n=1,p=df['pa0'],size=len(df)) #Simulating binary exposure based on probability
df['predy'] = 18 + 10df['a0'] - 9df['z0'] + 0df['a0']*df['z0'] #Simulating probability of outcome
df['t'] = np.random.weibull(2,size=len(df)) * df['predy'] #Setting survival times from Weibull dist
df['t2'] = np.where(df['t']>5,5,df['t']) #Administrative censoring time is 5
df['delta'] = np.where(df['t']>5,0,1) #Event indicator if event happened before time 5
return df
`


Add a sleep in Jupyter Backend poll loop

Re: conversation at https://noteables.slack.com/archives/C02FHK2RVLH/p1661454678229579?thread_ts=1661442230.269859&cid=C02FHK2RVLH

Current dependencies on/for this PR:

main

PR #37   👈

This comment was auto-generated by Graphite.

Codecov Report

Merging #37 (671167b) into main (c95f327) will decrease coverage by 0.33%.
The diff coverage is 81.81%.

@@            Coverage Diff             @@
##             main      #37      +/-   ##
==========================================
- Coverage   95.71%   95.37%   -0.34%
==========================================
  Files           8        8
  Lines         467      476       +9
==========================================
+ Hits          447      454       +7
- Misses         20       22       +2

Impacted Files
Coverage Δ

sending/backends/jupyter.py
97.67% <81.81%> (-2.33%)
:arrow_down:

Continue to review full report at Codecov.

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


Update version to 1.0.69-3 on staging

Update version to 1.0.69-3

🚀 Deployed to staging in version: 1.0.69-3🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production in version: 1.0.73-3🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


[feat] Use new restoreNVM API to restore NVM backups

Describe the solution you'd like
Z2M currently uses the restoreNVMRaw API to restore a backed up NVM. This API requires the the source and target controller NVMs are compatible. node-zwave-js v8.9.0 added a new restoreNVM API which automatically converts the NVM backup to the correct target format. Using the new one would allow users to backup from a 500 and restore to a 700, for example.
Unless I'm missing something, there doesn't seem to be any reason for applications to use restoreNVMRaw anymore with this new API available.

@kpine Thanks! It's a quick fix, just need to change the api name


add example:docker easy-container-env-variables

I will work on 15 create a new container with some env variables from terraform variables easy-container-env-variables

Good job. Thank you. Welcome to the club.


adding loci-geometry config

As per http://catalogue.linked.data.gov.au/resource/143

I would action this by hand (i.e. add in the content manually to avoid the large conflict) however the GDS appears to be down. When it's back up, let me know (mostly so I can test) and I will add.

I'm closing this as the GDS is long gone and there's a new regime in place for LocI geometries now - the FSDF datasets.


Creative way of allowing option as a parameter

Is there some creative way of using ReScript syntax to allow each argument to be either a string or an option? It would be nice to have such a convenience.

You could have your own type with all possible variants you want.
type t = Cnst(string) | Optional(option<string>) | Boolean(string, bool)
But this way you would wrap all of your values, and pattern-match them later

I don't think it's possible w/o ppx, which is not gonna make it into this library for the reasons I elaborated on here.


Optimize Kotlin compiler context reuse

The main bottleneck on first use of a multi-project build involving hundreds of projects is Kotlin compilation time.
The following tasks can be executed from a Gradle checkout to analyse the Kotlin compilation behaviour:
 $ ./gradlew ktsManyProjects
  ...
  Generating test project 'project99' into .../gradle/subprojects/performance/build/ktsManyProjects/project99
 $ cd subprojects/performance/build/ktsManyProjects
 $ ../../../../gradlew tasks --recompile-scripts
  ...
  BUILD SUCCESSFUL

  Total time: 37.239 secs

Hey @ligee, I'm assigning this one to you because of the conversation we had some time ago:

we’re trying to keep the context ourselves, so this is a good opportunity to see whether we are doing it correctly. I think we should start from analyzing the log.

Please let me know if you need changes to the code generation scripts.


incomplete message doesn't update when changing language without refreshing page or submitting

Reproduction
https://formkit.link/605d0604f01195f5e11b61e47dac5a27
Describe the bug
We use i18n to switch between norwegian and english, and we get all other instances of the form - the labels, error messages etc to change language instantly when pressing a language change button, but not the incomplete message. If I refresh the page or hit the submit button again, it updates.
We use Vue3, composition api - and I have tried different methods like watch, watchEffect etc with no luck.
Hard to reproduce, but have put the example in the playground. The incomplete message wont update, but the other example with the name input updates as expected without need of refreshing.
I asked in the discord server, and Andrew Boyd answered that this might have to do with that attribute not being reactive.
Environment
• OS: [e.g. iOS]
• Browser [e.g. chrome, safari]
• Version [e.g. 22]

Resolved in 1.4.0 👍

great! thanks for the quick fix!


Issues with exporting trained models for inference

Hi,
I trained a resnet50 model using your keras-resnet script, after that I got the .meta, .index and .data files which look like a standard tensorflow checkpoint. However, when I tried to load it, freeze it for inference. The checkpoint files seem to have some problems. Please take a look at the following code.
output_node_names = 'fc1000/kernel'
output_graph = './data/resnet50_0221/output_graph.pb'

with tf.Session(graph=tf.Graph()) as sess:
    new_saver = tf.train.import_meta_graph('./data/resnet50_0221/model/graph-0221-040251.meta',clear_devices=True)
    new_saver.restore(sess, './data/resnet50_0221/model/model-31575')

    frozen_graph_def = tf.graph_util.convert_variables_to_constants(
        sess,
        tf.get_default_graph().as_graph_def(),
        output_node_names.split(','))

    with tf.gfile.GFile(output_graph, "wb") as f:
        f.write(frozen_graph_def.SerializeToString())
    print("%d ops in the final graph." % len(frozen_graph_def.node))

It finally print out this:
INFO:tensorflow:Restoring parameters from ./data/resnet50_0221/model/model-31575 INFO:tensorflow:Froze 1 variables. Converted 1 variables to const ops. 1 ops in the final graph.
which is wrong because there should be 161 variables.
The same code worked fine with my other models, therefore I'm sure it's something wrong with the checkpoint files generated by tensorpack.
Do you have any idea? Thanks

Because you set the output node name to the variable, only the variable will be left.

Ohhh, thanks! I just realise that 'fc1000/kernel' is a variable. So the output node should be 'fc1000'? But it says AssertionError: fc1000 is not in graph after trying.

The name would depend how keras implement the layer so I don't know -- and it may change across versions. Look at the graph or print the nodes to find it out.

Thanks for your advices. However, I found more than 50k node names in the graph and I have no idea which are the input and output node names.

That's why I would recommend you to build the inference graph by yourself. See http://tensorpack.readthedocs.io/en/latest/tutorial/inference.html#inference-after-training


[BUG] - FrameLayout cannot be cast to CropImageView

I m getting this error
Unable to start activity ComponentInfo{com.test/com.canhub.cropper.CropImageActivity}: java.lang.ClassCastException: android.widget.FrameLayout cannot be cast to com.canhub.cropper.CropImageView
when using
CropImageContractOptions options = new CropImageContractOptions(outputUri, new CropImageOptions())
.setGuidelines(CropImageView.Guidelines.ON_TOUCH);
cropImage.launch(options);

Hey welcome 👋 to the library and thanks for the Issue.
Can you please follow the template we have for issues? 🙏
There are many points on the template and will help me to help you.
I will close 🔴  this issue so you can reopen following the template,
Is important to remember that my work here is volunteer and to save time and have better answer help me that people use the templates.
Thanks!! 🎉


Issue with chefdk 0.11.0

I have recently upgraded chefdk from version 0.10.0 to 0.11.0
Everything worked perfect till now but after the upgrade when trying to execute knife runs list

 $ knife runs list myNode -VV
INFO: Using configuration from /home/dimka/Projects/MyCHEF/.chef/knife.rb
/home/dimka/.chefdk/gem/ruby/2.1.0/gems/knife-reporting-0.4.1/lib/chef/knife/runs_list.rb:65:in `run': uninitialized constant Chef::REST (NameError)
    from /opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.7.2/lib/chef/knife.rb:416:in `block in run_with_pretty_exceptions'
    from /opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.7.2/lib/chef/local_mode.rb:44:in `with_server_connectivity'
    from /opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.7.2/lib/chef/knife.rb:415:in `run_with_pretty_exceptions'
    from /opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.7.2/lib/chef/knife.rb:214:in `run'
    from /opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.7.2/lib/chef/application/knife.rb:148:in `run'
    from /opt/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.7.2/bin/knife:25:in `'
    from /usr/bin/knife:50:in `load'
    from /usr/bin/knife:50:in `'

I can confirm that I am experiencing the same issue.  I installed chefdk-0.11.2 (Windows) and the knife runs list command throws the following exception:
D:\gitrepo>knife runs list servername.domainname.com -VV
INFO: Using configuration from D:/gitrepo/.chef/knife.rb
C:/Users/username/AppData/Local/chefdk/gem/ruby/2.1.0/gems/knife-reporting-0.4.1/lib/chef/knife/runs_list.rb:65:in `run': uninitialized constant Chef::REST (NameError)
        from C:/opscode/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.7.2-universal-mingw32/lib/chef/knife.rb:416:in`block in run_with_pretty_exceptions'
        from C:/opscode/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.7.2-universal-mingw32/lib/chef/local_mode.rb:44:in `with_server_connectivity'
        from C:/opscode/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.7.2-universal-mingw32/lib/chef/knife.rb:415:in`run_with_pretty_exceptions'
        from C:/opscode/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.7.2-universal-mingw32/lib/chef/knife.rb:214:in`run'
        from C:/opscode/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.7.2-universal-mingw32/lib/chef/application/knife.rb:148:in `run'
        from C:/opscode/chefdk/embedded/lib/ruby/gems/2.1.0/gems/chef-12.7.2-universal-mingw32/bin/knife:25:in `<top (required)>'
        from C:/opscode/chefdk/bin/knife:60:in `load'
        from C:/opscode/chefdk/bin/knife:60:in `<main>'

I'm running windows as well and I made the changes suggested in this thread and it seems to be working for me: https://github.com/chef/chef/commit/9bb3f00df9774d352e277fa9f03e7c2009e00598

@simxitnsz Thanks for the link and advice, the code change needs to be applied to runs_show and runs_list but works fine for me.

This was fixed in 0.5.0


Presenting Modally From Detached ViewController

Hi,
I'm trying to present a VC modally from the VC of one of the selected slide menu items. However, I keep receiving this warning:
Presenting view controllers on detached view controllers is discouraged.
Can someone, please, help me?

I can see the same problem. It's especially bad if you try to show a modal screen with another navigation in it, then you get unpredictable states.

If you use the version from cocoapods then please make sure that this line https://github.com/NinoScript/AMSlideMenu/commit/5df647e1a4e0bcd2b7d3fef8b87427f8c61f1042
exists  there. If not then just add it

That fixes it! You should absolutely create a pull request for this one. Thanks a lot!

Would be great if we can add this into a bug fix release, instead of closing this ticket.


composer validate and builds even empty main.echo file

Steps to reproduce
1> create an package with empty main.echo file
2> run composer validate and composer build
Actual Behaviour
It validates and builds without any error
Expected Behaviour
It should throw an error, build and validate should expect some mandatory fields
Screenshot

Issue verified and closing as fixed


feat: update installation scripts for quick install

Motivation

Currently quick installation requires cloning the repo locally which all we really need is the forge and cast binaries.

Solution

Updates installation scripts to pull and execute script from GH instead.

I suppose test / CI install workflow will fail until these changes are available on main ?