Error interpreting sql statements with blank lines

Hello, this topic was treated in this thread: http://dbeaver.jkiss.org/forum/viewtopic.php?f=2&t=669, although I don't know how to solve it.
This is a silly example (with postgres):
select *

from objeto;

returning this error:
SQL Error [42601]: ERROR: error de sintaxis en o cerca de «from»
  Position: 1

In Window->Preferences->Editors->SQL Editor->SQL Processing, Statement delimiter is set ; but I don't found anything about don't use blank lines as statement delimiter too.
Is it possible ignore blank lines as statement delimiter?
It is very useful when writing multiples PL procedures in the same script because I separate some of its code with blank lines to be more readable, and I want to execute only one of this statements (execute as statement not as script).
In Oracle sqlplus/sqldeveloper, the character / is used to specify the end of statement (function, trigger, anonymous block, etc).
Regards.

Currently statement execution command (ctrl+enter) threats blank lines as statement delimiter. At the moment there is no workaround for that. Mostly because some databases works that way. Also because it is simply convenient.
But you can select your statement (with any number of blank lines/special characters inside) and execute it (ctrl+enter).
Also you can execute it as script (alt+x) - in this case only statement delimiter (; by default) will be used as delimiter.
I could add some additional config option (like "Use blank line as statement delimiter") for SQL processing.. WDYT?

It should be wonderful. For example, for execute statement (CTRL+Intro), specify group of chars (Ex.: \n, ;, /, ...).
And there is same special cases (ex. Postgresql) where you can create dollar quoted string:
create function f() returns varchar as $$
    ...
$$ language ...;

$$ ... $$ is a string. But it is possible something like $any_text$ ... $any_text$. This is a particuarity of Postgresql, and I think dbeaver doesn't understand very well.

Hello, this might be a solution:
in org.jkiss.dbeaver.ui.TextUtils.java, add a new method to check dollar string delimiter:
    public static String[] hasDollarStringDelimiter(IDocument document, int line) throws BadLocationException {
        /*
         * It returns an String array. The first element is type of dollar string delimiter
         * (O: Open string delimiter, C: Close string delimiter). The second element is
         * the delimiter string (examples: $$, $body$, ...).
         * If there not found delimiter string in this line, it returns null.
         *
         * A string delimiter must be at the end of the line (if it is opening),
         * or at the begin of the line (if it is closing).
         *
         * Example:
         *    create ... $body$
         *      ...
         *    $body$ ....
         */
        IRegion region = document.getLineInformation(line);
        if (region == null || region.getLength() == 0) {
            return false;
        }

        String str = document.get(region.getOffset(), region.getLength());
        String matchStringBeg = "^[ \\t]*(\\$[a-zA-Z0-9]*\\$)([ \\t].*)?$";
        String matchStringEnd = "^.*[ \\t](\\$[a-zA-Z0-9]*\\$)[ \\t]*$";
        if (str.matches(matchStringBeg))
            return (String[]) {"O", str.replaceAll(matchStringBeg, "$1"))};
        else if (str.matches(matchStringEnd))
            return (String[]) {"C", str.replaceAll(matchStringEnd, "$1"))};
        else
            return null;
    }

and a modify in org.jkiss.dbeaver.ui.editors.sql.SQLEditorBase.java the method extractQueryAtPos:
    public SQLQuery extractQueryAtPos(int currentPos)
    {
        Document document = getDocument();
        if (document == null || document.getLength() == 0) {
            return null;
        }
        int docLength = document.getLength();
        IDocumentPartitioner partitioner = document.getDocumentPartitioner(SQLPartitionScanner.SQL_PARTITIONING);
        if (partitioner != null) {
            // Move to default partition. We don't want to be in the middle of multi-line comment or string
            while (currentPos < docLength && !isDefaultPartition(partitioner, currentPos)) {
                currentPos++;
            }
        }

        // Extract part of document between empty lines
        int startPos = 0;
        int endPos = document.getLength();
        try {
            int currentLine = document.getLineOfOffset(currentPos);
            int lineOffset = document.getLineOffset(currentLine);
            int linesCount = document.getNumberOfLines();

            // First search for dollar quoted strings.
            int firstLineDollar = currentLine, lastLineDollar = currentLine;
            int firstLine, lastLine;
            String[] dollarDelimiterOpen = null, dollarDelimiterClose = null;
            while (firstLineDollar > 0) {
                dollarDelimiterOpen = TextUtils.hasDollarStringDelimiter(document, firstLine);
                if (dollarDelimiterOpen != null) {
                  if (dollarDelimiterOpen[0] != "O") {
                      // It is not an open delimiter.
                      dollarDelimiterOpen = null;
                  }
                  break;
                }
                firstLineDollar--;
            }
            if (dollarDelimiterOpen != null) {
                while (lastLineDollar < linesCount) {
                    dollarDelimiterClose = TextUtils.hasDollarStringDelimiter(document, lastLine);
                    if (dollarDelimiterClose != null) {
                        if (dollarDelimiterClose[0] != "C") {
                            // It is not a close delimiter.
                            dollarDelimiterClose = null;
                        }
                        break;
                    }
                    lastLineDollar--;
                }
            }
            // Chek if there found valids strings delimiters.
            if (
                (dollarDelimiterOpen != null) &&
                (dollarDelimiterCloe != null) &&
                (dollarDelimiterOpen[1] == dollarDelimiterClose[1])
            ) {
                // They was found. Use they positions as new firstLine, lastLine.
                firstLine = firstLineDollar;
                lastLine = lastLineDollar;
            } else {
                // They was not found, so check only for blank lines.
                firstLine = currentLine;
                lastLine = currentLine;
            }

            // Search for blank lines.
            while (firstLine > 0) {
                if (TextUtils.isEmptyLine(document, firstLine)) {
                    if (isDefaultPartition(partitioner, document.getLineOffset(firstLine))) {
                        break;
                    }
                }
                firstLine--;
            }
            while (lastLine < linesCount) {
                if (TextUtils.isEmptyLine(document, lastLine)) {
                    if (isDefaultPartition(partitioner, document.getLineOffset(lastLine))) {
                        break;
                    }
                }
                lastLine++;
            }
            if (lastLine >= linesCount) {
                lastLine = linesCount - 1;
            }
            startPos = document.getLineOffset(firstLine);
            endPos = document.getLineOffset(lastLine) + document.getLineLength(lastLine);

            // Move currentPos at line begin
            currentPos = lineOffset;
        } catch (BadLocationException e) {
            log.warn(e);
        }
        return parseQuery(document, startPos, endPos, currentPos);
    }

I haven't test yet because I have not deply the project in any IDE.
What do you think?

Interesting, thanks. However I thought about modifying SQL syntax parser rules (extending delimiter rule). And also modifying extractQueryAtPos to make "blank lines as delimiters" configurable.
Don't forget about multi-database nature of SQL editor. All database specific stuff should be in SQLDialect/driver configuration. Dollar signs as delimiters are definitely not standard and should be somehow handled by PostgreSQL extension..

Will be added in the next version

Where is this setting? Searching the preferences screen for "blank lines" returns no result, also tried other combinations of related words/phrases.

just for reference there is now a setting
Windows --> Preferences --> Editors --> SQL Editor --> SQL Processing --> Delimiters --> Blank line is a statement delimiter


Arguments not following standard integration

I've noticed that Firefox is the only browser which has this behavior enable by default. The thing is that inside the window.scroll declaration it expects an object while it still gives supports for the old two number arguments.
window.scroll({
    top: 120,
    left: 20,
   behavior: 'smooth'
});

Since Firefox is expecting this, the script will return because it finds that smoothBehavioris implemented and never work.
If you are ok with this, I can create a pull request with all the necessary changes to adapt your script to this type of call which I guess is the standard one instead of the actual that doesn't have an options object.

So what's next... I found Firefox odd too.


Single quote the string predicates on the finder and consider other types

What?

Commented here: https://github.com/SugarRecord/SugarRecord/pull/87#issuecomment-67567564
Just how we have implemented the finder if you want to filter using an string you have to single-quote it which is something most of users forget about.
We could split the by() method into:
by(name: String, stringValue: String)
by(name: String, intValue: String)

Steps

[ ] Add methods to the SugarRecordObjectProtocol
[ ] Add methods to the Finder
[ ] Test SugarRecord objects methods:

[ ] Objective-C
[ ] Realm

[ ] Teste SugarRecordFinder methods:
Note: Basically the tests would consist on  checking if the predicate has the right format

Initially we had migrated from 1.0.3 to 1.0.6 to fix the persistant store on save() call as opposed to move to background...
func getItemById(value:String) -> ItemVO?{        let items = ItemVO.by("id", equalTo: value).sorted(by: "id", ascending: true).find()         let item = items.firstObject() as ItemVO!         return item }
This caused an error which was fixed by accepting the param as an NSString

There's another user reporting problems with Single Quotes: https://groups.google.com/forum/?utm_medium=email&utm_source=footer#!msg/sugarrecord/P5_qtKeWmic/HM_UIPLmc7oJ


missing x-ms-parameter-location: method for Parameters in policyDefinitionVersions.json

https://github.com/Azure/azure-rest-api-specs/pull/24542 introduces new api-version for Microsoft.Authorization. However, compared with existing definition, it misses x-ms-parameter-location: method for PolicyDefinitionName/PolicyDefinitionVersion of policyDefinitionVersions.json and PolicySetDefinitionName of policySetDefinitionVersions.json, which causes client signature breaking of SDK:

@calecarter Please help have a look, thank you.

The issue is fixed by https://github.com/Azure/azure-rest-api-specs/pull/29345/files


Wrong implementation of DEFAULT_ERROR_LIST in AuthenticationExceptionHandler

Hi,
I have a little suggestion of AuthenticationExceptionHandler  implementation.  As you can see at https://github.com/apereo/cas/blob/master/core/cas-server-core-authentication/src/main/java/org/apereo/cas/web/flow/AuthenticationExceptionHandler.java#L55
order of DEFAULT_ERROR_LIST is very important, so the question is why you have used HashSet. From the java doc:

Class HashSet
This class implements the Set interface, backed by a hash table (actually a HashMap instance). It makes no guarantees as to the iteration order of the set; in particular, it does not guarantee that the order will remain constant over time.

You're right. That's an oversight. Turn your suggestion to a PR and we can merge it. Thanks.


Output column names consistently using fieldnameAuthtypeAuthsubtype pattern

Relates to: https://collectionspace.atlassian.net/browse/DRYD-991
After doing this, re-generate CSV templates and RecordMappers.
Then test that collectionspace-csv-importer works as expected with the new format

Fixed by #75


[Bug]:  Browse OTU - disable/hide 'Browse current OTU' when there is no page available [Was:  No valid OTU page for valid taxon name Coccophagus funiculatus]

