Not able to get the selected rows in the UI-Grid.

Hi,
I am new to the ui-grid, trying to get the selected item from the ui-grid, but not able to get the selected item.
$scope.gridRolesList.onRegisterApi = function(gridApi){
                  //set gridApi on scope
                  $scope.gridApi = gridApi;
                  $scope.mySelectedRows = $scope.gridApi.selection.getSelectedRows();
                  gridApi.selection.on.rowSelectionChanged($scope,function(row){
                    console.log('row selected ' + row.isSelected);
                  });

                  //var g = $scope.gridApi.selection.getSelectedRows();
                  gridApi.selection.on.rowSelectionChangedBatch($scope,function(rows){
                    console.log('rows changed ' + rows.length);
                  });
                };

gridApi is showing undefined, and that's the reason the $scope.gridApi = gridApi;
$scope.mySelectedRows = $scope.gridApi.selection.getSelectedRows(); is not getting set.

What other errors do you have on your javascript console?  Is the grid rendering at all?  Where in your code is this function - is it part of grid initialisation before the grid initially renders?

There are no other errors.
Yeah Grid is rendering properly. And I am able to select the rows but not able to get the selected rows.
Below are the piece of code which I am calling before the  onRegisterApi.
                $scope.gridRolesList.rowHeight = 50;
                $scope.gridRolesList.enableSorting = false;
                $scope.gridRolesList.enableFiltering = false;
                $scope.gridRolesList.data = rolesCollection;
                $scope.gridRolesList.multiSelect = true;
                $scope.gridRolesList.enableRowSelection = true,
                $scope.gridRolesList.enableSelectAll = true,
                //$scope.gridRolesList.showGridFooter = true,
                $scope.gridRolesList.columnDefs = [
                    { name: 'roleName', displayName: 'Name' },
                    { name: 'Actions',
                    cellTemplate: '<button class="btn btn-primary" ng-click="grid.appScope.EditRole(row.entity.roleName)">Edit</button><button class="btn btn-primary" ng-click="grid.appScope.DisableRoles(row.entity.roleName)">Disable</button>'
                }];

It's working for me here: http://plnkr.co/edit/JNID2STlF0QELj2dpqzZ?p=preview
Just to be clear - are you getting the selected rows as soon as the grid has rendered (in the onRegisterApi)?  Surely there won't be any rows selected at that point?  Don't you have to wait until the user selects some rows before getting them?

A big thank you @PaulL1 .
As I said onRegisterApi was not getting initialized, reason was I had written it inside  $http.get() call.
Hence, gridApi was not getting initialized.
Now, it's working great.

HI PaulL,
I am facing the same issue with UIGrid in SharePoint 2013, I am using UI-Grid to select few Items in the grid to have certain action on it, but couldn't get the selected rows by using the function mentioned in your sample above, even though I've downloaded the same "ui-grid-unstable.js", "ui-grid-unstable.css" from the sample and almost copied the same steps of configuring the grid and how populate it with data, in addition to the note of initilizing gridAPi by siddhusper, all that and I am still unable to get selected rows by:
$scope.mySelectedRows = $scope.gridApi.selection.getSelectedRows();
so I used another method: gridApi.selection.on.rowSelectionChanged($scope, function (row) { ..}
to get each row and add it to my selectedrows = [];
its working fine with an issue that I couldn't handle  when "Select All/Deselect All" the ticker button of the Grid clicked, so I can reset my SelectedRows[] Array.
any help or advice is really appreciated.

@Basel78 you should read the documentation of ui.grid.selection. The secret is rowSelectionChangedBatch()
gridApi.selection.on.rowSelectionChanged(scope, singleSelect)
gridApi.selection.on.rowSelectionChangedBatch(scope, function (rows) {
    rows.forEach(singleSelect)
})

Thanks iPand for the reply,
Actually, the rowSelectionChangedBatch() is not fired in my case,
so is there a property needs to be enabled except , enableSelectionBatchEvent : true.
and where the event handler should comes in the code, as you mentioned its falling inside "gridApi.selection.on.rowSelectionChanged() {}" , but as for the documentations it's coming after it, and inside "$scope.gridOptions.onRegisterApi= " exactly like  rowSelectionChanged().
it seems that I have problem to fire rowSelectionChangedBatch function in my code, can you please describe how to do it in code.
I will put part of the code can explain to me what I did wrong?
$scope.gridOptions = {
enableRowSelection: true,
enableSelectAll: true,
selectionRowHeaderWidth: 35,
rowHeight: 35,
showGridFooter: true,
enableSelectionBatchEvent: true
};
$scope.gridOptions.columnDefs = [
{ field: 'ProductID', displayName: 'Product ID' },
{ field: 'ProductName', displayName: 'Product Name' },
{ field: 'SupplierID', displayName: 'Supplier ID' },
{ field: 'CategoryID', displayName: 'Category ID' },
{ field: 'QuantityPerUnit', displayName: 'Quantity Per Unit' },
{ field: 'UnitsOnOrder', displayName: 'Units On Order' }
];
$scope.info = {};
$scope.getSelectedRows = function () {
$scope.mySelectedRows = $scope.gridApi.selection.getSelectedRows();
};
$scope.gridOptions.onRegisterApi = function (gridApi) {
    $scope.gridApi = gridApi;

    gridApi.selection.on.rowSelectionChanged($scope, function (row) {
        $scope.SelectedProduct = row.entity;
  });

    gridApi.selection.on.rowSelectionChangedBatch($scope, function (rows) {

        rows.forEach($scope.SelectedProduct);
    });
};

After deep checking on my case, I found out that I am two UI grid in my page, with same
 $scope.gridApi = gridApi;
so what is happening is a conflict BTW the gridApi on both grids, and when I checked the items on the first gird , the gridApi is working on the second one that is having mo items yet :)
anyway I Listen here is we should use different "$scope.gridApi" for different grids on the same page!
Thanks lpand for your help.

You should really do $scope.gridApi1 and $scope.gridApi2 if you have two grids in the same controller.

how can I highlight ui-grid row on selection/click, which has 6 pinned columns?


Reconstruct the parsing for sqlTokens

Fixes #2275.

Pull Request Test Coverage Report for Build 7886

8 of 88 (9.09%)  changed or added relevant lines in 37 files are covered.
8 unchanged lines in 7 files lost coverage.
Overall coverage decreased (-0.02%) to 55.712%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/extractor/impl/common/table/TableExtractor.java
0
1
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/filler/common/dml/InsertColumnsFiller.java
0
1
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/filler/common/dml/SelectItemFiller.java
0
1
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/filler/encrypt/dml/EncryptSetAssignmentsFiller.java
0
1
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/filler/sharding/ShardingColumnFiller.java
0
1
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/filler/sharding/ShardingTableFiller.java
0
1
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/filler/sharding/dml/update/ShardingSetAssignmentsFiller.java
0
1
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/sql/segment/dml/column/ColumnSegment.java
0
1
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/sql/token/ItemsToken.java
0
1
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/sql/token/TableToken.java
0
1
0.0%

