[FEATURE] Add Custom Error-Page

Description
Sveltekit is able to display custom Error-Pages, we should create one.
https://kit.svelte.dev/docs/layouts#error-pages
Screenshots
No response
Additional information
No response

i would like to solve this issue.

Sure 🏀 🚀

Hey @bhaskar2840 just checking in do you need any help

Hi, just for sharing the documentation link which is updated for Svelete custom error page : https://kit.svelte.dev/docs/routing#error

Hey @Cahllagerfeld i would love to work on this issue. Also what kind of error page we are looking to create, some insight will be useful.

@Cahllagerfeld is this issue still open ??

I think we should have an error page, in case a 404 occurs, or something with the data-load goes wrong for example. For now I would just display a page with a status code. What do you think?
@bhaskar2840 are you still working on this? There hasn't been much activity on this issue, so in case you want to pick it, I can assign it to you 👍

@Cahllagerfeld sure i want to work on this.

@Cahllagerfeld may be we can provide some useful links (/home, /docs) along with 404 status code. I am also thinking to add some animated gif.

@Cahllagerfeld Can i solve this isssue??

I think it was closed with #299. closing this issue because of that 👍 feel free to pick another issue :)


How to visualize the sampling point cloud

I find that in the end of IA-SSD backbone.py, there are following codes
if self.model_cfg.SA_CONFIG.get('SAVE_SAMPLE_LIST',False) and not self.training:
import numpy as np
        result_dir = np.load('/home/qian/tmp.npy', allow_pickle=True)
        for i in range(batch_size)  :
            # i=0
            # point_saved_path = '/home/yifan/tmp'
            point_saved_path = result_dir / 'sample_list_save'
            os.makedirs(point_saved_path, exist_ok=True)
            idx = batch_dict['frame_id'][i]
            xyz_list = []
            for sa_xyz in encoder_xyz:
                xyz_list.append(sa_xyz[i].cpu().numpy())
            if '/' in idx: # Kitti_tracking
                sample_xyz = point_saved_path / idx.split('/')[0] / ('sample_list_' + ('%s' % idx.split('/')[1]))

                os.makedirs(point_saved_path / idx.split('/')[0], exist_ok=True)

            else:
                sample_xyz = point_saved_path / ('sample_list_' + ('%s' % idx))

            np.save(str(sample_xyz), xyz_list)
            # np.save(str(new_file), point_new.detach().cpu().numpy())

I think it is something about sampling points, so the tmp.npy is a np file? Should  I build an empty tmp.npy ?
How can i use this for the sampling points for visualization?
Looking forward to your reply.

Yes, the code segment here is used to verify the sampling effect, and the tmp.npy is just a record file for output path info.
You can just give it a valid address in your own style，or if you want to make it effective, please add code segment follows in def eval_one_epoch(cfg, model, dataloader, epoch_id, logger, dist_test=False, save_to_file=False, result_dir=None):
np.save('/home/yifan/tmp', result_dir)
Best wishes!


[GitLab] Push & Merge events not working properly.

Hello,
I've set up 3 statics angular 5 app with init --manual.
The first time, for each repo, everything went A-OK, I've even written a small medium post because it was not documented and I had to email the support (https://medium.com/captain-data-blog/how-to-deploy-any-git-repository-to-netlify-1a553b8e8e39).
However now whenever I push or merge, it just doesn't trigger the web hook.
When I try the web hook directly within GitLab, "push" events seems fine and I receive a 204, but I get a 422 when trying a "merge" event.
When I push to master, it doesn't trigger the web hook and I need to init --manual again ... which is really annoying :)
Any thoughts ?

Hmm, haven't heard of that behavior before and it shouldn't be the norm, @skwidy , so it's worth debugging for sure!  Could you tell us:

are you using gitlab.com or self-hosted gitlab?
is there any "interesting" ownership/permission setup on your repo (a sub-group of a group that is owned by an organization)
what does the webhook request/response say when you examine the individual requests after choosing to edit the webhook in your repo settings?
which site or sites is it affecting?

If you get to the point where you'd be chopping out portions of your response for privacy, it is probably saner to debug this in a support ticket than in this thread.

self-hosted on DigitalOcean

the ssh key generated during the init was setup in my profile settings and I'm the general admin/owner of the GitLab so I'd say there should be no problem on this side

You mean when I do this ? :

I'll think about it yep :)

It appears the SSH Keys were not added to the web hooks for some reason ...
I closed the issued :)


Passing Environment Variables to