Steps to reproduce the bug
There is only a combination page for the OTU Coccophagus funiculatus. It appears otherwise correct. When you click on browse current OTU it takes you back to the Browse OTU home page.

Screenshot
No response
Expected behavior
No response
Additional Screenshots
No response
Environment

[ ] Development (native)
[x] Development (docker)
[ ] Sandbox
[ ] Production

Sandbox Used
No response
Version
No response
Browser Used
No response

@jlpereira In this case there is no OTU for the  taxon_name of the valid id of this Combination.  We need to detect this and disable the blue button when it is the case.  Let me know if you need a response tweak.


Kafka Testcontainer should use bitnami/kafka image, or at least allow it as compatible

Module
Kafka
Proposal
Bitnami image has standard kafka version numbers and it's an standard distributor unlike confluent, a private company (who for instance, discontinued their widely used helmchart suddently in favour of a private delpoyment mechanism).
If the blocker was that bitnami kafka images did not support arm arch, they do from kafka 3.2
But at least, we would appriacte being able to make bitnami/kafka compatible as kafkacontainer, because right now it cant be used as image

Hi, have you tried using DockerImageName.parse("bitnami/kafka").asCompatibleSubstituteFor("confluentinc/cp-kafka")?

I think at least allow it as compatible would be better.
Ignoring confluentinc/cp-kafka completely is tricky because issues like https://issues.apache.org/jira/browse/KAFKA-7249 prove that most developers who want to use Kafka’s Docker Image will only Consider confluentinc/cp-kafka.

For now, there is no plans to support other images.


Removed Functional Test functional:productV2

Questions
Answers

Description?
Removed Functional Test functional:productV2

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?
N/A

Sponsor company
@PrestaShopCorp

How to test?
CI tested

@boubkerbribri It's merged


minor fix in e2e test

Signed-off-by: ashish<EMAIL_ADDRESS>Description
This PR fixes #
Notes for Reviewers
Signed commits

[x] Yes, I signed my commits.

Codecov Report

Merging #4843 (f2ec6ef) into master (48d07dd) will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #4843   +/-   ##
======================================
  Coverage    9.31%   9.31%
======================================
  Files          61      61
  Lines        5151    5151
======================================
  Hits          480     480
  Misses       4553    4553
  Partials      118     118

Flag
Coverage Δ

gointegrationtests
9.31% <ø> (ø)

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

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 9fee9fb...f2ec6ef. Read the comment docs.


git push fails on missing access key

core@micro-kube ~/example-go $ git push deis master
Counting objects: 3, done.
Compressing objects: 100% (3/3), done.
Writing objects: 100% (3/3), 267 bytes | 0 bytes/s, done.
Total 3 (delta 1), reused 0 (delta 0)
cat: /var/run/secrets/object/store/access-key-id: No such file or directory
To ssh://git@<IP_ADDRESS>:2222/ultima-waggoner.git
   5e2c829..550efea  master -> master

If your builder RC has volume mounts in the spec, You have to create secrets first and then deploy builder.

yea, I should have commented here - helm was not doing that properly. The patch to fix was merged: https://github.com/deis/charts/pull/36


Disabling "Stack Overrun checking" in RTX_config.h leads to linker error "L6218E: Undefined symbol osRtxThreadStackCheck (referred from irq_armv7m.o)."

The line 142 of irq_armv7m.S of CMSIS v6.0.0 pack uses osRtxThreadStackCheck without checking if the option "Stack Overrun checking" in RTX_config.h is enabled or not.
Other irq_armvxx.S may be also affected.

Hi Chen,
I can see the #ifdef RTX_STACK_CHECK here
https://github.com/ARM-software/CMSIS-RTX/blob/7b056a242e98587b66eb7c1c3a73ff2ab6cb0539/Source/GCC/irq_armv7m.S#L130-L142
Can you provide more details about how to reproduce your link issue?

Hi Jonatan,
you are right. I have overlooked that part. irq_armv7m.S is correct.
As I just show you in person, this is rather a Keil MDK µVision IDE problem.
The way to reproduce this issue:

Building an existing CMSIS-RTOS blinky example with the "Stack overrun checking" option enabled once.
Disabling the "Stack overrun checking" option
Click Build, instead of "Rebuild", the linker error above will occur.

In this case actually MDK µVision failed detecting the change on RTX_Config.h should lead to a rebuild of irq_armv7m.S module.
Making a complete "Rebuild" of the project can resolve the linker error above.
Thus, closing this issue


Refactor AbstractGenerator.php

Change If nesting conditions
Fix StyleCI issue

[Current coverage][cc-pull] is 86.09%

Merging [#35][cc-pull] into [master][cc-base-branch] will decrease coverage by 0.03%

@@             master        #35   diff @@
==========================================
  Files             5          5
  Lines           411        410     -1
  Methods          29         29
  Messages          0          0
  Branches          0          0
==========================================
- Hits            354        353     -1
  Misses           57         57
  Partials          0          0

Powered by Codecov. Last updated by [74f38a1...1f09db3][cc-compare]
[cc-base-branch]: https://codecov.io/gh/mpociot/laravel-apidoc-generator/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/mpociot/laravel-apidoc-generator/compare/74f38a1c21bf7200b2eb94e9c2568e8c029d32ea...1f09db38663f02193c47248a2ba52a3af980c36a
[cc-pull]: https://codecov.io/gh/mpociot/laravel-apidoc-generator/pull/35?src=pr


feature: telemetry

As mentioned in the UVP document, let's set up telemetry to collect usage information which helps to make RisingWave better. Example: https://docs.pingcap.com/tidb/dev/telemetry

Is it like Diagnostics reporting in Cockroach DB? 👀
https://www.cockroachlabs.com/docs/stable/diagnostics-reporting.html

Is it like Diagnostics reporting in Cockroach DB? 👀 https://www.cockroachlabs.com/docs/stable/diagnostics-reporting.html

yes, similar

Moved to next release.


Fix for class level validation (recursive adapter creation)

Specifically we don't want the .andThen(..) for the class level validation as then its recursive

Also note the path and field values are "null" and null currently:
  @Test
  void invalid_classLevelValidation() {
    var violation = one(new ATarnished("ok", 49, 50));
    assertThat(violation.message()).isEqualTo("put these foolish ambitions to rest");
    assertThat(violation.path()).isEqualTo("null");
    assertThat(violation.field()).isNull();
  }


Changing the method of disabling perms

To disable a permission that is given to default players you have to use that method.

What's wrong with unsetting the permission?
I mean you're disabling the permission from being used for that rank.. so you should be removing it from the 1PermisisonAttachment` not setting it to false

I'm going to close this as I don't think this does anything.


convertFahrenheitToCelsius in python

Why:

Closes #5027
What's being changed:

@anandfresh I just did


how to stop utarm_mvservo_trajectory from interfering with gazebo

Hi,
the moveit example doesn't work
when opening the rivz and gazebo without connecting the robot the model loads good

but if I tried any plan it will fail and stop mid way.
if you see the (end) right of the rqt graph there is gazebo and utarm_mvservo_trajectory, both of them are trying to communicate with move_group

and the terminal for rivz are outputting in high speed:
[ WARN] [1702377795.979577824,<PHONE_NUMBER>00]: processStatus: Previously received status from [/gazebo], but we now received status from [/utarm_mvservo_trajectory]. Did the ActionServer change?
[ WARN] [1702377796.052882026,<PHONE_NUMBER>00]: processStatus: Previously received status from [/utarm_mvservo_trajectory], but we now received status from [/gazebo]. Did the ActionServer change?
[ WARN] [1702377796.187705629,<PHONE_NUMBER>00]: processStatus: Previously received status from [/gazebo], but we now received status from [/utarm_mvservo_trajectory]. Did the ActionServer change?
[ WARN] [1702377796.257843731,<PHONE_NUMBER>00]: processStatus: Previously received status from [/utarm_mvservo_trajectory], but we now received status from [/gazebo]. Did the ActionServer change?
[ WARN] [1702377796.386407734,<PHONE_NUMBER>00]: processStatus: Previously received status from [/gazebo], but we now received status from [/utarm_mvservo_trajectory]. Did the ActionServer change?
[ WARN] [1702377796.457298436,<PHONE_NUMBER>00]: processStatus: Previously received status from [/utarm_mvservo_trajectory], but we now received status from [/gazebo]. Did the ActionServer change?
[ WARN] [1702377796.590359939,<PHONE_NUMBER>00]: processStatus: Previously received status from [/gazebo], but we now received status from [/utarm_mvservo_trajectory]. Did the ActionServer change?
[ WARN] [1702377796.662512541,<PHONE_NUMBER>00]: processStatus: Previously received status from [/utarm_mvservo_trajectory], but we now received status from [/gazebo]. Did the ActionServer change?
[ WARN] [1702377796.835774545,<PHONE_NUMBER>00]: processStatus: Previously received status from [/gazebo], but we now received status from [/utarm_mvservo_trajectory]. Did the ActionServer change?
and I think because of the 2 ActionServer racing, the execution of plans get failed and the moveit script file generates:
[ INFO] [1702377277.165643199]: Loading robot model 'utra6_850'...
[ INFO] [1702377277.169364599]: No root/virtual joint specified in SRDF. Assuming fixed joint
[ INFO] [1702377279.107153772, 93.896000000]: Ready to take commands for planning group manipulator.
[ INFO] [1702377279.297062779, 94.076000000]: ABORTED: CONTROL_FAILED
[ INFO] [1702377282.385252694, 97.071000000]: ABORTED: CONTROL_FAILED
[ INFO] [1702377282.596100302, 97.276000000]: ABORTED: CONTROL_FAILED
so what should I do?

the problem is gone after I killed the /utarm_mvservo_trajectory
but I don't know what will that effect If I connected the robot.

Hello
What instructions were run?
Run in Rviz and Gazebo simulator if you don't have the manipulator hardware

Hi,
I don't have hardware, yet.
I am using gazebo and Rviz, and Rvis is using moveit -as I understand- on ros neotic on ubuntu20.04 wsl.
the instructions run/steps to repreduce:
roslaunch utra6_850_gazebo gazebo.launch # launches gazebo and holds a terminal for warnings.
roslaunch utra6_850_moveit_config run_with_utra6_850.launch # launches rviz and moveit. and holds terminal for warnings.
# move the sphere to a new position
# press plan and succeed
# press execute to see the gazebo simulation starts moving -might even reach the point you desired- but the Rviz will output Failure state under the "Plan and Execute button" button.
# if you move the sphere to an another position and press plan you will see the Rivz planning from the home position to the new position -not from the old position to the new position-. if you pressed execute the gazebo will return to the home position first before moving to the new position.
# the terminal for rivz output the warnings at a very high speed as shown in the first post, that there is /gazebo and /utarm_mvservo_trajectory  are trying to send to the same rosnode.
#outputs:

[ WARN] [1702441271.301199113, 1177.462000000]: processStatus: Previously received status from [/utarm_mvservo_trajectory], but we now received status from [/gazebo]. Did the ActionServer change?
[ WARN] [1702441271.466351308, 1177.627000000]: processStatus: Previously received status from [/gazebo], but we now received status from [/utarm_mvservo_trajectory]. Did the ActionServer change?
[ WARN] [1702441271.501124307, 1177.662000000]: processStatus: Previously received status from [/utarm_mvservo_trajectory], but we now received status from [/gazebo]. Did the ActionServer change?
[ WARN] [1702441271.676016402, 1177.826000000]: processStatus: Previously received status from [/gazebo], but we now received status from [/utarm_mvservo_trajectory]. Did the ActionServer change?
[ WARN] [1702441271.714003901, 1177.861000000]: processStatus: Previously received status from [/utarm_mvservo_trajectory], but we now received status from [/gazebo]. Did the ActionServer change?
[ WARN] [1702441271.886034096, 1178.026000000]: processStatus: Previously received status from [/gazebo], but we now received status from [/utarm_mvservo_trajectory]. Did the ActionServer change?
[ WARN] [1702441271.922637295, 1178.062000000]: processStatus: Previously received status from [/utarm_mvservo_trajectory], but we now received status from [/gazebo]. Did the ActionServer change?
[ WARN] [1702441272.096813690, 1178.226000000]: processStatus: Previously received status from [/gazebo], but we now received status from [/utarm_mvservo_trajectory]. Did the ActionServer change?
[ WARN] [1702441272.136324789, 1178.260000000]: processStatus: Previously received status from [/utarm_mvservo_trajectory], but we now received status from [/gazebo]. Did the ActionServer change?
[ WARN] [1702441272.323913184, 1178.426000000]: processStatus: Previously received status from [/gazebo], but we now received status from [/utarm_mvservo_trajectory]. Did the ActionServer change?

# then trying to execute the script:
rosrun utra_ros_moveit_demo moveit_fk_demo # launches the ROS script (from https://github.com/umbratek-application/utra_ros_moveit_demo)

# outputs:
[ INFO] [1702441197.595994756]: Loading robot model 'utra6_850'...
[ INFO] [1702441197.600717955]: No root/virtual joint specified in SRDF. Assuming fixed joint
[ INFO] [1702441199.212345005, 1107.083000000]: Ready to take commands for planning group manipulator.
[ INFO] [1702441199.526976995, 1107.384000000]: ABORTED: CONTROL_FAILED
[ INFO] [1702441202.895197491, 1110.676000000]: ABORTED: CONTROL_FAILED
[ INFO] [1702441203.438504275, 1111.182000000]: ABORTED: CONTROL_FAILED

uninstalling and installing everything ros related solved it.


badge size

for me, the badge is too small, could you make function about its size?
thx, btw great lib :)