Files with Coverage Reduction
New Missed Lines
%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/old/parser/sql/dml/insert/InsertParserFactory.java
1
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/old/parser/dialect/postgresql/sql/PostgreSQLInsertParser.java
1
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/old/parser/dialect/sqlserver/sql/SQLServerInsertParser.java
1
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/sql/segment/common/TableSegment.java
1
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/old/parser/dialect/mysql/sql/MySQLInsertParser.java
1
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/old/parser/dialect/oracle/sql/OracleInsertParser.java
1
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/antlr/sql/segment/dml/item/ShorthandSelectItemSegment.java
2
0.0%

Totals

Change from base Build 7885:
-0.02%

Covered Lines:
8724

Relevant Lines:
15659

💛  - Coveralls


I wonder if this loss can work when the foregrounds are very small

Hi,
It may NOT, due to imbalanced label problem, but you could add a hyperparameter/weights between region_inside and region_outside.

thanks for your reply


Invalid RUST_SRC_PATH

Installed rust 1.6.0 via rsvm install 1.6.0 and activated it via rsvm use 1.6.0.
$RUST_SRC_PATH points to $HOME/.rsvm/current/src/rustc-source/src but rustc-source is missing inside of $HOME/.rsvm/current/src/.

@fpauser can test rsvm install 1.6.0 --with-rustc-source?
I will add to help message next time, and disable env variable if not installed source :'(

Thanks for the hint - this works perfectly!
Maybe it would be a good idea to install rust-versions with their rustc-sources by default and to provide an optional --without-rustc-source?

Ok. It will be help to many developers. I will change that :)

Hey, sounds great, thanks!


Added resort command that user can list person lists in a reversed…

… order

It's really hard to pass this code style checking...

@XinyiYS Please check the enhancement I added to this project... Thanks!


Added option to show bandwidth in camera stream

I think the text for the bandwidth should be in the build method for CameraStreamWidget rather than the MjpegStream, as it gives more control over the position of the text relative to the stream, and also gives control on what it should display when disconnected from network tables


Combine and simplify item decorators.

Currently, items are declared like:
import attrs
from zyte_common_items.util import export

@export
@attrs.define(slots=True, kw_only=True)
class SomeItem:
    ...

It would be nice if we can combine the two decorators into a single one to have a more concise code.

We can also drop the export decorator :)


[stable/prometheus-adapter] v0.4.1, readOnlyRootFilesystem, fixes

What this PR does / why we need it:

Bump version to v0.4.1
Set readOnlyRootFilesystem to true to improve security
Increases default metricsRelistInterval to 1m as recommended by upstream to address issues with unavailable metrics
Fixes the below error on helm install/upgrade:

Warning: Building values map for chart 'prometheus-adapter'. Skipped value (map[]) for 'resources', as it is not a table.

Which issue this PR fixes
Special notes for your reviewer:
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/assign @davidkarlsen
/assign @lachie83

/ok-to-test

/lgtm


Locking a mutex failed (sr_connect: Operation timed out)

When the netopeer server inited, it called sr_connect() to connect to sysrepo. This error occurred.
netopeer2 version: v1.1.34
libsysrepo version: v1.4.66
Why ? And, what I can do to solve it ?
Thanks！
I readed libsysrepo src code, and found this:
if ((err_info = sr_mlock(&main_shm->lydmods_lock, SR_MAIN_LOCK_TIMEOUT * 1000, __func__))) {
    goto cleanup_unlock;
}

Duplicate of sysrepo/sysrepo#2840.


Invalid format for BUILD_TAGS specified in web site

Page https://docs.facette.io/latest/installing/
states

build a version that suits to specific needs set BUILD_TAGS variable with a list a revelant tags, separated by commas.

You must use space instead of comma. For sample

make BUILD_TAGS="builtin_assets disable_connector_influxdb"

Also disable_driver_mysql disable_driver_pgsql must be used instead of disable_backend_mysql disable_backend_pgsql

Issue duplicated in the associated repository, see https://github.com/facette/facette-docs/issues/19.


Support 10-bit y4m.

FFmpeg has an extension to y4m that supports 10-bit and above. rav1e will need this at some point.
I'm not sure if it's best to just return a [u8] for this or try to add methods that return [u16].

I think the second option is better.

Thinking a bit more, it might be preferable to get planes as raw u8 values even if bitness is more than 8. E.g. for stream copying it doesn't make sense to convert to 10-bit values back and forth.
We may later add e.g. get_y_plane_native(&self) -> Iter<u16> iterators which would convert to the original bitness on the fly.


KAFKA-10370: rewind consumer to SinkTaskContext's offsets when init

As mentioned in https://stackoverflow.com/questions/54480715/no-current-assignment-for-partition-occurs-even-after-poll-in-kafka, another potential resolution could be "call consumer.seek(tp, offsets)" in onPartitionsAssigned() callback after subscribing


[T5A2][T16-B1] Ong Li-Jen

Changed files to add an isMutating override function and followed LSP to add an exception for command.

Ready for review


no configure file

