Improve 'subtree mostly changed' heuristics

See e.g. main.rs 4883edd90cc8011041a6cee0622805d6bc7847a0 hunk 5/11 or the res.append dot in e1ffa2af2a38917133a06c823fd0fc365f5eaa0d.

Old:
fn diff_file() {
    let rhs_binary = true;
    if rhs_binary {
        print!("{}", style::header(display_path, 1, 1, "binary"));
        return;
    }
    let extension = 1;
}

New:
fn diff_file() {
    let rhs_binary = true;
    if rhs_binary {
        return DiffResult {
            path: display_path.into(),
            language: None,
            binary: true,
            lhs_src: "".into(),
            lhs_positions: vec![],
            rhs_positions: vec![],
        };
    }
    let extension = 1;
}

Old:
fn diff_file() {
    let rhs_binary = true;
    if rhs_binary {
        print!("{}", style::header(display_path, 1, 1, "binary"));
        return;
    }
    let extension = 1;
}

New:
fn diff_file() {
    let rhs_binary = true;
    if rhs_binary {
        return DiffResult {
            path: display_path.into(),
            language: None,
            binary: true,
            lhs_src: "".into(),
            lhs_positions: vec![],
            rhs_positions: vec![],
        };
    }
    let extension = 1;
}

errors.mli in HHVM 71abf8d56763d497e95c1e79e13724e6e103c32d is another good example. It might benefit from never splitting a LHS top-level node over multiple RHS top-level nodes. IIRC Autochrome has some ideas in this space.

Giving top-level lists a bigger discount for being wholly novel might help matters.

Deadgrep 14c7d6b74c7891ed7294abe1a6f5914948e4ab49 has an interesting example where deadgrep--directory has its body factored out.

But deadgrep bdcdf138cd71b0a5a80ca64b3bd68b7355084757 is an example where defining a new function deadgrep--escape-backslash is considered as partially reused.


[Student View] Shift delete acad year button to last acad year row

More subtle than our current design

RESOLVED by commit 687b983


Provide a way to uninstall rubra.

Provide a way to uninstall rubra.
Currently I have to do the following steps to uninstall rubra.
curl -sfL https://get.rubra.ai/ | sh -s -- stop
 rm -r ~/.rubra
docker volume rm  rubra_mongodb rubra_milvus rubra_etcd

We need to support uninstall for cli and  dmg install.

https://github.com/rubra-ai/rubra/commit/5dc8c89f4a6be74ffca2ab5cc68a353eb8c3285d should resolve this
https://docs.rubra.ai/installation/uninstall

should we close this issue now?


add dependency groups to flattened vulnerability

Follow up on https://github.com/google/osv-scanner/pull/655

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

Comparison is base (cb3eeb2) 79.08% compared to head (4ba0154) 78.53%.
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #754      +/-   ##
==========================================
- Coverage   79.08%   78.53%   -0.55%
==========================================
  Files          86       86
  Lines        6121     6123       +2
==========================================
- Hits         4841     4809      -32
- Misses       1075     1107      +32
- Partials      205      207       +2

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


Rajkumar42/vsdbgtest

@gregg-miskelly @caslan @DustinCampbell  please review.

Never mind. I see that you already changed the minimum version.


totalCount seems to be wrong when using UseMongoDbPaging

Describe the bug
Hi everyone,
when i use UseMongoDbPaging with IncludeTotalCount=true , the totalCount field seems to be wrong when 'first' field is included in query.
To Reproduce
On MongoDB example (https://github.com/ChilliCream/hotchocolate-examples/tree/master/misc/MongoDB) ... When I write following query
{
  persons(where: { name: { contains: "Romeo" } }) {
    totalCount
    nodes {
      name
    }
  }
}

throws right number of items and totalCount (3):
{
  "data": {
    "persons": {
      "totalCount": 3,
      "nodes": [
        {
          "name": "Romeo McCloy"
        },
        {
          "name": "Romeo2 McCloy"
        },
        {
          "name": "Romeo3 McCloy"
        }
      ]
    }
  }
}

But when i include 'first' value, totalCount differs:
{
  persons(first:1, where: { name: { contains: "Romeo" } }) {
    totalCount
    nodes {
      name
    }
  }
}

{
  "data": {
    "persons": {
      "totalCount": 2,
      "nodes": [
        {
          "name": "Romeo McCloy"
        }
      ]
    }
  }
}

Expected behavior
totalCount should show always the total number of elements that matches the input filter, in above case, 3.
Additional context
Faced with HC 11.3 and also HC 12.0.0-preview.18

did you try this with 11.3? we reworked paging completly

did you try this with 11.3? we reworked paging completly

Sure, 11.3 and also 12.0.0-preview.18

I think there's a general flaw in the MongoDB pagination logic, since #3683 talks about the same issue, but for MongoDbOffsetPaging.

@PascalSenn I put both paging bugs on your deck for V12 since we need this to be working for our ccc project.


[Bug]: table input automatically insert square bracket and this leads to error

Apache Hop version?
2.3
Java version?
11
Operating system
Windows
What happened?
When I use table input transform and I select a table from a db in the query the program automatically adds square brackets and this lead to an error because the query results wrong. If I manually delete the extra square brackets it works but in this way I can't automatically select all the fields on the table if not with select *. This problem is present with all type of db connection, but I noticed that if the schema is beetween " " the problem is not present.

Issue Priority
Priority: 3
Issue Component
Component: Actions

Database type and version?

Ok sorry I checked better and with oracle and mysql I don't have the problem since there the schema is always between " ". The problem is with mssql (native) version: Microsoft SQL Server 2012 (SP3) (KB3072779) - 11.0.6020.0 (X64)
Oct 20 2015 15:36:27
Copyright (c) Microsoft Corporation
Standard Edition (64-bit) on Windows NT 6.3  (Build 9600: ) (Hypervisor)
and postregsql version: PostgreSQL 11.6 on x86_64-pc-linux-gnu, compiled by gcc (GCC) 4.8.5 20150623 (Red Hat 4.8.5-39), 64-bit

I have exactly the same issue.
Win 11, Hop 2.3, Java 11
SQL Database is an Azure SQL db.


Splash Proxy Cache

Hey,
It seems that Splash is caching my proxy credentials. If I change the username it seems to take around 30 seconds to propagate the change. Is there any way around this?
Thanks.

I'm not sure why it happens, I haven't seen it myself.


DSSP issue with building

Hi, Thanks for your work. I am trying to install dssp and ran into a problem.
The configuration is file ./configure --with-cif++=/home/yy/libcifpp
However, when I build with make I get the following error:
/usr/local/bin/mrc -o obj/mkdssp_rsrc.o /home/yy/libcifpp/rsrc/mmcif_pdbx_v50.dic
/usr/local/bin/mrc: error while loading shared libraries:  libboost_program_options.so.1.66.0: cannot open shared object file: No such file or directory
make: *** [GNUmakefile:139: obj/mkdssp_rsrc.o] Error 127
Would you be able to help?
Thanks!

This is a generic Linux related question and not something specific to DSSP. But yes, I can help you.
You have two options. The first and preferred one is to rebuild your boost libraries as static libraries. For this, you should first clean the boost directory (using ./b2 clean) and then rebuild the libraries using ./b2 link=static That way you get static libraries. If you now rebuild DSSP it will no longer need the shared libraries from boost.
The second option is to install boost (by running ./b2 install --prefix=/for/example/usr/local) and then add this path to the list of directories Linux looks for shared libraries. This is done by creating a file containing just this path in /etc/ld.config.so.d/ and then run ldconfig. As root.
I think the first option is easier :-)


chore: add awscli for nightly build

🤦 I did not realize that the CLI images were built in a container.  The container by default does not include the awscli.
Fixes: https://github.com/akuity/kargo/actions/workflows/release.yaml?query=event%3Aschedule

Thanks @34fathombelow!


chore(deps): upgrade compiler dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-compiler-dependencies-k8s-28-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3916

Questions ?
Please refer to the Backport tool documentation


Cannot use terminal after last update 2023.12.281947-latest

Describe the bug
Cannot use terminal, stuck in Connecting ... state. Cannot execute to pod.

To Reproduce
Steps to reproduce the behavior:

Go to File - Preferences - Terminal
Default value is powershell.exe

Try change to full path C:\Windows\System32\WindowsPowerShell\v1.0/powershell.exe
Still not work

Expected behavior
Terminal work normally.
Screenshots
If applicable, add screenshots to help explain your problem.
Environment (please complete the following information):

Lens Version: 2023.12.281947-latest
OS: [e.g. OSX] Windows 10 Enterprise 22H2 19045.3803
Installation method (e.g. snap or AppImage in Linux): Windows installer

I have the same problem.

I have the same problem too.

i have the same problem.
Lens Version: 2024.1.170738-latest
Chrome: 114.0.5735.289
OS: windows 10 enterprise 19044.3803
lens.log entry:
error: [SHELL-SESSION]: failed to open a local shell Failed to get directory containing kubectl: Error: Failed to symlink bundled kubectl {"stack":"Error: Failed to get directory containing kubectl: Error: Failed to symlink bundled kubectl\n    at C:\Program Files\Lens\resources\app.asar\node_modules\@lensapp\core-main\dist\index.js:4085:15"}

Lens Version: 2023.12.281947-latest
Chrome: 114.0.5735.289
OS: Microsoft Windows 10 Pro 10.0.19045 Build 19045
I had the exact same issue for Windows. I was able to resolve by closing Lens application, deleting the C:\Users<USER>\AppData\Roaming\Lens\binaries\kubectl folder and then relaunching Lens as Admin.
Hope this helps someone else!

Same for me on Mac. Not good, very bad.

Thanks for reporting, a fix for this will be release soon.

Issue still persist. Lens Setup 2024.1.300751-latest version. Cant connect with pod terminal. Same error.


Question about performance

Hi. After running some benchmarks with Acorn vs Esprima I noticed that Acorn - in my tests - are around 60% slower then Esprima. And in some other combos Acorn are 3 times slower. E.g module import and export. Esprima isn't the fastest parser either, but do the exist any plans to optimize Acorns performance?

No plans, no. You're welcome to try and find optimization opportunities, if you want, but we're not really interested in radical changes in architecture anymore at this point.

@ghost Could you please share test fixtures where you were seeing a 60% difference?


Consider logging more things as warn or even error

