Migrate decksite database to mysql.

Leaving cards.sqlite as is for now, because it's a mess and doesn't suffer race conditions anywhere near as often.
NB: Tests need to be fixed before this is mergable.

Ooh, exciting.

Any idea why pylint is complaining about similar lines in __init__.py?
http://ci.katelyngigante.com/job/Penny Dreadful/job/Penny-Dreadful-Tools/job/mysql/21/warnings48Result/


fix(CI): fix builder-plugin-swc CI failed on windows

PR Details

Description
Check CI on windows
Related Issue

Motivation and Context

How Has This Been Tested

Types of changes

[ ] Docs change / refactoring / dependency upgrade
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[ ] My code follows the code style of this project.
[ ] I have updated changeset
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

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

closed due to https://github.com/modern-js-dev/modern.js/pull/2177


Android Studio highlighting issue for excluded files

Expected Behavior
When in the detekt-config.yml  a file or path is excluded for a rule, the line which
violates the rule should not be highlighted
Observed Behavior
When in the detekt-config.yml  a file or path is excluded for a rule, the line which
violates the rule is still highlighted
Steps to Reproduce
open a .kt class, add function:
suspend fun testfun() {
var counter = 0
    val tasks = arrayOf<suspend () -> Unit>(
        { counter++ }
    )

    doParallel(*tasks)
}

in detekt-config.yml add
SpreadOperator:
active: true
excludes: "**/path/to/file"
after detekt check, the line doParallel(*tasks) is still highlighted with error color and
error "SpreadOperator: Using a spread operator causes a full copy of the array to be created before calling a method has a very high performance penalty."
is displayed
Context
We wand to accomplish the same behaviour of visual plugin inspection and authomatic gradle inspection
Your Environment

Version of detekt used: 1.0.0-RC15
Version of Gradle used (if applicable): 3.4.1
Operating System and version: macOS 10.14.6
Link to your project (if it's a public repository):

I couldn't reproduce your problem.
Can you please post your Gradle config which relates to configuring detekt?

I have set up a minimal detekt project,
Here is the config:
https://github.com/schauersbergern/detektMinimalSetup/blob/master/build.gradle
Note: Highlighting cannot be suppressed for the files in the excludes, e.g test, androidTest

So you are using the Detekt IntelliJ plugin, right?
https://github.com/arturbosch/detekt-intellij-plugin

Yes, thanks for the hint, I have filed the issue in the plugin repo


[🔧 lint]: request for lint migration rule for deprecated ESLMediaRuleList.parse method

It is requested to have a "migration" eslint rule created to help consumers replace or be notified about syntax that will be replaced in 5.0.0.
Background: #2498
Lint Rule AC:

The rule should be available under 4th group and should not be included in 5th version or 5th version defaults. So

if a user is on 4th version - the lint rule is included in defaults
if a user on the 5th beta it should be either constrained to  5.0.0-beta.24 (the nearest beta that will contain the lint rule but will not have related breaking change) or it should simply be optional and does not include into default or default-5 on the 5th version

The lit rule detects usages of ESLMediaRuleList.parse and treats them as warning-level vulnerabilities for users.

It is acceptable not to check method holder renames (so rule check parse called directly from ESLMediaRuleList only)

The fix script is not mandatory and could be considered as nice-to-have only.
4.1) In case of fix script implementation consider replacing parseTuple / parseQuery

Open Questions:

Do we need to treat single argument parse as vulnerable? (It's not defined explicitly. I'd rather say we should but there is no restriction in the ticket for that)

CC: @exadel-inc/esl-core-team

:tada: This issue has been resolved in version 4.18.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 5.0.0-beta.24 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Treat ERROR_SERVICE_ALREADY_RUNNING as success for start command

I suggest adding an STDOUT message that the service is already running. The rest looks good to me

A minor merge conflict needs to be fixed. My plan is to cut a release tonight, but we can release it later if it does not get into it


docs(readme): try to naturalize copy

@SebastianMeler thanks for merging. could you also respond to my various questions above?


Does not work on Safari

The video isn’t playing on Safari, though Firefox and Chrome seem to be OK.
p.s. Tested on macbook pro.

Hi! Has any progress been made on this?

naw yo figure it out

it's pretty easy these days to just use CSS with a video element or a YouTube embed @ljzerenHein


"tesseract_example" errors

Hello Egor.  I updated Visual Studio to 2017 version and tried to compile the project: "tesseract_example". There were some errors.

What could be cause of the errors?
I also tried to build "with_cppan", it works.

Don't know about VS errors. It's not the best place to ask about it.
Try to build in console cmake --build .. Works for me.


Add rate limit reset time to status.

Adds information about when the rate limit is going to reset to the status update.

Thank you Isabel :)
Could I get you to confirm that the contribution is under the Apache License 2.0?

Yes, sure it is. :)
Am 01.06.2017 8:26 vorm. schrieb "Henri Yandell"<EMAIL_ADDRESS>
Thank you Isabel :)
Could I get you to confirm that the contribution is under the Apache
License 2.0?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/amzn/oss-dashboard/pull/87#issuecomment-305400748,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAEVKbSTGVB2ztrls0RCEZiDjhKzKePrks5r_lmugaJpZM4NqH9Y
.


comments are interpreted as part of a selector

/* a comment */ div { color: red; }

produces the AST
[
  {
    "type": "block",
    "content": [
      {
        "type": "statement",
        "property": "color",
        "valueTokens": [
          {
            "type": "word",
            "source": "red",
            "length": 3,
            "index": 28
          }
        ]
      }
    ],
    "selectors": [
      "/* a comment */ div"   // <---- This here is obviously not what we want
    ]
  }
]

yep, parsing comments is not currently implemented, which is obviously a problem.
I may have time to look at this in the next few days, otherwise, a pull request would be awesome.

I got distracted looking at this and just did it.. Fixed and published.

Wow, that was quick. Thanks :-)


Trying to load config

Scenario/Intent
Just trying to understand the mechanics of starting to use the python OneView interface.
Environment Details

OneView SDK Version: 5.0.0
OneView Appliance Version: 4.20.01-0380241
OneView Client API Version: 1000
Python Version: 2.7.12
Platform: RHEL 7.6

Steps to Reproduce
I've downloaded the sdk to ~/python-hpOneView. Not wanting to alter the sdk code, I created my working directory as ~/tasks and a ~/config.json with contents:
config = {
  "ip": "<IP_ADDRESS>",
  "credentials": {
    "userName": "Administrator",
    "authLoginDomain": "",
    "password": 'mypassword'
  },
  "api_version": 1000,
  "ssl_certificate": "~/oneview.crt"
}

I ran python setup.py install --user
I then copied one of the examples over to my tasks directory and edited it. Contents of server_hardware_types.py:
import sys
sys.path.append('/home/mysid/python-hpOneView/examples')
from pprint import pprint
from hpOneView.oneview_client import OneViewClient
from config_loader import try_load_from_file
config = try_load_from_file('/home/mysid/config.json')
oneview_client = OnceViewClient(config)
server_hardware_types - oneview_client.server_hardware_types
print("\nGet all server hardware types")
server_hardware_types_all = server_hardware_types.get_all()
pprint(server_hardware_types_all, depth=3)