I'm trying to install 2.3.4 on EDA Playground (in a docker container) and the run is failing because there is no "configure" file. Which, looking at https://github.com/accellera-official/systemc, there isn't. I am following the instructions at https://github.com/accellera-official/systemc/blob/master/INSTALL.md, which is exactly what I did for 2.3.0 to 2.3.3) What am I missing?
Step 13/30 : RUN git clone https://github.com/accellera-official/systemc.git
---> Using cache
---> dd3c9b94d95d
-- snip --
Step 16/30 : RUN ls -l systemc
---> Running in 34525cf7d509
total 196
-rw-r--r-- 1 root root  2195 Jul 30 09:16 AUTHORS.md
-rw-r--r-- 1 root root 38946 Jul 30 09:16 CMakeLists.txt
-rw-r--r-- 1 root root  5640 Jul 30 09:16 CONTRIBUTING.md
-rw-r--r-- 1 root root 30718 Jul 30 09:16 INSTALL.md
-rw-r--r-- 1 root root 11358 Jul 30 09:16 LICENSE
-rw-r--r-- 1 root root  2633 Jul 30 09:16 Makefile.am
-rw-r--r-- 1 root root  3315 Jul 30 09:16 NOTICE
-rw-r--r-- 1 root root  4631 Jul 30 09:16 README.md
-rw-r--r-- 1 root root 34997 Jul 30 09:16 RELEASENOTES
drwxr-xr-x 2 root root  4096 Jul 30 09:16 cmake
drwxr-xr-x 2 root root  4096 Jul 30 09:16 config
-rw-r--r-- 1 root root 22074 Jul 30 09:16 configure.ac
drwxr-xr-x 4 root root  4096 Jul 30 09:16 docs
drwxr-xr-x 6 root root  4096 Jul 30 09:16 examples
drwxr-xr-x 3 root root  4096 Jul 30 09:16 msvc10
drwxr-xr-x 5 root root  4096 Jul 30 09:16 src
Removing intermediate container 34525cf7d509
---> c1df43997ad1
Step 17/30 : RUN mkdir -p systemc/objdir
---> Running in 81a1c130afd4
Removing intermediate container 81a1c130afd4
---> 0215055ffb6d
Step 18/30 : WORKDIR /install_data/systemc/objdir
---> Running in 1e838946a3e2
Removing intermediate container 1e838946a3e2
---> 0d42f997d8ac
Step 19/30 : RUN mkdir -p /playground_lib/systemc-2.3.4
---> Running in 03ac688a2921
Removing intermediate container 03ac688a2921
---> e8455ffdb159
-- snip --
Step 23/30 : RUN ../configure --prefix=/playground_lib/systemc-2.3.4
---> Running in 157f3742ee7a
/bin/sh: 1: ../configure: not found
The command '/bin/sh -c ../configure --prefix=/playground_lib/systemc-2.3.4' returned a non-zero code: 127