Hello. Thanks for feedback. Wait for improvments.

thanks....

Update 1.2.3 is on the rails.


Context-aware filters.

 This should allow players to use one filter for all of their pawns and mitigate issues like the one presented in #43.
Possible solution
No response
Details
No response

Out of scope.


Change Japanese explanatory texts simplify

Now, explanatory texts are too difficult to understand.

I didn't find it useful because it's not all in enlgish

@geirman
Thank you for your comment.
Have you try CHOOSE LANGUAGE?
You can change languages by this command.

I didn't notice it. I'll check it out again when I have a sec. Thanks for the tip :)


New package: ScTenifold v0.1.0

Registering package: ScTenifold
Repository: https://github.com/jamesjcai/ScTenifold.jl
Created by: @jamesjcai
Version: v0.1.0
Commit: 202c96c428aa1264fd96e7ce7e16bb7c1a288344
Git reference: main

@JuliaRegistrator register

@JuliaRegistrator register

Closing in favor of https://github.com/JuliaRegistries/General/pull/27721

Closing in favor of https://github.com/JuliaRegistries/General/pull/27721


Criando arquivo index

/cc

/assign  @devlware
I made some fixes, as requested.

I made some fixes, as requested.
/assign  @devlware

Hey @DonatoHorn,
It's been some day since last update, would you like to address issues in the previous comments so that we can move forward?

Hey @DonatoHorn,
It's been some day since last update, would you like to address issues in the previous comments so that we can move forward?

Sorry for the delay, it's ready for me.

Hi @DonatoHorn! Did you open a separate PR for this content?

Moved to pr 35520


[v1.7.0 deprecation] Remove features marked deprecated in #2993

#2993 (health check: add Redis custom health checker) introduced a deprecation notice for v1.7.0. This issue tracks source code cleanup.

@alyssawilk should I remove the warning logging only? Since I'm not sure we want to remove "RedisHealthCheck" health checker completely.
https://github.com/dio/envoy/blob/b76a69ccb73d7b6abc5e5e48bb38a92744a38ed3/source/common/upstream/health_checker_impl.cc#L57-L59

@dio I think the idea is to deprecate the redis specific config, and only allow driving it via the custom health checker config.


Update akka-http, akka-http-jackson to 10.2.9

Updates

com.typesafe.akka:akka-http
com.typesafe.akka:akka-http-jackson

from 10.2.8 to 10.2.9.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.typesafe.akka" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #736.


Fix make check on OS X

Resolve issues with make check on OS X

I'm gonna give @milancurcic a few hours to say "LGTM" or  comment on required changes, before merging.

LGTM!! :smile:

OK, I'm going to double check that the test program will exit non-zero on failure, and then if not, add stop 1 or something similar, because that releases me of the burden of parsing the output for the Homebrew formula test. If  there's already a stop n statement upon failure (with 0 < n < 255) then I'll mint a new release and submit my homebrew PR which I'm ready to pull the trigger on.

@zbeekman Take a look at the report_tests subroutine in src/tests/datetime_tests. This subroutine returns to a parent subroutine which returns to the end of the program right away. So I think we need this:
SUBROUTINE report_tests(tests)

  LOGICAL,DIMENSION(:),INTENT(IN) :: tests

  INTEGER :: n,ntests,nsuccess,nfailure

  ntests = SIZE(tests)

  nsuccess = 0
  nfailure = 0
  DO n = 1,ntests
    IF(tests(n))THEN
      nsuccess = nsuccess + 1
    ELSE
      nfailure = nfailure + 1
    ENDIF
  ENDDO

  WRITE(UNIT=STDOUT,FMT='(A,I3,A)')'Ran a total of ',ntests,' tests.'
  WRITE(UNIT=STDOUT,FMT='(I3,A,I3,A)')nsuccess,' tests PASSED, ',nfailure,' tests FAILED.'

  ! >>> Suggested code to add <<<
  IF(nfailure > 0)THEN
    STOP 1
  ELSE
    STOP 0
  ENDIF

ENDSUBROUTINE report_tests

What do you think?

I may have not been clear. There is currently no stop statements in the test program. I suggest we add this snippet:
  IF(nfailure > 0)THEN
    STOP 1
  ELSE
    STOP 0
  ENDIF

I did something similar, but less likely to break backwards compatibility: I added an optional logical argument to that procedure and then call stop in the main test subroutine.

@milancurcic Is what I have OK, or do you prefer your way?

Yours is perfect!

I'm going to mint a release and then open the Homebrew PR :beers:

:tada: :balloon: Awesome, it's beer time! :beers:

haha, well, :beers: is a homebrew logo of sorts... but yes, I'm having a lame Saturday night... well I'm having fun, but most would think it lame.

Feel free to edit the release notes, @milancurcic...


Can't use v-bind in css in background-image: url();

