Provide update method for Design Class

At present, the following functions call the Design.init() function due to the lack of an appropriate update function.
insert_design
set_active_design
copy_design_from
duplicate_design
It is proposed to develop an update() function to avoid losing the pointer to Design due to the constructor call

In this context we should be able to update the primitives property of the modeler whenever the design is updated. This will enable us to eliminate the modeler code:
@property
def primitives(self):
""" """
if self._primitivesDes != self._parent.project_name + self._parent.design_name:
self._primitives.refresh()
self._primitivesDes = self._parent.project_name + self._parent.design_name
return self._primitives.

From there we might consider doing the right thing and inheriting the Primitives classes into Modeler directly. That would allow the user to write aedtapp.modeler.create_box directly and eliminate a bunch of reduntant properties like oproject etc, with the primitives classes in the background. We could keep an aedtapp.modeler.primitives pointer for backwards compatibiility, i.e. self.primitives = self.  One to think about during the summer vacation ...

@maxcapodi78 I think this issue could be closed right? Design class has changed a lot and now these methods are independent


⚠ Bofrost.at has degraded performance

In 91b1252, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1084 ms

Resolved: Bofrost.at performance has improved in ab61e32.


Circular reference: Utils.js ->ClientAuthError.js -> Utils.js

I'm submitting a...

[ ] Regression (a behavior that used to work and stopped working in a new release)
[x] Bug report
[ ] Performance issue
[ ] Feature request
[ ] Documentation issue or request
[ ] Other... Please describe:

Browser:

[x] Chrome version 74
[ ] Firefox version XX
[ ] IE version XX
[ ] Edge version XX
[ ] Safari version XX

Library version

Library version: 1.0.0

## Current behavior
Build with rollup.aot identifies circular reference: "Circular dependency: node_modules\msal\lib-es6\Utils.js -> node_modules\msal\lib-es6\error\ClientAuthErr or.js -> node_modules\msal\lib-es6\Utils.js"

## Expected behavior
No circular reference.

## Minimal reproduction of the problem with instructions
1. Create Angular 7.x web application
2. Build with rollup (https://rollupjs.org/guide/en#rollup-rollup)

I think this is actually related to a bad Chart.js package in my site.  Closing.


Update openstack-ansible SHA

This commit updates the openstack-ansible SHA in preparation for the
next release.
The following issues are addressed by this change:
Issue: https://github.com/rcbops/rpc-openstack/issues/1112

👍  can confirm this OSA sha contains the necessary fix for issue #1112
$ git checkout 273912dab355deb4427240915b4fd8eb280d17f1
HEAD is now at 273912d... Migrate keystone v2 images to v3 during Liberty upgrade
[16:59:29] ~/working/.../openstack-ansible [(detached from 273912d)]
 $ git log --pretty=oneline|grep b3777c507b602ca9d44728e169e58e4811dda9bf
b3777c507b602ca9d44728e169e58e4811dda9bf Make ansible managed line in swift.conf static

Verified too.
👍


Updated readme, removed commented code

New Overview, Installation, and Usage sections in readme.
Removed some commented code from index.js

Example Joi validator:
// This is a Joi schema
var INSTANCE_SCHEMA = Joi.object().keys({
  timeToLive: Joi.number().integer().min(0), // duration in seconds
  size: Joi.string().valid("micro", "small", "medium", "large").default("micro"),
  sshKeyPair: Joi.string().required()
});

// Use the schema to validate a payload
var validated = Joi.validate(request.payload, INSTANCE_SCHEMA);
if (validated.error) {
  reply(Boom.badRequest(validated.error.message));
} else {
  var instance = validated.value;
  // do cool stuff with instance
}

Awesome, thanks Tyler! I'll incorporate all of your comments and make another PR.

All you have to do is push changes to your working branch. It will automagically update the PR :smile:

That is, push to jutonz:master

Oh, I didn't know that. Haha I'll do that then--thanks again!

PRs are great tools for having and capturing conversation about new code, seeing the changes in the same PR makes it easier to follow the history.

Yeah, that's definitely something I'm learning! I didn't realize how flexible they are. Thanks again for your feedback!


Legacy Tile Source fetching 2 layers

Hello there,
Curious to know as to what could be causing the Legacy Tile Source to fetch 2 levels upon initializing.
This is happening on my implementation but can also be witnessed on the Legacy Tile Source example from this page:
https://openseadragon.github.io/examples/tilesource-legacy/
If you watch the network request, there are two images being fetched for the last example (house with red roof).

Good question! In the case of the house with the red roof, I assume the lower res tile is for the navigator, and the higher res tile is for the viewer. Are you using a navigator in your version as well, or the reference strip?

@iangilman  thanks for responding.  I am not familiar with the navigator option.  Our implementation is very similar to the examples provided on the link I shared.  To summarize, we are providing 4 different 'levels' when initializing Open Sea Dragon.
The levels are in order of smallest to greatest, left to right.
Level 4 is the full resolution of the image.
In our implementation, OSD requests Levels 2 and 3.
I am uncertain if the request for multiple images is an error on our end, or if this is intended by OSD.
Can you link me to a documentation with references to the Navigator so I can confirm if this exists in our code?

@iangilman thanks for responding. I am not familiar with the navigator option. Our implementation is very similar to the examples provided on the link I shared. To summarize, we are providing 4 different 'levels' when initializing Open Sea Dragon.
The levels are in order of smallest to greatest, left to right.
Level 4 is the full resolution of the image.
In our implementation, OSD requests Levels 2 and 3.
I am uncertain if the request for multiple images is an error on our end, or if this is intended by OSD.
Can you link me to a documentation with references to the Navigator so I can confirm if this exists in our code?

@iangilman  thanks for that example.  I can confirm we do not use this feature.
Here's a public link to our implementation where you can see that OSD is fetching the 'quarter' and 'half' size images.
Looking forward to your response.
https://public.earthcam.net/earthcam/earthcampus/61_megapixelcam/view
thanks,

Great, thank you! Cool site, BTW :)
I think I've figured out what's going on... By default, OSD loads the smallest tile that covers the entire image first, and then loads higher resolution tiles afterwards. This makes a lot of sense with a tiled pyramid, because those higher resolution levels are composed of many tiles, and it may take a while for them all to load in. In your case, it's not as important, because all of your tile levels cover the entire image.
If you watch closely, you can see the low resolution image show up briefly and then get replaced by the higher resolution one. It's not much of a delay between them, so you might as well just be loading the higher resolution one in your case. I believe you can make OSD do that by using immediateRender: true in your options when creating the viewer.
It seems possible we should consider making that the default for LegacyTileSource. 🤔

@iangilman  thanks to your suggestion, I started going over each option that we added in our implementation to see what they do and was able to find that the property "homeFillsViewer" was causing the extra tile layer fetching since we are making sure that the image occupies the whole view and therefore requires a few steps of zooming-in.  "immediateRender" was already being assigned as true in our option.
Once again, thanks for looking into this with me!
I really appreciate your assistance.
Thanks!

Interesting, thank you for letting me know what worked!


Fix new year ArgumentOutOfRangeException in facebook's posts

Since there are no posts yet for facebook in this year (http://netdevelopers.pl/facebook/top-current-year) server currently throws an exception System.ArgumentOutOfRangeException in HLListPage.cs:25. I've decided not to use this class (sorry  @pawelklimczyk :D), change this to List<> and prompt user that there are no posts found.

@rwojcik it looks like it doesn't build. Check the AppVeyor logs

I will fix that in HandyLib. HLListPage was introduced because it has build-in pagination mechanism...
I will fix that tomorrow...

Well, if it's going to be fixed in HLListPage then there is no point in keeping this PR

Part of it was fixed in HLListPage
whole fix is here:
https://github.com/NetDevelopersPoland/NetDevPLWebsite/commit/97508f3dc006d1a61b94d248ec21a5ee1931972a


chore: add package-lock to gitignore, add note about yarn

as per comments in #2250

Having both yarn.lock and package-lock.json is not a good practice. In addition, we're disabling it in core repo as well.

@montogeek let me know please if there is anything else pending to get this one merged?

Thanks!

Please avoid merging master constantly, this repo changes quickly and every merge is wasting our limited CI time.

Sure, noted 👍🏽
Btw, a few days ago I noticed the same CI error, seems the requests are still not authenticated. Do you want me to have a look at it?

@byzyk Yeah, no idea :/


Remove deprecated cli flags

Proposal
Remove deprecated storage.tsdb.retention and remote-write-receiver cli flags for Prometheus 3.0.

I should check the correct branch before opening issues.


Add support for presigned URLs in the lakeFS API

This one requires discussion :)
Currently, lakeFS users have 2 options when reading and writing data:

Involve lakeFS in the data path: this typically means either using the s3 gateway to read/write data, or to use the data operations in the lakeFS API (get_object, put_object)
Use lakeFS for metadata only, with direct read/write to the underlying object store.