I'm having issues trying to pass environment variables (API Keys, etc) into angular. I've followed the recommendations in the documentation but Zone.current.get("PROPERTY"); is undefined both on the server and client-side. Here's my code:
Index.cshtml
... <app asp-prerender-module="ClientApp/dist/main-server" asp-prerender-data="new {         secretURL = @Environment.GetEnvironmentVariable("secretURL "),         secretKey = @Environment.GetEnvironmentVariable("secretKey ")         }">Preparing your content...</app> ...
boot-server.ts
... export default function (params: any) : Promise<{ html: string, globals?: any }> {     return new Promise((resolve, reject) => {         const requestZone = Zone.current.fork({             name: 'angular-universal request',             properties: {                 baseUrl: '/',                 requestUrl: params.url,                 originUrl: params.origin,                 preboot: false,                 specialKey: params.data.specialKey,                 specialURL: params.data.specialURL,                 document: '<!DOCTYPE html><app></app>'             },             onHandleError: (parentZone, currentZone, targetZone, error) => {                 reject(error);                 return true;             }         });         return requestZone.run<Promise<string>>(() => platform.serializeModule(AppModule)).then(html => {             resolve({                 html: html             });         }, reject);     }); } ...
app.module
... export class AppModule {     constructor() {         console.log(Zone.current.get("specialURL")); // undefined     } } ...
Alternatively, I've tried using webpack's DefinePlugin but was still unsuccessful in passing the properties to my application. The webpack code is below.
webpack.config.js
... var DefinePlugin = require('webpack/lib/DefinePlugin'); ... var clientBundleConfig = merge(sharedConfig, {     entry: { 'main-client': './ClientApp/boot-client.ts' },     output: { path: path.join(__dirname, clientBundleOutputDir) },     plugins: [         new webpack.DllReferencePlugin({             context: __dirname,             manifest: require('./wwwroot/dist/vendor-manifest.json')         }),         new webpack.DefinePlugin({             'process.env': {                 'SECRET_KEY': JSON.stringify('SecretKey')             }         })     ].concat(isDevBuild ? [ ...

Your code is mostly correct, but there are some inconsistencies that prevent it from working.
In your aspnet-prerender-data attribute, your values are called secretURL and secretKey, but in your boot.ts file, you're trying to read them as specialURL and specialKey.
Also in your GetEnvironmentVariable calls, you have trailing spaces (e.g., @Environment.GetEnvironmentVariable("secretURL ")) which might also prevent it from working.
Finally, when I tried, I had to use single quotes around the aspnet-prerender-data attribute value to avoid clashes with the double quotes inside it.
Having resolved all these, you should get your data passed through during server-side prerendering. Note that the same data won't be available during client-side rendering, because there's no logic in boot-client.ts to put it here (or at least you didn't imply that there is from your post).

Thanks for the reply, Steve. Sorry for the typos... those were added in place of the real values. The issues you mentioned (single quotes, naming consistency, space) are not present in my actual code. My question is actually with regard to adding logic to pass those variables during client-side rendering.

I would be interested in hearing this too: how to pass environment variables to the client side renderer, if possible at all.

I'm interesting too. Redux allows us to configure different files in function of the current environment (dev or production, example here: configureStore redux)
Can we access to the  $Env:ASPNETCORE_ENVIRONMENT in the client side, like process.env.NODE_ENV in NodeJs ?


Add UI to view DataLogEntry's metadata

WPILib's DataLogEntry objects have a Metadata property, which allows a String (usually JSON formatted) to be set as an entry's metadata. This can be set through the constructor or the setMetadata(String metadata) method. Currently, (to my knowledge), there is no way to view this data in AdvantageScope. WPI's DataLogTool has a column for this, and this is currently the only way to view it.
Describe the solution you'd like
I would like a tab, or other menu, that allows users to view the String contents of the metadata field of log data. Some of my logged data has additional details (such as unit) that would make it easier to go through logs if it could be easily viewed or referenced.

Another small enhancement is to syntax highlight the text if it's valid JSON.

Hopefully this addresses your use case. The metadata will now be shown when hovering on the field in the sidebar, and if the metadata is valid JSON it will be displayed in this key-value format.


Ensure that enable_tags and disable_tags options can be passed in a world SDF file to the gazebo_yarp_robotinterface of an included model

Similar to https://github.com/robotology/gazebo-yarp-plugins/issues/672, but for gz-sim-yarp-plugins .
As we do not have any gazebo_yarp_configurationoverride-like plugin in gz-sim-yarp-plugins, probably we first need to solve this before this one, see https://github.com/robotology/gz-sim-yarp-plugins/issues/159 that is kind of related (even if focusing just on initial joint positions).

@traversaro do you mean that having a gazebo_yarp_configurationoverride-like plugin is prioritary with respect to this? Do I have understood correctly?

Ok clear. After an initial investigation I will report what I've found and then we can decide how to proceed.


[$500] Avatar on Global Nav is cropped for users with a status

Version Number: v1.3.80-0
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail: N/A
Email or phone of affected tester (no customers): N/A
Logs: N/A
Expensify/Expensify Issue URL: N/A
Issue reported by: Internal engineer
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1696892206443779
Action Performed:

Sign in on staging web (v1.3.80-0)
Set a status

Expected Result:
The avatar shouldn't be cropped
Actual Result:
Notice avatar is cropped
Workaround:
No workaround
Platforms:

I only confirmed this on MacOS Chrome and iOS Native, but the issue is likely present on all platforms.

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

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

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

Upwork Job URL: https://www.upwork.com/jobs/~01eb806aeec36acb51
Upwork Job ID:<PHONE_NUMBER>640274432
Last Price Increase: 2023-10-09

This issue will potentially be combined with this issue

Closing this issue to be combined with this issue


[Support Job Resource] Add basic job definition

What this PR does / why we need it:
For supporting job resource in kube batch

/lgtm
/approve

@k82cn  what is required for a legal header?
https://travis-ci.org/kubernetes-sigs/kube-batch/builds/518170402#L491

what is required for a legal header?

not sure, try related update script.

/cc @Rajadeepan

@k82cn  the manually added file zz_generated.deepcopy.go should be removed from header check in script file.

/lgtm

the manually added file zz_generated.deepcopy.go should be removed from header check in script file.

good


minor improvements

@yegor256 помоги тут пожалуйста разобраться, почему таски не запускаются?

@Kartofanych also, read this: https://www.yegor256.com/2015/07/06/public-static-literals.html

here is examples about Public Static Literals but i dont have it - only private constants

@Kartofanych maybe some bug in GitHub. Just push some new changes to your branch, this will re-start the jobs.

unfortunately, didnt help

@Kartofanych try again please, I upgraded to macos 15

@Kartofanych try again please, I upgraded to macos 15

some strange issue in yamlint now - in my class there are no extra empty lines, and i didnt touch any other files

@Kartofanych I fixed yaml

@Kartofanych what is the motivation for these changes? I don't think the code became more readable. Instead, it's longer now and harder to read, in my opinion.

@Kartofanych this is not how this repository is designed: we don't use static literals for -1 and other magic numbers. We prefer using numbers, here is why: https://www.yegor256.com/2015/07/06/public-static-literals.html


Can't scroll page vertically on mobile when interacting with slider

I'm looking for a way to disable vertical scrolling in slider container and only allow to horizontal interactions. The problem is, that when you scroll page on mobile device and you stumble upon the slider, you can't scroll the page further. You can only scroll vertically. I think the problem lies in Pointer Events API.

Solved by deleting preventDefault() method called on touchstart event
https://github.com/lmgonzalves/momentum-slider/blob/545e86b49ed5b258266fbe11b8950b59c3cc6245/js/momentum-slider.js#L237


two-factor (steam guard code) doesn't work

doesn't work
proof :
http://prntscr.com/hfo5qs

Are you using 5 digit code or shared_key?

nvm it work I use shared key,,
but maybe can u add for hidden Recent Activity ???

What do you mean? Clear your recent activity from profile? There is 3 appID's that can hide that. Search on Google for the App's ID. You need to put them for last in the list.

like this
"games": [
766,
730,
399220,
399220,
399080
]
???


Migrate camel-k-runtime quarkus extensions into camel-quarkus

camel-k-runtime used to host some camel components and its quarkus extensions suitable for camel-k project, however today all camel components are already in camel, only the quarkus extensions remained in camel-k-runtime project.
It seems appropriate to move all camel-k-runtime quarkus extensions into camel-quarkus project.

I plan to work on this by working on a personal branch and later requesting review.

Inspecting the camel-k-runtime project, these are the structure and aspects to be migrated to camel-quarkus

docs: These pages lists the languages and how it works in Camel K, this is specific to Camel K and should not me migrated to camel-quarkus, but shoud go to Camel K documentation. They are published in Camel K website
examples: Migrate to https://github.com/apache/camel-quarkus-examples
camel-k-* quarkus extensions: Migrate to camel-quarkus, some of them like cloudevents, cron, knative, master,  webhook already exists in camel-quarkus, but are different. Other extensions as resume-kafka, runtime, are not present. These migrations are expected to be the main effort to bring into camel-quarkus
itests: Migrate as the quarkus extensions above.
catalog: Adapt to the camel-quarkus catalog and extend it whenever necessary.

There are some aspects relevant only to Camel K and we should not add these concerns to camel-quarkus, but look for ways to adapt Camel K to use camel-quarkus directly, this is yet to be discovered along the migration process.

examples: Migrate to https://github.com/apache/camel-quarkus-examples

I don't see those camel-k-runtime projects adding any real value given that we have camel-k-examples.

A description of each module (comprising of code), worth migrating.

Component name
Notes

camel-k-cloudevents
There is no customization in ck-runtime. Don't migrate.

camel-k-cron
@LoaderInterceptor @ConfigurerThere is CronShutdownStrategy to stop the camel context when there is a completed or failed event. Migrate CronShutdownStrategy to camel-quarkus and set a parameter to enable it ?

camel-k-knative
@LoaderInterceptor @Configurer

camel-k-master
@Customizer, @LoaderInterceptor @Configurer annotations.

camel-k-resume-kafka
@Customizer resume strategy

camel-k-runtimecamel-k-core
There are similar implementations in camel-quarkus, move them accordingly.

camel-k-webhook
Unsure if it's relevant to camel-k

support/  camel-k-annotations  camel-k-apt
What would be the augmentation in the build phase ?

support/  camel-k-catalog  camel-k-catalog-model  camel-k-maven-plugin
The Yaml catalog is the artifact used by camel-k-operator to resolve dependencies.The camel-k-maven-plugin generates the yaml catalog file.

camel-k-maven-logging
Structural log of maven output

camel-k-runtime-bom
This BOM is imported in the generated integration project created by camel-k-operator

camel-k-master, there's a PR that has been merged already https://github.com/apache/camel-quarkus/commit/17c9fdcfed19b14b12cbde928cd88aad31bb50ff

I migrated the camel-k-core and camel-k-runtime and their respective tests my camel-quarkus repository, branch merge_ckr for an early review

There are some parts that requires more investigation, like the use an existing camel-quarkus Runtime interface, instead of the one in the k-core/api package
Also, there are some tests disabled due to failures in the CamelContextCustomizer tests, which I am investigating.
There is the ApplicationConfigSourceProvider which I will see if it is really necessary to depend on microprofile config.
Any  feedback is welcome.

Wont do for now. The initial work was reverted in https://github.com/apache/camel-quarkus/pull/6388.
We have backups in the form the 3.14.x branch and the previous release tags.


fix(core): not working first plugin when icon option enabled

Hello! Just got not working plugin when enable icon option.
Problem:
not working config:
{
  icon: true,
  plugins: [ { removeDesc: false } ]
}

working config:
{
  icon: true,
  plugins: [ {}, { removeDesc: false } ]
}

Solution: add lodash merge function for concat arrays instead merge them and add test for it.

Ah nice, we bumped into the same problem today as well!
Thanks for the PR, that should fix the problem yes 🙌
One minor suggestion maybe: how about extracting the "config merging" into a separate function and test that? Relying on an SVG snapshot to test merged configs is kind of not intuitive. I/we can also open up a new PR to improve the testing if we don't want to do that in this PR.
WDYT?

@emmenko thank you for comments! just make improvements on your suggestions! how about it?

Cool, thanks! Much better now 👌
Let's see what the author has to say about it 😇
PS: can you run prettier on the changed/test files? The formatting looks weird.

@emmenko sorry for formatting! done with prettier.

Awesome thanks 🙏 !


Changed login to be with email instead of username.

In order to use these new changes, you must delete your previous migrations and db.

Don't merge yet.


SaveState Addition

suggest to add to saveState -> saveColumns
savedColumn.displayName = column.displayName;

This help to reconstruct grid.

In general the saveState feature aims to save state that the user can directly influence.  Can the user change the displayName in your grid?

Hi Paul, whilst writing a detailed response to you, I have found another way.  thanks.

Glad you found a solution!


Add a hyperlink option in Status page to the service/website it monitors

⚠ Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find similar feature request

🏷 Feature Request Type
UI Feature
🔖 Feature description
Option to add a hyperlink to an item in Status page
✔ Solution
On click it opens in new tab(window) specified link to the service/website it monitors. Fuctionality of Heimdall/Homer dashboard.
❓ Alternatives
No response
📝 Additional Context
No response

This has already been implemented


docs(README): change the website domain.

#3652  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rzvxa and the rest of your teammates on  Graphite

@Boshen The CI is stuck again, I don't understand why it keeps happening recently. Is it a GitHub issue or something with the protection policies put in place?

@Boshen The CI is stuck again, I don't understand why it keeps happening recently. Is it a GitHub issue or something with the protection policies put in place?

CI is skipped when markdown file are changed.

Now it makes sense😅


[TDP] Adding feature flags for TDP static and search pages

Add feature flags for TDP related wagtail (static) pages. Add feature flag for TDP Search Interface and Activity Detail pages.
Review

@chosak @willbarton @higs4281 @Scotchester

Checklist

[x] PR has an informative and human-readable title
[x] Changes are limited to a single goal (no scope creep)
[x] Code can be automatically merged (no conflicts)
[x] Code follows the standards laid out in the development playbook
[x] Passes all existing automated tests

OK. Will do.

Closing as superceded by #4502, opened to try and jog Travis.


Support multiple named schemas on dbtoyaml (and yamltodb)

dbtoyaml allows you to extract the definition for a single named schema:
Usage: dbtoyaml [options] dbname

Options:
  -n SCHEMA, --schema=SCHEMA
                        only for named schema (default none)

It also allows you to extract the definitions  for a list of named tables, ignoring the other tables:
Usage: dbtoyaml [options] dbname

Options:
  -t TABLIST, --table=TABLIST
                        only for named tables (default all)

It would be useful it we could use the same approach with schemas - i.e. default to extracting all schemas, but allow the --schema= option to accept a list of schemas.
Usage: dbtoyaml [options] dbname

Options:
  -n SCHEMALIST, --schema=SCHEMALIST
                        only for named schemas (default all)

If you also implement #19 then the new syntax should apply to yamltodb too.

yamltodb already supports this (added in 033f247). It shows in the --help output, but I see that docs/yamltodb.rst should use SCHLIST in the syntax description.


Improve support for variable blocksizes

This is a few different changes to improve SLATE's support for variable block sizes.

BaseMatrix::gridinfo now tries to detect the grid layout when unknown.  Fixes #102
condest was rewritten to not depend on a 2d block-cyclic layout.
Device memory is allocated for the largest tile, not the (0,0) tile.  Fixes #101
MatrixStorage::allocWorkspaceBuffer now takes a buffer size instead of assuming a size of tileMb(0) x tileNb(0)

Detecting the layout of a 2000x2000 tile grid (i.e., n=1e6 and nb=500 or n=32000 and nb=16) takes about 0.3s on Haswell.  And the result is cached with a 2d block cyclic layout is detected, so we pay that cost at most once per matrix.

@mgates3 I've updated is_2d_cyclic_grid to take int* and process_1d_grid_cyclic et al. to take GridOrder as discussed.  So, all your comments should be addressed.


[devinfra] remove go dependency

Remove needing to have go installed to build the project.

@bors-libra r=fbandersnatch

:pushpin: Commit 23f435d has been approved by fbandersnatch

:hourglass: Testing commit 23f435d311220add703196bd3d496512154254b6 with merge 568cf84f0aa19c5bcb1173bce55072ade1516ca7...

:sunny: Test successful - checks-circle_commit_workflow
Approved by: fbandersnatch
Pushing 568cf84f0aa19c5bcb1173bce55072ade1516ca7 to master...


Fixed infinite recursion bug in AuthFilter::authenticate

The current implementation of AuthFilter.authenticate does this:
requestContext.setSecurityContext(new SecurityContext() {
                @Override
                public boolean isSecure() {
                    return requestContext.getSecurityContext().isSecure();
                }
            });

Calling isSecure() on the filtered requestContext will recurse infinitely causing a StackOverflowError.
This PR fixes this bug and adds a test. The test is quite complex for such a small fix, but I can't think of a simpler way, due to the JAX-RS class design. Because of this, I'm fine with this being merged with or without the test.

@jplock I fixed the issues that you commented on.

Thanks, I guess I'm a little confused under what scenario secure will be true in this case?

Thanks, I guess I'm a little confused under what scenario secure will be true in this case?
secure will be true if the incoming requestContext is secure (which happens for example if the incoming request is made over HTTPS). We want our filter to keep the state of requestContext.getSecurityContext().isSecure().

Thanks!


Make Crocoddyl a Python library

In GitLab by @nmansard on Feb 18, 2019, 13:11
This implies:

installing the crocoddyl module
remove the symbolic link
test the examples with unittest
put the locomotion submodule back to crocoddyl

In GitLab by @nmansard on Feb 20, 2019, 10:00
Following the advice of @nim65s , this is what we should do:

remove the symlinks and call the python script from the root of the repo.
move locomotion back in croco folder, to make it a submodule. Locomotion object would be used with

from crocoddyl.locomotion import *

move the example / unittest related to locomotion in the example / unittest folders, with explicit names.
start documenting, it is useful for ipython ? operator and sphynx will be used later.

In GitLab by @nmansard on Feb 20, 2019, 10:03
For future work, we will make crocoddyl a mixed c++/python package, to be installed by robotpkg. This is what are the additional steps:

add a src folder where c++ sources are added. Keep crocoddyl folder for python code.
add a cmakelist, derived from Pinocchio. @nim65s is working on this at low priority. We should have it sometime.
c++ and python sources will be compile in a build/src/.o, build/src/libcrocoddyl.so build/src/.pyc. Unittest will be run from the build/src dir.
libcrocoddyl.so and *py will be installed.

In GitLab by @cmastalli on Feb 23, 2019, 10:08
@nmansard I will add that we need to format all Python files:

Sorting Python module inclusion (isort)
Linting analyses (e.g flake8)

This is a easy task that can be done automatically using your prefered IDE. However it can create plenty of conflicts during the merge request if we do it in a wrong moment. I would like to do it in order to ship quality code, this task should be the last one before v1.0.0 release.

In GitLab by @nim65s on Feb 23, 2019, 16:46
If you want to add this now, the best is to make on commit / MR just for it. I can do that if you want. After that, I can set it up to be ran in the CI, to enforce those rules to be followed.

In GitLab by @nmansard on Feb 28, 2019, 11:19
assigned to @nmansard

In GitLab by @nmansard on Feb 28, 2019, 11:49
I think we have to conflicting discussion here. This issue is about making crocoddyl a library, not about indenting the code.
@cmastalli please refer to #125 for discussion about indentation, it has nothing to do here.
@nim65s are you proposing to implement the indentation and enforce it? Or is your answer related to the initial topic of making it a library? In the first case, could you please propose it again in #125? In the second case, could you be more specific?

In GitLab by @nmansard on Feb 28, 2019, 11:50
@proyan if I move the locomotion files this afternoon, is it endangering your local development branch?

In GitLab by @nim65s on Feb 28, 2019, 11:51
I am proposing to implement standard style rules and enforce them, but this has nothing to do with the current issue.

In GitLab by @proyan on Feb 28, 2019, 12:23
@nmansard
I'm okay with moving the locomotion files.

In GitLab by @nmansard on Feb 28, 2019, 16:23
And ... it is not working as expected. We forget the specific case of the notebooks. They will not work anymore without either a symlink to the crocoddyl folder, or a proper installation in a path covered by PYTHONPATH.
@nim65s what would you propose to cope with this issue?
@proyan please let me handle the issue entirely (well, with the advises of @nim65s ).

In GitLab by @nmansard on Mar 7, 2019, 13:55
After discussing with @nim65s here is how we will process:

I will remove the symlinks as initially decided, except in the notebook subfolder. This is still an expected step for the V1.

Later, @nim65s will set up a cmakelist and reorganize the lib, to have a make install that would then allow to generate a robotpkg packet and enable the CI.

In GitLab by @nmansard on Mar 22, 2019, 15:43
mentioned in merge request !147

In GitLab by @nmansard on Mar 22, 2019, 15:43
Solved by !147

In GitLab by @nmansard on Mar 22, 2019, 15:44
closed

In GitLab by @cmastalli on Mar 23, 2019, 02:35
mentioned in commit 77c18f366345eba100bae67fc33f6bed69845e18

In GitLab by @wxmerkt on May 9, 2019, 22:52
Does it make sense to add a setup.py for installation via setuptools?

In GitLab by @nim65s on May 9, 2019, 23:55
This library strongly depends on Pinocchio which is not packaged with setuptools, so I don't see much interest in doing so.
Moreover, packaging pure python libraries with setuptools makes sense, but we are going to switch most of the backend to C++, so this will become less straitforward.

But yes, in the long term, it would be really nice to provide prebuilt wheels on PyPI for manylinux on most gepetto packages, but I fear that we are not ready yet.

In GitLab by @wxmerkt on May 10, 2019, 13:29
Okay, we do have use cases were people have Pinocchio from either robotpkg, source, or ROS and it would make it easier to install PyCrocoddyl from source (I am not suggesting a pip install for the C++ version). Happy to do a local workaround for now./

In GitLab by @nim65s on May 10, 2019, 13:46
Pinocchio & Crocoddyl are using the same CMake submodule, and they are packaged in the same way on robotpkg.
So if you want to install Crocoddyl from source, you can run the usual mkdir build; cd build; cmake -DCMAKE_INSTALL_PREFIX=… ..; make -j8; make install. Or you can also use robotpkg for source build & install.
About ROS… You are the expert :P


RedisConnection doesn't propogate redis connection exceptions when AbortOnConnectFail = true

My understanding of AbortOnConnectFail is that it's used to toggle whether connection issues against redis should propogate immediately or on next attempt, which allows the multiplexer to be kept alive and continue retrying.
Currently RedisConnection's Execute and ExecuteAsync have try/catches that catches any exception thrown from IDatabase's Execute(Async), and then rethrows it as a normal exception and with its own formatting instead of propogating the actual exception.
One side-effect of this is that you can't catch connectivity exceptions in your application without digging into the InnerException, which makes building your application resilient to connectivity issues awkward at best.
I suggest the try/catches be removed to allow the real redisexception to propogate up.
Alternatively, if the wrapping is really desired for message formatting reasons or otherwise, then for it to be rethrown as some kind of redis.om or StackExchange.Redis exception rather than a basic Exception. If so, it might also be prudent to differentiate catching a RedisTimeoutException from a RedisCommandException, and so on.
Also, I see that neither RedisClientException or RedisIndexingException inherit from RedisException, which might be preferable to centralize all redis-related errors under the RedisException umbrella (makes catching them a lot easier).

Hi @JuliusMikkela - so the issue here is that adding RedisException as a parent class of our internal exceptions is that it will increase the extent to which the Redis OM API is dependent on StackExchange.Redis' public API. To this date the only intermingling we have is that the RedisConnectionProvider will take a connection multiplexer (a highly localized API which can be kicked out to an extension one day if needed).
It's true that there isn't much (or really anything) by way of advanced exception handling for client interactions here. We should create a set of exceptions based off of common exceptions we might get from the inner client, but the rationale behind the catch/wrap/throw is to prevent an unnecessary comingling of the APIs

Alright @slorello89, if that's the intended design then I agree that it would be better to create an umbrella/base exception set for Redis.OM than expand on and propogate StackExchange's models.
Personally I'd probably let the dependency be high until the OM is developed enough to take on a life of its own, and then make a breaking major revision, but that's just personal preference.
I'll adapt my PR, and take an extra look around for places where StackExchange might bleed through.


jest snapshot tests are failing with invariant Violation

recently I've replaced mocha with jest and I love the snapshot testing feature. Unfortunately this does not work for components using the react-list. The tests are failing with an invariant violation:
Invariant Violation: getNodeFromInstance: Invalid argument.

      at invariant (node_modules/fbjs/lib/invariant.js:44:15)
      at Object.getNodeFromInstance (node_modules/react-dom/lib/ReactDOMComponentTree.js:162:77)
      at findDOMNode (node_modules/react-dom/lib/findDOMNode.js:49:41)
      at ReactList.getScrollParent (node_modules/react-list/react-list.js:191:18)
      at ReactList.updateScrollParent (node_modules/react-list/react-list.js:325:34)
      at ReactList.updateFrame (node_modules/react-list/react-list.js:310:14)
      at ReactList.componentDidMount (node_modules/react-list/react-list.js:149:14)
      at node_modules/react-test-renderer/lib/ReactCompositeComponent.js:265:25
      at measureLifeCyclePerf (node_modules/react-test-renderer/lib/ReactCompositeComponent.js:75:12)
      at node_modules/react-test-renderer/lib/ReactCompositeComponent.js:264:11

Googling for this I found some react issues regarding react-test-renderer and react-dom imported by 3rd party libs. Now I am just mocking the react-list in the tests:
// __mocks__/react-list.js
import React from 'react';

const ref = () => {};

export default function ReactListMock ({
    items,
    itemsRenderer,
    itemRenderer,
}) {
    const itemElements = items.map ((item, index) => itemRenderer (index, index));
    return itemsRenderer (itemElements, ref);
}

Does anyone have another solution?
Or even know whats going wrong in react internals?
Shall we add this to the FAQ section in the readme?

Just got similar error in rollbar

Invariant Violation: Minified React error #44; visit http://facebook.github.io/react/docs/error-decoder.html?invariant=44 for the full message or use the non-minified dev environment for full errors and additional helpful warnings.
  File "webpack:///../~/react-dom/lib/reactProdInvariant.js", line 31, in n
      var error = new Error(message);
  File "webpack:///../~/react-dom/lib/findDOMNode.js", line 53, in r
        !false ? process.env.NODE_ENV !== 'production' ? invariant(false, 'findDOMNode was called on ...
  File "webpack:///../~/react-list/react-list.js", line 218, in value
            var el = findDOMNode(this);
  File "webpack:///../~/react-list/react-list.js", line 352, in value
            this.scrollParent = this.getScrollParent();
  File "webpack:///../~/react-list/react-list.js", line 337, in value
            this.updateScrollParent();
  File "[native code]", line unknown, in value

the link contains
findDOMNode was called on an unmounted component.

I get similar error with jest.
TypeError: Cannot read property 'styleSheets' of undefined
      at Window.getComputedStyle (node_modules/jsdom/lib/jsdom/browser/Window.js:405:36)
      at ReactList.getScrollParent (node_modules/react-list/bin/react-list.js:170:24)
      at ReactList.updateScrollParent (node_modules/react-list/bin/react-list.js:301:32)
      at ReactList.updateFrame (node_modules/react-list/bin/react-list.js:286:12)
      at ReactList.componentDidMount (node_modules/react-list/bin/react-list.js:109:12)
      at node_modules/react-test-renderer/lib/ReactCompositeComponent.js:264:25
      at measureLifeCyclePerf (node_modules/react-test-renderer/lib/ReactCompositeComponent.js:75:12)
      at node_modules/react-test-renderer/lib/ReactCompositeComponent.js:263:11
      at CallbackQueue.notifyAll (node_modules/react-test-renderer/lib/CallbackQueue.js:76:22)
      at ReactTestReconcileTransaction.close (node_modules/react-test-renderer/lib/ReactTestReconcileTransaction.js:36:26)```

intentional beahviour by react-test-renderer
@see https://github.com/facebook/react/issues/7371#issuecomment-317396864
I will update the FAQ and do a pull request as soon as I can spare some time


Shasta extension for libbash

Integrates @sethsabar’s shasta for libbash work into the mainline. The tests pass (besides one bash expected failure).

The changes are mostly backwards compatible, but to accommodate bash some of the existing AstNodes are modified. Namely, references to fds (and the arg field in DupRedirectNode) in the AST have been changed to the tuple  (str, [list[ArgChar], int]), since in bash commands can redirect to variables that are then assigned an fd. Besides that, it’s just adding new optional fields. Are there projects using shasta that would break on these changes? (besides PaSh).

The tests for dash still pass, both in shasta and my branch of PaSh with the bash changes merged in, which is a good sign.

I’ll add some documentation on that. Repo file layout:

ast_node.py: AST nodes shared between dash and bash (by design). In principle, if you want to use libdash you can just ignore the bash fields.
json_to_ast.py: Libdash to shasta
bash_to_shasta_ast.py: Libbash to shasta
print_lib.py: Pretty prints the resulting AST in the same way for both bash and dash.
subst.py: Bash only helper functions.

If you’re using the library, the only difference is whether you import from json_to_ast (for dash) or bash_to_shasta_ast (for bash).

I now saw your overall comment, that is great, thanks a lot! The documentation plan sounds perfect!
Regarding the changes to AST nodes, could we have a dash-bash-diffs.md document that enumerates them? Are they only the ones you mentioned here? If so, things should be fine more or less, I don't suspect anything will break due to them (and if something does break we should be able to fix it pretty easily).
Once we get these merged we should also make a new major release of shasta so that downstream projects don't really break.
Thank you so much for the very diligent work here @BolunThompson !!

That’s weird — the tests seem to sometimes pass on my machine, and I’ve somehow messed up the state of my local git repo, so I‘ll work on this tomorrow.

Should be ready to review now!

I suspect shasta converts libbash into shasta incorrectly for a few edge cases because they compare roundtrip 1 and 2 (ex: there was a bug where shasta ignored whether a case node was supposed to fall through or not). Comparing the outputs of the original script and the roundtrips on valid bash tests could catch these (another issue?).

Do you mean that to catch these bugs we would need to compare the original script and the output after the roundtrip? instead of roundtrip 1 vs roundtrip 2? I think this would catch some bugs but would also make things a bit flaky because our pretty printing would affect tests (even though semantically equivalent). In any case, we should look at this in a different PR :)

That’d work, but as you said it’d be finicky. #6 clarifies.


Thiago+jmarino/rules ios manual fusing

Many changes to make things work with rules_ios for now. Some are not idiomatic yet and hard coded instead, will improve that in a follow up

@thiagohmcruz I was thinking to clean up some of the change I made in this branch into their own PRs and put WIP commits separately / clean them up. I can partition out these changes and send them in a series of PRs next week!

@thiagohmcruz I was thinking to clean up some of the change I made in this branch into their own PRs and put WIP commits separately / clean them up. I can partition out these changes and send them in a series of PRs next week!

K, we could also land and iterate on each fix for the FIXME items individually. It's a good way to 🔴  => 🟢  IMO having a working project "with hacks" and then iterate of removing each one keeping things 🟢
No strong opinions here though, we can also split this into multiple PRs upfront.


Add migration guide for mtls annotation.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: diemtvu
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: linsun
Assign the PR to them by writing /assign @linsun in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/assign @geeknoid
We need to have this for 0.8. Should I create PR to master and cherry pick it to 0.8 or just need to add to 0.8 branch?

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: diemtvu
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: geeknoid
Assign the PR to them by writing /assign @geeknoid in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/cc @wattli


group by select problem

Hi
this
select last(*) from occupancy group by psid
returns only one record
even though in console i see two records
my code ...
		String queryString = "select last(*) from occupancy group by psid";
		QueryResult queryResult = influxDBTemplate.getConnection().query(
				new Query(queryString, "parkdots"));
		queryResult.toString();

is the problem in front of the monitor or inside the box ?

Hey, sorry for the late response... I tried to reproduce this in my test environment (based on the docker-compose.yml file) and added an example to Application.java. In both, the application and the console, I only get one record with my test data.
Can you show me what kind of data do you publish to Influx? At least with my very very simple example I cannot reproduce this behavior...

this resulted from my bad influxdb knowledge
i was mixin fields and tags probably
now i still dont really understand the limitations but there should be some
i mean
one can group via tags and not via fields
or in some cases (if aggregating) one can display fields but not tags (this was probably the case)
tags and fields play different role in influx query
i solved this prob when i changed recording via fields and started using tags
still i must appologise as this was a result of my poor knowledge of the technology
i forgot the thread and now i wasted your time
soooory

No worries, isn't a big deal... ;-)


Change ModelResolver to better handle JsonIgnoreProperties

Rather than ignoring all properties with @JsonIgnoreProperties, we only ignore if it doesn't have the "allowGetters" = true
Since if it allowsGetters, you are going to receive it as part of the response
https://github.com/swagger-api/swagger-core/issues/3410
https://github.com/swagger-api/swagger-core/issues/4416
For example, the following line would cause var1 and var2 to be excluded:
@JsonIgnoreProperties(value = { "var1" , "var2" })
However if it has allowGetters, it isn't excluded
@JsonIgnoreProperties(value = { "var1" , "var2" }, allowGetters = true

Thanks!


tree.children[contentDir] is undefined; can't access its "get" property

There's this error when I try to push objects in the quadtree, it only happens when I push more objects than the number maxElement variable has.

Well, it appears that the error vanished. After some time I just tested if the error persisted and it doesn't shows anymore. Sorry for the trouble, but the universe solved it :relieved:

Sorry for the trouble, but the universe solved it 😌

Well good news then 😉. Thanks for letting me know!


the view blinked after ios 16.3 was turned off

.position(.top)
//.isOpaque(true) // When I use isOpaque(true), the view blinked after ios 16.3 was turned off, but not later than ios 16.7

Sorry @wscqs I didn't understand your sentence. What do you mean "ios 16.3 was turned off"? Do you mean there is an issue on ios 16.3 but not on ios 16.7? In any case, please attach a minimal compilable example and make sure the issue reproduces on this minimal example, a video of the issue, and specify your xcode and ios versions. Have a nice day

https://github.com/exyte/PopupView/assets/13110076/b315c541-8e60-44de-ba03-1139626eea5d

here is an issue on ios 16.3 but not on ios 16.7

@wscqs Unfortunately, usually we only choose to support the latest major versions of the iOS and not have minor version specific fixes, the code could turn into a mess in that case. Seems like it’s a bug in the exact version of the OS, since it works fine on iOS 15, 16.7 and 17


test: Add domain test for settlement report

I've updated the PR title according to this guideline.


How do you run terraform fmt in IntelliJ?

Missing instructions: How do I set up IntelliJ to run terraform fmt?

There's an action 'terraform fmt', you can invoke by searching in 'shift+shift' or 'cmd+shift+a'
Also you can run it before commit (checkbox in commit dialog)
Also 'file watcher' could be set up to run it after changes made in editor (usually on focus lose)

Got it, thanks.


Make table writes of aggregations async

Purpose
$subject to improve performance
Documentation
N/A
Security checks

Followed secure coding standards in http://wso2.com/technical-reports/wso2-secure-engineering-guidelines? yes
Ran FindSecurityBugs plugin and verified report? yes
Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets? yes

Codecov Report

Merging #1200 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@             Coverage Diff              @@
##             master    #1200      +/-   ##
============================================
+ Coverage      61.5%   61.51%   +<.01%
  Complexity       35       35
============================================
  Files           618      618
  Lines         24818    24824       +6
  Branches       3856     3857       +1
============================================
+ Hits          15264    15270       +6
  Misses         8268     8268
  Partials       1286     1286

Impacted Files
Coverage Δ
Complexity Δ

...o/siddhi/core/aggregation/IncrementalExecutor.java
81.88% <100%> (+0.89%)
0 <0> (ø)
:arrow_down:

Continue to review full report at Codecov.

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


Updates post discussion with Margie and Nick

this is duplicative - my other PR hasn't been merged yet so these changes are included in it. Closing this PR.


chore(deps): Upgrade vitest to 0.34.6

Reverting because of OOM errors https://github.com/wxt-dev/wxt/actions/runs/6521606212/job/17710355289


Return completed flag for use in script?

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like that [x])
Use Preview tab to see how your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2017.02.07. If it's not read this FAQ entry and update. Issues with outdated version will be rejected.

[x] I've verified and I assure that I'm running youtube-dl 2017.02.07

Before submitting an issue make sure you have:

[x] At least skimmed through README and most notably FAQ and BUGS sections
[x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ ] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[x] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

Description of your issue, suggested solution and other information
Is there a way for youtube-dl to load the manifest of videos to download and return a status or code (especially in quiet mode) to indicate if all of the videos have been downloaded or not?  I would like to use this status in a script to determine if work still needs to be done.  For example, if downloading an entire Youtube channel or Pluralsight course.

What signal are you talking about? When you run youtube-dl either as a process or a python script using YoutubeDL.download or so, it won't return until the work is done (assuming you you pass ignore errors option). When it returns then it's done - that is your signal.


add webp accepted content type only if WebP enabled

Send WebP as accepted content type only if it is enabled. In case it's not, stick with the standard formats in order to avoid the server sending us back WebP images.

Just spent way too long tracking down this problem, as our server got webp support recently and suddenly images stopped working in our app. Nasty one. Glad it's fixed now.


Support ACLs in database

Add new field - ACLField which is used to store Pyramid ACLs in mongo.
ACLField converts Pyramid ACL elements to strings when saving to DB.
ACL elements are validated before saving to database.
Skip fields _version and _acl when generating null values for fields.
Add BaseDocument property __acl__ that decodes string DB representation of ACL to valid Pyramid ACL.
When _acl field is not provided BaseDocument.__item_acl__ is used as default item ACL.

Fails because it relies on old version of nefertari (where ES type was lowercased)

created separate branch: acls_in_db


build retry bug

9.0.5

Version: 9.0.5

3.1

Target:
windows 32 x64

Steps to Reproduce:

i use  cmd:  electronize build /target win  ,its a  long time , waiting....    when i  ctrl+c   end of this cmd,
and then a try  electronize build /target win  agin ,i get an error:
i  must  to  reatart computer to fix it.

Build Electron Application...
Build ASP.NET Core App for win-x64...
Unhandled exception. System.AggregateException: One or more errors occurred. (The process cannot access the file '\?\D:\code\合智项目\ToolCabinet\obj\desktop\win' because it is being used by another process.)
---> System.IO.IOException: The process cannot access the file '\?\D:\code\合智项目\ToolCabinet\obj\desktop\win' because it is being used by another process.
at System.IO.FileSystem.RemoveDirectoryInternal(String fullPath, Boolean topLevel, Boolean allowDirectoryNotEmpty)
at System.IO.FileSystem.RemoveDirectoryRecursive(String fullPath, WIN32_FIND_DATA& findData, Boolean topLevel)
at System.IO.FileSystem.RemoveDirectory(String fullPath, Boolean recursive)
at System.IO.Directory.Delete(String path, Boolean recursive)
at ElectronNET.CLI.Commands.BuildCommand.b__19_0() in C:_WorkRepos\Electron.NET\ElectronNET.CLI\Commands\BuildCommand.cs:line 88
at System.Threading.Tasks.Task1.InnerInvoke()    at System.Threading.Tasks.Task.<>c.<.cctor>b__274_0(Object obj)    at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(Thread threadPoolThread, ExecutionContext executionContext, ContextCallback callback, Object state) --- End of stack trace from previous location where exception was thrown ---    at System.Threading.ExecutionContext.RunFromThreadPoolDispatchLoop(Thread threadPoolThread, ExecutionContext executionContext, ContextCallback callback, Object state)    at System.Threading.Tasks.Task.ExecuteWithThreadLocal(Task& currentTaskSlot, Thread threadPoolThread)    --- End of inner exception stack trace ---    at System.Threading.Tasks.Task.ThrowIfExceptional(Boolean includeTaskCanceledExceptions)    at System.Threading.Tasks.Task1.GetResultCore(Boolean waitCompletionNotification)
at System.Threading.Tasks.Task`1.get_Result()
at ElectronNET.CLI.Program.Main(String[] args) in C:_WorkRepos\Electron.NET\ElectronNET.CLI\Program.cs:line 59

🎉🚀 New Electron.NET version 23.6.1 released 🚀🎉
With native Electron 23 and .NET 6 support. Your problem should be fixed here. If you continue to have the problem, please let us know. Please note the correct updating of your API & CLI. Info in the README. Have fun!


Update for CUDA 12.1.1

xref: https://github.com/conda-forge/cuda-feedstock/issues/11

@conda-forge-admin, please rerender

Thanks Alex! 🙏


Use system's temp directory instead of /tmp/

Use system's default directory for temporary files instead of /tmp/ for database. It closes #4.
Is it required to include modules path and os?
var path = require('path');
var os = require('os');

@amaurymartiny I fixed imports.


Modify model manager to load / unload model based on dependency. Enable ...

ensemble config tests

@GuanLuo You need to manually rebase...


[zh] update content/zh/docs/concepts/services-networking/dual-stack.md

done @tengqm

/lgtm
/approve


Consul port configuration

Is there anyway to change the default consul 8500 port?
Regards

Create your ConsulClient object and pass in a delegate to set the Address property of the configuration to a valid URI during construction:
var client = new ConsulClient((c) => { c.Address = new Uri("http://<IP_ADDRESS>:1337"); });


Save user data

The model for User already existed, just hadn't added it as a SearchList yet. Done here: https://github.com/slackhq/python-slackclient/commit/b7cbd27eb56009fad627c922abedc24189eec3ee

Great, thanks!


Authentication with token

Hi,
is it possible to use cdp when the ChromaDB is configured to use authentication with x-chroma-token header?
I'm using the docker image with this configuration:
docker run
--detach
-p 8080:8000
--name chroma
-v /home/zzz/chroma-storage:/chroma/chroma
--restart=always
-e CHROMA_SERVER_AUTH_CREDENTIALS_PROVIDER="chromadb.auth.token.TokenConfigServerAuthCredentialsProvider"
-e CHROMA_SERVER_AUTH_PROVIDER="chromadb.auth.token.TokenAuthServerProvider"
-e CHROMA_SERVER_AUTH_TOKEN_TRANSPORT_HEADER="X_CHROMA_TOKEN"
-e CHROMA_SERVER_AUTH_CREDENTIALS="xxxxxx"
chromadb/chroma
Regards
Andreas

@awn70, let me add auth support for import/export commands.

That would be super useful. @tazarov, any updates on this?

@awn70, @alex-golubtsov, working on this now. Appologies for late reaction to this

Quick rundown of the feature's implementation. Currently the baseline URI scheme that I have e.g. http://<chroma_url>:<port>/<collection> supports auth (this is standard URI scheme):

basic - http://<user>:<pass>@<chroma_url>:<port>/<collection>
token - http://__auth_token__:<pass>@<chroma_url>:<port>/<collection> (you can also use __x_chroma_token__ if you need the custom auth header)

On top of the above I'm also adding support for env vars:

basic - export CHROMA_BASIC_AUTH="user:password"
token - export CHROMA_TOKEN_AUTH="token" (or export CHROMA_XTOKEN_AUTH="token" for custom token)

I just need to add some tests and this should be ready to go

@tazarov thanks a lot!
http://__auth_token__:<pass>@<chroma_url>:<port>/<collection> was a little misleading at first sight.
As I figured out from the code __auth_token__ is just a placeholder text, which is not used and the real token goes to <pass>

@alex-golubtsov, using__auth_token__ or sometimes simply __token__ is a common URL schemes for things like Docker, GitHub, JFrog Artifactor etc to indicate that the auth should be token based instead of basic. Some of the reasons to use it is:

Clarify - the static string makes it clear that it is token-based auth
Compatibility - this is comaptible with existing URL auth schemes (RFC 3986)

Overall URI-based schemes are nice but I'ld suggest using env vars as exposing sensitive info over CLI tooling is always tricky.

Cool, thanks for the clarification!


add new maintainers

Per our last call, I propose to officially add two maintainers:

Dawn Foster
Georg Link

This pull request is for 'voting'.
Should we also announce this pr on the mailing list for greater visibility? (In line with recent CHAOSS discussion on how to make decisions)

I would be happy to a maintainer for this repo!
Just a quick note that the link to my GitHub account in the commit is incorrect :)

Thanks @geekygirldawn I fixed it!

I agree this is needed, and you are the right people.


Update ignored dirs

Added the .elixir_ls and .git dirs to the list of ignored directories in the Paths module. Otherwise I was getting duplicate runs due to files in the .elixir_ls directory being changed too often during the execution of the tests. I'm not sure why files in that directory are getting changed but they are, and this prevents the double test run.

Nope, never mind. I only thought that fixed the problem, but it didn't. Something else is going on that may have nothing to do with this library at all. Back to the drawing board.


What will happen with the data of the custom attribute when you remove a custom attribute

Hi,
I have a data problem in my B2C instance. I have a custom attribute called userType (small letter u). I store and query my data with the microsoft graph api SDK (C#) as extension_{number here}_UserType (capital U). A few weeks back (around june 16th) my app went down because querying data with the custom attribute name UserType (capital U) couldn't find the records anymore. The result of the microsoft graph api was suddenly with small letter u. I think the graph team is now strictly (case sensitive) following the naming of the custom attribute.
See also issue #78065.
For consistency I want to rename the custom attribute from userType to UserType. Because I already store the data with capital U and query the data still with capital U.
In B2C it is not possible to rename the custom attribute. So what will happen when I remove the custom attribute userType and create a new custom attribute UserType?!?!
Will the data (so the values of the custom attribute) be deleted when I remove the custom attribute (direct or in a background process) or will it still be there? Because if stays there, then I can safely create the new attribute with the capital letter U and then all the problems are gone...
Thanks

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

ID: c29c26fd-dad6-8435-0694-ba82c784818a
Version Independent ID: fab7c0ac-b35b-83f5-5a1e-c8a6ce26dc8b
Content: Define custom attributes in Azure Active Directory B2C
Content Source: articles/active-directory-b2c/user-flow-custom-attributes.md
Service: active-directory
Sub-service: b2c
GitHub Login: @msmimart
Microsoft Alias: mimart

The UI in Azure B2C says that all data will be removed but I tested it and the data is still there after an hour.
Steps to reproduce:

Create custom attribute shoeSize
Patch data of a user xxx with custom attribute name ShoeSize (capital S) and as value '45'
Query ShoeSize (with capital) and see '45' as a result
Delete the custom attribute shoeSize (with small letter s)
Try to query ShoeSize or shoeSize but no data returned
Create new custom attribute ShoeSize (with capital letter S)
Query ShoeSize (with capital) and see '45' as a result again. This was after an hour. I'm going to wait this weekend and see if the data is still there. But I really want confirmation....

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

@JamesTran-MSFT I just checked the same data as last week (4 days ago). See comment above. The custom attribute ShoeSize with value '45' is still there after 4 days as stated in step 7 in the comment.
What do you think? Is it save to delete the custom attribute and recreate it immediately with the correct capital letter?

@JamesTran-MSFT any update on this? We are now 7 days further since my last comment and the data is still there. So I guess my workaround would work...

@LockTar
Thank you for the follow ups on this! Tagging @James-Hamil-MSFT since he's assigned to this issue.

Hi @LockTar , so sorry for the delay. From my perspective, and from the information you provided, it looks like the workaround is a good alternative. However, I have assigned the author to verify that this is safe and if so to update the documentation with your workaround. Thank you so much for your patience!

Hi @James-Hamil-MSFT any update on this. I really want to remove my hotfix

Hi @LockTar , I'm going to escalate this as I have not received a response yet.

@James-Hamil-MSFT we are now 27 days further since I tested my workaround. The data is still there. I think it's save to say that my workaround works.
I think it's to close on my holiday. So if the data is still there after my holiday I will do the workaround on production.
Hopefully you will get an official answer during my holiday.

The Azure AD schema article mentions something interesting.  It uses the term "inaccessible". In the notes it says that after you deleted an extension attribute it's counted as an existing one. And The only way to remove the property value from consideration once it has been set is to explicitly set it to null.
So, sounds like you are right. The extension attribute is not deleted, it's just inaccessible.
To be on the safe side, I'd recommend you to open a GitHub issue in the article above, and get a definitive answer from the MS Graph team. Because the same issue happens when you use the MS Graph API (directly without the B2C).

@yoelhor, As said before. I was on holiday. I opened a new ticket. See above.


Feature request: Add a function to create multiple data streams at once

This allows easier flow handling when creating multiple streams. For example, error handling, or when the streams are required to exist to be able to continue.
The uptime example has also been changed to demonstrate this.

@fsaravia done

@grilix Looks good! Feel free to merge it, and remember to update the MINOR version number after you do it :)

@fsaravia cool, thanks for your help!


vgo tags?

Any plans to support vgo tags?

Hi @silasb, honestly I'm a bit out-of-the-loop here... is there a doc that explains what are vgo tags and how to use them?

Instead of doing versioning via package names you can now version via a git tag.  I jumped the gun on this GH issue, there is no such  thing as a vgo tag. It's just nicer to work with if there is a git tag. https://research.swtch.com/vgo-tour

Yes.

It seems that also a go.mod file is required, I think I'll wait a bit more until a stable release of go tooling to see the best way forward.

Currently using vgo build in existing project with go-serial as dependency will fail with this error:
import "go.bug.st/serial.v1": cannot find module providing package go.bug.st/serial.v1

If I add the dependency manually to go.mod it will fail with below error:
go<EMAIL_ADDRESS>unrecognized import path "go.bug.st/serial.v1" (https fetch: Get https://go.bug.st/serial.v1?go-get=1: net/http: TLS handshake timeout)

I guess that's because vgo wasn't able to access that URL, but I don't really know how to fix this.

I fixed it via: https://github.com/silasb/betaflight-app/blob/master/go.mod
If the repo was tagged with a git version then I think it would be easier
to work with.
—Silas
On Sun, Aug 19, 2018 at 10:29 PM, maddie<EMAIL_ADDRESS>wrote:

Currently using vgo build in existing project with go-serial as
dependency will fail with this error:
import "go.bug.st/serial.v1": cannot find module providing package go.bug.st/serial.v1
If I add the dependency manually to go.mod it will fail with below error:
go<EMAIL_ADDRESS>unrecognized import path "go.bug.st/serial.v1" (https fetch: Get https://go.bug.st/serial.v1?go-get=1: net/http: TLS handshake timeout)
I guess that's because vgo wasn't able to access that URL, but I don't
really know how to fix this.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bugst/go-serial/issues/49#issuecomment-414180559, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACe3k-ortf2cLA8f7H3c0DNw8A7tARJks5uSh7wgaJpZM4UJnyD
.

go mod is now supported with the import go.bug.st/serial.


Better Title

Title barely tells about how awesome the library is. Maybe something like PodPlaygrouds?

podgrounds? :)

Ooh, or maybe CocoaGrounds? Good logo possibilities there!

Thanks for the suggestions, but I'm actually very happy with the project's name :)


Thumbnails Types - Add delete and generate events

For my Image Optimizer add-on it'd be handy if an event was fired when thumbnails are deleted or regenerated. When that happens, I know that they need to be re-optimized afterwards.
I propose these events:

on_thumbnail_delete, when a thumbnail is deleted. Can be fired in deleteThumbnail, I think.
on_thumbnail_generate, when a thumbnail is generated. Can be fired in generateThumbnail, I think.

This makes sense.


nz-input 建议加上disabled时的阴影样式

What problem does this feature solve?
禁用input 样式
What does the proposed API look like?
nzDisabled = true

https://github.com/NG-ZORRO/ng-zorro-antd/issues/2497


Update rstudio dockerfile

Resolves #31 .
Additionally, tidyverse:4.2.2 comes with openssl v3.1.1, so an update to the openssl command in the add_users.sh script was also required. Specifically: -crypt was removed in v3.0 (according to the migration guide); I decided to encrypt with SHA256 as replacement.

Before implementing these changes, the Rstudio environment did not have synapser available, thus resulting in errors when I was dry-running the modules.  I have since rebuilt the server using this proposed Dockerfile, and so far, things are working as expected! 🎉


Follow someone else

Would be cool to follow someone else’s trades somehow and tag along. Future feature

V2 Feature Request -- Keeping around.

meh


Can't add a new api ->  'operator-sdk add api' can only be run for Go operators.

Bug Report
The following command
operator-sdk add api --api-version=component.k8s.io/v1alpha1 --kind=Component

executed within my project : https://github.com/snowdrop/component-operator/tree/operator-0.1.1
generates this error
2018/11/09 10:34:27 'operator-sdk add api' can only be run for Go operators.

Version used
operator-sdk version v0.1.1+git

@cmoulliard Hello, what kind of an operator were you building? Ansible or go?

go operator

This command also generates the same error
operator-sdk generate k8s
2018/11/09 10:57:21 'operator-sdk generate k8s' can only be run for Go operators.

when executed at the root of my project

@cmoulliard The error is because operator-sdk is expecting the main.go file to be at cmd/manager/main.go path.

Many thanks. Can you update please then the migration guide as I don't think that this is documented ?

@cmoulliard There is a mention of it, but feel free to maybe point out how we can be more clear in that so we can update it: https://github.com/operator-framework/operator-sdk/blob/master/doc/migration/v0.1.0-migration-guide.md#copy-changes-from-maingo Thanks!

I propose to do 2 things:

Add another section within the Guide or Migration Guide to mention that it mandatory that the main.go file is created under the path cmd/manager due to the fact the operator-sdk is scanning this path to determine if the project is a Go Operator vs something else
Update the message reported by the operator-sdk client to explain why it thinks that the project is not a Go Operator and what should be the rules validated


Add missing Subrc %3D { sy-subrc }

One improvement a day :-)

the commit description is due to https://github.com/larshp/abapGit/issues/2448


How to keep the data in multi-line configuration

log_format main '$remote_addr - $remote_user [$time_local] "$request" '
'$status $body_bytes_sent "$http_referer" '
'"$http_user_agent" "$http_x_forwarded_for" "$request_time" "$host" ';
My original nginx configuration is multi-line, and after parsing it, it becomes one line. How to keep the data in multi-line configuration;
Note: The spaces and line breaks I use are\t \n

regarding this request, I have already submitted a PR:
#54 Keep parameters in their relative lines when in multi-line configuration.
check it out

thanks @mofantor , let's polish it and we merge


Use ApplicationInsightsSampler

Instantiate the appinsightssampler as part of distro configure and allow a way to configure it.

Handled by enabling sampler configuration in the OTel library:
https://github.com/open-telemetry/opentelemetry-python/pull/2972


Bump versions on azure-devops-node-api to 10.2.2 to address CVE-2021-23358

  High            Arbitrary Code Execution

  Package         underscore

  Patched in      >=1.12.1

  Dependency of   tfx-cli [dev]

  Path            tfx-cli > azure-devops-node-api > typed-rest-client >
                  underscore

  More info       https://npmjs.com/advisories/1674

  High            Arbitrary Code Execution

  Package         underscore

  Patched in      >=1.12.1

  Dependency of   tfx-cli [dev]

  More info       https://npmjs.com/advisories/1674

found 2 high severity vulnerabilities in 230 scanned packages
  2 vulnerabilities require manual review. See the full report for details.

According to https://github.com/microsoft/azure-devops-node-api/pull/440 underscore was bumped up on 10.2.2.

What's the ETA to get this fixed? Also duplicate in #392.

Hi everyone, let us take a look at it this week.

The updated package was published (0.10.0)


adds imageGridViewer

Codecov Report

Merging #30 into master will decrease coverage by 0.25%.
The diff coverage is 54.16%.

@@            Coverage Diff             @@
##           master      #30      +/-   ##
==========================================
- Coverage   61.34%   61.09%   -0.26%
==========================================
  Files          39       40       +1
  Lines         564      586      +22
  Branches       49       51       +2
==========================================
+ Hits          346      358      +12
- Misses        210      220      +10
  Partials        8        8

Impacted Files
Coverage Δ

...iscovery-elasticsearch-app/src/components/index.ts
100% <100%> (ø)
:arrow_up:

...y-elasticsearch-app/src/components/ResultsList.tsx
91.66% <50%> (ø)
:arrow_up:

...asticsearch-app/src/components/ImageGridViewer.tsx
52.38% <52.38%> (ø)

Continue to review full report at Codecov.

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


Checkbox option for the auto-tag function (search online or not)

Adding a checkbox in the auto-tag window for whether it does an online search or not .
search online : use online database and mass write metadata
or not : mass writes metatags based on current form (imported from filename)

Thinking out loud here but a path-filename talker might be an answer here. Which could then tie into the idea of using custom regexs. It wouldn't be great in the GUI (series and issue list would only have 1 record) but who would be using it for that?

It would actually fit better as a form of metadata, but that's for a different issue.


[Feature] - Templates for OCR (Zonal OCR) using KULL

Hi
I already saw your pinned post about automation. Looks great! I exactly need a solution which automatically sorts, and tags my PDFs accrding to their content.
Now i have a feature request:
Many of the expensive solutions offers Zonal OCR. They implemented something called OCR-Templates. This is nothing else than just a file which defines several boxes where OCR searches for Text. One possibility to select such zones is this project:
https://jsoma.github.io/kull
https://github.com/jsoma/kull
I have also recorded a short gif

Now the trick:
There are multiple zones defined

unamed0
unamed1 and so on.

If we have the possibility to do RegEx or any other StrComp function on every zone itself, we would have an extremly powerfull detection engine.
AND:
If we have the possibility to use some of the content of these fields as metadata, we would have one of the most powerfull intelligent classification engine out there...
What do you think?
I have not looked through the code, but if possible, i would like to help to implement this feature.
Maybe this feature could be implemented together with Milestone 1.5.0 ?
Thank you very much.

very interesting idea! thanks for the tip!

But be carefull... It looks like, that KULL outputs wrong coordinates. At least y1 and y2 are switched...

Maybe using ocrmypdf is an option here: Embeds the OCR result into the PDF and does document improvements on imports as well. Maybe #169 is relevant for this, too.

Zonal OCR would be a HUGE feature to implement to this great DMS.


add cleanup docker script to master

It builds up slower, but still needs to be cleaned up.
It also needs docker-py and python3-dateutils to support that.

Deployed in https://github.com/ros-infrastructure/buildfarm_deployment/commit/25bd8fe6d59ea426e9c7950a3fd2144858952665 via #37


Please publish latest version to npm

latest commits are not available in version installed via npm.. thanks

version 0.0.2 released


VSCode Engine 1.11.1 causes npm install error

I updated to the v1.11.1 and ran npm install and got this:
node ./node_modules/vscode/bin/install

Detected VS Code engine version: ^1.11.1
Found minimal version that qualifies engine range: 1.11.1
Fetching vscode.d.ts from: https://raw.githubusercontent.com/Microsoft/vscode/1.11.1/src/vs/vscode.d.ts
Error installing vscode.d.ts: Error: Request returned status code: 404
Details: 404: Not Found

https://raw.githubusercontent.com/Microsoft/vscode/1.11.1/src/vs/vscode.d.ts works fine. Maybe a fluke from Github ?

Yeah, it 404d me in my browser earlier but seems ok now. I'll change it back later and check.

I also got a few hiccup with GitHub today, but all seems fine now. Closing it.


JavaScript Error: Uncaught TypeError: window.gbmedFormsOptions is undefined

PHP Version
7.4
Shopware Version
v<IP_ADDRESS>
Expected behaviour
No JavaScript Error in Developer Console (Firefox).
Actual behaviour
JavaScript Error in Developer Console (Firefox):
TypeError: window.gbmedFormsOptions is undefined -> all.js:3:26786
How to reproduce

Create a customer account
Login to customer account
Place an order
Go to Account > My Orders
Press the button "..." and select "Change Payment method"
On the Page "account/order/edit/" the JavaScript Error is thrown

Can you please give a link or actual name of the extension? I have not installed an extension like that.

The extension name is GbmedForm

The extension name is GbmedForm

Thank you, I see.


Refactor CommonJS rewriting to be a normal pass

Reworks CommonJS module rewriting as well as imported script rewriting into normal passes. Moves them into the default pass configuration.
Rewriting JSON files will be moved in a separate PR.

@blickly Changes made. Let me know when you are ready to import and I'll squash the commits.

LGTM, let's squash and import.
Thanks Chad!

@blickly Sqaushed


mediainfo crashes while inspecting a tv series episode.

I was trying to check a movie with mediainfo.
it crashes with that tv series episodes. with all of them.
it happens with both laptop x64 and armv7.
on armv7 it says:
[1]    17768 segmentation fault (core dumped)  mediainfo ./S01E01.mkv
on laptop:
mediainfo ./S01E01.mkv
terminate called after throwing an instance of 'std::bad_alloc'
  what():  std::bad_alloc
[1]    541719 IOT instruction (core dumped)  mediainfo ./S01E01.mkv

I used mkvtoolnix and removed all subtitles and audios and so that if video only gives that error and it does.

Fedora 38?
Upgrade to latest version (the important package is libmediainfo, but we don't have a armv7 build so if it is fine on your x64 you'll have to use the Fedora testing repo).

Fedora 38? Upgrade to latest version (the important package is libmediainfo, but we don't have a armv7 build so if it is fine on your x64 you'll have to use the Fedora testing repo).

sorry I forgot to add needed info.
I have archlinux on both.
MediaInfoLib - v23.04

what about this development package?

what about this development package?

That seems to work.
is that a regression that occures from  - v23.04?

I can confirm this as well. Both archlinux packages (mediainfo and mediainfo-git) produce std::bad_alloc. The development package you linked to however does work.

@FallingSnow the issue is in libzen-dev, so this one need to be updated before compiling libmediainfo.


using custom wire pins, useful for esp-based boads

For esp-based boards, wire.begin(sda, scl) should be called, otherwise i2c is not working.
https://github.com/esp8266/Arduino
So I've added a method, which can help update the sda and scl pins before begin() is called.

any comments on this?


check-mc-ping: Cannot connect to a replica set

Even though OpenShift is able to connect to the MongoDB replica set via oo-admin-chk, check-mc-ping seems to be failing with the following error:
/opt/rh/ruby193/root/usr/share/gems/gems/mongo-1.9.2/lib/mongo/util/pool_manager.rb:265:in `get_valid_seed_node': Cannot connect to a replica set using seeds server1.example.com:27017, server2.example.com:27017, server3.example.com:27017 (Mongo::ConnectionFailure)
	from /opt/rh/ruby193/root/usr/share/gems/gems/mongo-1.9.2/lib/mongo/util/pool_manager.rb:171:in `connect_to_members'
	from /opt/rh/ruby193/root/usr/share/gems/gems/mongo-1.9.2/lib/mongo/util/pool_manager.rb:66:in `block in connect'
	from <internal:prelude>:10:in `synchronize'
	from /opt/rh/ruby193/root/usr/share/gems/gems/mongo-1.9.2/lib/mongo/util/pool_manager.rb:61:in `connect'
	from /opt/rh/ruby193/root/usr/share/gems/gems/mongo-1.9.2/lib/mongo/mongo_replica_set_client.rb:202:in `block in connect'
	from <internal:prelude>:10:in `synchronize'
	from /opt/rh/ruby193/root/usr/share/gems/gems/mongo-1.9.2/lib/mongo/mongo_replica_set_client.rb:191:in `connect'
	from /opt/rh/ruby193/root/usr/share/gems/gems/mongo-1.9.2/lib/mongo/mongo_client.rb:693:in `setup'
	from /opt/rh/ruby193/root/usr/share/gems/gems/mongo-1.9.2/lib/mongo/mongo_replica_set_client.rb:495:in `setup'
	from /opt/rh/ruby193/root/usr/share/gems/gems/mongo-1.9.2/lib/mongo/mongo_replica_set_client.rb:168:in `initialize'
	from /usr/share/zabbix/lib/openshift_mongo.rb:48:in `new'
	from /usr/share/zabbix/lib/openshift_mongo.rb:48:in `initialize'
	from /usr/share/zabbix/bin/check-mc-ping:99:in `new'
	from /usr/share/zabbix/bin/check-mc-ping:99:in `main'
	from /usr/share/zabbix/bin/check-mc-ping:182:in `<main>'

The MONGO entries in the /etc/openshift/broker.conf look like:
MONGO_HOST_PORT="server1.example.com:27017,server2.example.com:27017,server3.example.com:27017"
MONGO_USER="user"
MONGO_PASSWORD="password"
MONGO_DB="openshift_broker_db"
MONGO_TEST_DB="openshift_broker_test_db".
MONGO_SSL="true"
MONGO_WRITE_REPLICAS=1

That appears to be the issue, yes. The check does not appear to work with MONGO_SSL=true.


API - exchange_rate_period: Monthly exchange rates

Jira link
HOTT-3475
What?
I have added/removed/altered:

[ ] Created serializers for exchange rate list, exchange rates and files
[ ] Create api for returner the JSON api schema with tests

Just noticed few test files might not have been included in the last rebase will check out 👀


Strip ftp:// and ssh://

I accidentally pressed ENTER while creating the issue. 😄
Is it possible to strip ftp:// and ssh://?
I tried these options but it didn't worked. Actually I want strip all protocols.
defaultProtocol: "ftp:",
stripProtocol: true,
normalizeProtocol: true

It only strips protocol for http: and https:. I should make that clearer in the docs. Stripping any protocol would make the URL become ambiguous. Why are you trying to do this?

It only strips protocol for http: and https:. I should make that clearer in the docs. Stripping any protocol would make the URL become ambiguous. Why are you trying to do this?

I get the URL from an input, and it might contains ftp:// and ssh:// (and other protocols). So I want to strip all protocols. I would replace them with empty string but I wanted to know if the library can do this. If this isn't the target of the library, I'm OK with that.
By the way I agree with you in making that clearer in the docs.

That's not a good use for this package. You can just use a simple regex instead. Docs improved.


CLLocationManager extension can call reject multiple times

In the iOS-9 simulator don't set a default location
Invoke CLLocationManager.promise(.WhenInUse).then { ... }
Allow location services
The locationManager delegate didFailWithError is getting called twice by location manager.   This is causing the PromiseKit reject method (see CLLocationManager+Promise.swift line 64) to get called twice which cases the following warning:

PromiseKit: Warning: reject called on already rejected Promise: Error Domain=kCLErrorDomain Code=0 "(null)"

Looks like this is resolved in the latest

This issue only effects the swift implementation as the objective-c implementation shuts down the location manager after an error occurs.

Merged: https://github.com/mxcl/PromiseKit/commit/633fa83ca0c309dc54837db889fab820db5f27a3


Remove support for .NET 7.0 (EOL May 14, 2024)

Once .NET 7.0 reaches EOL, support for .NET 7.0 should be removed.  This is primarily affects the release infra including the release checklist.  Documentation should be scanned as well for 7.0 references.

This PR updated the release pipeline - https://dev.azure.com/dnceng/internal/_git/dotnet-release/pullrequest/40022


Please update NpgsqlPropertyAnnotations to fix issue #509

The following code change will fix #509 . Is this something you can implement? I compared the code against MySql's MySqlPropertyAnnotations class.
The bug is that when using owned types, the primary key returns ValueGenerated.Never for the ValueGenerated property so they use FindSharedTableRootPrimaryKeyProperty to get the primary key.
BEFORE:
 var relationalProperty = Property.Relational();             if (!fallbackToModel                 || Property.ValueGenerated != ValueGenerated.OnAdd                 || relationalProperty.DefaultValue != null                 || relationalProperty.DefaultValueSql != null                 || relationalProperty.ComputedColumnSql != null)             {                 return null;             }
AFTER:
`var relationalProperty = Property.Relational();
if (!fallbackToModel
|| relationalProperty.DefaultValue != null
|| relationalProperty.DefaultValueSql != null
|| relationalProperty.ComputedColumnSql != null)
{
return null;
}
        if (Property.ValueGenerated == ValueGenerated.Never)
        {
            var sharedTablePrincipalPrimaryKeyProperty = Property.FindSharedTableRootPrimaryKeyProperty();
            return sharedTablePrincipalPrimaryKeyProperty?.Npgsql().ValueGenerationStrategy;
        }`

Thanks for drawing my attention to this, it indeed seems to be an issue with Npgsql after all. Let me investigate exactly what's going, SQL Server has a slightly different code fragment from the one you quote above (from MySQL).
Regardless, closing this issue in favor of #509 (which I've reopened), I'll fix this for the next patch release.

Thanks for drawing my attention to this, it indeed seems to be an issue with Npgsql after all. Let me investigate exactly what's going, SQL Server has a slightly different code fragment from the one you quote above (from MySQL).
Regardless, closing this issue in favor of #509 (which I've reopened), I'll fix this for the next patch release.

Hi @roji
When will  the next patch be released?

@NazmiAltun EF Core just released 2.1.2, I'll try hard to release on our side this weekend.

@roji Thanks man, looking forward to it.


Update host_marbles_locally.md

Explained how to troubleshoot the error with the admin credentials.
Corrected the enumeration in the Markdown text.

thanks


CodeCov not uploading coverage report

I am trying to upload coverage reports to codecov using circle ci.
The report doesn’t seem to get uploaded
This is what i see on the console :
+ /Users/distiller/workspace/reports/Rentals/cobertura.xml bytes=4309697
==> Appending adjustments
http://docs.codecov.io/docs/fixing-reports
-> No adjustments found
==> Gzipping contents
==> Uploading reports
url: https://codecov.io
query: branch=dev.ZUM-29613-aggregateReports-ZUM-29614-slack&commit=33762ecc2b7bd439c20e0fd03896483d5261fc7a&build=2725&build_url=&name=&tag=&slug=zumper%2Fios&service=circleci&flags=Rentals&pr=&job=0
-> Pinging Codecov
https://codecov.io/upload/v4?package=bash-tbd&token=secret&branch=dev.ZUM-29613-aggregateReports-ZUM-29614-slack&commit=33762ecc2b7bd439c20e0fd03896483d5261fc7a&build=2725&build_url=&name=&tag=&slug=zumper%2Fios&service=circleci&flags=Rentals&pr=&job=0
-> Uploading
X> Failed to upload
-> Sleeping for 30s and trying again…
-> Pinging Codecov
https://codecov.io/upload/v4?package=bash-tbd&token=secret&branch=dev.ZUM-29613-aggregateReports-ZUM-29614-slack&commit=33762ecc2b7bd439c20e0fd03896483d5261fc7a&build=2725&build_url=&name=&tag=&slug=zumper%2Fios&service=circleci&flags=Rentals&pr=&job=0
-> Uploading
X> Failed to upload
-> Sleeping for 30s and trying again…
-> Uploading to Codecov
-> Sleeping for 30s and trying again…
-> Sleeping for 30s and trying again…
-> Sleeping for 30s and trying again…
-> Sleeping for 30s and trying again…
X> Failed to upload coverage reports```

Can you please post this on the community forums?


Add the originalEvent to the jquery event before triggering the touch event

This change allows you to stop the propagation of the original event, thus making sure that the click event is not triggered after a tap/dbltap event.

awesome thanks


Wrong data type conversion

When I try to add value "@thlon.202!", It will be forced to render as a date type, but it is a string

Version : 1.8.3

@liu-wh Please provide an example where you can reproduce the error -> https://codesandbox.io/p/sandbox/determined-framework-l44s2d?utm_source=dotnew&file=%2Fsrc%2FApp.js

@jaywcjlove link, click "+", set value "@thlon.202!"

@jaywcjlove sorry,  please use this link to test link

@liu-wh Upgrade v1.12.2

@liu-wh Upgrade v1.12.2
v1.12.2 s ok, thanks

The bug is also reproducable with "\S+:\d{3,4}" so something like "some_text:9000" or"some_text:900"  will return a date object with the year 9000 instead of the given text.

@MaeenBadea I didn't think of a good solution.


Our db stream methods won't stop reading as expected

Describe the bug
Our db stream methods such entriesStream have a serious flaw where the returned AsyncGenerator won't act as such.
If you create an entriesStream with opts such that reads all the keys, and then break out of the for await of loop, it won't actually stop reading. It will continue reading all the keys in the background.
The issue comes from the many layers of wrapping:

leveldb abstract down returns an iterator https://github.com/Level/level-js/blob/master/iterator.js
levelup wraps that with which returns a readable stream https://github.com/Level/iterator-stream
We wrap that with an it-pushable subscribing to the readable stream's callback

When we just subscribe to the callbacks the iterator just does whatever in the background. it-pushable instance has no control over it.
Expected behavior
If you break early out of entriesStream it should stop reading from the DB and cleanup.

can this be closed now that #2435 is merged?


Simplified Build Process

This patch simplifies the REX-Ray build process and introduces Travis-CI build stages to make builds faster and easier to understand.

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@3126d3c). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #937   +/-   ##
=========================================
  Coverage          ?   33.74%
=========================================
  Files             ?       37
  Lines             ?     2928
  Branches          ?        0
=========================================
  Hits              ?      988
  Misses            ?     1838
  Partials          ?      102

Impacted Files
Coverage Δ

libstorage/api/types/types_version.go
0% <ø> (ø)

Continue to review full report at Codecov.

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


fix: adjust import analysis behavior to match Node

Description
fixes #16714

 Submitted with StackBlitz.

/ecosystem-ci run vike


migration: ux and warnings

@urcades @jamesacklin here's my attempt at signaling what's happening during migration. I ended up putting the same tooltip over the pending migration button as in the sidebar. This addresses #1260.

@patosullivan looks like that group is partially migrated maybe since one of the channels is joinable. who hosts the solutions channel?


Webpack 2, build is slower?

Hi, strangely, happypack is giving me a slower build time after i upgraded to webpack 2. And the previous config was not using a threadpool. Any ideas why? Current buildtime is 18541ms, it was around 14000ms in webpack 1.
node: v6.7.0
npm: 3.10.3
Webpack: 2.6.1
Happypack: Version: 3.0.2.
  var module = {
    rules: [
      {
        test: /\.js$/,
        exclude: /node_modules/,
        loader: 'happypack/loader?id=hjs'
      },
      {
        test: /\.scss$/,
        loader: 'happypack/loader?id=hcss'
      },
      {
        test: /\.(png|jpg|jpeg|gif|svg|woff|woff2)$/,
        exclude: /node_modules/,
        use: [{
            loader: 'url-loader',
            options: {
             limit: 10000
            }
        }]
      },
      {
        test: /\.yml$/,
        exclude: /feature_config\.yml$/,
        use: ['json-loader',
          'yaml-loader'
        ]
      }
    ]
  };

const happyThreadPool = HappyPack.ThreadPool({ size: 10 });
  var plugins = [
    new HappyPack({
      debug: debugMode,
      verbose: debugMode,
      cache: true,
      cacheContext: {
        env: process.env.NODE_ENV
      },
      id: 'hjs',
      threadPool: happyThreadPool,
      loaders: [{
        loader: 'babel-loader',
        query: {
          cacheDirectory: '/tmp/',
          presets: ['es2015', 'stage-0', 'react'],
          plugins: jsPlugins[env]
        }
      }]
    }),
    new HappyPack({
      debug: debugMode,
      verbose: debugMode,
      cache: true,
      cacheContext: {
        env: process.env.NODE_ENV
      },
      id: 'hcss',
      threadPool: happyThreadPool,
      loaders: scssloader[env]
    }), ...

Sadly, webpack 2 has proven to be much slower for us as well so we went back to using webpack 1. Not much we can do on this side of things, it just has a higher overhead.

Which version of happypack to be used for webpack v1.12.15


My First Post

https://benjaminwen.com/post/my-first-post/

My first comment in GitTalk!

My first comment in GitTalk!


Can't do write operation, but can do read operation. When i do write operation, dbeaver happens "SQL error [08003]: No operations allowed after connection closed”.

Could you add info accordingly to template?
Thanks in advance

What do you mean by info accordingly? For example

there has been no update on the issue for a long time.
If it is still actual - feel free to reopen it.


Storybook: "Advanced -> Parent resize" has no effect

https://mauricius.github.io/vue-draggable-resizable/iframe.html?id=advanced--parent-resize
According to this story the component will resize. But resizing the window in any way doesn't change the component.
Is this story outdated? Am I doing something wrong? What should happen?

Firefox 114.0.2 (64-bit) OSX
Chrome Canary Version 117.0.5865.0

Edit:
~Also for this one the behaviour is unclear:~
https://mauricius.github.io/vue-draggable-resizable/?path=/story/basic--basic-component-with-scale-prop
~The scale is being applied to the parent but then there's no change applied on the components?~
Edit:
This only affects the internal movement I guess.
"Parent resize" is still unclear to me.

https://mauricius.github.io/vue-draggable-resizable/#/story/stories-advanced-parentresize-story-vue?variantId=_default
It's just to show that the component respects the parent if it gets resized.