Description
Maybe those could be logged as warnings..it would make scraping the archive.log file a bit easier for more unusual scenarios like the following (which to give more context may be related to the 'arithmetic_overflow' panic I had at this day (I can't be sure tho as the panic - see other issue - wasn't logged):
2023-04-14 11:23:48 (UTC) iota_sdk::client::node_manager::http_client DEBUG GET: 26 ms for 404 Not Found https://api.testnet.shimmer.network/api/core/v2/transactions/0xb6c448d3af8c8d065305e33f534c5e0cdd55a800189652361e331ebe312e5805/included-block
2023-04-14 11:23:48 (UTC) iota_sdk::wallet::account::operations::syncing::transactions DEBUG [SYNC] conflicting transaction 0xb6c448d3af8c8d065305e33f534c5e0cdd55a800189652361e331ebe312e5805

Are you planning to do it yourself in a pull request?
Maybe.

Closed by https://github.com/iotaledger/iota-sdk/pull/958


History code appearing upon refreshing page?

I have been testing the software and I think I've found a bug?
I'm using Firefox Nightly 41,
Problem:
Texts unexpected appended to editor upon refresh
Expected Text:
space
Resulted Text:
First time. spaceWoofmarkSelectionOpenMarkergbdhcWoofmarkSelectionCloseMarkergbdhc
Second time. spaceWoofmarkSelectionOpenMarkerecaacbhWoofmarkSelectionCloseMarkerecaacbh
Procedures:
Input space or any text into WYSIWYG mode, refresh page.
Analysis:
The string "WoofmarkSelectionOpenMarker" + code string + "WoofmarkSelectionCloseMarker" + repeated code string is appended to text upon every refresh on WYSIWYG mode with some text input.

Weird. These are used to persist the selection across input modes, but shouldn't stay in the editor. Can you invetigate this issue yourself? Relevant code is here: https://github.com/bevacqua/woofmark/blob/master/src/rememberSelection.js#L24

Is it something to do with my newer issue on it wraps ` tags around content upon refresh? If so, the detecting function might break because of it?

https://github.com/bevacqua/woofmark/blob/master/dist/woofmark.js#L3488-3489

Somehow, your script saved the <p> tags to the chunks. Matching a regex in chunks.before with the rmarkers failed.

You script failed to find the rmarker because the page has been refreshed and a new code is generated.
https://github.com/bevacqua/woofmark/blob/master/dist/woofmark.js#L3456

Could you create a PR with the fix?

Replacing https://github.com/bevacqua/woofmark/blob/master/dist/woofmark.js#L3457-3459
with
var code = Math.random().toString(18).substr(2).replace(/\d+/g, '');
var open = 'WoofmarkSelectionOpenMarker{{' + code + '}}';
var close = 'WoofmarkSelectionCloseMarker{{' + code + '}}';
//var rmarkers = new RegExp(open + '|' + close, 'g');
var rmarkers = new RegExp('WoofmarkSelectionOpenMarker{{.+}}|WoofmarkSelectionCloseMarker{{.+}}', 'g');

solves the problem with refreshing the page in WYSIWYG mode, but I'm sure the code is there for a reason. If the regex matches any code, would it break the program?

I can't reproduce this either. Please help me reproduce

Similar to the other issue, I'm using Nightly, but Firefox should do.
Steps:

Use http://bevacqua.github.io/woofmark/
Use "wysiwyg" mode.
Delete everything and paste in space.
Click the refresh button on the right of the address bar.
You should notice the editor now contains WoofmarkSelectionOpenMarkerefcdagfWoofmarkSelectionCloseMarkerefcdagfspace

@bevacqua I discovered that, on Firefox, it tried to keep the content, but on Chrome, the editor resets.

I haven't done thorough testing on Firefox. Can you work out a fix for this issue?

@bevacqua If you look up, I've tried a fix for the problem, accounting that Firefox tries to save what was inputted. I wrapped the code with brackets so they can be detected and removed.
Sorry, but I'm really no expert in JavaScript, my "fixes" might just be silly codings :(

The input autocompletion is probably the culprit here. You can try disabling the autocompletion as per the linked article if you do not want the browser to "remember" the modified content on refresh.


Got banned huh

To test this cheat I created a fresh account (without prime enabled) and got almost instantly banned (1 deathmatch game). I was not using any skinchanger, only esp (which didn't even work). I used AlisaAlis Manual Map DLL Injector for injection. Where lies the problem with detection? Will modifying code heavily change the outcome or do I have to use different injector/injection method?

its probalby the hook that is detected

Injecting via manual map doesn't work, use a thread hijacking method instead to avoid features not working.

i guess they ban instantly for bytepatching (hooking) and not lower your trust like they did in csgo. youll have to deal with the detection on your own

Thank you for ideas

I just tried minhook but still get dedects, are there anything to suggest for making it ud?


Compiler Options: GCC 13.1 analyzer options

GCC 13.1 is now out. We should review its analyzer options, e.g.: https://gcc.gnu.org/gcc-13/changes.html
I think it'd make sense to maximally enable the analyzers in most cases... but I'm happy to hear alternative views.

Since this document is more about security and memory saftey IDK if it should go too deep into "general code quality" if thats' what a lot of the analyze rules do.
If there are plenty of worthwhile security checks in analyze then maybe it is more sensible to recommend maximally enable

Also I didn't realise gcc could output sarif, very interesting
https://gcc.gnu.org/onlinedocs/gcc-13.1.0/gcc/Diagnostic-Message-Formatting-Options.html#index-fdiagnostics-format

Since this document is more about security and memory saftey IDK if it should go too deep into "general code quality" if thats' what a lot of the analyze rules do. If there are plenty of worthwhile security checks in analyze then maybe it is more sensible to recommend maximally enable

There are actually a lot of security-relevant checks in the gcc analyzer like allocation tracking, but they're more heuristics than definite issues.  Unfortunately some of the stringop warnings are heuristics too, which muddies the waters a bit.

Also I didn't realise gcc could output sarif, very interesting
https://gcc.gnu.org/onlinedocs/gcc-13.1.0/gcc/Diagnostic-Message-Formatting-Options.html#index-fdiagnostics-format

FWIW there's some more information on GCC's SARIF support here: https://github.com/oasis-tcs/sarif-spec/issues/531 (and I'm now on the SARIF technical committee)

I think whether or not an analyzer should be added should be considered case-by-case, considering how likely it is to identify a vulnerability (benefit) vs. likelihood of a false positive (cost). If it's finding vulnerabilities in many cases, but has occasional false positives that can be suppressed when they happen, that's likely to be worthwhile. At the least, that would make it worth discussion.

Many GCC analyzer are linked to CWE. CWE give useful technical details about potential securities issues. these links to CWE give a first list of GCC analyzers related to vulnerabilities : arbitrary code execution (buffer overflow, use after free, double free, uninitialized buffer, and more), arbitrary memory access (read and write), denial of service and crash.

Here a list of GCC analyzers a made based on vulnerabilities described by CWEs from GCC official documentation.
GCC - Static Analyers.pdf

This issue is in relation to the hardening guide. There are a few GCC 13 flags in the doc but IDK if we've covered all new relevant options in 13.1

Revisiting this after analyzers were again brought up again in #330. The conclusion from this thread seems to be have been that it does not make sense to systematically cover the GCC static analyzers, but rather consider specific analyzers on a case by case basis. Since this issue is about analyzers as a whole, and I think the discussion here didn't narrow down specific analyzers to consider, can this issue be closed?

Before closing it, let's document why we're not doing it.
What does compiler options hardening not do?
What does compiler options hardening not do?
Compiler options hardening is not a silver bullet; it is not sufficient to rely solely on security features and functions to achieve secure software. Security is an emergent property of the entire system that relies on building and integrating all parts properly. However, if properly used, secure compiler options will complement existing processes, such as static and dynamic analysis, secure coding practices, negative test suites, profiling tools, and most importantly: security hygiene as a part of a solid design and architecture.
Just before the section "What is our threat model, goal, and objective?" let's add a short section like this:
** Why are general static analysis flags out-of-scope? **
This guide identifies compiler hardening flags for generating instrumented test code and production code. We include flags that help identify constructs that are likely to lead to vulnerabilities, if they have few false positives and those are easily systemically silenced.
Many compilers can also act as a general static analysis tool that heuristically identifies suspicious constructs that may lead to bugs or vulnerabilities. However, these heuristics may often have false positives (reporting problems where none exist) and/or be difficult to silence to generate executable code. Using a compiler as a general static analysis tool, where a human is expected to read the reports instead of using the generated code, can be valuable but is outside the scope of this document.

(GCC analyzer author/maintainer here) Unfortunately there will be false positives (and false negatives) from the analyzer, so I can't recommend enabling it by default. I have a longer post about this that looks into the false positives rates per warning; I hope it will go live in the next week or so; sorry that it's not quite ready yet.

FWIW the article I wrote last year about this is here; see the "Fixing false positives" section.


Use new db ex common

Update ODBC driverů je na PR https://github.com/keboola/db-extractor-snowflake/pull/87

@tomasfejfar @ondrajodas may I ask the status of this?  I am going to work on fixing #1 which will use the new driver download method, just wondering if we can merge or not

The new driver download method is already merged (https://github.com/keboola/db-extractor-snowflake/pull/87) I am not sure why it still shows up here. I'll examine it later. Anyway, you should be fine using master (that already has #87).

Thanks!  I've had to abandon it though because it can't quite work with the current docker-runner (https://keboola.slack.com/archives/C03JBGZDG/p1565794087002400 if you're interested)

It just wasn't rebased. That's why it contained also the merged commits. I rebased and force pushed. Make sure you do git pull --rebase to update your local branch now, @ondrajodas...

řeší se v #89 - zavírám


System test with minikube demo

Minikube is a lightweight Kubernetes implementation that creates a VM on your local machine and deploys a simple cluster containing only one node. https://kubernetes.io/docs/tutorials/kubernetes-basics/create-cluster/cluster-intro/
About setup-minikube action https://github.com/medyagh/setup-minikube

build/deploy/test your application against a real Kubernetes cluster in GitHub Actions.
maintained by minikube maintainers

@coretl check out what @marcelldls has done for ec system tests. I think this is really cool and could come in useful for many other K8S projects.

merging this one too.


Not guarding

Hi , i've installed and everything is showing fine , but when i fill
public $guarded = ['password'];
it doesn't guard the value it shows everything.
the guarded value of the model is shown with the value but the return shows the field password. how can i guard attributes?

$guarded prevents mass-assignment, it has nothing to do with displaying values.  Try $hidden.

i've tryed this too , same problem  https://gist.github.com/tallesdantas/f4721f0050ff65fddd12

What do you mean by "it shows everything?"  How are you "showing" the model?  $hidden will only remove the attributes from the output of toArray() and similar functions.  This is not a laravel-mongodb issue.

sorry didnt know this, thx for the dumb question answer btw xD


Move parsing logic to server-side

so to pass less payload to clients.

Hope you can also provide a commit to stream the response of openai on front end.


Feature: Object literals

resolves #2046
Add the new syntax for object literals using #{. For this first version an object literal can only contain other object literal and other literals(string, number, boolean))
There is a few things to figure out:

[ ] Name conflict with the existing Object type used in projection
[ ] DO we need #[] for a tuple literal?
[ ] Add valueof support

:x: There is undocummented changes. Run chronus add to add a changeset or click here.
The following packages have changes but are not documented.

:x:@typespec/compiler

You can try these changes at https://cadlplayground.z22.web.core.windows.net/prs/3022/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/prs/3022/

Some thoughts on the design of values in general (I think these are reflected, but just putting things into spec form).
Overall, it seems very important to separate types and values as cleanly as possible. It's tempting to have things serve as both depending on context, but this can cause significant confusion. For example, unions outside of signatures of decorators and templates should not allow both types and values, and most type references should not allow values, e.g. a model property like prop: #{} should be an error..
That said, there are exceptions. For example, certain literals can represent types or values (referred to as valuish types for want of a better word):

string literals - either a string literal type whose only assignable value is a string with that value, or a string of that value.
numeric literals - either a numeric literal type whose only assignable value is a number of with value, or a number of that value. Note also that numeric literals are often coerced to either a type or value of a numeric subtype depending on context.
boolean literals - either a boolean literal type whose only assignable value is that boolean, or a true/false value.
enum members - either an enum member type, or a value of that enum member's value.

It is often not ambiguous whether the string or value is required. For example, default values are always values, type references like the type of a model property are always types.
However, it is possible to define a parameter to a template or decorator signature that creates this ambiguity. Consider a decorator with a parameter of type string | valueof string. If we pass "hello", we are not sure if we are passing the string literal type or the string literal value.
We could say this is not a problem, but I believe it can be - types can carry additional metadata on them that a decorator may want to access. Notably, enum member types contain a reference to their container, whereas the enum value is just a string/number.
We could introduce new syntax to represent either the value or type versions, but this would be deeply annoying.
We could say that it's an error to declare such a signature, but @extension is a common decorator where you want to allow any type or any value, because you want to allow both pushing literal JSON into e.g. a JSON Schema, but also pushing a type as a schema. For example, @extension("is-thing", true) should emit "is-thing", true, @extension("itemSchema", string) should emit "itemSchema: { "type": "string" }.
A proposal that might work: valueish literals/references are always passed as value in situations where there is any ambiguity, and we provide a mechanism to explicitly pass as type when required (discussed in next section).
ValueType
A template that takes a value and returns a type whose only allowable value is exactly the provided value. For example:

ValueType<10>: the numeric type literal 10.
ValueType<SomeEnum.value>: the enum member type.
ValueType<#{ x: 1, y: 1}>: The model type {x: 1, y: 1}.

This allows disambiguating what the meaning of "foo" is when using @extension: @extension("itemSchema", ValueType<"foo">) emits "itemSchema: { "type": "string", "const": "foo" }..


Osram LED strip has no Color Temperature in Home.app

Hi,
I have two LED strips connected to the Hue base and accessible via Homebridge.
They are recognized as the following:
[10/13/2020, 1:44:35 AM] [Hue] Philips hue: 7CB03EAA0A008D49: OSRAM LIGHTIFY Indoor Flex RGBW "Lichtschlauch"
[10/13/2020, 1:44:35 AM] [Hue] Philips hue: /lights/1: Extended color light "Lichtschlauch"
[10/13/2020, 1:44:35 AM] [Hue] Philips hue: /lights/1: config: {"on":true,"bri":true,"ct":true,"xy":true,"speed":false,"colorloop":true,"lastBoot":false,"lastSeen":false,"resetTimeout":500,"waitTimeUpdate":20,"minCT":125,"maxCT":666,"gamut":{"r":[0.6877,0.3161],"g":[0.1807,0.7282],"b":[0.1246,0.058]},"unknown":false}
[10/13/2020, 1:44:35 AM] [Hue] Philips hue: 00124B00213EEE28: GLEDOPTO GL-MC-001 "Monitorbeleuchtung"
[10/13/2020, 1:44:35 AM] [Hue] Philips hue: /lights/3: Extended color light "Monitorbeleuchtung"
[10/13/2020, 1:44:35 AM] [Hue] Philips hue: /lights/3: config: {"on":true,"bri":true,"ct":true,"xy":true,"speed":false,"colorloop":true,"lastBoot":false,"lastSeen":false,"resetTimeout":500,"waitTimeUpdate":20,"minCT":155,"maxCT":495,"gamut":{"r":[0.7006,0.2993],"g":[0.1387,0.8148],"b":[0.151,0.0227]},"unknown":false}
GLEDOPTO supports color temperature via Home.app. However, the OSRAM LIGHTIFY Indoor Flex RGBW does not work with color temperature via Home.app, instead it translates to setting hue and saturation, leaving the dedicated white LED aside.
 [10/13/2020, 2:28:42 AM] [Hue] Monitorbeleuchtung: set homekit hue from 37˚ to 33˚
 [10/13/2020, 2:28:42 AM] [Hue] Monitorbeleuchtung: set homekit saturation from 61% to 99%
I assume, the support for this OSRAM LED strip is not yet there. Do you plan support for it, or at least provide me a hint, where to embed it?
Thanks

The Home app supports either Hue and Saturation or Color Temperature, but not both.  In case of colour lights, the Temperature wheel sends Hue and Saturation, just as the Colour wheel.  This is OK for RGB lights that mix colour temperature using the RGB channels, but not for RGB+CCT lights that use separate WW/CW channels for colour temperature.  I'm unfamiliar with the GLEDOPTO model you're using, but I suppose it has only RGB channels?  I'm unfamiliar with the OSRAM model you're using, but I would expect it to mix in the W channel for lower saturation values.  It would still need to use the RGB channels to mix the colour temperature, as it only has one W channel.
To drive the WW/CW channels from HomeKit, Homebridge Hue exposes Colour Temperature using the old custom characteristic, used by the Hue bridge before HomeKit supported the standard Color Temperature.  Custom characteristics are not supported by Home, but other HomeKit apps, like Eve, do support it.  You can create a scene in Eve containing the custom characteristic, and still recall that scene from Home.

Thanks @ebaauw,
GLEDOPTO is a LED RGB CW WW strip and works flawlessly with color temperature.
OSRAM LIGHTIFY Indoor Flex is a LED RGB WW strip, without additional CW LED.
So, your assumptions are true ;-)
I was wondering if it does make sense to add this.model = 'RGBW' to the models section for OSAM, in order to support the basic W-behavior via the color temperature setting?

I'm not sure what you mean by "basic W-behaviour"?  Homebridge Hue doesn't know about the number of channels, and exposes the OSRAM and GLEDOPTO to HomeKit in the same way.  Both expose themselves as Extended Color Light.  You can set the colour temperature from HomeKit through Eve.
You could try through the API setting various values for xy and ct to see if the two strips behave differently, or use the GUI with Move to Color (specifying Current X and Current Y) or Move to Color Temperature (specifying Color Temperature).

Thanks. Yes, setting color temperature via Eve works for both strips.
I was just wondering why these two stips behave differently in Home.app; when their config-strings appear to be the same.
[10/25/2020, 2:30:58 PM] [Hue] GLEDOPTO: homekit color temperature changed from 367 mired to 492 mired uses the ct-features, while
[10/25/2020, 2:32:19 PM] [Hue] OSRAM: set homekit hue from 20˚ to 321˚
[10/25/2020, 2:32:19 PM] [Hue] OSRAM: set homekit saturation from 29% to 73% uses the xy-feature.
Anyways, the workaround via Eve scene works. It allows for some basic colour-tempature settings for the OSRAM strip.


test/deserialize-from-json

Description of change
I have added json files for the Transferstates and Actions and changed the test files to work with them.
Some info in the json files is now hardcoded, I kept the previously accompanying code in the test files commented out but not deleted.
Links to any relevant issues
fixes issue #51 .
How the change has been tested
I have tested the changes by running again the same test files I edited.
I have also added a 'async fn pretty_print_test' for testing the serialization and deserialization json output.
Definition of Done checklist
Add an x to the boxes that are relevant to your changes.

[x] I have followed the contribution guidelines for this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

The intended structure for the fixture files is as follows:
Location of the new folder:
src-tauri/tests/fixtures

With subfolders:
src-tauri/tests/fixtures/actions
src-tauri/tests/fixtures/states

In your current update they are structured like this:
src-tauri/tests/tests/fixtures-<test-name>/actions
src-tauri/tests/tests/fixtures-<test-name>/states

I think it is better to throw all state json files into src-tauri/tests/fixtures/states since in the future 'mermaid' update all the <test-name>'s will not exist anymore. That also means though that the names of the state json files must be a bit more explicit so that it is easier to understand what's inside. For example the current tests/tests/fixtures-qr_code_scanned/states/read_cred.json should be renamed to something like tests/fixtures/states/active_profile_with_credential_offer.json.
The Actions are easier to rename, for example the current tests/tests/fixtures-get_state/actions/create_new1.json should be renamed to tests/fixtures/actions/app_get_state.json, named after the actual ActionType.
Some are a bit trickier, like tests/tests/fixtures-qr_code_scanned/actions/handle_oid4vp_1.json and tests/tests/fixtures-qr_code_scanned/actions/handle_siopv2_1.json. They can be renamed to tests/fixtures/actions/qr_code_scanned_with_vp_token.json and tests/fixtures/actions/qr_code_scanned_with_id_token.json respectfully.
Other than that some of the code does not seem to be formatted as intended. You can easily formatted by using cargo fmt.
If these minor issues are solved than this PR is good to go!

Thanks for resolving everything! I just commented two more small issues. Also the fixtures folder is still located in src-tauri/tests/tests while a better location imo would be to move it up one level to src-tauri/tests

Nice work! 🎉 Please consider renaming some json files to more descriptive names.

Naming can be quite tricky sometimes in programming, and this is no exception. So if we want to do it right(-ish) from the start I would propose these name changes:
active_pf_password_required.json                    -> profile_password_required.json
active_pf_redirect_me.json                          -> profile_redirect_me.json
no_profile_redirect_welcome.json                    -> no_profile_redirect_welcome.json
pf_accept_connect.json                              -> profile_accept_connection.json
pf_credential_offer.json                            -> profile_credential_offer.json
pf_with_did_redirect_me.json                        -> profile_did_redirect_me.json
pf_with_driverl_and_personal_info_redirect_me.json  -> profile_with_two_credentials_redirect_me.json
pf_with_person_info_redirect_me.json                -> profile_with_credential_redirect_me.json
pf_with_person_info_share_cred.json                 -> profile_with_credential_share_credentials.json

Alternatively I would perhaps even prefer to omit the profile_ prefix since most state json files will have an active_profile and thus it does not add any descriptive benefit.


Missing "using"

If you take the first snippet of code, copy and paste, it won't compile. Needs to have added two "using"
using System;
using System.Threading.Tasks;

Thanks!

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

ID: 83e95a3f-447b-8ca0-beb5-f0363896fe29
Version Independent ID: b24e3ee5-404c-98a2-042e-6ac2d363b842
Content: Get started with Dapr
Content Source: docs/architecture/dapr-for-net-developers/getting-started.md
Product: dotnet-architecture
GitHub Login: @amolenk
Microsoft Alias: nanil

Good catch @fergek, we will add the statements.

Hi,
The sample is for .NET 6. If you follow the instructions closely those using statements shouldn’t be necessary because of the implicit usings feature of .NET 6.

Thanks @amolenk for the response, you are right. We can close this issue now.


error: could not compile t6 (bin "t6") due to 1 previous error; 2 warnings emitted

= note: /usr/lib/gcc/x86_64-alpine-linux-musl/13.2.1/../../../../x86_64-alpine-linux-musl/bin/ld: cannot find -lssl: No such file or directory
/usr/lib/gcc/x86_64-alpine-linux-musl/13.2.1/../../../../x86_64-alpine-linux-musl/bin/ld: cannot find -lcrypto: No such file or directory
/usr/lib/gcc/x86_64-alpine-linux-musl/13.2.1/../../../../x86_64-alpine-linux-musl/bin/ld: cannot find -lz: No such file or directory
collect2: error: ld returned 1 exit status

Hi.
I bet you are trying to build a statically linked binary. I was never actually tried that but first consider disabling the default set of crate features (default-features = false) and enable only that you actually use. Secondly — if TLS support is necessary, then use rustls-tls instead of native-tls.

use env: RUSTFLAGS=-C target-feature=-crt-static


domain is wrong

use this url 'https://test-passport.jindashi.cn/TDOA/page/pfyh/quick.html'
the result domain is  hi.cn  not jindashi.cn

Surprised this issue didn't come up sooner :P
It's fixed in v2.5.1 (node package also).
Thx :-)


Roll Flutter Engine from 2adad88a39f4 to da440cb597f3 (1 revision)

https://github.com/flutter/engine/compare/2adad88a39f4...da440cb597f3
2024-01-27<EMAIL_ADDRESS>Roll Fuchsia Linux SDK from WHlwlOwznFknNm5IS... to GBTh3gOOgmndwT70X... (flutter/engine#50111)
Also rolling transitive DEPS:
fuchsia/sdk/core/linux-amd64 from WHlwlOwznFkn to GBTh3gOOgmnd
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC jacksongardner@google.com,rmistry<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


[DH-496] Contact filters

This brings Contact filters in line with requirements, which state we need 4 filters:

Company Name (with partial matching)
Sector
Country
UK Region

Codecov Report

Merging #417 into develop will increase coverage by 0.06%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           develop     #417      +/-   ##
===========================================
+ Coverage    95.88%   95.94%   +0.06%
===========================================
  Files           86       86
  Lines         2938     2938
  Branches       254      254
===========================================
+ Hits          2817     2819       +2
+ Misses          83       82       -1
+ Partials        38       37       -1

Impacted Files
Coverage Δ

datahub/search/contact/views.py
90.9% <ø> (ø)
:arrow_up:

datahub/search/elasticsearch.py
95.32% <ø> (ø)
:arrow_up:

datahub/search/dict_utils.py
100% <0%> (+7.14%)
:arrow_up:

Continue to review full report at Codecov.

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


Can OIH records be 'metadata only'

Does an OIH metadata record have to resolve to a dataset?
If so, can thar be a portal 'landing page' or a publication (ie PDF) or does it need to be a machine actionable endpoint? (eg OGC API or direct file download)

That seems fine to me, just leave the https://schema.org/distribution property empty.

Does an OIH metadata record have to resolve to a dataset?

No, but a schema:Dataset JSON-LD record should point to the data it describes via the distribution property
other types need not point data sets

If so, can thar be a portal 'landing page' or a publication (ie PDF) or does it need to be a machine actionable endpoint? (eg OGC API or direct file download)

if it's a landing page or something similar, you can use the url property. But it must point to the dataset and not some document about the dataset (like a paper). One can use the citation property to point to resources that talk about the subject data

Slight tangent, so are the records from MEDIN schema:Dataset ? Because there's many, many on there that don't have any value in the distribution property.  Will they be dropped from OIH?

Slight tangent, so are the records from MEDIN schema:Dataset ? Because there's many, many on there that don't have any value in the distribution property.  Will they be dropped from OIH?

Most are, other Types are fine too
No distribution will not affect their discoverability in ODIS or OIH, but it will make rapid, programmatic download harder.


Compilation error - mismatched types

Bug report
I'm trying to compile the mssql_fdw wrapper on a Raspberry Pi 4 and encountering an issue when the compiler gets to supabase-wrappers.

[x] I confirm this is a bug with Supabase, not with my own application.
[x] I confirm I have searched the Docs, GitHub Discussions, and Discord.

Describe the bug
error[E0308]: mismatched types
    --> supabase-wrappers/src/import_foreign_schema.rs:84:18
      |
      |             ret.push(command.as_pg_cstr());
      |                 ---- ^^^^^^^^^^^^^^^^^^^^ expected `*mut i8`, found `*mut u8`
      |                 |
      |                 arguments to this method are incorrect
      |
      = note: expected raw pointer `*mut i8`
                 found raw pointer `*mut u8`

To Reproduce

Install Postgres 15
Install pgrx-0.11.3
Run cargo pgrx install --no-default-features --features pg15,mssql_fdw --release in ~/wrappers/wrappers

Expected behavior
Build completes
Screenshots
N/A
System information

Raspberry Pi 4B
OS: Raspberry Pi OS March 2024
Browser (if applies) N/A
Version of supabase-js: N/A
Version of Node.js: N/A

Additional context

The return type of as_pg_cstr() changes based on the platform. I thought I had fixed that. The previous line should be     let mut ret: PgList<ffi::c_char> = PgList::new();


created string constants for hardcoded strings in InitalizationUtil

Since the strings were mentioned to be abstracted specifically for the InitalizationUtil file, I just created a string constant for "csv" & "properties" each & replaced the strings with their respective variables at all their occurrences in the file.
Kindly tell if I need to add string resources rather than creating those constants.

Fixes odk-x/tool-suite-X#296


Some users give a error 500 when trying to doppelgang her/him

For example this user:
https://ep2016.europython.eu/admin/auth/user/2005/login/
He/she got a ticket from:
https://ep2016.europython.eu/admin/auth/user/2000/
I think this might be the issue, I remember we had a fix for this from last year but can't figure it out now.
The same might be happening with the owners or receivers of owners of the following tickets:
ERROR:root:Could not find a profile for ticket_id 2079.
ERROR:root:Could not find a profile for ticket_id 2223.
ERROR:root:Could not find a profile for ticket_id 2262.
ERROR:root:Could not find a profile for ticket_id 2290.
The ticket_id 2079 is the one I described above.

For the case of the ticket_id 2223.
Exactly the same happens to:
https://ep2016.europython.eu/admin/auth/user/2110/login/
Who received a ticket from:
https://ep2016.europython.eu/admin/auth/user/2109/

I am using the command:
python manage.py ticket_profiles ep2016 --ticket-id 2223 --raise
avaiable in this branch:
https://github.com/PythonSanSebastian/epcon/tree/alxfix

This is the traceback we're getting from the system:
Traceback (most recent call last):

  File "/usr/local/lib/python2.7/site-packages/django/core/handlers/base.py", line 113, in get_response
    response = callback(request, *callback_args, **callback_kwargs)

  File "/usr/local/lib/python2.7/site-packages/django/utils/decorators.py", line 91, in _wrapped_view
    response = view_func(request, *args, **kwargs)

  File "/usr/local/lib/python2.7/site-packages/django/views/decorators/cache.py", line 89, in _wrapped_view_func
    response = view_func(request, *args, **kwargs)

  File "/usr/local/lib/python2.7/site-packages/django/contrib/admin/sites.py", line 202, in inner
    return view(request, *args, **kwargs)

  File "/usr/src/ep2016/assopy/admin.py", line 388, in create_doppelganger
    auth.login(request, user)

  File "/usr/local/lib/python2.7/site-packages/django/contrib/auth/__init__.py", line 93, in login
    request.session[BACKEND_SESSION_KEY] = user.backend

AttributeError: 'AnonymousUser' object has no attribute 'backend'

These are user records created by the system when someone assigns a ticket to an email address for which there is no existing user record in the system. Apparently, the system doesn't also create a profile object for the user.


rs-measure.cpp in python

do we have rs measure in python if yes can any one share it

A RealSense community member did some conversions of Measure into Python.  Links to their scripts, and advice from their creator, can be found in the link below.
https://github.com/IntelRealSense/librealsense/issues/3948#issuecomment-491228272
There is also the 'box_dimensioner_multicam' example program for Python:
https://github.com/IntelRealSense/librealsense/tree/master/wrappers/python/examples/box_dimensioner_multicam

thanks for the reply
https://github.com/IntelRealSense/librealsense/issues/2204#issuecomment-414497056

Hello @scizors  @MartyG-RealSense, based on this issue. Is there any solution that can help me get de width and height of an detected object using the depth? Or something other than the pixels, since it isn't very accurate? Can you help?
Thanks for your help!

Hi @pfcouto  If you are using Python then the RealSense SDK's TensorFlow wrapper may be helpful to you.
https://dev.intelrealsense.com/docs/tensorflow-with-intel-realsense-cameras

Hi @MartyG-RealSense I am using Detectron2 with pytorch to inference. You don't know any way to get the width of an object?


Inconsistent ensembl info display

Issue
Internally VarFish displays gene/transcript information associated with GRCh37, but it displays results from GRCh38 while querying Ensembl. This mix leads to confusion.
Example
Gene PPP2R2C (chr4:6,349,605)
The link provided by VarFish refers to GRCh38 :
http://www.ensembl.org/Homo_sapiens/Gene/Summary?g=ENSG00000074211;r=4:6320578-6563600
...but displayed internally GRCh37:
http://grch37.ensembl.org/Homo_sapiens/Gene/Summary?db=core;g=ENSG00000074211;r=4:6322305-6565327
Expected behaviour
Consistent information displayed internally and queried from external sources, i.e., same genome version.
Additional info
Issue also impacts VariantValidator:
https://rest.variantvalidator.org/VariantValidator/variantvalidator/hg19/chr4-6349605-C-T/all?content-type=application%2Fjson
https://rest.variantvalidator.org/VariantValidator/variantvalidator/hg38/chr4-6347878-C-T/all?content-type=application%2Fjson

Root Cause Analysis
We do two link-outs to Ensembl, one for the variant and one for the gene. As it was not specified which link-out was meant, I checked both:
Linkout for the variant

This links correctly to the GRCh37 genome.
Linkout for gene in fold-out
Ensembl ID in the gene card links wrongly to the GRCh38.
Resolution Proposal
Linkout to ensembl gene needs to be changed.
Affected Components

VarFish server

Affected Modules/Files

geneinfo/templates/gene_info/gene_info.html

Required Architectural Changes
None.
Resolution Sketch
www.ensembl.org to grch37.ensembl.org

@eudesbarbosa I fixed the broken link-out (at least to my understanding). I couldn't follow the issue with variantvalidator. this sounds like an separate issue. In this case, could you please open a new bug report for this?

@stolpeo, I scan the merge request and just to clarify, is the used link depending somehow on the Genomebuild info? I'd assume that we have already samples processed with GRCh38 and that could lead to problems for those.
Regarding the VariantValidator, I will write a different issue.

@eudesbarbosa Yes, the link is based on genomebuild. Right now, the GRCh38 development branch is not yet merged into the main branch. I would prefer to have the link switch in the GRCh38 branch rather than integrating GRCh38 related stuff before introduction of the actual feature.

Thank you for the clarification.
Regarding VariantValidator, I could not replicate the error. It seems the transcripts displayed in the query is that same as the ones displayed in the 'Transcript information' table. Plus, the warning displayed can be ignored - according to their website:
RefSeqGene record not available:
Only some genes have an NCBI RefSeqGene record. This warning simply indicates that no RefSeqGene record exists for the gene in which the variant that is being validated resides.

I changed my mind and included the switch anyway after revising @holtgrewe MR #220 and not finding a change in the according lines. So this commit should make the ensembl link-out future proof.


Add file reader tests

First attempt at adding some basic FileReader checks.
The Bit Twiddling change is for @DanRStevens to review! I think this is a legitimate check to prevent seeking before start of stream bounds?
-Brett

See first draft of interface testing. This uses GoogleTest Typed Tests as defined here https://github.com/google/googletest/blob/master/googletest/docs/advanced.md. The sample here https://github.com/google/googletest/blob/master/googletest/samples/sample6_unittest.cc was very helpful.
I'm not good at template code, so hopefully everything is mostly correct??
It appears to test the code actually using the interface (Creating a MemoryReader and then testing it by calling it as a SeekableReader).
-Brett

The defined constructor for the MemoryReader is broken since the buffer is deleted after passing into MemoryReader. FileSliceReader's test will fail if added to the test suite due to seeking before 0 being unhandled. Ran out of time, so sorry if this explanation is a bit rough.
-Brett

Ok, I've taken a look at this more thoroughly now. I played around and got things to compile, though tests are still failing.
It seems I missed a bit of detail in my earlier reading. I'm going to update the test issue with some of my findings and recommendations. I'll then try to clean up what I've done here into easy to follow steps.

Ok, I think this is the general structure we are looking for. I haven't addressed the specifics of the tests themselves. The code is compiling. The tests are running.
Side note: The CI servers aren't currently running the tests. Hence they may still show as passing, even though the tests are failing. We'll get to that.
As for the tests, I'm currently seeing 10 failures on Linux. They are all the same, and are kind of a silly, in that the test data files "EmptyFile.txt" and "SimpleStream.txt" are not found. The reason likely being that I'm running the test executable from a different directory, and the tests use relative paths.
Perhaps we should generate the test files in the system temp folder, rather than store them in the repository. Mind you, that adds the complication of finding a platform specific temp folder. It's tempting to try and use the executable path as a base, but that doesn't work since the executable file could be placed anywhere relative to the source it was compiled from. Similarly, using the __FILE__ predefined macro doesn't work because the current working directory is unrelated to the source root folder. In fact, the source might not even be available to the compiled test suite after it's been compiled. Though in practice, it should be a reasonable assumption that the test source folder will be available somewhere for the test executable to examine.

I updated the Visual Studio project file by removing the deleted .cpp file and everything worked properly on Visual Studio. Nice work getting the tests switched over to the 'more complex' format.
I think that I would prefer leaving the test files in the test source code as it keeps things a bit more simple. I could look at updating the Visual Studio environment starting location to be the same directory as the Linux build, which would solve the relative directory issues. I don't know how difficult it would be to update the makefile test portion to use the test directory as it's default directory?
-Brett

Okay, I just updated the Visual Studio environment to start at OP2Utility instead of the test directory. Unfortunately, this is considered a user setting and is not stored in the repository, so others who download the library would have to manually set this property to get the tests to pass. It is an easy settings change though.
I'd like to figure out a better permanent project setting at some point, but this should get it compiling on both Linux and Windows for the time being.

Ok, I think what we can do then is to update the Linux makefile so make check runs the test from the "test" subfolder. Then you should be able to remove the Visual Studio user setting, and tests should work with default settings anywhere.

Ok, we've solved the big issues on this branch. We now have a good template structure for new tests. The tests should be passing on both Linux and Windows (awaiting verification) with default project settings. We've also set an expected test run folder, so we have a clear way to access test data files.
Should we move the test data files to a subfolder? Right now the folder is an arbitrary mingling of test and data files, which makes it harder to see what and where the tests are.

I'd still like to address some issues of test quality, though that's a deeper issue and may take some learning. We could defer that refactoring to future branches. There's plenty of value in this branch that I'd be happy to merge now.
As an example of future refactoring, we should consider moving ZeroSizeStreamHasSafeOperations to a header file and making it type parameterized. It would need to go in a separate test case, as it's written specifically for streams of size 0. Alternatively, we could split it into components that are valid elsewhere. In particular, each section relies on different assumptions:
TEST_F(EmptyMemoryStreamReader, ZeroSizeStreamHasSafeOperations) {
	// Length and position (expects a seekable stream of length 0)
	EXPECT_EQ(0, stream.Length());
	EXPECT_EQ(0, stream.Position());
	// Seek to current position (should not cause error) (expects a seekable stream)
	ASSERT_NO_THROW(stream.Seek(0));
	ASSERT_NO_THROW(stream.SeekRelative(0));
	// Read 0 bytes is safe (valid for any stream, of any length)
	ASSERT_NO_THROW(stream.Read(nullptr, 0));
	EXPECT_EQ(0, stream.ReadPartial(nullptr, 0));
}

I pushed all the test data into a directory called data. I'm going to merge the branch now so we can start new branches to continue other endeavors. Everything is compiling on Windows. No problem putting the data in a modified directory from this, but I don't want to keep fiddling with this branch.
I think you have a very specific idea of what you want for the stream::reader test refinement, so I would probably leave that effort to you.
I'd like to start creating a couple of simple tests for other classes in the library. This would populate the test source code files so they exist in sort of skeleton form as we currently only have the stream::reader covered at all.
-Brett

Alright, I agree. Time to merge this and move on. We can make improvements in future branches.
I noticed the Windows Pull Request build has failed. That would seem to indicate there will be a problem after merging? Looks related to the file rename from moving test data into a subfolder.
Hmm, indeed. The merge commit also failed.


Failed to refresh token - Error details: Manually created ServicePrincipalToken does not contain secret material to retrieve a new access token

Which version of the AzCopy was used? 10.2.1
Which platform are you using? Linux - CentOS 7.6
What command did you run?
./azcopy copy /tmp/hi https://adlsdb2woc.dfs.core.windows.net/datalake/hi --overwrite=false --from-to=LocalBlobFS --cap-mbps 1
What problem was encountered?
[root@mythos1 azcopy_linux_amd64_10.2.1]# ./azcopy copy /tmp/hi https://adlsdb2woc.dfs.core.windows.net/datalake/hi --overwrite=false --from-to=LocalBlobFS --cap-mbps 1
INFO: Scanning...
INFO: Using OAuth token for authentication.
INFO: failed to refresh token, please check error details and try to log in with azcopy login command again. (Error details: Manually created ServicePrincipalToken does not contain secret material to retrieve a new access token)

Job 9bf81cbf-045f-5440-6166-8353a218755b has started
Log file is located at: /root/.azcopy/9bf81cbf-045f-5440-6166-8353a218755b.log

0 Done, 0 Failed, 1 Pending, 0 Skipped, 1 Total,

Job 9bf81cbf-045f-5440-6166-8353a218755b summary
Elapsed Time (Minutes): 0.0335
Total Number Of Transfers: 1
Number of Transfers Completed: 0
Number of Transfers Failed: 1
Number of Transfers Skipped: 0
TotalBytesTransferred: 0
Final Job Status: Failed

azcopy.log
How can we reproduce the problem in the simplest way?
See above
Have you found a mitigation/solution?
No

That's a weird error.  @adreed-msft Can you please take a look at this on Friday or next week?

That does seem odd. @jeeva86 Are you authenticating via a certificate or via a secret? That should help me figure out what causes this.

Furthermore, could you show me the login command/env vars you used (with secret details redacted)?

I logged in via:
[root@mythos1 azcopy_linux_amd64_10.2.1]# ./azcopy login
To sign in, use a web browser to open the page https://microsoft.com/devicelogin and enter the code A27JZRQ3N to authenticate.

INFO: Logging in under the "Common" tenant. This will log the account in under its home tenant.
INFO: If you plan to use AzCopy with a B2B account (where the account's home tenant is separate from the tenant of the target storage account), please sign in under the target tenant with --tenant-id
INFO: Login succeeded.

The plot thickens-- You are using OAuth and not service principal auth.

I'll have to investigate what could cause this kind of error to be generated from a typical OAuth token.

Note that I don't receive this error with azcopy v8, so I've always resorted to using that v. but I can't this time because I need to pull/push into ADLS Gen2 :).

Doubly odd, You're getting this error on ADLSG2? This error occurs when we're trying to create a blob pipeline, not a blobFS pipeline.

I get the error there and in Blob(https://fiftypct.blob.core.windows.net). v10 never liked me...

`azurerm_virtual_machine.tf-vm-test[0]: Still creating... [52m40s elapsed]
azurerm_virtual_machine.tf-vm-test[1]: Still creating... [52m40s elapsed]
Error: azure.BearerAuthorizer#WithAuthorization: Failed to refresh the Token for the request to https://management.azure.com/subscriptions/xxxxxxxxxxxxxxxxxxxxxxxxxxxxxx/providers/Microsoft.Compute/locations/australiaeast/operations/da4d39bb-b6c5-4f59-9916-60a8edf9c50f?api-version=2019-03-01: StatusCode=0 -- Original Error: Manually created ServicePrincipalToken does not contain secret material to retrieve a new access token
on test_tpi_standalone.tf line 117, in resource "azurerm_virtual_machine" "tf-vm-test":
117: resource "azurerm_virtual_machine" "tf-vm-test" {
Error: azure.BearerAuthorizer#WithAuthorization: Failed to refresh the Token for the request to https://management.azure.com/subscriptions/xxxxxxxxxxxxxxxxxxxxxxxxxxx/providers/Microsoft.Compute/locations/australiaeast/operations/298d85c7-0fee-4ce9-95f4-4b76e89c4cfa?api-version=2019-03-01: StatusCode=0 -- Original Error: Manually created ServicePrincipalToken does not contain secret material to retrieve a new access token
on test_tpi_standalone.tf line 117, in resource "azurerm_virtual_machine" "tf-vm-test":
117: resource "azurerm_virtual_machine" "tf-vm-test" {
`

Took an hour to provision two VM's which manually can take by 5 minutes.

@adreed-msft any update on this?

I'm a bit befuddled, honestly. The latter bit of the error is returned from a SetAuthenticationValues call in ADAL. We attempt to create a service principal token from an OAuth token, which returns a secret type of ServicePrincipalNoSecret. Which means this call could only be triggered during a refresh if your OAuth token somehow contained no refresh information.
If the scope parameter of the device code login contained offline_access, a refresh token is issued. Lucky for us, According to this article, offline_access is enabled by default.
Is it possible you may have disallowed some permissions?

I used azcopy login to authenticate. What permissions should I be looking for?

I've just tried to log in under a new account and I didn't see any special prompts... I'm honestly unsure of what might trigger a scenario like this other than the inferred offline_access scope not kicking in for some reason.
@JohnRusk and @zezha-msft, We can't currently explicitly add any scope parameters in the autorest ADAL Go SDK. Perhaps we should send out a PR to fix that, and explicitly place the two inferred permissions (user.read and offline_access) into that scope. That would probably resolve this issue.
For now though, we can at least error out on azcopy login if no refresh token is available for OAuth. It doesn't FIX this issue, but it'd save us some similar confusion in the future.

@amishra-dev thoughts on this issue?

we have the same problem any update?

Encountered same issue when running terraform to create resources. Any updates on this issue?

Have you seen this? https://github.com/hashicorp/go-azure-helpers/pull/43

nov. 2019 kl. 22:33 skrev AnuradhaAshavathaRao<EMAIL_ADDRESS>

Encountered same issue when running terraform to create resources. Any updates on this issue?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

@AnuradhaAshavathaRao could you please clarify how is that related to AzCopy?

Have you seen this? hashicorp/go-azure-helpers#43

That may have fixed this issue. I haven't run into token expiration the last few months.

I'm still getting this problem btw:
INFO: Scanning...
INFO: Authenticating to source using Azure AD
INFO: failed to refresh token, please check error details and try to log in with azcopy login command again. (Error details: Manually created ServicePrincipalToken does not contain secret material to retrieve a new access token)
INFO: azcopy: A newer version 10.8.0 is available to download

INFO: Any empty folders will not be processed, because source and/or destination doesn't have full folder support

failed to perform copy command due to error: cannot start job due to error: cannot list files due to reason -> github.com/Azure/azure-storage-blob-go/azblob.newStorageError, /Users/runner/go/pkg/mod/github.com/!azure/azure-storage-blob-go@v0.10.0/azblob/zc_storage_error.go:42

Version 10.5.0


[Minor Bug] Version 1.3.3 not on Assetlib

Godot version
4.2.2
Plugin version
Specify a numerical Spatial Gardener version you are using. Include Git commit hash if possible.
Example:
None currently, but was using 1.3.1. Deleted and went to the assetlib to try and update, but it seems that the latest version is 1.3.2
Platform
What platform you were using, when you encountered this bug?
Example:
Windows 11
Issue description
In the assetlib, it says that the version of Spatial Gardener on the assetlib is 1.3.2. I'm sorry if this is not too much of an issue.

Taken on 21/5/2024
Steps to Reproduce
Describe steps for others to reproduce this issue
Example:
"1. Open Godot
2.  Click on Assetlib
3. Search for "Spatial Gardener"
4. Click on "Spatial Gardener
5. Observe the version number: 1.3.2

Minimal reproduction project
If applicable, please paste a link to minimal reproduction project that shows this issue in action.

Thank you for noticing!
The download link on AssetLib leads to 1.3.3, but the version in description was not updated. Submitted the edit request, should be updated shortly


Add option to display some custom fields on small profile

Some organisations would like to have some custom profile fields displayed in
the small profile, instead of being shown only in the full profile, which is one
extra click away. This commit offers a way of choosing fields to be displayed in
the small profile. #10135
Testing Plan:

Add a custom field and mark it to be displayed on small profile
Set a value for it
Verify that the value appears on the small profile

GIFs or Screenshots:

I've also added some casper docs fixes in this PR, since I ended up breaking some tests on casper by writing some arrow functions in the popovers.js module. I had to remote debug the casper tests to find the problem, and found a bunch of outdated documentation.
Also, would it be worth adding a linter for this case to prevent folks from writing super new js?

Ah, I just found this comment by @timabbott which says Casper "catches" ES6 code.

I think a good first step would be to add that as a note to the run-casper script. I'll add a new commit for that.

@timabbott I've resolved the conflicts on this branch (for the documentation) commits.

@punchagan thanks for working on this!  I merged most of the prep comments, because they looked good.
I'm a bit worried about the design impact of having additional height on that widget; I kinda want to wait until #12442 is merged (which should be very soon), because it's possible with that higher-metadata feature, we'll be able to at least make external accounts display very efficiently (e.g. a row with an icon for the email address and icons for any configured external accounts).

Sure! I'll watch that PR and follow-up on this PR when that gets merged

I accidentally deleted the branch and that closed the PR.  And I'm unable to re-open it even after doing a bunch of restore to older commits and force pushing. :man_facepalming:


Add field for source of occurrence data?

Description:
I would like to suggest the addition of a field to occurrence cubes that contains information about the source of the observations.
Background:
I am implementing general biodiversity indicators, such as species richness and evenness, using the occurrence cubes. One of the main challenges in implementation is that there are temporal, spatial, and taxonomic biases that I am attempting to mitigate or account for in some way. However, there are different biases associated with different data source types, and it would be very useful to be able to deal with these biases separately, or at least to compare them.
Suggested Implementation:
One possible way to add this field would be to include it as a new column, which could be populated with either numerical codes linked to source types (e.g., citizen science data, scientific survey, museum collection) or with source names. However, I do not know what is feasible.

Hi @shawndove
Dataset source
GBIF currently doesn't have source information (citizen-science, survey, etc.) per dataset, at least not in a structured way. I think there were talks to add machine-generated tags to datasets (e.g. source: citizen science, spatial-resolution: gridded), but I don't know if that is still on the radar. To work as a dimension in the cube, each dataset should also just have a single source assigned (and not more than one).
What currently is possible however, are:
basisOfRecord
Each occurrence has a single basisOfRecord (provided by the publisher), which broadly characterizes it as PRESERVED_SPECIMEN, MACHINE_OBSERVATION, HUMAN_OBSERVATION, etc. This is not at the level you are looking for, but can potentially help as a dimension. It is also possible to filter out certain of these categories before creating a cube. E.g. this is a query for machine observations only: https://www.gbif.org/occurrence/search?basis_of_record=MACHINE_OBSERVATION&advanced=1
datasetKey
Each occurrence has a single datasetKey (the identifier of the dataset to which the occurrence belongs). This too can be used as a filter or dimension. As a dimension, it would give you a much higher cardinality than you are looking for, but you could look up and assign your own "source" to each datasetKey to reduce the number of categories to something that works for you.
Both basisOfRecord and datasetKey have been suggested as dimensions in the spec.

Thank you @peterdesmet for a clear explanation of the options. I think the datasetKey would be appropriate for my use case.


History API is ignored if a page is opened in a new tab (target "_blank")

Environment
App version: <IP_ADDRESS>
Device model: iPhone 12 pro
iOS version: 14.6
Steps to reproduce the bug

Start with a simple website (the initial website) that opens a different website (the opened website) in a new tab when clicking on a link:

Open in new tab

On the opened website, use the History API's pushState method (e.g. when clicking on a button) to navigate to a new page

history.pushState({ currentPage: "page1" }, "Page 1")

Press the back button in the browser to go to the previous page

Expected behavior
It should go back to the first page of the opened website.
Actual behavior
It goes back to the initial website and skips over the first page (or more intermediate pages) on the opened website.
Example
I've made a small repository with example code and hosted it so you can try it yourself without having to write any code.

Open the following URL (= the initial website) in DuckDuckGo's browser for iOS:
https://simonbackx.github.io/push-state-example/open-in-tab.html
Click on the link Open in new tab
Hit one or more buttons Home, Page1, or Page2
Go back using the browser buttons. You'll see that any intermediary pages are skipped

Now compare this with the normal behavior:

Open the URL of the opened website by entering it directly in DuckDuckGo's browser for iOS, instead of opening it in a new tab using the initial website:
https://simonbackx.github.io/push-state-example/
Hit one or more buttons Home, Page1, or Page2
Go back using the browser buttons. You'll see that everything works as expected

You can check the source at https://github.com/SimonBackx/push-state-example

Hi @SimonBackx - thanks for submitting this and sorry for not replying sooner.  We determine back/forward using the signals we get from WKWebView.
I'm not sure there's much we can do about this, unless you have any ideas? It might better to create a bug for this in the WebKit issue tracker. https://webkit.org/reporting-bugs/

As far as I know, WKWebView doesn't have any support for target _blank out of the box, so the current behaviour is probably custom. I'll look into it and if you are open for it, open a PR with a fix.

Yep, check:

https://github.com/duckduckgo/iOS/blob/develop/DuckDuckGo/TabViewController.swift#L1268

And:

https://github.com/duckduckgo/iOS/blob/develop/DuckDuckGo/TabViewController.swift#L1181

But essentially we just open a new tab with a WebView and let it carry on from there.
if you can work out what’s going, a PR would be welcome, Thanks!

Any thoughts @SimonBackx ?

Hey @brindy,
I didn't mange to setup the right provisioning profiles to sign the app for development. Probably because DuckDuckGo is using a protected entitlement (e.g. com.apple.developer.web-browser) for the browser which I can't add to my App ID's at the moment. Am I right that I need to request permission to use this entitlement by emailing<EMAIL_ADDRESS>Automatic provisioning profiles aren't working either. Do you have some steps to set this up?

@SimonBackx that's the right email but you won't get the entitlement without having an actual browser app on the App Store already - you sort of have to be proven already before you can get it.  Basically they don't want like gaming (or more malicious) apps being able to handle links and things like that so they are quite judicious about who they give it to.
Obvious thing to do is remove that entitlement, but I appreciate that's a little awkward.
@samsymons do you have any thoughts about this given you just merged a config related PR :)

Sorry for the delay! I was on vacation and didn't have time to pick this issue up afterwards. I'll try to pick this issue back up the coming days 😊


Check and update documentation and potentially stale info about Docker images [Cross filed in jupyter/jupyter]

FYI to developers working on docker-demo-images. We will be working on documentation freshness and accuracy. See details at https://github.com/jupyter/jupyter/issues/87
Any documentation update work needed on docker-demo-images can be left in this issue.

Closing this issue due to inactivity.


no dependencies found on a ruby project

This project should have over 24 dependencies but ORT found none and did not report any issues: https://github.com/cucumber/cucumber-ruby.git
log:
sudo /fossid/lib/ort/bin/ort --info analyze --allow-dynamic-versions -f JSON -i /fossid/uploads/files/scans/16 -o /fossid/uploads/files/dependency_analysis/16  2>&1
Spooled info: "________ _____________________
\\_____  \\\\______   \\__    ___\/ the OSS Review Toolkit, version 0.1.0-SNAPSHOT.
 \/   |   \\|       _\/ |    |    Running 'analyze' under Java 11.0.6 on Linux with
\/    |    \\    |   \\ |    |    ORT_USER_HOME = \/root\/.ort
\\_______  \/____|_  \/ |____|    SHELL = \/bin\/bash
        \\\/       \\\/
More environment variables:
TERM = unknown

The following package managers are activated:
	Bower, Bundler, Cargo, Conan, DotNet, GoDep, GoMod, Gradle, Maven, NPM, NuGet, PhpComposer, PIP, Pipenv, Pub, SBT, Stack, Yarn
Analyzing project path:
	\/fossid\/uploads\/files\/scans\/16
WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by retrofit2.Platform (file:\/fossid\/lib\/ort\/lib\/retrofit-2.8.1.jar) to constructor java.lang.invoke.MethodHandles$Lookup(java.lang.Class,int)
WARNING: Please consider reporting this to the maintainers of retrofit2.Platform
WARNING: Use --illegal-access=warn to enable warnings of further illegal reflective access operations
WARNING: All illegal access operations will be denied in a future release
07:45:14.826 [main] INFO  org.ossreviewtoolkit.analyzer.PackageManager$Companion$findManagedFiles$2 - Not analyzing directory '\/fossid\/uploads\/files\/scans\/16\/cucumber-ruby\/.git' as it is hard-coded to be ignored.
07:45:14.831 [main] INFO  org.ossreviewtoolkit.analyzer.Analyzer - Unmanaged projects found in:
07:45:14.831 [main] INFO  org.ossreviewtoolkit.analyzer.Analyzer - 	.
07:45:14.866 [analyzer-1] INFO  org.ossreviewtoolkit.analyzer.PackageManager - Resolving Unmanaged dependencies for '\/fossid\/uploads\/files\/scans\/16'...
07:45:15.289 [analyzer-1] WARN  org.ossreviewtoolkit.analyzer.managers.Unmanaged - Analysis of local directory '\/fossid\/uploads\/files\/scans\/16' which is not under version control will produce non-cacheable results as no version for the cache key can be determined.
07:45:15.293 [analyzer-1] INFO  org.ossreviewtoolkit.analyzer.PackageManager - Resolving Unmanaged dependencies for '16' took 0s.
Found 1 project(s) in total.
Writing analyzer result to '\/fossid\/uploads\/files\/dependency_analysis\/16\/analyzer-result.json'.
"

Requirements:
root@13c4a023cc61:/# /fossid/lib/ort/bin/ort requirements
________ _____________________
\_____  \\______   \__    ___/ the OSS Review Toolkit, version 0.1.0-SNAPSHOT.
 /   |   \|       _/ |    |    Running 'requirements' under Java 11.0.6 on Linux with
/    |    \    |   \ |    |    ORT_USER_HOME = /root/.ort
\_______  /____|_  / |____|    TERM = xterm
        \/       \/

Scanners:
	- Askalono: Requires 'askalono.linux' in version =0.4.2. Tool not found.
	- BoyterLc: Requires 'lc' in version =1.3.1. Tool not found.
	- Licensee: Requires 'licensee' in version =9.13.0. Tool not found.
	- ScanCode: Requires 'scancode' in version =3.0.2. Tool not found.

PackageManagers:
	- Bower: Requires 'bower' in version =1.8.8. Tool not found.
	* Bundler: Requires 'bundle' in version >=1.16.0 <2.2.0. Found version 1.17.3.
	- Cargo: Requires 'cargo' in no specific version. Tool not found.
	- Conan: Requires 'conan' in version =1.18.0. Tool not found.
	- GoDep: Requires 'dep' in no specific version. Tool not found.
	- GoMod: Requires 'go' in no specific version. Tool not found.
	* Npm: Requires 'npm' in version >=5.7.0 <6.15.0. Found version 5.8.0.
	- PhpComposer: Requires 'composer' in version >=1.5.0. Tool not found.
	- Pip: Requires 'pip' in no specific version. Tool not found.
	- Pipenv: Requires 'pipenv' in version >=2018.10.9. Tool not found.
	- Pub: Requires 'pub' in version >=2.2.0. Tool not found.
	- Sbt: Requires 'sbt' in version >=0.13.0. Tool not found.
	- Stack: Requires 'stack' in version >=2.1.1. Tool not found.
	* Yarn: Requires 'yarn' in version >=1.3.0 <1.23.0. Found version 1.22.4.

VersionControlSystems:
	- Cvs: Requires 'cvs' in no specific version. Tool not found.
	* Git: Requires 'git' in no specific version. Found version 2.20.1.
	- GitRepo: Requires 'repo' in no specific version. Tool not found.
	- Mercurial: Requires 'hg' in no specific version. Tool not found.

Other tools:
	* PythonVersion: Requires 'python3' in no specific version. Found version 3.7.3.
	* VirtualEnv: Requires 'virtualenv' in version >=15.1.0 <20.1.0. Found version 15.1.0.

Prefix legend:
	- The tool was not found in the PATH environment.
	+ The tool was found in the PATH environment, but not in the required version.
	* The tool was found in the PATH environment in the required version.
Usage: ort requirements [OPTIONS]

Error: Not all tools were found in their required versions.

ruby 2.5.5p157 (2019-03-15 revision 67260) [x86_64-linux-gnu]
Debian 10 (in Docker)
Result file:
analyzer-result-cucumber-ruby.json.txt

The line
WARN  org.ossreviewtoolkit.analyzer.managers.Unmanaged - Analysis of local directory '\/fossid\/uploads\/files\/scans\/16' which is not under version control will produce non-cacheable results

is quite odd, as it means ORT was not able to find any package manager file in that directory, let alone a Gemfile. Could you please double-check that the Git working tree actually has files checked out, and a Gemfile is among them?

And could you please anyway try again now that https://github.com/oss-review-toolkit/ort/pull/2497 is merged?

The Gemfile was indeed missing in the original package we were testing. It looks it's working fine now. Thanks!


Implement custom exceptions

add ursgal/exceptions.py to collect ursgal specific exceptions
raise custom exceptions in upeptide_mapper_1_0_0, if input file
contains no sequences to map
add tests that check that all peptide_mappers raise UrsgalError and
EmptyUrsgalCsvError
addresses #120

I opened the pull request against dev, since we wanted to adapt the gitflow workflow

Not sure why AppVeyor is failing, but besides that two comments:

If the top level error name is UrsgalError (which I like), then I would prefer if all sub errors start with UrsgalError, so the csv error would then be UrsgalErrorEmptyCsv.
For the upeptide mapper solution, I would not remove the output files but instead keep them as empty csvs. Otherwise it's in my opinion a weird intermediate where the engine started running, modified the json but didn't produce an output file. I think it should be either back to the original (i.e. also setting back the json to the original state) or passing through with an empty file, with the latter making more sense to me since a workflow that would allow to pass through the raised error would also expect some output file (not necessarily, I know, but could be easier). What do you think?

If the top level error name is UrsgalError (which I like), then I would prefer if all sub errors start with UrsgalError, so the csv error would then be UrsgalErrorEmptyCsv.

I would prefer to have every exception end with UrsgalError (I think especially Error should be at the end) rather than starting with that, but thats a matter of taste I guess.
But yeah, having Ursgal and Error just somewhere in the exception is probably not the best idea.

For the upeptide mapper solution, I would not remove the output files but instead keep them as empty csvs. Otherwise it's in my opinion a weird intermediate where the engine started running, modified the json but didn't produce an output file. I think it should be either back to the original (i.e. also setting back the json to the original state) or passing through with an empty file, with the latter making more sense to me since a workflow that would allow to pass through the raised error would also expect some output file (not necessarily, I know, but could be easier). What do you think?

This is also fine for me.
I will update this and add the artifacts of the tests to .gitignore.
This would also solve the failing appveyor build:
os.remove(output_file) PermissionError: [WinError 32] The process cannot access the file because it is being used by another process:

If the top level error name is UrsgalError (which I like), then I would prefer if all sub errors start with UrsgalError, so the csv error would then be UrsgalErrorEmptyCsv.

I would prefer to have every exception end with UrsgalError (I think especially Error should be at the end) rather than starting with that, but thats a matter of taste I guess.
But yeah, having Ursgal and Error just somewhere in the exception is probably not the best idea.

True, UrsgalError at the end would be even better.

For the upeptide mapper solution, I would not remove the output files but instead keep them as empty csvs. Otherwise it's in my opinion a weird intermediate where the engine started running, modified the json but didn't produce an output file. I think it should be either back to the original (i.e. also setting back the json to the original state) or passing through with an empty file, with the latter making more sense to me since a workflow that would allow to pass through the raised error would also expect some output file (not necessarily, I know, but could be easier). What do you think?

This is also fine for me.
I will update this and add the artifacts of the tests to .gitignore.
This would also solve the failing appveyor build:
os.remove(output_file) PermissionError: [WinError 32] The process cannot access the file because it is being used by another process:

Sounds good

I don't see the need for an UrsgalError. Both cases can be covered with regular python ErrorExceptions and thus all it does creates more unreadable and complex code.


REFACTOR: Remove inplace kwarg from query compiler clip() arguments

As the docs say,

This parameter serves the compatibility purpose.

https://github.com/modin-project/modin/blob/67efa96e14e7cd2339c68f90e81703f1fa755b3e/modin/core/storage_formats/base/query_compiler.py#LL797C13-L797C61
This param shouldn't go to the query compiler at all. API layer handles the inplace parameter: https://github.com/modin-project/modin/blob/67efa96e14e7cd2339c68f90e81703f1fa755b3e/modin/pandas/base.py#L1089

This parameter serves the compatibility purpose.

I no longer see this comment in the pandas docs. Looks like we should support inplace=True. @mvashishtha?
https://pandas.pydata.org/pandas-docs/version/1.5.3/reference/api/pandas.DataFrame.clip.html?highlight=clip#pandas.DataFrame.clip

This parameter serves the compatibility purpose.

@anmyachev I think I meant that this was in the documentation of the query compiler method. What I meant was that we should handle this at the API layer and the query compiler methods should not accept an inplace parameter.

@mvashishtha ok, done in https://github.com/modin-project/modin/pull/5954


Add support for deserialize

See branch https://github.com/chmp/serde_arrow/tree/feature/deserializer

Implemented in 0.4.0


Create Alfa-college.txt

@robsieljes There is similar domain which is already listed, students.alfa-college.nl. Please let us know why it's essential to add the upper-level domain.
Also, please rename the file path, there is no necessity to have uppercase letters there, it can lead to a confusion (lib/domains/nl/Alfa-college.txt -> lib/domains/nl/alfa-college.txt)
Thank you!

Hello,
Thanks for the fast reply.
The student.alfa-college.nl http://student.alfa-college.nl/ is only for students. The alfa-college domain is for teachers.
Is it possible to add that one to?
--
Met vriendelijke groet,
Rob Sieljes
Tel. 06-15215269
Mail<EMAIL_ADDRESS><EMAIL_ADDRESS>
Op 14 sep. 2016, om 11:32 heeft Philip Torchinsky<EMAIL_ADDRESS>het volgende geschreven:
@robsieljes https://github.com/robsieljes There is similar domain which is already listed, students.alfa-college.nl. Please let us know why it's essential to add the upper-level domain.
Also, please rename the file path, there is no necessity to have uppercase letters there, it can lead to a confusion (lib/domains/nl/Alfa-college.txt -> lib/domains/nl/alfa-college.txt)
Thank you!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/JetBrains/swot/pull/2287#issuecomment-246957827, or mute the thread https://github.com/notifications/unsubscribe-auth/AGGAaZs6eXU30nsHRZPiUcIYt3kQK-btks5qp79DgaJpZM4J8kXB.

@robsieljes Thank you. Absolutely. Pull request merged.

Thanks!
--
Met vriendelijke groet,
Rob Sieljes
Tel. 06-15215269
Mail<EMAIL_ADDRESS><EMAIL_ADDRESS>
Op 14 sep. 2016, om 11:36 heeft Philip Torchinsky<EMAIL_ADDRESS>het volgende geschreven:
@robsieljes https://github.com/robsieljes Thank you. Absolutely. Pull request merged.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub https://github.com/JetBrains/swot/pull/2287#issuecomment-246958641, or mute the thread https://github.com/notifications/unsubscribe-auth/AGGAadb5W6P-DEkQb4-2TACJ4NuX_EO4ks5qp8APgaJpZM4J8kXB.


download-mind.sh demo fails as downloaded resource no longer is available

Describe the bug
The file download from ./bin/download-mind.sh demo is broken
To Reproduce
Steps to reproduce the behavior:

Execute ./bin/download-mind.sh demo
Error message
Archive:  ./mind/train.zip
End-of-central-directory signature not found.  Either this file is not
a zipfile, or it constitutes one disk of a multi-part archive.  In the
latter case the central directory and zipfile comment will be found on
the last disk(s) of this archive.
unzip:  cannot find zipfile directory in one of ./mind/train.zip or
./mind/train.zip.zip, and cannot find ./mind/train.zip.ZIP, period.

Screenshots

Environment (please complete the following information):

OS: CentOS
Infrastructure: self-hosted
Versions 7

Looks like they have removed public access to the 'demo' dataset used in this sample application. The download script is light on the error checking and the downloaded file is a NoAccess message which fails with the mention unzip failure.
curl -L -o train.zip https://recodatasets.blob.core.windows.net/newsrec/MINDdemo_train.zip
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
100   248  100   248    0     0    515      0 --:--:-- --:--:-- --:--:--   515
(base) Jos-MacBook-Pro:bigann bergum$ less train.zip
(base) Jos-MacBook-Pro:bigann bergum$ cat train.zip
<?xml version="1.0" encoding="utf-8"?><Error>PublicAccessNotPermitted<Message>Public access is not permitted on this storage account.
RequestId:cceee300-f01e-008f-6177-7acf06000000

The small and large seem to be ok though.

Yes, I used the small dataset instead to go through the learning material.

Should now be fixed with the above PR.


[Feature Fix] Add wrap in pre on Firefox [OSF-3532]

Purpose
In Firefox, word-wrap: break-word will not break the line in pre-tag.  To test it, you can run the link and compare the result of firefox with any other major browsers.
resolve [OSF-3532]
Change
There are two possible solutions. One is to add new pre css and apply it to all browsers -- Link. But it need to apply all the other browser. And also it requires to override a lot of css on bootstrap.
So I tried another method that is to add firefox specific css on mfr.css.

The tricks to target css on firefox  -- Link. Add code specific for Firefox

@-moz-document url-prefix() {
  .selector {
     color:lime;
  }
}

Use white-space and word-break to mimic word-wrap in firefox  and make it as the same as Chrome and other browsers. [link] http://stackoverflow.com/questions/4282757/how-to-make-word-break-on-firefox-using-css

/* For Firefox */
white-space: pre-wrap;
word-break: break-all;

/* For Chrome and IE */
word-wrap: break-word;

Side Effect
None

CR1 complete; no changes requested.
Confirmed that this change works as intended when running locally; FF and Chrome.
Note to reviewers: to view this locally, make sure that waterbutler is updated to work with the newest osf develop branch. (changes landed earlier today)

CR2 complete. Looks good.


Authoring Gradle Builds new introduction section in Gradle User Manual

Details
This is a Documentation change ONLY.
Context

Add an Introduction section to Running Gradle Builds.
Update the Tutorial section to Running Gradle Builds.
Add an Additional Topics section to Running Gradle Builds.
Add an Introduction section to Authoring Gradle Builds.
Add a Tutorial section to Authoring Gradle Builds.
Move overflow to an Additional Topics section to Authoring Gradle Builds.

Reviewers:
Link to Build: here
Ready for review of finished sections highlighted below.
Progress:
Running Gradle Builds

[X] Getting Started
[X] Introduction
[X] Tutorial
[X] Additional Topics

Authoring Gradle Builds

[X] Introduction
[ ] Tutorial
[X] Additional Topics (nothing new here, just overflow of pages)

Notes:

The Running Gradle Builds section is for Developers that just want to use Gradle (think riding a bike - how to pedal).
The Authoring Gradle Builds section is for Build Engineers. You should assume that they already have the knowledge from the Running Gradle Builds sections. Information in those sections should NOT be repeated in these new sections. This section is about how Gradle works and how to customize Gradle (think changing the pedals on the bike).

@bot-gradle test BD

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

With regards to the new layout, couple of comments:

It might be better to have the plugins section before tasks - hmmm I'm not strongly opinionated on this though
With regards to authoring - for consistency I wouldn't use "Settings Scripts" or "Build Scripts" - either use settings file or settings configuration
In the wrapper section, maybe go into details a little about bin vs all and the wrapper properties file

For the diagrams - I find it hard to distinguish between black highlights and gray normal areas - might be worth exploring other color schemes

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@bot-gradle clean

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

@bot-gradle test BD

I've triggered the following builds for you. Click here to see all build failures.

BuildDistributions build

Moved to https://github.com/gradle/gradle/pull/26992


fix grammar

Get all the text
Put text into Grammarly
Keep the most obvious suggestions

Very nice, thanks!


Added Owl Carousel to Projects Section

Fixes Issue
Closes #35
Changes proposed
Added an owl carousel to the projects section instead of a grid
Screenshots

Note to reviewers
It is an automatic carousel which moves with time delay of 5secs

Hello and thank you for your contribution. However, I decided to keep it in the current version because it does not appear in this manner.


fix(pipelinerun): block pipelinerun spec updates once the pipelinerun has started

fix #8148
Typically, the spec field of a PipelineRun resource is not allowed to be updated after creation, such as the timeouts configuration.

Changes

Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[ ] Has Docs if any changes are user facing, including updates to minimum requirements e.g. Kubernetes version bumps
[x] Has Tests included if any functionality added or changed
[x] pre-commit Passed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[x] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings). See some examples of good release notes.
[ ] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

/kind bug
Release Notes
Fix: Once a PipelineRun is created, most of the fields in the spec is not allowed to be updated; only status can be updated.

/test check-pr-has-kind-label

/test pull-tekton-pipeline-go-coverage-df

/ok-to-test


some command can run in powershell5.1,but cant run in powershell7 ,like  ssh ;

the powershell version is ：
$PSVersionTable
Name                           Value
PSVersion                      7.1.3
PSEdition                      Core
GitCommitId                    7.1.3
OS                             Microsoft Windows 10.0.18362
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

when i use the command  ssh、get-localuser will occur error:
ssh: The term 'ssh' is not recognized as a name of a cmdlet, function, script file, or executable program. Check the spelling of the name, or if a path was included, verify that the path is correct and try again.

I suspect you installed x86 PowerShell 7 on an x64 machine.

I suspect you installed x86 PowerShell 7 on an x64 machine.

thanks a lot！！！


New web

🛠 Fixes Issue (#1377 )
Closes #1377
@swapnilsparsh  your website as you wanted .
✅ Check List (Check all the applicable boxes)

[ X] My code doesn't break any part of the project
[X ] This PR does not contain plagiarized content.
[X ] My Addition/Changes works properly and matches the overall repo pattern.
[ X] The title of my pull request is a short description of the requested changes.

📄Please mark Gssoc 24 and level 3

📷 Screenshots

https://github.com/swapnilsparsh/30DaysOfJavaScript/assets/154527778/f595f8d8-05a7-48e0-b378-8ced172e729f

@swapnilsparsh ok

@swapnilsparsh is this correct
#red shadow on hover

@swapnilsparsh please cheak this

@swapnilsparsh I modified web page as instructed and add all 112 games
https://github.com/swapnilsparsh/30DaysOfJavaScript/assets/154527778/07fa5f54-3c11-4630-a4f3-c626af4a5bae

@swapnilsparsh please mark this  Gssoc and level 3

@swapnilsparsh please review and merge with Gssoc and level 3 label

@pandeyji711

@swapnilsparsh ok

now img is working fine

@swapnilsparsh all changes are done please review

@swapnilsparsh please review and merge

@swapnilsparsh ### please review and merge with Gssoc and level 3 label

@swapnilsparsh please review this

@swapnilsparsh please review i have fixed the  issue

Still this is not fixed (You don't have to define onClick on every button as you have to use tag instead)
You don't have to remove pagination.

Still this is not fixed. @pandeyji711

@swapnilsparsh ok gotit fixing pagination

@swapnilsparsh please review i implimented all the  changes that you asked for and mark it for Gssoc24 level 3
fix #1377

@swapnilsparsh please review

@swapnilsparsh please review and merge , mark Gssoc 24 level 3

@pandeyji711 Everything is working fine just fix this

@pandeyji711 Also the footer should stick to bottom

@swapnilsparsh    Ok

@pandeyji711 Footer is not fixed yet and resolve the conflicts also where every project description is added.

@swapnilsparsh I created Pr with other branch https://github.com/swapnilsparsh/30DaysOfJavaScript/pull/1559
becouse this gives conflict