Expected Result
A listing of the server hardware types
Actual Result
Traceback (most recent call last):
File "server_hardware_types.py", line 9, in
oneview_client = OneViewClient(config)
File "build/bdist.linux-x86_64/egg/hpOneView/oneview_client.py", line 124, in init
self.__connection = connection(config["ip"], config.get('api_version", self.DEFAULT_API_VERSION), config.get('ssl_certificate', False),
TypeError: string indices must be integers, not unicode

If I comment out the config = try_load_from_file line and insert the contents of config.json at that point, then the script runs correctly.  What am I doing wrong?

@bradawk1,
Problem is in this line -    config = try_load_from_file('/home/mysid/config.json')
Replace that line with this - config = try_load_from_file(filename='/home/mysid/config.json')
try_load_from_file(config, file_name=None) -  takes one default argument(config) and one keyword argument(file_name).  In your code filapath string  is getting assigned to first argument "config" not to file_path argument.

Great!  I'll give that a try in the morning.  Thanks!

Got it working.  I had to make three changes.  First was to my config.json file.  I had to remove the 'config = ' at the beginning of the file.  In the script, if I just try:
config = try_load_from_file(file_name='/home/mysid/config.json')
it says,
"TypeError: try_load_from_file() takes at least 1 argument (1 given)"
Which really doesn't make much sense.  :)
So, I tried,
try_load_from_file(config,file_name='/home/mysid/config.json')
It complains:
NameError: name 'config' is not defined
So, I added on the line before that:
config=''
Then it worked.  Thanks!

@bradawk1 ,
Sorry I missed it.
In Python , default arguments are mandatory but keyword arguments are not.
For try_load_from_file method, config is a mandatory argument and file_name is not mandatory as its a keyword argument and its already have a default value.
It's not mandatory to use try_load_from_file method, it's just an example for loading the configuration.
You can define the config dictionary in the same file and pass that to OnceViewClient as below.
config = {
"ip": "<IP_ADDRESS>",
"credentials": {
"userName": "Administrator",
"authLoginDomain": "",
"password": 'mypassword'
},
"api_version": 1000,
"ssl_certificate": "~/oneview.crt"
}
oneview_client = OnceViewClient(config)
Let me know if it doesn't work.

Hi Sijeesh,
Yes, that does work.  It was how I got something working in the first place.  :)
I just didn't want to have to put that in every script.  So, it is good I got the load from file working.  My next step is to figure out how to create a server profile template.  The example given doesn't show much detail about creating a template.  Mostly it shows how to extract.

Please check the Server profile template example file for more examples.
https://github.com/HewlettPackard/python-hpOneView/blob/release/v4.8.0/examples/server_profile_templates.py
Closing the issue as the problem got solved.


Timezone handling in timesync needs work

I tried using the timesync scheduler to update the time on my inverter through this autumn's DST change. Did not go well.
I used:
scheduler:
  enabled: true
  timesync:
    enabled: true
    cron: "25 * * * *"

My cheap tariff runs 21:30 - 02:30 so my intention was to "fix" the time 5 minutes before any scheduled changes were due to take effect to ensure the inverter was doing the right thing. In autumn in the UK the clocks change at 2am BST, back an hour to 1am GMT.
I think there's two problems to solve.
1) an invalid time panic when requesting the time at 01:25 BST: (35 minutes before the clocks went back)
Oct 30 01:25:00 vps lxp-bridge[3436684]: [2022-10-30T01:25:00.627 DEBUG lxp_bridge::coordinator] RX: TranslatedData(TranslatedData { datalog: x, device_function: ReadHold, inverter: x, register: 12, values: [22, 10, 30, 1, 25, 8] })
Oct 30 01:25:00 vps lxp-bridge[3436684]: [2022-10-30T01:25:00.627 DEBUG lxp_bridge::mqtt] publishing: lxp/x/hold/12 = 2582
Oct 30 01:25:00 vps lxp-bridge[3436684]: [2022-10-30T01:25:00.628 DEBUG lxp_bridge::mqtt] publishing: lxp/x/hold/13 = 286
Oct 30 01:25:00 vps lxp-bridge[3436684]: [2022-10-30T01:25:00.628 DEBUG lxp_bridge::mqtt] publishing: lxp/x/hold/14 = 2073
Oct 30 01:25:00 vps lxp-bridge[3436684]: [2022-10-30T01:25:00.628 DEBUG rumqttc::state] Publish. Topic = lxp/x/hold/12, Pkid = 41, Payload Size = 4
Oct 30 01:25:00 vps lxp-bridge[3436684]: [2022-10-30T01:25:00.628 DEBUG rumqttc::state] Publish. Topic = lxp/x/hold/13, Pkid = 42, Payload Size = 3
Oct 30 01:25:00 vps lxp-bridge[3436684]: [2022-10-30T01:25:00.628 DEBUG rumqttc::state] Publish. Topic = lxp/x/hold/14, Pkid = 43, Payload Size = 4
Oct 30 01:25:00 vps lxp-bridge[3436684]: thread 'main' panicked at 'invalid time', /home/chris/.cargo/registry/src/github.com1ecc6299db9ec823/chrono-0.4.22/src/date.rs:98:42
Oct 30 01:25:00 vps lxp-bridge[3436684]: note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
Oct 30 01:25:00 vps systemd[1]: lxp-bridge.service: Main process exited, code=exited, status=101/n/a

This seems to be due to an "impossible time in the requested timezone"? I think it boils down to feeding the values in the reply ([22, 10, 30, 1, 25, 8]) into chrono::Local here.
But 2022-10-30 01:25:08 BST is a valid time so not sure what to make of this.
2) the scheduler skipped an hour when scheduling across a DST change.
After the above crash, upon restart lxp-bridge scheduled the next check:
Oct 30 01:25:05 vps lxp-bridge[3706044]: [2022-10-30T01:25:05.935 INFO lxp_bridge::scheduler] scheduler starting
Oct 30 01:25:05 vps lxp-bridge[3706044]: [2022-10-30T01:25:05.947 INFO lxp_bridge::scheduler] next timesync at 2022-10-30 02:25:00 +00:00, sleeping for PT7194.052159789S

Just under 2 hours away, so it skipped the time check at 01:25 GMT (25 minutes after the clocks went back). So the inverter had the wrong time for a while after all :(
This could probably be avoided by using a more granular check period but I don't want to spam the inverter with time requests either.
Just removing the cron stuff and setting a check interval in seconds or minutes is probably a reasonable workaround to this, then timezone changes wouldn't affect the sleep duration.

(2) is fixed in #107 but the actual timesync code is still broken in DST changes thanks to (1) so its only a half-fix.


Reproduce results on MatterportLayout dataset

References:

3D Manhattan Room Layout Reconstruction from a Single 360 Image
MatterportLayout dataset
LayoutNetv2
DuLa-Net

TODO:

Dataset preparation script
Pre-trained model
The new evaluation metric in the paper

Mostly done. The report :clipboard: on MP3D.
Will check the different implementation of depth metric.


Add parameter to Add-VSTeamWorkItem to support setting parent work item

PR Summary
Add parameter to Add-VSTeamWorkItem to support setting parent work item. This is based on the info in issue https://github.com/DarqueWarrior/vsteam/issues/98 and should resolve this issue.
PR Checklist

[x] Write Help
[x] Write Unit Test
[x] Update VSTeam.psd1
[x] Update CHANGELOG.md

Thanks for the support. I am running it through the integration tests now. Will publish tomorrow if not sooner.


Support for custom serialization for value-format?

I am using the google protocol buffer format in my Kafka messages, but I can't seem to properly deserialize them in my consumer with this extension. Is there a work around to this issue? Seems unlikely, but I thought it might be worth asking. Thanks.
https://github.com/jlandersen/vscode-kafka/blob/master/docs/Consuming.md#Deserializer
https://developers.google.com/protocol-buffers
Also, if I wanted to write a local custom serializer is that possible?

Any contribution are welcome!


The method 'goIdle' was called on null. NoSuchMethodError was thrown while notifying listeners for AnimationController

This is my error's logs:
flutter: ══╡ EXCEPTION CAUGHT BY ANIMATION LIBRARY ╞═════════════════════
flutter: The following NoSuchMethodError was thrown while notifying listeners for AnimationController:
flutter: The method 'goIdle' was called on null.
flutter: Receiver: null
flutter: Tried calling: goIdle()
flutter:
flutter: When the exception was thrown, this was the stack:
flutter: #0      BallisticScrollActivity._tick (package:flutter/src/widgets/scroll_activity.dart:535)
flutter: #1      _AnimationController&Animation&AnimationEagerListenerMixin&AnimationLocalListenersMixin.notifyListeners (package:flutter/src/animation/listener_helpers.dart:126)
flutter: #2      AnimationController._tick (package:flutter/src/animation/animation_controller.dart:750)
flutter: #3      Ticker._tick (package:flutter/src/scheduler/ticker.dart:228)
flutter: #4      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding._invokeFrameCallback (package:flutter/src/scheduler/binding.dart:1012)
flutter: #5      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding.handleBeginFrame.<anonymous closure> (package:flutter/src/scheduler/binding.dart:928)
flutter: #6      __InternalLinkedHashMap&_HashVMBase&MapMixin&_LinkedHashMapMixin.forEach (dart:collection-patch/compact_hash.dart:367)
flutter: #7      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding.handleBeginFrame (package:flutter/src/scheduler/binding.dart:926)
flutter: #8      _WidgetsFlutterBinding&BindingBase&GestureBinding&ServicesBinding&SchedulerBinding._handleBeginFrame (package:flutter/src/scheduler/binding.dart:856)
flutter: #12     _invoke1 (dart:ui/hooks.dart:233)
flutter: #13     _beginFrame (dart:ui/hooks.dart:172)
flutter: (elided 3 frames from package dart:async)
flutter:
flutter: The AnimationController notifying listeners was:
flutter:   AnimationController#36d81(▶ -75.000; paused; DISPOSED; for BallisticScrollActivity)

This happen when I scroll very fast to the top of the scroll view which is embedded into a NotificationListener widget. Does anyone know the problem? Is this the best approach to do something while scrolling or not? Thanks you in advance. Arcangelo :)

Hi @pisa97, Can you post a reproduction of the issue? We will need more information to help you resolve it.

same error happen to me,      Help me... I cant reproduce this exception, but happens many on production

I am facing the same issue any solution
Error: The method 'goIdle' was called on null. Receiver: null Tried calling: goIdle()
File: (package:flutter/src/widgets/scroll_activity.dart:535:16)
void _tick() {
if (!applyMoveTo(_controller.value))
delegate.goIdle();
}

I am facing the same issue here as well

A possible solution is that they can validate that the controller does the animation only if the position is different from null
  if (_pageController.position != null) {
          _pageController.position.animateTo(
            _screenSize.width * 0.3,
            duration: Duration(milliseconds: 1000),
            curve: Curves.elasticOut,
          );
        }

In this way the error will stop coming out.


Remove all of the legacy WORKSPACE-related code.

We are no longer supporting legacy Bazel WORKSPACE projects. We an remove the build and test targets that exist to support this mode.

#1231: Removed mentions of the legacy workspace from the release generation.
Still need to remove the go_deps.bzl generation, etc.


AmazonProductAdvertisingClient.Config not getting my aws_access_key_id

alias AmazonProductAdvertisingClient.ItemSearch
alias AmazonProductAdvertisingClient.Config

config = %Config{}

returning
%AmazonProductAdvertisingClient.Config{AWSAccessKeyId: nil, AssociateTag: nil,
 Service: "AWSECommerceService", Version: "2013-08-01"}

@kakawamura For clarification, have you set aws_access_key_id in the config.exs? It should  look something similar to:
config :amazon_product_advertising_client,
  associate_tag: "YourAssociateTag",
  aws_access_key_id: "YourAccessKeyID",
  aws_secret_access_key: "YourSecretAccessKey"

