text:
Refactor GP unit tests to use data fixtures

A lot of the GP unit tests are defining the same data in multiple test cases, resulting in huge code files with large redundant parts.
This task is to use the fixture concept to pull data generation to a class that the tests then extend.
There are some tests in Shogun that are doing this (right @vigsterkr ?). It is a nice task to get familiar with fixtures, googletest and Shogun in general

@vigsterkr can you give reference to those tests?

grep for GaussianProcess in shogun/tests/unit

Hello, I am trying to work on this issue as no one seems to have worked on it for quite some time.
I have identified these two files that have GP unit tests:
classifier/GaussianProcessClassification_unittest.cc
regression/GaussianProcessRegression_unittest.cc
I also went through some other shogun unit tests where fixtures have been used such as FisherLDA_unittest.cc, BinaryLabels_unittest.cc, and RandomForest_unittest.cc.
In the file GaussianProcessClassification_unittest.cc test cases GaussianProcessClassificationUsingSingleLaplaceWithLBFGS, GaussianProcessClassificationUsingKLCovariance, GaussianProcessClassificationUsingKLCholesky, etc. define and use the same data. Since the test case names are different in these, they cannot use the same test fixture class. Would it be fine to create a base fixture class and create several other classes with appropriate names for each test case that inherit from this base class?

Hi @durovo and welcome! Great that you want to work on this
That would be ok. In fact, anything that reduces the bloated code.