Version
3.2.31
Reproduction link
sfc.vuejs.org/
Steps to reproduce
use v-bind in css in background-image: url();
but both the local src and the remote src will report an error.
What is expected?
can use v-bind in css in background-image
What is actually happening?
Syntax Error: ModuleNotFoundError: Module not found: Error: Can't resolve './var(--c9asdsad-src' in 'E:\Code\xxx'

should be
const remoteSrc = 'url(https://cdn4.buysellads.net/uu/1/41334/1550855401-cc_light.png)'

see sfc

oh, it's my problem. Thanks for your reply.


Need help overriding styles

Hi,
I'm trying to override some of the tree styles, like I tried adding:

to my component template.  But it's not working.
Any suggestions on how to override tree styles?

Hey,
no reason it shouldn't work, I'm using ViewEncapsulation.None everywhere.
Can you share your code?

        Organizational Metadata Tree

  <Tree #tree
        class="metatree"
        [nodes]="nodes"
        [options]="customTemplateStringOptions"
        (onActivate)="selectNode($event)"
        (onDeactivate)="deselectNode($event)">
    <template #treeNodeTemplate let-node>
      <node [node]="node"
            (createChildNode)="addChildNode($event)"
            (deleteChildNode)="deleteNode($event)"></node>
    </template>
  </Tree>

  <selected-metadata [selectedMetadata]="selectedNode">
  </selected-metadata>

  <files></files>

Hey, you should add styles to your component using the styles attribute,
not using
Organizational Metadata Tree
<Tree #tree
class="metatree"
[nodes]="nodes"
[options]="customTemplateStringOptions"
(onActivate)="selectNode($event)"
(onDeactivate)="deselectNode($event)">
<template #treeNodeTemplate let-node>
<node [node]="node"
(createChildNode)="addChildNode($event)"
(deleteChildNode)="deleteNode($event)">

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/500tech/angular2-tree-component/issues/171#issuecomment-280784547,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA2SSlNl4z8U1mf99o4HuDT6c7yzKfsTks5rdh-PgaJpZM4L94JO
.

@adrian-mitangi any news?

Hi,
I'll give it a try today and report back to you.  Thanks for following up.
Adrian

Awesome. I'll close the issue and if it doesn't work let me know


Use the function (closure) form of tweens

Use the simpler function / closure form for implementing a Pyramid tween, rather than the class form.
The class form is a little dodgy:

You should avoid mutating any state on the tween instance. The tween is invoked once per request and any shared mutable state needs to be carefully handled to avoid any race conditions.
The closure style performs slightly better and enables you to conditionally omit the tween from the request processing pipeline (see the following timing tween example), whereas the class style makes it easier to have shared mutable state and allows subclassing.
https://docs.pylonsproject.org/projects/pyramid/en/latest/narr/hooks.html?highlight=add_tween#creating-a-tween

Also split things up into four separate, small tweens that each do one thing rather than two big tweens that do multiple things. This makes a couple of things simpler:

We can conditionally add or not add each tween depending on which features are enabled by the settings. So the setting-based logic can now all live in one place includeme() rather than being split between the includeme() and the tweens themselves.

The tweens themselves are now dirt simple and each do only one job. No more settings logic in the tweens. One tween per feature

It's no longer necessary for includeme() to add the SanitySettings object to the registry, because the tweens no longer need it

SanitySettings.ingress_required and SanitySettings.egress_required are no longer needed

This actually seems to break the extension. Now when I do http 'http://<IP_ADDRESS>:5000/?f%FC=123' to h it's 500ing instead of returning Bad Request. The extension is raising InvalidQueryString though


⚠ Jenkins Plugins Artifactory has degraded performance

In f61adba, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1281 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 18953c7 after 27 minutes.


Vapor fails to open AdamPhillips dataset

To reproduce on OSX and CentOS:

Open a source build of Vapor and load the following VDC
/glade/p/cisl/vast/vapor/data/VDC/vapor-3.5.0/CAM/AdamPhillips/F2000C5_new120_025/F2000C5_new120_025-3.5.0.nc

Error:
Failure to open data set of type "vdc"
Failed to initialize horizontal coordinates
Failed to initialize derived coord variable
Invalid map projection : +proj=eqc +ellps=WGS84 +lon_0=-504.5 +lat_0=383.5
pj_init_plus(+proj=eqc +ellps=WGS84 +lon_0=-504.5 +lat_0=383.5) : lat_0, lat_1 or lat_2 >= 90
------------------
Vapor 3.5.0.<debug>
OS: Mac OS X 11.6.0

Alternatively, run step 1 with the recent installer build on 06ab5f4.  Vapor will crash instead of throwing an error.

I believe that this is a duplicate of #2982 and has been fixed on main

@clyne I'm still seeing it on HEAD bbff02844468db022eeb43feba3c363d292725e0 :\


Added support for a yotta_plugins folder containing command modules

Also moved a copy of yt out to the root yotta directory to allow execution from within the cloned repository folder (temporary and unnecessary for deployment)
@karlj please review.

Just the one thought. Otherwise it looks good.

👍


Removed IbmDb2 adapter

This storage is not defined in the dependent library so it is not possible to use this adapter. As it is dead code it should be removed.

[x] Is this related to quality assurance?

This repository has been closed and moved to laminas-api-tools/api-tools-oauth2; a new issue has been opened at https://github.com/laminas-api-tools/api-tools-oauth2/issues/3.

This repository has been moved to laminas-api-tools/api-tools-oauth2. If you feel that this patch is still relevant, please re-open against that repository, and reference this issue. To re-open, we suggest the following workflow:

Squash all commits in your branch (git rebase -i origin/{branch})
Make a note of all changed files (`git diff --name-only origin/{branch}...HEAD
Run the laminas/laminas-migration tool on the code.
Clone laminas-api-tools/api-tools-oauth2 to another directory.
Copy the files from the second bullet point to the clone of laminas-api-tools/api-tools-oauth2.
In your clone of laminas-api-tools/api-tools-oauth2, commit the files, push to your fork, and open the new PR.
We will be providing tooling via laminas/laminas-migration soon to help  automate the process.


Calling scalar on QNITensor with no blocks gives error

Currently, if a scalar (order-0) QNITensor T has no blocks (nzblocks(T)==0), then calling scalar(T) or T[] results in an error. Would it not be better to have scalar(T) return 0.0 for this case?
Example to reproduce:
s = siteinds("S=1/2",4;conserve_qns=true)
A = ITensor(s[1]'',dag(s[1]'))
B = ITensor(s[1]',dag(s[1]''))
A[1,1] = 1.0
B[2,2] = 1.0
R = A*B
@show scalar(R) #gives an error

Yeah that sounds right.


v4 Tooltip/Popover auto-placement left/right, then top/bottom

I think it would be advantageous if the "auto" positioning algorithm checked all 4 directions, rather than just 2.  Right now, if I set a popover's placement to "auto right" and the element is on the right side of the page, then the popover will be positioned to the left of the element.  However, if my element is 100% width, i.e. on both the left and right edges of the screen, the popover appears off screen.  If the algorithm were to notice that, then check the "top" and "bottom" position, it would ensure the popover was visible if at all possible.
I realize I can write my own positioning function - but I thought this could be baked in.

Is this a limitation of Tether? Curious if it's something we need to investigate or if we punt on it if it's not supported there.

Closing since our auto placement is no more. #19796


The notes fields for spells and elsewhere should expand vertically as necessary.

I think I tried to do this once, and failed. It may be that Roll20 doesn't support this functionality. But maybe there is a way to have two html elements, one that shows only when you are not editing and resizes, and one that shows only when you are editing, and doesn't resize if roll20 doesn't support multi-line editors.

Let me see if I can get this working better.

Notes can be a text area, but I didn't find a way to expand vertically.
Here is an example:


Should changing Network>Hostnames result in /etc/init.d/dnsmasq restart?

When I add or change a host's IP in Network>Hostnames I hope that when I hit Save & Apply that the change happens straight away.  I have to SSH in and run /etc/init.d/dnsmasq restart (or use a web script I've written).
I'm using:
LuCI Master (git-15.167.71950-8005ecb) / OpenWrt Chaos Calmer 15.05-rc3
Is my expectation wrong?  Is it a caching issue elsewhere?

@c3iq Can you verify this happens with LuCI and OpenWrt/LEDE trunk; I'm not sure it's still an issue.  Also, can you look at /var/etc/dnmasq*.conf and verify (even on 15.05) that the file is or is not updated?  If is, then restart was done and the issue is that DNS may have been cached on your client.

Procd reload triggers should take care of this nowadays.


Overfitting Issue with PICAI Baseline Model nnDetection

Dear PICAI team,
Recently, I have been working with the baseline model provided for the PICAI dataset. Although I carefully followed the instructions, I keep encountering significant overfitting issues. Attached, you can find a plot illustrating the training process.

I’m wondering if there might be an issue with my training environment, or if there are adjustments I should consider making to the trainer.
Here are some details about my setup:

Training environment: Official picai_baseline Docker on A100 40GB
Cross-validation: Custom stratified 5-fold CV
Preprocessing: I used prepare_data.py and obtained 1295 cases, 80% of which were used in the 5-fold CV to train the model.

Thank you in advance for your assistance, and I hope you’re having a great summer!

Dear @CollapsarEcho,
For the PI-CAI baseline, we haven't made any adjustments to the parameters of the nnDetection framework, so it seems your setup is correct. To verify this, I visualized the mAP@0.1,0.5,0.05 metric for the first fold of the nnDetection (supervised) algorithm, which should closely match your implementation.

The graph looks very similar, suggesting that you've implemented the baseline correctly, and we both observe overfitting occurring quite quickly.
We noticed a significant performance improvement when incorporating the AI-derived annotations (to train the semi-supervised nnDetection). However, overfitting still seems to persist:

I think if you're able to combat the overfitting issue, that could lead to improved performance! I don't have any leads in this direction, unfortunately.
All the best,
Joeran

Thank you for your response! I’m glad to hear that I was on the right track with the reproduction. I’ll definitely explore some additional methods to address the issue. Additional institutional datasets seem like a promising approach, I'll give that a try.


Update pytest-variables to version 2.0.0

Checklist

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

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

@conda-forge-admin, please rerender

@birdsarah @davehunt
This is ready to review. Updates to support version 2.0,

@davehunt can you also add me as maintainer here? Thanks!

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

There are 1 too many lines.  There should be one empty line at the end of the file.

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

@davehunt can you also add me as maintainer here? Thanks!

@nicoddemus I've added you to the maintainers list.

Thanks, unfortunately that will only have effect when it lands on master.
@davehunt and @birdsarah, I have reviewed and everything looks good, please go ahead and merge it when you have the chance. :+1:


save_checkpoint consumed many GBs of gpu ram.

🐛 Bug
This is a split of from https://github.com/huggingface/transformers/issues/8403 where a much larger memory leak is discussed (has to do with native amp usage in particular circumstances) (note: there was also an attempt to move the discussion to https://github.com/PyTorchLightning/pytorch-lightning/issues/4614 but the original prevailed).
I found another unrelated strange huge gpu memory requirement in PL. It happens during save_checkpoint - As I'm stepping through it with debugger I observe the gpu memory growing from 2GB (training was complete) to 5GB.
It's very difficult to debug PL due to dozens of indirections :( here is the stack trace (in v1.0.6)
dump_checkpoint, checkpoint_connector.py:337
save_checkpoint, checkpoint_connector.py:389
save_checkpoint, properties.py:207
_save_model, model_checkpoint.py:333
_update_best_and_save, model_checkpoint.py:583
_save_top_k_checkpoints, model_checkpoint.py:534
save_checkpoint, model_checkpoint.py:232
on_validation_end, model_checkpoint.py:186
on_validation_end, callback_hook.py:177
call_hook, trainer.py:833
on_evaluation_end, evaluation_loop.py:109
run_evaluation, trainer.py:620
run_training_epoch, training_loop.py:589
train, trainer.py:493
train_or_test, accelerator.py:74
train, gpu_accelerator.py:63
fit, trainer.py:444
generic_train, lightning_base.py:398
main, finetune.py:413
<module>, finetune.py:446

So the gpu ram is growing through dump_checkpoint to about 1GB, and then jumps by another 2GB at:
        # give the model a chance to dump a few things
        model.on_save_checkpoint(checkpoint)

Would this be something that can be done on CPU? This is a checkpoint saving - shouldn't need more gpu RAM here in theory.
Ideally this function should need 0 extra gpu ram. Otherwise one may have a successful training but the program would OOM at the end of it during checkpoint saving which is not best. or if it's intermediary saving then it's definitely a problem.
The setup where I discovered this is identical to the first post: https://github.com/huggingface/transformers/issues/8403
which some of you have already replicated while debugging https://github.com/PyTorchLightning/pytorch-lightning/issues/4614
Environment
* CUDA:
        - GPU:
                - GeForce RTX 3090
                - GeForce GTX 1070 Ti
        - available:         True
        - version:           10.2
* Packages:
        - numpy:             1.19.2
        - pyTorch_debug:     False
        - pyTorch_version:   1.6.0
        - pytorch-lightning: 1.0.6
        - tqdm:              4.51.0
* System:
        - OS:                Linux
        - architecture:
                - 64bit
                - ELF
        - processor:         x86_64
        - python:            3.8.5
        - version:           #57-Ubuntu SMP Thu Oct 15 10:57:00 UTC 2020

@stas00 mind share some example to reproduce, and/or some more details about your training:

what distributed you used
have you used AMP for training

it would be also interesting to know how much memory it took on GPU/CPU while training if you use batch size 1

As the OP mentions this is a split off from https://github.com/huggingface/transformers/issues/8403 (re-played partially at https://github.com/PyTorchLightning/pytorch-lightning/issues/4614) where a huge memory leak had been discovered. As I was trying to isolate the cause, I found this one as well - it was suggested I post it separately. So all the reproduction details are in the https://github.com/huggingface/transformers/issues/8403 - if it'd help to re-paste the details please let me know. I know at least @SeanNaren already setup the reproducible case as described in https://github.com/PyTorchLightning/pytorch-lightning/issues/4614 as he was trying to help to debug the issue.
Oh I see, in this Issue I linked to the issue you guys created, I will fix it shortly to link to the source.

what distributed you used

None - accelerator_gpu

have you used AMP for training

--fp16 (native amp)

it would be also interesting to know how much memory it took on GPU/CPU while training if you use batch size 1

As the OP says, it was 2GB at the end of training and jumped to 5GB during save_checkpoint - hence the issue.
If you need any other details please let me know.
Thank you!

@SeanNaren how far have you got with this leak debugging?

AFAIK @SeanNaren only worked and reproduced the main part of https://github.com/huggingface/transformers/issues/8403, which has now been diagnosed and the problem is coming neither from PL nor transformers, but pytorch's autocast caching mechanism, Though it's very likely whatever the solution will be either or both projects might have to add some extra code to prevent such situations in the future. So we are waiting for the pytorch developers to tell us what's going on and how to move forward.

@stas00 is this tied to fp16 or completely separate?

Good question, @SeanNaren - it's actually the most pronounces with apex. But it'll require some better setup to do proper testing.
However here is a quick reproduction of the problem w/o needing any complex debugger setup.

I did:

diff --git a/pytorch_lightning/callbacks/model_checkpoint.py b/pytorch_lightning/callbacks/model_checkpoint.py
index d257e1ea..3ec381e0 100644
--- a/pytorch_lightning/callbacks/model_checkpoint.py
+++ b/pytorch_lightning/callbacks/model_checkpoint.py
@@ -181,7 +181,10 @@ class ModelCheckpoint(Callback):
         """
         checkpoints can be saved at the end of the val loop
         """
+        print(f"before: peak {torch.cuda.memory_stats()['allocated_bytes.all.peak']>>20}MB")
         self.save_checkpoint(trainer, pl_module)
+        print(f"after: peak {torch.cuda.memory_stats()['allocated_bytes.all.peak']>>20}MB")
+        torch.cuda.reset_peak_memory_stats()

     def on_save_checkpoint(self, trainer, pl_module) -> Dict[str, Any]:
         return {

Now when running:

cd examples/seq2seq
PYTHONPATH="../../src" CUDA_VISIBLE_DEVICES=0 python  -W ignore finetune.py --learning_rate 3e-5 --gpus 1 --do_train --val_check_interval 1 --num_train_epochs 1 --freeze_encoder --freeze_embeds --data_dir cnn_dm --max_target_length 142 --val_max_target_length 142 --train_batch_size 1 --eval_batch_size 1 --gradient_accumulation_steps 1 --model_name_or_path sshleifer/student_cnn_12_6 --tokenizer_name facebook/bart-large --warmup_steps 1 --output_dir distilbart-cnn-12-6 --overwrite_output_dir --num_sanity_val_steps 0 --n_train 1 --n_val 1 --fp16 --amp_backend=apex

Epoch 0: 100%|██████████████████████████████████████████████████████████████████████████████████| 2/2 [00:03<00:00,  1.91s/it, loss=2.935, v_num=199]
before: peak 1547MB
after: peak 4279MB

Epoch 0: 100%|██████████████████████████████████████████████████████████████████████████████████| 2/2 [00:09<00:00,  4.63s/it, loss=2.935, v_num=199
before: peak 1162MB
after: peak 1162MB

As you can see the first time it runs on_validation_end - it allocates an additional 2.7GB.
With native amp I can't easily test at the moment since it consumes 10x gpu RAM so the peak is at 19201MB, so a 3gb temp fluctuation won't appear on a radar. It will require a different kind of memory tracing. But since it's easy to see with apex, then it's probably good enough for now.
Without fp16 I get:
Epoch 0:
before: peak 2464MB
after: peak 2464MB
Epoch 0: 1
before: peak 2323MB
after: peak 2323MB

So peaking at all.
So it does point to fp16 as the culprit.

Going to close this now @stas00 given it was fp16 specific and seems related to the resolved fp16 memory leak that you helped fix! If the problem persists let us know.

@SeanNaren, the problem is still there with apex as originally reported. I have just retested. Same details as https://github.com/PyTorchLightning/pytorch-lightning/issues/4642#issuecomment-728273362
And now that the native amp leak has been fixed, I can finally test with native amp and this problem doesn't exist there (but I never reported it was there since I couldn't even test that case):
PYTHONPATH="../../src" CUDA_VISIBLE_DEVICES=0 python  -W ignore finetune.py --learning_rate 3e-5 --gpus 1 --do_train --val_check_interval 1 --num_train_epochs 1 --freeze_encoder --freeze_embeds --data_dir cnn_dm --max_target_length 142 --val_max_target_length 142 --train_batch_size 1 --eval_batch_size 1 --gradient_accumulation_steps 1 --model_name_or_path sshleifer/student_cnn_12_6 --tokenizer_name facebook/bart-large --warmup_steps 1 --output_dir distilbart-cnn-12-6 --overwrite_output_dir --num_sanity_val_steps 0 --n_train 1 --n_val 1 --fp16
[...]
Using native 16bit precision.
Epoch 0:
before: peak 1774MB
after: peak 1774MB
Epoch 0:
before: peak 1550MB
after: peak 1550MB


Possibility to build daal4py with only one algorithm?

OneDAL can be compiled for individual algorithms and test them separately, can that built algorithm be used to build daal4py? Doing this lead me to the following runtime error when importing daal4py (after building just kmeans in onedal):
>>> from daal4py import kmeans
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/ubuntu/.local/lib/python3.8/site-packages/daal4py/__init__.py", line 38, in <module>
    from daal4py._daal4py import *
ImportError: /home/ubuntu/.local/lib/python3.8/site-packages/daal4py/_daal4py.cpython-38-x86_64-linux-gnu.so: undefined symbol: _ZTIN4daal10algorithms12linear_model10prediction10interface15InputE

Is this achievable by just commenting out other algorithms in cython generator files or is more work needed to make this work?

Theoretically, it's possible to remove mentions of other algorithms in daal4py generator and other places where these algorithms are explicitly mentioned like this, but it's huge amount of work.
daal4py wasn't designed to use one-algorithm builds of oneDAL.

Thanks for the insights! Closing this thread.


feat: Modify segment_index type in Relocatable to i64

At the moment, segment_index type in Relocatable is u64. To add some temporary segments, we should be able to handle negative segment indexes:

Rust implementation
Python implementation

As a consequence, the type of segment_index should be modified to i64 in order to handle this use case.
We should also add here unit tests for the negative index and the counter.

@Godspower-Eze Perfect just assigned


dependency: rfbrowser init gives: UnicodeEncodeError: 'charmap' codec can't encode characters in position 36-43: character maps to

Bug-Description
When trying to execute
"rfbrowser init"
the following Stacktrace is printed into the console:
2023-09-27 01:04:10,492 [INFO    ] ==============================================================================================================
2023-09-27 01:04:10,492 [INFO    ] Installing node dependencies...
2023-09-27 01:04:20,099 [INFO    ] Installing rfbrowser node dependencies at C:\hostedtoolcache\windows\Python\3.9.13\x64\lib\site-packages\Browser\wrapper
2023-09-27 01:04:21,033 [INFO    ] npm WARN config production Use --omit=dev instead.
2023-09-27 01:05:24,891 [INFO    ]
2023-09-27 01:05:24,891 [INFO    ] added 61 packages, and audited 62 packages in 1m
2023-09-27 01:05:24,891 [INFO    ]
2023-09-27 01:05:24,891 [INFO    ] 7 packages are looking for funding
2023-09-27 01:05:24,891 [INFO    ]   run npm fund for details
2023-09-27 01:05:24,891 [INFO    ]
2023-09-27 01:05:24,891 [INFO    ] found 0 vulnerabilities
2023-09-27 01:05:24,906 [INFO    ]
2023-09-27 01:05:24,906 [INFO    ]
2023-09-27 01:05:24,906 [INFO    ]
2023-09-27 01:05:24,906 [INFO    ]
2023-09-27 01:05:24,906 [INFO    ] Installing browser binaries to C:\hostedtoolcache\windows\Python\3.9.13\x64\lib\site-packages\Browser\wrapper
2023-09-27 01:05:24,906 [INFO    ] npx --quiet playwright install
2023-09-27 01:05:27,367 [INFO    ] |                                                                                |   0% of 118.6 Mb
--- Logging error ---
Traceback (most recent call last):
File "C:\hostedtoolcache\windows\Python\3.9.13\x64\lib\logging_init_.py", line 1086, in emit
stream.write(msg + self.terminator)
File "C:\hostedtoolcache\windows\Python\3.9.13\x64\lib\encodings\cp1252.py", line 19, in encode
return codecs.charmap_encode(input,self.errors,encoding_table)[0]
UnicodeEncodeError: 'charmap' codec can't encode characters in position 36-43: character maps to
Call stack:
File "C:\hostedtoolcache\windows\Python\3.9.13\x64\lib\runpy.py", line 197, in _run_module_as_main
return _run_code(code, main_globals, None,
File "C:\hostedtoolcache\windows\Python\3.9.13\x64\lib\runpy.py", line 87, in run_code
exec(code, run_globals)
File "C:\hostedtoolcache\windows\Python\3.9.13\x64\Scripts\rfbrowser.exe_main.py", line 7, in
sys.exit(cli())
File "C:\hostedtoolcache\windows\Python\3.9.13\x64\lib\site-packages\click\core.py", line 1157, in call
return self.main(*args, **kwargs)
File "C:\hostedtoolcache\windows\Python\3.9.13\x64\lib\site-packages\click\core.py", line 1078, in main
rv = self.invoke(ctx)
File "C:\hostedtoolcache\windows\Python\3.9.13\x64\lib\site-packages\click\core.py", line 1688, in invoke
return _process_result(sub_ctx.command.invoke(sub_ctx))
File "C:\hostedtoolcache\windows\Python\3.9.13\x64\lib\site-packages\click\core.py", line 1434, in invoke
return ctx.invoke(self.callback, **ctx.params)
File "C:\hostedtoolcache\windows\Python\3.9.13\x64\lib\site-packages\click\core.py", line 783, in invoke
return __callback(*args, **kwargs)
File "C:\hostedtoolcache\windows\Python\3.9.13\x64\lib\site-packages\click\decorators.py", line 33, in new_func
return f(get_current_context(), *args, **kwargs)
File "C:\hostedtoolcache\windows\Python\3.9.13\x64\lib\site-packages\Browser\entry.py", line 321, in init
_rfbrowser_init(skip_browsers, silent_mode, with_deps, browser)
File "C:\hostedtoolcache\windows\Python\3.9.13\x64\lib\site-packages\Browser\entry.py", line 164, in _rfbrowser_init
_process_poller(process, silent_mode)
File "C:\hostedtoolcache\windows\Python\3.9.13\x64\lib\site-packages\Browser\entry.py", line 61, in _process_poller
_log(output, silent_mode)
File "C:\hostedtoolcache\windows\Python\3.9.13\x64\lib\site-packages\Browser\entry.py", line 53, in _log
logging.info(message.strip("\n"))
Message: '|\u25a0\u25a0\u25a0\u25a0\u25a0\u25a0\u25a0\u25a0                                                                        |  10% of 118.6 Mb'
Reproduction
Establish the following version
Python 3.9.1
pip 23.2.1
robotframework-browser 17.15.1
robotframework  6.1.1
robotframework-assertion-engine 2.0.0
############################## Execution reproduction
run in cmd task in azure pipeline the following dependency

script: rfbrowser init
displayName: 'Install dependencies rfbrowser init'

Expected behavior
robotframework setup executes and the appropriate drivers are installed

In which Windows OS and version this happens?

that happens in:
Windows 10 Enterprise (64-bits)

And the version is?

Correction:
from the build server (Azure) is this:
Windows Server 2022
OS Version: 10.0.20348
Image Version: 20230918.1.0
Hopefully you have enough information

@allcontributors please add  @JoostW21111 for bugs

Logging the progress bar is difficult in windows. Will suppress the error and continue.

I'm witnessing this with robotframework-browser-18.5.1:
UnicodeEncodeError: 'charmap' codec can't encode characters in position 36-43: character maps to <undefined>
Call stack:
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code

rfbrowser init is run by Azure pipeline agent 3.239.1. Python version is 3.11.9
OS is:
Edition	Windows 10 Pro
Version	22H2
Installed on	09/11/2022
OS build	19045.4412
Experience	Windows Feature Experience Pack 1000.19056.1000.0

Yes, the error happens, but should be only cosmetic thing (but really annoying one.) I think we could detect if the platform is Window (this seems to only happen in Windows) and either just suppress the error or clean the message to contain only ascii.

Moved discussion in #3633


Validation message styling through validationHtmlClass

Needed support for custom styling of validation messages . https://github.com/Textalk/angular-schema-form/issues/443

Hi @prabhatsharma and thank you for the PR!
I'm a bit reluctant adding another watch,could this be solved by CSS in combo with htmlClass?

Understand your concern around adding something extra, but we have tried fixing the styling through various means (css, htmlClass and overriding help-block) with success in many cases. However all scenarios are not getting covered with these and overriding help-block impacts other fields as well and is not really a good idea.
Would appreciate your consideration in getting this incorporated. This would make styling a lot simpler for validation messages.

@prabhatsharma I know it has been a long time, but with David no longer on the development can you please give me an idea of the use cases you feel this supports that couldn't be worked out in any other way, like perhaps in a plunker?

Hi there, Idea of having this  PR was to be able to add styling to the validation message that happens on schemaform fields. In absence of this implementing custom styling is difficult and we had to resort to creating custom schemaform components. Schemaform already provides styling for label and field for teh component, but not for validation message.

So I basically have one thing here and that is for pure traceability, I do not like that the code block has moved the way it have, it means that diffs involving this code will always be problematic because of this commit.
I couldn't/wouldn't even find the change.
That is, with the very limited amount of effort I was prepared to put into it. :-)

@prabhatsharma Nicklas is Referring to the dist folder, that will need to be reverted before any PR could be accepted.
When I asked about what you can't do, I mean if you use the parent tag it gets has-error and also names field type ie. .schema-form-section, so I am just wondering what scenario you are having trouble supporting with CSS?
Whatever you add to form.htmlClass you could use as:
.my-css-class.has-error  .help-block {
}
.has-error .help-block {
}

If there are any specific templates that don't behave in a consistent way which made that difficult I would gladly accept a PR that made them behave in a way that made a simple CSS rule apply across all templates.

@prabhatsharma, @Anthropic  I am sorry, and thanks for the save there @Anthropic but that was just me being an idiot. :-)
I didn't see it was the dist file, I thought it was the source file. Disregard my earlier statement.
(however it is kind of why we don't want dist files to be included in commits)


Create selectionsort.py

For short term contributors: we understand that getting your commits well
defined like we require is a hard task and takes some learning. If you
look to help without wanting to contribute long term there's no need
for you to learn this. Just drop us a message and we'll take care of brushing
up your stuff for merge!
Fixes #{ISSUE_NUMBER}
By submitting this pull request I confirm I've read and complied with the
below declarations.

[x] I have read the Contribution guidelines and I am confident that my PR reflects them.
[x] I have followed the commit guidelines for this project.
[ ] My code follows the standard code structure.
[ ] This pull request has a descriptive title. For example, {Tag}: Add {Algorithm/DS name} [{Language}], not Update README.md or Added new code.
[ ] This pull request will be closed if I fail to update it even once in a continuous time span of 7 days.
[ ] This pull request shall only be reviewed and merged once all the checks passes. No maintainer or supporter shall be obliged to review it before this condition is met.
[ ] I have mentioned the issue number correctly (with hyperlink) in this pull request description.

After you submit your pull request, DO NOT click the 'Update Branch' button.

fixes #446

Hey @richabh456,
Something went wrong with the build.
TravisCI finished with status errored, which means the build failed because of something unrelated to the tests, such as a problem with a dependency or the build process itself.
View build log
TravisBuddy Request Identifier: 2a02ee90-f03c-11e9-983c-41f05746e0df


contractkit: use @celo/abis package

Description
This PR aims to introduce @celo/abis package usage to contractkit.
Other changes
Changes the way @celo/abis package is generated.
Tested
Not yet.
Related issues

Fixes #10719

Backwards compatibility
TBD
Documentation
TBD

I dont think the readme are being copied to the directory the packages are published from

https://github.com/celo-org/celo-monorepo/pull/10781 should mean the error in the SortedOracles ts tests no longer matters


LeagueProdKit can't be started : error Module not found

Hello,
Since 2 days when I tri to start a fresh new league Prod Kit instance, I have the error "Module not found" at the step "finish moving files to frontend"

I tried to install other version :
=> 1.2.11 : NOK
=> 1.2.10 : NOK
=> 1.2.9 : NOK
=> 1.2.8 : NOK
For information, I don't encountered any errors when I use "install.bat", I install all themes and modules for League of Legends and valorant.
Do you know how can produce this error ?
Step to reproduce :

download last release
execute "install.bat"
execute "start.bat"

Thanks you in advance for your help :)

This was a bug in the league-static module (unrelated to the toolkit version).
The bug is fixed with v3.6.15 of module-league-static and the toolkit should start without issue on a fresh install.


feat(ferry_exec): Pass context to GQL Links

feat: pass context to gql links
feat: update generated code

Generated code was updated with "void" statements. I am not sure why

Feature request: #575

In order to create advanced Links that can execute custom logic per request, we need a way to extend request with ContextEntries.
Currently, generated code for a request is:
  @override
  Request get execRequest => Request(
    operation: operation,
    variables: vars.toJson(),
    // no passed context. Thus, the default is `context = const Context()`
  );

gql_exec has a method .withContextEntry on Operation that we could use.
I'd be happy to contribute if you think it is a good issue 🙂
(😅 Hope I did not miss something in the API that already allows us to create context entries) Reference: #556

I'd be happy to fix verify_commits but I did not understand why it failed

It was the name of the PR. edited it for you and reran the action :)
(this is so CHANGELOGs can be created automatically with melos)

LGTM!


On-hover button-icons for Source Control "changes" can be easily mis-clicked

Issue Type: Bug
The buttons that appear on the changes list (Source Control panel), are shifted to the right by other decorations; this makes it very easy to miss-click on Stage or Discard buttons.

VS Code version: Code - Insiders 1.23.0-insider (1fdf60b882f0edc6753fc964b253783eb46d56ef, 2018-04-19T13:37:07.476Z)
OS version: Windows_NT x64 10.0.16299

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-4200U CPU @ 1.60GHz (4 x 2295)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
3.91GB (0.86GB free)

Process Argv
C:\Program Files\Microsoft VS Code Insiders\Code - Insiders.exe

Screen Reader
no

VM
50%

Extensions (11)

Extension
Author (truncated)
Version

spellright
ban
2.2.16

vscode-icontheme-nomo-dark
be5
1.3.5

unique-lines
bib
1.0.0

gitignore
cod
0.5.0

vscode-markdownlint
Dav
0.14.1

EditorConfig
Edi
0.12.1

vscode-apache
mrm
1.1.1

material-icon-theme
PKi
3.3.0

gitconfig
sid
1.0.1

ignore-gitignore
stu
1.0.1

ayu
tea
0.14.0

(4 theme extensions excluded)

you mean they are shifted to the left, correct?

By the change count, yes

are shifted to the right by other decorations

Do you mean shifted by the error decoration (the +9)? Iff so, know that uou can turn that off via "problems.decorations.enabled": false
@joaomoreno what's the bug here?

@jrieken Good tip. I believe the bug is that the user often misclicks the actions.

We should discuss this with UX folks


Installation doesn't work on Google Colab

I followed the steps given in the Installation guide somehow it doesn't work.
I am attaching link to the notebook.
Link : https://bit.ly/2AVyhK4

Hey, @salilmishra23 I tried to replicate the same steps on google colab and it worked without any issues. I am attaching the link of the notebook (https://colab.research.google.com/drive/1Zj9_g_8uVKS4tvNfp_pMqpcVDkDWhUUR). Please check and let me know if you have any issues.

try to add library path to system path:
import sys sys.path.insert(0,'/content/pystacknet')
change this line (in /content/pystacknet/pystacknet/pystacknet.py)
from pystacknet.metrics import check_regression_metric, check_classification_metric
--> from pystacknet.pystacknet.metrics import check_regression_metric, check_classification_metric
if you want to import StackNetClassifier:
from pystacknet.pystacknet.pystacknet import StackNetClassifier

Hi guys, the same issue appears on my MAC and Windows PCs, I cannot import models
StackNetClassifier, StackNetRegressor.
Other parts of the package works,
import pystacknet - works
from pystacknet.metrics import check_regression_metric, check_classification_metric - works
from pystacknet.pystacknet import StackNetClassifier - does not work
from pystacknet.pystacknet.pystacknet import StackNetClassifier - does not work
the same errors shows as in Google Collab
ImportErrorTraceback (most recent call last)
 in ()
----> 1 from pystacknet.pystacknet import StackNetClassifier
build\bdist.win-amd64\egg\pystacknet\pystacknet.py in ()
ImportError: No module named metrics

Hey, @salilmishra23 I tried to replicate the same steps on google colab and it worked without any issues. I am attaching the link of the notebook (https://colab.research.google.com/drive/1Zj9_g_8uVKS4tvNfp_pMqpcVDkDWhUUR). Please check and let me know if you have any issues.

this link is no longer available, can post a new one?

#17 Check this out!


@nuxtjs/font-awesome needs to be updated to version 5

What problem does this feature solve?
Icons are shown as square when using the nuxtjs/font-awesome package and font-awesome version 5.
E.g.
i class="fas fa-arrow-alt-right"
What does the proposed changes look like?
Use latest version of font-awesome (version 5)

This feature request is available on Nuxt community (#c9886)

Please open issue in a proper project. This is for nuxt itself, while your issue is for module that located in https://github.com/nuxt-community/modules/tree/master/packages/font-awesome


How to customize styles?

We could have a custom.css file that gets pushed into the header? Key, val pairs that we build into it? Defining a giant string to replace DEFAULT_HEADER seems bad. So does DEFAULT_HEADER for that matter.

I think the answer is custom.css but given that I have to turn the CSS into inline styles, I'm tempted to not worry about custom.css just yet.  But it should work the same as the default.css


[Proposal] Add a way to instantiate different scenes without the need of several NodeInstantiators, to avoid duplicate code.

The Problem:
There's currently no way to instantiate different scenes with the same NodeInstantiator, (or with just the call of a function on perhaps the GDSync autoload, that also sounds nice).
For my case, this is needed for performance reasons. Not that it would be a massive headache probably, but it would likely do some good by avoiding duplicate code:
I am making an RTS prototype. I would currently have to use a NodeInstantiator in each instance of a building to spawn units, e.g. two barracks's will have a NodeInstantiator each to spawn the same unit.
Proposed Solution:
Have a way to instantiate any scene using code, with the same functionality as NodeInstantiator. I thought of 2 ways to achieve this:

Modify NodeInstantiator, so it would work like so: node_instantiator.instantiate_node(scene_path: String, target: Node)
Have a function in GDSync: GDSync.instantiate_node(scene_path: String, target: Node, replicate_on_join := true, sync_starting_changes := true). This would probably reduce duplicate code more.

I came up with another solution that I implemented myself, where I made an autoload called GlobalInstantiator, copied the code of NodeInstantiator into it and modified it and NodeTracker too, to accept GlobalInstantiator.instantiate_node(scene_path: String, parent := get_tree().current_scene) while not breaking NodeInstantiator itself. I have no clue if replication works but I probably don't need it lol.
As final thoughts, this is just an idea and perhaps it's not needed massively, but I would like to know your thoughts. I am happy to share the modified NodeTracker code and my GlobalInstantiator code if needed.

Great suggestion! A global way to instantiate scenes sounds indeed very handy. It has been added to our roadmap.

This feature has now been implemented, you can download it here from the repo. You can now use GDSync.multiplayer_instantiate(scene : PackedScene, parent : Node, sync_starting_changes : bool = true, excluded_properties : PackedStringArray = [], replicate_on_join : bool = true) to instantiate Nodes without using a NodeInstantiator.
Please let me know if you encounter any issues with it.

If enabled, any changes made to the root Node of the instantiated scene within the same frame will automatically be synchronized.

This is the description of the sync_starting_changes parameter. It synchronizes any changes made to the root Node of the instantiated scene. Lets pick a bullet as an example. We can set the position, speed and direction after instantiating it. These changes will automatically be applied on all other clients when the bullet is instantiated there.
When a Node is instantiated remotely it goes as followed:
_ready() is called -> starting changes are applied -> _multiplayer_ready() is called
As for scene : PackedScene or just scene_path : String, it does not have a performance difference. Every user still loads the scene themselves. When sending the remote instantiate request it only sends the resource_path of the PackedScene.
Closing this issue as the new function seems to work as expected.


GPU support for OpenWhisk Actions

It is not an issue. But just a general query from my end. Does OpenWhisk support GPU acceleration?
I was going through this proposal about AI-Actions in OpenWhisk. The proposal there has the WIP [Work In Progress] tag. But that proposal itself is six years old.
I was wondering whether there is a mechanism in OpenWhisk using which I can offload actions to get them executed on GPU.
The tag-based-scheduling mentions about things like:
wsk action update params tests/dat/actions/params.js -i -a invoker-resources '["v2", "gpu"]'

I do not understand its utility. Does it solve the purpose of offloading actions to GPU? If so, I would like to be helped with some documentation or tutorials explaining how I could achieve the same.

We could use K8s with CUDA-K8s driver, I think it can work.


Dropdown option width issue in ipad

I have long texts for dropdown options. When I use bootstrap select, the dropdown options extends the viewport and causes design issue.

I am having this issue on iPhone as well. I tried setting the width with data-width but that only changes the width of the select, not the dropdown. Would appreciate a fix or guidance in case I'm doing something wrong.

i have this problem too,  have you solve it  ?   could you help me about this ?
tks & bestregards

Please try with latest version.


fix(es/resolver): Correctly check strict mode

Description:

BREAKING CHANGE:

Related issue (if exists):
Closes #8842

CI failed and it does not look like a random failure.
Diff < left / right > :
 "use strict";
 function f() {
<    var x, g1;
>    var x, g;
     g();
     x = 10;
     throw Error("foo");
 }
 f();

Consider the terser output, I believe the new output is desired.
"use strict";
function f() {
    g();
    x = 10;
    throw new Error("foo");
    var x;
}
f();


ALAssetsLibrary In WAStickers is deprecated

The ALAssetsLibrary has been replaced by the new Photos framework - the app doesn’t compile in the latest Xcode as a result.

I will refactor the code in the next Fluffcorn Stickers app update.

@CaptainMurdo ALAssetsLibrary is being used by YYImageCode which is part of the WhatsApp stickers (WAStickers) code which was imported from https://github.com/WhatsApp/stickers/blob/main/iOS/WAStickersThirdParty/YYImageCoder.m
The latest version of WAStickers code from WhatsApp still uses ALAssetsLibrary and updates to the WAStickers code will be imported from the official WhatsApp stickers repository when fixed by WhatsApp.
I can still successfully compile the current project in Xcode even though it warns about the deprecated code. If you don't need WhatsApp integration, you could remove the WhatsApp code files and compile without them.


Adding corner plots to paper

[ ] ...a corner plot would be very complicated and hard to read, since the large number of parameters, but I still recommend to make such a plot only for some key parameters, e.g., for the planet mass and eccentricity vectors.

I need to add the datasets to the data.zip file so that these can be accessed in travis.

Two corner plots have been created, one for the TTV analysis, and another for the photo dynamical analysis.  These have been placed in an appendix.


OFFSET without LIMIT returns odd results

Bug report
Ubuntu, nightly
Steps to reproduce:
1. Write some data
> create database mydb
> use mydb
Using database mydb
> insert mymeas,color=yellow value=2
> insert mymeas,color=yellow value=3
> insert mymeas,color=yellow value=4
> insert mymeas,color=yellow value=5
> SELECT * FROM mymeas
name: mymeas
time				color	value
----				-----	-----
2016-11-02T23:27:55.659009728Z	yellow	2
2016-11-02T23:27:57.532365177Z	yellow	3
2016-11-02T23:27:59.352763724Z	yellow	4
2016-11-02T23:28:01.064617644Z	yellow	5

2. Run an OFFSET query without LIMIT
> SELECT * FROM mymeas OFFSET 1
name: mymeas
time				color	value
----				-----	-----
2016-11-02T23:27:57.532365177Z	yellow	3

3.  Run an OFFSET query with LIMIT
> SELECT * FROM mymeas LIMIT 2 OFFSET 1
name: mymeas
time				color	value
----				-----	-----
2016-11-02T23:27:57.532365177Z	yellow	3
2016-11-02T23:27:59.352763724Z	yellow	4

Expected behavior:
I'd expect the query in step 2 to return an error about needing LIMIT or maybe the output below:
name: mymeas
time				color	value
----				-----	-----
2016-11-02T23:27:57.532365177Z	yellow	3
2016-11-02T23:27:59.352763724Z	yellow	4
2016-11-02T23:28:01.064617644Z	yellow	5

Actual behavior: [What actually happened]
I can't really explain the results in step 2.

This is happening on macOS 10.12.2 and Influx v1.1.1.

We should reproduce this on 1.5.x

This doesn't appear to happen anymore. I wrote three points to the database of the latest release and used OFFSET 1 without a limit and got the last two points.

Happens for me on 1.5.1.


#5749 fix tableDisplay memory leaks

cool thx, works well but one small problem: I am not seeing the triangles that should appear when I click on a column to sort it.


only show filetype icon for tabs that have a path

Turns out that was easy. Tabs that don't belong to an editor don't have a data-path attribute in the title. Since this is only about file-type icons, you can just make the selector a bit more specific.
Fixes #103

Thanks so much :grinning:


Puppeteer: resizeWindow() does not change window bounds

What are you trying to achieve?
I want to resize the browser window
What do you get instead?
I get a bigger viewport size, but the browser window size stays the same.
See also https://github.com/GoogleChrome/puppeteer/issues/1183
Details

CodeceptJS version: 1.1.5
NodeJS Version:  8.9.4
Operating System: Windows
Puppeteer (1.0.0)

I would recommend considering the following code for the resizeWindow function which seems to work well

async resizeWindow(width, height) {
    await this.page.setViewport({height, width});

    // Window frame - probably OS and WM dependent.
    height += 85;

    // Any tab.
    const {targetInfos: [{targetId}]} = await this.browser._connection.send(
      'Target.getTargets'
    );

    // Tab window.
    const {windowId} = await this.browser._connection.send(
      'Browser.getWindowForTarget',
      {targetId}
    );

    // Resize.
    await this.browser._connection.send('Browser.setWindowBounds', {
      bounds: {height, width},
      windowId
    });
  }

I just used the above code to implement browser window resizing, thanks

I just tried this and got an error of TypeError: Cannot read property 'setViewport' of undefined
Where / How do you integrate this?

Modified version for puppeteer with slight code update (it works with chrome app mode):
async function resizeWindow(browser, page, width, height) {
  await page.setViewport({height, width})

  // Window frame - probably OS and WM dependent.
  height += 85

  // Any tab.
  const targets = await browser._connection.send(
    'Target.getTargets'
  )

  // modified code
  const target = targets.targetInfos.filter(t => t.attached === true && t.type === 'page')[0]

  // Tab window.
  const {windowId} = await browser._connection.send(
    'Browser.getWindowForTarget',
    {targetId: target.targetId}
  )
  const {bounds} = await browser._connection.send(
    'Browser.getWindowBounds',
    {windowId}
  )

  const resize = async () => {
    await browser._connection.send('Browser.setWindowBounds', {
      bounds: {width: width, height: height},
      windowId
    })
  }

  if(bounds.windowState === 'normal') {
    await resize()
  } else {
    await browser._connection.send('Browser.setWindowBounds', {
      bounds: {windowState: 'minimized'},
      windowId
    })
    await resize()
  }
}

const browser = await puppeteer.launch({
  executablePath: '/path/to/chrome',
  args: ['--app=http://localhost:8080'],
  headless: false,
  defaultViewport: null
})

const page = (await browser.pages())[0]

setTimeout(() => {
  console.log('resize')
  resizeWindow(browser, page, 800, 600)
}, 1000)

having fought with this for a few days, none of the googled solutions worked for me.
Although some of what's been posted will result in the window size being set, the values won't be readable from window.outerWidth after a page navigation.
However...
function setWindowDimensions ({width, height}) {
  window.outerWidth = width;
  window.outerHeight = height;
}

(async () => {
  const dimensions = await page.evaluateOnNewDocument(setWindowDimensions, {width, height});
})();

Executing this before navigating to the target URL will allow a script on that page to properly read the window size.
Tested in both headless and non-headless mode (Chromium)

Here is a slightly improved version of @hubidu's snippet:
  async resizeWindow(width: number, height: number) {
    const page = await this.getActiveTab();
    const session = await page.target().createCDPSession();
    await page.setViewport({height, width});
    const {windowId} = await session.send('Browser.getWindowForTarget');
    await session.send('Browser.setWindowBounds', {
      bounds: {height, width},
      windowId,
    });
  }

I removed usages of private properties and call to 'Target.getTargets', because 'Browser.getWindowForTarget' returns the window with active CDP session (docs).


E2E testing setup for nuxt3 portal

Portal environment variables need to be added to secrets.
General tests are migrated from the nuxt2 portal to the nuxt3 portal.

Updated selectors
Fixed bugs
Created a new yaml file to trigger testing for PR, schedule, and manual

examples for the new yaml file

PR - github action, cypress result
Schedule - github action, cypress result
Manual - github action, cypress result

Is the MarkdownTest used anywhere?

No, it was originally there in nuxt2. It looks like someone created that 4 years ago. Let me know if I need to delete it.

Is the MarkdownTest used anywhere?

Not in nuxt3 at the moment. Someone created that 4 years ago in nuxt 2. Let me know if I should remove it.

We should delete it, there is no command to run it.


Performance: fix memory leak in battle/chat room deletion

Personal rooms' log writing streams weren't being cleaned up properly.

??
They should have a shared stream that shouldn't get closed...

The personal room ones create their own stream rather than using the one attached to Rooms

...the correct answer is to make them not create their own stream.

Wait shit, I made a retarded mistake; this should be closing the streams if the room's not a personal room

Um, we delete a room maybe once or twice a month, I don't think this leak is a serious problem...

I realize that now, but I've already wasted enough time thinking this as more serious than that.


dpctl: Error connecting to switch unix:/tmp/s1. (No such file or directory)

Hi all,
I use mininet to make the experiment and I want to add the queue to the port of the switch which is the ofsoftswitch13. When I use "sudo dpctl unix:/tmp/s1 queue-mod 1 1 80" to add the queue, there is a error.
RECEIVED (xid=0xF0FF00F0):
error{type="QUEUE_OP_FAILED", code="EPERM", dlen="56"}
I find a answer(http://stackoverflow.com/questions/33677006/adding-queues-to-switch-of-ofsoftswitch13-implementation-doesnt-work?answertab=active#tab-top) for this problem. Because of mininet set the user switch "--no-slicing", failing to add the queue. Then I follow the answer to modify the mininet, but it doesn't work and the switch can't negotiate with floodlight controller.
May 14 06:36:10|00001|vconn_unix|ERR|/tmp/vconn-unix.23705.0: connection to /tmp/s1 failed: No such file or directory
dpctl: Error connecting to switch unix:/tmp/s1. (No such file or directory)
I don't know how solve it, please help me?

The stackoverflow "solution" is bogus for multiple reasons, not the least of which is that you don't need to change Mininet to set options - please read the documentation!

I think your dpctl command is also wrong.


pin setuptools

Checklist

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

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

requirements: host: setuptools<60.0 must contain a space between the name and the pin, i.e. setuptools <60.0

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


Java 11

Is your feature request related to a problem? Please describe.
The last LTS version of Java is 11. We should upgrade DBTarzan to that version
Describe the solution you'd like
Change the build.sbt file to produce Java 11 based DBTarzan applications.
Also the publish scripts for the various platforms (Windows, Linux, Mac) should be upgraded
Additional context
See if it is possible to publish native Java 11 applications, since Oracle does not publish JRE 11 setup files

The new realease 1.20 uses Java 11 and JavaFX 11 (which is not compatible with previous Java version).
Since from version 11 JavaFX must be included in the jar file of the application (together with some native libraries), we changed the build.sbt so that it produces 3 different jars, one for Windows, one for Linux and one for MacOSX. These jars are also used to produce the installation files.
The debian package (.deb), is based on the LInux jar and therefore it now includes JavaFX, but is dependent by OpenJDK 11.
The Snap component remains the same.
The Windows installation file  (DBTarzan....exe) changed considerably. From this version the JRE  is bundled with the installation file, so that no previous installation of Java is needed.
Sadly there is no MacOSX app file now, because the included JavaFX causes the MacOSX (at least Mojave) to give errors when the application is executed.


added new optional prop to Tooltip that allows you to set the timeout

New prop on ToolTip called 'tooltipTimeout' default is 500 ms

a

Thanks!


License?

Please forgive me if I've just missed it, but this project doesn't appear to have a license attached. Can you please select one and attach it?

It's Apache licensed. It's set in setup.py and shows up on PyPI.

Great, thank you!


Intl.formatMessage method not working because it's not extracted?

version: 3.3.0
node version: 11.x
npm (or yarn) version:
Do you want to request a feature or report a bug?: Maybe bug
I think its not a problem with this lib, but I saw here that a pull request added support for intl.formatMessage. But https://formatjs.io/docs/tooling/babel-plugin tooling shows that

The default message descriptors for the app's default language will be extracted from: defineMessages(), defineMessage(), and ; all of which are named exports of the React Intl package.

So apparently intl.formatMessage is not extracted and then auto id does not find the extraction to parse.
So how does the pull request here for intl.formatMessage support should work if format-js does not extract?

Sorry about that, further more in documentation found a .babelrc configuration for this
https://formatjs.io/docs/tooling/babel-plugin#extractfromformatmessagecall
["react-intl", {
      "messagesDir": "./app/translations/",
      "extractFromFormatMessageCall": true
    }]


1.18.2, Forge || Log spam, unknown cause

Whenever I enable this mod specifically, the output log seems to be spammed with the letter 'a' thousands of times over.

oh my! looking into this, i probably forgot to remove something...


add cleanup command

this pull request adds a cleanup command to scoop which will remove older versions of apps while preserving persistent data.

@jfmherokiller  A bit late, but what does fsutil.exe reparsepoint delete $file.FullName do?
If I test this on file/directories with the LinkType: HardLink I get an Error: The file or directory is not a reparse point.


workflow: optimize use template

📑 Description
Closes https://github.com/goravel/goravel/issues/?

Summary by CodeRabbit

Chores

Simplified GitHub Actions workflows for linting and testing by using reusable workflows from an external repository.
Consolidated testing jobs into a single job, reducing duplication and streamlining maintenance.

✅ Checks

[ ] Added test cases for my code

@coderabbitai full review

Great 👍


SAK-31346 adding exception catch on createEntity and use getEntity method if thrown

Sometimes, when unzipping an archive in resources, the content collections and resources are created in an illogical order, causing an IdUsedException to be thrown.
For example, in one file I have a structure like:
mobile/1.pdf
mobile/2.pdf
mobile/
It creates the entities in top-down order. When it gets to "mobile/" it already exists, therefore the error is thrown.
This will get the content entity edit object instead if IdUsedException is thrown.

This LGTM. I'm just wondering about the consequences of the originating branch being in sakaiproject/sakai rather than your own account (if there are any at all).

Not sure. I can redo the PR from a fork, if it's preferred. I've always just created a branch with the name of the Sakai JIRA from master. :)

Ya, I'm just going off what our Git document says:
https://confluence.sakaiproject.org/display/SAKDEV/Git+Setup
To be honest, I'm not sure if it makes much of a difference, other than to keep things clean via separation. Anyone else have any insight into this?

Yeah, there shouldn't be a problem with the merge but it's highly preferred that everyone forks Sakai and create PR's from your forked version. Otherwise we have to spend time cleaning up these branches if they are not removed. (And might remove something before it's merged)
And someone might also inadvertently merge something directly into master and we are prevented from rewriting the history of that.

I'll go ahead and redo the PR from my own fork. Gimme 2 mins :)

Here’s the new one: https://github.com/sakaiproject/sakai/pull/2801 https://github.com/sakaiproject/sakai/pull/2801
—Bob

Robert Long
Software Engineer
Technical Account Manager - Sakai
Unicon, Inc.

M.A., Instructional Technology
Saginaw Valley State University

On Jun 10, 2016, at 08:24, Code Hugger (Matthew Jones<EMAIL_ADDRESS>wrote:
Yeah, there shouldn't be a problem with the merge but it's highly preferred that everyone forks Sakai and create PR's from your forked version. Otherwise we have to spend time cleaning up these branches if they are not removed. (And might remove something before it's merged)
And someone might also inadvertently merge something directly into master and we are prevented from rewriting the history of that.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/sakaiproject/sakai/pull/2794#issuecomment-225213137, or mute the thread https://github.com/notifications/unsubscribe/AANwZ3uxtmh4IIKxL4d3UVS0khI6S3kvks5qKYGSgaJpZM4IyebK.


Expose a wrapped version of Primal's vector class for use in Lua input files

Summary

This PR is a feature
It does the following:

Replaces vector component expansion (vec -> x,y,z) in Lua functions with a Lua usertype that supports basic vector operations
Additionally, resolves a rule of three warning in Primal due to a user-defined copy ctor but not copy op=

Also, please update the RELEASE-NOTES

Also, please update the RELEASE-NOTES