This requires a pretty hard tradeoff: (1) has implications regarding security (data going through the lakeFS server) as well as added operational complexity (i.e. the need to size a lakeFS installation for the required bandwidth) and potentially cost implications if the lakeFS installation is outside the AWS account and VPC used by its consumers (such as in lakeFS Cloud).
As for (2), it has a different set of problems: Once the metadata operation is done, direct access to the object store requires authentication and authorization. This means managing 2 different auth* systems: one for lakeFS and one for the underlying object store. Keeping both of them in sync is very hard (i.e. how do I give every user permissions to read only the exact underlying objects in a repository?)
This PR introduces pre-signed URLs as a middle ground. It allows receiving short-lived pre-signed URLs from GetObject, StatObject and ListObjects with the optional presign=true query parameter, as well as limited write support using GetPhysicalAddress where the address returned is pre-signed for a write operation.
This PR currently does not handle Multipart uploads, this could be added later (and is probably a requirement before it resolves #3160)

This is currently in draft and requires discussion. I don't plan on merging this unless we agree this is desired.

Yay, this is a great direction!
One worry is how to use this from lakeFSFS, because I would really like to use this from lakeFSFS.  While this is certainly not the only client, it's a pretty important one.  Here I am not sure how to use this and get the optimizations from S3A and other underlying FileSystems.

@ozkatz Still missing tests for most of the APIs to get the links, including validations that links actually work.

You're right @itaiad200 - Merged too early. Tests added in #5035 5035


Update dependencies

Armeria 0.95.0 -> 0.96.0
Jackson 2.10.0 -> 2.10.1
Micrometer 1.3.0 -> 1.3.1
Spring Boot 2.1.9 -> 2.1.10
Build:

Gradle 5.6.3 -> 6.0
JMH 1.21 -> 1.22
json-unit 2.11.0 -> 2.11.1
shadow 5.1.0 -> 5.2.0

Let me merge this and cut a new release to make some progress tonight. Will release a new version if there's something I need to address.


Set up the development environment

**As a** developer
**I need** to set up the environment
**So that** I can complete tasks for ecommerce project

### Details and Assumptions
    * [document what you know]
### Acceptance Criteria
    gherkin
    Given [some context]
    When [certain action is taken]
    Then [the outcome of action is observed]


After updating to 1.4.1 the store breaks -> No exports found from React components[BUG]

Describe the bug
So I am developing a new store with Hydrogen (1.3.1) and Sanity, but after updating to 1.4.1 I won't see any page anymore. Visiting the routes will lead to "Page not found". In the server console there are warnings that there are default exports found.
WARN: /src/routes/index.server.tsx doesn't export a default React component or an API function
WARN:  /src/routes/robots.txt.server.ts doesn't export a default React component or an API function
WARN:  /src/routes/sitemap.xml.server.ts doesn't export a default React component or an API function
WARN:  /src/routes/account/index.server.tsx doesn't export a default React component or an API function
WARN:  /src/routes/account/login.server.tsx doesn't export a default React component or an API function

+31 etc

To Reproduce
Upgrade to 1.4.1 and run yarn dev
Expected behaviour
Expects everything to run like before on 1.3.1

Hydrogen version V 1.4.1
Node version V16.14.2
Device details: Mac OS M1 Max

I think there might be something weird going on with your dependencies. Can you try the following?

Upgrade to Node 18
Remove node_modules and run yarn again to install everything
Run yarn dev --force


How can I add hover shadow on selected side menu item?

I just want to give a sign on selected side menu item if its already clicked when side menu is open
How can I figure out it?

In your tableViewController, in viewDidLoad function :
    // Uncomment the following line to preserve selection between presentations
    self.clearsSelectionOnViewWillAppear = false

compile time error on "clearsSelectionOnViewWillAppear"

What error?

How about a ViewController ,I don't want the action to happen inside the shifted front view controller when the Side menu is opened , but just tapping on it should close the drawer back.
How to do this ???
My Class is CustomerViewController : MyBaseviewController and its not UITableViewController

@vickyrappier thanks for pointing out this issue I'm concerning this as well

I fixed it with solution provided by the #hardluckbaby on stack link as below. You may provide an upgrade as I am using pods so can take the latest fix.
http://stackoverflow.com/questions/22601448/swrevealviewcontroller-remove-interaction-on-frontview-when-rearview-is-reveale

#Side menu for panic and settings
pod 'SWRevealViewController', '~> 2.3'
Pods that I used or if you wanna suggest anything different for Pods when upgrade as i was IOS 9.0 onwards. Great Component although so far that i saw and compared with. Good Job

I wrote a little sample to illustrate this thread:
SWSample.zip

Thanks John, I shall have a look and use the same. Appreciate your help.
Thanks,
Regards,
Vicky S Dhas
Contact: +65 9183 9203
Email :vickydhas<EMAIL_ADDRESS>On Wed, Apr 13, 2016 at 6:46 PM, iDevelopper<EMAIL_ADDRESS>wrote:

I wrote a little sample to illustrate this thread:
SWSample.zip
https://github.com/John-Lluch/SWRevealViewController/files/216865/SWSample.zip
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/John-Lluch/SWRevealViewController/issues/568#issuecomment-209363390

I'm not John but a user like you!

Another sample in Swift:
MapSample.zip

Thank You iDevelopper!
Your sources help me very well.

@azsha916 , you're welcome.


Question abouth the sparse encoder

Hello,
Thanks for your inspiring work, and I am trying to run the code you provided.
I met the following error, which indicates the shape mismatch in the sparse encoder. I have scrutinized the shape of all LiDAR scans in the NuScenes but cannot find any error.
Any suggestions for solving this?
Traceback (most recent call last):
File "tools/train.py", line 89, in
main()
File "tools/train.py", line 78, in main
train_model(
File "/root/paddlejob/workspace/whole/bevfusion-main/mmdet3d/apis/train.py", line 128, in train_model
runner.run(data_loaders, [("train", 1)])
File "/root/paddlejob/workspace/whole/bevfusion-main/env/guangrui_cuda11/lib/python3.8/site-packages/mmcv/runner/epoch_based_runner.py", line 127, in run
epoch_runner(data_loaders[i], **kwargs)
File "/root/paddlejob/workspace/whole/bevfusion-main/mmdet3d/runner/epoch_based_runner.py", line 15, in train
super().train(data_loader, **kwargs)
File "/root/paddlejob/workspace/whole/bevfusion-main/env/guangrui_cuda11/lib/python3.8/site-packages/mmcv/runner/epoch_based_runner.py", line 50, in train
self.run_iter(data_batch, train_mode=True, **kwargs)
File "/root/paddlejob/workspace/whole/bevfusion-main/env/guangrui_cuda11/lib/python3.8/site-packages/mmcv/runner/epoch_based_runner.py", line 29, in run_iter
outputs = self.model.train_step(data_batch, self.optimizer,
File "/root/paddlejob/workspace/whole/bevfusion-main/env/guangrui_cuda11/lib/python3.8/site-packages/mmcv/parallel/distributed.py", line 52, in train_step
output = self.module.train_step(*inputs[0], **kwargs[0])
File "/root/paddlejob/workspace/whole/bevfusion-main/mmdet3d/models/fusion_models/base.py", line 78, in train_step
losses = self(**data)
File "/root/paddlejob/workspace/whole/bevfusion-main/env/guangrui_cuda11/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
return forward_call(*input, **kwargs)
File "/root/paddlejob/workspace/whole/bevfusion-main/env/guangrui_cuda11/lib/python3.8/site-packages/mmcv/runner/fp16_utils.py", line 128, in new_func
output = old_func(*new_args, **new_kwargs)
File "/root/paddlejob/workspace/whole/bevfusion-main/mmdet3d/models/fusion_models/bevfusion.py", line 194, in forward
feature = self.extract_lidar_features(points)
File "/root/paddlejob/workspace/whole/bevfusion-main/mmdet3d/models/fusion_models/bevfusion.py", line 128, in extract_lidar_features
x = self.encoders["lidar"]["backbone"](feats, coords, batch_size, sizes=sizes)
File "/root/paddlejob/workspace/whole/bevfusion-main/env/guangrui_cuda11/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
return forward_call(*input, **kwargs)
File "/root/paddlejob/workspace/whole/bevfusion-main/env/guangrui_cuda11/lib/python3.8/site-packages/mmcv/runner/fp16_utils.py", line 128, in new_func
output = old_func(*new_args, **new_kwargs)
File "/root/paddlejob/workspace/whole/bevfusion-main/mmdet3d/models/backbones/sparse_encoder.py", line 122, in forward
x = encoder_layer(x)
File "/root/paddlejob/workspace/whole/bevfusion-main/env/guangrui_cuda11/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
return forward_call(*input, **kwargs)
File "/root/paddlejob/workspace/whole/bevfusion-main/mmdet3d/ops/spconv/modules.py", line 132, in forward
input = module(input)
File "/root/paddlejob/workspace/whole/bevfusion-main/env/guangrui_cuda11/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
return forward_call(*input, **kwargs)
File "/root/paddlejob/workspace/whole/bevfusion-main/mmdet3d/ops/spconv/modules.py", line 136, in forward
input.features = module(input.features)
File "/root/paddlejob/workspace/whole/bevfusion-main/env/guangrui_cuda11/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1102, in _call_impl
return forward_call(*input, **kwargs)
File "/root/paddlejob/workspace/whole/bevfusion-main/env/guangrui_cuda11/lib/python3.8/site-packages/torch/nn/modules/batchnorm.py", line 168, in forward
return F.batch_norm(
File "/root/paddlejob/workspace/whole/bevfusion-main/env/guangrui_cuda11/lib/python3.8/site-packages/torch/nn/functional.py", line 2280, in batch_norm
_verify_batch_size(input.size())
File "/root/paddlejob/workspace/whole/bevfusion-main/env/guangrui_cuda11/lib/python3.8/site-packages/torch/nn/functional.py", line 2248, in _verify_batch_size
raise ValueError("Expected more than 1 value per channel when training, got input size {}".format(size))
ValueError: Expected more than 1 value per channel when training, got input size torch.Size([1, 32])

Looks like this is a training-related problem. Maybe you can wait until we release full training details. To me, it seems like some BN1d layer has batch size = 1 during training, which is definitely not allowed.

Thanks for your response, and look forward the full training code.


fix(devicedescriptors): fix UA in DeviceDescriptors

https://github.com/kblok/puppeteer/pull/7/commits/cd8d75062826ee9dcb511126d000773413a9c681

@kblok  I think we should create a .csx file that will generate the cs file similar to the puppeteer project in  fetch_devices.js


[Fix #1200] Make Rails/TimeZone aware of safe navigation

parse raises on nil so autocorrect is not possible in this case.

Before submitting the PR make sure the following are checked:

[x] The PR relates to only one subject with a clear title and description in grammatically correct, complete sentences.
[x] Wrote good commit messages.
[x] Commit message starts with [Fix #issue-number] (if the related issue exists).
[x] Feature branch is up-to-date with master (if not - rebase it).
[x] Squashed related commits together.
[x] Added tests.
[x] Ran bundle exec rake default. It executes all tests and runs RuboCop on its own code.
[x] Added an entry (file) to the changelog folder named {change_type}_{change_description}.md if the new code introduces user-observable changes. See changelog entry format for details.
[ ] If this is a new cop, consider making a corresponding update to the Rails Style Guide.

Can you rebase with the latest master branch?

Yeah, sure!

Thanks!


Remove upper bound on react-native peerDependency

Issue: #1418
The goal is to allow new react-native versions without having to maintain an upper bounded version in Storybook.
What I did
Updates Storyboard's react-native module to allow any versions >=0.27.0, rather than keeping a capped upper-bound. As described in https://github.com/storybooks/storybook/issues/1418#issuecomment-313234328, if a breaking change is later introduced the next version of storyboard can set the new minimum version. This removes the need to maintain an upper bound every time react-native rotates out a new release. I personally think it's okay to be a little less strict here.
Note: we cannot use ^0.27.0 since this is a prerelease, which will not be satisfied by semver
How to test

Create a new react-native project on a version later than 0.43.x
Locally link @storybook/react-native and observe a successful installation without an invalid peerDependency message

@tmeasday can you take a look at this perhaps?

I guess it comes down to how "solid" we want to be at the expense of being a bit slow. It's not difficult for someone to check each new version of RN and bump the range when we see that it works. But obviously the fact that it hasn't happened shows that no maintainer is particularly on top of this right now.
In any case I think most people ignore peer dependency warnings anyway so I suspect it's all a bit academic. I don't think npm uses them to decide which version to use or anything like that. So ultimately, it's about: for the subset of users that would be bothered by a warning about SB being incompatible with RN, is it better to be "loose" and hope for the best (not show the warning if we aren't sure we are compatible), or "tight" (show the warning if we aren't sure).
Thinking about it like that, I would suggest the latter. What do you all think?

@tmeasday The concern is that for anyone still using shrinkwrap, peer dependency issues mean actual errors. In that sense, erring on the side of being too "tight" can be a very real problem as well.

Ahh! Ok so I am wrong when I say users don't typically care about peer deps.
I'm OK with loosening it then, at least until someone who is actively using RN wants to take responsibility for bumping our versions.


WEBAPP-403: Add plugins for lodash

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword WEBAPP.

@Integreat/app-web-admin ping


Issues with game events

I have no idea what is causing this but I think there are issues with game events.
For several months, I noticed the "death" event wasn't properly emitted in caracAL. I ended up using "target_hit" to check on .death instead.
Now I get a similar issue: I'm using game.on("hit") to get all hits around me and I only consider the ones that are done by my party but I'm not sure it's working as intended: it's not reporting the characters that are not mine.
On the screenshot below, you can see I don't get as much data. On the left where we have lots of lines, I was running using the steam client and on the right where only three lines are left, I'm using caracAL (with the same code, of course). The only lines left are my own characters and nothing else.

This was my mistake, sorry.
I still think there is something strange with the death event, though.


People lists have an extra row

Seems like there is an extra row, that's not needed here:    It's throwing all people lists too wide

fixed in the gios2-api: https://github.com/gios-asu/gios2-php/commit/14fd48bdd980a15b0660059cb3bbeee4a16dcd74


Pasting text does not change the model

Hi there!
I've found that the model does not change, if I past text into the tinymce input.
I try to analyze the problem.
https://plnkr.co/edit/YRfHj2EpWldiH1856CxK?p=preview

I figured out that tinymce doesn't change the dirty flag by pasting text, unless the "past" plugin is loaded.
https://plnkr.co/edit/2Q1KfaUdh77XutpuhzsP?p=preview
So you think thats a bug in tinymce? Anyway, it is not an error in ui-tinimce!

Closing as a duplicate of #240


Integrate new Symbol DCE infra in MLIR to IREE

This should eliminate a lot of unused symbols and related code from VM binaries.

As an update, some of this has started to make its way in (for example, I'm tagging vm.imports with symbol visibility so they'll go away), but there's still a comprehensive sweep required to clean things up.

This would be a great starter bug! Geoff: maybe Phoenix could take a look
at this?
On Thu, Mar 19, 2020 at 10:18 AM Ben Vanik<EMAIL_ADDRESS>wrote:

As an update, some of this has started to make its way in (for example,
I'm tagging vm.imports with symbol visibility so they'll go away), but
there's still a comprehensive sweep required to clean things up.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/google/iree/issues/614#issuecomment-601308194, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AF3RZM3M3OKISCFA4AA22FLRIJHVPANCNFSM4KNLM26Q
.

A remaining issue I found today was that some of our input paths and tests have all symbols as public, which means that if you use that as your signal as to whether to export or not you'll end up exporting private functions (and trying to run them through the ABI mangling and such). Should just be a burn down on the unit tests until things pass adding SymbolTable::Visibility::Private to things.
I'm tagging a few bits of code with this github issues - there shouldn't be much to cleanup once solved and finding code paths that break is mostly trying things when iree.module.export is disabled.

@GMNGeoffrey I vaguely recall chatting about removing iree.module.export a week or two ago - were you looking into that? That may be enough to call this one complete.

I'm not actively pursuing it, no. Just remember you saying it shouldn't be necessary on one of my changes, so I tried removing it and determined that it was definitely still necessary :-D

Darn :P Well, if you're looking for something to poke at on the compiler side this would be a good one.


Update to 6.0.0a15 (r20240)

Bug fixes for the C API regarding creating grids from user defined
matrices.

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


[BanyanDB] some tests are missing in convert package in skywalking-banyandb

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
Currently, there are some tests are missing in convert package in skywalking-banyandb
Use case
The proposed solution is to address this by adding the necessary tests for the files in the engine package, which would help in detecting bugs early, supporting CI/CD, reducing debugging time, providing safety for this codebase etc...
Related issues
No response
Are you willing to submit a pull request to implement this on your own?

[X] Yes I am willing to submit a pull request on my own!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Which parts? Have you talked with @hanahmily

Which parts? Have you talked with @hanahmily

I didn't talk with @hanahmily, but while going through the code base I found some tests are missing in convert.

I appreciate your finding and welcome you to contribute to this task.
It's a good starting task to help you become familiar with the project.


Enable, or allow toggling, Fast Tanh

It would be great to get the fast tanh code out there - either by default, or as a toggle.
It drastically improves performance, and could do a lot to mitigate the high-CPU usage complaints people have.
PR https://github.com/sdatkinson/NeuralAmpModelerCore/pull/28 makes it very easy to integrate into the plugin.

Whoa...I didn't realize they were this close!

Yeah - I just hardcoded it enabled in the LV2 plugin. I figure I'm much more likely to have people complain about CPU usage than any perceptible impact of the fast tanh.
It would be interesting to do a null test comparing the output of a model using normal vs tanh. The impact to the audio signal could be more or less than the error of the activation function - I don't have a good intuition which is more likely.

It would be interesting to do a null test comparing the output of a model using normal vs fast tanh

...which is what I was literally doing before switching back over here haha
Seems great to me: Source signal: -16.5 LUFS integrated; Null test: -62.4 LUFS (-45.9 dB)
...Which is lower than nulling my Deluxe Reverb against itself (two reamps back to back).
Case closed 😄

Cool - I figured that would probably be the case, but it is good to have actual confirmation.


Run unittests on travis server

Travis is a continuous integration service that can be uses for open source projects located on github. They need a .travis.yml file that contains the commands to be executed after each push in the (master)
This branch contains that travis file. If the upstream project also enables the travis service, then each commit can be tested automatically.

I'm wary of making the recommended pydot and docopt hard dependencies, but using the this is incomprehensible enough with the CLI, so let's go ahead and require docopt. ...And if we're going to start requiring more than just plain python, let's just add pydot while we're at it.


Error when logging into world

Here's the issue on NEI addons: https://github.com/bdew/neiaddons/issues/54#issuecomment-161328859
I can't say which mod is causing the error and kicking me to menu, but it appears to be an issue with Forestry 4.2 in both mods.

Both NEI Addons and NEI Integration are causing errors here. Updating NEI Integration should fix my error (my Forestry support was moved into Forestry itself).


Add a test that checks that ClangImporter chooses only exported modules

This test checks that when ClangImporter qualifies types with a module name, we choose a visible module.
This is a test for SR-13032, taken from PR https://github.com/apple/swift/pull/32465.

@swift-ci please test


Persistent sessions

Is your feature request related to a problem? Please describe.
It would be very helpfull if there are persistent sessions in drogon, so for example you dont need to re-login everytime the app restarts. It also would help with load-balancing application so sessions can get syncronized across app instances via the help of a database for example.
Describe the solution you'd like
Ideally, there would be a system that let's a app react on each change of a session: create, update (insert & erase), destroy.
I think the best solution would be to implement #1853 in addition with being able to access the session object itself in AdviceStartSessionCallback and AdviceDestroySessionCallback, i.e. by adding a parameter to pass SessionPtr.
If #1853 is not implemented, all of Session's mutation methods would to be changed to allow to notify any custom logic about a changed session, this change notification then maybe could be reacted to with a drogon::app().registerSessionChangeAdvice() that also gets the SessionPtr as a parameter.
Describe alternatives you've considered
Only alternative right now, would to be write an custom solution for this and thus ignoring drogon's session feature completly.

I agree, every time I restart Drogon I have to accept cookies on my website
when in real life I only want the client to accept cookies once in their
lifetime for the domain.
On Mon, Jun 17, 2024, 12:31 AM Mai-Lapyst @.***> wrote:

Is your feature request related to a problem? Please describe.
It would be very helpfull if there are persistent sessions in drogon, so
for example you dont need to re-login everytime the app restarts. It also
would help with load-balancing application so sessions can get syncronized
across app instances via the help of a database for example.
Describe the solution you'd like
Ideally, there would be a system that let's a app react on each change of
a session: create, update (insert & erase), destroy.
I think the best solution would be to implement #1853
https://github.com/drogonframework/drogon/issues/1853 in addition with
being able to access the session object itself in
AdviceStartSessionCallback and AdviceDestroySessionCallback, i.e. by
adding a parameter to pass SessionPtr.
If #1853 https://github.com/drogonframework/drogon/issues/1853 is not
implemented, all of Session's mutation methods would to be changed to
allow to notify any custom logic about a changed session, this change
notification then maybe could be reacted to with a
drogon::app().registerSessionChangeAdvice() that also gets the SessionPtr
as a parameter.
Describe alternatives you've considered
Only alternative right now, would to be write an custom solution for this
and thus ignoring drogon's session feature completly.
—
Reply to this email directly, view it on GitHub
https://github.com/drogonframework/drogon/issues/2071, or unsubscribe
https://github.com/notifications/unsubscribe-auth/BBZFRV36522ZO6UU7SBYSVTZH2GDHAVCNFSM6AAAAABJNN4V6KVHI2DSMVQWIX3LMV43ASLTON2WKOZSGM2TMNRQGQ3DKNA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I have made a library for the purpose of identifying users, keeping their session IDs, and doing database manipulations on their sessions
There's drogon-user, for identifying users in a single dimensional flat way, for example in auth applications, game servers, ungrouped chatting applications, or as a wrapper for WebSockets in general
Then there's drogon-group, which is for grouping users into multiple groups, this can be used for chatting applications where groups are present, or real-time systems for organizations and groups.
But this one is less modular and you have to know some of the APIs to get it to work
My plan is to make drogon-user be similar to Facebook's auth, we'll see where it goes

@Mis1eader-dev thats nice, but I want an solution inside drogon. I also do not want any authentication or similar things, as I want to roll / use my own technology and/or stack for it. All this issue should be kept about, is to create an way inside drogon to allow an developer to persist an session. If this is done by the way I layed out, or an extra adapter that by default is a null-adapter, a file-adapter (like php) or something entirly else is irrelevant to me. I just want a way to do it on my own. Thanks.


Visualize check thresholds with giraffe threshold viewer

Checks should be visualized with CheckPlot.
Check view should include all viewPlot properties.

 What do you think about extracting out a helper like
const getThresholdType = (threshold: Threshold): 'range' | 'greater' | 'lesser' => {
// ...

I think removing this will mean that when a user drags the max below the min, then the threshold type will suddenly change from within: true to within: false, which sounds really jarring.
I imagine a way we can handle this is by determining the within value of a threshold when it is first instantiated, storing that in state, then treating the threshold accordingly. But that sounds complicated. I would really love to get those old Threshold types back.


eframe transparent window not fully transparent

On Windows
Creating a simple transparent Window with eframe only makes it semi transparent.
Tried to look into it, the window looks exactly like this, already mentioned in winit : https://github.com/rust-windowing/winit/issues/1859. In winit its fixed though.
The Window:
    let native_options = eframe::NativeOptions{
        decorated: false,
        transparent: true,
        ..eframe::NativeOptions::default()};

@kurtn3x did you set App::clear_color as mentioned in the docs https://docs.rs/eframe/latest/eframe/struct.NativeOptions.html#structfield.transparent ?

Oh, yeah sorry i'm blind, that fixed it. Thank you.


Linux Arm on Azure

Checklist

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

@conda-forge-admin, please rerender

As originally suggested by @chrisjsewell in https://github.com/conda-forge/pymatgen-feedstock/pull/124


lines like " float depth = -0.00fg1f;" are not generating obvious syntax errors

VSCode Version: Code 1.8.1 (ee428b0eead68bf0fb99ab5fdc4439be227b6281, 2016-12-19T14:41:20.664Z)
OS Version: Linux x64 4.8.10-040810-generic
Extensions:

Extension
Author
Version

unity-tools
Tobiah
0.1.9

unity-debug
Unity
1.1.0

unity-snippets
YclepticStudios
0.1.2

vscode-astyle
chiehyu
0.4.0

cs-syntax-visualizer
itn3000
0.0.2

csharpextensions
jchannon
1.1.0

docomment
k--kato
0.0.9

csharp
ms-vscode
1.6.1

omnisharp
ms-vscode
0.3.3

vscode-format
ryannaddy
0.0.6

Steps to Reproduce:

write: "float depth = -0.00fg1f;"
no errors are generated, where it should complain about the "fg" inside the number

@Guedez what is the language are you using when writing the snippet float depth = -0.00fg1f;

@Guedez any update?

Moving to omnisharp-vscode for further investigation.

This issue was moved to OmniSharp/omnisharp-vscode#1094


Intuition behind hessian and min_child_weight

I tried to understand the role of hessian and min_child_weight but not everything is clear from the documentation.

Why it is better to restrict hessian than number of samples?
I tried to go deeper with hessian of log loss and min_child_weight. Hessian of logloss is the derivative of sigmoid. So it is positive, takes maximum at zero and minimum at +-inf. What is intuition behind restricting such hessian with min_child_weight? Why node with observations with large hessian (that means class label forecast from previous iteration close to zero) is better for split than node with high absolute forecasts?
What is the role of hessian in gain calculation?

https://arxiv.org/pdf/1603.02754.pdf

benoitdescamps, I read this article and know these formulas but they don't answer my questions.

@sergun while I agree that the PDF does not magically give all the answers, the answer to the (3) is there, see eq. 7.

@superboby right, I saw the equation bu exact role is not clear. We can have two split candidates. Both with the same Gleft^2, Gright^2, (Gright+Gleft)^2 but differ in Hleft and Hright so one split will be better. It's not clear what is the meaning of such normalization by hessian during split find. It's complitely clear for RMSE loss when H is equal to number of observations but not clear for logloss when H is direvative of sigmoid

@superboby right, I saw the equation bu exact role is not clear. We can have two split candidates. Both with the same Gleft^2, Gright^2, (Gright+Gleft)^2 but differ in Hleft and Hright so one split will be better. It's not clear what is the meaning of such normalization by hessian during split find. It's complitely clear for RMSE loss when H is equal to number of observations but not clear for logloss when H is direvative of sigmoid

The equation following eq. 9 is a rather intuitive representation of the role of hessian:

@khotilov Thanks!
But l'm still out of understang of hessian role. I understand his meaning for RMSE loss when hessian=1 and this means number of samples in a node. But for hessian of logloss it is not clear why it is useful in th algorithm that it's value is large for small predictions from previous round and small for large predictions. Is it something like: "observations with are already good described by ensemble(=large values of prediction) should have smaller influence in a node to be splitted"? It is also interesting how this logic works in case of class imbalance when uncertanty of prediction is shifted from zero.

In the case of binary classification, you might assume that each observations are i.i.d. Bernouilli B(1-p). Recall that Bernouilli has mean 1-p and variance p(1-p). In the case of logloss notice that these are proportional to the mean and variance respectively.
Have a look at equation(5) in the paper. The weight is the ratio of the averaged probability of each  observation, hence roughly the average probability of the leaf up to a factor, divided by the sum of the variance of each observation, hence the variance of the probability of the leaf up to a factor.
This is roughly the Z-score of the leaf.
So basically when boosting, you are updating the probabilities of each observations with the Z-score of the new partition they belong to.
I dont know, if this help, but if your intuition for RMSE satisfies you, then this should be enough.


[MM-19327] Remove Finder from desktop app

Summary
Removing the Finder functionality from the desktop app in favour of this PR: https://github.com/mattermost/mattermost-webapp/pull/7718
Issue link
https://mattermost.atlassian.net/browse/MM-19327

/update-branch


odd issue

for some reason Api messes with my console command on the mod loader by not letting me close it when i removed api i was able to close my console command

hmmm not much I can do with that little information.
All the api does with the console is add a command itself.


Show that union ranges are broken

references #2231

@vasily-kirichenko I really don't get that. This test shows that the compiler is indeed emmitting the correct range.  Any ideas what's going on here?

No. But I've almost fixed it :)

Ok the bug only appears if the union is the last char of the whole document and it's unrelated to double backticks:

just checked. this is a regression and worked in VS2015

just checked with the compiler. it still emits the correct range. so this is broken in IDE

ok found the bug - will send a fix soon

fixed. please review and merge asap.

after applying this error reporting and renaming works as expected:


Ionic 8 support

👋 Hi! Thanks for this great package. Are there plans to upgrade to Ionic 8? It seems this project is pinned to 7.
https://ionicframework.com/docs/updating/8-0

done @khromov  - thanks for mentioning
8.2.2 is now active
I have not tested it :) :) :)

When I ran npm create ionic-svelte-app@latest, I still got Ionic version 7, but I was able to manually update the package.json to Ionic 8.

Thanks for reporting
Will look into it

Fixed

Thanks, I'm now getting 8 as expected when I create a new app.


Added a11y document for Menu

Closes #1545
Description
[English]

[Japanese]

Is this a breaking change (Yes/No):

No

@108yen
should MenuItem have menuitem role?
https://www.w3.org/WAI/ARIA/apg/patterns/menubar/

When reading the following descriptions, also keep in mind that:
Focusable elements, which may have role menuitem, menuitemradio, or menuitemcheckbox, are referred to as items.

@taiseiiiii
Thanks for the review.
Fixed!

@taiseiiiii
Thanks for pointing that out.
Fixed.


[HOLD for payment 2024-11-29] Verify account - "Code requested" only shows in 2FA page after user opens contact method page

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

Version Number: 9.0.60-0
Reproducible in staging?: Y
Reproducible in production?: N/A - new feature, doesn't exist in prod
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Go to staging.new.expensify.com
Log in with a new Gmail account.
Go to Settings > Security > Two-factor authentication.
Note that there is no "Code requested. Please check your device." message.
Go to Profile > Contact method.
Click on the email.
Go back to Security > Two-factor authentication.
Now it shows "Code requested. Please check your device." message.

Expected Result:
In Step 4, 2FA page should show "Code requested. Please check your device."
Actual Result:
In Step 4, 2FA page does not show "Code requested. Please check your device."
It only shows the message (Step 8) after user opens Contact method page in Step 6.
Workaround:
Unknown
Platforms:

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

Screenshots/Videos
https://github.com/user-attachments/assets/7a1b7ea2-d2a1-4dd4-9021-15f47b950855

View all open jobs on GitHub
Issue OwnerCurrent Issue Owner: @adelekennedy

Proposal
Please re-state the problem that we are trying to solve in this issue.
In Step 4, 2FA page does not show "Code requested. Please check your device."
It only shows the message (Step 8) after user opens Contact method page in Step 6.
What is the root cause of that problem?
We are calculating hasMagicCodeBeenSent based on login data and this is not correct when we are calling API resendValidateCode so in step 4 we don't see "Code requested. Please check your device." message
after entering contact method loginData?.validateCodeSent is updated so we go back to 2FA it will show
https://github.com/Expensify/App/blob/b2d6a0eace9d22bb93b5e37aa83dda3bbe7eec1b/src/pages/settings/Security/TwoFactorAuth/Steps/CodesStep.tsx#L51
What changes do you think we should make in order to solve the problem?

We should change this line to use ONYXKEYS.VALIDATE_ACTION_CODE
    const [validateCodeAction] = useOnyx(ONYXKEYS.VALIDATE_ACTION_CODE);
    const hasMagicCodeBeenSent = !!validateCodeAction?.validateCodeSent;

What alternative solutions did you explore? (Optional)

It's coming from this PR here most likely: https://github.com/Expensify/App/pull/51882

Don't think it's a DB but I can handle it as a regression since the contributor can't be available

@hungvu193 Ok, let's do that if the original author can't handle it!

We can close, this was fixed by another PR.


chore(development): release 0.7.0

:robot: I have created a release beep boop
0.7.0 (2023-01-10)
Features

update parse function (#16) (4843d0e)

Bug Fixes

correction spelling (f096968)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/JonDotsoy/envuse-rust/releases/tag/v0.7.0 :sunflower:


Add VSCode configuration

Before submitting, please confirm you've

[x] read and understood our Contributing Guidelines
[x] completed Mattermost Contributor Agreement
[x] executed npm run lint:js for proper code formatting

Please provide the following information:
Summary
Add VSCode configuration. It would make easier to debug the app.
Issue link
N/A
Test Cases
Develop/Debug with VSCode.
Additional Notes
N/A

Thanks @yuya-oc is this for v4.2?

Rebased against for the current master branch. Somehow "Debug Renderer Process" is no longer working so it's commented out. But "Debug Main Process" will help us enough.


Case insensitive matching with glossary terms

Fixes #547

If you could add  to some term with a different case than the value in the glossary match so we can test this on the review app (https://dvc-org-pr-572.herokuapp.com/doc) that would be nice.
For example Output in static/docs/commands-reference/status.md

thanks, @algomaster99 !


support custom install requires for torch blade

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.bohua.cbh seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Section parser for articles

Hey, cool! Feel free to re-open this if you want me to merge it and I'll take a look.

Hi there,
I will surely re-open it soon. I've only just finished my initial
implementation so I was hoping to test it a bit more thoroughly before
submitting it for review.
Let me know if you have any questions, or if you want me to re-open the
pull request anyways.
Cheers
On Thu, Oct 15, 2015 at 12:34 PM, Tal Yarkoni<EMAIL_ADDRESS>wrote:

Hey, cool! Feel free to re-open this if you want me to merge it and I'll
take a look.
—
Reply to this email directly or view it on GitHub
https://github.com/neurosynth/ACE/pull/2#issuecomment-148498813.

No worries,  I'll just wait till you're ready. Thanks!
On Oct 15, 2015 4:11 PM, "mbelmadani"<EMAIL_ADDRESS>wrote:

Hi there,
I will surely re-open it soon. I've only just finished my initial
implementation so I was hoping to test it a bit more thoroughly before
submitting it for review.
Let me know if you have any questions, or if you want me to re-open the
pull request anyways.
Cheers
On Thu, Oct 15, 2015 at 12:34 PM, Tal Yarkoni<EMAIL_ADDRESS>wrote:

Hey, cool! Feel free to re-open this if you want me to merge it and I'll
take a look.
—
Reply to this email directly or view it on GitHub
https://github.com/neurosynth/ACE/pull/2#issuecomment-148498813.

—
Reply to this email directly or view it on GitHub
https://github.com/neurosynth/ACE/pull/2#issuecomment-148522410.

Oh, and let me know if you have questions about anything (though it looks
like you have it pretty well figured out). I apologize for the poor (well,
really nonexistent) state of the documentation.
On Oct 15, 2015 4:11 PM, "mbelmadani"<EMAIL_ADDRESS>wrote:

Hi there,
I will surely re-open it soon. I've only just finished my initial
implementation so I was hoping to test it a bit more thoroughly before
submitting it for review.
Let me know if you have any questions, or if you want me to re-open the
pull request anyways.
Cheers
On Thu, Oct 15, 2015 at 12:34 PM, Tal Yarkoni<EMAIL_ADDRESS>wrote:

Hey, cool! Feel free to re-open this if you want me to merge it and I'll
take a look.
—
Reply to this email directly or view it on GitHub
https://github.com/neurosynth/ACE/pull/2#issuecomment-148498813.

—
Reply to this email directly or view it on GitHub
https://github.com/neurosynth/ACE/pull/2#issuecomment-148522410.

Sounds good! I think I mostly just need to implement the feature for the
remaining publisher platforms/Sources, and also have the feature
configurable through the config file (I haven't looked at this part yet,
but I'm thinking of something similar to the way parse_article works.) And
of course, test it!
No need to apologize, the code, comments and examples were plenty to get me
started. Thank you for open sourcing your work!
Cheers,
On Thu, Oct 15, 2015 at 3:36 PM, Tal Yarkoni<EMAIL_ADDRESS>wrote:

Oh, and let me know if you have questions about anything (though it looks
like you have it pretty well figured out). I apologize for the poor (well,
really nonexistent) state of the documentation.
On Oct 15, 2015 4:11 PM, "mbelmadani"<EMAIL_ADDRESS>wrote:

Hi there,
I will surely re-open it soon. I've only just finished my initial
implementation so I was hoping to test it a bit more thoroughly before
submitting it for review.
Let me know if you have any questions, or if you want me to re-open the
pull request anyways.
Cheers
On Thu, Oct 15, 2015 at 12:34 PM, Tal Yarkoni<EMAIL_ADDRESS>wrote:

Hey, cool! Feel free to re-open this if you want me to merge it and
I'll
take a look.
—
Reply to this email directly or view it on GitHub
https://github.com/neurosynth/ACE/pull/2#issuecomment-148498813.

—
Reply to this email directly or view it on GitHub
https://github.com/neurosynth/ACE/pull/2#issuecomment-148522410.

—
Reply to this email directly or view it on GitHub
https://github.com/neurosynth/ACE/pull/2#issuecomment-148542623.


New package: SimonG4ELI.GeminiAmplifier version 1.2.0

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] This PR only modifies one (1) manifest
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.6 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Resolve: #138631

For moderators
This account is automated by Github Actions and the source code was created by CoolPlayLin. If you have any questions about any pull request, don't hesitate to ping Exorcism0666.

[!warning]
With the new version of Komac, numerous Pull Requests may contain bugs or unwanted elements. If such issues arise, please tag russellbanks so that these bugs can be addressed in future updates.

[!important]
Please carefully review these Pull Request before merging. If it is a Pull Request for removing incorrect content and the URLs are issue, free upon manual checking, please close this Pull Request directly. (It is best to inform Exorcism0666 of the closure.)

良い一日をお過ごしください！

Automatic Validation ended with:

2024-02-10 11:34:10.826 [CORE] DeliveryOptimization error: 0x80190190, extended error: 0x00000000 2024-02-10 11:34:10.827 [FAIL] D:\a_work\1\s\external\pkg\src\AppInstallerCommonCore\DODownloader.cpp(289)\WindowsPackageManager.dll!551957EB: (caller: 55195E1C) Exception(1) tid(1ff4) 80190190 Bad request (400). MSI (s) (3C:D8) [11:33:58:539]: Note: 1: 2228 2:  3: ServiceInstall 4: SELECT ServiceInstall,Name,DisplayName,ServiceType,StartType,ErrorControl,LoadOrderGroup,Dependencies,StartName,Password,ComponentId,Directory_,FileName,Arguments FROM ServiceInstall, Component, File WHERE ServiceInstall.Component_ = Component.Component AND (Component.KeyPath = File.File) AND (Action = 1 OR Action = 2)  MSI (s) (3C:D8) [11:33:58:852]: Windows Installer installed the product. Product Name: Microsoft Visual C++ 2013 x86 Minimum Runtime - 12.0.40664. Product Version: 12.0.40664. Product Language: 1033. Manufacturer: Microsoft Corporation. Installation success or error status: 0. MSI (s) (3C:14) [11:33:59:305]: Note: 1: 2228 2:  3: ServiceInstall 4: SELECT ServiceInstall,Name,DisplayName,ServiceType,StartType,ErrorControl,LoadOrderGroup,Dependencies,StartName,Password,ComponentId,Directory_,FileName,Arguments FROM ServiceInstall, Component, File WHERE ServiceInstall.Component_ = Component.Component AND (Component.KeyPath = File.File) AND (Action = 1 OR Action = 2)  MSI (s) (3C:14) [11:33:59:758]: Windows Installer installed the product. Product Name: Microsoft Visual C++ 2013 x86 Additional Runtime - 12.0.40664. Product Version: 12.0.40664. Product Language: 1033. Manufacturer: Microsoft Corporation. Installation success or error status: 0. [11B4:1804][2024-02-10T11:33:56]w363: Could not create system restore point, error: 0x80070422. Continuing...

(Automated response - build 720.)


expose partial in ObservableQuery.currentResult

This PR just exposes the partial property when calling ObservableQuery.currentResult.

@machard what problem does this PR solve?

@helfer
related to this PR that i am preparing for react-apollo : https://github.com/apollostack/react-apollo/compare/master...machard:review-skip-2
I will make a good explanation of what it solves when publishing to github.
to answer you in the apollo-client context, look at that for example:

you create a request with forceFetch=true noFetch=false
when calling currentResult (like it's done in react-apollo) you have loading = true => loading = false
if you refetch it (when noFetch goes to true then back to false for example) you will have again loading = true => loading = false BUT i would like to be aware that the result is already full (by exposing partial that would be false in this case). it's kind of emulating the stale property in relay https://facebook.github.io/relay/docs/guides-ready-state.html#onreadystatechange (partial=false && loading = true <=> stale = true && loading = true)

the user use case is that you don't want to redisplay a loading state when updating a query in the background (each time you go back to the screen for example). you want to display the result as soon as partial=false.

@machard okay, thanks! I'll take a look after the weekend.

@machard thanks for the PR! I can see how this could potentially be useful, and I don't think it will lead to any unnecessary re-renders, but it's a change in the external API, so I've changed the base to the 0.6 branch.

Before I can merge it, please make sure there's an entry in the changelog, and a corresponding PR to the docs where necessary. If you don't delete the PR template next time, it'll actually tell you what needs to be done :wink:

ok 'ill do that

@helfer  i updated changelog, however i am not sure what you expect me to do for the docs as it seems it gets the data from the AC source ?
https://github.com/apollostack/core-docs/blob/e20246bb4dc8ca8ea77719274e0838f43bbd08a7/source/apollo-client-api.md#L51
http://dev.apollodata.com/core/apollo-client-api.html#ApolloCurrentResult

@machard thanks!
Yes, that's right, the docs are generated. I just have to figure out how we used to do that ... 🤔
What would be very useful though is if you added docstrings to the ApolloQueryResult so that the docs will actually have an explanation for what each of the members in ApolloQueryResult is. Could you do that? Shouldn't take more than 5 minutes. 🙏

@hellfer, i was going too right now but i don't see other data structures with doc string so I am not sure what the syntax :/

@machard As far as I know you can only document functions, so for this one you should document the return value of watchQuery. Here is some more info on typedoc: http://typedoc.org/guides/doccomments/

@helfer done, and i rebased

@machard Thanks a lot, it's merged! Btw, we're planning to get rid of partial responses for 1.0, and return only valid GraphQL responses or nothing for queries.


Update da.json

as proposed in https://github.com/MichMich/MagicMirror/commit/05f0d1855c0b49823864ca571480442e43945711#commitcomment-85730050

Codecov Report

Merging #2930 (54111b4) into develop (7694d6f) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop    #2930   +/-   ##
========================================
  Coverage    63.82%   63.82%
========================================
  Files            9        9
  Lines          293      293
========================================
  Hits           187      187
  Misses         106      106

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


refactor: update files, components, directives following style-guide

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

This PR was created in order to update the repository following the  style-guide. The main changes are:

All files had their names changed (camelCase to kebab-case). Ex: 'smartTables' to 'smart-tables';
Components had their signature changed. Ex: 'App' now is 'AppComponent';
Components/directives that uses the lifecycle hook methods (OnInit, OnDestroy...)  now implements them;
Some simple changes, like including spaces between import {Component},  and also in declaraction, like: let x:number (spaces after the colon);

@nnixaa Is there any problem with this PR?


Documentation needs to be updated ...

Was experimenting with T4 text templates using Visual Studio 2022 Version 17.8.2.   Followed the documentation directions, but noted the following:
In the section entitled To create a run-time text template, the documentation states:
In the Add New Item dialog box, select Runtime Text Template.

In Visual Studio 2022 no such item seems to exist, the only option found when using the Add New Item was Text Template -- the Runtime Text Template seems to be missing from Visual Studio 2022.   Not sure if this is a bug in Visual Studio or the documentation needs to be updated with instructions on how to install the Runtime Text Template if not available.

I was able to use the Text Template and set the file's Custom Tools property to TextTemplatingFilePreprocessor as per the documentation.  However, when I compiled the program, the following build errors occurred:
Error	CS1069	The type name 'CompilerErrorCollection' could not be found in the namespace 'System.CodeDom.Compiler'. This type has been forwarded to assembly 'System.CodeDom, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cc7b13ffcd2ddd51' Consider adding a reference to that assembly.

The documentation needs to be updated to include the following additional step before compiling / using templates:

Add the System.CodeDom NuGet package to your project.

Adding the above NuGet package fixed the problem.  Hope the above will help others new to templates.

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

ID: 7a170869-23a9-96c9-d85b-372adcb19230
Version Independent ID: d900eb64-973b-78dc-47eb-799aad1ee592
Content: Run-Time Text Generation with T4 Text Templates - Visual Studio (Windows)
Content Source: docs/modeling/run-time-text-generation-with-t4-text-templates.md
Product: visual-studio-windows
Technology: vs-ide-modeling
GitHub Login: @mgoertz-msft
Microsoft Alias: mgoertz

@bdcoder2
I'm seeing the RuntimeTextTemplate option presented in Add New Item in a recent release of Visual Studio Community, without installing any Individual Components, at least when trying this with a .NET 8 Console App project.  What project type are you trying to use?


net: shell: iface: Add command to set gateway

Add a command net iface set_ipv4_gateway that allows setting IPv4 gateway for an interface from net shell.

I also thinking about putting it behind ipv4. As suggested, moved it there and added argc check

if you have the time to spare, it would be nice to do the same for the netmask.

Preferably after this change #68419

if you have the time to spare, it would be nice to do the same for the netmask.

Preferably after this change #68419

Sure, I can take a look into it after this PR is merged


Upgraded Mongo to 7.0 from 5.0

Closes #451
Summary
Updated MongoDB to 7.0.
Local Tests
Before following the update procedure, be sure to download the latest Mongo package.
To update follow this instructions:

Stop kytos
Change kytos/docker-compose.yml mongo images to 6.0 (There are 4)
From mongosh, enter db.adminCommand( { setFeatureCompatibilityVersion: "5.0" } ). This change can be checked with db.adminCommand( { getParameter: 1, featureCompatibilityVersion: 1 } )
Stop the containers that kytos creates: mongo-rs-init, mongo1, mongo2 and mongo3. To check their status enter the console command docker ps -a. Example:

CONTAINER ID   IMAGE       COMMAND                  CREATED              STATUS                          PORTS                                                      NAMES
493d530e4104   mongo:6.0   "/scripts/rs-init.sh"    About a minute ago   Exited (0) About a minute ago                                                              mongo-rs-init
ad7a9149a38c   mongo:6.0   "/usr/bin/mongod --b…"   2 minutes ago        Up About a minute               <IP_ADDRESS>:27017->27017/tcp, :::27017->27017/tcp              mongo1
fbec48992a5d   mongo:6.0   "/usr/bin/mongod --b…"   2 minutes ago        Up About a minute               27017/tcp, <IP_ADDRESS>:27018->27018/tcp, :::27018->27018/tcp   mongo2
607ee9359347   mongo:6.0   "/usr/bin/mongod --b…"   2 minutes ago        Up About a minute               27017/tcp, <IP_ADDRESS>:27019->27019/tcp, :::27019->27019/tcp   mongo3

Stop each container with docker stop $CONTAINER_ID. E.g. docker stop 493d530e4104.

Run Kytos to create containers with mongo 6.0
Close Kytos
To mongosh, enter db.adminCommand( { setFeatureCompatibilityVersion: "6.0" } )
Stop the containers that kytos creates
Change kytos/docker-compose.yml mongo images to 7.0
Run Kytos. The result can be verified with mongosh with the command mongosh mongo1:27017,mongo2:27018,mongo3:27019.

Current Mongosh Log ID: 6632a69a07964f2a047b2da8
Connecting to:          mongodb://<IP_ADDRESS>:27017/mongo1%3A27017%2Cmongo2%3A27018%2Cmongo3%3A27019?directConnection=true&serverSelectionTimeoutMS=2000&appName=mongosh+2.2.2
Using MongoDB:          7.0.8  # <-- HERE
Using Mongosh:          2.2.2
mongosh 2.2.5 is available for download: https://www.mongodb.com/try/download/shell

Discrepancies:

If there is any error while running Kytos after updating docker-compose.yml and stopping the containers. Try deleting the containers with docker rm $CONTAINER_ID. Usually there is a warning stating that the container of mongo1 is already in use.
Sometimes the port 27017 will be in use when running Kytos. This port needs to be free. For Linux this command sudo fuser -k 27017/tcp worked.

End-to-End Tests
The e2e tests depend on this PR
+ python3 -m pytest tests/ --reruns 2 -r fEr
============================= test session starts ==============================
platform linux -- Python 3.11.2, pytest-8.1.1, pluggy-1.5.0
rootdir: /tests
plugins: rerunfailures-13.0, timeout-2.2.0, anyio-4.3.0
collected 263 items

tests/test_e2e_01_kytos_startup.py ..                                    [  0%]
tests/test_e2e_05_topology.py ..................                         [  7%]
tests/test_e2e_06_topology.py ....                                       [  9%]
tests/test_e2e_10_mef_eline.py ..........ss.....x.....x................  [ 24%]
tests/test_e2e_11_mef_eline.py ......                                    [ 26%]
tests/test_e2e_12_mef_eline.py .....Xx.                                  [ 29%]
tests/test_e2e_13_mef_eline.py ....Xs.s.....Xs.s.XXxX.xxxx..X........... [ 45%]
.                                                                        [ 45%]
tests/test_e2e_14_mef_eline.py x                                         [ 46%]
tests/test_e2e_15_mef_eline.py .....                                     [ 47%]
tests/test_e2e_16_mef_eline.py .                                         [ 48%]
tests/test_e2e_20_flow_manager.py .....................                  [ 56%]
tests/test_e2e_21_flow_manager.py ...                                    [ 57%]
tests/test_e2e_22_flow_manager.py ...............                        [ 63%]
tests/test_e2e_23_flow_manager.py ..............                         [ 68%]
tests/test_e2e_30_of_lldp.py ....                                        [ 69%]
tests/test_e2e_31_of_lldp.py ...                                         [ 71%]
tests/test_e2e_32_of_lldp.py ...                                         [ 72%]
tests/test_e2e_40_sdntrace.py ..............                             [ 77%]
tests/test_e2e_41_kytos_auth.py ........                                 [ 80%]
tests/test_e2e_42_sdntrace.py ..                                         [ 81%]
tests/test_e2e_50_maintenance.py ............................            [ 92%]
tests/test_e2e_60_of_multi_table.py .....                                [ 93%]
tests/test_e2e_70_kytos_stats.py ........                                [ 96%]
tests/test_e2e_80_pathfinder.py ss......                                 [100%]

Hi team,
Great work, Aldo! very nice done!

Excellent to see e2e passing. On GitLab https://gitlab.ampath.net/kytos/kytos-end-to-end-tester/-/blob/master/.gitlab-ci.yml needs to be updated too, check out the services there on 5.0. We also need to decide on a strategy, if we'll upgrade there to 7.0, and then upgrade amlight/kytos-end-to-end-tester on 7.0 to be able to run e2e on demand, but then we can maybe also create a new repo copy of amlight/kytos-end-to-end-tester to still have it on 5.0 just so if any patch on 2024.1 still needs to be patched then we can also test on 5.0 in the CI? and then we maintain the 5.0 there until prod upgrades to 7.0 when they deploy the future version 2024.2 probably by the end of the year? Let me know if you have any other suggestions. In practice, no expected issues to also test future patches of 2024.1 with 7.0, but for completeness let's be ready for that case too. This also needs to include @italovalcy in the conversation.

Another approach would be having both tests on the end-to-end daily execution (5.0 and 7.0), just to make sure no surprises will be detected.

Good idea having two parallel pipelines on end-to-end project nightly on 7.0 and on 5.0. Let's also consider this, if the available GitLab workers can pick up the work in parallel, and then we maintain it until prod gets upgraded to 7.0.

Test description:

Created 100 EVCs.
60 disabling/enabling were done. 40 seconds between each other.
From mef_eline interval and timings were reduced to 30.
Ringtopo topology was used.

Overall Results
Timing measure for bulk_write

Database created and tests performed in mongo 5.0:
Overview

Trace

Request POST flow_manager install flows

Request POST flow_manager delete flows

Database created and tests performed in mongo 7.0:
Overview

Trace

Request POST flow_manager install flows

Request POST flow_manager delete flows

Database created in mongo 5.0 and tests performed in mongo 7.0:
Overview

Trace

Request POST flow_manager install flows

Request POST flow_manager delete flows

Great to see that overall the DB ops latencies are similar as we'd expect, that's a good start. On flow_manager delete if you could also include in the screenshot how long the DB operation was taking (on 5.0 you included that correctly, but that was truncated on 7.0 screenshot):

I'm just waiting now the the rest of the points on this comment to be addressed to approve it

@Alopalao if you could also post the output of vegeta for one of the endpoint stress tests for the record on both Mongo 5 and 7 results that'd be great, vegeta output is very helpful since it includes many stats including some percentiles. So, that of couse it's only from a client point of view, but if the overall latencies were similar, that's already a great sign

Vegeta report results from jq -ncM '{method: "POST", url: "http://localhost:8181/api/kytos/flow_manager/v2/flows/00:00:00:00:00:00:00:01", body: { "force": true, "flows": [ { "priority": 10, "match": { "in_port": 1, "dl_vlan": 100 }, "actions": [ { "action_type": "output", "port": 1 } ] } ] } | @base64, header: {"Content-Type": ["application/json"]}}' | vegeta attack -format=json -rate=200/1s -duration=60s -timeout=120s | tee results.bin | vegeta report
Mongo 7.0
Requests      [total, rate, throughput]         12000, 200.02, 148.80
Duration      [total, attack, wait]             1m1s, 59.994s, 603.263ms
Latencies     [min, mean, 50, 90, 95, 99, max]  10.2ms, 507.339ms, 519.413ms, 773.648ms, 848.971ms, 996.941ms, 1.239s
Bytes In      [total, mean]                     381289, 31.77
Bytes Out     [total, mean]                     1464000, 122.00
Success       [ratio]                           75.14%
Status Codes  [code:count]                      202:9017  503:2983
Error Set:
503 Service Unavailable

Mongo 5.0
Requests      [total, rate, throughput]         12000, 200.02, 143.44
Duration      [total, attack, wait]             1m1s, 59.995s, 692.008ms
Latencies     [min, mean, 50, 90, 95, 99, max]  7.965ms, 509.928ms, 483.188ms, 831.703ms, 906.246ms, 1.101s, 1.267s
Bytes In      [total, mean]                     375985, 31.33
Bytes Out     [total, mean]                     1464000, 122.00
Success       [ratio]                           72.54%
Status Codes  [code:count]                      202:8705  503:3295
Error Set:
503 Service Unavailable

The error from both tests come from concurrency limit:
2024-05-13 12:38:44,102 - WARNING [uvicorn.error] (MainThread) Exceeded concurrency limit.


Feature/add documentation

Adding documentation and comments to the codebase

ping @edenspiekermann/backend

Thanks @zfhui, & @martzoukos :)
Documentation is a first class citizen in Elixir - you don't need to generate documentation from it, it's becomes part of the code itself. https://hexdocs.pm/elixir/writing-documentation.html


Admin prepopulated_fields for slug not working with ChainedForeignKey

I'm using django 1.9.5 with django-smart-selects 1.2.6
I have a field in my model that uses ChainedForeignKey and that model also has a SlugField. Unfortunately, with the ChainedForeignKey, the prepopulated_fields doesn't work for SlugField in Admin.
I'm getting this error in my browser console:
Uncaught TypeError: $(...).data(...).prepopulate is not a function
$(field.id).data('dependency_list', field['dependency_list'])                .prepopulate(field['dependency_ids'], field.maxLength, field.allowUnicode);
Commenting out the ChainedForeignKey field fixes this problem.

Nevermind, I updated Django to the latest version: 1.10.4 and it now works flawlessly.


[ZEPPELIN-2438]: Add local winutils.exe in build step and use if required …

What is this PR for?
Add local winutils.exe in build and use as temporary HADOOP_HOME if required on run (windows only).
The build of the spark dependencies now downloads winutils.exe and installs it in interpreter/spark/winutils.
The zeppelin-env.cmd (only used if we run on windows) then checks if HADOOP_HOME exists, if not, it will set it to point to winutils. This is required to have the spark interpreter work in Windows.
In order to have this active, the zeppelin-env.cmd had to be removed from the .gitignore.
What type of PR is it?
[Bug Fix | Improvement]
Without this fix, the binary distribution or a local build of Zeppelin will have the spark interpreter (default interpreter) fail on a Windows platform (unless the user follows additional / manual installation steps or already has a local HADOOP). This will frustrate users trying to checkout Zeppelin.
For users having HADOOP or macOS/Linux users, the chance will have no effect.
Todos
What is the Jira issue?
https://issues.apache.org/jira/browse/ZEPPELIN-2438
How should this be tested?

Non-Regression Test
Standard Unit Test
Run Zeppelin without additional installations (except Java) on Windows

Screenshots (if appropriate)
N/A
Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

I think it's intentional that we ship a template of conf/zeppelin-env.cmd.template?

Again, I'm not sure about releasing with a binary embedded in resource and how that correspond to the Apache release process.
But, I understand where you are coming up, and if you have a way to bootstrap the download only on the Windows platform, we could take a look at that?

@felixcheung I don't know if it would be an issue embedding and .exe binary in the binary distribution. If that is the case, the either
a) default-dependency on hadoop should be removed
or
b) documentation should be added and - if possible - a dialog prompting the used for the download (s.th. "This Spark interpreter is pre-configures for Hadoop and requires winutils.exe. ....")

(close and reopen to kick Jenkins to recognise Travis)


Volume source allowed to be relative path

docker compose up works with volumes paths that are relative to the compose file; this PR will fix dockerode-compose to support relative paths.

Merged https://github.com/apocas/dockerode-compose/pull/19 instead.


Error when installing 'Python' extension. Please check the log for more details.

Environment data

VS Code version: VSCodeUserSetup-x64-1.60.0
Extension version (available under the Extensions sidebar): Python v2021.9.1191016588
OS and version: Windows 10
Python version (& distribution if applicable, e.g. Anaconda): Miniconda python 3.9
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): miniconda
Relevant/affected Python packages and their versions: Python 3.9.5 in windows
Relevant/affected Python-related VS Code extensions and their versions: None
Value of the python.languageServer setting: English? I didn't change it.

[NOTE: If you suspect that your issue is related to the Microsoft Python Language Server (python.languageServer: 'Microsoft'), please download our new language server Pylance from the VS Code marketplace to see if that fixes your issue]
Expected behaviour
Should allow me to install Python extension in VS code. Then I can configure the Python interceptor to run the python file with miniconda.
Actual behaviour
When I install Python extension in VS code, show the message: "Error when installing 'Python' extension. Please check the log for more details.". The log is saying below, and even I download "ms-python.python-2021.9.1191016588 (1).vsix" and manually install from VISX it's not working :
[2021-09-08 11:13:54.276] [renderer1] [error] extension/package.json not found inside zip.,extension/package.json not found inside zip.: unknown: extension/package.json not found inside zip.,extension/package.json not found inside zip.
at vscode-file://vscode-app/c:/Users/sshi126/Microsoft%20VS%20Code/resources/app/out/vs/code/electron-browser/sharedProcess/sharedProcessMain.js:48:17341
at Array.reduce ()
at g (vscode-file://vscode-app/c:/Users/sshi126/Microsoft%20VS%20Code/resources/app/out/vs/code/electron-browser/sharedProcess/sharedProcessMain.js:48:17327)
at O.joinAllSettled (vscode-file://vscode-app/c:/Users/sshi126/Microsoft%20VS%20Code/resources/app/out/vs/code/electron-browser/sharedProcess/sharedProcessMain.js:48:9895)
at async O.installExtension (vscode-file://vscode-app/c:/Users/sshi126/Microsoft%20VS%20Code/resources/app/out/vs/code/electron-browser/sharedProcess/sharedProcessMain.js:48:7440)
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

Open VS code
Install Python extension

Logs

@legend03312000 I verified again, it seems to be installing fine. Can you verify that the SHA1 checksum is d137b2f03ddf455134e93b38ca6e4bc806753ff5 for your download? You can also rename the vsix to zip and open it up to see if it has package.json. As far as I can tell, it seems to install fine for me, and it seems to have the file:

I have a similar problem trying to install the extension on one on a mac M1. BigSur 11.4 VSCODE Version: 1.60.0

@205mdp Can you verify the hash of the file (see the one I have posted in my previous comment). They should match. If they match and this continues to fail let us know. I used sha1sum to command to get the hash.


Changing of namespace not allowed

First, thank you very much for sharing with us your awesome tool.
I have an issue using kubens for changing my namespace
Demo :
INFO: k8s-vault session started

❯ kubens myns
error: open /home/myuser/.kube/k8s-vault-kubeconfig-1101772.yaml: permission denied
❯ ll /home/myuser/.kube/k8s-vault-kubeconfig-1101772.yaml
.r-------- myuser myuser 974 B Wed Dec  7 13:59:08 2022  /home/myuser/.kube/k8s-vault-kubeconfig-1101772.yaml

I suspect kubens wants to update the default context in the YAML k8s-vault generated..

give the v0.4.3 a try

Fixed quickly. Thanks a lot. I'll try the new release tomorrow indeed !

I confirm, It's working right now.
WARNING: k8s-vault session terminated
❯ kbx myns
Switched to context "myns".


[issue/62] fix CommonJs file extension

https://github.com/NickMcBurney/storybook-vue3-router/issues/62

@NickMcBurney Any chance of getting this merged and having a new release? It doesn't seem possible to use stories in tests without this PR and it seems a pretty simple PR.

For now I published storybook-vue3-router-fixed to the npm repo and added this to my package.json:
  "pnpm": {
    "overrides": {
      "storybook-vue3-router": "npm:storybook-vue3-router-fixed"
    }
  },


New update error

New update
Bash not support (" no color") 😅

Syntax is turned on?
set syntax on

How did you install micro? I did not use package manager, pacman in my case, just downloaded  and unpacked static build archive. The shell syntax highlighting works.

Same here, theme is not working

I am termux user so I use pkg install micro

I am termux user so I use pkg install micro

They fixed it, use pkg update && pkg upgrade to fix

I am termux user so I use pkg install micro

They fixed it, use pkg update && pkg upgrade to fix

Not working

I am termux user so I use pkg install micro

They fixed it, use pkg update && pkg upgrade to fix

Not working

There's no way it won't work. 'pkg upgrade micro'

@dmaluka @JoeKar for close?

Looks outdated and wasn't updated for more than one year now.
In case the problem still exists please trigger this issue again.


Increment version for dns releases

Increment package version after release of Azure.ResourceManager.Dns

API change check
API changes are not detected in this pull request.

/check-enforcer override

Failure is unrelated to these changes; merging.


Renderer messaging: do not use generics

Testing #125016
Currently in the renderer messaging api we use <TSend = any, TReceive = TSend>. I suggest to instead simply use any since we already use that in other messaging api we have: debug protocol and for the WebView.

I think the generics make sense in this case. Having them lets you optionally enforce typings on the messaging object
Webviews use any because postMessage belongs to the entire WebView object instead of a dedicated messaging class like we have here


Move Bot-Full into main

Merge by necessity -- bot-full represents the current bot state anyways

merge by de-facto main


Support animations for glTF/GLB format

I create ModelRenderable with glb model file that contains a few skeleton animations. But animationDataCount is zero. Do you plan to support animations for glTF/GLB formats?

Right now we only have support for animations via FBX imported to SFB and glTF/GLB only support static assets (whether loaded at runtime or imported into an sfb).  At this point we don't have any new animation formats to announce, but I will mark this as a feature request to track this thanks!

Thanks! It will be a very useful feature!

Please implement the Animations from glb/gltf! 😎

GLTF animations would be very helpful!

@tpsiaki are there any workarounds possible? E.g. Filament has a gltf loader that supports animation, is it possible to use it directly? Possibly via a custom version of Filament (as in #329)

@jfaust Do you found a way to play animated gltf models?

@trikolasu Not yet.

How is the model-viewer doing that? Go to https://www.google.com/search?q=panda on your android
Just give us that :D

Please implement the Animations from glb/gltf.
Or create an easy and stable online converter without need in Android Studio. Its not a job for developers to convert hundreds of objects, and marketing people just cannot use the existing converter properly.

Hi there:
Truly hoping this support is added to the next Android SDK release, as ARCore web-component uses it already from several previous versions (I suppose that using a port of the aforementioned Filament existing functionality).

Hi there:
Any advice about the way to to implement/hack within Scenform by ourselves the already available ### Filament Engine GLTF/GLB functionality .
BTW, it's me or seems very strange not being to be possible to use this (basic) functionality in ARCore/Sceneform, considering that:

GLTF/GLB and USDZ are the formats to go in the AR/VR playground;
ARCore is a technology by Google and this one had a revenue along 2018 of 136224M$, so they have enough cash for hire a few decent developers here and there, in order to have this basic functionality implemented;
Sceneform uses Filament and Filament has it already implemented (see previous link), both technologies of Google;
as remarked before by arifullahjan, ARCore for Web (via model-viewer web component) already implements it.

The lacks of this and others basic technologies make me think that Google prefer (as by example Vuforia does) that people use a third party game engine (Unity and Unreal) in order to tackle AR projects instead the good native-way. Do not misunderstand me, I have not problem with that focus when in fact a "game" is being developed, but I think that's a complete (and bloating) overkill in many other scenarios.
Thanks for your time.
Best regards.

Hi again dear folks:
After writing the previous post, I've just seen in the Filament's GitHub repository that, with date 07/09/19, a GLTF/GLB animation implementation has been added to the engine (by the guy in charge of it: Philip Rideout).
This way I think is only a question of time that this functionality is propaged to the ### Android Studio's Sceneform plugin and so to ARCore.
In the meanwhile, I wonder whether a way to implement new-Filament-engine-in-new-Sceneform- engine is suitable, without the use of that plugin.
Best regards.

+1
I suggest to focus to implement this feature to simplify the developer work...

Hi everybody:
As a result of the fix in implementation of GLTF/GLB on Filament, I think now there are not excuses for finally incorporate GLTF/GLB animations to the native ARCore/Sceneform functionality.
Best regards.

@tpsiaki Still not working on Scenefrom 1.14 ... c'mon guys :(

So weird it is still not working in 1.14... maybe for 1.15?

Hi I'm also in requirement of this feature ASAP.
I'm trying to load animated 3d objects at runtime. But animation is only supported in .obj which in turn can't be loaded at runtime as it needs to be converted into sfb. On the other hand runtime supportive gltf/glb formats can't have animation. Such a vicious circle. Please guys implement this ASAP!

1.15 still not implement animated GLTF/GLB functionality.
Supporting all the work on the oldie FBX format is a real bottleneck in each serious pipeline. Very disappointing for the CG artist!
C'mon guys!

How to load animation 3d object in-app by HTTP URL?    Is AR core support animated object in runtime by using URL assets of 3d object if yes, then which format I should use?
Thanks

Finalliy 1.16 release support animated GLTF/GLB by means of gltfio library and open access to Filament APIs.
Hallelujah!!


test cases for the service account

test cases for the service account for the node observability operator

/lgtm - if you don't mind fixing the nit: Josef pointed out

if you don't mind fixing the nit: Josef pointed out

sure, fixed.

/lgtm
/approve


Dexter's Attempt at Completing the Beginning Git Video Course

Hi my name is Dexter and I enjoyed the course Beginning Git! Sam is a cool guy!

Hey @Berend27—congrats on submitting a PR at the end of the git course. I'm hope that you found the course valuable, and that you were able to follow along easily and learn lots.
If you weren't already using git for your code, hopefully this has inspired you to do so. I hope that you enjoy the "Mastering Git" video course as well—you'll soon be able to get git to bend to your will :)
You can now wear this #GitWarrior badge with pride—and Tweet your historic victory!

sam


Github Fine-grained Personal Access Tokens

Our organization has recently prohibited the use of "classic" personal access tokens and we've so far been unable to figure out the right set of permissions for the new "Fined-grained" personal access tokens that will allow Release Please to run successfully.
Here's our latest attempt:

Which fails with
 Found pull request #2064: 'chore(main): release 10.20.0'
✔ Building release for path: .
❯ type: go
❯ targetBranch: main
✔ Creating 1 releases for pull #2064
Error: release-please failed: Resource not accessible by personal access token

If anyone is using a fine-grained token, could you please share the set of permissions your token has?

We apparently were able to stumble into the right set of permissions and now the Release Please action is working. I'm sure this token has way more permission than it needs, but it's working now so I'm not going to mess with it...
If anyone does have specific guidance they can offer, it would be appreciated.


bump support lib to v27

minor version bump
(interesting that android studio stable bumped to com.android.tools.build:gradle:3.1.0-alpha01)

Thanks for the heads up.
I'll be closing this because it was easier to add the required changes locally rather than merging the PR.
I've got one additional feature I'd like to finish, but if that doesn't progress fast enough I'll try to make a new release this week.


Use Aurelia-UX(theme) within skeleton-esnext-webpack got a error

Hello friends, i using the newest skeleton-esnext-webpack and  aurelia-ux, but got a error when build:
ERROR in ./src/app.html
Module not found: Error: Can't resolve 'common/styles/common-ux.css#ux' in '/Users/xujin/My_Projects/NodeJS/skeleton-esnext-webpack/src'
 @ ./src/app.html
 @ ./src/app.js
 @ ./src/main.js
 @ ./~/aurelia-webpack-plugin/dist/aurelia-entry.js
 @ multi aurelia-webpack-plugin/dist/aurelia-entry aurelia-polyfills aurelia-loader-webpack (webpack)-dev-server/client?http://localhost:8080 aurelia-bootstrapper

I try to rename common-ux.css to common-ux.css#ux, but got another error:
ERROR in ./src/common/styles/common-ux.css#ux
Module build failed: Unknown word (16:23)

  14 | }
  15 | styles.primary {
> 16 |   background-color: ${$design.primary};
     |                       ^
  17 | }
  18 | styles.left_font_color {
  19 |   color: ${$design.left_font_color};

 @ ./src/app.html
 @ ./src/app.js
 @ ./src/main.js
 @ ./~/aurelia-webpack-plugin/dist/aurelia-entry.js
 @ multi aurelia-webpack-plugin/dist/aurelia-entry aurelia-polyfills aurelia-loader-webpack (webpack)-dev-server/client?http://localhost:8080 aurelia-bootstrapper

main.js fragment
import 'babel-polyfill';
import * as Bluebird from 'bluebird';
import i18nextBackend from 'i18next-xhr-backend';
import moment from 'moment';
import {AppRouter} from 'aurelia-router';

Bluebird.config({ warnings: { wForgottenReturn: false } });

export async function configure(aurelia) {
    aurelia.use
        .standardConfiguration()
        .developmentLogging()
        .plugin(PLATFORM.moduleName('aurelia-animator-css'))
        .plugin(PLATFORM.moduleName('aurelia-ux'))
        .feature(PLATFORM.moduleName('common'));
        ......

Node: 6.11.0
aurelia-ux: 0.2.0
skeleton-esnext-webpack: the master branch

i guess this is already known, it currently just doesn't work, webpack gathers all .css files, but the dynamic css is not valid css...  so it fails on compiling them, and the whole build fails.
see https://github.com/aurelia/ux/issues/50

You need webpack to treat it as text I imagine. I'm not a webpack guru though.

Have you found a solution? :(

Hello,
I found a solution :)
in webpack.config.js

add ModuleDependenciesPlugin dependencies

const { AureliaPlugin } = require('aurelia-webpack-plugin');
replace with
const { AureliaPlugin, ModuleDependenciesPlugin } = require('aurelia-webpack-plugin');

replace these rules

{         test: /\.css$/i,         issuer: [{ not: [{ test: /\.html$/i }] }],         use: extractCss ? ExtractTextPlugin.extract({           fallback: 'style-loader',           use: cssRules,         }) : ['style-loader', ...cssRules]       },       {         test: /\.css$/i,         issuer: [{ test: /\.html$/i }],         // CSS required in templates cannot be extracted safely         // because Aurelia would try to require it again in runtime         use: cssRules       },
with
{         test: /\.css$/i,         issuer: [{ not: [{ test: /\.html$/i }] }],         use: extractCss ? ExtractTextPlugin.extract({           fallback: 'style-loader',           use: cssRules,         }) : ['style-loader', ...cssRules],         exclude: /aurelia-ux/       },       {         test: /\.css$/i,         issuer: [{ test: /\.html$/i }],         // CSS required in templates cannot be extracted safely         // because Aurelia would try to require it again in runtime         use: cssRules,         exclude: /aurelia-ux/       },

add these rules

{         test: /\.css$/i,           loader: 'raw-loader',         include: /aurelia-ux/,       },

add in ModuleDependenciesPlugin aurelia-ux dependencies

.... plugins: [     new AureliaPlugin({       includeSubModules: [{ moduleId: "aurelia-ux" }]     }),     new ModuleDependenciesPlugin({       'aurelia-ux': [         'aurelia-ux/button/ux-button-theme.css',         'aurelia-ux/input/ux-input-theme.css',         'aurelia-ux/form/ux-form-theme.css',         'aurelia-ux/form/ux-field-theme.css',         'aurelia-ux/button/ux-button-theme.css',         'aurelia-ux/checkbox/ux-checkbox-theme.css',         'aurelia-ux/chip-input/ux-chip-theme.css',         'aurelia-ux/chip-input/ux-chip-input-theme.css',         'aurelia-ux/chip-input/ux-tag-theme.css',         'aurelia-ux/icons/ux-icon-theme.css',         'aurelia-ux/input-info/ux-input-info-theme.css',         'aurelia-ux/textarea/ux-textarea-theme.css'       ]     }),     new ProvidePlugin({ ....

and magic :D works

@mrazvan92 works for me on latest webpack-typescript skeleton, just had to npm install raw-loader (used in the solution). thx!
...if someone else tries this out now: i had this related issue https://github.com/aurelia/validation/issues/445#issuecomment-305650958
...deleting contents of the aurelia-ux/node_modules folder solved it

@mrazvan92 Thanks! This works for me too on a webpack-based Aurelia CLI project. :)

@jemhuntr or you can check https://t.co/ajjHO9N1EL


Why are some document titles missing?

Thank you for the amazing repo.
I am curious why are some titles missing from the tfidf index. It seems that during evaluation we get multiple such warnings:
Oranjegekte_0 is missing
James Gunn_0 is missing
..

I assume this means that some document titles are not found in the database. Is that normal? could you explain?
Thanks!

Hi, sorry for my late response! Could you share the command you are running and in which dataset you have that issue?
I think I have seen the same issue when the Wikipedia title (id) cannot be matched with any of the ids in the database. In particular,

the code cannot handle well some Unicode characters
the Wikipedia entity titles have been changed or directed to the new one

Thanks for the response. This happens with HotpotQA when I run the following command or similar commands.
python run_graph_retriever.py \
        --task hotpot_open \
        --bert_model bert-base-uncased --do_lower_case \
        --dev_file_path path/to/hotpotqa/dev \
        --output_dir path/to/output \
        --model_suffix 3\
        --max_para_num 10 \
        --tfidf_limit 50 \
        --beam 4\
        --eval_chunk 200 \
        --eval_batch_size 64 \
        --split_chunk 1000\
        --pruning_by_links \
        --example_limit 128

I think the main issue is that some titles are retrieved by the tfidf retriever, but when trying to retrieve their content using tfidf_retriever.load_abstract_para_text(), it outputs this warning for some documents. Not sure if this could affect the performance of the model, though. I was able to reproduce your results with the warning happening many times.