Proposal to revise naming for transcriptional repressors & add a few grouping classes

On a related ticket, we've been discussing how to shorten some long and confusing names for transcriptional repressors, e.g.
RNA polymerase II activating transcription factor binding transcription factor activity involved in negative regulation of transcription
Ruth wrote a comment on the original ticket with a general proposal for renaming and grouping transcriptional repressor terms.  I've made a fresh ticket for discussion here so that original ticket can be closed ones the specific issue it covers has been adressed.

PMID:24418103 describes:
Mechanisms of transcriptional repression can be divided into three categories;

inhibition of the basal transcriptional machinery: repressors that interfere with the binding of RNA polymerase, TATA box-binding protein, and/or general transcription factors to transcription start sites; thus, preventing transcription initiation
ablation of transcriptional activator function: repressors targeting transcriptional activators and co-activators, resulting in their degradation, nuclear export, or inhibiting their ability to bind DNA.
remodeling of chromatin : recruitment of chromatin remodeling factors, which cause epigenetic alterations on the genome.

Plus 4: non-coding RNAs in transcriptional repression by interacting with chromatin modifying factors (Wang et al., 2010).
PMID:12785719 describes:
General repression: General repressor protein (or complex) either sequesters or modifies a central component of the Pre-initiation complex (PIC) or RNA polymerase II. Thus general repressor will down-regulate expression of all genes transcribed by this RNA polymerase.
Eg 1: Phosphorylation of RNA polymerase core subunit by herpes virus ICP22, UL13  kinases, which generates a non-functional polymerase and general repression[ ref 14].
Eg 2: Nucleosomes generally repress promoter activity by sequestering promoter DNA into chromatin [ref 15].
Gene-specific repression: Gene-specific repressors act by decreasing the concentration of a functional activator/co-activator at the promoter or by counteracting the stimulatory effect of these proteins on transcription. In addition, some repressors inhibit transcription by interacting in a promoter-specific fashion with components of the PIC or by recruiting chromatin-remodelling proteins.
Gene-specific repressors may:
Bind directly or indirectly to DNA,
Regulate transcription from binding sites proximal [short-range repressors] or at a distance [long-range repressors] from the promoter [ref 16].
So considering the type of experiments we are annotating would it be possible to create MF terms with sensible names to represent these different classes of proteins?
Firstly do we have to have ‘RNA polymerase II’ included in the name, as the ontology will provide this information, and this can be included in the exact synonym?
If this can be removed we could have:
NEW GO Term 1. General transcription repressor activity (provide more specific terms as the data is annotated?)
Exact synonyms:
RNA polymerase II general transcription repressor activity
NEW GO Term 2. Transcription factor binding repressor activity
Exact synonyms:
Transcription factor binding gene-specific transcription repressor activity;
RNA polymerase II transcription factor binding gene-specific transcription repressor activity
NEW GO Term 3 From initial inquiry and child of NEW GO Term 2 above: activating transcription factor binding repressor activity
Exact synonyms: RNA polymerase II activating transcription factor binding transcription factor activity involved in negative regulation of transcription;
Activating transcription factor binding gene-specific transcription repressor activity;
RNA polymerase II activating transcription factor binding gene-specific transcription repressor activity
NEW GO Term 4. DNA binding transcription repressor activity
Exact synonyms:
DNA binding transcription gene-specific transcription repressor activity;
Gene-specific DNA binding RNA polymerase II transcription repressor activity
NEW GO Term 5. Pre-initiation complex binding transcription repressor activity
Exact synonyms:
Pre-initiation complex binding gene-specific transcription repressor activity;
Gene-specific Pre-initiation complex RNA polymerase II transcription repressor activity
NEW GO Term 6. Transcription repressor activity by regulation of chromatin-remodeling
Exact synonyms:
Gene-specific transcription repressor activity by regulation of chromatin-remodeling;
Gene-specific RNA polymerase II transcription repressor activity by regulation of chromatin-remodeling
Note that several repressors will be annotated to more than one of these functions.
Plus the definitions would have to make it clear that these are repressing terms and not for co-repressing terms.
I am happy to help sort out definitions for these if the concept is agreed.
In addition, this should be discussed with the transcription factor group in Norway.
Best
Ruth
Reported by: dosumis
Original Ticket: geneontology/ontology-requests/11716

Hi @RLovering Do you still want these terms (or the concepts?) It seems slightly different from the direction we took in the current remodeling.
Thank you in advance for your response, Pascale

Hi Pascale
As mentioned at the top of the ticket, it seems to me that the following repressors are well known to the transcription field, although not easy to define: PMID:24418103 describes:
Mechanisms of transcriptional repression can be divided into three categories;

inhibition of the basal transcriptional machinery: repressors that interfere with the binding of RNA polymerase, TATA box-binding protein, and/or general transcription factors to transcription start sites; thus, preventing transcription initiation
ablation of transcriptional activator function: repressors targeting transcriptional activators and co-activators, resulting in their degradation, nuclear export, or inhibiting their ability to bind DNA (or inhibiting a TF activity in some way).
remodeling of chromatin : recruitment of chromatin remodeling factors, which cause epigenetic alterations on the genome.

It seems to me that the recent discussions led to an agreement for terms to cover activities described in 1 and 3, but I don't think there is any term to describe a protein with one of the (several) functions described in 2. I don't want to edit the protein binding TF terms until there is a term I can use if it turns out that one of the proteins has a function described in 2, otherwise I will have to go back and reannotate again. Plus as we mentioned in the discussion the other week. Many of the proteins currently associated with the GO terms now describing 1 or 3 were associated with the term(s) assuming these described the function in 2. But we can't revise these until we have these other terms.
I appreciate that you could just cover these 'activity' as processes but I think people will expect to see a MF term to describe the repressors that act on the DNAbinding TFs.
Thanks
Ruth

NEW GO Term 1. General transcription repressor activity (provide more specific terms as the data is annotated?)
Exact synonyms: RNA polymerase II general transcription repressor activity

Covered by 'GO:0003714 transcription corepressor activity'

NEW GO Term 2. Transcription factor binding repressor activity
Exact synonyms:
Transcription factor binding gene-specific transcription repressor activity;
RNA polymerase II transcription factor binding gene-specific transcription repressor activity

Also covered by 'GO:0003714 transcription corepressor activity'

NEW GO Term 3 From initial inquiry and child of NEW GO Term 2 above: activating transcription factor binding repressor activity
Exact synonyms: RNA polymerase II activating transcription factor binding transcription factor activity involved in negative regulation of transcription;
Activating transcription factor binding gene-specific transcription repressor activity;
RNA polymerase II activating transcription factor binding gene-specific transcription repressor activity

Not needed anymore.

NEW GO Term 4. DNA binding transcription repressor activity
Exact synonyms:
DNA binding transcription gene-specific transcription repressor activity;
Gene-specific DNA binding RNA polymerase II transcription repressor activity

Already in the ontology.

NEW GO Term 5. Pre-initiation complex binding transcription repressor activity
Exact synonyms:
Pre-initiation complex binding gene-specific transcription repressor activity;
Gene-specific Pre-initiation complex RNA polymerase II transcription repressor activity

Also covered by corepressor

NEW GO Term 6. Transcription repressor activity by regulation of chromatin-remodeling
Exact synonyms:
Gene-specific transcription repressor activity by regulation of chromatin-remodeling;
Gene-specific RNA polymerase II transcription repressor activity by regulation of chromatin-> remodeling

Also covered by corepressor

resulting in their degradation: p53 on gene, when phosphorylated by PIC it leaves promoter.
and
nuclear export: nucleoplasmic shuttling eg NFkappaB by IKKB, not part of TF ontology restructure. need GO term eg Db Transcription factor inhibitor/repressor activity include in defintion that this often occurs in the cytoplasm and may include HSPs egs would be steroid receptor/NkappaB binding proteins.

are related to sequestering, see #16157
Closing this.


Proposal: a safe predicate for JSON booleans