I had this problem too. For some reason compiling the dependencies a few times fixed it for me.


Issues with $record in visible() closure on Tables\Columns\TextColumn: Filament is unable to resolve the record

Package
filament/filament
Package Version
3.2.82
Laravel Version
11.8.0
Livewire Version
3.5.0
PHP Version
8.2.22
Problem description
I have a table with simple TextColumn, so defined:
           Tables\Columns\TextColumn::make('order_suppliers_to_be_approved_by_shipyard')
                ->visible(function (Model $record) {
                    return auth()->user()?->isShipyard() || (auth()->user()?->isSupplier() && auth()->user()->supplier->isLeaderSupplierForOrder($record->id));
                })
                ->label(__('Order suppliers to be approved by shipyard'))
                ->state(function (Model $record) {
                    return $record->orderSuppliersToBeApprovedByShipyard()->count();
                })
                ->color(fn ($state) => $state != 0 ? 'warning' : '')
                ->icon(fn ($state) => $state != 0 ? 'heroicon-m-exclamation-triangle' : ''),

The $record is not found, here the error that I get instead:

App\Filament\Resources\OrderResource::App\Filament\Resources{closure}(): Argument #1 ($record) must be of type Illuminate\Database\Eloquent\Model, null given, called in /Users/zoba-mac/Sites/laravel/navisafe/vendor/filament/support/src/Concerns/EvaluatesClosures.php on line 35

Consider that, few lines below I've the state() declaration for the same column:
                    ->state(function (Model $record) {                         return $record->orderSuppliersToBeApprovedByShipyard()->count();                     })
...which gives no errors (if I comment the visible() invocation).
Expected behavior
I expect the $record to be correctly evaluated as usual.
Steps to reproduce
1 - create a Filament Resource
2 - add "table" method with any given column
3 - choose any column, call the "visible" method on it, passing a Closure with a Model $record as parameter
Reproduction repository (issue will be closed if this is not valid)
https://github.com/giovanniledda/navisafe
Relevant log output

Illuminate \ Contracts \ Container \ BindingResolutionException
PHP 8.2.22
11.8.0
An attempt was made to evaluate a closure for [Filament\Tables\Columns\TextColumn], but [$record] was unresolvable.

Donate 💰 to fund this issue

You can donate funding to this issue. We receive the money once the issue is completed & confirmed by you.
100% of the funding will be distributed between the Filament core team to run all aspects of the project.
Thank you in advance for helping us make maintenance sustainable!

Sorry, duplicate of https://github.com/filamentphp/filament/issues/7757


Allow adding media attribute to picture element

Describe the feature
Hey,
I love your plugin. It is super helpful. I now have the case where I would like to add media attributes to a picture element with your plugin, but unfortunately it is not possible right now, correct? When I add media property to the sources array, it doesn't show up in the code.
So I would like to get something like this:
  <picture>
    <source
      media="(orientation: landscape)"
      srcset="image-320w.jpg 320w, image-640w.jpg 640w"
    />
    <source
      media="(orientation: portrait)"
      srcset="image-320w.jpg 320w, image-640w.jpg 640w"
    />
    <img src="/images/foo.jpg" />
  </picture>

Is it possible to allow this for the sourcesObject:
const exampleSources = [
  {
    media: "(orientation: landscape)",
    srcSet: "image-320w.jpg 320w, image-640w.jpg 640w",
  },
  {
    media: "(orientation: portrait)",
    srcSet: "image-320w.jpg 320w, image-640w.jpg 640w",
  },
];

Thanks for your help in advance.
Best regards
Chris
Additional information

[ ] Would you be willing to help implement this feature?
[ ] Can you think of other implementations of this feature?

Final checks

[X] Check existing issues.

Hi there!
Thanks for the enhancement idea. While I don't have a roadmap for unlazy, media support for source sets is definitely on the list. It should be fairly easy to implement.
Unfortunately, nor do I have the usecase, neither the time to implement it at the moment. PR welcome! 🙌

Hey Johann,
thanks for the feedback. I wanted to create a pull request, but when I fork and clone the project and start it locally, I get this error all the time: "Pre-transform error: Failed to resolve import "unlazy" from "src/runtime/components/UnLazyImage.vue". Does the file exist?"
It seems, that it can't resolve the aliases in vite.resolve.path {}. I also tried starting the vue moduel and the nuxt module, but in both I get the same error. Do you know why this is happening?
Best regards
Chris

Right, there is currently a bug in unbuild with dev mode. That's why you need to run pnpm build once at root level or in packages/unlazy to build the core package first, so that the Nuxt module can use the unlazy package.

Ah okay thanks for the help. Unfortunately I can't build the unlazy package. Any idea why this is happenning? Sorry for bothering you.
RollupError: Could not resolve entry module "../../../../Local%20Development/unlazy/packages/unlazy/src/bundle/index.iife.ts".
│     at getRollupError<EMAIL_ADDRESS>│     at error<EMAIL_ADDRESS>│     at ModuleLoader.loadEntryModule<EMAIL_ADDRESS>│     at async Promise.all (index 0)

Seems like a path resolving issue on Windows. I will have to check that further. I'm busy at the moment but will try to take a look into it in the next days. 🙋♂

I am on Mac OS, but thanks so much for looking into it, when you have the time for it :)

Can you please try again?
pnpm i && pnpm build
cd packages/nuxt
pnpm dev

Hey Johannes,
thanks for looking into it. I can now build everything, but when I start the dev server, I still get the error:
Pre-transform error: Failed to resolve import "unlazy" from "src/runtime/components/UnLazyImage.vue". Does the file exist?
I also have completely recloned the whole project, so every dep and everything ist up to date.
Do you have any more ideas?
Best regards
Chris

Got it to work. I had to change your calculcation of the current directory to __dirname:
    resolve: {
      alias: {
        // '@unlazy/core': `${resolve(currentDir, '../../core/src')}/`,
        // 'unlazy': `${resolve(currentDir, '../../unlazy/src')}/`,
        'unlazy': resolve(__dirname, '../../unlazy/src'),
        '@unlazy/core': resolve(__dirname, '../../core/src'),
      },
    },

Maybe the files couldn't be resolved because of the space in your paths. I have refactored to use fileURLToPath in each instance, which should fix your path issues. (Hopefully)

Hey Johannes, thanks for that. I started implemented the media feature, but experienced another bug, I think:
Right now we get a layout shift, when you specify only width/height. But this should we work, because I have tested with a normal img tag and there we don't get a layout shift.
For the implementation of the media attribute, it should be possible to only specify width/height, because we can have different width/height values for different sources and so we don't know the aspect ratio, which wie normally can set with style aspect-ratio.
Maybe you can clone my repo and then start the dev server to see it in action:
https://github.com/chrispreisler/unlazy/blob/main/packages/nuxt/src/runtime/components/UnLazyImage.vue
https://github.com/chrispreisler/unlazy/blob/main/packages/nuxt/playground/app.vue
The media implementation for itself is working without problems.
https://github.com/johannschopplich/unlazy/assets/5261762/9d92e602-1193-444f-af55-1c0c5daa0563

Hi there, sorry, but I neither have the usecase, nor the time to implement your feature request at the moment. However, PR welcome and I will review it. Thanks! 🙏


Getting address as coordinates

This not an issue am a newbie to programming , after picking an address how do i get the coordinates of that address latitude and longitude..and thanks for creating such an awesome package

its depending on your code
if you use OpenMapPicker then you can get the picked location in onSaved  or onChanged  paramter
    OpenMapPicker(
      decoration: const InputDecoration(hintText: "My location"),
      onSaved: (FormattedLocation? newValue) {
        print(newValue?.lat);
        print(newValue?.lon);
      },
      onChanged: (FormattedLocation? newValue) {
        print(newValue?.lat);
        print(newValue?.lon);
      },
    ),

onSaved when you save form > search for flutter forms
onChanged. called when user tap done button and confirm location

if you use the scond option
Navigator.of(context).push(MaterialPageRoute(builder: (BuildContext context) {
    return OpenStreetMaps(
      options: OpenMapOptions(),
       onDone: (FormattedLocation? newValue) {
        print(newValue?.lat);
        print(newValue?.lon);
      },
      bloc: CustomBloc(const OpenMapState.selected(SelectedLocation.single(null))),
    );
  }));

You are welcome


added jwt token validation

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

Codecov Report

Merging #24 into master will decrease coverage by 4.6%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master      #24      +/-   ##
==========================================
- Coverage   70.85%   66.24%   -4.61%
==========================================
  Files           6        6
  Lines         446      477      +31
==========================================
  Hits          316      316
- Misses        116      147      +31
  Partials       14       14

Impacted Files
Coverage Δ

src/caduceus/caduceus_type.go
100% <ø> (ø)
:arrow_up:

src/caduceus/caduceus.go
0% <0%> (ø)
:arrow_up:

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 2f9b26c...4eccfb9. Read the comment docs.


ci: Add integration tests for exec/signal/tcp trace gadgets

Add integration tests for exec/signal/tcp trace gadgets
These three gadgets were missing for tests.
TODO

[x] Rebase once https://github.com/inspektor-gadget/inspektor-gadget/pull/1053 is merged.

This PR is now ready for review.


Fix fmt errors

With the recent changes to .drone.yml, we should double check that make fmt-check (or something that has it as a dependency) always runs in CI.

depends on #2547 ?

@lunny yes

LGTM

Codecov Report

Merging #2544 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #2544   +/-   ##
=======================================
  Coverage   27.21%   27.21%
=======================================
  Files          85       85
  Lines       16955    16955
