text:
Replace flatbuffers with json serialization library

https://github.com/nlohmann/json
Why?

Has easier serialization APIs for C++ STL storage
Can add custom types (C++ structs -> json)
Can output in human readable (json) and binary format (ubjson, cbor, messagepack etc)

Problems with Flatbuffers

Harder to create a C++ to FBS interface
For JSON like output need to use flexbuffers instead of flatbuffers (no array like root_type allowed)
Schema -> C++ conversion is harder, since there is no explicit support for hashmaps/hashtables/maps/tables.

#223 Initial JSON Serialization library + CustomGenerator addition

#225 JSON Serialization for Target

#226 Remove flatbuffer/flexbuffer usage and library from BuildCC


text:
你好，诚心请教～

你好，我对你开源的项目很有兴趣，在我花费了一周的时间，终于搭建好了环境，但是在开始测试时，出现一些问题。所以诚心想您请教。
1.我根据你发布的教程，进行测试时，和文档中显示的结果不同，original字段没有内容，请问这个是哪里的问题呢？

2.我这边的目的是想训练一个业务客服的机器人，根据提供的业务文档，回复有关业务的问题。请问我想要实现目的，使用哪种模型更合适，同时如何训练模型呢？ （我是一名Android方向的开发者，对于AI 了解的不是很多，所以问题比较多，实在不好意思～～）
期待你的回复，灰常感谢～～

可以留个邮箱 我发你微信我们沟通一下
@Remielpub R

@Remielpub 老哥，我也是Android开发，最近我们公司也要搞这个，有机会交流一下啊。