Here's a snippet of code I've been using to check whether something is true or not, regardless of whether null and the booleans are parsed as symbols or not:
(defun true-p (yason-bool)
  (case yason-bool
    ('yason:false nil)
    (:null nil)
    (otherwise yason-bool)))

It's especially helpful when writing libraries using YASON where you don't want to make assumptions about what the user might have bound *parse-json-booleans-as-symbols* and *parse-json-null-as-keyword* but still need to reliably check truth values.
The question, before I go ahead and make a patch out of it, is how strictly it should adhere to what is true and false in JavaScript. In JavaScript the number 0 and the empty string is also false, which is just two more clauses in the case expression, but empty array in JavaScript is true, so is empty array in Lisp, but that conflicts with parsing arrays as lists, or bools as t/nil.
I would prefer to keep it as it stands, lispers are already used to using zerop and checking sequence lengths, and the latter is probably too much of a fringe case to be worth the trouble.

I'm not sure I understand what this predicate is meant to be used for.  I
wonder whether it would not be better if a library that uses YASON properly
configures it rather than making any guesses about its parse outcome.
Also, I don't think that JavaScript semantics are applicable in this
context, but I might be wrong.  Can you try to explain a use case?
2015-01-08 13:36 GMT+01:00 Grim Schjetne<EMAIL_ADDRESS>
Here's a snippet of code I've been using to check whether something is
true or not, regardless of whether null and the booleans are parsed as
symbols or not:
(defun true-p (yason-bool)
(case yason-bool
('yason:false nil)
(:null nil)
(otherwise yason-bool)))
It's especially helpful when writing libraries using YASON where you don't
want to make assumptions about what the user might have bound
parse-json-booleans-as-symbols and parse-json-null-as-keyword but
still need to reliably check truth values.
The question, before I go ahead and make a patch out of it, is how
strictly it should adhere to what is true and false in JavaScript. In
JavaScript the number 0 and the empty string is also false, which is just
two more clauses in the case expression, but empty array in JavaScript is
true, so is empty array in Lisp, but that conflicts with parsing arrays as
lists, or bools as t/nil.
I would prefer to keep it as it stands, lispers are already used to using
zerop and checking sequence lengths, and the latter is probably too much
of a fringe case to be worth the trouble.
--
Reply to this email directly or view it on GitHub
https://github.com/hanshuebner/yason/issues/30.

My current use case is a library for interfacing with Elasticsearch. I could of course as you said hard-code the behaviour to ensure the data always survives the round-trip, but that would require users to always check the symbols, even if they don't care about preserving JavaScript semantics. A compromise could be to assume the former is what they want, and have a special variable to override it with.

I still don't understand, bear with me:  You are worried that a user of
your library might also use YASON and configure it in a way that breaks
your library?  Or something else?
2015-01-12 11:05 GMT+01:00 Grim Schjetne<EMAIL_ADDRESS>
My current use case is a library for interfacing with Elasticsearch. I
could of course as you said hard-code the behaviour to ensure the data
always survives the round-trip, but that would require users to always
check the symbols, even if they don't care about preserving JavaScript
semantics. A compromise could be to assume the former is what they want,
and have a special variable to override it with.
--
Reply to this email directly or view it on GitHub
https://github.com/hanshuebner/yason/issues/30#issuecomment-69548947.

The Elastic library parses JSON coming from the database. The user may want this parsed either as symbols or t and nil depending on the specifics of their own application. This is a way of passing the choice transparently on to the user.
It may also be of use to others who uses the symbols for any other purpose, which is why I'm making the suggestion. Otherwise I'll just keep it in my own library.

Given that it seems to be hard to explain precisely what is going on and
how, I'll prefer if you keep the function in your own library.  I would
suggest that you do not open up YASON configuration as a means to configure
your Elasticsearch library.
2015-01-12 14:15 GMT+01:00 Grim Schjetne<EMAIL_ADDRESS>
The Elastic library parses JSON coming from the database. The user may
want this parsed either as symbols or t and nil depending on the
specifics of their own application. This is a way of passing the choice
transparently on to the user.
It may also be of use to others who uses the symbols for any other
purpose, which is why I'm making the suggestion. Otherwise I'll just keep
it in my own library.
--
Reply to this email directly or view it on GitHub
https://github.com/hanshuebner/yason/issues/30#issuecomment-69567656.

The complete opposite effect is achieved. No configuration change to YASON alters the behaviour of my library, giving the user more control and making the library more robust.
Precisely what's going on is trivial:
(true-p (parse "false") => nil
(true-p (parse "null") => nil

regardless of the configuration.
The general use case is whenever someone wants to use the symbols option, but still want to use the values in conditionals without the wrong part evaluating. I do it all the time, and if it's useful for anyone else, by all means, take it, otherwise, don't.

Well, either I'm dumb or you are failing to explain what you're trying to
get into YASON.  Let me put it this way:  If you cannot describe how the
function that you want to add would be used, then I don't want it.  I'm
probably dumber than I would like to admit, but I like to believe that I am
not the dumbest person on earth, so for the sake of the users of YASON, let
us assume that I need to understand it.
Maybe you could start freshly and come up with a 4 sentence description of
what your function does and how it would be used by another YASON user.
-Hans
2015-01-12 17:30 GMT+01:00 Grim Schjetne<EMAIL_ADDRESS>
The complete opposite effect is achieved. No configuration change to YASON
alters the behaviour of my library, giving the user more control and making
the library more robust.
Precisely what's going on is trivial:
(true-p (parse "false") => nil
(true-p (parse "null") => nil
regardless of the configuration.
The general use case is whenever someone wants to use the symbols option,
but still want to use the values in conditionals without the wrong part
evaluating. I do it all the time, and if it's useful for anyone else, by
all means, take it, otherwise, don't.
--
Reply to this email directly or view it on GitHub
https://github.com/hanshuebner/yason/issues/30#issuecomment-69597984.

It's really no more useful than a way for letting lazy programmers such as myself check that the value they're dealing with is neither yason:false nor yason:true. There's no real downside to having people define such a function themselves. I would go on for two more sentences, but there's a risk it would just turn into a gross exaggeration of the merits of the idea. :)


Non-numeric ESN not Failed in Test 1.9

Serial number ending in xxxxxxxx was accepted in both 3.1.0 and 3.0.12.
A-1 criteria is potentially not supported.
<IP_ADDRESS> Fail Criteria
a. Fail if there are no positive responses (serial number SP 588 not supported by any OBD ECU).
b. Fail if none of the positive responses are provided by the same SA as the SA that claims to be function 0 (engine). (SP 588 ESN not supported by the engine function).
c. Fail if the serial number field (SP 588) from any function 0 device does not end in 5 numeric characters (ASCII 0 through ASCII 9).
d. Fail if the make (SP 586), model (SP 587), or serial number (SP 588) from any OBD ECU contains any unprintable ASCII characters.17
See Section A.1 for detailed examples of failure and warning criteria.

Error message wasfound


Clicking on a day should show a graph for that day

Clicking on a day should show a graph for that day. Either open a new page or auto scroll the graph up top to the correct day.

I think that's a good idea, except weather forecasts more than 3 days in advance are very inaccurate.
It's probably better to have some sort of overview similar to Geometric Weather's:

@RageGamerBoi it is now implemented


Remove technical debt for the Facebook Graph API Call

In the ManageUserController there is a call to the Facebook Graph API to retrieve the email claim.
This is now built into App Service Authentication & Authorization.

Is this a dup of #31?

Fixed in #54


Add translations to attributes with type=combo to their values

1. Why is this change necessary?
While you can translate labels of an attribute field you can't translate the values itself on a combo field.
2. What does this change do, exactly?
Iterate over arrayStore in options and try to translate those based on their key
3. Describe each step to reproduce the issue or behaviour.
Create a new attribute like
$crudService = $this->container->get('shopware_attribute.crud_service');
        $crudService->update(
            's_articles_attributes',
            'my_cool_plugin',
            'combobox',
            [
                'label' => 'Dropdown',
                'displayInBackend' => true,
                'arrayStore' => [
                    ['key' => 1, 'value' => 'Value 1'],
                    ['key' => 2, 'value' => 'Value 2'],
                    ['key' => 3, 'value' => 'Value 3'],
                ],
            ]
        );

Afterwards you can translate or import translations for the label but not its values. With this PR those will be translated
4. Please link to the relevant issues (if any).
5. Which documentation changes (if any) need to be made because of this PR?
6. Checklist

[ ] I have written tests and verified that they fail without my change
[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.

Hey @pascalheidmann,
thank you for your pull request.
Unfortunately I am failing to understand, why you would want to translate actual values.
Could you please take some time to enlighten me on that?

Your example and the changed code do not match.
Can you stop creating pull requests without testing it 🙈

@PaddyS the naming might be confusing as the key in the arrayStore is the actual value and value is the label which should be translated.
@shyim you are right, I played with that service around but looks like I then committed the wrong changes 🙈I changed that so that this should work as expected.

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-25311
Please use this issue to track the state of your pull request.

Thanks for your Contribution! :blue_heart:


Re-enable binary classification threshold tuning by default

We added this feature on the #346 feature branch, then backed it out in #606 because it was recomputing predict and slowing down automl.
We should re-enable this by default. In order to do so, we'd have to cache the prediction output, which is currently computed in score. The long-term solution is to memoize predictions with a cache (#466), but short-term we should be able to do something.
This also relates to #579, which tracks cleaning up the duplicate code between the pipeline classes' score methods.

@freddyaboulton you're not working on this, right? Can @jeremyliweishih take it?

@jeremyliweishih @dsherry Please take it! The initial analysis showed that simply enabling tuning doesn't improve scores. Using a different data splitting strategy might help!

Moving back to Dev Backlog and will follow through with this after more data splitting work.

Moving back to Dev Backlog and will follow through with this after more data splitting work.

@bchen1116 and I discussed, and we feel this is necessary for #973


Make show-open-prs-of-forks's cache case-insensitive

Something I spotted when commenting on #2921, it doesn't cause any issues as far as I'm aware but it's there if you want it ;)

Good point. I think this specific case isn't an issue though. getRepoURL() is end-user-dependant, because it's just a URL they entered, but getForkedRepo() comes straight from GitHub's DOM.
Unless it's being compared to getRepoURL() anywhere, I don't think it needs to be normalized.


Allow specifying custom pager

Currently, the less pager is assumed to exist in the users PATH. The STACKIT CLI should allow users to specify their preferred pager using the PAGER environment variable.
Additionally, the STACKIT CLI should fall back to not paging at all, if the less command is not found in the user's environment.

Hi @Kumm-Kai,
with the latest STACKIT CLI release v0.6.0, it is now possible to specify a custom pager.


Continuing almost WIP for TSP API

Description
Added blackouts to the model and fleshed out more of the Shipments and TSP endpoints.
Alphabetized the definitions and paths sections.
Reviewer Notes
Make sure I haven't broken anything in active use. I want to keep landing this so a) folks can see what's going on & b) I don't want it to diverge to much from Master
Verification Steps

[*] All tests pass.
[n/a] The requirements listed in Querying the Database Safely have been satisfied.
[n/a] There are no aXe warnings for UI.
[n/a] This works in IE.

👀 I took a look and will keep watching as the parts that touch blackout dates, in particular, take shape.


Not allowed to configure database ID in Notero

Dear Developpers,
Thank you for creating this cool tool! When I attempted to use Notero, I encountered an issue where I couldn’t input or paste the database ID, as shown in the figure below. Do you have any suggestions on how to resolve this?
Some software information for your reference.

OS: MacOS Monterey 12.7.1
Zotero version: 6.0.37
Notero version: 0.5.15

Thanks for your help!

Hi @glycohezhu! The setup process has slightly changed recently, so you no longer need to manually input your database ID. The grayed-out field is a drop-down that's supposed to populate with names of the databases that Notero has access to.
I see in your screenshot it says "no databases are accessible." Can you retry step 4 in the Configure Notion section of the readme to ensure your database is connected to the Notero integration?

Hi @dvanoni, it works now. Thank you very much.


Exec tachikoma update<PHONE_NUMBER>4222

:hamster::hamster::hamster:   Strategy none
Powered by Interval Pull Request as a Service - Tachikoma.io

Tachikoma.io detects file structure for david, you can set strategy, if you put .tachikoma.yml :dog:
#.tachikoma.yml
strategy: 'david'

Enjoy! :poodle:
Tachikoma.io Custom Configure

Coverage remained the same when pulling e5a037105d02da8780c00f2fad657bb301c8f8a6 on tachikoma/update-20150121024222 into 4f25865565d063bb412ec932d426a5fbcf99f40c on master.


in ayushossl sending pull request to ayushsawl

Signed-off-by: ayushsawl<EMAIL_ADDRESS>

please change minor kernel version also


Dont feed data

https://play.nativescript.org/?template=play-vue&id=5UQuCj&v=58
My code does not work.
Why?
The currencies obtained trought from fetch method, do not feed the currencies data.

As I could see in https://play.nativescript.org/?template=play-vue&id=5UQuCj&v=61 you were able to do it. You can find some documentation https://docs.nativescript.org/vuejs/ns-ui/AutoCompleteTextView/aync-data . This issue is not related with Playground itself so I will close it.


When a Symbol with an "as" Alias is present, handle it correctly

Fixes #20
Extract separation logic into a new function and make the regex into
a constant

Build failure doesn't look related to this change.


optionally remove line numbers

part of https://github.com/elifesciences/issues/issues/5515
This tries to handle line numbers as an interim solution. It doesn't attempt to cover all case (e.g. right side line numbers do not seem to happen in our dataset and are not handled).
This feature can be enabled via the property:
grobid.features.remove_line_numbers = true

Some related results:

The one with (line no) are the results for having included this feature - the models haven't been re-trained yet. The drop in author names for DL segmentation model is due to it extracting authors of references for one of the documents.


docs: Generate manifests after push images

What this PR does / why we need it:
Just a small update for the getting-started guide. After doing make && make push I was still having old images deployed in my test cluster. I think it worth to mention in the docs that the manifests should be rebuilt.
Bazel push images creates digest files in
bazel-bin/push-<image-name>.digest

The SHA sums are taken from those files and if the manifests are not regenerated after pushing they will contain the old sums.
Special notes for your reviewer:
Release note:

NONE

/lgtm
/approve


Add 1.9.1 to interop matrix

Add 1.9.1 to interop matrix for PHP, Python, C#, C++, and Ruby.
Node left out because there was a comment on 1.7.x that makes me unsure if I should add it.
I also changed the Git repo URL to utilize the https:// scheme as opposed to SSH, because it does not ask for security key, etc.

@matt-kwong  Please let me know whether Node should be added or not (and if yes, which versions?)

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

@murgatroid99 There's a comment under the last node portability test entry in this file:
         #{'v1.7.1': None}, Failing tests

For that reason, I refrained from adding new entries to this worrying about potential test failures. Please consider adding newer versions of Node if it is appropriate. Thanks!


docs: consistently link to docs/imagepullsecret.md in all charts

Description

server already uses the link
enforcer had the "Container Registry Credentials" section but didn't
use the link

now it just links out

kubeenforcer mentioned this, but in a differently named "Deploy The
Helm Chart"

that section was removed and "Installing the Chart" was moved up
a level consistent with every other chart
"Prerequisites" section was created with "Container Registry
Credentials" under it with the link

TLS auth section was moved under it as well since it too is
a prereq

scanner didn't even mention imagepullsecret

so created "Prerequisites" and "Container Registry Credentials"
sections with the link

Tags
I would specifically only merge this after #103 has been merged, because that fixes a bug in that doc that other docs don't have.

@agilgur5 conflicts

@niso120b conflicts have been resolved.

KubeEnforcer change conflicted with standardization of ToC in #108
Scanner change conflicted with #106 as that change happened to be at the bottom of ToC and this change right after the bottom of ToC


runtime/interpreter: Check the paths first for faster and cheaper detection

Signed-off-by: Kemal Akkoyun<EMAIL_ADDRESS>https://pprof.me/beb2cec956f42f3b78f38fc2a941ffdf

LGTM! Perhaps I missed it but I could not see Python or Ruby stacks in the profile, could you add them if they aren't present?

The profile just from the agent, not the targets.
I can add them as well.

We don't have integration tests for the interpreters, while I don't think this PR introduces any bugs, I think it would be a good idea to verify that everything still works as expected

We don't have integration tests for the interpreters, while I don't think this PR introduces any bugs, I think it would be a good idea to verify that everything still works as expected

Yes, of course, I verified it. I'll attach a profile.


Report notable status, such as failure, queued, uploading, auto retrying, manual retrying, deleting.

Already completed as part of 0.1.0.


eventpp削除

Can I ask why? :-)

Hello @wqking,
Don't mind, this is not an issue with eventpp itself.
Currently this library holds eventpp's callbacklist so that users can add callback to it. This forces the users to install and use eventpp, even if they have different ways to manage events or do not use callback at all.
So I'm thinking that removing dependency to eventpp (and some other dependencies as well) would make it easier to distribute it, considering that this repository is a library but not a standalone executable.
But I think eventpp is still a great library and I want to use it in my other project someday. Thanks!

Thanks for so fast response and nice explanation.
One thing I would like to remind, not about any specified library. You can always include the full source code of a small library in your project if the library doesn't need special build process, make it as if it's part of your project, just keep any license information in the library, then your project won't have dependence on that library. For example, I always do this with CATCH2 unit test library.


Robotics cyborg parts proper ordering

Description
Change

To

Pretty much like the rest of borg types.

borg arm and leg have different ids because they used to exist for drones and shit before borging existed so it trolls the sorting

Sorting seems to work fine if names are changed to be consistent. Ordering happens by name, not by id. See PR for details.

damn, I was too slow


configuration not read with laravel 5.5

Using Laravel 5.5.
Published configuration is not read. adding the filename to the tag helps
private function getApiEndPoint()
{
return config('battlenet-api.battlenet-api.domain');
}

Let me check it why this is the case.

This is because the battlenet-api.php config file has a battlenet-api index in the array.  remove this index, and it works perfectly fine.
Link to my working battlenet-api.php file: https://gist.github.com/sdousley/af4aa0ee6a151c559345eb667dcba75d
Masked my API key for obvious reasons

Hi,
Cool, makes sense. Thanks!
Ola

On 5 Dec 2017, at 14:25, sdousley<EMAIL_ADDRESS>wrote:
This is because the battlenet-api.php config file has a battlenet-api index in the array. remove this index, and it works perfectly fine.
Link to my working battlenet-api.php file: https://gist.github.com/sdousley/af4aa0ee6a151c559345eb667dcba75d https://gist.github.com/sdousley/af4aa0ee6a151c559345eb667dcba75d
Masked my API key for obvious reasons
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/xklusive/laravel-battlenet-api/issues/22#issuecomment-349303019, or mute the thread https://github.com/notifications/unsubscribe-auth/AJqOUro31Uw8CgYXnDy1q0wFB5UaOFoVks5s9URAgaJpZM4Q1CiU.

Didn't work, here is a bit of analysis. This is my findings for this specific issue. Happy to hear if somebody has more complete explanation
removing the tag, i.e. config/battlenet-api.php contains:
return [
// no tag here
[
'api_key' => 'the key',
'locale' => 'the locale',
'domain' => 'https://eu.api.battle.net',
'cache' => true,
],
];
calling config('battlenet-api') will return an indexed array with 2 elements [0] being the default one in the 'vendor' directory and [1] will return my overrides in my config folder. So to get the value from my example from my initial post needs to be
config('battlenet-api')[1]['domain];
So, the your-config.php needs to be an associative array and thus contain tags for all entries. Original format is fine but querying the contents needs to contain the filename, i.e.
config('filenamewithoutextention.tag.tag');
config('battlenet-api.battlenet-api.domain');
filename needs to be in the request.
This is done in Laravel 5.5, php 7.2. Hope it helps
Ola

On 5 Dec 2017, at 22:04, Dániel Hagyárossy<EMAIL_ADDRESS>wrote:
Yeah looks like it might be the reason. Unfortunately I still dont have a working laravel environment to test this :)
I will finish the setup today and apply the change to the repo and push a new version.
PS: Do not store your key in the configuration file. I know it is currently the only option if you don't change the config manually. I will fix that so you can store the key in the .env file as you should.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/xklusive/laravel-battlenet-api/issues/22#issuecomment-349440878, or mute the thread https://github.com/notifications/unsubscribe-auth/AJqOUiXk2sJtGSMsKAWNCyslC2RWhKF7ks5s9a_hgaJpZM4Q1CiU.

Hi Ola
My apologies, I probably was a little unclear in what I said compared to what I meant!
It's not just a case of removing the array index tag, you need to remove the whole top level of the array in that file.
What you need (no comments) is:
return [
'api_key' => 'the key',
'locale'
....
];
This will allow you to access using config('battlenet-api.api_key') etc

I've put up a pull request from my own fork of this to fix this

That’s it. nice and clean!
Ola

On 6 Dec 2017, at 09:39, sdousley<EMAIL_ADDRESS>wrote:
Hi Ola
My apologies, I probably was a little unclear in what I said compared to what I meant!
It's not just a case of removing the array index tag, you need to remove the whole top level of the array in that file.
What you need (no comments) is:
return [
'api_key' => 'the key',
'locale'
....
];
This will allow you to access using config('battlenet-api.api_key') etc
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/xklusive/laravel-battlenet-api/issues/22#issuecomment-349571636, or mute the thread https://github.com/notifications/unsubscribe-auth/AJqOUvGTLLpKs1dbFvukFgVtXZe2NooSks5s9lLFgaJpZM4Q1CiU.

I created a new release v1.0.1 with the fixes.
Thanks for the help


Drop nbscuid name

I renamed nbscuid/ -> cupid/, and tried to replace every instance of nbscuid with cupid (including the conda environment names).
I've also updated the README file and verified the instructions provided still work -- this required changing a few paths in the nblibrary YAML file because directories on scratch had been scrubbed.

I also double checked all of the nbscuid places that I found against this list of changes, and it looks great to me!


Improve user select prevention for keyboard on iOS

iOS needs the -webkit prefix to actually disable user’s selection.
See: https://caniuse.com/mdn-css_properties_user-select

Perfect! I forgot about this, I'm using android myself but even though I was made aware of this issue early on I never remembered to investigate it.


Fix literal support in token tree to ast item list

This PR implements following things :

Expose next_token from ra_parse
Fix the literal conversion in token_tree_to_ast_item_list
Add test for the conversion

bors r+
Thanks!

Im checking now, seem like it break others thing which something is passed to as literal but not actual is in heavy_tests

it might be a bug in our lexer as well!

 My added test in the lib.rs didn't even test the real thing, it only test the tt::TokenTree conversion, but what i really want is testing for TtTokenSource.

@matklad What is gen_test failed ? Should i run cargo gen test ?

Yep, cargo gen-tests should do the trick. See
https://github.com/rust-analyzer/rust-analyzer/blob/master/docs/dev/architecture.md
and
https://github.com/rust-analyzer/rust-analyzer/blob/master/.cargo/config

I am Windows user, seem like the line ending break the gen_test content comparison when i run the full cargo test, which ask me to regenerate all the tests. Have anyone tried this in Windows before?

Ah, that's because git on windows changes \n on \r\n on checkout. THe proper solution here is to add an .gitattributes files to the test data folder, which instructs git to checkout and commit files as is.

Ah, that's because git on windows changes \n on \r\n on checkout. THe proper solution here is to add an .gitattributes files to the test data folder, which instructs git to checkout and commit files as is.

I will create another PR for that.

Problem fixed. It can be reviewed again.

bors r+


Defaults are not properly dealt with on MariaDB 10.2.7+ (including NULL)

I know it says in the README that the library supports MySQL 5.7+, but since MariaDB is commonly used to replace MySQL (e.g in linux distros, notably CentOS/Debian...), or at the very least the most common replacement for MySQL, I thought it'd be worth a mention.
MariaDB 10.2 supports expressions on defaults, which MySQL does not, and from MariaDB 10.2.7 onwards, the information schema default column is now evaluated and stored as an expression, and not a string (see: https://mariadb.com/kb/en/library/information-schema-columns-table/)
To be explicit, from MariaDB 10.2.7+:

When the column default is NULL, the COLUMN_DEFAULT column has a string value of "NULL". This breaks detection of NULL defaults, because they're treated as a "NULL" string.
When the column default is 'ABC', the COLUMN_DEFAULT column has a string value of "'ABC'", simple quotes included. This also probably breaks things, but I haven't tested it, as I ran into the NULL defaults problem first.

There's of course no way to handle both MySQL and MariaDB at the same time unless the engine is detected. I suppose SELECT @@GLOBAL.version; could be parsed and defaults parsed accordingly, but I'm not sure if there's an interest from your side to support MariaDB moving forward.
Edit: Alternatively, a mariadb_new_default_schema option could be added (or some name like that), which is more reliable than parsing a server variable

Hi @alxgb Thank you very much for your good feedback.
The question is, what if you run MariaDB in dev and MySQL on the production server? Then the generated migration would be optimized for MariaDB, but "fail" on the Prod server.

Hi,
Indeed, I hadn't really considered that scenario (and I guess, the inverse - a developer has MySQL on their PC but there's MariaDB in production). Ok, let's ditch mariadb_new_default_schema then, here's another idea:
New option, treat_defaults_as_expr, with 3 levels:
0: Generated code as it is now
1: Generated code assumes all defaults are expressions, and doesn't quote them
2: Dynamically query the engine in order to use expr or no expr. The 'default' options in this case would not be a literal, but a function call to a code-generated default() function that yields the right default based on the schema
How about that? I didn't want to add the dynamic query always since it's some overhead and there's a chance it may not be 100% guaranteed to work (e.g if the user altered the version variable - which I'm not sure right now is possible).

Due to time reasons I cannot offer support for MariaDB. For this reason I have to close the ticket. Thank you very much.


BalenaSound (balenaOS 2.48.0+rev1) not working after installing JustBoom Digi HAT on Raspberry Pi 4 #473

Hi,
I have been enjoying my BalendaSound with my new Raspberry Pi 4. But when I installed the JustBoom Digi HAT and edited the configuration it suddenly stopped working. The console is outputting some errors, but I really don’t know how to debug them…
Any help would be appreciated thanks!

HOST OS VERSION
balenaOS 2.48.0+rev1
SUPERVISOR VERSION
10.8.0
CURRENT RELEASE
f49e2692bbf334167693b8d607d7a504

@jonrrivera you used the wrong dtoverlay. Please change justboom-dac to justboom-digi and report me the result.

that was it! thank you so much!


Fail to bind openldap

It seems the ldap interface is for AD , I try to config an openldap Authorization Source but fail .
How and  help to give an openldap config example , thanks .

Hmm... did you solve the problem?

yes , it's ok now , thanks .

Cool!


App gets closed when Remote JS Debugging is enabled

Description
When the JS Debugging is started from the Dev menu, the app on the mobile gets closed on its own after a while. It does not happen immediately, but after a minute or two, sometimes after 20-30 seconds.
It only happens if there is a debugger actually attached. If the option is enabled, but the debugger is closed immediately, the app runs as it should. E.g. if the Chrome window localhost:8081/debugger-ui is closed, or whatever other debugger is used.
It doesn't sound like a feature to me, so I suppose it is a bug somewhere. Maybe it is not in react-native itself, I'm not sure. But it definitely makes the logging and debugging much harder.
This happens on an Android 6.0 device connected by USB, not on an emulator.
Reproduction

Create a new project with "react-native init ..."
Run it with "react-native run-android"
Open the Dev menu on the device and start the remote JS debugging.
This will open Chrome to the debugger-ui page (or some other debugger). Don't close the page and wait for a few minutes. The app will close on the mobile on its own.

Additional Information

React Native version: 0.42
Platform: Android
Development Operating System: Windows 7
Dev tools: adb is v1.0.36, Android SDK platform-tools 25.0.3

There's already an issue for this: #9665

Thanks. It is exactly the same behaviour. I read the comments, but unfortunately after 7 months the issue is still there and it is a major one.

kill $(lsof -t -i:8081) works for me.


Bank transfer as payment method

Hello @sandervanhooft and others, quick question: is it possible to use Mollie's bank transfer payment method with the package? And if so, how does it work when the subscription is to be renewed? I see dueDate and billingEmail properties in Mollie\Api\Resources\Payment, but not much else...
(It doesn't seem possible to test the bank transfer method in Mollie's dev mode.)

Hi @pecuchet
Not sure what you mean but let me try to clear it up.
This package works by obtaining a payment mandate through Mollie's checkout and then schedules/triggers mandated payments when these are due.
Only payment methods eligible for "first payments" can be used in that checkout to obtain the mandate.
Except for creditcard and PayPal, all other eligible payment methods will result in a bank transfer mandate.
Alternatively, you could create that mandate manually using the Mollie API by providing some details such as the SEPA reference. If you store that mandate id along with the customer id on the billable model, this package can handle the remainder of the subscription logic.

Hi @sandervanhooft,
Indeed I meant a manual bank transfer, like SEPA, and wondered if it is eligible as a first payment method. If I understand your last paragraph correctly, it's not out of the box with your package. So if I want to support it, I have to implement a custom first payment flow for that method?

You can collect the data using your own form and create the mandate through Mollie's API, without the customer having to go through Mollie's checkout.
Something like:
$mandate = mollie()->mandates->create([
    'customerId' => $billable->mollieCustomerId(),
    // rest of payload
]);

$billable->mollie_mandate_id = $mandate->id;
$billable->save();

See https://docs.mollie.com/reference/v2/mandates-api/create-mandate for the actual payload contents and more.

Ok, thanks @sandervanhooft !

No problem, enjoy!


Is it possible for y axis in reversed order?

Instead of

can I get something like this?

I would love this as well! Happy to contribute but don't know the codebase very well.

@yuan9778 Have you find a solution to reverse Y axis value of graph?


Set Matrix Zeroes in Java

Describe the solution you'd like
Given a 2D matrix. If an element in the matrix is zero and put all the elements in that row and column to zeroes.

Describe the problem and solution (at each steps)
write the complexity

Hello,
I would like to work on this issue. Please assign it to me.

Hello,
I would like to work on this issue. Please assign it to me.

Go for it 👍

Hey can i work on this ?


Youtube Enhancer doesn't have the skip ad anymore

They had to remove it per Google pressure.
Do you have plans to work around it, or do you have another solution to not have ads on youtube ?

Working around it is not realistic since the feature simply does not exist anymore. Technically you could use an older version of YT Enhacer and use it till that feature can no longer function. As far as other plans are concerned, there are none. The simplest solution to eliminate YouTube ads is by using uBlock Origin which does not seem to be detected.


feat(front): update ticket hash

URL to Jira task (this should explain the task)
https://hackachain.atlassian.net/browse/HYC-207?atlOrigin=eyJpIjoiNzY0MmRhMWVhZjFlNDQ0OTliNTg4ODAxMGI0YjVlZTAiLCJwIjoiaiJ9
Explain what the code alterations do
Updated note to use hex
What has to be tested? How to test it?
Test deposit and withdraw with new note
Any new dependencies? Why were they added?
no

/deploy

/deploy

/deploy

/deploy

/deploy

/deploy


Using setRequestHeader in ajaxSettings

I'm trying to set an x-authorization header for an easyAutocomplete but it isn't being set. I can't figure out why not so I wonder if someone might be able to help me out.
The request is as follows:
$options = {
	url: function(phrase){
		return $app.domain + '/' + 'lang/airport/' + phrase + '/format';
	},
	getValue: "Text",
	ajaxSettings: {
		dataType: "jsonp",
		beforeSend: function(xhr) {
			xhr.setRequestHeader("x-authorization", $app.license);
		},
	},
   	requestDelay: 300,
}

$(this).easyAutocomplete($options);

How do I add the x-authorization header?

I realised that I needed to set the method to GET for this to work
ajaxSettings: {
		dataType: "jsonp",
		beforeSend: function(xhr) {
			xhr.setRequestHeader("x-authorization", $app.license);
		},
                 method: 'GET'
	},


Major overhaul

Fixes Sigil II support.
Improves Fredoom support as individual episodes.
Adds Fredoom1 music.
Adds Freedoom intermission graphic.
Other minor fixes.

Drops Windows support, will need to be added back in if desired.
I don't have/use Windows so I dropped it as I can't do anything with it.

Minor fix needed


Makefile to Compile the files

Do you have a Makefile to compiled all the files as a single archive/ static library? Thanks

Hi @heshananupama,
You request has been forwarded to our development teams. I will be back to you as soon as they provide me with their feedback.
With regards,

Hi @heshananupama,
I hope you are fine. Regarding your request of a Makefile, the Cube IDE already allows creating one for your project. Please follow the instructions detailed in this tutorial.
I hope this answers your question. Please do not hesitate if you still have something to ask.
With regards,

Hi @heshananupama,
I hope you found the tutorial I referred to in my previous comment helpful. Please allow me to close this issue now.
With regards,


plot/examples: Add more examples for plot

Right now vsl.plot supports the following graph types: Bar, Heatmap, Pie, Scatter, Scatter 3D, Surface. We need to add more examples for them!
We are using Python/Plotly as backend so you can check at their doc to know more about supported graphs and look for inspiration there!
Feel free to submit any amount of PRs for this issue!

Hi I would like to work on it

Great! Feel free to create any PR for it and ask any question you have about it!

Hello Ulises.
why don't response recently?
I am waiting your request.
please send me message in telegram. @ulisesjcf


feat: Configurable link format

TODO:

[x] Resolve more kinds of links using: document title, file path, file name, file stem (path wo/ ext).
[x] Refactor the completion code to allow the configurability (#120)
[x] Heading completion should resolve against doc using the logic above
[x] Add a config option for wiki-links doc completion
[x] Rename refactor should honor the kind of a file link (e.g. rename title shouldn't change a path-like link)
[x] Docs

Yeah, it's all a bit fuzzy. The reasoning for the current implementation is that if you stick to one particular style in your document folder and configure the completion accordingly, there won't be ambiguity in links.
I'm currently working on the completion which is a bit involved. Let's see how it plays out.

Thanks for leaving this open to get feedback.
Things that affect me:

[[Foo/Bar]] should resolve to ./Foo/Bar.md, at least. This works in emanote/obsidian
[[./Bar]] should not match ./Foo/Bar.md (although [[Bar]] may). This works in emanote + obsidian (basically emanote follows obsidian so I think they agree for most things)

A possible axis for config would be whether to search subdirs for "stem"-based links without an explicit ./, ../, or / prefix. Since most of the other tools are permissive here tho, I would tend to leave that option out.
Also, I think this could be a future feature, but  [[Title|Target]] (gh wiki) vs [[Target|Title]] (obsidian etc) (see https://hackage.haskell.org/package/commonmark-extensions-<IP_ADDRESS>/docs/Commonmark-Extensions-Wikilinks.html) if/when marksman supports pipes in wiki links (I don't use these much but I think some people do).
I'd really like to put together a set of shared test cases and engage various tool authors to try to come to some consensus about the way wiki links should work. I had been plotting a lint/refactor tool to go along with it but honestly marksman seems like a good place for that to live (since, you know, it already does that), with .marksman.toml acting as a repo-wide policy description for linting.

I do notice that marksman makes my CPU spin for longer than I would think is needed (~45s on one core for my moderate size wiki with ~1400 fairly densely linked md files) but I hadn't used marksman much prior to this branch so I don't have a baseline to say it this is a regression. Are there any performance/stress test cases that could be used to detect a performance regression?

@edrex perf tests is something I'd like to setup but haven't gotten to yet as it's a bit fiddly in the context of an LSP server.
I'm curious though, at what point do you see the CPU to spin? Is it during initialisation? Or when processing keystrokes? Or something else?

I guess I wouldn't worry as much about perf as getting the outputs right. Perf can come later IMO, even if it's a big regression.

Perf results: In a few days testing I found marksman turns 1 core into a heater for ~10s whenever a buffer in my moderate sized wiki

is opened
is changed

Switching buffers and moving around in the file, as expected, don't trigger the costly operation.
I haven't tested with your changes from today.
Are this and the completion branch ready to start testing together?

Thanks for testing @edrex! Calculating diagnostics might be the main factor here - it's not very efficient really. This would explain the CPU load on buffer change. But not when the buffer is open.
Diagnostics calc is done asynchronously outside of the main thread though, so I'd expect the server to be responsive and serving quickly other request like completion, symbols, etc. Does this match your experience @edrex?

Testing with alternate wiki.style values.

file?(-path)-stem styles preserve stem case, which I very much appreciate.
It would be cool to have a style which uses minimal paths. So from foo/bar.md

For sibling docs, just use the stem: foo/baz.md -> baz (or ./baz?  not sure which is more broadly supported - test matrix soon)
For descendants of the current document directory, use relative path, eg foo/ban/qux.md -> ban/qux (or ./ban/qux?)
Otherwise use path from root, like current file-path-stem style, eg quux/waldo.md -> /quux/waldo`
I think this would satisfy the needs of folks who want exactly resolved relative paths, but I can't guarantee that all the tools out there will resolve these paths correctly

Coming together nicely!

This would explain.. But not when the buffer is open.
Oh, I meant "opened" as an event rather than a state. So when I first open a buffer.

Diagnostics calc is done asynchronously outside of the main thread though, so I'd expect the server to be responsive and serving quickly other request like completion, symbols, etc. Does this match your experience @edrex?

Yeah, it does. I don't notice any delay in completion etc when CPU is high.

With the file-(path-)?stem styles, I notice that completion doesn't match the typed text against  document h1 titles (although those titles are shown in the result list entry). IMO, matching should happen against all possible fields (title, stem, path) and the setting should only affect what the search text is replaced with when a completion entry is selected.

It would be cool to have a style which uses minimal paths.

A minimal path wiki link style could be done in a follow-up PR, even as a "good first contrib" issue.

IMO, matching should happen against all possible fields (title, stem, path)

This is how candidate search works now, yes (or at least is supposed to work 🙂).

... and the setting should only affect what the search text is replaced with when a completion entry is selected.

Yes, this is idea. But there's a couple layers of extra filtering involved:

Through LSP's filterText attribute,
Through editor's own heuristics.

My first guess would be that it's filterText that is incorrect in some cases. @edrex could you give an example of a completion that doesn't work as you'd expect?

I'm late to this thread, and haven't read all the comments above: but just wanted to state that it is generally a good idea to support the wiki-link convention that Obsidian itself supports. We don't have a clear standard on these things, but Obsidian seems to be the leader here, as far as future-proof Markdown notes goes.
Interesting extension, by the way. I might switch over from vscode-memo to this one in https://github.com/srid/emanote-template once it supports Obsidian/Emanote-style wikilinks.

@edrex I think the problem with missing completions was due to wrong 'filter text' of a completion; this has been fixed in https://github.com/artempyanykh/marksman/pull/117/commits/8a9a5c33567cad7a1aa048f924d4458e891d6bb8.
I think the basic functionality should work now (but do give it a spin and report bugs :) ). There are several more things around file link resolution and completion that I want to improve, but this will likely go into separate PRs. @srid thanks for chiming in, by the way! "substring search in path hierarchy" makes sense.

"sub-string match in path hierarchy".

@srid that's a helpful explaination. I feel like I understand the motivation a lot better now. Thanks! (meant to respond much earlier, echoing artempyanykh's reply). Only the tail of the path should match tho right?
Installed the release in my profile, and will be testing.

Yes, it is similar to the tails function (but without the empty list).
Here's emanote's implementation that returns the candidates:
https://github.com/srid/commonmark-wikilink/blob/72c880b5dbe8521f8c424c7bd39cb11b26e8dbb8/src/Commonmark/Extensions/WikiLink.hs#L151-L164


move List import to generated Handler

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! :smile:


Added cegeptr

Just added a new email address to the list.

Here are some details about this college.
Website: http://www.cegeptr.qc.ca/

If you would like to get more information on the program that is doing web development:
http://www.cegeptr.qc.ca/programmes/informatique/
However the page is in French.

Merged, thank you and sorry for delay!


Add missing wait_for_nodes param in Cluster health Reference REST API

Description
Document missing wait_for_nodes param in Cluster health Reference REST API
Additional improvements:

The master_timeout is marked as deprecated and cluster_manager_timeout is suggested instead
A required permissions is also documented
The JSON example response is completed

Issues Resolved
Closes: #552
Check List

[x] Commits are signed per the DCO using --signoff

Signed-off-by: Lukáš Vlček<EMAIL_ADDRESS>By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

I went ahead and took over this PR. See #599


why my extensions atre not downloading on vs code

Type: Bug
entensions are not installing good network but also extensions are not downloading
VS Code version: Code 1.89.1 (dc96b837cf6bb4af9cd736aa3af08cf8279f7685, 2024-05-07T05:13:33.891Z)
OS version: Windows_NT x64 10.0.22631
Modes:

Hi @SmiSow ,
Thank you for reaching out to the VS Marketplace Team. We are unable to reproduce the issue. Are you still facing difficulties installing this Python extension?
Regards,
VS Marketplace Team

Since we have not received the required information, we are closing the issue. Please feel free to reopen or create a new issue if the problem still persists.
Regards,
VS Marketplace Team


Regular Angular (not js)

Hi, Is this compatible for regular Angular 11?

Hi @Avramo!
Unfortunately not, this was built for AngularJS.
Cheers, Pablo.

Thanks!


Add and expose auth types: MultiAuth ApiToken*

This adds more auth classes designed to be used when initializing a
Consumer subclass. There are two generic AuthToken types that require
a subclass to define the relevant query parameter or header. There is
also a MultiAuth type which allows for passing in a list of auth
methods, so that they all take effect.
The purpose of the generic AuthToken types is to make it simple for
uplink-based libraries to provide a standardized way to authenticate
with their api.
The purpose of MultiAuth is to allow for multiple auth methods. This may
be required when ProxyAuth needs to be combined with another auth
method, or when an API requires multiple auth tokens.
Attention: @prkumar

I don't think very many (if any) APIs require multiple forms of auth on the server side.
I'm designing an API that might require multiple tokens on the server side, not just for intermediary auth, but it's a very specific use-case peculiar to our internal infrastructure. I don't know of any generally available APIs that require multiple forms of auth on the server side.
However, many clients require multiple forms of auth to, for example, authenticate to an HTTP proxy and to the API server.

@prkumar done including tests (codecov @ 100%).

[x] expanded the MultiAuth to make it act more like a list, allowing for building it in whatever way makes sense to the API library consumer.
[x] expanded the generic ApiTokenParam and ApiTokenHeader classes to allow using them directly ApiTokenParam(param="api_token", token="****") as well as by subclassing with class attributes. (See my comments and commit messages above).
[x] refactored BasicAuth and ProxyAuth to reuse the ApiTokenHeader.
[x] added/adjusted tests for all of the above.

What do you think? I can revert the changes to BasicAuth and ProxyAuth if you'd like (changing that made it more consistent in internal implementation details, but is otherwise unnecessary).

@cognifloyd - This looks great! Thanks so much for adding coverage and for extending the ApiTokenParam and ApiTokenHeader classes to be used directly, too. I'll go ahead approve the changes and merge this in ASAP.
After merging, I'll also make a separate commit to add you to the AUTHORS.rst file. Please let me know what name and/or public handle (e.g., @cognifloyd) that you'd like me to append to the file .Typically, I'd use the name and handle of your GitHub account, but I can use a different account/name if you'd prefer. For example, using your GitHub, here's what I'd add to AUTHORS.rst:
Jacob Floyd (@cognifloyd)

Alternatively, you could make a separate commit to this PR with that change. Whatever works best for you! Thanks for contributing!

I added myself to AUTHORS.rst

Squashed and rebased on master.


Autoloader: Recursive autoloading on submodels

Implemented recursive autoload support
organizations = Organization.includes(users: :roles)

has_one autoload support
Query#first and #last autoload support

Specs passed locally


fix: Loosen package.json parsing to avoid failures.

package.json files can be written in far too man ways, so this removes all the fields we don't care about.

Awesome. This looks good and thanks for tackling it so quickly.


Update to 1.2.1

Checklist

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

@conda-forge-admin, please rerender

@cburing Can you take a look at this pull request? It fixes the issues with https://github.com/conda-forge/pyaml-env-feedstock/pull/5 and https://github.com/conda-forge/pyaml-env-feedstock/pull/6

@conda-forge/core Can one of you merge this pull request? The maintainer seems to be inactive.


Smoke test for Docfx v3

Docs Build status updates of commit ed95de4:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/SmokeTest.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Finding template placeholder

Description
Small enhancement of Finding template allowing to keep "original" content of finding with a placeholder.
Matching based on Title of findings.

Using {{original}} placeholder allows to keep original outputs of the finding and can be include inside the template.
eg: Templating output of a tool automatically and keep including tool output in the finding.

Test results
Ran "Finding" related tests.
Documentation
N/A
Checklist
This checklist is for your information.

[x] Make sure to rebase your PR against the very latest dev.
[x] Features/Changes should be submitted against the dev.
[ ] Bugfixes should be submitted against the bugfix branch.
[x] Give a meaningful name to your PR, as it may end up being used in the release notes.
[x] Your code is flake8 compliant.
[x] Your code is python 3.11 compliant.
[ ] If this is a new feature and not a bug fix, you've included the proper documentation in the docs at https://github.com/DefectDojo/django-DefectDojo/tree/dev/docs as part of this PR.
[x] Model changes must include the necessary migrations in the dojo/db_migrations folder.
[ ] Add applicable tests to the unit tests.
[x] Add the proper label to categorize your PR.

Do you need any further info on that ?

@Siryu6 There's a bunch of failing tests that need to be addressed before this can be reviewed further

Hi @mtesauro !
Thanks for the checks it appears that I missed that 😅

Do you mind adding some unittest, please?

Not sure what I failed here as we just added some tests :')

Hi @Siryu6,
regarding

python: can't open file '/home/runner/work/django-DefectDojo/django-DefectDojo/.github/scripts/git_protect.py': [Errno 2] No such file or directory

it is not connected to your work, you can ignore it for now. Guys are trying to fix it by #8953 and #8954
Regarding

Do you mind adding some unittest, please?

I see you added nice integration tests (dir tests/). But can you add unit tests (dir unittests/) as well?
You can test template_replace_placeholder with different values (empty fields, placeholder {{original}}, fields with {{original}} or {{{{original}}}}), how does it behave based on different content of findings_to_replace and each of the impacted fields (empty, one ;, two ;, three ;, four ;, )?
And do you mind describing this behaviour in documentation or field description as well? E.g. I see it impacts fields severity, cvssv3, description, mitigation, impact and references but it is not obvious until you do not check the code.
Thank you

Hi @kiblik
Thanks for your answer ! I'll do some doc about that !
Quick question, would you mind check that unittest (https://github.com/Siryu6/django-DefectDojo/blob/unit-tests/unittests/test_apply_finding_template_placeholder.py) ? I'm kinda not used to Django testing we didn't pushed unittest cause I am struggling on our test assertion 😅

First, I highly recommend checking https://github.com/DefectDojo/django-DefectDojo/blob/master/readme-docs/DOCKER.md#running-the-unit-tests (just how to start tests - you do not need to run all of them, only yours ./dc-unittest.sh --profile ... --test-case unittests.test_apply_finding_template_placeholder)
I see, you found inspiration in test_apply_finding_template.py but this specific one is not the best (might be written much better - simplified and access via API for basic tests is not necessary).
Find inspiration in

test_cleanup_alerts.py - for use-case "what will happen when I call apply_cwe_mitigation"
test_api_sonarqube_updater.py- for use-case "if I call template_replace_placeholder with specific parameters, what will be the result?"

Closed as stale


Add ande-ronayt to Contributors list

Thank you for allowing me to do my first pull request :)

@ande-ronayt, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?


fix: fix keymanager api for prysm

Summary
This PR fixes the keymanager api feature for prysm which was not working. In detail it:

delete the unknown http flags and replaces them with the grpc-gateway flags since all REST APIs are exposed as gateway to the general grpc protocol
deletes the unused/unnecessary grpc port for the public port exposure

Lets wait for this PR to be merged once this is no longer a develop only change.

Changing the default prysm img to point at latest stable instead of latest develop: https://github.com/ethpandaops/ethereum-package/pull/793

Actually, it looks like prysm stable branch is still using grpc-gateway, and latest develop uses http.
You might have had some older version of the prysm image locally.

is this pr still needed?
Prysm made a new stable release that does http instead of grocery.

Please keep it open for now. I just need it as reference because I think there is still a bug in the prysm config (not related to grpc). I will test in the next days and will close it once I did all testing.

Closed as described here.


https://github.com/MicrosoftLearning/PL-400_Microsoft-Power-Platform-Developer/blob/d48e1ffb49c7c31a93d8c7e07ce51065018e80dc/Instructions/Labs/LAB[PL-400]_Lab07_APIs.md

Module: 00
Lab/Demo: 00
Task: 00
Step: 00
Description of issue
Repro steps:

Closing as no information provided


s3: optimize info/exists calls

Described in https://github.com/dask/s3fs/issues/470#issuecomment-829097599

Please ping me to have a thorough review of this.
First a couple of thoughts:

loosing version information could be a dealbraker for some, so we would only want to do this when the filesystem is not marked as version-aware
for the case where the user does not have list permission, we would still want to fall back to HEAD (but not bother trying to list for the directory)

This might be a good place to include a test against #473 and fix, if necessary.

That seems a very different bug, though since it is trivial a fix is included in this PR.

There is a separate permission for listing files with a prefix versus listing with a prefix that happens to end with "/"?

I think we hit that before, though @efiop probably has more context than me. It is not a specific permission per se, but rather a bucket-level rule declaration that allows you to list /bucket/something/* but not /bucket/somethingXXX etc.

Correct, here is a ticket for it https://github.com/iterative/dvc/issues/5130

Can you please also add the test shown in #475 here? I suspect the changes in this PR will have fixed that.

If I am not missing anything, I already did https://github.com/dask/s3fs/pull/472/commits/0eb3620b788775c1a4badec6f2861dad5e53a748

Specifically the test
@pytest.mark.parametrize("find", [False, True])
def test_lsdir(s3, find):
    if find:
        s3.find(test_bucket_name)

    d = test_bucket_name + "/test"
    assert d in s3.ls(test_bucket_name)

This might be a good place to include a test against #473 and fix, if necessary.
Can you please also add the test shown in #475 here? I suspect the changes in this PR will have fixed that.

wait you are talking about different issues?

this PR doesn't interact with ls/find in any way, so I don't think there is any reason to include a fix for #475.

OK, that will have to be dealt with separately.

Okay, I'll add a few comments

(I'll merge later - going out prospecting for a vaccine)


Prepare the BOM for completing the second test equipment

Since we currently have a partial test equipment ready, we want to check which components are missing and prepare the BOM of the them to have a second complete test equipment. In this way it is easier to support the testers on their work.
cc: @valegagge

Please @MSECode add a purchase issue as @Fabrizio69 suggested.

BOM for completing the 2FOC test equipment 2:

Wingst Number
Quantity
Description
Notes

4570_B
1
2FOC_ADC_CABLE, IIT - Cable set for testing equipment for 2FOC boards
Items from 13069 to 4584_B are part of this set. Add just needed and what we are not sure about if we have or it is actually still needed. Good to make a check about that.

13069
1
STRIP6toAMP8, IIT - Programming cable, PICkit™ 3 programmer to AMP8 L=250mm
Present but should be the one for PICkitv4. Apparently is the same. Maybe just check and update the procedure. It comes with the programmer anyway

4577
1
CAN6toDB9, IIT - CAN cable, Molex 6 pins to DB9 w/o power-supply, w/o termination, L=250mm
not sure if we have the correct one since it is without the power supply the one we have. to be checked

4578
1
2xCAN6toDB9, IIT - Double CAN cable, 2x Molex 6 pins to DB9 w/o power-supply, w/o termination, L=250mm
not sure why we need this cable also but it is needed. missing

4579_A
1
TESTIO, IIT - cable for 2FOC IO test, 2x Molex 7 pins to switch and leds, L=150mm
No idea which cable are we talking about. Custom cables with no images and/or documents attached. Need help in identification

4579_B
1
TESTIO, IIT - cable for 2FOC IO test, 2x Molex 7 pins to switch and leds, L=150mm
No idea which cable are we talking about. Custom cables with no images and/or documents attached. Need help in identification

4580
2
TESTLOAD, IIT - Cable for 2FOC load test, 3x star connected resistor, L=250mm
No idea which cable are we talking about. Custom cables with no images and/or documents attached. Need help in identification

4581
1
BANANAStoWIRECAP, IIT - Generic Power supply cable with capacitor, L=500mm
I think it is not needed. We just need the 4255. The capacitor is on the board

4583
1
PICO7toPICO7, IIT - Cable for 2FOC IO test, Molex 7 pins to Molex 7 pins, L=150mm
No idea which cable are we talking about. Custom cables with no images and/or documents attached. Need help in identification

4584_A
1
PICO6toPICO6, IIT - Cable for 2FOC IO test, Molex 6 pins to Molex 6 pins, L=100mm
Not sure if already present. Need help in identification

4584_B
1
PICO6toPICO6, IIT - Cable for 2FOC IO test, Molex 6 pins to Molex 6 pins, L=100mm
Not sure if already present. Need help in identification. Why we need also this one. Why different revision with respect to the previous. Why not using 2 of rev B

1437
5
Jumper with extractor, 2 contacts, 1 rows, 2,54mm, Straight, 3A, gold, Red
Needed

3336_F
1
2FOC, IIT - Electronic board with CAN interface, dual driver for brushless motors, power supply 18-48V, 20A full Band width, for iCub, new heatsink without terminal blocks and MOSFETs
Needed. I think we have a very old one

3336_K
1
2FOC, IIT - Electronic board with CAN interface, dual driver for brushless motors, power supply 18-48V, 20A full Band width, for iCub, TMCS1100A2 and 31A overcurrent
I need to check the revision

4573_A
1
2FOC_ADC, IIT - Electronic board, Testing equipment for 2FOC boards
I cannot see the BOM of this component to get the code of the needles since for this component we just need to clean it and change needles in JJ19, 20, 22, 26 (which are broken ) and check the others

cc: @Fabrizio69


Feature request: Selective re-generation by area

I'm having trouble finding a workflow to use when a part of a large dataset changes, with changes confined to one specific area. For instance, I add a couple of data sources to opentrees.org in the Netherlands. Currently, my only option is to re-generate the entire tileset (~40 minutes and growing).
I tried generating tilesets for different regions then merging them together with tile-join but that doesn't really work if the regions are close together - per-tile limits quickly get exceeded, which tile-join resolves by keeping neither tile.
Would it be possible to perhaps have an option to selectively update an existing tileset, within an area specified in either lat/long or tile coordinates?

This would be great, but unfortunately I think it is too ambitious for the small amount of attention I can still give to Tippecanoe.


Fixed customStyle + propogates down to Day component

I'm relatively new to the world of react-native but it looks like the customStyle functionality was broken. It relied on _.merge() with the object and component defined styles which was created using StyleSheet.create(). Unfortunately StyleSheet.create() doesn't output doesn't output objects but instead internally stores the styles and outputs an object of references in the form of integers.
For example StyleSheet.create({ foo: { fontSize: 12 }, bar: { fontSize: 14 } }); would then just result in { foo: 1, bar: 2} which when merged with another object doesn't result in what we want.
I'm guessing StyleSheet.create() used to behave differently and this used to work.
Anyway, it's fixed now and in addition to that the customStyle is passed down to the Day component for further styling opportunities.
With these changes I was to style this:

Yay! 👏🏼👏🏼👏🏼

Hmm, I'm using custom styles in my app and it's working fine. Are you talking about defining style names that aren't already defined in react-native-calendar?

Hmm, nope doesn't work for me with react-native 0.10.1.
This way also allows us to use StyleSheet.create() on our customStyle rather than it having to be a straight object. This is better for performance from what I understand.

Ok, I see what's happening now.
_.merge(styles, this.props.customStyle) results in existing styles to be completely overwritten, as opposed to customStyle being complimentary to the existing styles (and overriding where necessary).
For example, existing CalendarIOS.js defines styles as
var styles = StyleSheet.create({
  title: {
    flex: 1,
    textAlign: 'center',
    fontSize: 15,
  },
  day: {
    fontSize: 16,
    alignSelf: 'center',
  },
  ...
});

This results in styles being { title: 123, day: 234, ... } (123 and 234 being some numbers, StyleSheet internally increments this number).
So now when I pass in customStyle
{
  title: {
    fontSize: 24,
    fontWeight: 'bold',
  }
}

and it gets merged via _.merge(styles, this.props.customStyle) the result is
{
  title: {
    fontSize: 24,
    fontWeight: 'bold',
  },
  day: 234,
  ...
}

So the customStyle.title passed in completely hijacks the style as opposed to just overriding fontSize and adding fontWeight. It's not a huge deal in this particular case but in the case of say dayButtonFiller style, we would lose the calculated width (based on DEVICE_WIDTH) and other properties.
The PR basically sidesteps all this and applies styles in cascading order the way they are meant to in react-native. customStyle passed in override existing style properties but leave the ones defined by the CalendarIOS component itself. In addition to this, this PR allows for the Day component to be styled which was not previously possible.
Does this make sense?

Yes, that makes sense. It was known that the styles are completely overwritten. It's definitely better to just add styles where needed instead of having to completely redefine the style.
So this PR would be good for me. :+1:
@christopherdro what do you think?

I agree, looks pretty good. I just need to make sure we have this working with the most recent stable release.

Do we need to make any changes to the docs after this PR is merged?

No, works as it does before (but better).

:+1:


⚠ Fortnite has degraded performance

In f94dc34, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1653 ms

Resolved: Fortnite performance has improved in da5bf6c.


Update version to 1.0.0!!!

Description
🎊 🎊 🎊 Version 1.0.0! 🎊 🎊 🎊
Checklist

[x] I have read the contributing guide and understand what's expected of me
[x] I have signed the CLA
[x] I have run this code in development and it appears to resolve the stated issue
[x] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[x] I have run changie new to create a changelog entry

Codecov Report
Attention: Patch coverage is 0% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 55.53%. Comparing base (a4caf20) to head (1449a3b).

Files
Patch %
Lines

dbt_common/__about__.py
0.00%
1 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #102   +/-   ##
=======================================
  Coverage   55.53%   55.53%
=======================================
  Files          50       50
  Lines        3065     3065
=======================================
  Hits         1702     1702
  Misses       1363     1363

Flag
Coverage Δ

unit
55.53% <0.00%> (ø)

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

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


Compiling for MacOS arm64 targets

Until now, the project did not compile on my MacOS M1 laptop.
It worked perfectly fine on my old x86-64 Linux laptop, but I wanted to try to make it compile properly on my current laptop as well.
After some trial and error, I've figured out that the proper way to do this, seems to be to pass the option CMAKE_OSX_ARCHITECTURES=arm64 to cmake.
What I don't know, however, is what the best way to set this setting properly on the actual device the Makefile ends up executing on.
Any ideas?

Hi @Qqwy, it works fine on my M2 MBP. I wonder if you have any error logs for that?

This is the log when running `mix clean && mix compile` on a037709fc969e99b52399ed0e61468ac8955792b.
➜   mix clean && mix compile
==> elixir_make
Compiling 6 files (.ex)
Generated elixir_make app
==> cc_precompiler
Compiling 3 files (.ex)
Generated cc_precompiler app
==> adbc
Downloading precompiled NIF to /Users/qqwy/Library/Caches/adbc-nif-2.16-aarch64-apple-darwin-0.1.0.tar.gz
Error happened while installing adbc from precompiled binary: "precompiled \"adbc-nif-2.16-aarch64-apple-darwin-0.1.0.tar.gz\" does not exist or cannot download: :enoent".

Attempting to compile adbc from source...
CMake Warning:
  Ignoring empty string ("") provided on the command line.

Not searching for unused variables given on the command line.

-- The C compiler identification is AppleClang <IP_ADDRESS>30022
-- The CXX compiler identification is AppleClang <IP_ADDRESS>30022
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /Library/Developer/CommandLineTools/usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /Library/Developer/CommandLineTools/usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Performing Test CXX_LINKER_SUPPORTS_VERSION_SCRIPT
-- Performing Test CXX_LINKER_SUPPORTS_VERSION_SCRIPT - Failed
-- Found SQLite3: /Library/Developer/CommandLineTools/SDKs/MacOSX13.3.sdk/usr/include (found version "3.39.5")
-- ---------------------------------------------------------------------
-- Arrow version:                                 0.4.0
--
-- Build configuration summary:
--   Generator: Unix Makefiles
--   Build type: Release
--   Source directory: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/3rd_party/apache-arrow-adbc/c
--   Install prefix: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv
--
-- Compile and link options:
--
--   ADBC_CXXFLAGS="" [default=""]
--       Compiler flags to append when compiling ADBC C++ libraries
--   ADBC_GO_BUILD_TAGS="" [default=""]
--       Build tags to append when compiling ADBC Go libraries
--   ADBC_BUILD_STATIC=OFF [default=ON]
--       Build static libraries
--   ADBC_BUILD_SHARED=ON [default=ON]
--       Build shared libraries
--   ADBC_PACKAGE_KIND="" [default=""]
--       Arbitrary string that identifies the kind of package
--       (for informational purposes)
--   ADBC_GIT_ID=a037709fc969e99b52399ed0e61468ac8955792b [default=""]
--       The Arrow git commit id (if any)
--   ADBC_GIT_DESCRIPTION="" [default=""]
--       The Arrow git commit description (if any)
--   ADBC_NO_DEPRECATED_API=OFF [default=OFF]
--       Exclude deprecated APIs from build
--   ADBC_USE_CCACHE=ON [default=ON]
--       Use ccache when compiling (if available)
--   ADBC_USE_PRECOMPILED_HEADERS=OFF [default=OFF]
--       Use precompiled headers when compiling
--   ADBC_ARMV8_ARCH=armv8-a [default=armv8-a|armv8-a+crc+crypto]
--       Arm64 arch and extensions
--   ADBC_ALTIVEC=ON [default=ON]
--       Build with Altivec if compiler has support
--   ADBC_RPATH_ORIGIN=OFF [default=OFF]
--       Build Arrow libraries with RATH set to $ORIGIN
--   ADBC_INSTALL_NAME_RPATH=ON [default=ON]
--       Build Arrow libraries with install_name set to @rpath
--   ADBC_GGDB_DEBUG=ON [default=ON]
--       Pass -ggdb flag to debug builds
--
-- Test and benchmark options:
--
--   ADBC_BUILD_EXAMPLES=OFF [default=OFF]
--       Build the Arrow examples
--   ADBC_BUILD_TESTS=OFF [default=OFF]
--       Build the Arrow googletest unit tests
--   ADBC_BUILD_INTEGRATION=OFF [default=OFF]
--       Build the Arrow integration test executables
--   ADBC_BUILD_BENCHMARKS=OFF [default=OFF]
--       Build the Arrow micro benchmarks
--   ADBC_TEST_LINKAGE=shared [default=shared|static]
--       Linkage of Arrow libraries with unit tests executables.
--
-- Lint options:
--
--   ADBC_GENERATE_COVERAGE=OFF [default=OFF]
--       Build with C++ code coverage enabled
--
-- Checks options:
--
--   ADBC_TEST_MEMCHECK=OFF [default=OFF]
--       Run the test suite using valgrind --tool=memcheck
--   ADBC_USE_ASAN=OFF [default=OFF]
--       Enable Address Sanitizer checks
--   ADBC_USE_TSAN=OFF [default=OFF]
--       Enable Thread Sanitizer checks
--   ADBC_USE_UBSAN=OFF [default=OFF]
--       Enable Undefined Behavior sanitizer checks
--
-- Thirdparty toolchain options:
--
--   ADBC_DEPENDENCY_SOURCE=BUNDLED [default=AUTO|BUNDLED|SYSTEM|CONDA|VCPKG|BREW]
--       Method to use for acquiring arrow's build dependencies
--
-- Advanced developer options:
--
--
-- Project components options:
--
--   ADBC_DRIVER_FLIGHTSQL=OFF [default=OFF]
--       Build the Flight SQL driver
--   ADBC_DRIVER_MANAGER=ON [default=OFF]
--       Build the driver manager
--   ADBC_DRIVER_POSTGRESQL=OFF [default=OFF]
--       Build the PostgreSQL driver
--   ADBC_DRIVER_SQLITE=ON [default=OFF]
--       Build the SQLite driver
--   ADBC_DRIVER_SNOWFLAKE=OFF [default=OFF]
--       Build the Snowflake driver
-- Configuring done
-- Generating done
-- Build files have been written to: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/cmake_adbc
gmake[1]: Entering directory '/Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/cmake_adbc'
[ 33%] Building CXX object driver_manager/CMakeFiles/adbc_driver_manager_objlib.dir/adbc_driver_manager.cc.o
[ 33%] Building C object vendor/nanoarrow/CMakeFiles/nanoarrow.dir/nanoarrow.c.o
[ 33%] Building C object driver/common/CMakeFiles/adbc_driver_common.dir/utils.c.o
[ 44%] Linking C static library libadbc_driver_common.a
[ 44%] Built target adbc_driver_common
[ 55%] Linking C static library libnanoarrow.a
[ 55%] Built target nanoarrow
[ 77%] Building C object driver/sqlite/CMakeFiles/adbc_driver_sqlite_objlib.dir/statement_reader.c.o
[ 77%] Building C object driver/sqlite/CMakeFiles/adbc_driver_sqlite_objlib.dir/sqlite.c.o
[ 77%] Built target adbc_driver_manager_objlib
[ 88%] Linking CXX shared library libadbc_driver_manager.dylib
[ 88%] Built target adbc_driver_manager_shared
[ 88%] Built target adbc_driver_sqlite_objlib
[100%] Linking C shared library libadbc_driver_sqlite.dylib
[100%] Built target adbc_driver_sqlite_shared
Install the project...
-- Install configuration: "Release"
-- Installing: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/lib/libadbc_driver_manager.4.0.0.dylib
-- Installing: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/lib/libadbc_driver_manager.4.dylib
-- Installing: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/lib/libadbc_driver_manager.dylib
-- Installing: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/lib/cmake/AdbcDriverManager/AdbcDriverManagerConfig.cmake
-- Installing: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/lib/cmake/AdbcDriverManager/AdbcDriverManagerConfigVersion.cmake
-- Installing: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/lib/cmake/AdbcDriverManager/AdbcDriverManagerTargets.cmake
-- Installing: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/lib/cmake/AdbcDriverManager/AdbcDriverManagerTargets-release.cmake
-- Installing: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/lib/pkgconfig/adbc-driver-manager.pc
-- Installing: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/include/adbc.h
-- Installing: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/include/adbc_driver_manager.h
-- Installing: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/lib/libadbc_driver_sqlite.4.0.0.dylib
-- Installing: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/lib/libadbc_driver_sqlite.4.dylib
-- Installing: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/lib/libadbc_driver_sqlite.dylib
-- Installing: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/lib/cmake/AdbcDriverSQLite/AdbcDriverSQLiteConfig.cmake
-- Installing: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/lib/cmake/AdbcDriverSQLite/AdbcDriverSQLiteConfigVersion.cmake
-- Installing: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/lib/cmake/AdbcDriverSQLite/AdbcDriverSQLiteTargets.cmake
-- Installing: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/lib/cmake/AdbcDriverSQLite/AdbcDriverSQLiteTargets-release.cmake
-- Installing: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/lib/pkgconfig/adbc-driver-sqlite.pc
gmake[1]: Leaving directory '/Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/cmake_adbc'
CMake Warning:
  Ignoring empty string ("") provided on the command line.

Not searching for unused variables given on the command line.

-- The C compiler identification is AppleClang <IP_ADDRESS>30022
-- The CXX compiler identification is AppleClang <IP_ADDRESS>30022
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /Library/Developer/CommandLineTools/usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /Library/Developer/CommandLineTools/usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- Using PRIV_DIR: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv
-- Using ERTS_INCLUDE_DIR: /Users/qqwy/.asdf/installs/erlang/25.0/erts-13.0/include
-- CMAKE_TOOLCHAIN_FILE:
-- Configuring done
-- Generating done
-- Build files have been written to: /Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/cmake_adbc_nif
[ 33%] Building CXX object CMakeFiles/adbc_nif.dir/c_src/nif_utils.cpp.o
[ 66%] Building CXX object CMakeFiles/adbc_nif.dir/c_src/adbc_nif.cpp.o
[100%] Linking CXX shared library adbc_nif.so
[100%] Built target adbc_nif
Compiling 10 files (.ex)

12:30:13.271 [error] Process #PID<0.336.0> raised an exception
** (Protocol.UndefinedError) protocol String.Chars not implemented for {:load_failed, 'Failed to load NIF library: \'dlopen(/Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/adbc_nif.so, 0x0002): tried: \'/Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/adbc_nif.so\' (mach-o file, but is an incompatible architecture (have \'x86_64\', need \'arm64\')), \'/System/Volumes/Preboot/Cryptexes/OS/Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/adbc_nif.so\' (no such file), \'/Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/adbc_nif.so\' (mach-o file, but is an incompatible architecture (have \'x86_64\', need \'arm64\'))\''} of type Tuple
    (elixir 1.14.1) lib/string/chars.ex:3: String.Chars.impl_for!/1
    (elixir 1.14.1) lib/string/chars.ex:22: String.Chars.to_string/1
    lib/adbc_nif.ex:11: Adbc.Nif.load_nif/0
    (kernel 8.4) code_server.erl:1317: anonymous fn/1 in :code_server.handle_on_load/5

12:30:13.272 [warning] The on_load function for module Elixir.Adbc.Nif returned:
{%Protocol.UndefinedError{
   protocol: String.Chars,
   value: {:load_failed,
    'Failed to load NIF library: \'dlopen(/Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/adbc_nif.so, 0x0002): tried: \'/Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/adbc_nif.so\' (mach-o file, but is an incompatible architecture (have \'x86_64\', need \'arm64\')), \'/System/Volumes/Preboot/Cryptexes/OS/Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/adbc_nif.so\' (no such file), \'/Users/qqwy/Programming/Personal/elixir/rust_c/adbc/_build/dev/lib/adbc/priv/adbc_nif.so\' (mach-o file, but is an incompatible architecture (have \'x86_64\', need \'arm64\'))\''},
   description: ""
 },
 [
   {String.Chars, :impl_for!, 1, [file: 'lib/string/chars.ex', line: 3]},
   {String.Chars, :to_string, 1, [file: 'lib/string/chars.ex', line: 22]},
   {Adbc.Nif, :load_nif, 0, [file: 'lib/adbc_nif.ex', ...]},
   {:code_server, :"-handle_on_load/5-fun-0-", 1, [...]}
 ]}

Generated adbc app

Maybe there is some weird configuration setting missing on my machine that makes Make/CMake think it should compile for x86 instead?

@Qqwy I think it is something related to Rosetta? Either make or something else is compiling the wrong version. What happens if you set CMAKE_OSX_ARCHITECTURES=arm64 as an env var in your shell?

What happens if you set CMAKE_OSX_ARCHITECTURES=arm64 as an env var in your shell?

It does not change the output; it seems cmake does not read those -D settings (looking into its docs, they seem to be called 'cache entries') from the environment variables.

At the very least it's good to know that it works on your computers, because then it is more likely that it is a problem with my laptop rather than something that needs to be improved in the library 😅

Emmm, that seems to be weird. Could you please add one line in the CMakeLists.txt, below line 14?
message(STATUS "CMAKE_HOST_SYSTEM_PROCESSOR: ${CMAKE_HOST_SYSTEM_PROCESSOR}")

It should look like this
... skipped
-- Using PRIV_DIR: /Users/cocoa/Workspace/Git/adbc/_build/test/lib/adbc/priv
-- CMAKE_HOST_SYSTEM_PROCESSOR: arm64
...

which is weird, how did it end up in the non-Rosetta path?

Maybe the path to the Rosetta bin presents before the non-Rosetta one in $PATH? 🤔
Or it might be linked with --force option when installed by the x86_64 homebrew?


Changing primary and/or secondary color

Hi,
Thanks for the great skin!
When changing the primary and secondary colors in _variables.scss (or adding any color via a custom map-merge), these colors do NOT get a changed :root --bs-primary-rgb value (which should result in a changed .bg-primary class value (RGB).
What are the best steps I should take to make these changes?

@ropalko could you help here?

ok it works!
I added this underneath my mapmerged theme-colors part:
$theme-colors-rgb: map-loop($theme-colors, to-rgb, "$value");
The merge part in _variables.scss now looks like this:
$theme-colors: ();
$theme-colors: map-merge(
    (
        "primary":      $primary,
        "secondary":    $secondary,
        "danger":       #842029,
        "dark":         $gray-600
   ),
   $theme-colors
);
$theme-colors-rgb: map-loop($theme-colors, to-rgb, "$value");
Thanks


Update video issue

Pull Request Test Coverage Report for Build 3745

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 36.996%

Totals

Change from base Build 3744:
0.0%

Covered Lines:
2709

Relevant Lines:
6555

💛  - Coveralls


TrackingReader does a poor job reporting where a parse error occurred.

When a parsed value contains invalid data, an error is returned. However the location that will get reported to the user will be the location after the end of the last read instead of before it.

I got rid of TrackingReader because of this, and instead added more logging so we can still find out where an error occurred. Beyond that, I hope to use thiserror's backtrace functionality when it's finally available on stable Rust.


Bjorncs/container thread pool

I confirm that this contribution is made under the terms of the license found in the root directory of this repository's source tree and that I have the authority necessary to make this contribution on behalf of its copyright owner.

I also suggest that you run a system test or 2 first to verify.

Rebasing with master to resolve merge conflict with ModelContextImpl

@baldersheim PTAL. The two last commits are new since previous review.

Another merge conflict...

Rebased with master - no other changes.


Battery lifetime simulation with 3rd party ownership doesn't work

It appears that if you run a PV+battery with a 3rd party ownership model and lifetime mode, the simulation will run and return: "Replacements are enabled without running lifetime simulation, please run over lifetime to consider battery replacements".  Essentially, it seems as if the simulation is run in single-year mode regardless of whether lifetime is selected.

Pending pull request: https://github.com/NREL/ssc/pull/200


knife vsphere vm snapshot problem - can not find folders

Hi,
I am using knife vsphere plugin , and I trying to list all exist snapshots on particular machine without success.
The command I used:
knife vsphere vm snapshot LiorVM -f "/vm/Lior VM's/" --list -VV
The result:
INFO: Using configuration from /root/chef-repo/.chef/knife.rb
DEBUG: value for config item vsphere_host: lab-admin-vc1.domain.com
DEBUG: value for config item vsphere_path: /sdk
DEBUG: value for config item vsphere_port: 443
DEBUG: value for config item vsphere_nossl:
DEBUG: value for config item vsphere_user: lioryantov
DEBUG: value for config item vsphere_pass: ***********
DEBUG: value for config item vsphere_insecure: true
DEBUG: value for config item proxy_host:
DEBUG: value for config item proxy_port:
DEBUG: value for config item folder: /vm/Lior VM's/
DEBUG: value for config item vsphere_dc: Domain-Lab
no such folder /vm/Lior VM's/ while looking for vm
Although when I running command "knife vsphere folder list" I received correct directory topology.
Folder: vm
Folder: Lior VM's
What is the problem there, what I am doing wrong in my query ?
Many Thanks,
Lior

Hi,
knife-vsphere is not part of core chef. Please report your issue at https://github.com/chef-partners/knife-vsphere/issues/new and you'll be able to get some assistance there.