OOP school library: decorate a class

Hi there 👋,
In this particular project, I :

Check the usage of Decorator pattern in Ruby
Think about how I can use two decorators in order to capitalize and trim people's names.
In the following steps I will get instructions for implementing the Decorator design pattern for this case.

Interface

Create a class Nameable.

Implement a method called correct_name that will raise a NotImplementedError.

Turn my Person class to Nameable

Person class inherits from Nameable
This class has a method correct_name implemented. It simply returns the name attribute.

Prepare base Decorator

It inherits from Nameable.
In the constructor assign a nameable object from params to an instance variable.
Implement the correct_name method that returns the result of the correct_name method of the @nameable.

Prepare CapitalizeDecorator and TrimmerDecorator

For the CapitalizeDecorator:

Create a class that inherits from the base Decorator class.
Implement a method correct_name that capitalizes the output of @nameable.correct_name.

For the TrimmerDecorator:

Create a class that inherits from the base Decorator class.
Implement a method correct_name that makes sure that the output of @nameable.correct_name has a maximum of 10 characters. If it's longer it should trim the word.

See my decorators in action
\Try the following code and check if you managed to decorate your person:
person = Person.new(22, 'maximilianus')
  person.correct_name
  capitalizedPerson = CapitalizeDecorator.new(person)
  capitalizedPerson.correct_name
  capitalizedTrimmedPerson = TrimmerDecorator.new(capitalizedPerson)
  capitalizedTrimmedPerson.correct_name

Hi @Olivier-Kango, kindly add the key features and future features section to your readme

Hi @Olivier-Kango
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You have submitted a project, that  Kindly add the key features and future features section of your app to your readme, all sections from the microverse template should be present in your readme
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification @prolajumokeoni.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.


add instance id configuration item

Please answer these questions before submitting pull request

Why submit this pull request?
[ ] Bug fix
[x] New feature provided
[ ] Improve performance

Done!


Hi 👋 is this project still maintained somehow?

Hey 👋 It is. It hasn't seen much activity simply due to the fact that it satisfies its functional goals.
There are one or two bugs, but they're being addressed already. I have a 3.0 in the works locally (and have had it for some time), it's just been a lower priority on my TODO list since it is already in working order.
Thanks for making sure!

It hasn't seen much activity simply due to the fact that it satisfies its functional goals.
Yep, still use it actively at work and my own projects :P


Can I get email notification if new RSS feeds are loaded with contents?

Newsboat version (copy the output of newsboat -v or the first line of git show):
Version is 2.18.0
Can I get a update on emails if new feeds are found or observed?

You could use notify-program config option.

I'll add one more pointer: How to send a mail directly using Mutt?. There are of course other programs too, e.g. sendmail.
Also, depending on what you want, rss2email might be a better fit than Newsboat.
I think this answers the question, so I'm going to close this, but feel free to comment below—we can always re-open.


feat(webp): use ril with rust-native image-webp feature

Closes #133, moving all the dependencies to native Rust instead of binding to libwebp-sys.

was just testing this out and noticed it's quite a lot faster than catwalk on main:
Benchmark 1: ./catwalk-main
  Time (mean ± σ):      1.216 s ±  0.010 s    [User: 1.180 s, System: 0.034 s]
  Range (min … max):    1.193 s …  1.231 s    10 runs

Benchmark 2: ./catwalk-winston
  Time (mean ± σ):     405.3 ms ±   5.7 ms    [User: 370.4 ms, System: 34.9 ms]
  Range (min … max):   396.7 ms … 418.2 ms    10 runs

Summary
  ./catwalk-winston ran
    3.00 ± 0.05 times faster than ./catwalk-main

nice!

I'd like to update this PR with my latest changes on my ril PR, but I don't have write permissions to this branch anymore.

I've added you as a Maintainer. c:

@backwardspy

was just testing this out and noticed it's quite a lot faster than catwalk on main:

I can pretty much replicate these results 1:1 on my own x86_64 hardware, but it seems like the compression factor is worse?

Here I'm using the PR version of catwalk to generate preview.webp in the VSCode port, preview-before.webp is the current version, and then I reencode it with ImageMagick's convert.
The ImageMagick version has the smallest resulting size, catwalk with this PR encodes images in a way that's usually 2-3x larger than the main branch. 🤔

that is interesting, i didn't think to check file size.
i get similar results with imagemagick coming out on top:
296K    preview-magick.webp
304K    preview-cwebp.webp
304K    preview-main.webp
628K    preview-feat-ril-webp.webp


Integration with Adidas Runtastic

I want to upload runs (maybe other indoor activities as well?) to Adidas's Runtastic's portal.
Here is activity uploading: https://runtastic.stoplight.io/docs/public-api-docs/1f94a4ec016b1-create-activity
It supports both FIT and TCX, I'd rely on FIT.
Supported session message attributes:

timestamp
start_time
sport
sub_sport
total_timer_time
total_elapsed_time
total_distance
total_calories
total_ascent
total_descent
max_speed
enhanced_max_speed

Supported record message attributes

timestamp
cadence
heart_rate
speed
enhanced_speed
altitude
enhanced_altitude
position_lat
position_long
distance
temperature

Supported event message attributes

timestamp
event
event_type

Note, that it doesn't support watt readings, so we may need a FIT file which doesn't contain that?
For authentication I'll need to obtain CLIENT_ID and CLIENT_SECRET.
https://runtastic.stoplight.io/docs/public-api-docs/bfcb4e574b11d-authentication
I wrote to<EMAIL_ADDRESS>
Subject: Mobile app uploading FIT/TCX activities to Runtastic
Body: Dear Runtastic Partner API,
I have a mobile app which currently has integrations to some major training platforms (including Strava, MapMyRun, SUUNTO, TrainingPeaks).
I'd like to add activity upload support for Runtastic. Questions:

Do I need to register an API app to obtain a CLIENT_ID and CLIENT_SECRET for authentication? This could be needed so I can obtain a Bearer token.
Since this is a mobile app the redirect address would have a fictious protocol, do you have such partner apps? (the training platforms above support that with no problem, but Google Health does not so far).
Where can I register my app to get the CLIENT_ID and CLIENT_SECRET?

Let me know if you have any questions about my app.
https://trackmyindoorworkout.github.io/
(Although my app is mainly for indoor workouts, it generates a GPS track based on the running pace (just like Kinomap or some other apps do))
Thanks,
Csaba Toth

So far I hear crickets from Adidas...

Maybe we should try this: https://runtastic.stoplight.io/


New version: OISF.Suricata version 6.0.12

- [X] Have you signed the Contributor License Agreement?