=======================================
  Hits         4615     4615
  Misses      11665    11665
  Partials      675      675

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 23645fe...06a9970. Read the comment docs.

LGTM

make L-G-T-M work


Video.js 6.x compatibility

This PR adds compatibility with Video.js 6.x (and future versions) by using getComponent('html5')  for older versions and videojs.getTech('html5') for version 6.x and higher.
Thanks very much!

Will this be merged ever? We are looking into use video.js, but we need the newer version. The fork seems to be working, but we wish to use the main repo instead.

The recently production code sync should work fine on videojs 6. :smile:

Hey guys! Sorry for being silent her for too long. The current implementation https://github.com/streamroot/videojs5-hlsjs-source-handler/blob/master/lib/videojs5-hlsjs-source-handler.js#L168 should work in both videojs 5&6, as videojs.getTech is available in both of them.
@jienco @citosid Please confirm that it works and I'll close the PR.
And again -- sorry for being silent that long, we are overwhelmed by other projects. We'll try to improve, and thank you very very much for you feedback and this PR!

Hi @ruslandinov , no problem :) I tested the latest version of the plugin with VideoJS 6.7.4 and it plays HLS well (See https://embed.plnkr.co/97AbtBGwmreNRnIvmqV1/ - must add the source in the textbox manually). Thank you!


fix account change issue and styling

Fixes #153
Also makes the styling more robust to work on all screens

@bhavya2611 seeing a regression to the fat user panel here:

also seems to be overflowing its container horizontally


Fix bugs reported by mercury in batch #3

Bugs fixed:

[x] support for few, next few days
[x] past week, past Monday
[x] the is not ignored, "Between November 15th through the 19th"
[x] spacing problem, a m/a. m.
[x] a regression bug caused by regex typo for after/before
[x] bugs for parsing duration with next/past as date or range
[x] resolution for a month ago
[x] in+duration (resolution <= day will be absolute datetime while resolution > day will be range)

@rubio41,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot


Invalid version for git resolver

$ shards install

Resolving dependencies

Fetching https://github.com/oprypin/crystal-dbus.git

Fetching https://github.com/freyamade/discordcr.git

Fetching https://github.com/freyamade/hydra.git

Fetching https://github.com/freyamade/noir.git

Fetching https://github.com/woodruffw/notify.cr.git

Fetching https://github.com/andrewsuzuki/termbox-crystal.git

Invalid version for git resolver: v0.1.0+git.commit.c1902634ce60acbd13cb445d441b3ca32631021e

I have this problem too. Neither building from source or using the PKGBUILD is working for me.

This is also happening for me, if anyone has any solution, please let me know.

Been taking a bit of a break from the dev side of things but I'm starting to get back into it, I'll have a look at this when I get the chance, sorry for the absence on my part ^^"

This will take a bit of personal work in the background but I'll get this up to compatibility with Crystal 1.0 myself as soon as I can


UnusedPrivateMember and fun interfaces

I'm not really sure if it must be considered a bug, but I have the following code.
fun interface A {
    fun doSomething(a: Int, b: Int)
}

fun f(g: A) {
    g.doSomething(1, 2)
}

fun a(a: Int, b: Int) { ... } // <-- reports unused variable

val x: Unit = f(::a)

As you can see, function a cannot omit arguments, it'll break types.
On the other hand, I'm not sure if it's possible to detect such cases.

You could code it like this:
fun interface A {
    fun doSomething(a: Int, b: Int)
}

fun f(g: A) {
    g.doSomething(1, 2)
}

fun a() { ... }

val x: Unit = f { _, _ -> a()}

I understand that you want to ues the ::a notation and that this is just a sample and the real code could be way more complex. But If we allow this case we should not report any function parameter because of this.
I feel that this is not a false-positive bebause with a little change in your code you can fix the unused parameter. And I imagine that this is not something common in your code base so maybe an @Suppress have sense in this scenario.
A third option would be to have a flag in this rule to enable/disable the check of function parameters. But I think that we shouldn't go in that direction.

My use case includes a function with 9 arguments.
I can rewrite it as
val x: SomeParametrizedInterface<Type1, Type2, Type3> = SomeParametrizedInterface { argument1, anotherArgument, somethingElse, .... ->
}

but as you can see, it'll create a very long line and it'll fail on max line length check x)
I think too that @Suppress is a good candidate for such cases, I just wanted to highlight the issue, perhaps someone from dev team has a better idea.

Closing this then due to the arguments given in
https://github.com/detekt/detekt/issues/5126#issuecomment-1193280299


Live syntax Heigithing for PHP