SystemC 2.3.4 is not released by Accellera (see https://www.accellera.org/downloads/standards/systemc) so it would be rather confusing for the community to release something under this version number. Actually this repository is based on SystemC 2.3.3 and includes the bug fixes. I suggest we rename the configure script first with the version 2.3.3_fixes before you publish on EDA playground.
Is there is specific reason why you need this version instead of the latest Accellera stable SystemC 2.3.3 ?

No. I know very little about SystemC, but had just installed 2.3.3 on EDA Playground and a colleague asked why not 2.3.4. I said, because 2.3.3 was the latest version on the Accelera website and asked where I could find 2.3.4. He pointed me to here. But if it's not officially released then that explains why it's not on the Accelera website and I'll wait until it is released. Thanks for your help.


Created permutations of string.md file

Hi, Please read the Important Announcement.
PR can only be made after the issue is assigned to you.


Unable to start selenoid with safari browser

I'm trying to start selenoid docker with safari browser, when I run docker-compose up -d having
{
"safari": {
"default": "latest",
"versions": {
"latest": {
"image": ["/usr/bin/safaridriver"]
}
}
}
}
in browsers.json file and then try to open manual session to verify if browser is present. It doesn't open the browser and I checked logs which shows,
2020/03/04 13:03:12 [7] [STARTING_PROCESS] [[/usr/bin/safaridriver --port=42995]]
2020/03/04 13:03:12 [7] [SERVICE_STARTUP_FAILED] [cannot start process [/usr/bin/safaridriver --port=42995]: fork/exec /usr/bin/safaridriver: no such file or directory]
I have added safaridriver in /usr/bin in my host. Is there any thing more to be done to get safari started?
Thanks in advance.

@ratankumarangadi Selenoid in that case should be running as a process and not as Docker container.

@vania-pooh oh okay , so is there no way to make Safari this way? instead of running VM inside container? Sorry I read about the solution of running a VM inside container for Mac and Windows if I am not wrong

There are no public available images for MacOS and Windows.
ср, 4 мар. 2020 г., 22:07 ratankumarangadi<EMAIL_ADDRESS>
@vania-pooh https://github.com/vania-pooh oh okay , so is there no way
to make Safari this way? instead of running VM inside container? Sorry I
read about the solution of running a VM inside container for Mac and
Windows if I am not wrong
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/aerokube/selenoid/issues/878?email_source=notifications&email_token=AAKY23MZYG4EA73F57RC3MTRF2RINA5CNFSM4LBFZJE2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENZVBTY#issuecomment-594759887,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAKY23JM6AQGE32JPBN22LDRF2RINANCNFSM4LBFZJEQ
.

@ratankumarangadi yes, you are right. Such approach will work but anyway you need to build such images yourself using these instructions: https://github.com/aerokube/windows-images

Just to follow up , should this work ? https://docs.microsoft.com/en-us/microsoft-edge/webdriver-chromium#docker Microsoft has pre-built image with microsoft Chromium which is latest edge browser

@ratankumarangadi yes, it should.


Can't run in macos Apple M2 Ultra

build in macos，
the log as following：
(base) fanmac@M2-Ultra-Lan SenseVoice.cpp % ./build/bin/sense-voice-main -m ./models/gguf-fp32-sense-voice-small.bin ./asr_example_zh.wav -t 4
sense_voice_small_init_from_file_with_params_no_state: loading model from './models/gguf-fp32-sense-voice-small.bin'
sense_voice_init_with_params_no_state: use gpu    = 1
sense_voice_init_with_params_no_state: flash attn = 0
sense_voice_init_with_params_no_state: gpu_device = 0
sense_voice_model_load: version:      3
sense_voice_model_load: alignment:   32
sense_voice_model_load: data offset: 444480
sense_voice_model_load: loading model
sense_voice_model_load: n_vocab = 25055
sense_voice_model_load: n_encoder_hidden_state = 512
sense_voice_model_load: n_encoder_linear_units = 2048
sense_voice_model_load: n_encoder_attention_heads  = 4
sense_voice_model_load: n_encoder_layers = 50
sense_voice_model_load: n_mels  = 80
sense_voice_model_load: ftype  = 1
sense_voice_model_load: vocab[25055] loaded
sense_voice_model_load: Metal total size =   935.21 MB
sense_voice_model_load: n_tensors: 1197
sense_voice_model_load: load SenseVoiceSmall takes 0.317000 second
sense_voice_backend_init_gpu: using Metal backend
ggml_metal_init: allocating
ggml_metal_init: found device: Apple M2 Ultra
ggml_metal_init: picking default device: Apple M2 Ultra
ggml_metal_init: using embedded metal library
ggml_metal_init: GPU name:   Apple M2 Ultra
ggml_metal_init: GPU family: MTLGPUFamilyApple8  (1008)
ggml_metal_init: GPU family: MTLGPUFamilyCommon3 (3003)
ggml_metal_init: GPU family: MTLGPUFamilyMetal3  (5001)
ggml_metal_init: simdgroup reduction support   = true
ggml_metal_init: simdgroup matrix mul. support = true
ggml_metal_init: hasUnifiedMemory              = true
ggml_metal_init: recommendedMaxWorkingSetSize  = 154618.82 MB
sense_voice_init_state: kv pad  size  =    3.67 MB
ggml_backend_metal_buffer_type_alloc_buffer: error: failed to allocate buffer, size =     0.00 MiB
ggml_gallocr_reserve_n: failed to allocate Metal buffer of size 0
zsh: segmentation fault  ./build/bin/sense-voice-main -m ./models/gguf-fp32-sense-voice-small.bin  -t

Unfortunately, the Metal GPU backend is not supported yet, only the CPU can be used for now. I’ll do my best to adapt it.

cool work, I run it ok by -ng parameter.


add IUPACSequence.to_regex method

IUPACSequence.to_regex should return a regular expression object that accounts for degenerate characters. There's a quick implementation of this in the SciPy 2015 talk here.

@patena is working on this.

Implementation from scipy talk:
def seq_to_regex(seq):
    result = []
    for base in str(seq):
        if base in DNA.degenerate_chars:
            result.append('[{0}]'.format(
                ''.join(DNA.degenerate_map[base])))
        else:
            result.append(base)

    return ''.join(result)


Add a feature flag to disable inline spec

By default, the inline specs will be enabled. Only if the flag is set to true, the inline spec would be disabled. This is to increase security of pipelines.
Fixes #5359

Changes

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

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

Release Notes
NONE

/kind feature

/test pull-tekton-pipeline-integration-tests

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

/hold cancel

/retest all

/test all

We probably want to update PipelineRun, TaskRun and Pipeline specs to also highlight that inline/embedded spec might be disabled by this configuration (linking to that heading)

How do I update these specs? Did you mean document? Or description on parameter?

We should cover Pipeline as well as it allows for embedded Taskspec.

Should I disable PipelineSpec also for Pipeline which is used for Pipeline in Pipeline feature?  We can still use pipelineRef.

/assign @Chitrang @Andrea Frittoli
Can we get this for v0.59.0? Some of our customers are waiting for this.

I was wondering why we need everything to be referenced? If the top-level config is referenced, isn't that already good enough? i.e. for a single TaskRun, we need the Task to be referenced but for a PipelineRun, we need the pipeline to be referenced.
That's because if we can trust the Pipeline then we should be able to implicitly trust the pipelineTasks.
WDYT?

/approve cancel

That's because if we can trust the Pipeline then we should be able to implicitly trust the pipelineTasks. That way, even if certain pipelineTasks need to be inlined by the author, thats ok.

That's what I was thinking also but the use-case came to not give anyone any ability to create a pipeline without defined tasks isn't satisfied. So only tasks referenced from the clusters can be utilized in the pipelines.

/test pull-tekton-pipeline-alpha-integration-tests

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

/approve

That's because if we can trust the Pipeline then we should be able to implicitly trust the pipelineTasks. That way, even if certain pipelineTasks need to be inlined by the author, thats ok.

That's what I was thinking also but the use-case came to not give anyone any ability to create a pipeline without defined tasks isn't satisfied. So only tasks referenced from the clusters can be utilized in the pipelines.

What about remote resolution?
I guess that needs to be disabled too, otherwise someone could reference a malicious Task somewhere.

My original requirement was to be able to safely grant developers the ability to create PipelineRuns (and maybe TaskRuns) without essentially handing them admin in the namespaces, due to inline task specs allowing pods to be created to execute any code, using any service account and any secret.
I'm not very familiar with resolvers so this may also need locking down.
Pipelines with inline task specs would still be okay, because RBAC could enforce developers are not able to create Pipelines themselves - they would only have the ability to create PipelineRuns that use safe Pipelines authored by someone else.

@afrittoli remote resolution can already be disabled (from the configuration — and by not deploying the resolvers).
Here, we only need to handle inline spec. We could go ahead and only disallow them on the Run types and not Pipeline for example (as @chitrangpatel said). My suggestion was more for consistency but maybe it's gonna be too much – or we should have a different feature-flag, aka not a boolean but something where we can list things we allow or disallow ?

Here, we only need to handle inline spec. We could go ahead and only disallow them on the Run types and not Pipeline for example (as @chitrangpatel said). My suggestion was more for consistency but maybe it's gonna be too much – or we should have a different feature-flag, aka not a boolean but something where we can list things we allow or disallow ?

Yea. Having inline disabled everywhere is more consistent behavior.

Thinking outloud, maybe something like the following could work as well ?
# all
disable-inline-spec: "all"
# only pipelinerun
disable-inline-spec: "pipelinerun"
# only taskrun
disable-inline-spec: "taskrun"
# a mix
disable-inline-spec: "pipelinerun,pipeline"

Not sure it's worth doing it but, throwing it out there.

I like this. It gives users more control. We don't have to implement them all right now.

Right, but we have to decide on this vs "boolean" now though 😛

I like this. It gives users more control. We don't have to implement them all right now.

Right, but we have to decide on this vs "boolean" now though 😛

I vote for string based value over a boolean.

@khrm ^^

@vdemeester Sure. I will change it to string implementation.

# all
disable-inline-spec: "all"
# only pipelinerun
disable-inline-spec: "pipelinerun"
# only taskrun
disable-inline-spec: "taskrun"
# a mix
disable-inline-spec: "pipelinerun,pipeline"

What about:
# all
disable-inline-spec: "pipelinerun,taskrun,pipeline"
# only pipelinerun
disable-inline-spec: "pipelinerun"
# only taskrun
disable-inline-spec: "taskrun"
# pipelinerun and pipeline
disable-inline-spec: "pipelinerun,pipeline"

Does `all` make sense?

/test pull-tekton-pipeline-integration-tests

@vdemeester you're review is still marked as "changes requested" - I believe the changes you requested have been implemented. Since you're on leave for some time, I will take the liberty of dismissing your "changes requested" now, so that we may proceed with this PR. We can address any further concern as a follow-up.


refactor: combine both submodules

Description
Combine the analyzer and combine submodules into one as Picocli can handle subcommands. So no need to build two different programs.
Checklist

[x] My code follows the style guidelines of the project
[x] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation

:tada: This PR is included in version 1.11.5 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Hygiene DAL error

Clicking "Permissions" in Office 365 Security & Compliance gives a client error of "The Hygiene DAL encountered a permanent exception." Following the instructions to do the same in powershell gives the same error.
Is there a workaround to this?

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

ID: 99e7332f-6c1d-19cf-dc71-4cebaa93c1cc
Version Independent ID: a8e4bd77-725f-af7a-63bc-95d8ea92f554
Content: Give users access to the Office 365 Security & Compliance Center - Office 365
Content Source: microsoft-365/security/office-365-security/grant-access-to-the-security-and-compliance-center.md
Service: o365-seccomp
GitHub Login: @chrisda
Microsoft Alias: chrisda

@deniz-yilmaz - Thank you for submitting feedback.
I think the best way forward is if you can open a service ticket in your tenant so this can get resolved ASAP. Based on outcome, let us know if it is something that can be called out in the docs.
If you don't mind please keep us posted here on the resolution so we can get the applicable information you are discovering back into the docs.


Generated Assets File has variable names with backslashes

When I run any of the build commands, the Assets file is broken because the variable names it creates contain backslashes.
Here is an example with only one asset in assets/images.
// GENERATED CODE - DO NOT MODIFY BY HAND

// **************************************************************************
// Total assets: 1.
// Generated by https://pub.dev/packages/assets_gen.
// **************************************************************************

// ignore_for_file: constant_identifier_names,non_constant_identifier_names,lines_longer_than_80_chars
class Assets {
  static const String package = 'mypackage';

  static const String assets\images\bottomCurvesPng = 'assets\images\bottom-curves.png';
  static const String mypackage$assets\images\bottomCurvesPng = 'packages/mypackage/assets\images\bottom-curves.png';
}

I have no idea if I'm doing something wrong or what the issue is.
Would be nice if it can be addressed any time soon.
Thanks!

Hi, are you using a windows os?

Yes.
Ah, that would have been good information to provide.

I'll fix this issue tomorrow, thanks.

I don't have a windows computer at hand, could you please try 1.5.0 and give me a feedback if it solves this issue? 😄

My setup is like this:
assets_gen:
  class_name: Assets
  code_style: lowerCamelCase
  format_dart_code: true
  exclude:
    - assets/i18n/**
  output: assets.dart

I have one image "some-image.png" in the subfolder images.
I'd expect, that after generating I can access it with Assets.images.someImage
But actually I have two constants, and they look like this:
Assets.assetsImagesSomeImagePng;
Assets.bildersprache$assetsImagesSomeImagePng;

So yes, it works in a way that it doesn't produce an error anymore, but it doesn't work as expected (described).

Yeah, assets_gen generates a "single-line"(not nested) file as so far, and it will provide a nested production in the future.


Add Custom Message option to can_definition

I would like to continue this unresolved CanCan issue here.
https://github.com/ryanb/cancan/issues/103
As was mentioned by @mcfiredrill, there's no way to have different authorization error messages for the same action+subject combination but for different rules. It's quite important to be able to tell the user not only that he is not able to do something, but also why he isn't able to do it. So it would be really great if there's a way to specify a separate error message for every rule (fallbacking to existing message mechanism, if not specified).

+1

+1

@Exoth :+1: I can't say this would be a super high priority at the moment, but it would be something I could see coming into the project for sure.

+1

Am I wrong or this is already implemented? https://github.com/ryanb/cancan/commit/a5f838a964f8717c3243eb6e2d5cd3a949a8986a#diff-d4e0a70d43eb4c389e623945f417d6e0R203
You have simply to define them into yml file
en:
  unauthorized:
    default: "You don't have the required permissions to execute this action"
    create:
      group: "You cannot create the group because you didn't pay enough"

Thanks! It seems like you are right.
I followed this article and it seems to do the job for my purposes:
https://github.com/CanCanCommunity/cancancan/wiki/exception-handling

can you make an example?

can :edit, Message do |message|
  current_user.present? && message.creator == current_user
end

not logged in -> "You need to be logged in to edit a message"
wrong creator -> "You can only edit your own messages"

Nice example. Any suggestions on how to implement such feature in your case? Anyway, hope this will be implemented in CanCanCan

From my point of view this feature would try to archive too much. What if you want to display different partials or execute different algorithms depending on the same condition? Cancancan cannot  (and shouldn't) manage that

I tend to agree with @coorasse here, we want to keep cancancan's responsibilities as limited to authorization as possible.
What message is displayed to the user is the responsibility of either the controller or the view layer.
Typically "not logged in" is an authentication failure, not an authorization failure and should be handled at that layer of the application

What is we need to show different messages for two different rules like that:
1)logged in, but subscription is not paid - > "you have to subscribe to edit a message"
2)logged in, subscribed, but tries to edit someone's else message -> "sorry, you can't edit a message created by other гыукЭ
In that case we still need somehow to understand what rule is used to deny access.


fix: (helm) service template now respects defined port and is only templated if prometheus is enabled

I've noticed that the service port as defined in values.yaml is not respected. As such I created a pull request to respect the value and, by doing so, noticed that the service is only used for metrics. Therefore the service can also be only templated if prometheus.enabled is true.
Let me know if anything is wrong.

@rafaribe Please squash the commits into one and sign-off the resulting commit (see https://docs.cilium.io/en/v1.11/contributing/development/contributing_guide/#developer-s-certificate-of-origin).

@rafaribe Please squash the commits into one and sign-off the resulting commit (see https://docs.cilium.io/en/v1.11/contributing/development/contributing_guide/#developer-s-certificate-of-origin).

There you go.

There seem to be two errors in the static checks:
  Signed-off-by: Rafael Ribeiro<EMAIL_ADDRESS>  Running on 1af8247a1e04df7e76846266ff48714db3c7860a
  Error: ERROR:MISSING_SIGN_OFF: Missing Signed-off-by: line(s)

  NOTE: For some of the reported defects, checkpatch may be able to
        mechanically convert to the typical style using --fix or --fix-inplace.

  "[PATCH]  fix: (helm) service template now respects defined port and" has style problems, please review.

  NOTE: Ignored message types: C99_COMMENTS C99_COMMENT_TOLERANCE COMMIT_LOG_LONG_LINE COMMIT_MESSAGE COMPLEX_MACRO CONSTANT_CONVERSION CONST_STRUCT EMAIL_SUBJECT FILE_PATH_CHANGES FROM_SIGN_OFF_MISMATCH GIT_COMMIT_ID JIFFIES_COMPARISON LEADING_SPACE LONG_LINE_COMMENT MACRO_WITH_FLOW_CONTROL MULTISTATEMENT_MACRO_USE_DO_WHILE NOT_UNIFIED_DIFF PREFER_DEFINED_ATTRIBUTE_MACRO PRINTK_WITHOUT_KERN_LEVEL TRAILING_SEMICOLON TRAILING_STATEMENTS VOLATILE

  NOTE: If any of the errors are false positives, please report
        them to the maintainer, see CHECKPATCH in MAINTAINERS.
  Error: ERROR:CUSTOM: Please avoid long commit subjects (max: 75, found: 155)

I don't understand why the Signed-off-by failure is there, the lines seem identical to me.

I've issued the commit again using git commit -S -s -m "<message>"  Should be fixed now.

I've issued the commit again using git commit -S -s -m "<message>"  Should be fixed now.

Ah, thanks! above seems to fix the signing issue. Could you also change the commit message so that the first line it's shorter?
How about:
helm: fix metrics template

template now respects defined prometheus port (tetragon.prometheus.port) value.
Also, it only adds the service if prometheus is enabled
(tetragon.prometheus.enabled)


Exposing Element node_index

I currently have an use case in which I'd like to filter an iterator of elements that fall inside a range of node indices.  As of 0.3.2 however, node_index is a private field, and I couldn't find an existing public function that returns it.
My proposed solution is an additional function of the same name,
pub fn node_index(&self) -> usize

Currently I have a working example with a test to cover.  In this case, I thought the sane thing to would be to follow precedent  where, for example, the text field was exposed via a text() function.
That sound okay @yggie ?
PR: https://github.com/yggie/rquery/pull/6

@ancamcheachta that all looks great, I took the liberty of bumping the version published on crates.io to include the changes you introduced, let me know if this solves your use case.

@yggie indeed, that's solved.  Thanks for that!
On a separate note, it seems to me the documentation page linked in the crate (https://yggie.github.io/rquery/rquery) is frozen on 0.2.0.  Is that deliberate?  If not I can open new issue/PR.  docs.rs would appear to have the latest

@ancamcheachta nope that isn’t deliberate, I will update that shortly


startNfc() not returning firstname and lastname

the startNfc() is not returning firsname and lastname. It only returns expiryDate, cardNumber and cardType.

names should work in this :  branch

i tried for 3 cards and with 2 of them i couldnt get first and last name.
here what i get;
firstName: "java.util.regex.Matcher[pattern=([0-9]{1,19})D([0-9]{4})([0-9]{3})?(.*) region=0,38 lastmatch=5351774593097672D25112262942853500000F]"
lastName:"2942853500000F"

i tried for 3 cards and with 2 of them i couldnt get first and last name.
here what i get;
firstName: "java.util.regex.Matcher[pattern=([0-9]{1,19})D([0-9]{4})([0-9]{3})?(.*) region=0,38 lastmatch=5351774593097672D25112262942853500000F]" lastName:"2942853500000F"
can you help me with that @jackbayliss

I got the similar output


Can't use GroupHandler in FingersCrossedHandler

When I try to use GroupHandler in FingersCrossedHandler I got an exception:
UnexpectedValueException(code: 0): The nested handler of type Monolog\\Handler\\GroupHandler does not support formatters. at /app/vendor/monolog/monolog/src/Monolog/Handler/FingersCrossedHandler.php:237
FingersCrossedHandler expects that wrapped handler will implement FormattableHandlerInterface and GroupHandler doesn't implement it.

Could you add the code to show what you are trying to do?

This should work fine as long as you don't configure a formatter on the FingersCrossHandler, because then it must forward it to the nested handler and that will fail on GroupHandler.
But if you configure the formatters where they belong it should work fine.

Well, FingersCrossedHandler implements FormattableHandlerInterface so it means you can use setFormatter() method, but it works fine only if the "inner" handler also implements FromattablehandlerInterface.
GroupHandler doesn't implement it so it means you can't use it in FingersCrossedHandler and try to set formatter.
Kinda understand it, but Laravel doesn't :)
I was using Laravel and in LogManager it automatically tries apply formatter if handler implements FormattableHandlerInterface.
LogManager:
    protected function prepareHandler(HandlerInterface $handler, array $config = [])
    {
        if (isset($config['action_level'])) {
            $handler = new FingersCrossedHandler($handler, $this->actionLevel($config));
        }

        if (! $handler instanceof FormattableHandlerInterface) {
            return $handler;
        }

        if (! isset($config['formatter'])) {
            $handler->setFormatter($this->formatter());
        } elseif ($config['formatter'] !== 'default') {
            $handler->setFormatter($this->app->make($config['formatter'], $config['formatter_with'] ?? []));
        }

        return $handler;
    }

I will find workaround. Probably I will need to write own Handler which extends FingersCrossedHandler.
You can close this one, I think.
Thanks

Ok, IMO this should be reported to laravel and fixed there. Having a default formatter across all handlers makes little sense as they have such different needs. So if a specific formatter isn't configured on a given handler it shouldn't call setFormatter at all I'd say.
Every handler comes with its own sensible default formatter anyway out of the box so overriding that with a single one at framework level is just harmful.


Jetbrains Plugin is very out of date - please update

Bug description
Please update the Jetbrains prisma plugin to be compatible with the most recent features. From what I can tell, it's at least 2 months behind the VSCode one, resulting in an inability to use new features.
For example, attempting to use prismaSchemaFolder in previewFeatures while on version 5.17.0 results in:
The preview feature "prismaSchemaFolder" is not known. Expected one of: deno, driverAdapters, fullTextIndex, fullTextSearch, metrics, multiSchema, nativeDistinct, postgresqlExtensions, tracing, views, relationJoins. If this is unexpected, it might be due to your Prisma VS Code Extension being out of date.

That last sentence is a typo on your end, as this is in Jetbrains and not VS Code. All of the recent reviews are asking for the same thing:

How to reproduce

Install prisma plugin in Jetbrains
Try to use a feature introduced in 5.15

Expected behavior
I can use features at the same time VSCode can
Prisma information
Version: 5.17.0
Environment & setup

OS: Windows
Database: PostgreSQL
Node.js version: v18.15.0

Prisma Version
prisma                  : 5.17.0
@prisma/client          : 5.17.0
Computed binaryTarget   : windows
Operating System        : win32
Architecture            : x64
Node.js                 : v18.15.0
Query Engine (Node-API) : libquery-engine 393aa359c9ad4a4bb28630fb5613f9c281cde053 (at<EMAIL_ADDRESS>Schema Engine           : schema-engine-cli 393aa359c9ad4a4bb28630fb5613f9c281cde053 (at<EMAIL_ADDRESS>Schema Wasm             : @prisma/prisma-schema-wasm 5.17.0-31.393aa359c9ad4a4bb28630fb5613f9c281cde053
Default Engines Hash    : 393aa359c9ad4a4bb28630fb5613f9c281cde053
Studio                  : 0.502.0

Hey @flipperbw, the Prisma Jetbrain extension is developed by Jetbrains themselves. Please open an issue on their repository https://github.com/JetBrains/intellij-plugins/tree/master/prisma. Thank you 🙏


Add a build option for watching the file system

The command line is --watch-fs.
Fixes #12053

The build option approach currently doesn't work, since they are picked up too late. https://github.com/gradle/gradle/pull/13075 would take care of that.

It seems like https://github.com/gradle/gradle/pull/13075 isn't enough to make a build option work for file watching. File watching is enabled in a afterStart hook. With the changes from https://github.com/gradle/gradle/pull/13075, the start parameter is still updated too late.
So either we start watching later, or we load the environment earlier.

@lptr PTAL!
After this PR we probably should do a round of renaming and replace VfsRetention with WatchFileSystem in our code.

@lptr PTAL!


My second ko translation.

This has translated this means.
265 : Holy smokes -> Surprise [놀라다]

Thanks :+1:


Sample Program for Receiving C2D messages in Python

@schirag1993
We are working on it and the Python Service Client will be in the next release.
Also moving this issue to the Python repo: https://github.com/Azure/azure-iot-sdk-python
Thanks,
Zoltan

Closing as the Python Service Client has been released:
https://github.com/Azure/azure-iot-sdk-python/releases
Thanks,
Zoltan


报错：AttributeError: 'FeedExporter' object has no attribute 'slot' & IndexError: list index out of range

You need to modify pornhub.csv permissions or modify the path of CVS in setting.py


Distributed v2 - review OpLog

[x] pruning
[ ] reconciliation

Implemented truncate of oplog for reconciliation.
Needs to be integrated with the tx flow


Support vLLM for beacon models

https://github.com/vllm-project/vllm/issues/2676

Hi, Activation Beacon introduces extra parameters to the Llama model, including the beacon_embed_tokens.weight in your issue. You can try to use our modeling_llama.py instead. Nevertheless, supporting vLLM is currently not our priority.


Please do not change the font style of anything by default

https://github.com/QB64Official/vscode/blob/3f6e6633a6d325d660b981a4bdbab245f59e64af/src/decoratorFunctions.ts#L95
https://github.com/QB64Official/vscode/blob/3f6e6633a6d325d660b981a4bdbab245f59e64af/src/decoratorFunctions.ts#L100

We can instead use the settings.json and "fontStyle": "bold" in the "settings" block if bold is desired.
I cannot unset it, I have tried.
    "editor.tokenColorCustomizations": {
        "textMateRules": [
            {
                "scope": [
                    "userfunctions.QB64",
                    "entity.name.function.QB64",
                    "meta.function.QB64",
                    "source.QB64",
                    "entity.name.function"
                ],
                "settings": {
                    "fontStyle": ""
                }
            }
        ]
    }

The setting is working, just is bold by default which should not be forced on the user.

Agreed.  This and any new font styles should be controlled by a setting.

This is ready to test in dev.

The setting doesn't seem to work, and the highlighting does not factor int the return type variety:

I have in my settings:
    "qb64.isBoldingSubsAndFunctionsEnabled": true,

And it's not bolded.
Note that a function that ends in a type suffix is also not matched.

Actually. I think it was the font rendering on the Mac. I was using my Mac.
On PC, it's absolutely working.
Sorry for the false info @LordDurus

There is a bug though. It looks like you need negative lookahead for '

See how it's highlighting the comment too?

Also some strange formatting here:

Attaching the .BAS for these bugs for your own tests.
MONEY.zip

Another strange one:

From my strings.bi here:
''
' QB64_GJ_LIB
' GRYMMJACK'S STRINGS LIB
'
' Some commonly used functions that I missed in QB64 coming from PHP
'
' USAGE:
'   Insert '$INCLUDE:'path_to_GJ_LIB/STRINGS/STRINGS.BI' at the top of file
'   Insert '$INCLUDE:'path_to_GJ_LIB/STRINGS/STRINGS.BM' at the bottom of file
'
' @author Rick Christy<EMAIL_ADDRESS>' @uses STRINGS.BM
'
$LET GJ_LIB_STRINGS_INC_BI = 1
CONST GJ_LIB_MAX_STR_POSITIONS = 50

DECLARE FUNCTION str_implode$ (arr$(), delim$)
DECLARE SUB str_explode (target$, delim$, dest$(), numParts%)
DECLARE SUB str_find_pos (target$, search$, arrFound%(), numFound%)
DECLARE FUNCTION str_insert$ (s$, ins$, p%)
DECLARE FUNCTION str_remove$ (s$, del$)
DECLARE FUNCTION str_del (s$, del$)
DECLARE FUNCTION str_replace$ (s$, search$, replace$)
DECLARE FUNCTION str_rep$ (s$, search$, replace$)
DECLARE FUNCTION str_slice_pos (s$, startPos%, endPos%)

Last thing: good job on fixing the type suffixes on functions and subs highlighting. Works good:

I think I have the issue with declare function fixed.
The problem was the regex was matching function and not declare function.

Still need to fix:

I think I have it working.  Let me know if you find anything else.
I pushed this to dev in v.0.9.1

Testing :)

Found another case of the comments getting highlighted (GOTO command)

Took awhile, but looks good now


TSV file columns not rendering on website

Hi,
I have a bunch of tab-delimited tsv files in an OSF project. Most of them render correctly on the website, but for 2 files the separate column values are being squished into a single column. I have checked the downloaded files and the delimiters are correct. I've also checked that web browser/versions are not causing the problem.
Here are the 2 files with the incorrect display:

Here is an example of another tsv displaying correctly:

I'd appreciate any suggestions for how to fix this!

@felliott looks like an issue for the MFR repo.

Hi - I wanted to follow-up on this issue, was there any resolution? thanks!

Hello maintainers - I'm having the same issue with one of my osf projects with TSV files. Some files are rendered as expected while others have a single cell per row. Is there any quick fix or solution to this issue? Thank you!


Headless render to texture

It should be possible to render to texture even without the application in the main Mixed Reality mode loop. This would be for applications that wish to draw a flatscreen 3d viewport, and then transition in and out of MR mode.
Maybe this isn't render to texture? Perhaps its switching between flatscreen mode and mixed reality mode with a flexible output system?

In v0.3.9 preview.


Problem with finetuning Qwen 2.5 model

There is an error, when loading Qwen 2.5 model and trying to finetune it. Minimal code for reproduction (in Google colab, though I tried the same thing on 1 A100 gpu with unsloth[cu121-ampere-torch230] installation and there is the same error):
# installation step
!pip install unsloth
!pip uninstall unsloth -y && pip install --upgrade --no-cache-dir "unsloth[colab-new] @ git+https://github.com/unslothai/unsloth.git"

# prepare model
from unsloth import FastLanguageModel
import torch
max_seq_length = 2048 # Choose any! We auto support RoPE Scaling internally!
dtype = torch.bfloat16 # None for auto detection. Float16 for Tesla T4, V100, Bfloat16 for Ampere+
load_in_4bit = False # Use 4bit quantization to reduce memory usage. Can be False.

model, tokenizer = FastLanguageModel.from_pretrained(
    model_name = "unsloth/Qwen2.5-Coder-1.5B-Instruct",
    max_seq_length = max_seq_length,
    dtype = dtype,
    load_in_4bit = load_in_4bit,
)
model.to(torch.device('cuda'))

# example with some text
text = "def func_foo(): return None"
tokenized = tokenizer(text, return_tensors='pt').to(model.device)
tokenized['labels'] = tokenized['input_ids'].clone()

output = model(
    input_ids=tokenized["input_ids"],
    labels=tokenized["labels"],
    attention_mask=tokenized["attention_mask"],
)
output.loss.backward()

Output:
RuntimeError: one of the variables needed for gradient computation has been modified by an inplace operation: [torch.cuda.FloatTensor [1, 7, 1536]], which is output 0 of MulBackward0, is at version 1; expected version 0 instead. Hint: enable anomaly detection to find the operation that failed to compute its gradient, with torch.autograd.set_detect_anomaly(True).

@Pstva Wait you have to set up the HuggingFace Trainer since we internally update it. Also get_peft_model must be used


Support curl's --connect-to parameter

Proposal
Implement curl's --connect-to parameter
The draft PR https://github.com/tsenart/vegeta/issues/576 seems to work, and would be an improvement over the current situation.
I have rebased it and updated the usage message to provide slightly better guidance; result is https://github.com/tsenart/vegeta/pull/692
The remaining downside is that it doesn't support empty fields as curl does, but maybe the added example in the usage message will make it clear enough for new users.
(See https://github.com/curl/curl/blob/58772b0e082eda333e0a5fc8fb0bc7f17a3cd99c/lib/url.c#L2888 etc shows that the first two fields match any source if empty;
https://github.com/curl/curl/blob/58772b0e082eda333e0a5fc8fb0bc7f17a3cd99c/lib/url.c#L2782 shows that the final two fields may be empty and mean "no change" in that case.)
Background
In an environment where DNS always gives the wrong answer, and cannot be changed, it is common for people to use curl --connect-to to test connections and debug.  Let's make it easier for them to adopt vegeta.
Workarounds
Set up a custom DNS server, but that's not always convenient or possible.

Closed by 647dacc
Release: https://github.com/tsenart/vegeta/releases/tag/v12.12.0


Xcode 9.0 beta compile error

I can't compile your library (i used cocoa pods to attach it)

I will be working on Xcode 9 compatibility sometime this week. I'll make sure this is fixed.

post_install do |installer|     installer.pods_project.targets.each do |target|         if target.name == 'AmazonS3RequestManager'             #puts "#{target.name} "             target.build_configurations.each do |config|                 config.build_settings['SWIFT_VERSION'] = '3.2'             end         end     end end

Solution #66


1.7.10 Forge Server and Client, Connection gets Terminated right after successful join

So i setup a Proxy server handling connections from clients to a paper server (using protocolsupport plugin) fine, they then get redirected to my mod server fine aswell.
As soon as the client connected and the World is displayed on the client, the connection gets terminated and my client gives me a few stacktraces [1]. Travertine itself just gives me a single error on the console [2].
This happens aswell if i take the paper server out of the equation and just use Client -> Travertine -> Forge Server.
Client -> Forge server works without any issues.
After looking at the Stacktraces of my Client and some researching i think my problem is related by a custom packet generated by the "Code Chicken Lib" [3]
[1] https://paste.crystalnet.org/?da1008dd38712027#kVWjS0FEC/cGUMl6OjT3/0TzA1i0V5xeHe9Nw2ryh+8=
[2] https://paste.crystalnet.org/?c444e5f08707cd73#hxFAB6Kb8NyBWpp4aKVVa1U2Gw/blja4uNGhTP4M/Gk=
[3] https://github.com/TheCBProject/CodeChickenLib/blob/master/src/main/java/codechicken/lib/packet/PacketCustom.java

Additional Info:
Similar Modded 1.12 sponge server works fine…
Ill compare the mods loaded on each later on, just know that codechickenlib 1.8+ is in use there (not sure how extensively it was rewritten.)

I can plus one this issue that @psych0d0g  has had with the latest travertine build

Can confirm, this is still an issue.

Pastes have expired, would need to see new ones

Client log when Fatal Error happens - https://pastebin.com/dF67VVGa
Travertine just shows disconnected, this error does not happen on an older version of travertine that we use to prevent this from happening, as to what build that is I'm not sure.

By any chance, is the older version directly connecting to that modded server?
Does connecting to the modded server directly through the proxy (default server, forced hosts) does it work?

Hey @electronicboy this is when we used a forced host to connect so from our IP: inf.brierie.net which forced to our infinity server which is a 1.7.10 server

@electronicboy  The older version was connecting our servers from 1.7.10 to 1.12.2 the new version does 1.10.2 - 1.12.2 only it seems due to the issue above

The old build that we are using that works is Build-40 if that helps.

@electronicboy Think this is something that can be fixed as the new build of course does fix other issues with our 1.12.2 servers

This is something I'd need to look into, ForgeMultipart is throwing an exception, yet that exception is getting squashed out because of another exception thrown due to their bad exception handling, meaning that we can't see the original exception which caused the issue
@Keaton1188 There are 33 builds between now and 40, if you can identify the exact build/commit which broke it, that would be of much more use

I'll do some testing and will report back when we find out anything


refactor(meta): maintain snapshot of running actors instead of resolving it every time for barrier

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
Some changes in this PR:

maintaining a snapshot of running actors in barrier state, it will be incrementally update according to the actor changes of each barrier.
drop table fragments directly before issuing a drop barrier, which is the main purpose of introducing this refactoring, so that we can easily support sql-backend.
some changes on the drop and reschedule command to expose actor changes for barrier manager.

Checklist

[ ] I have written necessary rustdoc comments
[ ] I have added necessary unit tests and integration tests
[ ] I have added test labels as necessary. See details.
[ ] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[ ] My PR contains breaking changes. (If it deprecates some features, please create a tracking issue to remove them in the future).
[ ] All checks passed in ./risedev check (or alias, ./risedev c)
[ ] My PR changes performance-critical code. (Please run macro/micro-benchmarks and show the results.)

[ ] My PR contains critical fixes that are necessary to be merged into the latest release. (Please check out the details)

Documentation

[ ] My PR needs documentation updates. (Please use the Release note section below to summarize the impact on users)

Release note
If this PR includes changes that directly affect users or other significant modifications relevant to the community, kindly draft a release note to provide a concise summary of these changes. Please prioritize highlighting the impact these changes will have on users.

Does this resolve https://github.com/risingwavelabs/risingwave/issues/8015?

Does this resolve #8015?

👍 Yes exactly.

Is it possible (and easy) to keep the original impl so that we can add a debug assertion to check if everything work well with the incremental maintenance?

🥺

Is it possible (and easy) to keep the original impl so that we can add a debug assertion to check if everything work well with the incremental maintenance?

🥺

Oh, I added the assertion check in function pre_apply and post_apply. Is this what you mean to keep? 🥵