[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?

[X] Does your manifest conform to the 1.4 schema?

Resolves #109899

Microsoft Reviewers: codeflow:open?pullrequest=https://github.com/microsoft/winget-pkgs/pull/110511&drop=dogfoodAlpha

@wingetbot waivers Add Validation-Executable-Error


Poor quality after down sizing

What are you trying to achieve?
Resize and compress images so they can be used on a website. The target width they'll be displayed at is 800 px. I was trying to achieve the best visual quality / file size ratio and noticed that it is actually better to scale the image way above the final display width while using much lower output quality. In other words it is much better if the image is scaled down by the browser at render time than by the Sharp library before hand.
For example I took this original image and resized it with
sharp('./input/3.jpg').jpeg({quality: 100})
        .resize(800)
        .toFile('./output/3_800_100.jpg')

This resulted in a 814 KB file. When viewing it side by side to the original with 800 physical pixels the original looks much better.
Why am I loosing quality despite the quality parameter being at its maximum value? Why is the Sharp's downsizing producing lower quality than the computer's downsizing?
Then I resized the original file with
sharp('./input/3.jpg').jpeg({quality: 60})
        .resize(1500)
        .toFile('./output/3_1500_60.jpg')

This resulted in a 375KB file. When displayed with 800 physical pixels this file looks significantly better than the former one while also being significantly smaller in bytes.
I was expecting, that when exporting image to be displayed on 800px display, the best approach is to export an 800px wide image and then tweak the compression ratio to an acceptable level. However, the above observation leads me to assume, that from some point it is impossible to achieve higher levels of visual quality without also increasing the pixel width. Interestingly, that doesn't necessarily mean the file size (in bytes) has to increase, because we can use more aggressive compression to (even over) compensate for the increase in pixel width.
Is my assumption right?  If yes, why? Why can't I simply resize jpeg image to 800px so that it looks the same as viewing 2500px wide jpeg image on 800px screen? Is this a fundamental property of jpeg format, or is this caused by the algorithms Sharp uses for downsizing?
Btw: here's my repo with a script I used to generate various size/quality variations of several images for comparison. It also contains the exported files I linked above.
Have you searched for similar questions?
Yes
https://github.com/lovell/sharp/issues/2140 -> mentioned suggestions have no effect
Are you able to provide a minimal, standalone code sample that demonstrates this question?
See above.
Are you able to provide a sample image that helps explain the question?
See above.
Thank you.

Hi, did you see the kernel and fastShrinkOnLoad options of resize() that alter the way in which images are resized?
https://sharp.pixelplumbing.com/api-resize#parameters

Yes. Using fastShrinkOnLoad: false had no visible effect. Out of all the kernels the default lanczos3 is subjectively the best looking one.

Just found out this article that confirm my findings and explains them with

The bottomline is that heavy compression doesn’t affect the final image as much as you would expect. This is because of the greater amount of pixels in the [bigger in pixel size] image, compression artifacts are scaled down and therefore almost unnoticeable.

Thanks for the update, glad you worked it out.

For anyone stumbling upon this: see also this SO question, which pretty much explains all my questions above.


AMMFactory should create a mapping of [market][sharetoken] to AMM

This is easier for everyone to look it up and is cheaper.

Duplicate of 9263


Does it support face recognition?

Not yet. Delta supports NLP and speech tasks now.

It is easy to support face recognition, welcome to contributing.

How to support face recognition? Like this: https://github.com/AKSHAYUBHAT/DeepVideoAnalytics
------------------ 原始邮件 ------------------
发件人: "Hui<EMAIL_ADDRESS>发送时间: 2019年8月14日(星期三) 下午2:54
收件人<EMAIL_ADDRESS>抄送<EMAIL_ADDRESS><EMAIL_ADDRESS>主题: Re: [didi/delta] Does it support face recognition? (#14)
It is easy to support face recognition, welcome to contributing.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.


multiple ng-select with size attribute freezes in IE 11 and Edge

I have 5 different selects with size=10 to display as listbox.  Code similar to
<select size="10"
  class="selectFilter"
  ng-model="selectedWorkload"
  ng-options="workload.Title for workload in workloads"></select>

I have a table with an ng-repeat similar to
data-ng-repeat="contentpack in contentpacks | filter: selectedProduct.Value | filter: selectedWorkload.Value | filter: selectedOfferingType.Value | ....
the code runs fine if I don't have the size=10 and filters properly.  adding the size=10 jams up IE11 and Edge Browsers and page does not properly load.

Can you provide a demo in a plnkr.co or similar?

I think this might be the same as https://github.com/angular/angular.js/issues/11314 which I just fixed on the current snapshot

I'm pretty sure this was fixed in 1.4.7. Please reopen if you think this is till an issue.


Bumps frameworks to latest

chore: bump

Bumps open-aea to ==1.50.0
Bumps open-aea-ledger-ethereum to ==1.50.0
Bumps open-aea-ledger-ethereum-flashbots to ==1.50.0
Bumps open-aea-ledger-ethereum-hwi to ==1.50.0
Bumps open-aea-ledger-cosmos to ==1.50.0
Bumps open-aea-ledger-solana to ==1.50.0
Bumps open-aea-cli-ipfs to ==1.50.0
Bumps open-autonomy to ==0.14.10
Bumps open-aea-test-autonomy to ==0.14.10

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.78%. Comparing base (5fdd71d) to head (69c07fe).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #132   +/-   ##
=======================================
  Coverage   94.78%   94.78%
=======================================
  Files          75       75
  Lines        2357     2357
=======================================
  Hits         2234     2234
  Misses        123      123

Flag
Coverage Δ

unittests
94.78% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


DX: remove unused FixerFileProcessedEvent::STATUS_UNKNOWN

Coverage remained the same at 92.913% when pulling 432c17a359c1592d3fd57e62ee4970121c3d541b on keradus:unknown into 953b8380de208061f0d7d2aa4ec4dbc775e0a577 on FriendsOfPHP:master.

nice, please fix the tests related to printing the legend and lets ship :+1:

done @SpacePossum !


Do not cache bad ufs client

What changes are proposed in this pull request?
Do not cache bad ufs client.
Why are the changes needed?
Fix #17212.
Return the created client instead of handling exceptions at the caller. At the same time, in order to avoid caching the bad client, check the client in both caches.
Does this PR introduce any user facing changes?
Please list the user-facing changes introduced by your change, including

change in user-facing APIs
addition or removal of property keys
webui

@jja725 Do you mind take a look? Thanks!

@ssz1997 Handling on the caller is fine, but there are many callers. Calling exists twice is just to not cache the bad client. Another way to consider is to remove mUfs. I don’t know if this will affect performance.


Max line length mismatch

🐛 Bug
.pep8speaks.yml has max-line-length set to 119 whereas it is 120 in pyproject.toml
Expected behavior
line-length should be consistent

mind send a fix PR? :]

Sure!


Roadmap request

I am interested in using this library, i noticed there are some breaking changes in master (between v0.2.4 and master) are there many more breaking changes planned for the next release? or have you thought about what might be left before you get to version 1 of this library?

There is no roadmap, the code is quite stable and used in production. The minor version change (for example 0.2.4 -> 0.2.5) is BC compatible (should be at least). We raise the patch version when we have to do BC break (for example 0.2.5 -> 0.3.0).


Better error handling for systems

What problem does this solve or what need does it fill?
Handling Result<T, E> inside systems currently feels a bit awkward. The few options we have right now:

Call .unwrap() everywhere. This feels very unidiomatic in Rust.
Write custom log messages everywhere. This feels very boilerplaty for what could've been a single ?.
Have your system return Result<T, E>, then .pipe() it into bevy::utils::{warn, error, ...}.

I'm liking the third option the most. But the problem is that the log messages does not include any traceback information whatsoever, such as the name of the system where the error originated from. This makes it very hard to debug.
What solution would you like?
The usability of bevy::utils::{warn, error, ...} should be improved to make it easier to work with.
Or maybe there should be a built-in support for systems that returns Result<T, E>.

Another option is to use bevy_mod_sysfail (disclaimer, I am the author)

This has been discussed extensively in the past. I'm surprised you are actually the first to open an issue regarding error handling in bevy systems.
The best you are likely to get for now are the pipe error handling systems and bevy_mod_sysfail.
Regarding the traceback in the pipe error handling systems, there is an open issue: #8638

Thanks for the crate! I tried it out and the log messages now displays the module the error originated from, which is way better than the default .pipe(). But it still does not log the actual name of the system. Is there a way to configure this?

A future release will automatically add the system name to the log message. It's going to be published early next week, see https://github.com/nicopap/bevy_mod_sysfail/pull/5

I'm closing this as a Duplicate of #10874 since it's really just asking for the same thing.


GitHub changelog generator attempt

Pivotal story
This PR adds a CHANGELOG to this terraform module that has been built using the github-changelog-generator tool.
Also adds a note about how to do it.

Makes sense to me @eeeady. I'll add that note.


Make e2e ping check compatible with Ubuntu

Ping output differs a little on Ubuntu and CirrOS, let's support both

This change is

Reviewed 1 of 1 files at r1.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable


could not import github.com/go-flutter-desktop/go-flutter

Do in need to go get go-flutter to fix the following issue?

But when I try to  go get -u github.com/go-flutter-desktop/go-flutter
I get following error
# github.com/go-flutter-desktop/go-flutter/embedder
/usr/bin/ld: cannot find -lflutter_engine
collect2: error: ld returned 1 exit status

I have not issues with comipling withhover run or hover build
Thx

I am still getting error but all of sudden issue seem to be fixed

Now I have quesion how and where we are importing option variable from option.go file, because setup works. Is somthing wrong with linter?

I can see that you are using LSP, the gopls is famous for been a bit freakish.
The issue from: go get -u github.com/go-flutter-desktop/go-flutter occurs because your CGO_LDFLAGS isn't correct. When using the tool hover, we are setting the CGO_LDFLAGS to the correct path (on linux: export CGO_LDFLAGS="-L/home/drakirus/.cache/hover/engine/linux/")
For linting I use, ALE, it works out of the box. Having linting error in main.go isn't a issue as the file souldn't be modified. You don't seems to have LSP errors in option.go that what matters the most.

If you don't have any other question, or if my answer is enough, please close the issue. :wink:

Hello ! I also work in VSCode, and I have this "LSP error" in all the go files where "github.com/go-flutter-desktop/go-flutter" is imported.
Setting CGO_LDFLAGS env variable doesn't change anything to that.
Any idea ?

Hello ! I also work in VSCode, and I have this "LSP error" in all the go files where "github.com/go-flutter-desktop/go-flutter" is imported.
Setting CGO_LDFLAGS env variable doesn't change anything to that.
Any idea ?


Cannot install istio-init

In "Install the Istio CRDs on AKS" section, I was unable to run "helm install install/kubernetes/helm/istio-init --name istio-init --namespace istio-system".
This was the error received: "Error: failed to download "install/kubernetes/helm/istio-init" (hint: running helm repo update may help)"
To fix this I have cloned the repo of istio in my workspace and launch the command from the repo.
This could be a solution for who run in my same problem.

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

ID: 68b5d00d-529b-a8be-47bf-773c967f0e72
Version Independent ID: 0cf044ef-b26f-a8fa-213f-1e3fffaf53d6
Content: Install Istio in Azure Kubernetes Service (AKS)
Content Source: articles/aks/istio-install.md
Service: container-service
GitHub Login: @paulbouwer
Microsoft Alias: pabouwer

@Ettores88 Thanks for the feedback! We are currently investigating and will update you shortly.

@Ettores88 I think you missed the note under the "Install the Istio CRDs on AKS"
Same note is available in the Istio install section also.
This issue is already reported #26733. After that Author added the node under Install the Istio CRDs on AKS.
Refer to the authors comment here.

@Ettores88 i am closing this issue because the doc is already updated with the required details.
If you need additional help please mention me in the comment


geo-callback only run once.

Changes in the value of position or center moves the marker or map, but the function in geo-callback is called only once.

Example
<!DOCTYPE html>
<html lang="en" ng-app="app">
  <head>
    <meta charset="UTF-8">
    <link rel="stylesheet" href="style.css">
  </head>
  <body ng-controller="controller as ctrl">
    <div map-lazy-load="https://maps.google.com/maps/api/js">
      <ng-map center="{{ctrl.location}}"
              geo-callback="ctrl.print('geo-callback')">
      </ng-map>
    </div>
    <input ng-model="ctrl.location"></input>
    <script src="https://opensource.keycdn.com/angularjs/1.5.3/angular.min.js"></script>
    <script src="ng-map.min.js"></script>
    <script>
      angular
        .module('app', ['ngMap'])
        .controller('controller', function () {
          this.location = 'Statue of Liberty';
          this.print = function (message) {
            console.log(message);
          };
        });
    </script>
  </body>
</html>

I have the same problem, here is an exemple: http://plnkr.co/edit/mijd1N?p=preview
The geo-callback is called only once, when the script loads.

I am trying to find the source of this bug.
It seems that the whole linkFunc function (doesn't matter which directive it belongs to - map, marker...) is only called once, whatever you try.

I found something interesting. When options are modified on a directive (like the position of a marker), callbacks will not be triggered. But if you delete this marker and create a new one with the new position, it will work just fine... once.

geo-callback is not designed to be used to callback of all geo related properties. It's only used for once when map or marker is loaded.
Other properties can be watched with {{ }} because it can be mapped to google function.
position -> setPosition
center -> setCenter
However geo-callback is not done that way. I cannot put all geo-callback to all possible geo related properties. It's simply too much. :(
PR would be appreciated though.


[v3] Separator not respecting order in sidebar

Describe the bug
Regression, used to work in v2.
Following file structure:
pages/overview.mdx
pages/getting-started.mdx
pages/about.mdx

Create the following _meta.js, where separator follows the top item with no item after that:
export default {
  "overview": "Overview",
  "---": {
    "type": "separator"
  },
}

Expected
Sidebar should be:
Overview
---separator---
- rest ordered alphabetically

Actual
Sidebar becomes:
Overview
- rest ordered alphabetically
---separator---

can confirm, need to fixed here somewhere https://github.com/shuding/nextra/blob/a742821f3c125a6bd2e300ba1b5411be6c2320c8/packages/nextra/src/client/normalize-pages.ts#L166-L202

@arno-fukuda Ok, seems I fixed it after big refactoring in https://github.com/shuding/nextra/pull/3512, I will release canary instead of stable first, please definitely try it if I didn't miss something and share your feedback
Also I found a way to fix another issue to validate _meta keys https://github.com/shuding/nextra/issues/2664

try nextra-theme-docs/nextra 3.1.0-canary.0

Thank you for the quick turnaround 🙇
Tested the canary.

Separator location is now correct for sidebar.

had a redundant object in _meta.js that was ignored previously but failed dev this time.
This is in combination with a 404.js file. Deleting the below line corrected it.

// _meta.js
export default {
   ...
  "404": {
    "type": "page",
    "display": "hidden",
    "theme": { "layout": "raw" }
  }
}

[breaking] _meta.js menu items are missing.

// _meta.js
export default {
  "getting-started": {
    "title": "Getting Started",
    "type": "menu",
    "items": {
      "setup": {
        "title": "Setup"
      },
      "best-practices": {
        "title": "Best Practices"
      },
      "tutorials": {
        "title": "Tutorials"
      }
    }
  }
}

<img width="134" alt="Screenshot 2024-10-20 at 11 35 14" src="https://github.com/user-attachments/assets/a9285f8e-a75f-4e72-831d-39fff39719aa">
You should be able to repro with docs template on the version menu:
https://github.com/shuding/nextra/blob/6511ed4cf45ffe68c08b1446941dd5a6543e28d4/docs/pages/_meta.ts#L20

You should be able to repro with docs template on the version menu:

hm I can't repro this in https://nextra.site/
Can you help me repro it, so i will found how to fix it :)

@dimaMachina
https://github.com/arno-fukuda/nextra/tree/ISSUE-3485

Hi @arno-fukuda try nextra/nextra-theme-docs 3.1.0-canary.1, should be good now, thanks for reporting!

works now 👍 Thanks as always
Will update our app when its out of canary

did a stable release 👍


MM-30258 Fixing title text in select team icon.

Summary
Fixing title text in select team icon.
Ticket Link
https://mattermost.atlassian.net/browse/MM-30258

Cherry pick is scheduled.


Bug: false value is blocked in required rule

Direction
This is a bug question. I had found the error cause.
Is the empty function in required rule.
A disclaimer: You can clean more the code to a best development experience.
Describe the bug
When I use the required rule, the value false is blocked.
CodeIgniter 4 version
The composer version presents this error.
Affected module(s)
The module Validation/Rules is the only affected.
Expected behavior, and steps to reproduce if appropriate
To reproduce this error, you can create a database table with a boolean value or number in a field.
Before, you try add false  value in the field.
Context

OS: [Manjaro Linux, latest]
Web server [spark server, latest]
PHP version [8.0.8]

@OzRib please add example to reproduce error.
In fact, boolean columns is supported only in mysql and stored and work as tinyint (1).

Hello @WinterSilence .
An example?
Try it:
create a SQL table in your database with a boolean column
create table foo(bar boolean not null);

after, create a model to the table
<?php

 namespace App\Models;

use CodeIgniter\Model;

class Test extends Model
{
  protected $table = 'foo';
  protected $validationRules = [
    'bar'=>'required'
  ];
  protected $validationMessages = [
    'bar'=>[
      'required'=>'You can insert a value in bar'
    ]
  ];
}

If you try to insert a false value with the save, insert or another similar function you get the configured error

I have started a fork fixing this @WinterSilence .
Can I send a pull request? Is finished.

@OzRib

Can I send a pull request? Is finished.

why not? (:

Sended (:

@OzRib hmm.. strange solution:

it's stored as tinyint and CI check it as int
your solution is hmm.. "strange"

public function required($str = null): bool
{
    if (is_object($str) || is_bool($str)) {
        return true;
    }
    return is_array($str) ? ! empty($str) : (trim($str) !== '');
}

I tried to turn this more readable. But if is strange I change this. No prob!

@OzRib it's less readable, work slower and BC...

@OzRib  problem not in validation, you need add pre-filter - look to Model::$beforeInsert/Model::$beforeUpdate`

@WinterSilence Okay! Working

See the discussion on the PR, but this is actually working as intended (as far as Validation is concerned): false should not pass required.
There are some other discussions about how Query Builder and the database layer should handle boolean values that might be a more appropriate place for this issue.

Okay


Re-introduce app provided model view override

Closes: #3711
What's Changed

TODO

[ ] Explanation of Change(s)
[ ] Added change log fragment(s) (for more information see the documentation)
[ ] Attached Screenshots, Payload Example
[ ] Unit, Integration Tests
[ ] Documentation Updates (when adding/changing features)
[ ] Example Plugin Updates (when adding/changing features)
[ ] Outline Remaining Work, Constraints from Design

w8... wot?
What is the code in the retrieve to render the plugin's view instead of the core one? I would have thought we accidentally removed that.
Two points:

Let's use nautobot-ui components in our example: https://github.com/nautobot/nautobot/pull/3721/files#diff-d4a3993cc893833a26c36f02b1e8774b12a594c8b1dbba604613fc98d4eb14b7R1
Can you have it provide a view for its own model too?

Can you have it provide a view for its own model too?
Example Plugin has a model view

The latest commit has the following:

ExampleModel uses the Nautobot Default ObjectRetrieveView.

AnotherExampleModel uses a Customized ModelView from example_plugin.

Finally, the Location model uses OverridenView provided by example_plugin_with_view_override

Removed changes in example_plugin_with_view_override, since we already proved that we can override the Location Detail View from an app.

Code looks good to me. At some point we need to document this pattern in the app developer guide - is that in scope here?

I think this story only entails the re-introduction of the app overrides. I will open a follow-up issue that tracks this.

Code looks good to me. At some point we need to document this pattern in the app developer guide - is that in scope here?

I think this story only entails the re-introduction of the app overrides. I will open a follow-up issue that tracks this.

https://github.com/nautobot/nautobot/issues/3773


eml: 0.36.0-5 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository eml to 0.36.0-5:

upstream repository: http://pr2packages.clearpathrobotics.com/third-party/eml/eml-r36.tar.gz
release repository: https://github.com/ros-gbp/eml-release.git
distro file: indigo/distribution.yaml
bloom version: 0.5.17
previous version for package: null

So.. 2 hours? @dirk-thomas .. I'll hold my tongue until it passes a build on Jenkins :P

This will not work since previously eml had version 1.8.13-1 (http://www.ros.org/debbuild/indigo.html?q=eml). You can not release a lower version.

Thanks dirk. Will fix and re release asap.
On Feb 4, 2015 11:11 PM, "Dirk Thomas"<EMAIL_ADDRESS>wrote:

This will not work since previously eml had version 1.8.13-1 (
http://www.ros.org/debbuild/indigo.html?q=eml). You can not release a
lower version.
—
Reply to this email directly or view it on GitHub
https://github.com/ros/rosdistro/pull/7096#issuecomment-72975879.


Corrected images filenames

iphoto filenames corrected to correct format, and replace AddressBook with TeamContact4/7 title in index.md.

Codecov Report

Merging #40 (26dcd86) into master (3ea2429) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #40   +/-   ##
=========================================
  Coverage     72.15%   72.15%
  Complexity      399      399
=========================================
  Files            70       70
  Lines          1232     1232
  Branches        125      125
=========================================
  Hits            889      889
  Misses          311      311
  Partials         32       32

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 3ea2429...26dcd86. Read the comment docs.


Update quick_start_images.md

"docker run -it -p 80:80 microsoft/iis cmd" can't into container.
Only return following log:
Service 'w3svc' started
Because specify ENTRYPOINT to "C:\ServiceMonitor.exe w3svc"

Same pull request.
https://github.com/Microsoft/Virtualization-Documentation/pull/458


git-quick-stats: 2.1.8 -> 2.1.9

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/arzzen/git-quick-stats/releases.
meta.description for git-quick-stats is: "A simple and efficient way to access various statistics in git repository"
meta.homepage for git-quick-stats is: "https://github.com/arzzen/git-quick-stats"
meta.changelog for git-quick-stats is: ""
Updates performed

Version update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS

The tests defined in passthru.tests, if any, passed

Warning: no invocation of /nix/store/jrsnp2vaqjjlca8vi4c749ms31gq0vsp-git-quick-stats-2.1.9/bin/git-quick-stats had a zero exit code or showed the expected version

/nix/store/jrsnp2vaqjjlca8vi4c749ms31gq0vsp-git-quick-stats-2.1.9/bin/.git-quick-stats-wrapped passed the binary check.

0 of 2 passed binary check by having a zero exit code.

1 of 2 passed binary check by having the new version present in output.

found 2.1.9 with grep in /nix/store/jrsnp2vaqjjlca8vi4c749ms31gq0vsp-git-quick-stats-2.1.9

found 2.1.9 in filename of file in /nix/store/jrsnp2vaqjjlca8vi4c749ms31gq0vsp-git-quick-stats-2.1.9

directory tree listing: https://gist.github.com/f2aabb86fd101b039d8e818cf35d71ee

du listing: https://gist.github.com/cda4459dd8a57f1e69b346062d698977

Rebuild report (if merged into master) (click to expand)

4 total rebuild path(s)

1 package rebuild(s)

1 x86_64-linux rebuild(s)
1 i686-linux rebuild(s)
1 x86_64-darwin rebuild(s)
1 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
git-quick-stats

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/jrsnp2vaqjjlca8vi4c749ms31gq0vsp-git-quick-stats-2.1.9 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A git-quick-stats https://github.com/r-ryantm/nixpkgs/archive/d923d640ec9ea8f685c768ae2257c5d6d0acc704.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/jrsnp2vaqjjlca8vi4c749ms31gq0vsp-git-quick-stats-2.1.9
ls -la /nix/store/jrsnp2vaqjjlca8vi4c749ms31gq0vsp-git-quick-stats-2.1.9/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

git-quick-stats

Maintainer pings
cc @kmein for testing.

Result of nixpkgs-review pr 121218 run on x86_64-linux 1

1 package built:

git-quick-stats


Internal Server Error

Brilliant API but now reporting internal server error and does not work.

Fixed!

Thanks for reporting


The low risk encounters and the dp3t protocol

Hi
I asked the authors of the Spanish radar covid app, to implement the feature of low risk encounters based on
https://github.com/corona-warn-app/cwa-documentation/blob/master/transmission_risk.pdf
They answered me that

Sorry, that's not possible for the moment because we are using the dp3t protocol and this functionality is not supported currently.

is this true? Can anybody please comment on this.
thanks and regards

@ouboub
I don't know but maybe it would be better if you ask this question here: https://github.com/DP-3T

@ouboub
I don't know but maybe it would be better if you ask this question here: https://github.com/DP-3T/documents

Fair enough


implement REST API for /api/customers*

As for now, there will be three models on which the API should be based.

Both purchase and repair models have a one-to-many relationship with a customer.
Endpoints:

/api/customers

GET /api/customers - return list of customers
query parameters: search, limit
POST /api/customers - create new customer
accept json

/api/customers/<customer_id>

GET /api/customers/<customer_id>  - return customer details
PATCH /api/customers/<customer_id>  - edit customer details
DELETE /api/customers/<customer_id>  - delete customer and all it's relations

/api/customers/<customer_id>/purchases

GET /api/customers/<customer_id>/purchases return list of purchases
query parameters: search, limit, filter
POST /api/customers/<customer_id>/purchases create new purchase
accept json

/api/customers/<customer_id>/purchases/<purchase_id>

GET /api/customers/<customer_id>/purchases/<purchase_id> - return purchase details
PATCH /api/customers/<customer_id>/purchases/<purchase_id> - edit purchase details
DELETE /api/customers/<customer_id>/purchases/<purchase_id> - delete purchase details

/api/customers/<customer_id>/repairs

GET /api/customers/<customer_id>/repairs return list of repairs
query parameters: search, limit, filter
POST /api/customers/<customer_id>/repairs create new repair
accept json

/api/customers/<customer_id>/repairs/<repair_id>

GET /api/customers/<customer_id>/repair/<repair_id> - return repair details
PATCH /api/customers/<customer_id>/repair/<repair_id> - edit repair details
DELETE /api/customers/<customer_id>/repair/<repair_id> - delete repair details

https://github.com/radekska/customer-manager/issues/14


abort-without-identified-hostname flag

I am using wamp v3.
configured hosts to use
127:0:0:1 mml
127:0:0:1 abc.mml
127:0:0:1 xyz.mml
http://abc.mml/anyroute works
http://mml/anyroute - 404 error
until I changed abort-without-identified-hostname from true to false.
is this the expected behavior?
basically in production I would like:
https://mysite.com for all users to go and register
as well as faqs, about, etc not to worry about tenancy
The user registers at https://mysite.com/register
From there a tenant is created - abc.mysite.com
Am I going about this correct?
Should I do it another way?
How should a site differentiate between tenant site and global site?
Thanks
Randy

The app wasn't able to identify mml as a hostname, so having that config set to true is doing exactly what it's supposed to do.
You have a few options.

You can look into the "Fallback" functionality: https://laravel-tenancy.com/docs/hyn/5.x/fallback
You can leave that config set to false and implement the abort functionality yourself
Or you can simply add that domain to the hostnames table without associating it to any tenants/websites.  This will allow the hostname resolution to succeed and let your routes be handled appropriately.

I'm sure there are other options that I've missed.  Ultimately, it's up to you how you do it.

Thanks


note where .env.sample can be found

there is no .env.sample in the readraptor repo I'm assuming it must be referring to the one in the meta repo

nevermind I thought the go get was installing readraptor stuff, the readme was missing the git clone step.


Add perturb before or after node encoder?

To my understanding, for the graphprop tasks, the perturb is add to the input node embedding, output of the node encoder while in the nodeprop tasks, the perturb is added directly to the node feature. Have you test the difference?
# graphprop
def forward(self, batched_data, perturb=None):
    x, edge_index, edge_attr, batch = batched_data.x, batched_data.edge_index, batched_data.edge_attr, batched_data.batch

    ### computing input node embedding
    tmp = self.atom_encoder(x) + perturb if perturb is not None else self.atom_encoder(x)

# nodeprop
def train_flag(model, x, y_true, train_idx, optimizer, args, device) :

    forward = lambda perturb : model(x[train_idx] + perturb)
    model_forward = (model, forward)
    y = y_true.squeeze(1)[train_idx]
    loss, _ = flag(model_forward, x[train_idx].shape, y, args, optimizer, device, F.nll_loss)
    return loss.item()

Hi, thanks for your interest in our work. The reason we add perturbation on the output of node encoder in the graph classification task is to achieve the goal of doing data augmentation in continuous feature space. The raw input node features in the graph task are categorical features. It will be suboptimal to add continuous perturbation on that. While on the node task the features are already preprocessed and projected into continuous space so we can directly add perturbation on that.

I'm considering the scale of node encoder output. As the encoder is updated during training, the feature scale changes while the step size of flag is fixed. This might damage the robustness I think.
I'm interesting in the graph augmentation field and repeat your work recently. Looking forward to discussing more with you.

This is an interesting idea. The reason we did not perform such fine-grained investigation on the augmentation technique is that we want to make our method widely applicable to various tasks and datasets. And yes I believe there is definitely room for improvement. Hope things work out for you!


Implement Email-Sending Capability in email-service with Kafka Communication

Pull Request Description
Summary
This PR introduces the implementation of an email-sending capability in the email-service. The service is designed to listen to Kafka topics for events and send emails based on the event payload. This ensures that notifications, alerts, and other communications can be reliably delivered to users.
Key Changes

Kafka Listener:

Configured email-service to subscribe to the email-sent Kafka topic.
Implemented a Kafka listener to consume and process incoming messages containing email-related data.

SMTP Configuration:

Integrated SMTP settings for email delivery using a Resend.

Email Templates:

Added support for customizable templates for different email types to be created in the future.

Error Handling and Logging:

Implemented robust error handling for failed email sends, with detailed logs for diagnostics.

Acceptance Criteria

The email-service listens to the email-notifications Kafka topic and processes events correctly.
Emails are sent using SMTP settings, and content is dynamically customized with templates.
Errors are logged, and retry logic handles transient failures gracefully.

Story Details

Story Points: 13
Priority: P1
Risk: High

Next Steps

Monitor email-service performance in production for edge cases and optimize as needed.
Extend the template library to include more email types as required by the system.
Add retry logic for transient SMTP errors (e.g., timeouts).
Test the microservice especially as it starts to grow.

@wade3hamati can you delete the .idea directory? Use the git rm .idea --cached command.


Drag and click event confusion

Sometimes node is clicked, but the drag event is firing. Need to address this

Can be fixed by adding .clickDistance(10) to d3.zoom()
Example :
// Get zooming function
behaviors.zoom = d3.zoom()
.clickDistance(10) // Added to allow a leeway distance of 10 before registering a click as a drag/pan event
.on('start', (event, d) => attrs.onZoomStart(event, d))

@yyqsg888 wow, did not know about it, and every time I had to implement that clickDistance-like method by myself. Thank you for mentioning it


⚠ Bofrost.es has degraded performance

In f04941c, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1477 ms

Resolved: Bofrost.es performance has improved in 64df887 after 6 minutes.


Initialize GitHub repository not working

I have a private GitHub repo that I have been using for a few week. But on the 18th, after reinstalling the app, the initialize repository function wasn’t working properly. I put the clone url and then Code App says that the repo has been initialized but I can’t see any files from the repo and the initialize repo button doesn’t disappear. If I try to re-enter the GitHub url, I get a message saying the repo already exists.
I just want to be able to access my repo and edit it when needed, please help

Nvm it started working by itself somehow

Nice! I'm glad you could figure it out!


BIG BUG : problems are no more showed

Hi,
a very big bug : when I create an error, for example when I create a new node with no tag, ID doesn't react : nothing in the tab "problems", nothing when I select the node (no problem shown in the left), nothing when I want to save (yesterday and before, it was a blocking error : it was impossible to save. Today you may !!).
Best regards

Hi,
the next day everything was working again.
It was not a problem of internet connexion because i'm using public internet, I can use 2 different connexions and I had the same problem with both.
Maybe the server was out or so slow that the controls never arrived.
For 4 days all is working  so I close this issue.
Best regards


[Bug]: Codec Numbers do not appear on 1920x1080 internal resolution

Issue Summary:
Internal Resolution set to Enabled causes codec numbers to not appear
MGSM2Fix & Game Version Number:
MGSM2Fix Version 2.2
Metal Gear Solid Master Collection Version Ver.1.5.0

Log Files:
CPU - AMD Ryzen 5800X3D
GPU - AMD Radeon 6750X
RAM - 32GB Corsair
Reproduction:
in [Internal Resolution] set config to this
Enabled = false
Height = 0
Widescreen = true

seems to be a duplicate of #15

Duplicate of #15.


Custom converter specified on binding is called only once at startup and not with the every OnPropertyChanged event

Describe the bug
Custom converter specified on binding is called only once at startup and not with the every OnPropertyChanged event as is expected. This is a new behavior related to 11.1.0-rc1, everything worked just fine with 11.0.10/11.0.11. Minimal sample to simulate this behavior is attached.
TestApp.zip
To Reproduce

Start the app
TextBlock content on the second line should change with the every click on button "Click me" (toggling between suffixes -cs/cz and -en/en) but this is not happening

Expected behavior
Text on the second line should change with the every click on button "Click me" because OnPropertyChanged event is fired for the binded property.
Avalonia version
11.1.0-rc1
OS
No response
Additional context
No response

You should consider writing an IMultiValueConverter and move the _toggle state into the ViewModel. IMO, converters are best left as pure functions without local state.
I can confirm the regression from 11.0.11 with the repro though, so it should be fixed.
Also, I see a use of BindingNotification which might not work until RC2 or stable.

As @stevemonaco said, this is a bit of an edge case as you're raising PropertyChanged on a property which hasn't changed and then relying on a stateful IValueConverter to mutate the state differently each time. The issue is that we now check for a changed bound value earlier than before and so the converter isn't being called.
I cross-checked with WPF and it also does call the converter even if the value hasn't changed so I'm in two minds about whether we should support this use-case. On one hand we try to match WPF's behavior where it makes sense, but on the other hand it feels like this is an anti-pattern.

I've run across a related issue to this that I never filed an issue for. The situation is this:

Have a custom IsNullOrEmptyConverter bound to an ObservableCollection property. That converter is used to show "list empty, please add an item" type messages. So ListBlock.IsVisible="{Binding ListProperty, Converter={StaticResource IsNullOrEmptyConverter}}" in pseudocode.
In the view model, manually force the converter to update visibility by calling OnPropertyChanged("ListProperty") which should force re-calculation of the converter and hide the message. In Avalonia it does not.

This is a pattern I've used since WPF as it saves one additional property on the view model that adds no new information. For now I've had to work-around this with a "IsListEmpty" property.
I really wish the binding systems were smart enough to raise property change notification on the ListProperty when it's a collection that has INotifyCollectionChanged but we aren't there yet. So I've been using this method.
Overall though, I think there always needs to be a way for apps to force re-evaluation of bindings. That's needed in edge cases we've discussed separately anyway.

As @stevemonaco said, this is a bit of an edge case as you're raising PropertyChanged on a property which hasn't changed and then relying on a stateful IValueConverter to mutate the state differently each time. The issue is that we now check for a changed bound value earlier than before and so the converter isn't being called.
I cross-checked with WPF and it also does call the converter even if the value hasn't changed so I'm in two minds about whether we should support this use-case. On one hand we try to match WPF's behavior where it makes sense, but on the other hand it feels like this is an anti-pattern in that it's generally bad practise to rely on side-effects in bindings. It also feels like a misuse of PropertyChanged in that if that event is raised on a property that hasn't changed, a listener should be free to ignore the event.

I provided the minimal sample to simulate this behavior. In real use I need to raise the property change for the property (flags) even if its value does not change because I need to translate the flags value into another language in runtime. I do not figure out another way how to do that.

@robloo
ObservableCollection<T>.Count can be binded to as it raises PropertyChanged for Count.
I think stateful converters are bad practice, but I also think the ViewModel or some component in-between should be responsible for restricting notifications, if necessary.
@viktorspacil
To clarify my earlier answer: Use a IMultiValueConverter and move the _toggle value to the ViewModel. You bind both TestEnum and Toggle to the converter. Then when you mutate either one from the VM, the converter will be reevaluated and the binding target updated.

@stevemonaco

ObservableCollection.Count can be binded to as it raises PropertyChanged for Count. Granted, that's a lot of notifications and converter calls since ObservableCollection doesn't well-support range operations.

lol, I've done it using the collection itself for so many years I never thought to check that. I never knew INotifyPropertyChanged fired for Count. Always learning, thanks!

I think stateful converters are bad practice, but I also think the ViewModel or some component in-between should be responsible for restricting notifications, if necessary.

I would say an IsNullOrEmptyConverter isn't stateful. It's a transform of the raw information and adds nothing to it or maintains any state itself. Transforming information is exactly what converters are for. Simply speaking we want to minimize duplicate information (properties) in the view model. I'm sure there is some information theory rationale here.

Anyway, I still think we shouldn't try to outsmart application developers. Just as in WPF, if the view model wants an update it should get an update. I'm sure I can find other cases where it's needed if I looked a while.

I've been thinking about this, and as much as I dislike some of the patterns this is used for, Avalonia 11.0 supports it and WPF supports it so Avalonia 11.1 should also support it.


Are function types supposed to be working

Problem
I don't know if these are considered implemented or not but saw functions are parsed as types. is the following suppose to work
use dep::std;
fn main() {
    let a = 1;
    let c = foo(a, bar);
    std::println(c);
}

fn foo(a: Field, f: Fn(Field) -> Field) -> Field {
    f(a)
}

fn bar(b: Field) -> Field {
    b
}

Happy Case
.
Alternatives Considered
No response
Additional Context
No response
Would you like to submit a PR for this Issue?
Maybe
Support Needs
No response

Yes, but you have the syntax a bit off. The function type uses a lower-case fn like rust's function types. Capital Fn is reserved for function traits once traits are implemented. Here's a fixed version of your example:
use dep::std;
fn main() {
    let a = 1;
    let c = foo(a, bar);
    std::println(c);
}

fn foo(a: Field, f: fn(Field) -> Field) -> Field {
    f(a)
}

fn bar(b: Field) -> Field {
    b
}

Ah, I should also note that you can use lambdas as well: |a, b| a + b. We're in the process of implementing closures though, so trying to capture variables from the environment is broken at the moment..


bot is not downloading images.

no files found after "ddg history (channel id)"

how did you fixed it?


test function so broken

Type: Bug
Just see the log:
2023-07-17 15:12:17.555 [info] Discover tests for workspace name: FIN-social - uri: /Users/kaichuan/Repos/FIN-social
2023-07-17 15:12:17.558 [info] > conda run -n fin --no-capture-output python ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/get_output_via_markers.py ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir . -s --cache-clear tests
2023-07-17 15:12:17.559 [info] cwd: .
2023-07-17 15:12:17.826 [error] Error discovering pytest tests:
n [Error]:
EnvironmentLocationNotFound: Not a conda environment: /Users/kaichuan/mambaforge/envs/fin/envs/fin
at ChildProcess.<anonymous> (/Users/kaichuan/.vscode/extensions/ms-python.python-2023.12.0/out/client/extension.js:2:241817)
at Object.onceWrapper (node:events:628:26)
at ChildProcess.emit (node:events:513:28)
at maybeClose (node:internal/child_process:1121:16)
at Process.ChildProcess._handle.onexit (node:internal/child_process:304:5)

2023-07-17 15:12:27.936 [info] > conda run -n fin --no-capture-output python ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/get_output_via_markers.py -c "import pytest"
2023-07-17 15:12:28.101 [info] Send text to terminal: conda install --name fin pytest -y
2023-07-17 15:12:28.102 [info] > conda run -n fin --no-capture-output python ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/get_output_via_markers.py -c "import pytest"
2023-07-17 15:19:24.627 [info] > conda info --json
2023-07-17 15:19:46.486 [info] Discover tests for workspace name: FIN-social - uri: /Users/kaichuan/Repos/FIN-social
2023-07-17 15:19:46.489 [info] > conda run -n fin --no-capture-output python ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/get_output_via_markers.py ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/printEnvVariables.py
2023-07-17 15:19:46.489 [info] shell: bash
2023-07-17 15:19:47.113 [info] > conda run -n fin --no-capture-output python ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/get_output_via_markers.py ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/testing_tools/run_adapter.py discover pytest -- --rootdir . -s --cache-clear tests
2023-07-17 15:19:47.113 [info] cwd: .
2023-07-17 15:19:47.366 [error] Error discovering pytest tests:
n [Error]:
EnvironmentLocationNotFound: Not a conda environment: /Users/kaichuan/mambaforge/envs/fin/envs/fin
at ChildProcess.<anonymous> (/Users/kaichuan/.vscode/extensions/ms-python.python-2023.12.0/out/client/extension.js:2:241817)
at Object.onceWrapper (node:events:628:26)
at ChildProcess.emit (node:events:513:28)
at maybeClose (node:internal/child_process:1121:16)
at Process.ChildProcess._handle.onexit (node:internal/child_process:304:5)

2023-07-17 15:19:50.185 [info] > conda run -n fin --no-capture-output python ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/get_output_via_markers.py ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/printEnvVariables.py
2023-07-17 15:19:50.185 [info] shell: bash
2023-07-17 15:19:50.675 [info] > conda run -n fin --no-capture-output python ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/get_output_via_markers.py -c "import pytest"
2023-07-17 15:19:50.820 [info] > conda run -n fin --no-capture-output python ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/get_output_via_markers.py -c "import pytest"
2023-07-17 15:19:50.933 [info] > conda info --json
2023-07-17 15:19:51.304 [info] Send text to terminal: conda activate fin
2023-07-17 15:19:51.810 [info] Send text to terminal: conda install --name fin pytest -y
Here are two pieces of log that I especially want to complain about:
EnvironmentLocationNotFound: Not a conda environment: /Users/kaichuan/mambaforge/envs/fin/envs/fin
2023-07-17 15:19:51.810 [info] Send text to terminal: conda install --name fin pytest -y
Here are some of my comments:

Why can't it locate the Python environment that I have, like all other extensions or functions of VSCode does?
How is the "conda install --name fin pytest -y" a good solution to the "EnvironmentLocationNotFound" error?
Please don't put "-y" after any "conda install" that you decide to initiate. You don't know what you can mess up.

VS Code version: Code 1.80.1 (74f6148eb9ea00507ec113ec51c489d6ffb4b771, 2023-07-12T17:20:58.115Z)
OS version: Darwin arm64 22.5.0
Modes:

System Info

Item
Value

CPUs
Apple M2 (8 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmetal: disabled_offmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
4, 3, 3

Memory (System)
8.00GB (0.07GB free)

Process Argv
--crash-reporter-id e7641681-a248-4b39-b8fd-a92e2c3e5516

Screen Reader
no

VM
0%

Extensions (9)

Extension
Author (truncated)
Version

copilot
Git
1.96.255

vscode-language-pack-zh-hans
MS-
1.80.2023071209

python
ms-
2023.12.0

vscode-pylance
ms-
2023.7.20

jupyter
ms-
2023.6.1101941928

jupyter-keymap
ms-
1.1.2

jupyter-renderers
ms-
1.0.17

vscode-jupyter-cell-tags
ms-
0.1.8

vscode-jupyter-slideshow
ms-
0.1.5

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30410667
py29gd2263cf:30789497
vsclangdf:30486550
c4g48928:30535728
dsvsc012:30540252
pynewext54:30695312
azure-dev_surveyone:30548225
vsccc:30610678
282f8724:30602487
pyind779:30671433
f6dab269:30613381
pythonsymbol12:30671437
showlangstatbar:30737416
vsctsb:30748421
pythonms35:30701012
pythonfmttext:30731395
pythoncmvfstrcf:30756944
fixshowwlkth:30771522
showindicator:30785052
pythongtdpath:30769146
bgfeh915:30780428
pythonnosmt12:30779714
pythonidxpt:30784022
pythonnocebcf:30776496
h7j2d465:30786200
dsvsc013:30789517

Hi @eleanorjboyd Thanks for your feedback. I have added the setting and I think I am still facing the same issue. Following is the log:
2023-07-19 10:41:07.343 [info] Experiment 'pythonTestAdapter' is active
2023-07-19 10:41:07.343 [info] Experiment 'pythonPromptNewFormatterExt' is active
2023-07-19 10:41:07.344 [info] Experiment 'pythonPromptNewToolsExt' is active
2023-07-19 10:41:07.344 [info] Test server listening.
2023-07-19 10:41:07.344 [info] Python interpreter path: ~/mambaforge/envs/fin/bin/python
2023-07-19 10:41:07.392 [info] > conda info --json
2023-07-19 10:41:07.835 [info] > /usr/bin/python3 -I ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/get_output_via_markers.py ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/interpreterInfo.py
2023-07-19 10:41:08.524 [info] > conda run -n fin --no-capture-output python ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/get_output_via_markers.py ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/printEnvVariables.py
2023-07-19 10:41:08.524 [info] shell: bash
2023-07-19 10:41:08.563 [info] Starting Pylance language server.
2023-07-19 10:41:20.135 [info] Discover tests for workspace name: FIN-social - uri: /Users/kaichuan/Repos/FIN-social
2023-07-19 10:41:20.135 [info] Running discovery for pytest using the new test adapter.
2023-07-19 10:41:20.137 [info] > conda run -n fin --no-capture-output python ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/get_output_via_markers.py -m pytest -p vscode_pytest --collect-only tests
2023-07-19 10:41:20.137 [info] cwd: .
2023-07-19 10:41:20.525 [error] Error while trying to run pytest discovery,
Error:
EnvironmentLocationNotFound: Not a conda environment: /Users/kaichuan/mambaforge/envs/fin/envs/fin
2023-07-19 10:41:20.526 [error] pytest test discovery error
n [Error]:
EnvironmentLocationNotFound: Not a conda environment: /Users/kaichuan/mambaforge/envs/fin/envs/fin
at ChildProcess.<anonymous> (/Users/kaichuan/.vscode/extensions/ms-python.python-2023.12.0/out/client/extension.js:2:241817)
at Object.onceWrapper (node:events:628:26)
at ChildProcess.emit (node:events:513:28)
at maybeClose (node:internal/child_process:1121:16)
at Process.ChildProcess._handle.onexit (node:internal/child_process:304:5)

2023-07-19 10:41:39.721 [info] Discover tests for workspace name: FIN-social - uri: /Users/kaichuan/Repos/FIN-social
2023-07-19 10:41:39.722 [info] Running discovery for pytest using the new test adapter.
2023-07-19 10:41:39.724 [info] > conda run -n fin --no-capture-output python ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/get_output_via_markers.py -m pytest -p vscode_pytest --collect-only tests
2023-07-19 10:41:39.724 [info] cwd: .
2023-07-19 10:41:39.967 [error] Error while trying to run pytest discovery,
Error:
EnvironmentLocationNotFound: Not a conda environment: /Users/kaichuan/mambaforge/envs/fin/envs/fin
2023-07-19 10:41:39.968 [error] pytest test discovery error
n [Error]:
EnvironmentLocationNotFound: Not a conda environment: /Users/kaichuan/mambaforge/envs/fin/envs/fin
at ChildProcess.<anonymous> (/Users/kaichuan/.vscode/extensions/ms-python.python-2023.12.0/out/client/extension.js:2:241817)
at Object.onceWrapper (node:events:628:26)
at ChildProcess.emit (node:events:513:28)
at maybeClose (node:internal/child_process:1121:16)
at Socket.<anonymous> (node:internal/child_process:479:11)
at Socket.emit (node:events:513:28)
at Pipe.<anonymous> (node:net:757:14)

2023-07-19 10:42:04.031 [info] > conda run -n fin --no-capture-output python ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/get_output_via_markers.py ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/printEnvVariables.py
2023-07-19 10:42:04.032 [info] shell: bash
2023-07-19 10:42:04.747 [info] > conda run -n fin --no-capture-output python ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/get_output_via_markers.py -c "import pytest"
2023-07-19 10:42:04.997 [info] > conda run -n fin --no-capture-output python ~/.vscode/extensions/ms-python.python-2023.12.0/pythonFiles/get_output_via_markers.py -c "import pytest"
2023-07-19 10:42:05.103 [info] > conda info --json
2023-07-19 10:42:05.484 [info] Send text to terminal: conda activate fin
2023-07-19 10:42:05.989 [info] Send text to terminal: conda install --name fin pytest -y

Thank you for the additional logs- will investigate!

Hi @QihangYao, are you able to try this again on the newest version of insiders? I have made some bug fixes to how we handle environments and paths which might fix this problem. Let me know the status and I can investigate further if it still doesn't look correct. If you do run into an issue please include the logs from your run. Thanks


By submitting this review you agree to use of your data as outlined in our privacy policy

Great module!
Thank you!
I believe that the word 'review' should be changed to 'request' in this case?!

I believe that the word 'review' should be changed to 'request' in this case?!

You are right, this is obvious copy'n'paste error. Guess from what module :)

Yes, I imagined that may be the case :)


Target Link

Hi
Great plugin thanks. I am struggling to get the target link to work from the card, any ideas if put in the link
f?p=&APP_ID.:5:&SESSION.::&DEBUG.::::
but nothing happens? Does this feature work?

Thanks for liking the plugin. Sorry for answering after quite a while. The Target Link does not work right now. Hoping to Include this feature some day.


Add yaml support for themes

While this won't provide full YAML syntax support, it at least allows for users to specify the theme format using the json primitives as YAML syntax.
I used this in my own fork to allow support for comments in the theme files, so perhaps it might be of use here as well.
Feel free to suggest any changes!

Hello sir,
I do like this change, I will try it out on a VM in the following days to ensure that it works as expected and probably merge it. I don't see any reason that this would break any functionality at all.

Hello @JonasDe , it looks ok so I merged it. In the future (and this is optional) would you like to make a new PR where you provide an example of a theme config in YAML just as an example?


Update CBA availability status

Azure AD certificate-based authentication is GA and no longer in preview.
https://techcommunity.microsoft.com/t5/microsoft-entra-azure-ad-blog/azure-ad-certificate-based-authentication-cba-on-mobile-now/ba-p/1144695
https://learn.microsoft.com/en-us/azure/active-directory/authentication/concept-certificate-based-authentication

@Justinha
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

#sign-off


ARROW-454: pojo.Field doesn't implement hashCode()

unrelated C++ build failure in travis...
The java side passes.

We've been experiencing intermittent timeouts on OS X.


Updating the encoding in documentation

Correcting the documentation for encoding of rigid and wooden walls in 'board' array which would solve issue #196

thanks!


request help: Custom plugins have no effect

Issue description

容器内jar包启动后，name方法内有打印，但是做插件请求接口测试的时候，filter内无打印没反应。
之后测试了jwt-auth插件，是有效的，所以猜想是不是ext-plugin-pre-req内的自定义插件无效
Environment

apisix version 2.10
OS  Linux master 3.10.0-1160.6.1.el7.x86_64
apisix-dashboard version, 2.9.0

any releated logs in logs/error.log?

Logs are only logged at startup, but not at any other time

try other plugin------jwt-auth

Something strange, can you be sure that the request hit is the route you configured? From the logs it looks like some other route caught your request.
Give me a minimalistic reproduction of the use case, no pictures, use text.

use RewriteRequestDemoFilter. just add two log.info()
only one route configured :
{   "uri": "/data-processing-center/*",   "name": "prod_httpserver-ingress-getall_rule-getall",   "desc": "Created by apisix-ingress-controller, DO NOT modify it manually",   "plugins": {     "ext-plugin-pre-req": {       "conf": [         {           "name": "RewriteRequestDemoFilter",           "value": "bar"         }       ]     },     "jwt-auth": {       "disable": true     }   },   "upstream_id": "33ffc6f3",   "labels": {     "managed-by": "apisix-ingress-controller"   },   "status": 1 }
The test Url : http://<IP_ADDRESS>:30371/data-processing-center/config/findAll
when set     "jwt-auth": {"disable": true }  return normal result and no log   (RewriteRequestDemoFilter======filter)
when set     "jwt-auth": {"disable": false }  return  {"message":"Missing JWT token in request"}
It seems to RewriteRequestDemoFilter have failed

can you remove jwt plugin and test ext-plugin-pre-req?
I reproduced ext-plugin-pre-req and didn't find the problem you mentioned.

can you remove jwt plugin and test ext-plugin-pre-req?
I reproduced ext-plugin-pre-req and didn't find the problem you mentioned.

thanks !
The problem has already been solved.
add ext-plugin-pre-req  in config.yaml
plugins:                          # plugin list
······

ext-plugin-pre-req

I didn't realize to add this plugin  !
😢


Extend TransitionAbandonedReason enum

Currently the enum is
pub enum TransitionAbandonedReason {
    // NotLeader transitions have been abandoned because the replica is not
    // the cluster leader.
    NotLeader,
}

However, transitions can also be dropped by a replica if at some point it gets disconnected from majority of the cluster thinking it is still a leader accepting transitions; upon reconnecting to the rest of the cluster, such a replica will drop uncommitted transitions due to a discrepancy with consensus achieved by the cluster majority.

See also https://github.com/andreev-io/little-raft/issues/13.

Which part of the source code should I look into?

@suzaku The only place where follower Replicas might overwrite uncommitted transitions is here: https://github.com/andreev-io/little-raft/blob/master/little_raft/src/replica.rs#L537-L547. When the log vector is truncated, transitions get abandoned.

@suzaku I assigned this to you. Hope you don't mind. 😄

By the way, in the paper, this behavior is documented in section 5.3. https://raft.github.io/raft.pdf. Also see the highlighted text in the screenshot below.

I will take a look. The change seems straight forward enough, but I'll read the paper for the rationale behind the change for sure. I've forgotten most of the details about Raft.


Generate CMake frameworks from podspecs

The motivation for this change is that in #1525 we started down the path of adding yet another copy of the FirebaseCore/GoogleUtilities hand-written CMake integration code. With this change in place, all we should need there is this line in Firestore/CMakeLists.txt:
podspec_framework(${FIREBASE_SOURCE_DIR}/FirebaseAuthInterop.podspec)

The FirebaseAuthInterop target did not work with the existing xcodebuild-based mechanism because it's a header-only and seems to behave differently from the point of view of the way Xcode builds it. Rather than implement a different custom integration path for that, I bit the bullet and made it possible to just use the podspec as the source of truth for the CMake build.
At a lower-level this moves these frameworks away from external builds that shell out to xcodebuild to building the frameworks natively with CMake. There's now a ruby script that uses cocoapods Pod::Spec and related utilities to read the podspec and translate to the equivalent CMake commands.
Note that the trickiest part of getting this to work was to make header naming work. Xcode makes it possible to use framework imports from within a framework while building it, but CMake has no such mechanism. See comments in podspec_cmake.rb for how this is reconciled.

And GoogleUtilities. Note that this is just the Logger and Environment subspecs:
# This file was generated by podspec_cmake.rb
# from GoogleUtilities.podspec.
# Do not edit!

add_library(
  GoogleUtilities
  STATIC
  /Users/mcg/src/google/ios-sdk/GoogleUtilities/Environment/third_party/GULAppEnvironmentUtil.h
  /Users/mcg/src/google/ios-sdk/GoogleUtilities/Environment/third_party/GULAppEnvironmentUtil.m
  /Users/mcg/src/google/ios-sdk/GoogleUtilities/Logger/GULLogger.m
  /Users/mcg/src/google/ios-sdk/GoogleUtilities/Logger/Private/GULLogger.h
  /Users/mcg/src/google/ios-sdk/GoogleUtilities/Logger/Public/GULLoggerLevel.h
)

set_property(
  TARGET
  GoogleUtilities
  PROPERTY
  PUBLIC_HEADER
  /Users/mcg/src/google/ios-sdk/GoogleUtilities/Environment/third_party/GULAppEnvironmentUtil.h
  /Users/mcg/src/google/ios-sdk/GoogleUtilities/Logger/Public/GULLoggerLevel.h
  /Users/mcg/src/google/ios-sdk/GoogleUtilities/Logger/Private/GULLogger.h
)
set_property(TARGET GoogleUtilities PROPERTY FRAMEWORK ON)
set_property(TARGET GoogleUtilities PROPERTY VERSION 5.0.5)

target_compile_options(
  GoogleUtilities
  INTERFACE
  -F${CMAKE_CURRENT_BINARY_DIR}
)
target_compile_options(GoogleUtilities PRIVATE ${OBJC_FLAGS})

target_include_directories(
  GoogleUtilities
  PRIVATE
  /Users/mcg/src/google/ios-sdk/Debug/Firestore/Headers
  /Users/mcg/src/google/ios-sdk/Debug/Firestore/Headers/GoogleUtilities
)

target_link_libraries(GoogleUtilities PUBLIC "-framework Foundation")


Update missing var "type"

We need to add type in default var because, we are getting error here #L119 if we do not set it.

Thanks!

Thanks!

:+1:


require.context causes invalidated bundle even for non-matches

For example:
var testsContext = require.context(".", true, /\/(?!flycheck_).*\.spec$/);

We're trying to ignore flycheck's files, but even with this they cause bundle invalidation all the time.
The WatchIgnorePlugin does not work for this either. It doens't prevent the invalidation and it has this problem: #1378

facing the same issue

fwiw, we worked around w/ this @Metallium
  plugins: [
    new IgnoreFlycheckPlugin(),
  ],

var DirectoryWatcher = require('watchpack/lib/DirectoryWatcher');

function IgnoreFlycheckPlugin() {
}

function wrapWithFlycheckIgnore(watcher, property) {
  var original = watcher.prototype[property];

  watcher.prototype[property] = function(filePath, stat) {
    if (filePath.indexOf('flycheck_') !== -1) return;
    original.apply(this, [filePath, stat]);
  }
}

IgnoreFlycheckPlugin.prototype.apply = function(compiler) {
  compiler.plugin('after-environment', function() {
    wrapWithFlycheckIgnore(DirectoryWatcher, 'onChange');
    wrapWithFlycheckIgnore(DirectoryWatcher, 'onFileAdded');
    wrapWithFlycheckIgnore(DirectoryWatcher, 'onFileUnlinked');
  })
}

module.exports = IgnoreFlycheckPlugin;

same issus, does webpack really support regexp lookahead ?

I've created a minimal repro case for this: https://github.com/cletusw/webpack-issue-1379
Would love to see this fixed.

It's interesting that watch: true with webpack doesn't exhibit the same broken behavior. That may help with finding a fix.


Doesn't run due to missing node

Hi,
I was trying to run the code using codespace but got the error -
sh: 1: ts-node: not found
Any suggestions?

Hi I also got the same issue as well, for context here's the terminal. Thank you.

can you try npm install

yes, that worked. Thank you!

no problem, i will update the wiki. Sometimes Github codespace doesn't install automatically for you I guess.

Same here, it's running now. Thanks!


Error: "Installation not found"

I tried to deploy test prover from template, but face the issue "Installation not found"

We just landed a backend fix that should fix this problem. Could you try again please?
The suspected causes was that the newly created destination repo was not returned by GitHub for whatever reason, which caused the subsequent git push to fail. We added some wait-and-retry logic to mitigate the problem. Please let us know if it’s working for you.

We've recently implemented a new update addressing the issue you experienced. The underlying problem stemmed from the GitHub API's default setting, which only displayed a maximum of 30 repositories. We've adjusted this limit to show more repositories, which should resolve the error. Please feel free to reach out if the issue persists.
Thank you for your patience!

Deploy Success: poseidon


Setup deployment to Production

Set up the Azure web app environment which will include:

The application web server
+The postgres db server
Configuration of secret key and app environment for the application web server
Manual deployment from Github to the application web server

Fixed and deployed


Add --maximum-image-age option

This adds an option called --maximum-image-age and it defines
the number days old an image can get before the auto-selection
process no longer depends on only new OS patches.
This can be used to make sure refreshed images are picked that
may have updated docker caches or other state that is not tracked
by the OS patches.
We use this to allow our images to stay more "fresh" with respect
to the workload running in the cluster.
The default is 0, which means that only new OS patches will trigger
and new image to be created.  The value is in days and anything
less than 1 would effectively meaningles anyway.
Note that if there was an OS patch and the image age is too old,
we still prefer to pick up the patched nodes over the unpatched
nodes, as long as the number of patched nodes satisfies the
minimum candidates value.
Fixes #45

Now also fixes #48

Now also fixes #48

/lgtm

/lgtm


Give precedence to user-provided interceptors

Misk's default interceptors have precedence over user-provided ones which makes them less practical.
For example, I want to use a custom interceptor to transparently map application exceptions to WebActionException. With the current setup this isn't possible—and I can't think of many cases where the current behavior is useful.
I did a quick search for import misk.web.NetworkInterceptor and it returned only Misk entries which makes me think that the change won't break existing code. Alternatively, we can find a backward compatible way to control the order and preserve the existing default.
Do you have any thoughts or concerns about this proposal?

I ended up using ExceptionMapper for my needs. Closing this issue as reordering user interceptors is scary but maybe we'll consider more flexibility in the future (i.e., allow users to set their interceptors before or after Misk's).


Installating via pip seems to miss torch_scatter dependency

Hey,
I noticed that the installation via pip install wilds seems to miss the torch_scatter dependency that is also listed in the README. When e.g. trying to do from wilds.datasets.amazon_dataset import AmazonDataset I got
from wilds.datasets.amazon_dataset import AmazonDataset
  File "/Users/deul/Desktop/wilds/wilds/datasets/amazon_dataset.py", line 6, in <module>
    from wilds.common.utils import map_to_id_array
  File "/Users/deul/Desktop/wilds/wilds/common/utils.py", line 1, in <module>
    import torch, torch_scatter
ModuleNotFoundError: No module named 'torch_scatter'

As far as I can see, the solution should be as easy as adding torch_scatter>=2.0.5 to  the install_requires attribute in setup.py. In my case, the error was resolved after installing torch_scatter separately.

Hi! Unfortunately, we aren't able to install torch_scatter automatically, as it requires specifying the correct Torch and CUDA version. We mention this in the README here:

Running pip install wilds or pip install -e . will automatically check for and install all of these requirements except for the torch-scatter and torch-geometric packages, which require a quick manual install.

Thanks for bringing this up. When it does become automatically installable, we'll update it.

Thanks for the response and sorry for missing that!

Hi @kohpangwei , I wonder what is the proper version of torch-scatter and torch-geometric for torch-1.7.0/1.8.0? It seems hard to have a consistent version.

Hmm, did running something like
pip install torch-scatter -f https://pytorch-geometric.com/whl/torch-1.8.0+${CUDA}.html
work for you? (That's taken from https://github.com/rusty1s/pytorch_scatter#pytorch-180181.) This should give the latest version, which should hopefully work for you -- could you let us know if you run into problems with those (or with reproducibility in general) please?

It's strange that I find that only torch-scatter==2.0.2 works for me lol.

Oh, is that the version that's installed for you when you specify the version of torch and CUDA in the pip install torch-scatter... command above? Or did you have to manually specify it because the default version wasn't correct?


[reg_top] Add spurious WE check

This adds a spurious WE check using prim_onehot_check inside the CSR nodes (*_reg_top).
The check is only performed in the bus clock domain at the moment - i.e., async CSRs are not fully protected yet.
See #11036 for context.
Signed-off-by: Michael Schaffner<EMAIL_ADDRESS>

@weicaiyang @cindychip would it be easy to add a common CSR check for this? the error condition feeds into the same error as the bus integrity error.

@weicaiyang @cindychip would it be easy to add a common CSR check for this? the error condition feeds into the same error as the bus integrity error.

sure, as discussed, will enhance the bus integrity test to cover this.

Thanks for the review @vogelpi.

Squashed and rebased.


-v: Wrong parameter!

Hi,
quick question: I use Ubuntu live CD in order to recover some data from a zfs.
I tried to specify the device with the -v option, but no matter what I try, I always get "Wrong parameter error", e.g.
java -jar ./ZfsSpy.jar -p 8888 -i <IP_ADDRESS> -v <IP_ADDRESS>:/dev/nvme0n1p5 -d .
So I omitted the parameter and started the web app. But there it is a similar problem: No matter what I enter, I always get "Errors: File is not readable: ..."
Any idea, what's going wrong?
Thanks and regards
Roland
BTW: Output of lsblk -o NAME,FSTYPE,SIZE,MOUNTPOINT,LABEL is:
nvme0n1      zfs_member 894.3G                              rpool
├─nvme0n1p1  vfat         512M                              rpool
├─nvme0n1p2  ext4          50M                              rpool
├─nvme0n1p3  swap           2G                              rpool
├─nvme0n1p4  zfs_member     2G                              bpool
└─nvme0n1p5  zfs_member 889.7G                              rpool

Hi,
why <IP_ADDRESS>:/dev/... ? Should be only /dev/...!
Could you try out java -jar ./ZfsSpy.jar -p 8888 -i <IP_ADDRESS> -v /dev/nvme0n1p5 -d . ?
Does the user have read access rights to the device?
Greetings
Stefan

Thanks for the support. That helped.


Add usb fuzzing for KMSAN

Before sending a pull request, please review Contribution Guidelines:
https://github.com/google/syzkaller/blob/master/docs/contributing.md

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1183   +/-   ##
=======================================
  Coverage   67.47%   67.47%
=======================================
  Files         120      120
  Lines       20870    20870
=======================================
  Hits        14083    14083
  Misses       6126     6126
  Partials      661      661

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 491919b...58b069e. Read the comment docs.


Added support for device muting

Hey,
I really like your package and a way how you organized everything. Well done!
I searched thru all npm packages to find those who able to toggle mute for microphone on my Mac. No-one can do that so I decided to use you package as a code base.
Mu solution works well on my Mac with MacOs 10.15.2. I think it might me useful to have such functionality as well.

Hey @apollon thank you for the PR. Would you mind formatting the code with 2 spaces like it was and updating the PR? As it stands right now it changed every line of Swift code (to 4 spaces vs 2) so it's really hard for me to review what actually changed

@karaggeorge - Done. You can check.

Hi Guys, I've fixed issues that you found. Could you take a look?

@apollon Bump

Hi @apollon, I was just looking for this exact feature too so wondered if I might somehow help you in getting this over the finish line?
I'm not familiar with swift but if this hasn't been abandoned I'd be happy to try and work through completing this with someone.

TODO: need to think how to handle when hasInput && hasOutput

It seems that the above is the main outstanding item. Had you had any initial thoughts around this I could look into further?

@sindresorhus, @benswinburne - Sorry Guys, I didn't have a chance to finish my PR. Will try to do finish it this weekend 😆.

I have to integrate this feature from scratch because I've gotten a lot conflicts during merge 🙂.
New PR #13.


How can i integrate RAGtouille with DSPy

I have successfully created an index and I can easily search using
RAG = RAGPretrainedModel.from_index("my_index")
results = RAG.search(query,k=2,index_name="/m_index")
How do i integrate it in dspy?
What i want to do is ?
dspy.settings.configure(lm=lm, rm=RAG)
Issue:
retrieve = dspy.Retrieve(k=1)("hello")
TypeError: 'RAGPretrainedModel' object is not callable

Hi!
I know that my solution is kinda crutches, but it works for me so far. There is a method from langchain .as_langchain_retriever(k=k).
I haven't found how to integrate langchain retrievers in dspy frame, so I wrote an additional class to adapt it:
`from typing import List, Optional, Union
from dspy.retrieve.retrieve import Retrieve
from dspy.primitives.prediction import Prediction
class PassageWrapper:
def init(self, long_text):
self.long_text = long_text
class CustomLangChainRetriever(Retrieve):
def init(self, langchain_retriever, k=3):
super().init(k=k)
self.langchain_retriever = langchain_retriever
def forward(self, query_or_queries: Union[str, List[str]], k: Optional[int] = None) -> List[PassageWrapper]:
    k = k if k is not None else self.k
    if isinstance(query_or_queries, str):
        query_or_queries = [query_or_queries]

    results_lists = [self.langchain_retriever.invoke(query, k=k) for query in query_or_queries]

    passages = []
    for results in results_lists:
        for result in results:
            passages.append(PassageWrapper(result.page_content))

    return passages

rm = RAG.as_langchain_retriever(k=3)
custom_retriever = CustomLangChainRetriever(langchain_retriever=rm, k=3)
turbo = dspy.OpenAI(model='gpt-3.5-turbo')
dspy.settings.configure(lm=turbo, rm=custom_retriever)
`

This is what I tried. This can avoid the dependency

Create a method to query your index created using RAGatouille

from ragatouille import RAGPretrainedModel

def query_index(query: str, index_name: str, topk: int):

    # provide the path to your index
    index_path = ".ragatouille/colbert/indexes/"+index_name
    try:
        RAG = RAGPretrainedModel.from_index(index_path=index_path)
        return RAG.search(query=query, k=topk)
    except FileNotFoundError as e:
        print(f"Index not found: {e}")

Call that method in retriever steps in your DSPy pipeline.

class RAG(dspy.Module):
    def __init__(self, topk=2, index_name="my_index"):
        super().__init__()
        self.generate_answer = dspy.ChainOfThought(GenerateAnswer)
        self.topk = topk
        self.index_name = index_name

    def forward(self, question):
        # calling custom method
        context = query_index(query=question, index_name=self.index_name, topk=self.topk)
        prediction = self.generate_answer(context=context, question=question)
        return dspy.Prediction(context=context, answer=prediction.answer)

Hope DSPy will bring some support to this in the future.

What @hasalams posted looks like all that is needed in the medium term IMO

Had a PR for DSPy repo regarding this.

What @hasalams posted looks like all that is needed in the medium term IMO

Curious to learn more. What is the long term vision? i.e., what is missing from this example @okhat ?


Timeout in EC2 when listing a bucket, s3cmd works.

$  date; /usr/local/bin/s4cmd ls s3://my-redacted-bucket --debug; date
Thu Dec  3 17:16:53 UTC 2015
  (D)s4cmd.py:468  read S3 keys from $HOME/.s3cfg file
  (D)s4cmd.py:140  >> ls_handler(<__main__.CommandHandler object at 0x7fe855009a50>, ['ls', 's3://my-redacted-bucket'])
  (D)s4cmd.py:140  >> validate(<__main__.CommandHandler object at 0x7fe855009a50>, 'cmd|s3', ['ls', 's3://my-redacted-bucket'])
  (D)s4cmd.py:142  << validate(<__main__.CommandHandler object at 0x7fe855009a50>, 'cmd|s3', ['ls', 's3://my-redacted-bucket']): None
  (D)s4cmd.py:140  >> s3walk(<__main__.S3Handler object at 0x7fe855009a10>, 's3://my-redacted-bucket')
  (D)s4cmd.py:140  >> wrapper(<ThreadUtil(Thread-1, started daemon<PHONE_NUMBER>52416)>, <__main__.S3URL instance at 0x7fe854f7f950>, '', '', [])
  (E)s4cmd.py:359  [Errno 110] Connection timed out
  (D)s4cmd.py:140  >> wrapper(<ThreadUtil(Thread-2, started daemon<PHONE_NUMBER>15200)>, <__main__.S3URL instance at 0x7fe854f7f950>, '', '', [])
  (E)s4cmd.py:359  [Errno 110] Connection timed out
  (D)s4cmd.py:140  >> wrapper(<ThreadUtil(Thread-3, started daemon<PHONE_NUMBER>25344)>, <__main__.S3URL instance at 0x7fe854f7f950>, '', '', [])
  (E)s4cmd.py:359  [Errno 110] Connection timed out
  (D)s4cmd.py:140  >> wrapper(<ThreadUtil(Thread-4, started daemon<PHONE_NUMBER>35488)>, <__main__.S3URL instance at 0x7fe854f7f950>, '', '', [])
  (E)s4cmd.py:194  [Runtime Exception] [Errno 110] Connection timed out
  (E)s4cmd.py:196  Traceback (most recent call last):
  File "/usr/local/lib/python2.7/dist-packages/s4cmd.py", line 337, in run
    self.__class__.__dict__[func_name](self, *args, **kargs)
  File "/usr/local/lib/python2.7/dist-packages/s4cmd.py", line 141, in wrapper
    ret = func(*args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/s4cmd.py", line 838, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/s4cmd.py", line 916, in s3walk
    for key in bucket.list(s3dir, PATH_SEP):
  File "/usr/local/lib/python2.7/dist-packages/boto/s3/bucketlistresultset.py", line 34, in bucket_lister
    encoding_type=encoding_type)
  File "/usr/local/lib/python2.7/dist-packages/boto/s3/bucket.py", line 472, in get_all_keys
    '', headers, **params)
  File "/usr/local/lib/python2.7/dist-packages/boto/s3/bucket.py", line 398, in _get_all
    query_args=query_args)
  File "/usr/local/lib/python2.7/dist-packages/boto/s3/connection.py", line 664, in make_request
    retry_handler=retry_handler
  File "/usr/local/lib/python2.7/dist-packages/boto/connection.py", line 1071, in make_request
    retry_handler=retry_handler)
  File "/usr/local/lib/python2.7/dist-packages/boto/connection.py", line 1030, in _mexe
    raise ex
error: [Errno 110] Connection timed out

  (E)s4cmd.py:194  [Thread Failure] [Errno 110] Connection timed out
Thu Dec  3 17:51:27 UTC 2015
$

Sometimes it is due to unable network connection or S3 throttling. Maybe you can try to play with --retry-delay and --retry to see if you they can help reduce the problem?
Also, check out the new s4cmd 2.0 with more robust boto3 API.


Can't see the RDP access link

I requested the access on Sandbox environment details page, but no link appears to download the RDP file. I can only see the SQLserver\Database name with User name and Password. The docs site says to RDP to the machine and then open SQL server management studio. I am looking to access the entity store database.
Please advise, if I am missing anything.

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

ID: da145b17-ccd8-7bf7-7495-0ddb1bf5829f
Version Independent ID: 2a24d715-ecac-1c65-d2d4-8d14b55b8765
Content: Power BI integration with Entity store - Finance & Operations | Dynamics 365
Content Source: articles/fin-ops-core/dev-itpro/analytics/power-bi-integration-entity-store.md
Service: dynamics-365-cross-app
Product: ****
Technology: ****
GitHub Login: @MilindaV2
Microsoft Alias: milindav

You can't rdp to a tier 2 or above, only SQL access.


Show full path in breadcrumbs

Show full path in breadcrumbs instead of only home link and title of current page: eg: 'home >> ns1 >> ns2 >> page' instead of 'home >> page'

Sorry for being late and thank you for your pull request. Since I need the current breadcrumbs which only show the home and current page, I'll create an option whether to show a full path or not, and I'll use your code there when the full path breadcrumbs option is set to true.


Consider adding company/org names to URLs of pages in browser/

Compare the URLs/page names of the documents in pages/ vs. headsets/.

done, for most (except the obvious ones, such as chrome_for_android)


Shared lib

For projects that need a dynamic version of the ROSS library, libROSS.so is now compiled and installed, in addition to libROSS.a.

If this merge represents a feature addition to ROSS, the following items must be completed before the branch will be merged:

[ ] Document the feature on the blog (See the Contributing guide in the gh-pages branch).
Include a link to your blog post in the Pull Request.
[ ] One or more TravisCI tests should be created (and they should pass)
[ ] Through the TravisCI tests, coverage should increase
[ ] Test with CODES to ensure everything continues to work

Codecov Report

Merging #111 into master will decrease coverage by -0.48%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #111      +/-   ##
==========================================
- Coverage    73.3%   72.82%   -0.48%
==========================================
  Files          29       30       +1
  Lines        3648     3672      +24
==========================================
  Hits         2674     2674
- Misses        974      998      +24

Impacted Files
Coverage Δ

core/tw-signal.c
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 de26e58...fd72bf5. Read the comment docs.


Other components get formik value outside  onchange

❓Question
Hello, how can other components get formik value outside  onchange?
Here is the code  https://codesandbox.io/s/5wmrp396kp
I wanna show formik values on ComponentOne.js  and  ComponentTwo.js but i have no idea how to do that :(
please help, thanks so much.

if you dont want to pass value from render function, you always can use formik's context

You can also keep a local state in the component, and update it before passing to Formik. Sort  of this.

const App = () => {
  let [formValues, setFormValues] = useState(initialValues);
  return (
    <div className="app">
      <Formik ...>
        {props => {
          const {
            ...,
            handleChange,
          } = props;
          const onChange = e => {
            const targetEl = e.target;
            const fieldName = targetEl.name;
            setFormValues({
              ...formValues,
              [fieldName]: targetEl.value
            });
            return handleChange(e);
          };

          return (
            <form onSubmit={handleSubmit}>
              ...
              <input
                name="email"
                ....
                onChange={onChange}
              />
             ....
              <input
                name="email2"
                ....
                onChange={onChange}
              />
              ....
            </form>
          );
        }}
      </Formik>
      <ComponentOne formValues={formValues} />
      <ComponentTwo formValues={formValues} />
    </div>
  );
};

So any ideas how to resolve this problem?

@hyposlasher just use the hook useFormikContext(); like this: const { values } = useFormikContext();. values will have the updated fields

@otaviobps any idea if there's any substitute for userFormikeContext() in v 1.5.7? Its on a react 16.3.1 prehooks era.

@otaviobps any idea if there's any substitute for useFormikContext() in v 1.5.7? Its on a react 16.3.1 prehooks era.

Not sure, sorry but I only use formik with hooks


Scripts in ./scripts and hubot-scripts repo conflict.

I've got a hubot up and running on Heroku, and have written a couple scripts with names that conflict with scripts in the hubot-scripts repo.
As a result of this, when I specify the script name in my bot's hubot-scripts.json, and try to run the scripts through Campfire, both scripts are run.
Am I doing something wrong here, or is this a bug? If I don't specify the scripts in the hubot-scripts.json file, neither script is loaded.

Did you install from the hubot repo or from a released tarball?  You should not be deploying your bot from a hubot checkout.

Installed from a release tarball; specifically, v2.1.0.

Yeah, if you have a script with similar functionality/matches then you'll see duplicates.  Just remove the conflicting script from hubot-scripts.json.

That seems to have fixed it, thanks.


fix/tsconfig/Update tsconfig.cjs.json

Can you explain why this change is required?

Because testing on down stream project with jest will not run