It's possible to configure (without install php binaries) Bracket for syntax error heighliting (when you made error in Aptana this line is heighlighted).
I tried a lot of solutions and plugin but nothing dosen't work.
I want change Ecclipse to Brackets becasue Brackets is better (faster, modern) but this feature is very needed. Without these feature still i must use Eclipse :(
Regards

Currently brackets doesn't support PHP. As some plugins are available for that but those don't work that well. We have this feature in our roadmap. So closing this for now.


Parse failed when no document start present

Parser fais when document have no --- start node:
TypeError: Cannot read property '!!' of undefined
    at [object Object].parseNode (/home/ixti/projects/js-yaml/lib/js-yaml/parser.js:375:44)
    at [object Object].parseBlockNodeOrIndentlessSequence (/home/ixti/projects/js-yaml/lib/js-yaml/parser.js:324:17)
    at [object Object].parseBlockMappingValue (/home/ixti/projects/js-yaml/lib/js-yaml/parser.js:561:23)
    at [object Object].state (/home/ixti/projects/js-yaml/node_modules/jsclass/src/core.js:56:19)
    at [object Object].checkEvent (/home/ixti/projects/js-yaml/lib/js-yaml/parser.js:110:32)
    at [object Object].composeNode (/home/ixti/projects/js-yaml/lib/js-yaml/composer.js:89:14)
    at [object Object].composeMappingNode (/home/ixti/projects/js-yaml/lib/js-yaml/composer.js:190:24)
    at [object Object].composeNode (/home/ixti/projects/js-yaml/lib/js-yaml/composer.js:115:19)
    at [object Object].composeMappingNode (/home/ixti/projects/js-yaml/lib/js-yaml/composer.js:190:24)
    at [object Object].composeNode (/home/ixti/projects/js-yaml/lib/js-yaml/composer.js:115:19)

It can parse simple document like:
foo: bar

But fails on document with tags:
foo: !!str bar


implement Binary Exponent calculation in Dart

💥 Proposal
implement Binary Exponent calculation in Dart
Have you read the Contributing Guidelines on Pull Requests?
YES

/assign


[stable/datadog] Add nodeSelector for Cluster Agent Deployment

Signed-off-by: Ivan Sukhomlyn<EMAIL_ADDRESS>
What this PR does / why we need it:
Add the possibility to define nodeSelector object for Datadog Cluster Agent Deployment
Ref: https://kubernetes.io/docs/user-guide/node-selection/
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/ok-to-test

Hello,
@xlucas @mfpierre could you please take a look at the PR?
Thanks

Hi @lachie83 @cpanato
could you please take a look at the PR?

Hi @ivan-sukhomlyn
Your PR looks good, please bump again the chart version, and we will be able to approve it.

/assign clamoriniere

Hi @clamoriniere
Thanks for your reply!
I've rebased master branch and pushed changes with bumped Chart version

/lgtm
Thanks again for you contribution 👍

@clamoriniere And thank you for paying attention to my PR 👍


[SPARK-15150][EXAMPLE][DOC] Add python example for LDA

What changes were proposed in this pull request?
Add python example for LDA
How was this patch tested?
manual tests
./bin/spark-submit examples/src/main/python/ml/lda_example.py

Test build #57865 has started for PR 12927 at commit b98ad7c.

Test build #57867 has started for PR 12927 at commit 543df2f.

Test build #57865 has finished for PR 12927 at commit b98ad7c.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/57865/
Test PASSed.

Merged build finished. Test PASSed.

Test build #57867 has finished for PR 12927 at commit 543df2f.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/57867/
Test PASSed.

@dongjoon-hyun Sorry for cc a lot but it would be great if I can hear your thoughts.

Test build #57968 has started for PR 12927 at commit 2270148.

Test build #57968 has finished for PR 12927 at commit 2270148.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/57968/
Test PASSed.

Merged build finished. Test PASSed.

Test build #57991 has started for PR 12927 at commit d7916b4.

Test build #57991 has finished for PR 12927 at commit d7916b4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/57991/
Test PASSed.

Test build #58088 has started for PR 12927 at commit 8625754.

Test build #58088 has finished for PR 12927 at commit 8625754.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58088/
Test PASSed.

Merged build finished. Test PASSed.

Test build #58089 has started for PR 12927 at commit 0191d6a.

Test build #58089 has finished for PR 12927 at commit 0191d6a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58089/
Test PASSed.

Test build #58101 has started for PR 12927 at commit 5e26019.

Test build #58102 has started for PR 12927 at commit 1219ca1.

Test build #58101 has finished for PR 12927 at commit 5e26019.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58101/
Test PASSed.

Merged build finished. Test PASSed.

Test build #58102 has finished for PR 12927 at commit 1219ca1.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58102/
Test PASSed.

Test build #58316 has started for PR 12927 at commit 65dd7ca.

Test build #58317 has started for PR 12927 at commit eab9511.

Test build #58316 has finished for PR 12927 at commit 65dd7ca.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58316/
Test PASSed.

Test build #58317 has finished for PR 12927 at commit eab9511.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58317/
Test PASSed.

Test build #58329 has started for PR 12927 at commit eab9511.

Test build #58329 has finished for PR 12927 at commit eab9511.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58329/
Test PASSed.

minor comment on some comments, otherwise LGTM. @yanboliang ?

Test build #58345 has started for PR 12927 at commit f573fa6.

Test build #58345 has finished for PR 12927 at commit f573fa6.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58345/
Test PASSed.

Please rebase your PR to current master and fix merge conflicts

Test build #58356 has started for PR 12927 at commit 83918d4.

Test build #58356 has finished for PR 12927 at commit 83918d4.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/58356/
Test PASSed.

Merged build finished. Test PASSed.

LGTM. Thanks!

Merged to master and branch-2.0. Thanks!


Turned the regex in QueryModel class to a reusable constant

Closes #64

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
251

Relevant Lines:
274

💛  - Coveralls


animationFrameScheduler throws in SSR mode

Bug Report
Current Behavior
My Angular app is using a 3rd party library: ngx-highlightjs and it throws an error when running the app in SSR mode:

requestAnimationFrame is not defined

Digging into the stack trace revealed it's due to the use of this line:
animationFrameScheduler.schedule(() =>
  // ...
);

Reproduction

Repo link: I will update this shortly

Expected behavior
I'm actually not sure if it should be the responsibility of rxjs to make animationFrameScheduler work in NodeJS or if it should be the responsibility of the library author to write SSR-friendly implementation.
It looks like there are a lot of similar issues in the wild so my first impression is for this issue to be fixed at the rxjs level. Then all these other issues will miraculously be resolved! ✨
Environment

Runtime: Node v12
RxJS version: 6.6.3
angular/core: 9.1.13 (in case this is relevant)

Same as ajaxObservable which requires proper polyfill to XHR on node.js which doesn't support it out of the box, I believe this should be polyfilled by each application if it's needed to be used on runtime does not have suport for it.

Since RxJS positions itself not as a browser but as a multiplatform library, I believe a check for availability and fallback to setTimeout could be built in. I see that new animationFrames Observable also tries to access global requestAnimationFrame. Do you think this should be addressed, @benlesh or is it working by design and we should be careful using it cross-platform?

IMO

working by design

So I'm closing this.

great argumentation.

@cartant I see that new animationFrames function uses global const animationFrameProvider which has a delegate property, which is undefined. In that case it falls back to global rAF functions. Are we supposed to mutate that variable in SSR environment with a fallback to avoid this issue? Is this how this delegate was supposed to be used?

animationFrameProvider is internal, ATM, and is for testing purposes. It's possible that it might be exposed at some stage, but there is no plan to do so.
Fundamentally, it's not RxJS's responsibility to detect and mock features that applications might be using that are not available in particular environments. As OJ mentioned the same could be said for XHR, fetch, WebSockets, etc.


Implement type_of

Pull Request Description
This change implements a simple type_of method that returns a type of a given value, including for polyglot objects.
The change also allows for pattern matching on various time-related instances. It is a nice-to-have on its own, but it was primarily needed here to write some tests. For equality checks on types we currently can't use == due to a known feature which essentially does wrong dispatching. This will be improved in the upcoming statics PR so we agreed that there is no point in duplicating that work and we can replace it later.
Also, note that this PR changes Meta.is_same_object. Comparing types revealed that it was wrong when comparing polyglot wrappers over the same value.
Checklist
Please include the following checklist in your PR:

[x] The documentation has been updated if necessary.
[x] All code conforms to the
Scala,
Java,
and
Rust
style guides.
All code has been tested:

[x] Unit tests have been written where possible.
[ ] If GUI codebase was changed: Enso GUI was tested when built using BOTH
./run ide build and ./run ide watch.

I changed type_of tests to use Meta.is_same_object rather than the new function. I reused some of the logic from it though (see isIdentical) to fix wrong semantics for polyglot values. Added a bunch of tests, changed the subtyping for date/time to what was requested.
I couldn't use ==, as we all wished, to test types because that is currently broken and it is a known feature. Should be fixed once work on statics is finalized.


Reduce number of process workers in some tests to reduce time it takes to run on travis

Codecov Report

Merging #229 into master will decrease coverage by 0.02%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #229      +/-   ##
==========================================
- Coverage   85.82%   85.79%   -0.03%
==========================================
  Files          69       69
  Lines        3619     3619
  Branches      486      486
==========================================
- Hits         3106     3105       -1
  Misses        414      414
- Partials       99      100       +1

Impacted Files
Coverage Δ

petastorm/workers_pool/process_pool.py
92.98% <0%> (-0.59%)
: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 f6086fb...da37cb3. Read the comment docs.


Feature: lint less files with csslint

Closes #17
Here is an example of some of the options available in the .cssslintrc file https://github.com/twbs/bootstrap/blob/master/less/.csslintrc
Bootstrap use this in combination with csscomb, so may be worth using that too.

looking good so far :smile:

@edwardoparearyee I've added csslint rules and also configured csscomb with rules from bootstraps example, these are mainly consistent with out styleguide anyway. CSScomb will ensure less rules are formatted consistently and properties are ordered consistently.

:+1: LGTM


Add release notes for 14.1

Do we want to mention the windows_task rewrite since that should resolve a pile of longstanding issues?


open function reload, add guard when get the chatRoom to prevent crash

open function reload in QRoomList (need to override this function to implement delete conversation), add guard when getting the chatRoom variable to prevent crash when send message before room loaded completely

we cant open the func reload in QRoomList.

updated please check


Get Started with SAP Mobile Cards

Tutorial URL: https://developers.sap.com/tutorials/cp-mobile-cards-action.html
Please specify the step you are referring to : Step 5: Add Reject Action
Save step missing in step 5. Deploy a new version of the Card' can only be executed after save.
Regards
Sreejith

Hi @sreejithppsap,
Thank you so much for bringing this matter to our attention. I will add a step to save in the documentation.
We truly appreciate your support.
Warm Reards
Sandeep T D S

Thank you for your feedback. The issue seems to be resolved, so I am closing the issue.
If you still have questions, feel free to reopen the issue.


ImportError: No module named wox

After downloading and installing Wox v1.2.0-beta, I installed the Hackernews plugin using the wpm install command. When I enter the keyword hn it shows all the recent posts which is what it should do as shown below.

But when I run the main.py inside the Hackernews plugin folder, I get the message saying ImportError: No module named wox

When you run from Wox, Wox will import https://github.com/qianlifeng/Wox/blob/dev/PythonHome/wox.py automatically.


setuptools is outdated error

@Rubines you claim in #37  that you are not able to test local changes. I cannot approve an untested PR and I'm currently traveling so I'm not able to test on my end. Will try to reproduce the issue and test your PR next week

The latest change of thedockerfile.template restored the project and the project is compiling again.


LifeCycleModel notebook question

I'm trying to update the DemARKs to make them work with the most recent HARK release.
I'm confused by something in this notebook and I wonder if somebody could explain what's going on:
https://github.com/econ-ark/DemARK/blob/master/notebooks/LifeCycleModelTheoryVsData.ipynb
These are the tracked variables on LifecyclePop:
LifeCyclePop.track_vars = ['aNrmNow','pLvlNow','mNrmNow','cNrmNow','TranShkNow']

But in cell [9] there is the expression:
aGro41=LifeCyclePop.aLvlNow_hist[41]/LifeCyclePop.aLvlNow_hist[40]

How could that cell have been evaluated, if aLvlNow was not a tracked variable?

It was constructed 1 line above that, apparently.
On Mon, Jun 22, 2020 at 4:09 PM Sebastian Benthall<EMAIL_ADDRESS>wrote:

I'm trying to update the DemARKs to make them work with the most recent
HARK release.
I'm confused by something in this notebook and I wonder if somebody could
explain what's going on:
https://github.com/econ-ark/DemARK/blob/master/notebooks/LifeCycleModelTheoryVsData.ipynb
These are the tracked variables on LifecyclePop:
LifeCyclePop.track_vars = ['aNrmNow','pLvlNow','mNrmNow','cNrmNow','TranShkNow']
But in cell [9] there is the expression:
aGro41=LifeCyclePop.aLvlNow_hist[41]/LifeCyclePop.aLvlNow_hist[40]
How could that cell have been evaluated, if aLvlNow was not a tracked
variable?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/econ-ark/DemARK/issues/121, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADKRAFI5EZ44DVBTQK2MQC3RX626TANCNFSM4OE74BJQ
.

ah dumb question thank you.


Rich Text: Mechanism for custom ckeditor plugins needs to be ported to 0.6

Figure out how to do this nicely, without apos.data.

closed with aba6e329a1bb307a119f3fc762ec6fcb84ff0f01


[10.x] Update Facade::$app to nullable

Facade::$app can be null.
There are already some nullability checks within the same class (see swap() or resolveFacadeInstance()).
Making it explicit in the PHP blocks allows us to avoid errors with static analysis tools like PHPStan:
Parameter #1 $app of static method Illuminate\Support\Facades\Facade::setFacadeApplication() expects Illuminate\Contracts\Foundation\Application, null given.

@driesvints

@jose123v can you share a full stack trace?

@driesvints hi, look at this example, this fail only on 10.x https://github.com/barryvdh/laravel-dompdf/actions/runs/8084028258/job/22088401041?pr=1036


TagV2

So I don't really know how to make a draft of one of these I couldn't get the branches to compare and stuff...
I plan on adding more syntax as I see fit, read the comment at the top for the changes I have made. Indigo is my staple, please leave it as indigo.
For a demo join my smol guild: https://discord.gg/y3BH7rr74w
I think this is better fit as TagV2 since I added the syntax to it and a bunch of other things, I'll make a PR later to improve some basic things on the original tag system.
Changes:
 Syntax
 better color (indigo)
 new `;tag list` appearence
 RoleFilter on tag management
 Response is now an embed
 image support
 Each sub command has more options
 Shows who it was requested by

Syntax
I plan on adding Args support possibly idk. give a thumbs up if you like the idea.
Current
USER
{{User-String}}: User's name Crenshaw#1312
{{User-Username}}: User's username Crenshaw
{{User-Discrminator}}: 4 digits after username 1312
{{User-ID}}: User's ID<PHONE_NUMBER>30856458
{{User-Age}}: User's age 2 weeks 4 days and 1 hour
{{User-Mention}}:Mentions triggering user @Crenshaw
CHANNEL
{{Channel-Name}}: Channel name of channel pagstdb
{{Channel-Topic}}: Channel topic some topic for pagstdb
{{Channel-ID}}: Channel ID of trggering channel<PHONE_NUMBER>78650932
Example

Tag List
It now uses the css code block and is spaced out so tags don't get jumbled together and become a mess, I also had to remove certain characters do to splitting, and coloration
For now, or ever, there is no pagination via reactions. I am not a fan of it but I may add it so others can use it if the enjoy it, I'll try and get it to work with Joe's current one so a new pagination doesn't have to made.

I'm really bad at this whole pull request thing, this is a mess

glances at the mess that's known as #90

I AM DONE
@Jo3-L review and sumbit pls, leave color as indigo

could I delete ;tag info? since the ;tag list command essentially does it

Oh, just a comment, would like this "syntax" to be compatible (i.e be a subset of) YAGPDB CC if it is to be kept. I am skeptical as to the benefits as, well, users should use YAGPDB CC directly if they require actual scripting other than embeds + content, but up to you. Example:
{{ $data := sdict "User" (sdict "Username"  .User.Username) }}
{{ range reFindAllSubmatches `{{\s*((?:\.\w+)+)\s*}}` $USER_INPUT }}
    {{- $path := split (slice (index . 1) 1) "." }}
    {{- $val := $data }}
    {{- range $key := $path }}
        {{- /* can't use `and` because it doesn't short circuit */}}
        {{- if ne (kindOf $val) "map" }}
            {{- with $val.Get $key }}
                {{- $val = . }}
            {{- else }}
                {{- $val = "" }} {{- /* or another appropriate default value */}}
            {{- end }}
        {{- end -}}
    {{- end }}
    {{- /* prevent making too many reReplace call by split + join */}}
    {{- $USER_INPUT = joinStr $val (split $USER_INPUT (index . 0)) -}}
{{ end }}

Haven't tested this but that's the general idea.
I find parsing this generally a pain (other than the lazy approach demonstrated above) so that's why I'm not a huge fan of including this, again, your call to make though.

Please use the .go.tmpl file extension as of latest commit. Thanks!

Hi @Crenshaw1312, are you still interested in working on this?

interesting should be merged soon.

Stale, no activity for 2 months. Closing for now.
If you'd like to continue working on this ping me @Crenshaw1312.

Stale, no activity for 2 months. Closing for now.
If you'd like to continue working on this ping me @Crenshaw1312.


fix: Adding spot instances with AMI-ignore

Description
As per https://github.com/terraform-aws-modules/terraform-aws-ec2-instance/issues/361, replicated https://github.com/terraform-aws-modules/terraform-aws-ec2-instance/issues/239 to enable spot instances to also support the ignore_ami setting.
Breaking Changes
No
How Has This Been Tested?

[ ] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[ ] I have tested and validated these changes using one or more of the provided examples/* projects
[ ] I have executed pre-commit run -a on my pull request

thank you but unfortunately I don't think we will be taking on this additional change in the module


Store inbound messages

Adds an API endpoint that accepts inbound messages from Junebug and stores them in the database.
Migrations will be added once this is approved to prevent having to create them multiple times

@rudigiesler this is ready for re-review please


Additional derived quantities

[ ] TotalVolume
[ ] AverageVolume
[ ] TotalSurface
[ ] AverageSurface
[ ] MinimumVolume
[ ] MaximumVolume
[ ] MinimumSurface
[ ] MaximumSurface

I just started working on this issue, is there any specificities to keep in mind while doing it ?

Glad to hear! So I've opened a PR for TotalVolume #703 since I needed it for one of my branches but the rest is still left to do.
Maybe one difference with FESTIM 1 is that we would like derived quantities to be "stand alone".
In FESTIM-1 all the DerivedQuantity objects have to be bundled in a DerivedQuantities object, and this bigger object has a filename attribute to export to csv.
In FESTIM2, we would like to be able to export each DerivedQuantity independently which means they all have a filename attribute.
Later down the line, we'll think of how we can also bundle them to avoid having too many files, but I think this new architecture will be easier to deal with and more flexible.
Apart from that, I'm not aware of any specificity. I remember that finding the minimum and maximum of a solution in a subdomain in legacy-fenics was a bit tricky, I don't know how to do it in dolfinx. The fenics discourse group is your friend! 😄


feat: compatible mode

Support generating code that works with apache/thrift v0.13.0 while compiles well with apache/thrift v0.14.0 in spite of the breaking changes between the two versions by adding golang/extension/compatible.

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


gitkraken: 6.2.1 -> 6.3.0

Motivation for this change
https://support.gitkraken.com/release-notes/current/#version-630
Things done

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

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

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

Notify maintainers
cc @xNWDD

dpkg should be in nativeBuildInputs if it is only necessary to unpack deb package at build time.
Also when overriding installPhase it should start with runHook preInstall and end with runHook postInstall

Thanks!
Seems to work fine after the changes.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/explicit-runhook-in-derivation/4614/7


feat: add paysWithFeeOrigin support for MultiLocations

Summary

This updates polkadot-js to 10.11.1.
Allow for both multilocation and integer input for paysWithFeeOrigin.
Add transferAllowDeath support for balances::transfer
Update metadata for testing for system parachains

rel: https://github.com/paritytech/asset-transfer-api/issues/228


Node2D: look_at() return rotation angle

Figured it would be nice if the Node2D func look_at() would also return the angle of rotation instead of just being void.

This seems a little unnecessary. Can't you just save the original rotation, rotate it, then subtract the new rotation from the original?

I could also just implement all of this in GDscript only in very few lines, but since we have this shortcut function for the Node2D I figured it would be nice to have this shortcut too. I know it's a really small change, but if we have shortcuts we could also make them more shortcuttier :p

Can't you just call get_angle_to() in GDScript before doing the rotation?

look_at already calls to get_angle_to() so makes sense to make it more flexible, not that it would cause any severe performance drop.

I could also just implement all of this in GDscript only in very few lines, but since we have this shortcut function for the Node2D I figured it would be nice to have this shortcut too. I know it's a really small change, but if we have shortcuts we could also make them more shortcuttier :p

Oh no, believe me, I love shortcuts. But why would we need to replicate what get_angle_to() is already doing? It seems redundant, even if it is also actually rotating to that angle too. Why do we need two functions that perform the exact same function?

I'm kind of against adding return to function that do not really need it. For me it only makes sense when the operation done by the function requires to complex to compute value, which could be returned in that case. In any other situation, like this one, this only makes the API confusing.

I agree with @groud, there's no reason per se for look_at() to return an angle. It's simply not what it's meant for. Return values should only be used when what they provide is directly related to the logic they perform. Not other method in Node2D returns such value, but if we add it for look_at(), then should apply_scale() return the new scale, rotate() return the new rotation angle, translate() return the new position? That would require changing the whole Godot API to add return values which are already available by calling an existing property.

I think it just adds some semantic difference which messes up with human logic a bit:
apply_scale()
get_scale() # OK

rotate(rad)
get_rotation() # OK

look_at(pos)
get_angle_to(pos) # "FAIL", meant to get angle before `look_at`

@Xrayez But that sustains my point, get_angle_to() is not the natural result of look_at(). It's just an intermediate calculation done by the latter. There's not really more reason for look_at() to return that intermediate angle than to return a direction vector from self to the target position, or why not the distance between self and the target position.

Not other method in Node2D returns such value, but if we add it for look_at(), then should apply_scale() return the new scale, rotate() return the new rotation angle, translate() return the new position? That would require changing the whole Godot API to add return values which are already available by calling an existing property.

I get your point, you're right. But why not return any value at all in the first place? If somebody needs a specific value from some function like in this case, why do it in GDscript instead of just doing two things at once? It's not like it adds any bigger overhead or overcomplicates things. If you don't need it you won't even notice.
But it's up to you, IMHO it's not a bad idea but you know the engine better.

It's not like it adds any bigger overhead or overcomplicates things.

Actually, it will give a GDScript warning of an unused return value. Which means you need to add a #warning-ignore line, which does complicate things more.

Right, that's not really worth it. Thanks for your time.


Sourceforge down ?

Hello, problems with SourceForge is offline Msys2 not use the other available reposit please ? thanks

There are no other mirrors for MSYS2. We don't have infrastructure for this.

ok thanks , I will be patient^^

@elrhk Sourceforge should be  back now, have you been able to resolve your problem? If yes this issue can be closed. Otherwise, reading the error message, pacman db files were probably updated with wrong data which render them unusable.
Check out this thread from the mailing list for details about why this problem is there and how to solve it.

resolve thanks


EfficientV2_small  vs Resnet50

I use the EfficientV2_small as backbone.
But it seem to slower than Resnet50.
Is it normal？
I test the retinaface. And get the result.
EfficientV2_small : 19ms /per img     ,size 99.6mb
Resnet50 : 10ms /per img                 ,size 109.6mb

@alicera yes, it's normal, the two aren't close on any measure


Lab 1.3 - Minor step correction

Lab 1.3 - Import Solution
Exercise 1 - Import Fabrikam solutions
Task 1.1 - Main solution
Step: 2
Description of issue:
Step 2 says:
Make sure you are in your [my initial] Development environment.
It should say:
Make sure you are in your Dev One environment.
Also #### Step: 16
It says:
Select Import.
It should say:
Select Next, and then select Import.

Hi Tyler, there's currently some issues being investigated and remediated on Dev One so this is a workaround to use the old instructions for the time being.  It is currently anticipated that these issues should be resolved by early next week. Thanks for your understanding. - Simon


Add missing_inputs branches to non-alevin mapping

          Related, and we should probably fix at the same time:

In bulk-salmon.nf, spaceranger.nf and genetic-demux.nf we should update branching with the same pattern as af-rna.nf to prevent errors when raw data files do not exist.
Originally posted by @jashapiro in https://github.com/AlexsLemonade/scpca-nf/issues/670#issuecomment-1910462148
Adding detail here that this means changing branches for skipping mapping or quantification to check for the existence of the files directory variable and contents, and allowing for a missing_inputs branch. These updates should include sending missing_inputs branches to the log, as is done in af-rna.nf

Closed by #673


Should be first MSA-attention then Pairwise-Attention

Here, original evoformer should be first MSA-attention then Pairwise-Attention:
https://github.com/lucidrains/alphafold2/blob/2c574801bab437b25018c2591fbc06fe17dee06c/alphafold2_pytorch/alphafold2.py#L422-L428
plz have a check
Best

sorry, I didn't see the latter part.


Make the use and purpose of getRectForPosition() to be consistent across the codebase (Resolves #1614)

Make the use and purpose of getRectForPosition() to be consistent across the codebase (Resolves #1614)
It seemed that sometimes getRectForPosition() was being used to obtain a bounding box and other times to obtain a leading/trailing edge of content. This PR resolves this inconsistency by always returning a content bounding box with getRectForPosition().
Most of the changes in this PR are changes of code that uses getRectForPosition() and getRectForSelection(), but one foundational change is that BoxComponents now return their bounding box from getRectForPosition() regardless of whether the given position is upstream or downstream.
I discovered along the way that our HoriziontalRuleComponent and UnselectableHorizontalRuleComponent didn't include IgnorePointers. I'm not sure why this didn't break any tests previously, but it did break some tests in the course of this PR, so I added IgnorePointers.
I also added some test tools in this PR, as needed.

@angelosilvestre can you show me again where I can find the CI golden failure files?

@matthew-carroll You can go to the "Checks" tab and then click on the workflow. There will be a zip file at the end of the page.

@angelosilvestre I added a new document layout and component layout method called getEdgeForPosition() which returns the edge of the bounding box for the given position. That method returns the left side or right side of a box component depending on upstream or downstream.

FYI @angelosilvestre @rutvik110 I was able to get goldens passing again in this PR. My final change was to use smaller golden file sizes that still fit all the content, but aren't nearly as large as the default size.
I don't know why that worked. I still don't know why CI thought goldens were failing. When I downloaded the goldens they were identical as far as I could tell. Nonetheless, they're passing again.
Let's try to keep the goldens passing moving forward.


Revert "Connectedk8s - RBAC: Add "arc "mode in Guard for using OBO service and deprecate --app-id & --app-secret params"

Reverts Azure/azure-cli-extensions#6675

Thank you for your contribution! We will review the pull request and get back to you soon.


Cordova SQLite / Websql in-browser support

Hello there.
We have relational data that we would like to be able to represent in the same way on both the server and client. The reason we want it to work in the client is that our app works offline (with a custom sync solution). It would be absolutely amazing if this library was tested again the cordova sqlite implementation and the related websql browser equivalent. Would you ever consider this kind of thing as a goal of this project or is this kind of thing out of scope?

Hi,
websql is already supported by knex which we use for all query building and connection handling. And since websql is basically sqlite, moron.js already has tests for it. Assuming cordova's webgl implementation follows standards things should just work.
The problem is running moron.js in the browser. I'm pretty sure this cannot be done at the moment even if you use browserify. I would have to make a browser build.
If I get more requests like this in the future I will certainly consider the browser build. Until then I'll probably concentrate on other features.

That would be great indeed

It turns out, all packages moron.js uses are browser compatible. You can try running it in the browser using browserify. First create a file moron-browser.js with contents:
window.moron = require('./moron.js');

To the root rolder of moron.js. Then install browserify:
npm install -g browserify

And then run it
browserify moron-browser.js -o moron-bundle.js

The bundle can be used in a web page:
<!DOCTYPE html>
<html lang="en">
<head>
  <meta charset="utf-8">
  <title>moron.js</title>
  <script src="lodash.js"></script>
  <script src="bluebird.js"></script>
  <!-- The knex.js browser build -->
  <script src="knex.js"></script>
  <script src="moron-bundle.js"></script>
  <script>
    var knex = Knex({
      client: 'websql',
      connection: {
        filename: 'test.db'
      },
      // Disable pool.
      pool: {
        max: 0,
        min: 0
      }
    });

    function Test() {
      moron.Model.apply(this, arguments);
    }

    moron.Model.extend(Test);
    Test.tableName = 'Test';
    Test.knex(knex);

    knex.schema.createTable('Test', function (table) {
      table.bigincrements('id');
      table.string('text');
    }).then(function () {
      return Test.query().insert({text: 'test 1'})
    }).then(function () {
      return Test.query();
    }).then(function (models) {
      console.log(models);
    });

  </script>
</head>
<body>

</body>
</html>

Knex throws an error with the example above. Don't know if there is something wrong with my code, or if it is a bug in knex.

@koskimas Interesting. Thanks for looking at it. I'll take a peak later today. Excited!

Potentially related to Knex error: https://github.com/tgriesser/knex/issues/877

I'll definitely give this a try :+1:
Thanks !

There are two bugs in knex, both described in the issue @jimthedev attached. Until they are fixed you can get things running by using the version 2.9.27 of bluebird and by replacing the line:
err.message = SqlString.format(obj.sql, obj.bindings) + ' - ' + err.message;

with
try {
  err.message = SqlString.format(obj.sql, obj.bindings) + ' - ' + err.message;
} catch (ignore) {
   // `message` is sometimes a getter.
}

in the knex.js file. I also updated the example I posted before.

@koskimas Did you create a PR with the fix in Knex? Would this work long term?

Sorry, I forgot to answer this.
I didn't create a PR because I'm not familiar enough with knex's internals. My solution was a hack to make it work for our tests.
Unfortunately I cannot give any guarantees about this since I don't have tests for browsers. Making the integration tests run also on browsers seems like a lot of work, and I don't think I have time to do that any time soon.

None of my business, but i suspect this issue would better be labelled as "won't fix" instead of being closed.

I can add the wontfix label, but I don't want to keep these issues lying around forever when no-one seems to be interested in them.

@jimthedev , did you ever get this working? I'm still looking for the best ORM for CordovaSQLite and this solution looked promising, except all the working around of the Knex defects.


tests(behavior): add test for writer write overwrite

fix #4790

Now the behavior tests can run on PR. I have no idea about what happens to sftp.

Seems sftp does have some issues:
---- behavior::test_writer_write_overwrite ----
test panicked: assertion `left == right` failed: read content_two
  left: "eec0453ded52696abe7e34987d12d89daa382e5d70c0d0271c3ccfd4706e5245"
 right: "4e4c5edbadda8d42c622f28de6b643397820abfc37c6e35eeed67a360cb3f65e"

failures:
    behavior::test_writer_write_overwrite

test result: FAILED. 84 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 21.25s

I will double check there. Feel free to dive in if you are interested too.

Wow, perfect!


Link off to javadoc

Not my favorite here, but hopefully it'll start a conversation.
Refs #6

We should keep this use case in mind when we add references to the SDKs.


elastic 2.x

hi,thanks a lot for this repository.
will it support elastic 2.x? I'm asking because I know that elastic 2 'update' works differently
Lior

Hi, actually, I am not planning to update the repository. I do not have access to the corresponding blog posts so I could not change the content there.
However, Flink 1.1 comes with an Elasticsearch2 connector which might be helpful: https://ci.apache.org/projects/flink/flink-docs-release-1.1/apis/streaming/connectors/elasticsearch2.html


Laravel 5 Validations

Hi,
Thanks a lot for this nice package.
I'm using Laravel 5.1, please is there a way to validate fooTranslation attributes ?
For example I have the attribute 'name' required and unique in 'foo' model, so how can I update it with this validation ?
Example :
class Foo extends Model
{
use  Translatable;
protected $table = 'foos';
public $translatedAttributes = ['name', 'description'];

Does the normal Laravel Validation not work for this?
https://laravel.com/docs/5.1/validation

You should use the dot notation like fr.name, en.name, de.name or with laravel 5.2 you can use the star : *.name, *.description

Thanks for answer @sdebacker
Please could you post an example ^^

Using a Form Request class:
class FormRequest extends AbstractFormRequest
{
    public function authorize()
    {
        return true;
    }

    public function rules()
    {
        return [
            '*.name'        => 'max:255|required',
            '*.description' => 'required',
        ];
    }
}

Thanks a lot,
but what about unique validation ? for example the name  is it unique for model Foo or FooTranslation ?

I have '.slug' rule in a translatable model. I want validate if its unique combination 'locale' and 'slug' how can i do that:
'.slug' => 'required|unique:category_translations,slug,NULL,id,locale,'.?¿?¿?¿¿
Thanks

have slug rule in a translatable model. I want validate if its unique combination 'locale' and 'slug' how can i do that:
'*.slug' => 'required|unique:category_translations,slug,NULL,id,locale,'.?¿?¿?¿¿
Thanks

@asisti2-pruebas have a look at https://github.com/felixkiss/uniquewith-validator/

please check the laravel docs for validation.

I use '*.title' to validate title, it validated normally, but it creates other fields .title that exist in the table.

Did you find a solution to thisissue @hugofabricio

I came up with this solution:
 'sometimes|nullable|integer|min:0',
            'starred' => 'sometimes|boolean',
            'image_id' => 'sometimes|integer|exists:images,id',
            'page_id' => 'sometimes|integer|exists:images,id',
            'view_id' => 'required|integer|exists:views,id',
        ];

        $trans = [];

        foreach ($locales as $locale) {
            $trans = $trans + [
                $locale . '.title' => 'required|string|min:3|max:60',
                $locale . '.description' => 'required|string|min:3|max:155',
                $locale . '.slug' => 'required|string|min:3|max:200|unique',
            ];
        }

        return $page + $trans;
    }

}

this should return the required complete array for validationr

Has anyone found a solution on how to validate uniqueness?

@anaisabel19886 Validation rule unique requires at least 1 parameters
You're missing the table name

@stefkes
duh! you are absolutely right! :)
Anyway, I ended up making a custom validator since I needed something more specific..

I see i late, but maybe it can help someone in the future.
For validate translatable fields with unique,
in Laravel 5.3 i can use this custom syntax for unique, and it working:
public function rules()
{
    return [
        'ru.name' => 'required|max:255',
        'ru.slug' => [
            'required',
            'max:255',
             Rule::unique('category_translations', 'slug')
                 ->where(function ($query) {
                     $query->where('locale', 'ru');
                 })
        ]
    ]
}

@stefkes If you've found something in this time, I would be curious and happy to see it. Thanks!

@arturmamedov - I had a solution but I replaced it with your much cleaner method.

To bring this up again: do you think that we should have a validation rule in this package? Or provide a validation package for this one?
And if so: which cases should be covered/possible with the rule?

As announced here, the project has been deprecated. Please use astrotomic/laravel-translatable.


Ad company name, dependencies and assets

This code allows to add assets company name and dependencies to a nativefier project in roder to enhance an existing web site with local scripts and ressources.

@zucher do you still want to merge this? If yup, please:

fix the lint errors (check for errors with npm run lint)
rebase (note that https://github.com/jiahaog/nativefier/pull/417 worked on companyName, you might have conflicts)
and re-push.

@ronjouch Is usage of Promise possible ?, in that case, can I move the project to be ES6 compliant ?

@ronjouch what about code duplication, I have the same code between buildApp and gulp helper

Is usage of Promise possible ?, in that case, can I move the project to be ES6 compliant ?

@zucher the more ES6, the merrier, as long as our CI passes on Node 4.x

@ronjouch seems ok now ;-)

Seems to be ok now @ronjouch, except for the npm version.
Regards

Seems to be ok now @ronjouch, except for the npm version.
Regards

@zucher, is this ready for another review? Looks like @ronjouch has requested changes (I cannot predict if changes since satisfy those requests).

Do you still plan to work on this feature? It would be great if we can modify the company name!


Enable GCP alerts during business hours

Task
Enable GCP alerts during business hours in order to get a feeling on what issues CAPG clusters have.

Prometheus needs to have CSI working with workload identity. @alex-dabija create separate issue.

The separate issue to have CSI working with workload identity: https://github.com/giantswarm/roadmap/issues/1453 .

Blocked by: https://github.com/giantswarm/roadmap/issues/1453 .

@alex-dabija this is done as I see the team working on the alerts but can you confirm and we close? Thanks!

GCP alerts are enabled and working.


Binary Files in Secrets with Google Secret Manager

godaddy/Kubernetes-external-secrets:6.0.0 Controller fails to if the secrets uploaded to the Google secret manager is a binary file and the creation of Kubernetes secrets are encoded from hexadecimal that is encoded by google secret manager.
In a nutshell, my question is How to use godaddy/Kubernetes-external-secrets with binary data on Google secret manager?

Provide example of your external secret  :))
Did you set isBinary: true for the key with binary data?

Hi,
Thank you for the reply my case I am trying to store .p12 binary file and create the k8s secret with external secrets.  Can you let me know how I can set isBinary: true  as I could not find a way where I can enable.
Thanks Much.

https://github.com/external-secrets/kubernetes-external-secrets/blob/master/examples/hello-service-external-secret-gcp.yml

Hi Flydiverny,
Thank you for the suggestion, and it works for us. The tricky part here is on google secret value, I have to provide base64 coded value of binary data and set isBinary: true.
Once Again thank you.


Trajectory timing/smoothing refactor 2.0

This pull request finishes (for now...) the trajectory timing/smooth refactor that I started in #64 and #78. It also addresses the issue that @garthz raised in #73.
Here's a quick summary:

Added a new PostProcessPath function with three different post-processing pipelines:

for smooth trajectories (e.g. CHOMP), approximate the trajectory as a continuous curve
for constrained trajectories, stop at every waypoint and exactly follow the geometric path
otherwise, by default, shortcut the path and simultaneously smooth/retime it

PostProcessPath takes separate shortcut_, smoothing_, and retiming_options dicts (as per #73)
We determine which pipeline to use (i, ii, or iii) using the tags on the trajectory (#78)
Note: We current have no method of doing (i), so it temporarily falls through to (ii) and (iii)
Modified ExecutePath to call PostProcessPath, followed by ExecuteTrajectory
Added docstrings for PostProcessPath, ExecutePath, and ExecuteTrajectory

PostProcessPath (and, thus, ExecutePath) do not correctly handle base trajectories. I want to get @jeking04's input on this before I proceed, but I expect to:

Check if any affine DOFs are in the path passed to PostProcessPath; if:

Only regular DOFs use the current behavior
Only affine DOFs call RetimeAffineDOFTrajectory
Both are present raise an exception

Call ExecuteTrajectory as normal (I believe this already handles base trajectories)

Finally, there a few obvious next steps. I will create separate issues for these:

Implement a method retiming smooth paths; i.e. case (i)
Tag the trajectory with constraint TSRs
If a trajectory is tagged with constraint TSRs, use CBiRRT to do the smoothing/timing

@siddhss5 @psigen @jeking04 Thoughts?

We were just waiting for @jeking04 to collect data before merging this. I am going ahead and doing the merge now.


Media: Avoid repeated backfilling logic

Feature Description

In the local media provider we currently have lots of code duplication related to logic for backfilling media metadata (poster image, video's audio information, base color, and soon blurhash).
There's an opportunity to make this more DRY, in a way TBD.
Alternatives Considered

Additional Context

I believe this was handled by https://github.com/google/web-stories-wp/pull/9638

I thought we were originally talking about this:
https://github.com/google/web-stories-wp/blob/c4b6efa369aa450bbdb9191c68be2a9c1d9f8086/packages/story-editor/src/app/media/local/useContextValueProvider.js#L156-L209
Which is still very repetitive

It is very repetitive, but each check is different. I can't see a nice way to refactor this.

Let's close this for now then


Replace validation.js

validation.js is still used for e-Mail address check and badge ID number check.
The new e-Mail address validation should use JavaScript compatible regular expression from HTML 5.1 recommendation: https://www.w3.org/TR/2016/REC-html51-20161101/sec-forms.html#valid-e-mail-address
Badge ID number check should use a regular expression for accepting an unsigned integer.

Done in commit "Replace validation.js" on dev020 branch on 2017-01-31.


Change case for windows

Description of the new feature / enhancement
There is one extension from Chrome called Change Case, it would be awesome if something similar could be achieved with PowerToys across whole system.
Scenario when this would be used?
Everywhere there is text editing.
Supporting information
No response

Thanks for your message. This is a Duplicate of an other issue. Additional comments may be added there. /dup #20225


Build Configuration Failed

Hi @c0de-wizard , it is me again (I swear this is not a witch-hunt).  First, thank you so much for this project, it serves as an awesome reference. I was using your project to learn how to go about managing dependencies using gradle catalog, unfortunately the build is failing with a weird error. Similarly, the error can be reproduced in your other project [Stargazer]; for some reason android-studio cannot
find the reference of the android-app-plugin.
Script compilation error:

  Line 2:     `android-app-plugin`
              ^ Unresolved reference: `android-app-plugin`

If you do manage to look into this issue, thank you so much in advance.

Hi @GibsonRuitiari It's all good. 🙂  I forgot to commit this file and that's why it wasn't compiling. I have added it and updated some dependencies. Should be good not. Let me know if you run into any other issues.


Add "Set up environment" module

Description
A new module is created to guide the reader through the installation of the required software and the setting up of the development environment.
Create this new module required to move some of the existing chapters to different parts of the manual, including the new module.
Testing
The content was tested manually. The test was merely to check that the content of the manual was rendered as intended.
Is it Done?

[X] I (developer) have tested the changes manually
[ ] Reviewer has finished the code review
[ ] After the review, the developer has made changes accordingly
[ ] Customer/Teacher has accepted the implementation of the feature

I had to force-push. There were some problems with my branch, But I have applied all the required changes.

Yes, the extension will keep the publisher, otherwise, the extension will stop working for those who have already downloaded it. But, the new publisher "apluslms" should be able to completely manage the extension. I sent another email asking about changing the URL, but I have not gotten a response.