a class that generates the data shared by all these tests
a class that sets up the GP pipeline that is used in many of the tests (e.g. one for each of the inference methods, one for regression/classification, etc, .... anything that is set up more than twice should be grouped this way

Let me know if you have any questions

Hello @karlnapf ,  I have refactored GaussianProcessClassification_unittest.cc. However in GaussianProcessRegression_unittest.cc, the same data is almost never used more than twice in the nine tests. Also, since we can have only one fixture class for each TestName, I cannot think of any good way of refactoring it.

Closed in #3990

authors: Sahil333 durovo karlnapf

repo: shogun-toolbox/shogun


text:
WIP: Run CI Build with Silenium container

#6

Ok, it's working. Tomorrow need finish last step - delete all containers after all work completed or after any error.

@amitaibu ready for code review.

@amitaibu updated.

To confirm PR:

Fork https://github.com/amitaibu/shoov-behat
Re-create encrypted keys (use the POST /api/encrypt endpoint), and place in your .shoov.yml
Create a few CI build items (message) of the same repo
execute drush create-ci-build-item --verbose

source: gharchive/pull-request

authors: amitaibu solker

repo: shoov/shoov-nodejs

license: mit

license_type: permissive

license_source: bigquery


text:
Assistant transferres GMT+01:00 Order timestamps as UTC

Used Shopware Version: 6.5.8.2
Description:
when you migrate orders there seems to happen a timeshift on the timestamps when not both systems are set to utc?
Environment:
PHP 8.2.16,
see SW6 version + Assistance version,
Server time said Fri Feb 23 11:19:05 CET 2024
source and target database-version is
10.6.17-MariaDB-1:10.6.17+maria~deb11
and CURTIME() also always shows CET (currently UTC+01:00) timestamps
Steps to reproduce:
Use a SW6 account with Timezone set to "(UTC +01:00) Europe/Berlin" (not sure if relevant to reproduce) then configure the Migration Assistant - in this case we only migrated Customers + Orders (besides the enforced default Salechannel+Categories Migration setting)
Expected result:
Have the timestamps set and calculated according to the old systems
Actual result:
You happen to even see orders in the new SW6 Admin which are set to happen "in 34 Minutes" then just in the past (due to the one hour time shift)  [Screenshot_20240223_132750-1.jpg|https://sbp-issuetracker-attachments.s3.eu-west-1.amazonaws.com/1708691643_65d890bba313d_1708691596_65d8908c4b4b7_Screenshot_20240223_132750-1.jpg]  [Screenshot_20240223_130234-1.jpg|https://sbp-issuetracker-attachments.s3.eu-west-1.amazonaws.com/1708691644_65d890bc21256_1708691596_65d8908c78f40_Screenshot_20240223_130234-1.jpg]

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/MIG-946.
[created from MIG-946, comment 484786]

I can confirm this Bug, the data gets migrated with the exact date, but SW5 uses GMT+2, while SW6 uses UTC. Thus all Orders have a timeshift of 2Hours. New Orders are shown to be in the future, as was mentioned in the initial Post.
SW 6.6.6.1
Assistant: 13.2.0
Had the PHP-Timezone default at first, then tried out if setting it to "Europe/Berlin" and UTC would change anything.
Setting the Profile Timezone also did not help, I could change it to UTC to see the actual Times, but i guess that Customers in the Shop-Frontend would still see the wrong time.

[public] Hello,
Within our company roadmap and work capacity, we try to address each bug or improvement request but admit that not each one will be resolved. To continue our culture of honesty and openness, we are closing this issue to focus on our roadmap on behalf of all Shopware users.
If you feel like this issue is still impacting you, please create a new issue and let us know. Thank you again for your valuable feedback.
[created from MIG-946, comment 501322]

source: gharchive/issue

authors: danielDigitalArt shopwareBot

repo: shopware/shopware

license: MIT

license_source: github-api


text:
Aggregation may produce incorrect result, if the filtering produces LEFT JOIN queries

**Description: **
Example. If you filter the orders via order.lineItems.product.category_ids, and the order has several matching lineItems, the count and stat.sum are calculated incorrectly because due to LEFT JOIN this order is aggregated several times.
**Steps to reproduce: **

delete all today's orders
Place an order with two products in the same category
Stat-aggregation with filter on this category

Expected result:
count = 1, stat.sum = order.amount
Actual result:
count = 2, stat.sum = 2*order.amount
Example:
api payload
{
    "page": 1,
    "limit": 25,
    "filter": [
        {
            "type": "range",
            "field": "orderDateTime",
            "parameters": {
                "gte": "2024-01-07T23:00:00.000Z"
            }
        },
        {
            "type": "equalsAny",
            "field": "lineItems.product.categoryIds",
            "value": "513427899de64feba7a6c1f2e63f3d71"
        }
    ],
    "aggregations": [
        {
            "type": "histogram",
            "name": "order_count_histogram",
            "field": "orderDateTime",
            "interval": "day",
            "format": null,
            "aggregation": {
                "type": "stats",
                "name": "stats",
                "field": "amountNet"
            }
        }
    ],
    "associations": {
        "addresses": {
            "total-count-mode": 1
        },
        "billingAddress": {
            "total-count-mode": 1
        },
        "salesChannel": {
            "total-count-mode": 1
        },
        "orderCustomer": {
            "total-count-mode": 1
        },
        "currency": {
            "total-count-mode": 1
        },
        "documents": {
            "total-count-mode": 1
        },
        "transactions": {
            "sort": [
                {
                    "field": "createdAt",
                    "order": "ASC",
                    "naturalSorting": false
                }
            ],
            "total-count-mode": 1
        },
        "deliveries": {
            "total-count-mode": 1
        },
        "lineItems": {
            "associations": {
                "product": {
                    "associations": {
                        "categories": {
                            "total-count-mode": 1
                        }
                    },
                    "total-count-mode": 1
                }
            },
            "total-count-mode": 1
        }
    },
    "total-count-mode": 1
}

Then the following query is generated in \Shopware\Core\Framework\DataAbstractionLayer\Dbal\EntityAggregator::fetchAggregation:
SELECT MIN(`order`.`order_date_time`) as `order_count_histogram.key`,
COUNT(`order`.`id`) as `order_count_histogram.count`,
AVG(`order`.`amount_net`) as `stats.avg`,
MIN(`order`.`amount_net`) as `stats.min`,
MAX(`order`.`amount_net`) as `stats.max`,
SUM(`order`.`amount_net`) as `stats.sum`
FROM `order`
LEFT JOIN `order_line_item` `order.lineItems` ON `order`.`id` = `order.lineItems`.`order_id` AND `order`.version_id = `order.lineItems`.order_version_id
LEFT JOIN `product` `order.lineItems.product` ON `order.lineItems`.`product_id` = `order.lineItems.product`.`id` AND `order.lineItems`.`version_id` = `order.lineItems.product`.`version_id`
WHERE (`order`.`version_id` = 0x0FA91CE3E96A4BC2BE4BD9CE752C3425)
AND (((`order`.`order_date_time` >= '2024-01-07T23:00:00.000Z')
AND (JSON_CONTAINS(`order.lineItems.product`.`category_ids`, JSON_ARRAY('513427899de64feba7a6c1f2e63f3d71')))
))
GROUP BY DATE_FORMAT(`order`.`order_date_time`, '%Y-%m')
ORDER BY `order`.`order_date_time` ASC

Order is calculated twice for count and for sum because 2 rows are created due to LEFT JOIN
The filtering and aggregation should then be done separately, so that each order is only represented once for the aggregation
SELECT MIN(`order`.`order_date_time`) as `order_count_histogram.key`,
COUNT(`order`.`id`) as `order_count_histogram.count`,
AVG(`order`.`amount_net`) as `stats.avg`,
MIN(`order`.`amount_net`) as `stats.min`,
MAX(`order`.`amount_net`) as `stats.max`,
SUM(`order`.`amount_net`) as `stats.sum`
FROM `order` WHERE `order`.id IN (SELECT `order`.id FROM `order`
LEFT JOIN `order_line_item` `order.lineItems` ON `order`.`id` = `order.lineItems`.`order_id` AND `order`.version_id = `order.lineItems`.order_version_id
LEFT JOIN `product` `order.lineItems.product` ON `order.lineItems`.`product_id` = `order.lineItems.product`.`id` AND `order.lineItems`.`version_id` = `order.lineItems.product`.`version_id`
WHERE (`order`.`version_id` = 0x0FA91CE3E96A4BC2BE4BD9CE752C3425)
AND (((`order`.`order_date_time` >= '2023-12-31T23:00:00.000Z')
AND (JSON_CONTAINS(`order.lineItems.product`.`category_ids`, JSON_ARRAY('513427899de64feba7a6c1f2e63f3d71'))) ))
)
GROUP BY DATE_FORMAT(`order`.`order_date_time`, '%Y-%m')
ORDER BY `order`.`order_date_time` ASC

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/NEXT-33008.
[created from NEXT-33008, comment 486125]

[public] Hello,
Within our company roadmap and work capacity, we try to address each bug or improvement request but admit that not each one will be resolved. To continue our culture of honesty and openness, we are closing this issue to focus on our roadmap on behalf of all Shopware users.
If you feel like this issue is still impacting you, please create a new issue and let us know. Thank you again for your valuable feedback.
[created from NEXT-33008, comment 492327]


text:
Workaround for MariaDB >= 10.2.7 compatibility

1. Why is this change necessary?
When Shopware is run with MariaDB and the old articles attributes definition from before 5.3 is still present (s_articles_attributes) illegal attribute models are generated based on the column definition:
`attr17` date DEFAULT NULL

MariaDB is not officially supported but nevertheless "recommended" in the Shopware performance guide and also by some agencies.
The broken code is generated in /var/cache/production____REVISION___/doctrine/attributes/Article.php:
    /**
     * @var date $attr17
     *
     * @ORM\Column(name="attr17", type="date", nullable=true)
     */
    protected $attr17;

...

    public function __construct()
    {
        ...
        $this->attr15 = "NULL";
        $this->attr16 = "NULL";
        $this->attr17 = new \DateTime("NULL");
        $this->attr18 = "NULL";
        ...
    }

The following exception is thrown, for example when duplicating an article in the backend:
DateTime::__construct(): Failed to parse time string (NULL) at position 0 (N): The timezone could not be found in the database in var/cache/production____REVISION___/doctrine/attributes/Article.php on line 299
The attribute (s_articles_attributes.attr17) was normalized with Shopware 5.3, but this only applies for fresh installations not existing ones which are updated: https://github.com/shopware/shopware/blob/5325f6c831e4f6101ae1669494323475a38dce58/_sql/migrations/808-change-varchar-attributes.php#L32
And it's still possible to create problematic date attributes via the backend:

But the exception thrown for DATE columns is not the only problem, all attributes without a default value will be initialized with the string "NULL" (see attr15 above for example) with all MariaDB versions >= 10.2.7. The MariaDB handling of default values inINFORMATION_SCHEMA deviates from MySQL and the MariaDB devs decided that they do not want to change that anymore, but Doctrine is working on compatibility to both approaches. All of this discussion can be found here (including the linked issues): https://github.com/doctrine/doctrine2/issues/6565
2. What does this change do, exactly?
Ignore "NULL" (string) default values, which probably no shop owner needs/wants or would  intentionally set. This is a workaround but should be manageable.
Apparently doctrine/dbal 2.7.0 will resolve this as well, but even on the Shopware 5.4 branch version 2.5.4 is still shipped.
3. Describe each step to reproduce the issue or behaviour.
Use MariaDB => 10.2.7 (docker setup documented here) and run ant build-unit or generate the attribute models some other way. The will use "NULL" (string) as default value for all 20 article attributes that are shipped by default.
4. Please link to the relevant issues (if any).

https://issues.shopware.com/issues/SW-20274
https://issues.shopware.com/issues/SW-20343

5. Which documentation changes (if any) need to be made because of this PR?
MySQL 5.7 was released 2015-10-21 and made most of the reasons for recommending MariaDB (better performance and easier configuration/more sensible defaults) obsolete. So really MariaDB could be removed from the Shopware performance docs. It looks like in the future MariaDB will care even less about MySQL compatibility.
6. Checklist

[X] I have squashed any insignificant commits
[X] This change has comments for package types, values, functions, and non-obvious lines of code
[X] I have read the contribution requirements and fulfil them.
I've verified this with MariaDB 10.2.10 vs. MySQL 5.7.20 but I do not think there is a realistic way to write automated tests for this right now (either mocking a big part of dbal or adding additional travis builds for MariaDB, which is supported BTW)

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-20584
Please use this issue to track the state of your pull request.

authors: samuelvogel shopwareBot


text:
NEXT-31615 - fix language mapping in SalesChannelValidator

1. Why is this change necessary?
it solves the problem described in NEXT-19596 - #3363
2. What does this change do, exactly?
When a SalesChannel get updated, it remove inserts which are already set.
3. Describe each step to reproduce the issue or behaviour.
The issue is described in #3363
4. Please link to the relevant issues (if any).

#3363
NEXT-19596

5. Checklist

[x] I have rebased my changes to remove merge conflicts
[x] I have written tests and verified that they fail without my change
[x] I have created a changelog file with all necessary information about my changes
[x] I have written or adjusted the documentation according to my changes
[x] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

🤖 Generated by Copilot at 104a454
This pull request fixes a bug in the SalesChannelValidator that caused errors when updating sales channels with different languages. It also adds a changelog entry for the fix in changelog/_unreleased/2023-10-13-fix-sales_channel_language_validator.md.

🤖 Generated by Copilot at 104a454

Add a changelog entry for the fix (link)
Filter out inserts that do not match the language id of the record in SalesChannelValidator (link)

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

@celha what can I do that this PR gets merged?

@Scarbous there are two things open that I considered during testing.

Why did you delete the test case 'testInsertSalesChannelLanguageWhichAlreadyExist'?
How did you test your changes (excl. unit test)? I tried it and for me the changes didn't work. Maybe you can explain your steps so that I can validate that.

@celha sorry for the late reply.
I'll add the test again and correct it.
I'm testing the changes with an API driven import where we create/update multiple SlaesChannels.
While working on the import, I noticed the misbehavior.
While working on the bug, I booted up a development system with gitpod and ran it multiple times with the import on the development system.
We create a UUID for each SalesChannel, this allows us to address the SalesChannel at any time without knowing whether it exists in advance.
Below is an example of the creation of a SalesChannel:
$uuid = md5(SalesChannelDefinition::ENTITY_NAME . $mandant);
[
    'id' => $uuid,
    'typeId' => Defaults::SALES_CHANNEL_TYPE_STOREFRONT,
    'customerGroupId' => Defaults::FALLBACK_CUSTOMER_GROUP,
    'currencyId' => Defaults::CURRENCY,
    'language' => [
        "id" => $this->shopConfigService->getLanguageUuidByIso('de-DE'),
    ],
    "languages" => [
        [
            "id" => Defaults::LANGUAGE_SYSTEM,
        ],
        [
            "id" => $this->shopConfigService->getLanguageUuidByIso('de-DE'),
        ],
    ],
    "domains" => [
        [
            "id" => md5(SalesChannelDomainDefinition::ENTITY_NAME . $mandant),
            "url" => "http://localhost:8000/" . $mandant,
            "languageId" => Defaults::LANGUAGE_SYSTEM,
            "currencyId" => Defaults::CURRENCY,
            "snippetSetId" => '018a642ae64f7170816a1b96ec375dc4',
        ],
    ],
    "hreflangActive" => false,
    "homeEnabled" => true,
    'paymentMethodId' => $this->shopConfigService->getPaymentUuid(ShopwarePaymentMethod::DebitPayment),
    'paymentMethods' => [
        [
            "id" => $this->shopConfigService->getPaymentUuid(ShopwarePaymentMethod::DebitPayment),
        ],
        [
            "id" => $this->shopConfigService->getPaymentUuid(ShopwarePaymentMethod::CashPayment),
        ],
    ],
    'shippingMethodId' => $this->shopConfigService->getShippingMethodUuid(ShopwareShippingMethod::Standard),
    'shippingMethods' => [
        [
            "id" => $this->shopConfigService->getShippingMethodUuid(ShopwareShippingMethod::Standard),
        ],
        [
            "id" => $this->shopConfigService->getShippingMethodUuid(ShopwareShippingMethod::Express),
        ],
    ],
    'countryId' => $this->shopConfigService->getCountryUuid('DE'),
    'name' => $name,
];

The shopConfigService allows us to get UUIDs from the Shopware over the Admin-API.

Hello @Scarbous,
your changes were imported and the Pull Request was merged!
Thanks a lot for your contribution!

authors: CLAassistant MartinKrzykawski Scarbous celha


text:
Update logo.html.twig

Incorrect indentation

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

Hey @erguenyogurtcu  thanks for your contribution! Your PR just got merged 🎉 💙

authors: CLAassistant erguenyogurtcu ssltg


text:
Typo error_in_Update prompt_composition.mdx

Updated Spelling mistake @AashishSainiShorthillsAI

The Favourite 'F' should be in small case. Please fix this @AmitSinghShorthillsAI

Done @AashishSainiShorthillsAI

repo: shorthills-ai/langchain


text:
valkey passthrough test

Valkey is looking to be the dominant open source fork for Redis, makes sense to run this smoke against it. There hasn't been an official release so I've pinned to latest. This would work to our advantage because it will notify us immediately if anything changes that breaks Shotover compatibility.

I think we should give this some more time before we start supporting a redis fork.
Googling for valkey doesnt even bring up a repo.

Lets reconsider this once valkey is large enough and it has a version we can pin to.
Happy for this to remain up as a draft until then.

Can close now for https://github.com/shotover/shotover-proxy/pull/1809/files

authors: conorbros rukai

repo: shotover/shotover-proxy


text:
[blog-theme] Expose reading time in metadata

Problem
I want to show reading time in OG images but reading time is not accessible from metadata.
Sample OG image (hardcoded reading time)

Solution
If possible, expose reading time in metadata. Or in other way, that can easily be accessible from config.head

you can access it by
import { useBlogContext } from 'nextra-theme-blog'

const { opts } = useBlogContext()

const readingTime = opts.readingTime?.text

repo: shuding/nextra


text:
tooltip position bottom not working

In button, data-tooltip-position="bottom" means bottom position for tooltip is not working. however we can set top, left and right.
Step to reproduce
<button
type="button"
class="border-muted-200 hover:ring-muted-200 dark:hover:ring-muted-700 dark:border-muted-700 dark:bg-muted-800 dark:ring-offset-muted-900 flex h-9 w-9 items-center justify-center rounded-full border bg-white ring-1 ring-transparent transition-all duration-300 hover:ring-offset-4"
data-tooltip="Back to bots"
data-tooltip-position="bottom"
>
<Icon name="ph:arrow-bend-up-left" class="text-muted-400 h-5 w-5" />
</button>

Expected output,
when we set data-tooltip-position="bottom",
the tooltip need to show below the targeted element

Hello @sagardspeed2,
You can use data-tooltip-position="down" !

repo: shuriken-ui/nuxt


text:
Frontend: Receipt render bug

Describe the bug
In admin receipt viewing page, if there are multiple sessions with one or more sessions without any data, the page would not render other sessions that contains data.
To Reproduce
Steps to reproduce the behavior:

Create new sessions
Go to '/receipt'
Inspect element
See error

Expected behavior
Sessions with data should render without any error.
Additional context
Add any other context about the problem here.

Bug is patched by moving it to the homepage #29

repo: shuttleday/shuttleday

license: WTFPL


text:
[io] Fix path and target for serialization headers generation.

After some mentions of build problems I thought I would check the status of master.  I am unable to build with serialization turned on.  I also found that generating the serialization headers is broken.
This PR changes the doxygen dependency target which has changed I guess.  Probably kernel_xml4swig is not correct, maybe someone can suggest a better fix.
However, I think the patch to build_common.py is necessary.
Unfortunately even with this, I get errors in kernelPYTHON_wrap.cxx trying to build with serialization, but I cannot figure them out.
Many errors like this:
/usr/include/boost/serialization/access.hpp:116:11: error: ‘class LinearOSNS’ has no member named ‘serialize’; did you mean ‘serializable’?

which seems like the classes do not have serialization declared but it's weird because SiconosFullGenerated.hpp is definitely getting included, I used #error to check.  However I noticed that the generated headers have a lot of things re-ordered, which is supposed to be avoided by sorting them by priority and by name in io/tools/build_from_doxygen.py, but it seems to have decided to move a lot of things around so it's hard to tell if this is due to registration order.

I should mention that if I turn off serialization things seem to build fine.

Further info:
Without this patch, the errors are in new objects SiconosCone, SiconosCapsule, they do not have a "serialize" method.  I suppose they could be added manually, but the SerializersGeneration should work.
Then, with -DWITH_GENERATION=ON, then make SerializersGeneration leads to,
make[3]: *** No rule to make target '../io/doxygen', needed by '../io/src/SiconosFullGenerated.hpp'.  Stop.

So it is related to recent changes in docs.
With this patch now it depends on xml doxygen targets, and I have updated (rebase) the PR to look for xml in doxy2swig-xml/{components}/*.xml, and this works for generation, but I still get a lot of errors in kernelPYTHON_wrap.cxx when trying to compile.
So, currently, I don't know how to fully fix this.
But I think it's clear that there must be added a CI target that includes serialization and is preceded by SerializersGeneration, if this feature is to remain maintainable.

do we need python docstrings from swig to run serialization

Not sure, I don't think so.

(which is implied if kernel_xml4swig is a dep)

Yes, I just used it to ensure xml was generated.  Perhaps we need an xml4serialization target, but not sure why multiple "xml" targets are needed, there is just one set of xml outputs from doxygen, is this not the case?  It seems there are now multiple directories under Docs/build/xml, but I am not up to date on how you are rearranging things.

And I guess that we should be able to run serialization/generation even when PYTHON_WRAPPER is off ?

Agreed.  Recall that the doxygen xml was just used as a replacement for gccxml.  Previously I think I always used SerializersGeneration with WITH_DOCUMENTATION=1 so I am not 100% confident that the dependency chain is perfect.  It would be good for it to depend explicitly on doxygen xml generation.

do we need xml files for all components ? Only kernel ? Some others ? (I would say all ?)

It is used to generate SiconosFullKernelGenerated.hpp which should only depend on kernel, and SiconosFullGenerated.hpp which additionally includes classes from mechanics and control.
There is a dict called input_headers that maps these components to the headers they need the xml for in io/tools/builder_common.py.
The way it works is that classes_from_build_path in build_from_doxygen.py opens all xml files and scans for classes that support serialization.  These are compared against a list of classes to ignore.  The remaining classes are ordered according to priority and then alphabetically.  These are then written to the SiconosGenerated*.hpp headers along with all members and superclass dependencies.

Just a quick comment that I had the same thoughts as @fperignon about the dependency on kernel_xml4swig, and the fact that this would not work when the python bindings are not enabled.
I got lost in trying to come up with a good proposal.
@radarsat1 your work on that is very much appreciated.

ps

Another remark : if I understood properly the add_custom_command(s) in io/CMakelist.txt, headers (GENERATED_HEADER and GENERATED_HEADER_KERNEL) are generated into source dir rather than into binary dir. Isn't it a bit strange ?

Yes I always found that strange but never fixed it.  Principally I think it is because we do not include this as part of the build process, but rather we generate these files and then commit them as part of the source.  We could always change it to be a standard part of the build process, the only effect would be to require doxygen every time we build with serialization, but I do not think this is too bad a requirement.
By the way, with regards to CI, I found that turning on serialization increases requirements for building the python bindings so much that building kernel.i requires 8 GB of memory, which has the effect of crashing with out-of-memory errors in at least the debian salsa CI system, as well as my personal laptop.  (I have to close Firefox to compile Siconos successfully!)  So be careful.

The memory requirements are a big issues. It has to be documented: I had the same issue 4 years ago. The fix would be to split that gigantic file. However, IIRC, a hack is needed to have all the .o compiled into 1 python module (.so).

"Perhaps we need an xml4serialization target, but not sure why multiple "xml" targets are needed, there is just one set of xml outputs from doxygen, is this not the case? It seems there are now multiple directories under Docs/build/xml, but I am not up to date on how you are rearranging things."
yes there are multiple xml outputs and targets.
I tried to explain thing here : https://nonsmooth.gricad-pages.univ-grenoble-alpes.fr/siconos/devel_guide/write_and_build_doc.html https://nonsmooth.gricad-pages.univ-grenoble-alpes.fr/siconos/devel_guide/write_and_build_doc.html
and through comments in docs/gendoctools files.
To sum it up :
A first pipeline leads from headers/xml outputs to rst files for sphinx  to handle python/docstrings in swig

hpp —> xml —> .i (doxy2swig) —> rst .
More or less what is done in gendoctools/python2rst.py

Another pipeline leads from xml outputs  to rst files for sphinx to handle c++ comments :

hpp —> xml —> rst (breathe complient)
More or less what is done in gendoctools/cpp2rst.py

And to make things easy the doxygen config to generate xml files can not be the same for swig and breathe.
So we need different targets :
_xml4swig  (For python API in autodoc)  —> files in <cmake_binary_dir>/docs/build/doxygen/doxy2swig-xml/
-doxy2xml (For C++ API in autodoc)   —> files in <cmake_binary_dir>/docs/build/doxygen/xml4rst/
So, taking that into account and regarding Steve comments, I think a dependency to kernel_doxy2xml will be enough
I have commit something to fix this, or at least try to …

Le 5 avr. 2019 à 15:24, Stephen Sinclair notifications@github.com a écrit :
ps
Another remark : if I understood properly the add_custom_command(s) in io/CMakelist.txt, headers (GENERATED_HEADER and GENERATED_HEADER_KERNEL) are generated into source dir rather than into binary dir. Isn't it a bit strange ?
Yes I always found that strange but never fixed it. Principally I think it is because we do not include this as part of the build process, but rather we generate these files and then commit them as part of the source. We could always change it to be a standard part of the build process, the only effect would be to require doxygen every time we build with serialization, but I do not think this is too bad a requirement.

By the way, with regards to CI, I found that turning on serialization increases requirements for building the python bindings so much that building kernel.i requires 8 GB of memory, which has the effect of crashing with out-of-memory errors in at least the debian salsa CI system, as well as my personal laptop. (I have to close Firefox to compile Siconos successfully!) So be careful.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/siconos/siconos/pull/288#issuecomment-480273873, or mute the thread https://github.com/notifications/unsubscribe-auth/AHbsU7YG1XkVeKmgoT8Lz5ugKa-wIyktks5vd06EgaJpZM4cOTxe.

authors: fperignon radarsat1 xhub

repo: siconos/siconos


text:
如何用go-mysql搭建半同步binlog  server?

Hi,您好：
如何用go-mysql搭建半同步binlog  server?

enable https://github.com/siddontang/go-mysql/blob/master/replication/binlogsyncer.go#L48

谢谢。

authors: siddontang weiboyiyou

repo: siddontang/go-mysql


text:
useAuth() not defined when using custom login page

Environment

Reproduction

Describe the bug
mentioned code shows useAuth() not defined. there is no way mentioned in docs which guides regarding its import.
Additional context
No response
Logs
No response

Which version of nuxt-auth are you using?
If it's prior to v0.5, you should call useSession() instead of ustAuth()
https://sidebase.io/nuxt-auth/application-side#overview

Which version of nuxt-auth are you using? If it's prior to v0.5, you should call useSession() instead of useAuth()

...as he's using useAuth(), isn't that just an indication that he's using 0.5?

As there has not been a response by @baseerx for a while, I will close this issue. @baseerx if the issue still presists, please provide a reproduction to the issue on StackBlitz, so I can investigate futhur!

authors: andreasvirkus baseerx quroom zoey-kaiser

repo: sidebase/nuxt-auth


text:
Feature Request: Recursive update-dep

Say I have a workspace with a root package foo which has dependency chain:
foo->bar->baz->zomg
I make a change in zomg. I attempt to test it at foo level ("why" is another issue, but simply say I cannot test zomg changes when zomg is instantiated as part of a wit workspace directly).
It seems I can not currently, in my workspace, say wit update-dep zomg because foo does not itself have a top-level dependency on zomg?
So it seems I can't actually use update-dep until i get to wit update-dep bar and until then I have to manually copy-paste commit hashes into wit-manifest.json?
Couldn't a wit update-dep -r do this manual work for me?

I think help making recursive/cascading updates makes sense in some contexts, but I want to warn that it has some risks. If any of the commit hashes along the way gets squashed and merged or otherwise not accepted upstream, you can create broken links.
I think the answer to this issue is more of a workflow solution than recursive update-dep. It is perfectly reasonable to test zomg against foo. In fact, Wit explicitly supports this notion. The standard way of doing it would be to just add zomg at the commit in question to the workspace. Ie. just run wit add-pkg zomg && wit update.
Alternatively, if you have made a change to foo that requires an updated zomg, but bar and baz are unaffected, then you should simply add zomg as a dependency to foo: wit add-dep zomg. As with update-dep you'll need to commit the change to foo's wit-manifest.json before running wit update. The whole idea of Wit is to enable such dependencies, eg.
"foo depends on bar at commit X which transitively depends on zomg, but foo needs a newer version of zomg than it would get via its dependency on bar"
I really need to figure out how to better express these workflows in documentation, maybe a cookbook is the right style of documentation for this.

I think I was also not aware that I could use wit -C baz update-dep zomg which helped a lot of the pain.
The part I haven't wrapped my head around is what in a sentence does the above do. Is it correc that the command above means "Look in the current wit workspace, wherever you happen to be in the hierarchy at the moment, do the equivalent of cd zomg; git show), then modify baz/wit-manifest.json of baz to  match the commit hash result of the git show?

That's functionally correct. A little more formally I'd phrase it as something like:
Wit update-dep updates the version of a dependency package in a dependee package. The dependee is set by the current working directory. The dependency is set using the standard <package>[::<version>] argument pattern where the package must exist in the workspace and the version can be provided manually or will be inferred to be the currently checked out commit of the package.

repo: sifive/wit


text:
Feature/add kosovo

Added Kosovo
Right now Kosovo doesn't have its own official iso3166 standart code, but it is assigned temp xk/xkx codes.
Codes are temporary till Kosovo is assigned its own country codes.
Also there is no numeric country code (even temp/unofficial), so used phone code just to make code compiled.
Code was formatted with cargo fmt, to get verbose representation of relevant changes see this commit
Used data from:

https://en.wikipedia.org/wiki/Kosovo
https://en.wikipedia.org/wiki/.xk

Codes:

alpha2 - XK
alpha3 - XKX
numeric - 383 (### Used phone code because numeric id was not assigned, so the country doesn't still has it, added just to make code compile ###)
full name - Kosovo

any updates on this?

authors: globaltrouble paramako

repo: sifton/isocountry-rs


text:
Prevent TypeError

Maybe caused by asynchronous ajax requests?

@J2TeaM That's a weird one. Your change looks good, thanks for adding the fallback!

repo: siggy/beatboxer


text:
Support downloading dependencies via HTTPS like legacy

Furyctl Next does not currently support downloading dependencies via HTTPS instead of SSH, this is a feature that was added to Furyctl legacy because SSH traffic is blocked in some environments, breaking the modules' download.
Add the capability to download using HTTPS and a flag to enable it to furyctl next.

Closing as completed

authors: nutellinoit ralgozino

repo: sighupio/furyctl


text:
Upgrade go.opentelemetry.io

Release v1.15.1/v0.38.1
Contrib release v1.16.1/v0.41.1/v0.10.1

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.06 :tada:

Comparison is base (dc84f48) 81.95% compared to head (627d18c) 82.02%.

:exclamation: Current head 627d18c differs from pull request most recent head 908082a. Consider uploading reports for the commit 908082a to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2095      +/-   ##
==========================================
+ Coverage   81.95%   82.02%   +0.06%
==========================================
  Files          75       75
  Lines        2871     2882      +11
==========================================
+ Hits         2353     2364      +11
  Misses        452      452
  Partials       66       66

Flag
Coverage Δ

Linux
81.67% <100.00%> (+0.07%)
:arrow_up:

Windows
78.55% <100.00%> (+0.09%)
:arrow_up:

macOS
78.07% <100.00%> (+0.08%)
:arrow_up:

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

Impacted Files
Coverage Δ

instrumentation/database/sql/splunksql/metrics.go
78.18% <100.00%> (+5.45%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

authors: MrAlias codecov-commenter

repo: signalfx/splunk-otel-go


text:
Bump opentelemetry-collector-contrib to 0.80.0

Tested locally (data was send to Splunk o11y Cloud)

Codecov Report
Patch coverage has no change and project coverage change: -0.35 :warning:

Comparison is base (6dc3d60) 82.43% compared to head (725869b) 82.08%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2312      +/-   ##
==========================================
- Coverage   82.43%   82.08%   -0.35%
==========================================
  Files          76       76
  Lines        2886     2886
==========================================
- Hits         2379     2369      -10
- Misses        444      452       +8
- Partials       63       65       +2

Flag
Coverage Δ

Linux
82.08% <ø> (ø)

Windows
?

macOS
?

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

authors: codecov-commenter pellared


text:
Make application truly async

Right now we are running the main methods synchronously...

We also need to fix the wait graphic ... currently, it is just run synchronously...

repo: signebedi/gptty


text:
Using Asteroid STFTB instead of torch.STFT

What does this implement/fix? Explain your changes.
Using Asteroid STFTB instead of torch.STFT
Any other comments?

@Baldwin-disso please add asteroid to the environement files

@Baldwin-disso any news on this?

depency can be replaced with https://github.com/asteroid-team/asteroid-filterbanks

@Baldwin-disso I think you can remove cython and only add the new asteroid-filterbanks dependency

@Baldwin-disso sorry for the confusion. The use of torch windows is now supported...
https://github.com/asteroid-team/asteroid-filterbanks/pull/4

finally, it works

finally, it works

I benchmarked torch_stft_fb against torch.stft, and its about 15% slower on cpu. Should we make this optional then?
proposal --filterbank torchstft vs --filterbank asteroidstft ?

I benchmarked torch_stft_fb against torch.stft, and its about 15% slower on cpu. Should we make this optional then?
proposal --filterbank torchstft vs --filterbank asteroidstft ?

@Baldwin-disso @aliutkus

@Baldwin-disso @aliutkus

@Baldwin-disso @aliutkus ONNX support isn't doable with asteroid filterbanks at the moment

@Baldwin-disso @aliutkus ONNX support isn't doable with asteroid filterbanks at the moment

authors: Baldwin-disso faroit

repo: sigsep/open-unmix-pytorch


text:
Update README.md

added details for crane installation for those unfamiliar
Signed-off-by: Dan POP dan.papandrea@sysdig.com

cc @jonjohnsonjr @ImJasonH  we have a crane user over here!

closing PR to redo DCO....  the bane of my existence...

authors: danpopSD dlorenc

repo: sigstore/cosign


text:
Add unit tests for oauthflow.IssuerFromIDToken

Description
Similar to https://github.com/sigstore/fulcio/issues/405, oauthflow.IssuerFromIDToken is untested and could use some unit tests

And I am working on it :)

cc: @nsmith5

repo: sigstore/fulcio


text:
sign using keyless for ci containers

Summary

update key flag and add workflow-dispacth to trigger the build manually
sing using keyless approach

Ticket Link
n/a
Release Note

update key flag and add workflow-dispacth to trigger the build manually

Is this one all set or do we need to delete the lines?

@lukehinds @dlorenc made some changes, let me know your thoughts

will watch the job

green now https://github.com/sigstore/rekor/runs/4549547999?check_suite_focus=true

authors: cpanato dlorenc

repo: sigstore/rekor


text:
Format Tcl code using tclfmt

Announcing that tclint finally includes a formatter (tclfmt) by way of this PR!
The first commit is the result of running the formatter on the code (tcfmt --in-place .). It mostly follows tclint's style guide, and uses the same configuration. There are two categories of changes that are responsible for most of the diff that perhaps might be controversial:

tclfmt pushes the braces for multi-line expressions onto different lines. There were a couple reasons for doing this, the main one is that it consistently looks good across differently configured indent levels (the style currently used by this repo can sometimes have a case where the expression of an if statement is indented the same as the body, for example).
It breaks set alignment. This was just a case of being more opinionated for simplicity, although I'm open to making tclfmt respect the allow-aligned-sets config if you feel strongly about preserving these.

I wrote up some docs describing usage of the utility plus a description of the style guide with some rationale, so that might be interesting to look through: https://github.com/nmoroze/tclint/blob/v0.4.0/docs/tclfmt.md.
The second commit adds the formatter to CI. Turns out it's much easier to comprehensively enforce a consistent style by diffing against formatted code rather than checking for different style violations piecemeal. Because of this, tclfmt is gonna be the canonical way of enforcing style going forward, and I'll eventually remove style checks from tclint itself. To preview this future behavior and reduce output noise, tclint got a new flag --no-check-style. tclfmt also doesn't automatically reducing line length yet, so --no-check-style still reports line length violations.
As always, I'm open to any feedback!

Thanks for merging, sorry I dropped the thread re: expression indentation style.
I can look into alternatives if you'd still prefer a different style.

@nmoroze no worries, I don't feel so strongly about the indents, so I think it's fine. I'll work my way through the repos that would benefit from this and let you know if anything changes or if any issues arise.

authors: gadfort nmoroze

repo: siliconcompiler/siliconcompiler


text:
No folder specified. Please pass a folder as an argument or set SB_FOLDER environment variable. ERORS

No folder specified. Please pass a folder as an argument or set SB_FOLDER environment variable.

CLOSED

How are you running this? You need to run it from the command prompt, and pass in a folder pointing to a folder where your markdown files will be kept as an argument. Simply double clicking the .exe won't work (which you may be doing).

I don't understand. Can you help me?

How are you running this? You need to run it from the command prompt, and pass in a folder pointing to a folder w

https://github.com/silverbulletmd/silverbullet/issues/810#issuecomment-2002016426

Ok this is not helpful, looking at your GitHub profile I think you're looking at the wrong repository. Let me know if I'm wrong. Closing this.

repo: silverbulletmd/silverbullet


text:
Can not enforce our three-space indentation with var declarations aligned

We have long used three-space (never a tab) indentation because in standard CLI (and other editor/viewer) configs where a tab is five or eight spaces, the three space indentation easily stands out from tabs. The standard eslint indent rule will handle the three-space indentation. However, we also want var declarations to line up evenly so that variable names all start in the same column. This is not handled by the standard rule as it wants the var declarations to be a multiple of your standard indentation, but ours is not (it would be four spaces more than the previous line).
For example, we want our code to look like this:
var foo = 'bar',
    baz = 'abc';

function foo(xyz) {
   var def = 123,
       fed = 321;

   doSomething(xyz, def, fed);
}

This is the configuration that won't work:
'indent': [
   'error',
   3,
   { "VariableDeclarator": "IsBroken-RequiresMultipleNumber" }
],

We should be able to close this since #5

repo: silvermine/eslint-config-silvermine


text:
Paying of manual payment via OrderActionsForm only works with offsite gateways

When a user completes an order and uses a Manual payment (eg. Invoice), the Order has an outstanding balance.
With OrderActionsForm, the user can pay the outstanding amount, but this only seems to work with offsite gateways. When selecting an onsite gateway, there will just be a failure notice…

Closed via #511

repo: silvershop/silvershop-core


text:
BUG CSV Export functionality doesn't respect UTF8 extended characters

From https://github.com/silverstripe/silverstripe-userforms/issues/402
The CSV export in this add-on is not working for languages like Persian & Arabic and i can't set char-set to UTF-8

Also see https://github.com/silverstripe/silverstripe-cms/issues/1335

https://github.com/silverstripe/silverstripe-framework/pull/7833 may have resolved this

This definitely won't be a problem in 4.x (he says)
I'm not sure 3.x will see any more love around this; I did quite a bit of investigation in the BOM stuff and it was a pain. I'm loathed to handroll code to fix this. There's no realistic league/csv version to use that matches our PHP version requirements so I think think this is going to remain an issue except when someone decides to create their own UTF-8 exporter.

#wontfix for SS3 IMO.

authors: dhensby kinglozzer sminnee tractorcow

repo: silverstripe/silverstripe-framework


text:
NEW: Add login profile SVG for ssorg

Add two SVG files to replace images icons and add styles. Add Iframe ID's as classes to global toolbar body to target mobile/desktop versions separately.
Depends on https://github.com/silverstripeltd/ssorg/pull/93/files

Looks good otherwise =)

Cheers! 😄

authors: robbieaverill sachajudd

repo: silverstripe/silverstripe-globaltoolbar


text:
[Request] Detect user geolocation for country specific google exclusions

Leaving an open issue here so I can track this.
Description:
Would be cool if we could detect the users geolocation before CSP is engaged, that way we could reduce header size and apply a single rule for all of google https://www.google.* and the TLD/Country suffix would be applied.
Cause:
Seeing a lot of https://adservice.google.com|co.nz|.ae domains being reported. Only option at the moment is to list them all las new rules in the fragments.
NB: Could be some privacy/security concerns for this?

https://content-security-policy.com/strict-dynamic/

repo: silverstripeltd/silverstripe-csp


text:
What about using python typing

So now we are in Python 3.
What about changing the documentation guideline to use typing?
It's not about reworking everything, nor enforcing typing. It's just about changing the way we write documentation for the new code we design.
In Flint i mostly type everything. That's really useful. And as we already do it in the docstring, what about doing it with the python parser.
So this code:
    def getAlternativeImageData(self, copy=True):
        """Get the optional RGBA image that is displayed instead of the data

        :param bool copy: True (Default) to get a copy,
            False to use internal representation (do not modify!)
        :rtype: Union[None,numpy.ndarray]
        """
        pass

Can be written:
    def getAlternativeImageData(self, copy:bool=True) -> typing.Optional[numpy.ndarray]:
        """Get the optional RGBA image that is displayed instead of the data

        :param copy: True (Default) to get a copy,
            False to use internal representation (do not modify!)
        """
        pass

I dont have yet tried the result in sphinx.
What do you think?

Why am I the always finding older code more readable?
I must be old :(

Would be nice to see how sphinx process it.
@vasole, I also find the function signature less readable (In the end, I guess you get used to it), but I also find the current docstring syntax to describe the type not really nicer either..

Especially when you have to write :class:~bla.bla.bla.
To be honest, i use black, so i don't care much about the page layout.
But i always use the autocompletion of the IDE, and here is a big difference, cause the python code is parsed and validated (flake, pylint), not the docstrings.

So i did some tests. I checked 2 extensions (sphinx.ext.napoleon, sphinx-autodoc-typehints), and none provides links from arguments or returned type to the type description. So it's mostly useless for the HTML documentation.

Looking back into this, the sphinx-autodoc-typehints extension is doing it and it looks to work.
It is even packaged in ubuntu, conda-forge and debian but only from debian11 onward...

authors: t20100 vallsv vasole

repo: silx-kit/silx


text:
Error when using boolean columns in postgresql when used as a boolean expression

Describe the bug
Having a table such as
class MyTables extends Table {
  BoolColumn get myBool => boolean()();
}

and connecting to a postgresql database, running the following query fails:
(select(myTables)..where((_) => myTables.myBool)).get();

Message: [PostgreSQLSeverity.error 42804: argument of WHERE must be type boolean, not type integer ]. Stacktrace:
#0      _PostgreSQLExecutionContextMixin._query
package:postgres/src/connection.dart:513
#1      _PostgreSQLExecutionContextMixin.query
package:postgres/src/connection.dart:474
#2      _PgDelegate.runSelect
package:drift_postgres/src/pg_database.dart:108
#3      _BaseExecutor.runSelect
package:drift/…/helpers/engines.dart:68
#4      main

This is due to postgres, unlike sqlite, not implicitly converting integers to booleans. Instead, it has a dedicated boolean type. So, we could simply change the sqlTypeName of DriftSqlType.bool to be boolean instead of integer in postgres, but then we run into issues with the default constraints on bool columns (CHECK ("col_name" IN (0, 1))) producing errors since postgres can't compare a boolean to 0 or 1.
I'd be willing to write up a PR for this if you could give me some guidance on how to approach the issue - being a new user of drift, I'm not aware of how this stuff is usually handled.

Thanks for the report and for the suggestion on how to fix this.
I agree that using a native boolean datatype for postgres would be good. At the moment, the CHECK ("col_name" IN (0, 1)) constraint is unconditionally added by drift_dev. To fix this, we'd need to restructure that code to be aware of the current dialect at runtime.
We could change GeneratedColumn.defaultConstraints to be a void Function(GenerationContext) that writes the constraint into a context instead of just being a static string. Then, the generator could emit code aware of the current dialect (via GenerationContext.dialect) and only include the check for sqlite.
A PR for this would be most welcome! This could be approached with the callback taking a GenerationContext, but I'm also happy with alternative approaches if you come up with something simpler. If you have any questions on this, please let me know :)


text:
How to do relation classes probably structured

I am doing a new project, and still in the structure phase doing models ...etc, I am using Drift as the main database ( Local First )
Now I created tables like this
// @UseRowClass(SettingsRaw)
class SettingsTable extends Table {
  IntColumn get id => integer().nullable().autoIncrement()();
  IntColumn get account => integer().nullable()();
  // nullable fields
  TextColumn get token => text().nullable()();
  TextColumn get pincode => text().nullable()();
  DateTimeColumn get createdAt =>
      dateTime().nullable().withDefault(Constant(DateTime.now()))();

  // required fields
  IntColumn get theme =>
      intEnum<ThemeMode>().withDefault(Constant(ThemeMode.light.index))();
  BoolColumn get isAuthenticated =>
      boolean().withDefault(const Constant(false))();
  BoolColumn get isBiometricsEnabled =>
      boolean().withDefault(const Constant(false))();
  BoolColumn get isPincodeEnabled =>
      boolean().withDefault(const Constant(false))();
}

My problem is with joins and relationships, now to join the account which is from another table called accounts I have to create another class that is called SettingsWithAccount in order to map this, as there will be more relations not just account
Is there a way to just add the account class prop in the SettingsTables, so be settings.account
I searched and found UseRowClass but didn't quite understand how to use it to do that

No, drift has no support for things which would automatically add joins when selecting from a single table.

as there will be more relations not just account

For SettingsTable as well? If there is just a 1:1 relation between settings and accounts, it might be easier to just "inline" the fields from settings into the accounts table. If you want to avoid writing joins, you could store settings in a JSON structure with is stored on the accounts table in a text column with a type converter instead.


text:
Cant delete multiple selected instances

Cannot select 2 instances and delete both of them. Doing so just deletes one of them and leaves the other.

Intended, instances are currently very hard to maintain as they contain multiple units.

authors: Luffyyy RexTheWho


text:
Windows + non ascii symbols in filename path: not possible to open fits file

Fits error: FitsError { status: 104, message: "could not open the named file" }
Any ideas how to fix it?

Hi, thanks for the report. Do you have an example filename?
Because we use a c library to do the heavy lifting, we may struggle. I'm not at my computer right now but I'll check when I can.
There might be a way to open a file from a.file descriptor that we can use.

I thought the issue might've been because the specified filename isn't valid UTF-8, and (to my knowledge) the only way to convert an AsRef<Path> to a CString is with a valid UTF-8 filename.
https://github.com/simonrw/rust-fitsio/blob/1a2c4589ac223581a89663bfb0c15ba99b56cd77/fitsio/src/fitsfile.rs#L52-L59
... but that would've triggered the except in that code block.  @art-den can you give the file name you tried to use?  These error messages are pretty poor but the issue may be as simple as what's said on the tin.

Examples:
D:\test\Путь\HA_dzX2_KS_D_eco.fits
D:\test\Path\Имя_файла.fits

So it's not the non-ascii characters in the filenames - I've tested with Имя_файла.fits and it opens just fine in cfitsio and this crate. I suspect it's something to do with windows absolute paths, but I have recently deleted my windows VM so it might be a while before I can test.
@art-den can you try using relative paths (e.g. ..\Имя_файла.fits) to see if that still works?

Right: my windows VM still exists, and it cfitsio (the c library) struggles to open Имя_файла.fits after all. cfitsio expects the filename to be a char* (equivalent to rust u8) and does not seem to be doing any sort of unicode handling. In fact there's no mention of the word "unicode" in the cfitsio source code at all.
Windows unicode is utf-16 which is not what Rust or C expect at all. I will bow to someone with more windows experience than me but a workaround is to save files with ascii filenames for now.

Astrophotography software saves fits files into user's folders like C:\Users\<user name>\Desktop\SharpCap Captures (or something else) by default. And  can contains not only ascii characters ((
I will try to set default folder to C:\Users\<user name>\Desktop\SharpCap Captures and to open fits file by short file name (without full path). It will not solve problem with non-ascii in short file name but I think this is very rare situation.

I can't immediately think of a way around this. I recognise that the software has no control over the username of the user, but I think there would be a problem with cfitsio as well so we're not adding any problems here.
It's not ideal, but I don't understand enough about windows paths and non-ascii characters. Is this something that you can live with? Also: could you map the  path to e.g. Z:\ so it does not contain the username?

My heavyweight solution to the problem ))
#[cfg(target_os = "windows")]
mod win_fits_open {
    use std::{sync::Mutex, path::PathBuf};

    pub static OPEN_FITS_MUTEX: Mutex<()> = Mutex::new(());

    pub struct CurDirRestore {
        cur_dir: PathBuf,
    }

    impl CurDirRestore {
        pub fn new() -> anyhow::Result<Self> {
            Ok(Self {
                cur_dir: std::env::current_dir()?,
            })
        }
    }

    impl Drop for CurDirRestore {
        fn drop(&mut self) {
            _ = std::env::set_current_dir(&self.cur_dir);
        }
    }
}

#[cfg(target_os = "windows")]
fn fits_file_open_helper<F>(file_name: &Path, fun: F) -> anyhow::Result<FitsFile>
where F: Fn(&Path) -> anyhow::Result<FitsFile> {
    use win_fits_open::*;
    let result = match (file_name.parent(), file_name.file_name()) {
        (Some(file_dir), Some(rel_name)) => {
            let change_dir_locker = OPEN_FITS_MUTEX.lock();
            let cur_dir_restore = CurDirRestore::new()?;
            std::env::set_current_dir(file_dir)?;
            let rel_name = rel_name.to_str().unwrap_or("");
            let fits = fun(&Path::new(rel_name))?;
            drop(cur_dir_restore);
            drop(change_dir_locker);
            fits
        },
        _ =>
            fun(file_name)?,
    };
    Ok(result)
}

#[cfg(not(target_os = "windows"))]
fn fits_file_open_helper<F>(file_name: &Path, fun: F) -> anyhow::Result<FitsFile>
where F: Fn(&Path) -> anyhow::Result<FitsFile> {
    fun(file_name)
}

//////////////////////////////////////////////////

// using this

let mut fptr = fits_file_open_helper(
    file_name,
    |file_name| Ok(FitsFile::open(file_name)?)
)?;

// or

let mut fptr = fits_file_open_helper(
    file_name,
    |file_name| {
        Ok(FitsFile::create(file_name)
            .with_custom_primary(&image_description)
            .open()?)
    }
)?;

I'm sorry you have to work around this in this way, but that seems reasonable since cfitsio doesn't support non-ascii filenames :facepalm:
In general - are you happy that this fixes your problem going forwards? If so I will close this issue (and it will remain forever as documentation for others), otherwise I'll leave this open and we can consider a "proper" fix

It looks like this is issue of msys2 (and libfitsio also). My workaround fixes most common problem. So i'm happy ))

authors: art-den cjordan simonrw

repo: simonrw/rust-fitsio


text:
openfoam version222?

not sure if openfoam version222 is required for some reason but it leads to install fail
Makefile line 104
apt-get install -y --force-yes openfoam222
can be replaced by
apt-get install -y --force-yes openfoam30
for instance -  I don't know if it will cause problems later but at least install works.

The makefiles are for ubuntu 12.04. The latest version supported for that version of ubuntu is openfoam230

Please also note that the version of openfoam to be used is dictated by the openfoam simphony wrapper

Confirm 222 did not work. I am currently installing 230 on Precise, and 231 on Trusty (Because 230 is not working on Trusty, and 231 is not available on Precise)

authors: itziakos jeremy-rutman stefanoborini

repo: simphony/simphony-framework


text:
Explicitly lookup session and store services

closes #2302

remove setup-session-service initializer
add computed property to internal session to lazily lookup session-store.
add computed property to session service to lazily lookup session:main.
modifies tests to inject owner to entities that need to lookup dependencies.

closed in favor of #2315

repo: simplabs/ember-simple-auth


text:
Can't Get The Sign In To Work

Hello.
I've forked the repo and done the DB setup and I'm completely unable to properly sign in using either the in built admin account or a newly registered account.
This returns success in AccountController.Login post method
var result = await _signInManager.PasswordSignInAsync(model.Email, model.Password, model.RememberMe, lockoutOnFailure: false);
This returns false immediately on next render of the Login Partial
@if (SignInManager.IsSignedIn(User))
Any help in what I'm missing? I'm sure it'll be something obvious I'll kick myself about once it's pointed out.
Thanks

We haven't encountered this error.  If the sign in return success but out in the next request. I think there may be something wrong in the authentication cookie

Nevermind! Turns out clearing my cookies was all that was required. Schoolboy! Thanks for the pointer.
Thanks so much for your quick reply and help!

authors: HarleyBartles thiennn

repo: simplcommerce/SimplCommerce


text:
simple2d on Mac

Apologies for noise - I much prefer to develop in XCode than with editor-and-make
I have used homebrew to install the software on my Catalina-d Mac. The hello world triangle builds and runs - if I copy the source and create a new file and use simple2d build. Neat!
But it is obscure to me

where all this is installed - eg, where's the /test folder?
how to construct MacOS xcode projects within which to work.

Help much appreciated. I'd be happy to contribute 'pretty' versions of any such advice, together with eventual further example code (in xcode project form)

The same question from me.

Hi @petewilson-bsc. Homebrew will install frameworks to /usr/local/Cellar/simple2d/1.2.0/Frameworks. I've also added some example Xcode projects here. Hope that helps!

@blacktm I would like to include it via CMakeList. Are there any examples of how I can do that correctly?

Thanks!

On Oct 15, 2020, at 3:36 AM, Tom Black notifications@github.com wrote:
Hi @petewilson-bsc https://github.com/petewilson-bsc. Homebrew will install frameworks to /usr/local/Cellar/simple2d/1.2.0/Frameworks. I've also added some example Xcode projects here https://github.com/simple2d/deps/tree/master/xcode. Hope that helps!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/simple2d/simple2d/issues/121#issuecomment-708839725, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQQZ64OONJMVJECPY2QUMIDSKZG3RANCNFSM4Q25ZUDQ.

WARNING / LEGAL TEXT: This message is intended only for the use of the individual or entity to which it is addressed and may contain information which is privileged, confidential, proprietary, or exempt from disclosure under applicable law. If you are not the intended recipient or the person responsible for delivering the message to the intended recipient, you are strictly prohibited from disclosing, distributing, copying, or in any way using this message. If you have received this communication in error, please notify the sender and destroy and delete any copies you may have received.
http://www.bsc.es/disclaimer
http://bsc.es/disclaimer

@IevgeniiChuprun I don't have much experience with CMake, but if you figure it out, let me know and we'll add some guidance / code to the project.

@blacktm I would like to include it via CMakeList. Are there any examples of how I can do that correctly?

https://github.com/simple2d/simple2d/issues/130#issuecomment-890553494

authors: IevgeniiChuprun blacktm petewilson-bsc snandasena


text:
added the other noOp obj

Ticket: n/a
Context

adding more noOp items in keg-core for the Tap to use

Goal

adding more noOp items in keg-core for the Tap to use so I can use the noPropArr in the tap

Updates

repos/keg-core/core/base/utils/helpers/method/noop.js

added noOpObj, noPropObj, noPropArr

Testing

n/a

michael is doing it

repo: simpleviewinc/keg-hub


text:
Add softwareontheroad.com

By submitting this pull request, I promise I have read the contribution guidelines twice and ensured my submission follows it. I realize not doing so wastes the maintainers' time that they could have spent making the world better. 🖖
⬆⬆⬆⬆⬆⬆⬆⬆⬆⬆

Make an individual pull request for each suggestion. - https://github.com/sindresorhus/awesome-nodejs/blob/master/contributing.md

authors: santiq sindresorhus

repo: sindresorhus/awesome-nodejs


text:
Add Neuromorphic

Neuromorphic or spiking networks - brain-inspired, third generation neural networks, where neurons communicating via spikes.

https://github.com/artiomn/awesome-neuromorphic#readme
Spiking or neuromorphic neural networks are artificial neural networks that more closely mimic natural neural networks. In addition to neuronal and synaptic state, SNNs incorporate the concept of time into their operating model. The idea is that neurons in the SNN do not transmit information at each propagation cycle, but rather transmit information only when a membrane potential reaches a specific value, called the threshold.
This is the important and fast-grown area, that must be added to the Machine Learning page.
By submitting this pull request I confirm I've read and complied with the below requirements 🖖
Please read it multiple times. I spent a lot of time on these guidelines and most people miss a lot.
Requirements for your pull request

[x] Don't open a Draft / WIP pull request while you work on the guidelines. A pull request should be 100% ready and should adhere to all the guidelines when you open it. Instead use #2242 for incubation visibility.
[x] Don't waste my time. Do a good job, adhere to all the guidelines, and be responsive.
[x] You have to review at least 2 other open pull requests.
Try to prioritize unreviewed PRs, but you can also add more comments to reviewed PRs. Go through the below list when reviewing. This requirement is meant to help make the Awesome project self-sustaining. Comment here which PRs you reviewed. You're expected to put a good effort into this and to be thorough. Look at previous PR reviews for inspiration. Just commenting “looks good” or simply marking the pull request as approved does not count! You have to actually point out mistakes or improvement suggestions. Comments pointing out lint violation are allowed, but does not count as a review.

Other PR's Reviewed prior to this PR
https://github.com/sindresorhus/awesome/pull/2692
https://github.com/sindresorhus/awesome/pull/2690
https://github.com/sindresorhus/awesome/pull/2647
https://github.com/sindresorhus/awesome/pull/2622

[x] You have read and understood the instructions for creating a list.
[x] This pull request has a title in the format Add Name of List. It should not contain the word Awesome.

✅ Add Swift
✅ Add Software Architecture
❌ Update readme.md
❌ Add Awesome Swift
❌ Add swift
❌ add Swift
❌ Adding Swift
❌ Added Swift

[x] Your entry here should include a short description about the project/theme of the list. It should not describe the list itself. The first character should be uppercase and the description should end in a dot. It should be an objective description and not a tagline or marketing blurb.

✅ - [iOS](…) - Mobile operating system for Apple phones and tablets.
✅ - [Framer](…) - Prototyping interactive UI designs.
❌ - [iOS](…) - Resources and tools for iOS development.
❌ - [Framer](…)
❌ - [Framer](…) - prototyping interactive UI designs

[x] Your entry should be added at the bottom of the appropriate category.
[x] The title of your entry should be title-cased and the URL to your list should end in #readme.

Example: - [Software Architecture](https://github.com/simskij/awesome-software-architecture#readme) - The discipline of designing and building software.

[x] No blockchain-related lists.
[x] The suggested Awesome list complies with the below requirements.

Requirements for your Awesome list

[x] Has been around for at least 30 days.That means 30 days from either the first real commit or when it was open-sourced. Whatever is most recent.
[x] Run awesome-lint on your list and fix the reported issues. If there are false-positives or things that cannot/shouldn't be fixed, please report it.
[x] The default branch should be named main, not master.
[x] Includes a succinct description of the project/theme at the top of the readme. (Example)

✅ Mobile operating system for Apple phones and tablets.
✅ Prototyping interactive UI designs.
❌ Resources and tools for iOS development.
❌ Awesome Framer packages and tools.

[x] It's the result of hard work and the best I could possibly produce.
If you have not put in considerable effort into your list, your pull request will be immediately closed.
[x] The repo name of your list should be in lowercase slug format: awesome-name-of-list.

✅ awesome-swift
✅ awesome-web-typography
❌ awesome-Swift
❌ AwesomeWebTypography

[x] The heading title of your list should be in title case format: # Awesome Name of List.

✅ # Awesome Swift
✅ # Awesome Web Typography
❌ # awesome-swift
❌ # AwesomeSwift

[x] Non-generated Markdown file in a GitHub repo.
[x] The repo should have awesome-list & awesome as GitHub topics. I encourage you to add more relevant topics.
[x] Not a duplicate. Please search for existing submissions.
[x] Only has awesome items. Awesome lists are curations of the best, not everything.
[x] Does not contain items that are unmaintained, has archived repo, deprecated, or missing docs. If you really need to include such items, they should be in a separate Markdown file.
[N/A] Includes a project logo/illustration whenever possible.

Either centered, fullwidth, or placed at the top-right of the readme. (Example)
The image should link to the project website or any relevant website.
The image should be high-DPI. Set it to maximum half the width of the original image.

[x] Includes the Awesome badge.

Should be placed on the right side of the readme heading.

Can be placed centered if the list has a centered graphics header.

Should link back to this list.

[x] Entries have a description, unless the title is descriptive enough by itself. It rarely is though.
[x] Has a Table of Contents section.

Should be named Contents, not Table of Contents.
Should be the first section in the list.
Should only have one level of nested lists, preferably none.
Must not feature Contributing or Footnotes sections.

[x] Has an appropriate license.

We strongly recommend the CC0 license, but any Creative Commons license will work.

Tip: You can quickly add it to your repo by going to this URL: https://github.com/<user>/<repo>/community/license/new?branch=main&template=cc0-1.0 (replace <user> and <repo> accordingly).

A code license like MIT, BSD, Apache, GPL, etc, is not acceptable. Neither are WTFPL and Unlicense.
Place a file named license or LICENSE in the repo root with the license text.
Do not add the license name, text, or a Licence section to the readme. GitHub already shows the license name and link to the full text at the top of the repo.
To verify that you've read all the guidelines, please comment on your pull request with just the word unicorn.

[x] Has contribution guidelines.

The file should be named contributing.md. Casing is up to you.
It can optionally be linked from the readme in a dedicated section titled Contributing, positioned at the top or bottom of the main content.
The section should not appear in the Table of Contents.

[x] All non-important but necessary content (like extra copyright notices, hyperlinks to sources, pointers to expansive content, etc) should be grouped in a Footnotes section at the bottom of the readme. The section should not be present in the Table of Contents.
[x] Has consistent formatting and proper spelling/grammar.

The link and description are separated by a dash. Example: - [AVA](…) - JavaScript test runner.
The description starts with an uppercase character and ends with a period.
Consistent and correct naming. For example, Node.js, not NodeJS or node.js.

[x] Does not use hard-wrapping.
[x] Does not include a CI (e.g. GitHub Actions) badge.You can still use a CI for linting, but the badge has no value in the readme.
[x] Does not include an Inspired by awesome-foo or Inspired by the Awesome project kinda link at the top of the readme. The Awesome badge is enough.

Go to the top and read it again.

unicorn

Why was the previous pull request closed? There's really never any reason to open a new pull request. That just creates noise in the issue tracker.

Because the previous request did not totally comply with your rules.
You were warned before closing, but I didn't get a response for over a week.
After that, the request was fixed and reopened.

46485da130d3371c9ceb95c100a220ca64058cdb

Okay, I'll close my request next week because I think sindersohus won't merge it. I don't want to wait another few months again.

What do you gain by closing?

authors: Jogai Simeon221 artiomn sindresorhus

repo: sindresorhus/awesome


text:
Add Awesome Front Only link

Awesome tools for creating apps using only front end technologies such as html, css and javascript and awesome front only apps.
https://github.com/teles/awesome-front-only

Thanks for the suggestion, but there's almost no content in that list. Awesome lists are hard work. Put some more work into it.

Ok @sindresorhus
Thanks :smile:

authors: sindresorhus woliveiras


text:
Update index.js

This will also probably address https://github.com/sindresorhus/gulp-ava/issues/8.

Can you give the PR a proper title?

Done.

authors: opensrcken sindresorhus

repo: sindresorhus/gulp-ava


text:
Drop aggregate-error and p-cancelable

Both are now platform-native:

https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/AggregateError
https://developer.mozilla.org/en-US/docs/Web/API/AbortSignal

The whole module needs to be rewritten to make use of these APIs. I'm thinking that when they're all settled and count hasn't been reached, the module can just return Promise.any(input)

👍

I'm thinking that when they're all settled and count hasn't been reached, the module can just return Promise.any(input)

I don't understand. You want it to return a single value then, when the expectation is to get an array of values of the specified count? That sounds very wrong. We could potentially add an option to allow it to return less than count without throwing.

Promise.any() will return an aggregate error exactly like the module is doing, except it builds one for you. Is that not what's happening?

Ah, it was not clear to me that you meant that it should throw like Promise.any, since you used the wording "return".

authors: fregante sindresorhus

repo: sindresorhus/p-some


text:
consider to support -s and --relative-to as it does on linux

I'm looking for solution to provide realpath shim on macos to match the behavior as on linux.
I was going to make a cli package and did a name clash checking, then I saw your package :)

Done in https://github.com/sindresorhus/realpath/releases/tag/v3.0.0

authors: beenotung sindresorhus

repo: sindresorhus/realpath


text:
Remove newlines around TODO checkboxes in the editor

On https://github.com/sindresorhus/refined-github/issues/new, there is an "Add a task list" button. There is no autocomplete on Enter and I often use a button again to add a new item but Github adds 2 newlines around the TODO item with checkbox and in 90% of cases, that's not what I want (I tend to have headers for each task list, so I need to add newlines anyway). I would like to have a feature that can be toggled in the settings of the extension, e.g. disable-task-list-newlines that would make the "Add a task list button" simply add a new checkbox item on the next line (1 newline that is). This is the default behaviour of Gitlab if I recall correctly.
I am almost embarrased to ask for this as I cannot imagine I am the only one stuggling with constant loop of pressing the "Add a task list" and removing two extra lines (unless I get tired and copy an empty task list item with a following newline and paste a dozen). I did thoroughly check https://github.com/Mottie/GitHub-userscripts and the issue tracker.

Heh, that's kind of a ridiculous behavior, but reimplementing it is kinda difficult.
Look at this file: https://github.com/sindresorhus/refined-github/blob/master/source/helpers/smart-block-wrap.ts
That code replicates the current "block" injection that GitHub uses. Implementing this feature would mean copying that so it works when starting a new list but adjusting it so that checkboxes are injected together.
I can think 2 alternative solutions:

detect the injection of a checkbox and subsequently just drop the extra newline via textFieldEdit.replace (this creates a new undo entry)
reuse smartBlockWrap but drop the extra line before injecting the text via textFieldEdit.insert

Thank you, I will see if I can prototype a possible solution. Does Refined GH use text-field-edit already?

Yes

Natively supported!! https://github.blog/changelog/2020-12-15-markdown-list-syntax-now-autocompleted/

I’m super glad we don’t have to deal with this 😌

authors: berezovskyi fregante yakov116

repo: sindresorhus/refined-github


text:
Fixed PR filters popup being obscured

The sticky table-header-bar is obscuring the PR filter list:

DId you verify that this works as intended and that it doesn’t revert #1317?

@bfred-it I didn't rebuild and reinstall the extension, but I tested it using the dev tools. Any value for z-index between 20 and 30 (not inclusive) seems to work fine.

Verified and merged, thanks!

authors: bfred-it hubertgrzeskowiak


text:
QuillToolbarFontFamilyButton requires QuillToolbarProvider when is used in QuillBaseToolbar

I use QuillBaseToobar where I can customize the sequence of toolbar buttons but I get a crash because some buttons (like QuillToolbarFontFamilyButton) required QuillToolbarProvider. I think it is not required because QuillBaseToobar has its own configurations.
Actually it is not a bug because I can provide the QuillToolbarProvider but it confuses because now I have 2 similar configurations. One is from QuillToolbarProvider and another from QuillBaseToobar.

I use QuillBaseToobar where I can customize the sequence of toolbar buttons but I get a crash because some buttons (like QuillToolbarFontFamilyButton) required QuillToolbarProvider. I think it is not required because QuillBaseToobar has its own configurations.
Actually it is not a bug because I can provide the QuillToolbarProvider but it confuses because now I have 2 similar configurations. One is from QuillToolbarProvider and another from QuillBaseToobar.

Hi, thank you for the report, we will take a look at it soon.

I use QuillBaseToobar where I can customize the sequence of toolbar buttons but I get a crash because some buttons (like QuillToolbarFontFamilyButton) required QuillToolbarProvider. I think it is not required because QuillBaseToobar has its own configurations.
Actually it is not a bug because I can provide the QuillToolbarProvider but it confuses because now I have 2 similar configurations. One is from QuillToolbarProvider and another from QuillBaseToobar.

I use QuillBaseToobar where I can customize the sequence of toolbar buttons but I get a crash because some buttons (like QuillToolbarFontFamilyButton) required QuillToolbarProvider. I think it is not required because QuillBaseToobar has its own configurations.
Actually it is not a bug because I can provide the QuillToolbarProvider but it confuses because now I have 2 similar configurations. One is from QuillToolbarProvider and another from QuillBaseToobar.

This issue has been fixed in 8.6.0, please tell us if you still facing this issue

QuillToolbarProvider

You no longer need to use QuillToolbarProvider to use the buttons

I use QuillBaseToobar where I can customize the sequence of toolbar buttons but I get a crash because some buttons (like QuillToolbarFontFamilyButton) required QuillToolbarProvider. I think it is not required because QuillBaseToobar has its own configurations.
Actually it is not a bug because I can provide the QuillToolbarProvider but it confuses because now I have 2 similar configurations. One is from QuillToolbarProvider and another from QuillBaseToobar.

This issue has been fixed, if you still face it please let us know
Feel free to ask us any questions at anytime.

authors: bambinoua freshtechtips

repo: singerdmx/flutter-quill


text:
single-spa routing doesn't work after deployment to apache directory

The root application always loads default app.
My deployment location is http://example.co.jp/emp
On load I am able to load the following default application
http://example.co.jp/emp#login
But when I try to navigate to my second app using  http://example.co.jp/emp/user/list, I get redirected to default app with following url:
http://example.co.jp/emp/user/list#login
The APP_BASE_HREF for my second app is '/' and the router path is 'user'.
Everything seems to be working when my deployment location is http://example.co.jp , but because of some permission issues we can't make the final deployment to this location.
I am not able to figure out the root cause of this behavior. Tried apache rewrite rule as well but nothing seems to be working for this.
Any help would be really appreciated.

Hi @jyoti0291, I've transferred this issue to the single-spa-angular repo since it seems you're using Angular (APP_BASE_HREF).

But when I try to navigate to my second app using http://example.co.jp/emp/user/list, I get redirected to default app with following url:
http://example.co.jp/emp/user/list#login

Could you clarify what you expect to happen? Did you expect the URL not to have the #login part in it? Or what is wrong.
Additionally, if you could provide a demonstration of the issue, that would help a lot.

Thanks so much for quick response @joeldenning
Could you clarify what you expect to happen? Did you expect the URL not to have the #login part in it? Or what is wrong.
I want my second app to load when I hit http://example.co.jp/emp/user/list as per my following router configuration

But instead it loads the default app which is
Default emp app is working fine on initial load

Expected user screen on hitting http://example.co.jp/emp/user/list

But instead getting following screen which is again my default app:

The application is working as per the expectation when my deployment location is http://example.co.jp instead of http://example.co.jp/emp

Seems like you have two different paths: /emp/user and /user. The single-spa-layout config you showed only matches the /user route, not the /emp route. This could be caused by your link's href not starting with /. The link should be an absolute url -  not page-relative url.

The link should be an absolute url - not page-relative url.

Could you please elaborate this? Are you talking about the path in single-spa-layout config?
Actually the application and the routing is working properly when I am deploying the application to the root of the apache server e.g. http://example.co.jp because base href is / for the application
And if I am deploying it inside a directory e.g. http://example.co.jp/emp, emp is a directory is apache server. The routing doesn't work and I guess the root cause is base href "/" but I am not able to resolve the issue by adding /emp/ in the paths of the route.

But when I try to navigate to my second app using http://example.co.jp/emp/user/list, I get redirected to default app with following url:

/emp/user is the wrong URL for the user application, based on your single-spa-layout route definitions. This is what I mean.
<!-- wrong -->
<a [routerLink]="['user']">
  This is a relative link instead of absolute
</a>

<!-- correct -->
<a [routerLink]="['/user']">
  This is an absolute link
</a>

repo: single-spa/single-spa-angular


text:
Documentation is importing the "Config"

Issue description
The documentation for Plugins is still importing the Config component instead of the Plugins.
Steps to reproduce

Adding URLs
Enable Paging

Versions

yarn: 1.3.2
node 6.9.1
dekk: 1.0.0-5

thx .. will fix along the way.

authors: TimPietrusky pixelass

repo: sinnerschrader/dekk


text:
Snapshot plugin install link broken . Tag 5.4.2 wont exist

https://github.com/sirensolutions/sentinl/#snapshot-plugin-install
this link wont work ->
https://github.com/sirensolutions/sentinl/releases/download/tag-5/sentinl-v5.4.2.zip

rebulding of releases in progress, 5.4.2 will reappear momentarily - unfortunately our features have to be backported each time Elastic decides to add a comma to the code.

authors: ffahri lmangani

repo: sirensolutions/sentinl


text:
Adding here a few ideas for new features

Combinatory generators length
It would be nice to have handy functions to calculate the length of a combinatory generator (given the length of the input sequence). This way I can pick the nth item (for example).
flatten
A simple flatten (only one level), can be useful. For example I could use flatten(zipAll(...)) to interleave many iterators. You can do the same with "chain" and the spread operator, but would require to expand the argument in an array and it would be inefficient
window
window(3, range(100))
[0, 1, 2] [1, 2, 3] [2, 3, 4] [3, 4, 5] etc.

Better to pick up a different name though
collate
It takes as arguments n sorted iterable. It returns a single sorted iterable.
Reduce variant
Similar to the reduce but returning a iterable that return the accumulator.
I don't know yet how to call it.

I want a really simple one, perhaps called when, which would take a condition and an item, and would emit an iterable of 0 or 1 items depending on the value of condition. It would only be useful to the extent that it was more compact and readable than using a normal ternary.
For example:
[
  ...(condition1 ? [item1] : []),
  ...(condition2 ? [item2] : []),
  // vs.
  ...when(condition3, item3),
  ...when(condition4, item4),
]

A simple flatten (only one level), can be useful.

Why only one level though? ES native's array.flatten(n) accept any level.
I suggest this:
flatten(iterable) // flatten infinitely (until meeting a non-iterable)
flatten(n, iterable) // flatten n levels
flatten(fn, iterable) // flatten until fn(x) returns false (x is element/sub-element of iterable)

This looks a reasonable implementation though: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Array/flat

Added #79 #80

I may have to correct your TypeScript declarations.

There's nothing conceptually wrong with not treating strings as iterables for the purpose of a flatten, even though they technically are. I'd argue that even a not arbitrarily deep flatten implementation would want to do that. Obviously if you did let flatten into a string it would blow up as the string iterator returns other strings, which becomes an infinite recursion.
As for "all other types" being iterable, I'm sure I don't know what you mean. Almost every type in javascript is derived from Object, which is not iterable, and this is probably a big part of why. That leaves Array, Map, Set, and classes with iterators defined, which are all acceptable things to flatten.

I am undecided on 3 things:
About strings
I can see 2 possible approaches:

they are never considered iterables
they are not considered iterables when length == 1

About map, set, and others
There are cases where I don't want to flatten a map or a set.
About passing a function
I can see the positive side. It solves the problem above as I can decide when to flatten depending on depth and item type.
On the negative side, the behaviour diverge from Array.prototype.flat (but this is probably not a big deal).

On the negative side, the behaviour diverge from Array.prototype.flat (but this is probably not a big deal).

This is a reasonable trade-off I think, as the ability to decide when to flatten is incredibly powerful.
The next problem is how do we gonna invoke the callback function, there're few ways:
fn(depth, element) => boolean
fn(element, depth) => boolean
fn({ depth, element }) => boolean

About map, set, and others
There are cases where I don't want to flatten a map or a set.

We already made exception for string as it is special. We should treat Map and Set as any other iterables to stays consistent.
Also, there's a rule of thumb in designing software called Zero One Infinity.

BTW, @sithmel, if you ever decide to change behavior of flat(iterable) from depth = 1 to infinitely flatten, notify me so that I may update the type declarations.

I published iter-tools@6.1.8-next.1 with flat and cursor

Currently working at merge and asyncMerge:
It reads multiple iterable and uses a function to decide what item to return every time.
it allows to get something like the collate described above

Awesome. As regards flatten, could I suggest that depth 0 does a deep flatten?
Also for the docs, isNotAnIterableString is not the right name for the example callback your provide. The function you wrote is isNotANonIterableString which I think could be simplified to isNotACharacter.

Ok, with "isNotACharacter"
I am not 100% sure about 0 to trigger the deep flatten behaviour.
I think it is more obvious if 0 is "no flatten". You can still use Infinity for deep flatten. Array.prototype.flat works that way

Oooh ok that's fine with me. I think with the callback I can actually just use this flatten implementation for Sequins, which is great.

authors: KSXGitHub conartist6 sithmel

repo: sithmel/iter-tools


text:
Feature request: profile name in OSX Keychain Item Name

In the OSX Keyring implementation, keychain items are named awscli:key and awscli:secret.  It would be nice if the profile name was included in the item name, so they could be differentiated (e.g. in Keychain Access).

This would probably be a breaking change for existing users, unless you could migrate their items.

They should be differentiable by account, which is the profile name.

Yes, you can see the account if you drill into the individual item.  Here's the issue I was thinking of:

That's kinda just how Keychain works, I'm afraid:

My suggestion would be to pass in the profile as part of the service parameter to the keyring library's methods:
keyring.set_password("awscli:key-" + profile, profile, key)

It's not a big deal, but if you had a lot of them, there's no easy way to differentiate them in the list.

Keychain actually supports a label, for exactly this purpose.

authors: lox simpsora sj26


text:
mail_catcher.rb:23: Use RbConfig instead of obsolete and deprecated Config

This warning arises on 1.9.3 owing to the deprecation of Config, now replaced with RbConfig. I didn't find reference to this in the Ruby API docs, but I also didn't spend a long time looking, here's the exact error:
RbConfig instead of obsolete and deprecated Config

I see this is fixed in HEAD, bundler must be refusing to update for some reason. Apologies


text:
Bitbucket:349 HTMLクリップボード内のスクリプトエラーを無視する

そういやForms版では無視してたなーっと。
コピー内容によりスクリプト参照が狂うからユーザー責任じゃなくて実装側でお世話してあげねば。

Software: Pe
Version: 0.69.0.38641-6328525aa1e014abdf212350cd2e5dba53cb3ca4
Type: RELEASE
Process: 64
Platform: 64
OS: Microsoft Windows NT 6.1.7601 Service Pack 1
CLI: v4.0.30319

BitBucket から移植

項目
内容

元課題
#349

起票日時
2015-11-03T22:21:32Z

起票者
sk_0520

resolved #349
→ 《cset c5500742c661》

BitBucket から移植

項目
内容

元コメント
#349 コメント

コメント日時
2015-11-05T20:29:06Z

コメントユーザー
sk_0520

Removing component: PeMain-Clipboard (automated comment)

BitBucket から移植

項目
内容

元コメント
#349 コメント

コメント日時
2020-02-24T21:18:24Z

コメントユーザー
sk_0520


text:
Bitbucket:363 クリップボード重複判定の初期値を全件対象にする

せっかくハッシュ値持ってんだからちまちまやらなくてもいいと思う。

Software: Pe
Version: 0.70.9.23745-xxxxxxxxxxxxxxDEVELOPMENTxxxxxxxxxxxxxxx
Type: DEBUG
Process: 32
Platform: 64
OS: Microsoft Windows NT 6.1.7601 Service Pack 1
CLI: v4.0.30319

BitBucket から移植

項目
内容

元課題
#363

起票日時
2015-11-15T13:18:57Z

起票者
sk_0520

resolved #363
→ 《cset 96a49510c95c》

BitBucket から移植

項目
内容

元コメント
#363 コメント

コメント日時
2015-11-15T14:08:01Z

コメントユーザー
sk_0520

Removing component: PeMain-Clipboard (automated comment)

BitBucket から移植

項目
内容

元コメント
#363 コメント

コメント日時
2020-02-24T21:18:24Z

コメントユーザー
sk_0520


text:
Fatal Error: Variable inttime has the wrong data type, in UVREAD

GM reports an issue with inttime datatype when using miriad invert task:
Hi, I have set the missing items (restfreq,vsource,veldop) on the correlation_burst_141_20240110_13238_0.uv file prvided by
SA  (I have simply renamed as test.uv).
puthd in=test.uv/restfreq value=0.000000000d0
puthd in=test.uv/vsource value=0.00000
puthd in=test.uv/veldop value=0.00000

I’ve also set  jyperk and systemp (as I was used to do for aavs2 data):
puthd in=test.uv/jyperk value=1704.0
puthd in=test.uv/systemp value=300.0

After this, uvflag and mfcal works properly,
uvflag vis=test.uv select=auto flagval=flag
mfcal vis=test.uv refant=10 stokes=xx,yy select=‘uvrange(0.005,1)’ flux=24000.0,0.10,1.8590

However invert task complains:
invert vis=test.uv map=test.map beam=test.beam imsize=512 cell=1260.0 sup=0 stokes=xx select=‘uvrange(0.005,1)’ options=double,mfs
invert: Revision 1.28, 2019/07/04 06:32:48 UTC
Reading the visibility data ...
Applying bandpass corrections to test.uv
Applying gain corrections to test.uv
### Fatal Error: Variable inttime has the wrong data type, in UVREAD
I checked the value of inttime is there with:
uvlist vis=test.uv options=var,full
uvlist: Revision 1.8, 2013/08/30 01:49:21 UTC
UV Listing for data-set test.uv
Options: full,variables
------------------------------------------------------------
Header variables at 24JAN10:03:40:38
altitude:  359.50    antnames: ( 1992 elements)
antnums : ( 256 elements)            antpos : ( 768 elements)
cnt   :  1.0000    dec   : -26:43:35.76  degpdy :  360.99
dut1  : 1.28374E-02  epoch  : 2000.00    gst0  :  288.53
instrume:aavs3     inttime : 0.49545    ischan :    1
jyperk : 1704.00    latitud : -26:42:14.68  longitu : +116:40:12.84
nants  :   256    nbls  :  32896    nblts  :  65792
nchan  :    1    npol  :    4    nschan :    1
nspect :    1    ntimes :    2    obsdec : -26:42:14.68
obspa  : -2.52681E-03  obsra  : 18:44:01.21  phsframe:icrs
pol   :   -5    ra   : 18:42:33.31  rdate  :2024-01-09
restfreq:  0.0000    sdf   :  0.0000    sfreq  : 0.11016
source :zenith_at_jd2460319.653227        systemp : 300.000
telescop:aavs3      timesys :UTC       veldop : 0.00000
visunits:uncalib     vsource : 0.00000

I have also tried to change the data type (with puthd), with no success…
SA has replicated all these tasks on correlation_burst_141_20240110_13238_0.uv with their Miriad version.

pyuvdata writes this as a double:
        uv.add_var("inttime", "d")
        uv["inttime"] = self.integration_time[viscnt].astype(np.double)

On lines LK1949 and L2023 of miriad.py.

The error is being thrown inmiriad/src/subs/uvio.c:
  v = uv_locvar(tno,varname);
  if(v == NULL) ERROR('f',(message,
	"Variable %s is missing, in UVREAD",varname));
  else if(type != 0 && type != v->type)ERROR('f',(message,
	"Variable %s has the wrong data type, in UVREAD",varname));
  else if(v->buf == NULL || v->length <= 0)ERROR('f',(message,
	"Variable %s was not initialised before it was required, in UVREAD",varname));
  return(v);
}

It is expecting a float or 'real', based on the inttime def:
{
  double *restfreq,*tab;
  float bw,inttime,jyperk,*syst,*t1,*t2,factor;
  int i,j,bl,i1,i2,nants,nsyst,*nschan,start;
  off_t offset;
  LINE_INFO *line;
  VARIABLE *tsys;

Elsewhere in miriad code, inttime is expected to be a real, which is a 32-bit float:
typedef float *realptr, real;
...
real inttime(ATBASE),inttim

I've checked against the ATNF version too. So issue is that pyuvdata is writing float64 to a float32 field.