哈哈，原来都是道友～～ 要不加个QQ吧  咱们一起学习一下。
------------------ 原始邮件 ------------------
发件人:                                                                                "coderabbit214/bibliothecarius"                                                                                @.>;
发送时间: 2023年4月24日(星期一) 上午10:13
@.>;
@.@.>;
主题: Re: [coderabbit214/bibliothecarius] 你好，诚心请教～ (Issue #28)
@Remielpub 老哥，我也是Android开发，最近我们公司也要搞这个，有机会交流一下啊。
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

source: gharchive/issue

authors: Remielpub coderabbit214 panyy

license: MIT

license_type: permissive

license_source: github-api


text:
Add reapplyFilters (or similar name) to the TableFilterHeader

(Original issue 18 created by None on 2011-06-07T13:57:18+00:00)
As part of the previous issue, it would be interesting to being able to reapply the
filters.
In this case, if setFilterOnUpdates does not work, or is set to false, the filters
can be programmatically applied

Original issue reported on code.google.com by luismpena on 2011-06-07 13:57:18

(Original issue 18 last updated on 2011-06-07T13:57:18+00:00)
(Issue automaticaly closed due to status in Bitbucket: wontfix)

(Original comment by coderazzi on 2011-06-07T15:07:59+00:00)
(No text was entered with this change)

Original issue reported on code.google.com by coderazzi on 2011-06-07 15:07:59

(Original comment by coderazzi on 2011-11-09T17:12:37+00:00)
As reported on the previous bug (17),
the issue is a repainting problem on the underlying table,
that is, the filters are applied properly.
To solve the issue, it is enough to request a table.repaint()

Original issue reported on code.google.com by coderazzi on 2011-11-09 17:12:37

repo: coderazzi/tablefilter-swing


text:
Delete doesn't actually delete

code-server version: v1.604-vsc1.32.0
OS Version: macOS 10.14.4 (18E226)

Description
Deleting files from the file tree inside the IDE doesn't actually delete them from disk. It seems to simply remove them from the workspace until code-server is restarted

Expected Files should be sent to the trash/recycle bin
Steps to Reproduce

Run code-server
Right click on a file/folder
Choose delete
Check the local filesystem for that file/folder

also on Mac, and does not work.

@nol166 One part of your issue mentions the IDE doesn't actually delete them from disk while the other mentions Files should be sent to the trash/recycle bin. I personally interpret those two statements as different things. code-server is technically sending files to its equivalent of a trash/recycle bin (the .Trash-1000 folder), but it's not deleting the files immediately from disk. That's because code-server abides by your settings for Files: Enable Trash, which by default is set to true. If you set files.enableTrash to false, code-server it will immediately delete the file.

Looks like trash depends on macos-trash so I suppose we'll want to bundle that with the binary the way we do with ripgrep.

This happens on centos as well. Delete file then refresh fs -> boom file still in the directory

repo: codercom/code-server


text:
01-motivation: second example missing R kernel

In the second example in lesson 01, when opening https://github.com/timalthoff/activityinequality in mybinder, the R kernel is not loaded. According to mybinder documentation (https://mybinder.readthedocs.io/en/latest/using/config_files.html) the repository should have a runtime.txt or a DESCRIPTION for the R kernel to be included.
The simple solution would be to for the repository, fix and add create a pull request. We could use the fork in the meantime But do we want to depend on external repositories keeping up to date?

I think showing real projects here is good. Also that they have room for improvement. We don't expect them to keep up to date but we can discuss how one could improve them.
I am checking whether I get the same problem ...

Getting the same problem but I think it's good to have this problem. I am adding a hint/explanation.

authors: bast rantahar

repo: coderefinery/jupyter


text:
the-project.md: Initial "about the project"

This tries to cover the basic history and the way the project works
internally, so that others are able to join and take part.
The page is currently rather lacking and the opinions of one person,
so it needs careful checking and improvement.
Also, it outlines the kinds of things we need to decide about the
project, since we don't know ourselves yet.

Thank you!

source: gharchive/pull-request

authors: bast rkdarst

repo: coderefinery/manuals


text:
Full lesson summary

I am missing here a full lesson summary that brings everything together and talks a bit about when which of the tools are useful, and when not.

I guess this relates also to issue #237 -> giving the lesson more practical focus. 'What do I do with all this information and how and when do I actually use this stuff?'

repo: coderefinery/reproducible-research


text:
Session 1: Contributing to and reusing CodeRefinery lesson materials

Issue to discuss the full session 1. Please use new issues for single episode discussions and assignments to indicate that you are working on something.
Preliminary learning objectives

Understand how collaborative development of lesson material works
Be able to contribute to existing CodeRefinery lessons
Undertstand how the lesson template is used to create new lessons
Get to know the reasons and sources of inspiration behind major lesson and workshop updates

Initial schedule suggestion:

1 hour for lesson development
1 hour for lesson template
30 min for the survey

Instructor interests

lesson_development.md :  @bast , @rantahar  , @johanhellsvik  , @dianaiusan
lesson_template.md :  @bast , @dianaiusan , @rkdarst
post_workshop_survey.md : YW (needs to accept collaborator rights first)

Host for this session: @samumantha
Coordinator for this session: @rantahar
Related issues: #85 #84 and #88

Draft for lesson development session added (PR #103)

I want to say a bit more about exercises and assessments, but I still need to think a bit first.

15 min intro
45 min lesson design , 15 min break , 45 min lesson template, 15 min break, 45 min survey and feedback
lesson design and survey: RB
lesson template: RD, SW prepares questions

I will add timing info to instructor guide and then I will close the issue.

authors: bast rantahar samumantha

repo: coderefinery/train-the-trainer


text:
Update Splash Screen  - EcoFriendly Feature

Hi,  I have update a Splash Screen, which consist of eco friendly feature.
Attaching a screenshot

Hi @Yatra052 @codervivek5 ,
I am a contributor in GSSoC 24.
Can you please assign this issue to me ??

let's join meeting today at 10 pm link


text:
Role to appear on the profile page

When you open the profile page of any user can we add the Role of this user, So you can know what is his role in the organization.

The Role is already there.

The Role is already there.

authors: Mahmoud-Emad Nayer-Sobhy

repo: codescalers/CSHR


text:
Can't comment or ask questions on Kata solutions

Hello, hoping somebody can help me. I can't post a comment or ask a question on Kata solutions. My account has been verified. There are no errors in my console. I've tried on Safari, Chrome and Firefox ... not working on any of those.

Which solution? I'm guessing this is a duplicate of #2263 (happens for old kata with lots of solutions).

Hello Kazk, thank you for your quick response.
I tried to comment on several of the solutions for the Exes and Ohs challenge https://www.codewars.com/kata/55908aad6620c066bc00002a/solutions/javascript.
I've read through the page #2263 ...The technical detail was a bit over my head but I think I got the gist. The old way of doing this set some kind of limit on user input for comments and votes etc but to solve this issue is labour intensive and will take some time. I accept that... but I must say I found this frustrating and time consuming trying to figure out what was going on... when you're new, you're already trying to take a lot in familiarising yourself with the site so this is quite annoying.
It would be great if there was some explicit way of letting the user know that they can't post because it's an old katas rather than give the impression all is fine allowing you to create a comment but stopping short of actually posting on the click. Some kind of indication better than none to avoid frustration so early on. I really like the site and felt quite excited  by what gains I could make in learning but that positivity was soon replaced with frustration spending so long trying to figure out why I couldn't post a comment or question.

Hi Lulu, I'm having the same issue on several katas. Can't vote on solutions either and didn't find out why until I found your explanation of #2263. I agree with you that this should be either indicated on the page via a disclaimer and/or the comment and voting feature should be removed for old katas.

See https://github.com/codewars/codewars.com/issues/2263#issuecomment-1007176572

authors: aninaA kazk luluUK


text:
Internal Server Error

Seriously...


text:
Debug/sampling

No unit-test nor implementation handling existed for creating proving samples from padded data cells. The unit tests used a data setup where each slot would always contain a power-of-two number of cells, causing padding to never happen and sampling in the range of padding to never happen.
This PR adjusts the test data so padding happens, and adds tests to check that sampling in this range behaves as expected.

I've built an image of this branch and used it to verify this indeed fixes the node crash.

@benbierens  Can we resolve conflicts and get this merged?

authors: benbierens dryajov

repo: codex-storage/nim-codex


text:
Update index.mdx

Que tipo de Pull Request es?

[x ] Mejoras
[ ] Bug
[x] Docs / tests

Descripción
Por favor agrega una descripción de tu aporte para tener más contexto y poder avanzar más rápido. Si es de ayuda puedes usar plataformar como https://www.loom.com/ para grabar un video.

Forma parte de este proyecto.

Discord
Twitter
Youtube
Telegram

Que tipo de Pull Request es?

[x ] Mejoras
[ ]  Bug
[x]  Docs / tests

Descripción
Por favor agrega una descripción de tu aporte para tener más contexto y poder avanzar más rápido. Si es de ayuda puedes usar plataformar como https://www.loom.com/ para grabar un video.

Forma parte de este proyecto.

Discord
Twitter
Youtube
Telegram

No puedes irte sin antes darle like y compartir

repo: codigoencasa/bot-whatsapp


text:
Cpplint issues

I checked veles with cpplint, and cpplint have some issues about 80 characters:
src/parser/unpyc.cc:444:  Lines should be <= 80 characters long  [whitespace/line_length] [2]
src/util/settings/connection_client.cc:218:  Lines should be <= 80 characters long  [whitespace/line_length] [2]

I filled a bug in clang-format for the first case (https://bugs.llvm.org/show_bug.cgi?id=34693), the second one looks like an intended behavior for me - probably some compilers doesn't support splitting #warning strings well.

authors: janczer mkow

repo: codilime/veles

license_source: bigquery


text:
Order of items in QIF export

When importing a QIF in Gnucash 2.6.9 that was generated by the app the transactions for each date are added in reverse order in which they are in the app.
Example:

QIF file has

Day 1:

A
B

Day 2:

C
D

Gnucash on the Desktop will show

Day 1:

B
A

Day 2:

D
C

Could this may be fixed by adding a num field to the QIF file. Looking at the QIF spec, such a field is supported and Gnucash shows one, too. Or is there an easier solution I am overlooking?

@fefe982 this falls more in your expertise. Any input?

Well, a nicer solution would actually be for QIF to support a timestamp as well,
unfortunately it does not.
I also don't know, whether it is a bug in Gnucash, that it reads the
transactions in the QIF file in reverse order, but I guess my issue would also
be fixed if the data would be exported reversed, because than gnucash would
import it in the correct order again.

Ok, having looked at this a bit more, I noticed that the ordering in gnucash is no reverse, but basically arbitrary, due to the fact, that the ordering of gnucash for imports is:
"Date Posted, Num, Date Entered, Description, GUID, ..." [1], so all transactions are sorted alphabetically by their Description.
It would be great if gnucash on Android could keep a rolling (and user-settable) transaction number per account.

So can we close this issue @behrmann ?

Well, not necessarily. Although the bug is with Gnucash in assigning
basically arbitrary order, this could be fixed on this end by providing
a N field for a reference number, according to the QIF
specification.
I'll try to add one to a QIF file myself and have a look whether Gnucash
imports this correctly.

Well, not necessarily. Although the bug is with Gnucash in assigning basically arbitrary order, this could be fixed on this end by providing a N field for a reference number, according to the QIF
specification.
I'll try to add one to a QIF file myself and have a look whether Gnucash imports this correctly.

Yes, but that is a different issue altogether. I think reported in #108
But it would be interesting to know if the N field helps with the GnuCash import

authors: behrmann codinguser

repo: codinguser/gnucash-android


text:
Pie chart feature

Some notes about the implementation:

works on android 2.2 and above
accounts balances are taken in absolute values
displays only the first level of nesting non-placeholder accounts. E.g. if we have the following account hierarchy: Expences/Auto/Gas/Shell V-Power, then will be displayed only Auto as the sum of all sub accounts.

Thanks Oleksandr. :+1:
Looking forward to the other charts which you teased a while back :) ;P

It's nothing. :smile:

authors: aint codinguser


text:
Reuse tab9 binaries across different editors

please complete the following information:

OS version: Mac Mojave
Editor version: 1.51.1
Programming language: any
TabNine extension version: 3.1.10

Issue Details:
I use tabnine in VSCode and Intellij IDEs. Itellij IDEs download tab9 binaries into ~/.tabnine directory, while VSCode extension downloads them into ~/.vscode/extensions/<tabnine ext dir>/binaries. I'd suggest to use one global directory for tab9 executables to save disk space

Hey @fan-tom thanks for reporting this issue!
We are working on a solution to this problem, and sorry that this affects you. We will remove older versions automatically, so Tabnine won't be such a burden on disk space (each version is roughly 20mb in size).
This will take us some time (we want to make sure we do this right).
Sorry for the inconvenience, you can currently manually delete older versions of Tabnine's binary (we follow semver, so leave the 2 latest versions and remove the rest).

authors: boaz-codota fan-tom

repo: codota/tabnine-vscode


text:
Editor: Youtube resizing support

Context
This is a story of master feature #197
The editor is in components/editor
Expected Behavior
Youtube embed should be updated with the media resizer, This is currently already being used by IMG nodes and the same idea should ideally be applied to youtube embeds
Current Behavior
the height and width of youtube videos is set in TiptapExtions and centered in global.css (.Prosemirror iframe )
Steps to reproduce
Please provide detailed steps for reproducing the issue
go to alpha/create/new
add a youtube video
Additional info
https://tiptap.dev/api/nodes/youtube

closing - not in scope for v1

repo: codu-code/codu


text:
nameplates: mark Kui Nameplates as incompatible

Issue Fixed # no double nameplates when using Kui Nameplates @goldpaw

https://mods.curse.com/addons/wow/kuinameplates

repo: cogwerkz/DiabolicUI

license: mit


text:
API Warning: missing_version: Please supply API version as CB-VERSION

When running a cURL request to retrieve pricing for a given exchange rate buy request, I am passing the CB-VERSION header over to the API, however it doesn't appear as though this is always detected by the API, as sometimes it returns the following warning:
public 'id' => string 'missing_version' (length=15)

public 'message' => string 'Please supply API version (YYYY-MM-DD) as CB-VERSION header' (length=59)

Here's a snippet looping through available rates, almost always at least one returns the warning above, but there's a general mix in the # of rates which return the warning, which may display against BTC one run, and not the next.
$rates = [
      'BTC-USD',
      'BCH-USD',
      'ETH-USD',
      'LTC-USD',
];

foreach($rates as $rate)
{
       $url = 'https://api.coinbase.com/v2/prices/'.$rate.'/buy';

       $curl = curl_init();

       curl_setopt($curl, CURLOPT_URL, $url);
       curl_setopt($curl, CURLOPT_RETURNTRANSFER, 1);

       $headers = [
           'CB-VERSION: 2018-05-21',
       ];

       curl_setopt($curl, CURLOPT_HTTPHEADER, $headers);

       $result = curl_exec($curl);
       $data   = json_decode($result);
       var_dump($data);

       curl_close($curl);
}

This doesn't prevent pricing being retrieved, however it would be good to understand whether this is an endpoint issue and for it to be resolved, or get to the bottom if it's a localised issue.

It appears this is an API issue.
Having spent more time looking at this and sending requests via Postman, I can send requests without the CB-VERSION header and get a valid response back, subsequent requests may then throw the warning, and then work again. It seems like this is related to a Coinbase price change, as the state seems to persist until the price changes.
Will raise to api@coinbase.com
Closing, as unrelated to the PHP library, however may be useful for anyone searching this issue in future.

repo: coinbase/coinbase-php


text:
Cancelled order not in my book

I want to execute some code after a trade is completed/fully filled so I have the following event listener in place:
trader.on('Trader.trade-finalized', (msg: TradeFinalizedMessage) => {
    logger.log('info', 'Order complete', JSON.stringify(msg));
});

Now I'm getting the following message instead: 2017-10-28T21:07:28.294Z - warn: Trader: Cancelled order not in my book...
Why is this happening and how to fix it?

I had this similar issue quite often a couple weeks ago when trying. It seems that the book (for one's own orders, not the public orderbook) is buggy. I haven't spent much time debugging. Perhaps a contributor can take a look?

Is the order being placed by the Trader object? Or are you placing the trade manually somewhere else?

@CjS77 the order is being placed by the Trader object:
const order: PlaceOrderMessage = {
    time: null,
    type: 'placeOrder',
    productId: 'LTC-EUR',
    size: '1',
    price: '48.50',
    side: 'buy',
    orderType: 'limit',
    postOnly: true
};

trader.executeMessage(order);

repo: coinbase/gdax-tt


text:
Hacked sandbox support.

You can almost use the new sandbox working by setting up omniauth like this:
Rails.application.config.middleware.use OmniAuth::Builder do
  provider :coinbase, Settings.coinbase_client_id, Rails.application.secrets.coinbase_client_secret,
    scope: 'user', client_options: {site: 'https://sandbox.coinbase.com'}
end

But omniauth-coinbase tries to fetch the current user info from 'https://sandbox.coinbase.com/api/v1/user' which doesnt seem to exist, we need to use 'https://api.sandbox.coinbase.com/v1/user' when using the sandbox.
This patch fixes the problem in a slightly hacky way.

Sandbox is officially supported as of v2.1.0

authors: aianus williamcoates

repo: coinbase/omniauth-coinbase


text:
[syncer/storage] Pre-store Blocks

WARNING: THIS PR INCLUDES BREAKING CHANGES TO STORAGE!
Changes

[x] Pre-store blocks as soon as they are seen by the syncer (instead of waiting to store until blocks are sequenced)
[x] Replace accountEntry with *types.AccountCurrency
[x] Manually encode/decode *types.AccountCurrency
[ ] Ensure ctx changes in syncer do not introduce regression in rosetta-cli

Pull Request Test Coverage Report for Build 13337

101 of 154 (65.58%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 78.339%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

storage/encoder/encoder.go
101
154
65.58%

Totals

Change from base Build 13199:
-0.03%

Covered Lines:
5718

Relevant Lines:
7299

💛  - Coveralls

authors: coveralls patrick-ogrady

repo: coinbase/rosetta-sdk-go


text:
avoid requiring tmp_path introduce in pytest 3.9.0

Reverts #243 and avoids the use of tmp_path introduced by #230 instead.
Easier to read whitespace-ignoring diff: https://github.com/colcon/colcon-core/pull/246/files?w=1
@rotu FYI.

Codecov Report

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

@@           Coverage Diff           @@
##           master     #246   +/-   ##
=======================================
  Coverage   80.04%   80.04%
=======================================
  Files          54       54
  Lines        3107     3107
  Branches      514      514
=======================================
  Hits         2487     2487
  Misses        582      582
  Partials       38       38

Continue to review full report at Codecov.

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

Thanks for pointing out the problems!

authors: codecov-io dirk-thomas

repo: colcon/colcon-core


text:
Zod schema serialize/deserialize

How can I serialise/deserialize zod schema?
So that I can store it in database for example as a string and reuse later when needed.

Can you explain more about your use case? Perhaps a give a code example.

repo: colinhacks/zod


text:
어절 텔에비

SUSSY BAKA
어절 텔에비 라는 단어 추가좀요

ㅗ

SUSSY BAKA는 너가 추가하세요


text:
FIX: Update the important notice and github issue linking for this repo's pull request template

This pull request makes the following changes

Fixes collabo-community/issue-tickets-ready-for-fixing#8

General checklist

[x] My pull request adheres to the Collabo Community pull request guidelines.
[x] I have read and understood the Pull request guideline extension section found at the bottom of this pull request description/body.
[x] I have replaced the dummy text under the subheading Testing checklist below with the testing checklist from the issue ticket I'm working on.

Testing checklist

[x] The important notice and Github issue linking has been updated for the pull request template.
[x] I certify that I ran my checklist

Pull request guideline extension

[!NOTE]
Do not remove this note or section. This talks about extra things to take note of, in addition to the directions given in the Collabo Community pull request guidelines.

We appreciate the time taken to help tackle issue tickets within the Collabo Community. Please go ahead to submit your pull request even if it is not yet completed, and push to the pull request actively. This will allow us to know that you have not abandoned the issue ticket. However, you should only alert reviewers about your pull request as explained below, to avoid flooding their GitHub notifications unnecessarily.
"Ready for review" implies that things are working as expected, according to how it is specified in the issue ticket you are working on. Ensure that you test the code that you are working on, that it works as expected. Don't just request for review when your pull request is not yet done and working.
The only time you should be making people review something incomplete is: if you want to show them what's not working, so that they can send help or guidance. If this is the case, be explicit about this in our Discord community or in the comment of your pull request (after you have submitted the pull request).

Please see my comments on the Discord thread @Festusali

Necessary corrections effected. Please confirm it's now sorted @Ifycode

@all-contributors add @Festusali for code

authors: Festusali Ifycode

license: ISC


text:
nullable field isn't nullable if also restricted to a list of choices

Describe the bug
Fields with nullable=True aren't considered nullable when also using choices=...
To Reproduce
Create a model with the following:
from enum import Enum
from ormar import Model, Integer, String
import sqlalchemy
import databases

URL = "postgresql://..."  # url to postgres db

metadata = sqlalchemy.MetaData()
database = database.Database(URL)

class MyChoices(Enum):
    A = "A"
    B = "B"

class Item(Model):
    class Meta:
        tablename="items"
        metadata = metadata
        database = database

    id = Integer(primary_key=True)
    my_field = String(max_length=1, nullable=True, choices=list(MyChoices))

app = FastAPI()
app.state.database = database

@app.on_event("startup")
async def startup() -> None:
    database_ = app.state.database
    if not database_.is_connected:
        await database_.connect()

@app.on_event("shutdown")
async def shutdown() -> None:
    database_ = app.state.database
    if database_.is_connected:
        await database_.disconnect()

@app.post("/upload/")
async def upload(item: Item):
    await Item.save()

Upon trying a POST request to that endpoint, if my_field is set to None, it will fail with a ValueError:
{"detail": [{"loc": ["body", 0, "__root__"], "msg": "my_field: 'None' not in allowed choices set: ['A', 'B']", "type": "value_error"}]}

Expected behavior
The model accepts "A", "B", or None for my_field, because it has a set of valid choices and is nullable. It can be worked around by adding None to the enum, but that does not seem natural to me:
  class MyChoices(Enum):
      A = "A"
      B = "B"
+     NONE = None

Versions

Database backend used: postgres
Python version: 3.9.7
ormar version: 0.10.19
pydantic version: 1.8.2
fastapi version: 0.68.1

That's the same issue as described in #353, i will introduce the extra parameter on Model meta, that will allow to ignore or forbid the extra fields

I don't think it is the same issue. I am only sending it the required/allowed fields, but the choices= restriction on the nullable field isn't accepting None unless I put None in my list of choices

Yep you are right, nullable should still cause the field to be optional. Thank you for catching that!

I don't think it is the same issue. I am only sending it the required/allowed fields, but the choices= restriction on the nullable field isn't accepting None unless I put None in my list of choices

Sorry last post was in wrong issue 🤦

oh no problem, thanks!

Fixed in 0.10.20

authors: classabbyamp collerek

repo: collerek/ormar


text:
Toggle between search algorithms

Create a way to toggle between search algorithms so that the user can select the search algorithm for generating the maze.
Likewise, implement this solution for toggling between search algorithms that solve the maze.

Hey, I want to work on this please assign this to me.

Any update on this?

@colliecoder sorry for inconvenience I was busy in college projects. I will currently not able to work on it,

@A-Spiral-Forge No problem! School projects definitely have to come first - thanks for letting me know.

authors: A-Spiral-Forge colliecoder

repo: colliecoder/maze-winner


text:
How to use GPS information in pycolmap

Hello, is there a function interface like model_aligner that uses GPS information to adjust the model in pycolmap? thanks for your help.

Hi @TNTSAYou, you can use reconstruction.align(image_names, locations, min_common_images) or  reconstruction.align_robust(image_names, locations, min_common_images, max_error=12.0, min_inlier_ratio=0.1), which aligns the reconstruction to the gps-positions in-place and returns the transformation. locations should be a list of world coordinates [x, y, z] here.

@Phil26AT  Thanks for your help, I found this method, ready to apply to my dataset. Thanks again for your answer and the pycolmap project scheme.

repo: colmap/pycolmap


text:
ACL Implementation

Have ACLs been implemented? There were comments in December saying this change was going in :)

Hi @Priminus , I still haven't found the time to do so. It's the first thing on the list to address though :)

authors: Priminus beeman

repo: colmena/colmena-cms


text:
native mobile app (or - at least - offline)

A native app would let this work without internet connection and may also enable additional features not available to web pages.
Additionally, or alternatively, we could look into service workers to enable working offline.

More seemingly relevant info about service workers on iOS: https://adactio.com/journal/16619

repo: colorimetry/colorimetry-net


text:
Stop handling string filters as regexps

Previously, filter strings passed to add_filter have been handled as regexps. It means, for example, add_filter '.pl' matches against sample.rb because . is handled as any character in regexp.
As now we have regexp filter, there's no reason to handle filter strings as regexps.
However we need to think about semver with this change. If this behavior was intentional, we cannot introduce this change until next major version bump. Or we can do if this was a bug.
What do you think?

:wave:
Hi, first thanks for your contribution :tada:
This is an interesting issue and a very astute observation! A problem is that for all that I can tell it has been behaving like this forever.
However, the README doesn't seem to mention that it has regexp like behaviour:
All in all, I'm in favor, the example you provided is especially frightening :scream: So thanks for that!
There's another issue I might need to reach out to Christoph for, just wanna double check with him as I'm not aware of all the history/intentions.
Regarding version, in 0.y.z versions you don't need a major version bump for changes:

Major version zero (0.y.z) is for initial development. Anything may change at any time. The public API should not be considered stable.

If we fix it I'd internally classify it as a "bug" but as it breaks behaviour old as time I'd still bump a minor to release it so people can have a proper warning looking at the changelog because patch level updates (imo even in 0.y.z) should be safe.

If we fix it I'd internally classify it as a "bug" but as it breaks behaviour old as time I'd still bump a minor to release it so people can have a proper warning looking at the changelog because patch level updates (imo even in 0.y.z) should be safe.

I forgot the fact that simplecov is still under 1.0 👍

Hey all, sorry for my late reply! Thanks for the PR, looks very reasonable to me!
As to the potentially breaking change:
Personally, I'd say I originally built it like this out of shear laziness. I agree that it should not automatically expand into a regexp under the hood and magically. I also agree that this could be a breaking change for existing users, since a significant behaviour changes. Under SemVer that should warrant a major version bump. We really should finally declare something 1.0 :grin: I think what @PragTob suggested, to bump minor version should be fine :rocket:

Well then :shipit:
Thanks @colszowka

authors: PragTob colszowka yujinakayama

repo: colszowka/simplecov


text:
How to handle company specific extensions

Companies need some way to add their own specific extions to the SCL files.
Questions:
Where to store the XSD extensions?
How to validate the XSD extsions?
How to use them?

I think the most important part for now is the validator and there is a separate issue for that there (https://github.com/com-pas/compas-scl-validator/issues/116).
No need to have it in the JAXB2 Converter for now. And that's maybe also possible if a custom config is used and a JAR file is added for the Company Specific generated classes, but that probably also means some kind of custom component build by the company itself.
So close it for now.

repo: com-pas/compas-core


text:
[Bug]: Help text wrong with RebindAllTheKeys

Describe the Bug
If the RebindAllTheKeys mod is installed, I can rebind F3+Q to something else. With BetterF3, the help text will still tell me to press F3+Q.
Modloader Type
Fabric
Modloader Version
1.19.2
Mod Version
1.4.0

I think the best way to fix this would be to open the jar file with an archive extractor and manually modify the text.betterf3.line.help_press value.


text:
"The commit-hook project sounds interesting though. I would definitel…

…y merge something that runs flake8 and pylint on the modified files!"

pd0wm, https://github.com/commaai/openpilot/pull/1575#issuecomment-634974344

pylint needed some more config options and work to pass. See https://github.com/commaai/openpilot/pull/1580
Specifically it couldn't be run in an isolated virtualenv because it uses dynamic analysis and checks all the imports. Once I get that to replace our current pylint CI step I'll look at flake8.
Currently we have a whole list of violations we don't check for (see .pylintrc), we should make small PRs where we gradually remove those and fix them everywhere in the codebase.

repo: commaai/openpilot


text:
tici: fix ui rotation

iirc the frame looked stretched due to some hand coded aspect ratio assumption somewhere in paint.cc. can you also fix that?

Yes, will check that out too.


text:
params: safe and efficient async writing parameters

The original version of put_nonblocking may create many threads(which are not joined) waiting on the FileLock .
If the process exits while the thread is runing, these pending write operations will fail with exception.

Can you write a good test for this?

@deanlee seems there's still one test failure
=================================== FAILURES ===================================
_________________ TestPowerMonitoring.test_pandaState_present __________________

self = <selfdrive.thermald.tests.test_power_monitoring.TestPowerMonitoring testMethod=test_pandaState_present>

    def test_pandaState_present(self):
      pm = PowerMonitoring()
      for _ in range(10):
        pm.calculate(None, None)
      self.assertEqual(pm.get_power_used(), 0)
>     self.assertEqual(pm.get_car_battery_capacity(), (CAR_BATTERY_CAPACITY_uWh / 10))
E     AssertionError: 30000000 != 3000000.0

selfdrive/thermald/tests/test_power_monitoring.py:38: AssertionError

checking

fixed. it's a bug.the std::future<T>::~future  does not always block waiting for the thread to finish when running in python/pytest
fix: do not reply on ~future(), explicit call wait in dctor.

improved test case to catch this issue.

Nice cleanup!

Had a seemingly related failure on master: https://github.com/commaai/openpilot/actions/runs/6102074880/job/16559742075

I'll check again,  I think this issue may not be related to asyc write

@adeebshihadeh : think we still need to mock the nonblocking put to blocking put
the writing thread of static AsyncWriter will continue to run if the queue is not empty after the params is destroyed/re-create in TestPowerMonitoring.setUp(self).
this  causes  test_pandaState_present to fail if 'CarBatteryCapacity' is written by asyncWriter before init car_battery_capacity_uWh from self.params.get("CarBatteryCapacity")

reopend https://github.com/commaai/openpilot/pull/29808
pass param prefix  to asyWriter to fix test issues https://github.com/commaai/openpilot/pull/29808/commits/3fb13e429093507458513512d50628b454173a83

authors: adeebshihadeh deanlee


text:
How do I print the help screen?

 Issue by ptr727
Tuesday May 23, 2017 at 17:02 GMT
Originally opened as https://github.com/gsscoder/commandline/issues/445

How do I print the help output I've evaluated the logical options and something is wrong?
            // Parse the commandline arguments
            // https://github.com/gsscoder/commandline
            Parser parser = new Parser(with =>
            {
                with.CaseSensitive = false;
                with.HelpWriter = Console.Error;
            });
            ParserResult<Options> result = parser.ParseArguments<Options>(args);
            if (result.Tag == ParserResultType.NotParsed)
            {
                return -1;
            }
            Options options = ((Parsed<Options>)result).Value;

            // Process or monitor must be specified
            if (options.ProcessFolders == false && options.MonitorFolders == false)
            {
                // TODO : Call help output directly
                return -1;
            }

 Comment by NVanderEnde
Wednesday May 24, 2017 at 15:53 GMT

Are you looking for HelpText.AutoBuild(args)?

 Comment by ptr727
Friday May 26, 2017 at 16:09 GMT

How do I use it?

 Comment by NVanderEnde
Friday May 26, 2017 at 16:20 GMT

Like so:
  private static string HelpOutput(Parameters args)
    {
        var result = new StringBuilder();

        result.AppendLine("Hello, and welcome to the  console application.");
        result.AppendLine("This application takes in a data file and attempts to import that data into our systems.");
        result.AppendLine("Valid options are:");
        result.AppendLine(HelpText.AutoBuild(args));
        result.AppendLine("Press any key to exit");

        return result.ToString();
    }

where Parameters is some class that's been annotated with the Option attributes, etc.

 Comment by ptr727
Friday May 26, 2017 at 20:14 GMT

I get a runtime exception:
            // Parse the commandline arguments
            // https://github.com/gsscoder/commandline
            Parser parser = new Parser(with =>
            {
                with.CaseSensitive = false;
                with.HelpWriter = Console.Error;
            });
            ParserResult<Options> result = parser.ParseArguments<Options>(args);
            if (result.Tag == ParserResultType.NotParsed)
            {
                Tools.WriteLineError("Failed to parse commandline.");
                return -1;
            }
            Options options = (((Parsed<Options>)result).Value);

            // Process or monitor must be specified
            if (options.ProcessFolders == false && options.MonitorFolders == false)
            {
                Tools.WriteLineError(CommandLine.Text.HelpText.AutoBuild(result));
                Tools.WriteLineError("Failed to parse commandline, missing action.");

                return -1;
            }

Fails at runtime.
System.ArgumentException occurred
  HResult=0x80070057
  Message=Excepting NotParsed<T> type.
  Source=CommandLine
  StackTrace:
   at CommandLine.Text.HelpText.AutoBuild[T](ParserResult`1 parserResult, Int32 maxDisplayWidth)
   at Plex.Program.Main(String[] args) in C:\Users\piete\OneDrive\Projects\Plex\Plex\Program.cs:line 44

 Comment by veleek
Tuesday Jul 25, 2017 at 23:23 GMT

I believe that this is definitely a bug in the code.  If you use the overload of AutoBuild(...) that takes onError and onExample functions then it will write successfully.
It actually has an explicit block in the code which checks whether or not the ParserResult<T> being passed in is a NotParsed result in which case it gracefully attempts to write the error output.
if (onError != null && parserResult.Tag == ParserResultType.NotParsed)
{
    errors = ((NotParsed<T>)parserResult).Errors;

    if (errors.OnlyMeaningfulOnes().Any())
        auto = onError(auto);
}

So, as a workaround in the meantime you can just use
Console.WriteLine(HelpText.AutoBuild(result, null, null));

 Comment by cbertolasio
Wednesday Jun 14, 2017 at 18:10 GMT

I also get the same error at runtime using my own sample project.  Probably a different topic... but it is unclear what the best practices are to get the helptext to be output when the args follow the "help verb" or "help" construct using the v2.x codebase - documentation is dated and so is the sample.

static int Main(string[] args)
{
	var results = Parser.Default.ParseArguments<SetupDirectionCommandOptions>(args);
	var exitCode = 0;
	if (args[0].Equals("help", StringComparison.InvariantCultureIgnoreCase))
	{
		var text = HelpText.AutoBuild(results);
		Console.WriteLine(text.ToString());
		return 0;
	}
	else
	{
		exitCode = results.MapResult(
		(SetupDirectionCommandOptions opts) => RunSetupDirection(opts),
		errs => (int)1);
	}

	return exitCode;
}

this is what my verb looks like...

[Verb(Verbs.SetupDirectionKey, HelpText = "Setup a direction command")]
public class SetupDirectionCommandOptions : SubOptionsBase
{
	[Option('r', "rotation-type", Default = PivotRotation.None, HelpText = "Set the pivot direction")]
	public PivotRotation Direction { get; set; }

	[Option('v', "velocitymode-type", Default = VelocityMode.AutoVri, HelpText = "Set the velocity mode")]
	public VelocityMode VelocityModeType { get; set; }

	[Option("percent", Default = 100, HelpText = "Set the speed percentage")]
	public int Percent { get; set; }

	[Option('f', "flowrate", Default = 6, HelpText = "Set the flow rate")]
	public float FlowRate { get; set; }

	[Option("ispumpon", Default = true, HelpText = "Set the is pump on flag")]
	public bool IsPumpOn { get; set; }
}

The above workaround does not work for me (throws a NullReferenceException, using CommandLineParser 2.6.0), however only a simple change was needed to make it work:
Console.WriteLine(HelpText.AutoBuild(result, _ => _, _ => _));

repo: commandlineparser/commandline


text:
[Bug]:  Theme Button is not at the position when open in mobile phone

Describe the bug

Click on the drop-down menu
You will see the theme button will shift from it's postion and will appear on mid left side

Screenshot/ Video

@kaiwalyakoparkar please assign this issue to me . I would like to work on it

hey Neetesh Why are you sending a PR without assigning this issue to you?
@kaiwalyakoparkar @harkiratsm this is not fare!!
I solved this issue and waited to assigning this issue to me and Neetesh send PR without assigning.
So, Please assign this issue to me or should i also make a PR without assigning

repo: commclassroom/commclassroom


text:
dev postfinance

includes the implementation of "prepayment", "creditcard", "sofortüberweisung", "postfinance-card" "postfinance-efinance" and "paypal"
@lauraluiz @schleichardt pls review

I will not be able to review this soon.

I will review it tomorrow

authors: lauraluiz mht-dotsource schleichardt


text:
Display λ in prompt

i can not find any reason why we don't display λ in prompt but ‽. i mean in every prompt of a FP language, there will be a λ. 😅
Btw, there is one thing confusing me a lot while walking through the code base, why we need to a temp file, and all the infos are coming from the temp file, e.g. type-at ?

Btw, there is one thing confusing me a lot while walking through the code base, why we need to a temp file, and all the infos are coming from the temp file, e.g. type-at ?

That's orthogonal, but could that be to prevent races between the user changing code and the typechecking process?

@Blaisorblade Preventing races sounds reasonable :) Thx!

That's orthogonal, but could that be to prevent races between the user changing code and the typechecking process?

authors: Blaisorblade zjhmale

repo: commercialhaskell/intero


text:
FPT_ROT_EXT.2 Selections invoke other requirements

"computation and verification of a hash by trusted code/data" invokes FCS_COP.1/Hash.
"verification of a digital signature by trusted code/data" invokes FCS_COP.1/SigVer.
"generating an audit event" invokes FAU_GEN.1, FAU_SAR.1, FAU_STG.1, FAU_STG.4, and FAU_STG_EXT.1.
"Initiate a Recovery process as specified in FPT_RVR_EXT.1" invokes FPT_RVR_EXT.1
"in accordance with administrator-configurable policy" invokes a FMT_MOF_EXT.1, FMT_SMF.EXT.1, and also implies that there is an authenticated administrator, so...
The places where we say "administrator/user" should be changed to [selection: administrator, user]. If "administrator" is selected then all the requirements pertaining to administrator authentication are invoked, that is FIA_UIA_EXT.1, FIA_UAU.7, FIA_AFL_EXT.1, FIA_PMG_EXT.1 & more? FIA_UAU.5 should probably be sel-based rather than Optional.
The difference between admins and users is that admins are authenticated to the platform/TOE. Users might be authenticated to tenant software, but need not be authenticated to the platform.

Done. All loose ends tied up.

repo: commoncriteria/gpcp

license: Unlicense


text:
Fix crash(es) due to uninitialized notification channel.

The notification channel needs to be created for API versions greater than OR EQUAL to 26 (O).  Also, the channel does not need to be reinitialized if it already exists.
This will likely fix #1877

@dbrant Could you please rebase on 2.8-release and resubmit this PR to that branch? I can test as soon as you do that, thanks :)

Codecov Report

Merging #1904 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #1904      +/-   ##
=========================================
- Coverage    3.69%   3.69%   -0.01%
=========================================
  Files         192     192
  Lines        9912    9914       +2
  Branches      886     887       +1
=========================================
  Hits          366     366
- Misses       9519    9521       +2
  Partials       27      27

Impacted Files
Coverage Δ

...n/java/fr/free/nrw/commons/CommonsApplication.java
28.12% <0%> (-0.91%)
:arrow_down:

Continue to review full report at Codecov.

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

authors: codecov-io dbrant misaochan

repo: commons-app/apps-android-commons


text:
[GSoC] Full Screen Mode

Tasks

[x] Detect swipe gestures
[x] Swipe left to view the previous image
[x] Swipe right to view the next image
[x] Swipe up to select the picture (the equivalent of tapping it in non-fullscreen mode) and show the next picture (the picture that follows in the folder view, skipping pictures that have either already been uploaded or ignored).
[x] Swipe down to immediately mark that picture as "Not for upload" (the equivalent of selecting it then tapping "Mark as not for upload" in non-fullscreen mode), and show the next picture.

Wonderful, thanks for the fast fix!

There are some changes left to do. I will commit those soon.

Understood, thanks!

I get build errors indeed, such as Conflicting overloads: public abstract suspend fun findByImageSHA1

Yeah, I have seen that. I will create a new PR with required changes,
merging FS Mode and Hide/Unhide feature, I will solve this build issue on
that PR.
On Sat, Aug 20, 2022, 5:34 PM Nicolas Raoul @.***>
wrote:

I get build errors indeed, such as Conflicting overloads: public abstract
suspend fun findByImageSHA1
—
Reply to this email directly, view it on GitHub
https://github.com/commons-app/apps-android-commons/pull/5032#issuecomment-1221302456,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ7HSBIEAB56OFOPHCOI66TV2DCVJANCNFSM56ECOCOA
.
You are receiving this because you authored the thread.Message ID:
@.***>


text:
Add page titles

To save having to repeat the service name and GOV.UK each time, I wonder if we can use a helper, and just have pages provide a :title
Here’s an example: https://github.com/DFE-Digital/apply-for-teacher-training/blob/1a37fb2cce6c933529eddfa818e4283c56f90966/app/helpers/application_helper.rb#L7

authors: kosiakkatrina paulrobertlloyd


text:
GitLab Merge request remain not covered after docu-signed

Steps :

Go to gated Easycla PR
Click on Not covered
Proceed as an Individual Contributor
Click on Sing CLA
Fill docusign pdf form

Actual results : GitLab Merge request remain not covered after docu-signed
Refer video clip :
https://images.zenhubusercontent.com/194341141/464fc3e1-b6a5-432d-8939-b673edf2d172/notcovered.mp4

Tested issue on dev site & found that it is reproducible as reported, So making issue status QA-Failed
Refer video clip :
https://user-images.githubusercontent.com/75417507/220549895-683583ca-bb3a-4952-93e8-56cfbe8f12b8.mp4

MR : https://gitlab.com/marsgitlabgroup/repo01/-/merge_requests/32

Tested issue with multiple user & found that it is working fine except veerendrat user

Added separate tickets for failed user : https://app.zenhub.com/workspaces/easycla-5ed619832c71a855a76ef404/issues/gh/communitybridge/easycla/3816

Refer screenshot for more details :

@thakurveerendras confirmed its working as expected. Hence closing

authors: shyam-goshike thakurveerendras

repo: communitybridge/easycla


text:
Test

Signed-off-by:Adam Jesse ak1jesse@gmail.com

Are you sure you meant to send this PR to the main repo? Adding a letter "a" on a line by itself?

That was a mistake, my apologies.
-----Original Message-----
From: Instructor, Computer Systems Technology
Sent: 12 Feb 2015 00:43:48 GMT
To: comp4711/winter2015-lab04
Cc: AndroidAdam
Subject: Re: [winter2015-lab04] Test (#5)
Are you sure you meant to send this PR to the main repo? Adding a letter "a" on a line by itself?

Reply to this email directly or view it on GitHub:
https://github.com/comp4711/winter2015-lab04/pull/5#issuecomment-73996518

authors: AndroidAdam jim-parry


text:
uibModal service not being injected?

Having the same issue as [this user] (http://stackoverflow.com/questions/33605162/inject-provider-into-directive-not-working).
This is the console error
Unknown provider: $uibModalProvider <- $uibModal <- Lightbox <- Lightbox

ui.boostrap is being required correctly and used on other elements in the same controller
var accountApp = angular.module('accountApp', [
  'ngRoute',
  'ngResource',
  'messageModule',
  'accountControllers',
  'ui.bootstrap',
  'Devise',
  'braintree-angular',
  'ngFileUpload',
  'bootstrapLightbox',
]);

Which version of UI Bootstrap? I think this happens with v0.13.x and below.

Yep that was it.  I was using ui-bootstrap 0.13.4x and upgrading to 0.14.3x fixed it.
Thank you.

authors: compact jordangraft

repo: compact/angular-bootstrap-lightbox


text:
Idva6 430 bulk email sender in add association v2

Short description outlining key changes/additions
Integrate bulk email sender into addAssociation endpoint.
JIRA Ticket Number
IDVA6-430
Type of change

[ ] Bug fix
[x] New feature
[ ] Breaking change
[x] Infrastructure change

Pull request checklist

[x] I have added unit tests for new code that I have added
[x] I have added/updated functional tests where appropriate
[x] The code follows our coding standards

An exhaustive list of peer review checks can be read here

CI: Security warnings found!

authors: ch-code-analysis krishna-patel-ch

repo: companieshouse/accounts-association-api


text:
Add payment service integration

Add support for user web journey through GOV.pay
When form requires payment:

show 'Payment amount' value
use alternate submit button text
navigate to payment service

When form doesn't require payment:

hide 'Payment amount' value
use original submit button text
navigate to Confirmation page

also

remove redundant PaymentRequiredModel class
remove redundant paymentReference tokens in messages.properties
add unit test coverage

Note:
Requires update to private-api-sdk-java version from https://github.com/companieshouse/private-api-sdk-java/pull/83. POM dependency version is unversioned until that PR is merged and the new version is known.
Requires:

https://github.com/companieshouse/efs-submission-api/pull/5
https://github.com/companieshouse/private-api-sdk-java/pull/83

Note: Updated the POM with appropriate SDK release version

repo: companieshouse/efs-submission-web


text:
UAR-1619 Send new NOCs for Update/Remove journey Gov or PA BO

JIRA link
https://companieshouse.atlassian.net/browse/UAR-1619
Change description

Addition and change services updated
Unit tests added

Work checklist

[X] Tests added where applicable

Merge instructions
We are committed to keeping commit history clean, consistent and linear. To achieve this, this commit should be structured as follows:
<type>[optional scope]: <description>

and contain the following structural elements:

fix: a commit that patches a bug in your codebase (this correlates with PATCH in semantic versioning),
feat: a commit that introduces a new feature to the codebase (this correlates with MINOR in semantic versioning),
BREAKING CHANGE: a commit that has a footer BREAKING CHANGE: introduces a breaking API change (correlating with MAJOR in semantic versioning). A BREAKING CHANGE can be part of commits of any type,
types other than fix: and feat: are allowed, for example build:, chore:, ci:, docs:, style:, refactor:, perf:, test:, and others,
footers other than BREAKING CHANGE: <description> may be provided.

CI: No security warnings found

Can look again once the main branch has been merged in.

CI: No security warnings found

Looks ok, post merge.

authors: ch-code-analysis mattch1 mwestacott

repo: companieshouse/overseas-entities-api


text:
use version number instead of release in .travis.yml

release will change over time, but your REQUIRE file says this package supports julia 0.4 so it should continue to be tested

Coverage increased (+2.7%) to 95.89% when pulling 0a11690365757205ee0528774404e5deba52e1a5 on tkelman:travisver into 68ee995e3ede3e214eab58fc54259bdf76bb684e on compressed:master.

Thanks!

authors: compressed coveralls tkelman


text:
Exercise sheet 2, task 3 b) is too difficult

3 b) is too difficult especially because

Hint: Apply the same steps as we used in the lecture for the logistic regression

Is not correct. There was nothing of the sort in the videos. (I assume it used to be)

Updated pdfs:
exercise_2.pdf
solution_2.pdf

authors: HeidiSeibold tpielok


text:
Division object is projected to next frame

Reported by @Sagnik700 in #64
Testing micro-sam with 0.1.2.post1  version and with MiaPaCa200x200 dataset, daughter cell segmentation of a frame was visible in the previous frame where the parent cell was annotated as division in track_state. Screenshot 1(Frame 22) shows the parent cell segmentation in brown and the daughter cell segmentation in blue. The blue segmentation should not be visible in Screenshot 1 since it is appearing in Screenshot 2(Frame 23) onwards.
Steps to replicate the issue:

Segment the exact cell which is highlighted as division as track_state in frame 22
Segment the daughter cell as highlighted as track 2 as track_state in frame 23 and click on Track Object
Come back to frame 22 to notice the issue

Originally posted by @Sagnik700 in https://github.com/computational-cell-analytics/micro-sam/issues/64#issuecomment-1660880123

Hi @Sagnik700 ,
I created a separate issue for this. It should be fixed by #114 now, but I haven't checked if it's really fixed.
Can you pull the master and confirm that it's fixed?
Thanks!

Yes @constantinpape I just checked now and the issue seems to be fixed!!

Reopening the issue since the issue reappeared while checking with 117

repo: computational-cell-analytics/micro-sam


text:
Question: Different ways of running an automated module

The aim is to access the api object for sending a message anytime a message is sent in a thread. exports.match is always called for matches so my hack was to set a local variable with the state from exports.match and return true to access said variable in exports.run; check the state and send the appropriate message using api.
var state = false;
exports.match = (event, commandPrefix) => {
    state = true;
    return true;
}

exports.run = (api, event) => {
    if(state) {
        api.sendMessage(automatedFoo(), event.thread_id);
    }
}

Is there an easier/alternative way of doing this? I'm probably forgetting something fundamental provided by the API.

Hmmm. So to clarify you are just trying to use the api whenever a message is received from a particular integration? I probably need more detail on the specific task to give helpful suggestions.
Modifying your current example to be slightly less of a hack and avoid race conditions:
exports.match = (event) => {
    event.state = true; // event object will be preserved between match and run
    return true;
}

exports.run = (api, event) => {
    if(event.state) {
        api.sendMessage(automatedFoo(), event.thread_id);
    }
}

You could just match everything and perform your matching task inside run.
exports.match = () => true;
exports.run = (api, event) => {
    if (event.event_source === 'wherever') {
        // do your thing
    }
}

although I don't see currently why this check can't be done as follows:
exports.match = (event) => event.event_source === 'whatever';
exports.run = (api, event) => {
    // do thing
}

In general match should only be used to determine if your module should run, not how it should run or what to do when it is running - there are always exceptions to this (associate passes information from its match method to its run method to avoid a duplicated search).

Thank you for the pointers! I'm currently writing a module that basically checks the current date once Concierge has started (or once user interaction is received from an integration) and sends a message if it's the special day. It's like a birthday-wishing module.
Ideally, I'd like to perform the logic without interaction from a user; once integrations have successfully loaded.

once integrations have successfully loaded

That is a difficult one... some integrations like facebook report that they are loaded when they actually aren't (nothing we can do to fix that I am aware of). There is a current hacky approach to finding out if they are loaded in the say-hi module, but that only works a portion of the time. In the beta branch this problem still exists but can be handled a little more gracefully by listening for the start event.
Have you considered a different approach? Perhaps when the module is loaded, it could check on an interval timer if the date is a particular one (or even make a timeout that will callback at exactly the right time for the next event). Then you could use exports.platform.getIntegrationApis() to get the appropriate api and send the message.
This would mean you aren't waiting for someone to send a message to your module and it would work independently of the status of the integrations (if an integration isn't running when the callback occurs, just don't send the message).

Great idea! Thanks I'll try that; might have to store thread_id some way. I'll update here as things progress.
Thanks for the link

Does this issue still need to be open?

I believe not. The project has gone stale and I'll re-open if I REALLY need to.
Thanks

authors: ParasDPain mrkno

repo: concierge/Concierge


text:
first attempt to fix broken session

if an auth client in portal doesn't have redirect urls for lara, then after LARA tries to log in the session becomes broken.
With this broken session, if the user tries to log directly into the portal (not going through lara), it doesn't work.
The exception raised shows that the oauth code path is still being traversed.
Unfortunately the fix in this commit doesn't actually fix the problem, but it seems like one piece of it.
I think the reason the problem persists is because something is setting the 'user_return_to' session variable.  This session variable can be seen in the exception info printed in the log.
When the exception is raised initially due to the misconfigured auth client, this session variable remains set. I'm not sure yet where that variable gets set.

@pjanik if you have time can you take a look at this and see if you have any insights?
The issue causes problems for Ethan when he was setting up his dev environment. And I figured out how to reproduce it.

I'm looking at it. I think the problem was clear, the app had never an opportunity to delete session[:oauth_authorize_params], so there's a neverending loop.
Your solution seems perfectly fine and I'd expect it to fix the problem.
Although, I can see that session[:oauth_authorize_params] params are still present on the next page load / sign in attempt, even if we deleted that before. It seems like the session is not being saved properly when an exception is raised. But I don't know yet if it's a feature or bug.
I didn't any see references to user_return_to yet.

@scytacki, here's something that seems to brutally fix the problem:
https://github.com/concord-consortium/rigse/pull/689
I wish I knew why the session can't be updated normally. But maybe throwing an exception in after_sign_in_path_for isn't working the best. I'm wondering about user_return_to, but I haven't really seen it, and I could clearly see that my session isn't updated.
My 3rd approach that I was considering a long time ago would be even different:

Use after_sign_in_path_for only to construct path to oauth_authorize with query params saved in session[:oauth_authorize_params] and redirect there. So we're really getting back there after signing in.
Call AccessGrant.get_authorize_redirect_uri(current_user, params) in oauth_authorize method and deal with errors a bit nicer.

But still, we might need to use reset_session in after_sign_in_path_for rescue block, as even the simpler code defined in 1. can raise some error (it's just less likely). Also, I'm lost about auth_popup flow that would be more difficult to keep around. So it might not be worth it at this point.

@pjanik thanks for coming up with that solution. I've looked into this more and figured out that the user_return_to comes from Devise's failure app, we have customized this. I'm going to spend a bit more time on this and see if I can fix it without using reset_session.
One problem with reset_session is that if a user is logged into the portal, and then they try to log into a misconfigured client they will get logged out of the portal and it won't be obvious why.

OK I finally convinced myself that that the user_return_to has no effect on this. And it does seem right the issue is that the session is not saved to the database when an exception occurs. So I'm going with your PR.

authors: pjanik scytacki

repo: concord-consortium/rigse


text:
Change the resource_caches index to use md5 of version

The version can potentially become too long, and creating an index for
the columns will fail. See https://github.com/concourse/concourse-pipeline-resource/issues/22
for an issue.

@ebabani Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@ebabani Thank you for signing the Contributor License Agreement!

sorry we slept on this, could you rebase? the CI build should be more reliable now

Will do the rebase this weekend.

@ebabani We went ahead and merged it manually. Thanks!

authors: ebabani pivotal-issuemaster vito

repo: concourse/atc


text:
Allow sending multiple emails per connection

CommunityTranslation is logging a lot of timeouts when it tries to send notification emails.
I think this is caused by the SendGrid rate limits: 10 concurrent connections from a single server.
This is a reasonable limit: IMHO the problem is with the current implementation of our mail service: we open a new connection for every email to be sent.
What about adding an option to set the number of messages per SMTP connection (that defaults to 1 for backward compatibility)?

PS Sending multiple emails per connection is also ways faster

I just added a couple of commits.
@KorvinSzanto what about https://github.com/mlocati/concrete5/commit/fe95f41b5402662645d9d13e35748f2d6d377746 ? It would be good if everybody get the mail service instance the right way, but it would be a BC break if someone instantiates the class manually...

This looks good to me. Thanks for all the work and the effort changing the stuff I nitpicked!

@mlocati the one last thing on my mind is the rest of the API for that SMTP transport class. I recall you raised the valid point that anything assuming the transport was an SMTPTransport would be accessing methods that won't exist on this new decorator. Did you want to add those or are we just going to disregard?

anything assuming the transport was an SMTPTransport would be accessing methods that won't exist on this new decorator

I think it's safe enough to disregard this: by default we don't limit the number of messages per connection to be sent, so people by default will still get a Zend\Mail\Transport\Smtp instance in case of smtp transports.
If someone wants the transport underlying the decorator, they can use the getSmtpTransport method.

Sounds good to me.
On Thu, Mar 30, 2017 at 11:22 PM Michele Locati notifications@github.com
wrote:

anything assuming the transport was an SMTPTransport would be accessing
methods that won't exist on this new decorator
I think it's safe enough to disregard this: by default we don't limit the
number of messages per connection to be sent, so people by default will
still get a Zend\Mail\Transport\Smtp instance in case of smtp transports.
If someone wants the transport underlying the decorator, they can use the
getSmtpTransport method
https://github.com/mlocati/concrete5/blob/5dfec6e46a65c08bc6cfcc2fb05c0933fc211f15/concrete/src/Mail/Transport/LimitedSmtp.php#L49
.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/concrete5/concrete5/pull/5233#issuecomment-290627599,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA9fO1qU7-ItwLMPhSFhfrlY3HcxKlSHks5rrJuwgaJpZM4MjqL8
.

@KorvinSzanto Thank you very much for your review and feedback

authors: KorvinSzanto mlocati


text:
Fix dependencies

Checklist

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

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

@xhochy if you have time to review that, it would be great.

authors: conda-forge-linter xmnlab

repo: conda-forge/arrow-c-glib-feedstock


text:
Unbundle Try 2

Checklist

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

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

authors: conda-forge-linter hmaarrfk


text:
Update package pins now that we're building again.

Pretty straightforward.

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

authors: conda-forge-linter pkgw

repo: conda-forge/cairo-feedstock


text:
Enable Windows build

This requires boost-cpp 1.76 - we need to wait for that migration to happen. There is some weird error in boost 1.74 that was fixed later on.

See #4

repo: conda-forge/colmap-feedstock


text:
Restore py38 support

Checklist

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

Fixes https://github.com/conda-forge/eups-feedstock/pull/25#issuecomment-1001727127 .  cc @isuruf
We might need to mark build 0 as broken, at least on py=3.8?
I checked that this patch works with py38 and 39.

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

authors: conda-forge-linter erykoff

repo: conda-forge/crossenv-feedstock


text:
update 2021.4

Checklist

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

@conda-forge-admin, please restart ci

@conda-forge-admin, please restart ci


text:
updated 2.1.2

Checklist

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

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

authors: aschonfeld conda-forge-linter

repo: conda-forge/dtale-feedstock


text:
Fix executables installed on Linux

Newer Meson apparently is more aggressive about stripping rpaths from installed executables, so the binary tools installed on Linux didn't work anymore. Manually tell Meson to keep the rpaths in the installed executables, and add tests to try to catch this in the future.

Checklist

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

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

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

Failed to even lint the recipe, probably because of a conda-smithy bug :cry:. This likely indicates a problem in your meta.yaml, though. To get a traceback to help figure out what's going on, install conda-smithy and run conda smithy recipe-lint . from the recipe directory.

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


text:
render recipe (last render had failed)

Checklist

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

The last render-attempt had failed as the dependency ttach was not available on conda-forge yet. Now it is. So, just rerender.

@conda-forge-admin, please rerender

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

authors: conda-forge-linter sugatoray

repo: conda-forge/grad-cam-feedstock


text:
Enable Windows

Checklist

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

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

authors: conda-forge-admin jeongseok-meta

repo: conda-forge/inplace-abn-feedstock


text:
jupyterlab-gt-coar-theme 0.2.4

Checklist

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

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

@conda-forge-admin please rerender

authors: bollwyvl conda-forge-linter


text:
Fix license name

This package is GPLv2+, not 3+.
Checklist

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

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The build section contained an unexpected subsection name. error_overdepending is not a valid subsection name.
The build section contained an unexpected subsection name. error_overlinking is not a valid subsection name.

authors: conda-forge-linter duncanmmacleod

repo: conda-forge/lal-feedstock


text:
Add a Windows build

We really need this on Windows.
We have a really hacky Windows recipe here:
https://github.com/NOAA-ORR-ERD/orr-conda-recipes/tree/master/libgd
only known to work with py2.7 -- i.e. VS2008
I"m not sure how to make different versions with different compilers -- features?
but this could be a starting place.

Sorry, I don't have a windows machine, I cannot really help you here 👎

This had been done for a while :-)

repo: conda-forge/libgd-feedstock


text:
update release 1.0.1

Checklist

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

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

@conda-forge-admin, please rerender

authors: conda-forge-linter runette

repo: conda-forge/mdal-feedstock


text:
HDF5 linkage issues

Copying from @alimahmoudi29's comment.

I received the following error:
Traceback (most recent call last):
  File "/anaconda3/bin/mspms", line 4, in <module>
    import msprime.cli
  File "/anaconda3/lib/python3.6/site-packages/msprime/__init__.py", line 26, in <module>
    from _msprime import FORWARD  # NOQA
ImportError: dlopen(/anaconda3/lib/python3.6/site-packages/_msprime.cpython-36m-darwin.so, 2): Library not loaded: @rpath/libhdf5.10.dylib
  Referenced from: /anaconda3/lib/python3.6/site-packages/_msprime.cpython-36m-darwin.so
  Reason: image not found

I fixed the issue by
mv libhdf5.101.dylib libhdf5.10.dylib  in directory /usr/local/lib .

Would add that it appears that hdf5 is not pinned in this package as it should be. The package should be rebuilt with hdf5 correctly pinned. We will need to pull the packages that were not pinned once this is done.

Ugh, sorry for this @jakirkham, my bad. I was trying to muddle my way through without bothering you and the other maintainers, but it appears this has done more harm than good...
I've pinned HDF5 following the examples in other feedstocks, so hopefully it's OK now. Let me know if there's anything I can do to help clear up the mess.

No worries. If you see this issue other places, please us know.
Looks better. Have pulled the packages that didn't have hdf5 pinned, which should avoid users getting these accidentally. Please double check that we got them all and got the right ones.

Great, thanks. Just checked the packages on anaconda cloud, and they look right to me.

It works now, thanks

authors: alimahmoudi29 jakirkham jeromekelleher

repo: conda-forge/msprime-feedstock


text:
8.0.28

Checklist

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

Supersedes https://github.com/conda-forge/mysql-feedstock/pull/62

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

Thank you for rebasing the 2 patches :)

authors: conda-forge-linter jonashaag nehaljwani

repo: conda-forge/mysql-feedstock


text:
Update to 0.8.0

Checklist

[*] Used a personal fork of the feedstock to propose changes
[*] Reset the build number to 0 (if the version changed)
[*] Ensured the license file is being packaged.

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

authors: conda-forge-linter nnansters

repo: conda-forge/nannyml-feedstock


text:
make this cftime until the feedstock is created

@jhamman we will archive this feedstock once the proper one is created.

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

authors: conda-forge-linter ocefpaf

repo: conda-forge/netcdftime-feedstock


text:
MAINT: Bump to HDF5 1.14.1

Checklist

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

@conda-forge-admin, please rerender

repo: conda-forge/openmeeg-feedstock


text:
Fix openjdk upper version

Checklist

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

Running generate with openjdk 22 raises:
PogoException: java.lang.RuntimeException: Problems instantiating module org.tango.pogo.generator.PogoDslGeneratorMWE: Cannot resolve proxy: java:/Objects/java.lang.String#java.lang.String

@conda-forge-admin, please rerender

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

authors: beenje conda-forge-admin

repo: conda-forge/pogo-feedstock


text:
bump libctl and HDF5 versions

Checklist

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

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

Jinja2 variable references are suggested to take a {{<one space><variable name><one space>}} form. See lines [18].
License is not an SPDX identifier (or a custom LicenseRef) nor an SPDX license expression.

Documentation on acceptable licenses can be found here.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

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

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

authors: conda-forge-linter oskooi

repo: conda-forge/pymeep-feedstock


text:
meshpy missing dependencies gcc and pybind11 with 'conda install meshpy'

as i say in the title, when i tried to $ pip install meshpy,

i had an error with libstd++.h (??) I dont remember the exact name of .h missing)
i installed gcc : $ conda install gcc
Then i tried $ pip uninstall meshpy followed by $ pip install meshpy the installation complains about some clang++ and pybind11 missing
I installed pybind11 with $ conda install pybind11
finally $ pip uninstall conda followed by $ pip install conda succeeded.
and that's it, meshpy is installed<!--

This has nothing to with pyqt, closing.

repo: conda-forge/pyqt-feedstock


text:
Updated to 1.2.5

- Bump version: 1.2.4 → 1.2.5. [Martin Raspaud]

- Fix EWA fornav for integer data and add test. [davidh-ssec]

  There was a bug when doing the averaging in EWA where the current pixel was being rounded based on the previous pixel's value instead of the current pixel. This only affects integer data because values above 0 are rounded up by 0.5 and values below 0 are rounded by 0.5, for floats this round value is 0.0.

- Fix certain compilers not liking integers being passed to isnan.
  [davidh-ssec]

- Replace catch_warnings in all tests with astropy version. [davidh-
  ssec]

- Use catch_warnings from astropy (or at least try to) [davidh-ssec]

- Test removing version specific warning checks in `test_swath_wrap`
  [davidh-ssec]

- Move USE_CYTHON handling to if main block in setup.py. [davidh-ssec]

- Fix isnan definition only if a macro doesn't already exist. [davidh-
  ssec]

  Numpy does some special macro stuff to define a good npy_isnan function. Some systems define a macro for it, others don't. Hopefully this works for all systems. A better solution might be to define a templated isnan that calls npy_isnan if it isn't an integer.

- Fix EWA compile failure on windows python 3.5. [David Hoese]

- Make pykdtree install on appveyor optional. [davidh-ssec]

- Add pykdtree to appveyor dependencies. [davidh-ssec]

- Fix setup.py test on windows for multiprocessing tests. [davidh-ssec]

  On Windows when new processes are started the initially command is imported or re-executed. For setup.py this is a big problem since the usual boilerplate does not include `if __name__ == "__main__"` so the setup.py test command gets rerun and rerun. This results in the child processes never actually being run for newer versions of python (2.7+). There still seems to be an issue with `test_nearest_resize` on Windows.

- Merge pull request #41 from cpaulik/fix-windows-ewa. [David Hoese]

  Fix Windows CI import Error

- Install scipy in Windows CI to fix import problems. [Christoph Paulik]

- Fix copy/paste error in EWA fornav. [davidh-ssec]

  I had started rewriting EWA in cython then realized it was faster in straight C++ so copied/pasted the cython code and modified it. Seems like I missed this 'or' hanging around.

- Fix NAN constant/macro for EWA on Windows. [davidh-ssec]

- Merge branch 'add-windows-CI' into fix-windows-ewa. [davidh-ssec]

- CI: Add IOOS conda channel to get basemap for Windows and python > 2.
  [Christoph Paulik]

- Merge branch 'add-windows-CI' into fix-windows-ewa. [davidh-ssec]

- Add pyproj to conda install in Appveyor CI. [Christoph Paulik]

- Make extra_compile_args platform dependent. [Christoph Paulik]

- Add Appveyor CI configuration. [Christoph Paulik]

- Fix EWA resampling's isnan to work better with windows. [davidh-ssec]

- Update changelog. [Martin Raspaud]

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

repo: conda-forge/pyresample-feedstock


text:
@conda-forge-admin please rerender

Additional comment:
No response

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I rerendered the recipe in conda-forge/qt-webengine-feedstock#5.

repo: conda-forge/qt-webengine-feedstock


text:
R3.5

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (@conda-forge-admin, please rerender)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

Using toolchain directly in this manner is deprecated.  Consider using the compilers outlined here.

Hi! This is the friendly automated conda-forge-webservice.
I tried to re-render for you, but it looks like there was nothing to do.

authors: cbrueffer conda-forge-linter

repo: conda-forge/r-gee-feedstock


text:
MNT: Re-rendered with conda-smithy 3.1.12 and pinning 2018.10.11

R 3.5.1

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

repo: conda-forge/r-sda-feedstock


text:
Add sbpy to RSP

Checklist

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

@conda-forge-admin please rerender

repo: conda-forge/rubinenv-feedstock


text:
tpot 0.12.1 doesn't work with python 3.12

Solution to issue cannot be found in the documentation.

[X] I checked the documentation.

Issue
tpot 0.12.1 doesn't work with python 3.12 because it attempts to import imp in base.py. Suggesting you change tpot_python in the recipe to specify 3.11 as the max python version currently supported.
Installed packages
python tpot numpy pandas

Environment info
n/a

Probably no version of tpot ever has worked with python 3.12. So even if changed on the feedstock (on e.g. #38), it would take additional work to make something useful happen (a solver would just go walk back to the previous version).
If this support is important for you, it would likely be most useful to continue the conversation upstream on https://github.com/EpistasisLab/tpot/issues/1327

I see you updated the recipe. Thank you! What this means is that someone typing "conda create -n automl tpot numpy pandas" at least will get a functional environment as it'll know to not install python 3.12 but to bump it down to 3.11.

There is no guarantee of that, it might just pull an older tpot that doesn't have the pin.

Ahh... I see it now. I didn't fully understand that the solver might go
that direction instead.
On Mon, Feb 26, 2024 at 1:43 PM Nicholas Bollweg @.***>
wrote:

There is no guarantee of that, it might just pull an older tpot that
doesn't have the pin.
—
Reply to this email directly, view it on GitHub
https://github.com/conda-forge/tpot-feedstock/issues/39#issuecomment-1965120684,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AYLTXVLDLJCR7XF5HEPAUZDYVTQV3AVCNFSM6AAAAABD2EYFBOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNRVGEZDANRYGQ
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Peter VanWylen
Data & Technology Services and Consulting
Founder, United InfoLytics LLC
unitedinfolytics.com https://unitedinfolytics.com / 901-214-5317
https://unitedinfolytics.com

Could add a repodata patch for the old versions to add an upper bound
Admittedly some of them may need an even lower upper bound that just 3.12 (like if they don't work with 3.11 either for example)
Here is an example: https://github.com/conda-forge/conda-forge-repodata-patches-feedstock/pull/584

Yep, on it. noarch: python was added 0.11.2, so nothing prior to that is affected by this incompatibility.
Here's the basic idea (still waiting for the diff):
# uses now-removed `imp` module, see:
# - https://github.com/conda-forge/tpot-feedstock/issues/39
# - https://github.com/EpistasisLab/tpot/issues/1327
if:
  name: tpot
  version_lt: 0.12.2
  version_ge: 0.11.2
  timestamp_lt: 1708983815000
then:
  - tighten_depends:
      name: python
      upper_bound: 3.12

Got this up, but had some local issues vs anaconda.org: https://github.com/conda-forge/conda-forge-repodata-patches-feedstock/pull/673

authors: bollwyvl jakirkham petervanwylen

repo: conda-forge/tpot-feedstock


text:
ENH: update package version

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #34.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #34

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

No valid build backend found for Python recipe for package tsam using pip. Python recipes using pip need to explicitly specify a build backend in the host section. If your recipe has built with only pip in the host section in the past, you likely should add setuptools to the host section of your recipe.

Hi! This is the friendly automated conda-forge-webservice.
I tried to rerender for you, but it looks like there was nothing to do.
This message was generated by GitHub actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/11346641520.

authors: conda-forge-admin julian-belina

repo: conda-forge/tsam-feedstock


text:
Update to v2.20.0.20240331

Checklist

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

@conda-forge-admin, please rerender

repo: conda-forge/types-hdbcli-feedstock


text:
Update to v9.1.0946

Checklist

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

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

Thank you for the PR @blmaier! Would you like to add yourself as a maintainer here?

repo: conda-forge/vim-feedstock


text:
updated to v0.21.3

Checklist

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

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

authors: abloemert conda-forge-linter

repo: conda-forge/zeroconf-feedstock


text:
Token based authentication for jupyterlab users in Conda-Store

Right now there is no easy for a user to authenticate from jupyterlab launched via jupyterhub transparently. Currently there is a workaround to allow the gator extension to modify create/update/delete environments.
c.RBACAuthorizationBackend.unauthenticated_role_bindings = {
    '*/*': ['admin'],
}

This makes it so that the REST api/UI effectively considers anyone(unauthenticated) to be an admin.
In order to properly handle this we must accept a jupyterhub token from the jupyterlab gator extension and get the credentials for the user based on this. Possibly this means we should expose a token based authentication method for Conda-Store.

Is this still applicable @costrouc?

@trallard this issue is resolved as conda-store does have token based authentication and it is possible to generate new tokens.

authors: costrouc trallard

repo: conda-incubator/conda-store


text:
REL - 2024.11.2

Pull request checklist

[x] Did you test this change locally?
[x] Did you update the documentation (if required)?
[x] Did you add/update relevant tests for this change (if required)?

I guess it’s okay that the server and UI versions are not the same?

I guess it’s okay that the server and UI versions are not the same?

yep. We did a release of the server earlier in the month. So the next version should be 2024.11.2.


text:
AttributeError: 'str' object has no attribute 'content'

❗BEFORE YOU BEGIN❗
Are you on discord? 🤗 We'd love to have you asking questions on discord instead: https://discord.com/invite/a3K9c8GRGt
Describe the bug
AttributeError: 'str' object has no attribute 'content'
To Reproduce
class AWSBedrock(DeepEvalBaseLLM):
def init(
self,
model
):
self.model = model
def load_model(self):
    return self.model

def generate(self, prompt: str) -> str:
    chat_model = self.load_model()
    return chat_model.invoke(prompt).content

async def a_generate(self, prompt: str) -> str:
    chat_model = self.load_model()
    res = await chat_model.ainvoke(prompt)
    print(res)
    return res.content

def get_model_name(self):
    return ""

llm = BedrockLLM(model_id='mistral.mistral-large-2402-v1:0', client=bedrock,streaming=True)
aws_bedrock = AWSBedrock(model=llm)
Expected behavior
A clear and concise description of what you expected to happen.
Traceback
Traceback (most recent call last):
File "/home/ubuntu/scp-analyzer/rag-old/dpeval.py", line 185, in
synthesizer.generate_goldens_from_docs(
File "/home/ubuntu/.local/lib/python3.10/site-packages/deepeval/synthesizer/synthesizer.py", line 900, in generate_goldens_from_docs
return loop.run_until_complete(
File "/usr/lib/python3.10/asyncio/base_events.py", line 649, in run_until_complete
return future.result()
File "/home/ubuntu/.local/lib/python3.10/site-packages/deepeval/synthesizer/synthesizer.py", line 860, in a_generate_goldens_from_docs
return await self.a_generate_goldens(
File "/home/ubuntu/.local/lib/python3.10/site-packages/deepeval/synthesizer/synthesizer.py", line 676, in a_generate_goldens
await asyncio.gather(*tasks)
File "/home/ubuntu/.local/lib/python3.10/site-packages/deepeval/synthesizer/synthesizer.py", line 223, in _a_generate_from_contexts
res, _ = await self.a_generate(prompt)
File "/home/ubuntu/.local/lib/python3.10/site-packages/deepeval/synthesizer/synthesizer.py", line 91, in a_generate
return (await self.model.a_generate(prompt), 0)
File "/home/ubuntu/scp-analyzer/rag-old/dpeval.py", line 143, in a_generate
return res.content
AttributeError: 'str' object has no attribute 'content'
Desktop (please complete the following information):
python:3.10
deepeval:0.21.71
Additional context
Add any other context about the problem here.

This issue is difficult to read and lacks details. Anyway, I'll try to help you.
It seems to me that the model you are trying to use does not return the expected type.
Please have a look at the following AWS Bedrock example: https://github.com/confident-ai/deepeval/blob/main/docs/docs/metrics-introduction.mdx#aws-bedrock-example
The ainvoke method seems to return a str, see https://github.com/langchain-ai/langchain/blob/master/libs/core/langchain_core/language_models/llms.py#L360-L379
So in your existing code, you might just change the line return res.content to return res as the result already is the generated string.
Let me know if that works.

repo: confident-ai/deepeval