Correct config files typo

typos in #240 are corrected
Added a function to check Genome version.

I just wanted to be sure that we were using the right genome, but yeah,
make senses to allow people to use customs
On Fri, Mar 24, 2017 at 12:51 PM, Marcel Martin<EMAIL_ADDRESS>wrote:

@marcelm commented on this pull request.
In main.nf
https://github.com/SciLifeLab/CAW/pull/243#discussion_r107886108:

@@ -1142,6 +1144,13 @@ def checkFileExtension(it, extension) {
}
}

+def checkGenome() {

if (!(params.genome)) {
return false
}
return params.genome in ['GRCh37', 'GRCh38'] ? true : false

the in operator already returns a boolean. no need to check again whether
it is true or false. Use just
return params.genome in ['GRCh37', 'GRCh38']
And I would also not check this at all. And I would actually just check params.genome
in params.genomes. No need to artifically restrict this. We are going to
have a custom version of GRCh38, for example, and then it would be nice to
just name it GRCh38b or something.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SciLifeLab/CAW/pull/243#pullrequestreview-28885223,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA-O7Cbu7DaeRJr8O0K-YFjtHFfrrbY1ks5ro64ugaJpZM4MoIft
.

--
Maxime U Garcia | PhD, Bioinformatiker
BarnTumörBanken | Department of Oncology Pathology | Karolinska Institutet
Genomics Applications Development Facility | Science for Life Laboratory
<%2B46-723734336>
+46-723734336
<EMAIL_ADDRESS>| scilifelab.se
<EMAIL_ADDRESS>| ki.se

Yes, the wild card, just get all the files that match it
On Fri, Mar 24, 2017 at 1:09 PM, Marcel Martin<EMAIL_ADDRESS>wrote:

@marcelm commented on this pull request.
In configuration/genomes.config
https://github.com/SciLifeLab/CAW/pull/243#discussion_r107888649:

@@ -17,7 +17,7 @@ params {
dbsnp       = "$bundleDir/dbsnp_138.b37.vcf"
dbsnpIndex  = "${dbsnp}.idx"
genome      = "$bundleDir/human_g1k_v37_decoy.fasta"

 bwaIndex    = "${genome}.{amb,ann,bwt,pac,sa}"

 bwaIndex    = "${genome}.{alt,amb,ann,bwt,pac,sa}"

Does this work for GRCh37? That is, when the alt file does not exist?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/SciLifeLab/CAW/pull/243#pullrequestreview-28887967,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA-O7GfLJeUsFBHXGPMlP5L5XmC4xBV2ks5ro7KBgaJpZM4MoIft
.

--
Maxime U Garcia | PhD, Bioinformatiker
BarnTumörBanken | Department of Oncology Pathology | Karolinska Institutet
Genomics Applications Development Facility | Science for Life Laboratory
<%2B46-723734336>
+46-723734336
<EMAIL_ADDRESS>| scilifelab.se
<EMAIL_ADDRESS>| ki.se

Ok, thanks, then it’s different to how it works in the shell.

Can't find it now, but I'm sure I read it somewhere in the docs.
(and I did run the test before making the PR)


predict not True

I use the demo file to predict, the config file is like below.
dataset_builder: &ds_builder
table_path: 'example/table.txt'
img_shape: [32, null, 3]
max_img_width: 400
ignore_case: true
train:
dataset_builder:
<<: *ds_builder
train_ann_paths:
- '/content/gdrive/MyDrive/CRNN/CRNN.tf2/example/annotation_train.txt'
- '/content/gdrive/MyDrive/CRNN/CRNN.tf2/example/annotation_val.txt'
val_ann_paths:
- '/content/gdrive/MyDrive/CRNN/CRNN.tf2/example/annotation_test.txt'
batch_size_per_replica: 32
# Number of epochs to train.
epochs: 2000
lr_schedule:
initial_learning_rate: 0.0001
decay_steps: 600000
alpha: 0.01
# TensorBoard Arguments
# https://www.tensorflow.org/api_docs/python/tf/keras/callbacks/TensorBoard#arguments_1
tensorboard:
histogram_freq: 1
profile_batch: 0
eval:
dataset_builder:
<<: *ds_builder
ann_paths:
- '/content/gdrive/MyDrive/CRNN/CRNN.tf2/example/annotation_eval.txt'
batch_size: 1
probability: [0.9999411] , but result show is . It only displays numbers, not characters.
The result it returns is :
test_images/IMG_20211212_091803.jpg, y_pred: [b''], probability: [0.9999411] .
While I'm looking forward to is :"HSD".
Hope you answer.

Can you attach the first or second image?


General: Lag when selecting codes

Whenever I code for too long the toolkit starts to lag. The lag happens when I try to select new codes for a document, instead of being instantaneous the little loading circle appears instead of the mouse and it takes from 1-10 seconds for the click to happen. Here is how it happens, and I use and average of 3 codes per document:

Coding the first document. I open them on the data collection module by double clicking. When the window opens I select the codes I want and then close the window by pressing 'x'
Around 10-12 documents later the lag begins
Around 14-15 documents there is a 2-3 second lag before each action
Around document 20 the lag is 3-4 seconds long
30th document, 5 second lag
35th document, 6 second lag
It increases in increments until the 50th document, when the lag is around 10-11 seconds.
Around then I save my progress and close the toolkit
Wait for around 1 minute and then open it again
Load the last save and the lag is gone

I tried to just open the windows without coding anything and the same problem happened again. This time what I did was:

Open toolkit and load a file.
Open 50 consecutive windows on the data collection module, without selecting a single code.
When I tried to code the last opened window the lag was there and it was taking me around 10-11 seconds to do anything.

Issue recreated and debug + memory monitoring identified issue. Initial fix created and undergoing testing to verify issue has been resolved.

fix appears to be working on windows


Fix CMSIS-DAP connect under reset

The CMSIS-DAP firmware does not configure the nRESET pin for output
until it receives the DAP_Connect command. In the connect under reset
flow we try to toggle the pins via swj_pins before calling connect.
This fix addresses that by connecting if needed during swj_pins calls.

(from comment on #1159)
Which firmware in particular? The CMSIS-DAP specification doesn't say you have to connect before using swj_pins, and indeed hs-probe will always honour an swj_pins command for the nRESET pin (but not for the swd/jtag pins until connected), so it could be seen as a bug in one cmsis-dap firmware...
That said it's probably still best to call connect first to configure the probe pins and I can't see any reason not to do this before setting reset.

(To clarify: cmsis-dap is a particular USB protocol for debug probes, but there's many different firmwares/probes that implement it, which can have different behaviours around this sort of thing).

Which firmware in particular?

DAPLink - the ARM produced reference implementation that many probes run.  See here:
https://github.com/ARMmbed/DAPLink/blob/c782a5ba907377658bc28aa8d132a0fa44543687/source/daplink/cmsis-dap/DAP.c#L242
PORT_SWD_SETUP is what does the muxing.
I agree that the docs do not specify this as a requirement, but it seems to be for anything running this particular firmware, like the reported chip.  If we found other CMSIS-DAP probes where this was a problem we'd probably need some kind of quirks system for specific probes.  Hopefully we can avoid that given how simple the connect call is.

if anything I'm surprised we haven't noticed this issue already.

I suspect this is because the CMSIS DAP commands always succeed, so there's no error to observe.  If your project didn't really require the reset it would happily continue to work, even though probe-rs wasn't doing what you asked.  You'd only notice this if the reset was mandatory for your scenario.

Hmm maybe this could explain why some folks have still issues with ATSAMs and some don't ...

Thanks folks for fixing this!
bors r+


feat restrict partition num

Which issue does this PR close?
Closes #
Rationale for this change
Now partition num in table partition is not restricted,  you can set the partition num to an unaccpetable huge number.

What changes are included in this PR?
In this pr we make a restriction on it, it cannot be greater than MAX_PARTITION_NUM(1024 now).

Are there any user-facing changes?
User can not set a partition num greater than MAX_PARTITION_NUM while creating partitioned table.

How does this change test
Test by ut.

Codecov Report

Merging #598 (edf060f) into main (6d57582) will increase coverage by 3.36%.
The diff coverage is 92.30%.

@@            Coverage Diff             @@
##             main     #598      +/-   ##
==========================================
+ Coverage   62.61%   65.98%   +3.36%
==========================================
  Files         326      285      -41
  Lines       50232    44342    -5890
==========================================
- Hits        31454    29257    -2197
+ Misses      18778    15085    -3693

Impacted Files
Coverage Δ

sql/src/partition.rs
0.00% <ø> (ø)

sql/src/parser.rs
89.49% <92.30%> (+0.26%)
:arrow_up:

server/src/grpc/storage_service/write.rs
38.99% <0.00%> (-3.67%)
:arrow_down:

analytic_engine/src/sst/file.rs
76.17% <0.00%> (-1.09%)
:arrow_down:

analytic_engine/src/instance/write_worker.rs
72.60% <0.00%> (-0.58%)
:arrow_down:

analytic_engine/src/row_iter/merge.rs
88.58% <0.00%> (-0.19%)
:arrow_down:

server/src/http.rs
0.00% <0.00%> (ø)

server/src/server.rs
0.00% <0.00%> (ø)

server/src/handlers/sql.rs
0.00% <0.00%> (ø)

server/src/mysql/worker.rs
0.00% <0.00%> (ø)

... and 45 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Outbound traffic for SMTP port 587 stopped working after istio upgrade 1.7.8 to 1.8.6

Hi,
We have email service running which is configured with 'smtp.office365.com' SMTP server was running fine, we were able to send emails. We did istio upgrade from 1.7.8 to 1.8.6 that instance we are unable to connect SMTP server - 'smtp.office365.com'.
I am not able to find any logs in sidecar proxy. also confirms here there is no any change on the network.
For time being I have excluded port 587 by adding annotation 'traffic.sidecar.istio.io/excludeOutboundPorts' on the deployment. it started working.
Please help on the issue. Thanks in advance !!

dupe https://github.com/istio/istio/issues/34444


Non default versions

Fixes the incorrect logic in the MajorMinorIncrementalFilter that caused numerous issues.
Fixes #304 for which tests are included and this was targetted.
I also beleive (and there is some minimal test for it) #296 #295 #282
ITs pass on maven 3.5.2  as the ordering and parsing relies on some fixes in Maven 3.x they may or may not pass on maven 2.x.
@stephenc

test failure looks like infrastructure issues.
ERROR] Failed to execute goal org.codehaus.mojo:animal-sniffer-maven-plugin:1.14:check (check-java17) on project versions-maven-plugin: Failed to check signatures: Unexpected end of ZLIB input stream -> [Help 1]

@khmarbaise except any concern I will merge it.
let use know.
Thanks

If it fixes issues which is the case get it and merge it....


How can I specify the order of submodels?

I want to specify the order of submodels under Adminshell.
However, if you add a submodel using Modelreference, this order gets messed up and the result changes every time.
What function should I use to specify the order?
I am using the following code
    aas.submodel.add(model.ModelReference.from_referable(
        create_submodel(submodel_name, submodel_data, extract_columns_containing_word(data, submodel_name))))

I modified AssetAdministrationShell.Submodel set to List and solbed my issue

@SpicyYeol FYI, neither submodel nor submodel elements within a submodel don't have any order. It is wrong to rely on any specific order.

Hi @SpicyYeol, thanks for asking your question here.
@mhrimaz  is correct, an AssetAdministrationShell only has a "soft" reference to its Submodels, as by the specification. That means, an AssetAdministrationShell does not contain Submodels, which is a common misconception.
These references to the Submodel don't have any order, however each Submodel must have a globally unique Identifier (Submodel.id).
The suggested way to identify the correct Submodel you want to deal with is using that Identifier.


add l2 regularization and dropout

Hi,
Is it posible to add l2 regularization and dropout in the text_classification_cnn.py ?
Thanks

@rubenstern You can use contrib.layers.regularizers found here

@rubenstern, does this address your need. If so please close the bug.

Automatically closing due to lack of recent activity. Please reopen when further information becomes available.


Minor spec improvements

What was the end-user problem that led to this PR?
The problem was that some specs had harcoded paths from @indirect's file system. :sweat_smile:. These lockfiles are clearly incorrect but specs are not failing.
What was your diagnosis of the problem?
My diagnosis is that these lockfiles are unnecessary for these tests and don't affect what these tests are testing.
What is your fix for the problem, implemented in this PR?
My fix is to remove the lockfiles.
As a consequence of the change, the "comment numbering" got messed up. So I removed the numbering while keeping the comments to avoid that problem from happening again in the future.

@bundlerbot r+

👍


Does it work with 1.16? (OCT 17 release)

When I try to run ER with the patcher I only get a command prompt window popup and close down and ER goes from "launching" in steam and back to "Play".  And obviously the game refuses to launch.
Maybe it just doesn't work for the current version?
ps. tried using:
python er-patcher --rate 144 -uvca -- env WINE_FULLSCREEN_FSR=1 MANGOHUD=1 MANGOHUD_CONFIG=histogram %command%
and:
python er-patcher  --rate 120 --ultrawide --disable-vignette --increase-animation-distance --skip-intro -- %command%

1.16 is supported and it should work. Try copy pasting a cmd from readme, check your python version and check that you have the latest version of er-patcher.

Ok weird. I'll try a completely fresh install and copying from the readme. Maybe it's the python version? Does it need to be exactly 3.8 or 3.8 and above?

Getting the same issue with fresh install. I'm on win11 and I'm using python er-patcher --all --rate 30 --disable-rune-loss -- %command% for launch parameters through steam.

With a bit of tinkering and help from a friend I have now found the issue. I hadn't installed python with "add to path" option installed. It now seems to work properly :D!


error：CUB segmented reduce errorinvalid device function

thanks for your great job！ ！ ！
but i have some question ：
when i train the modelnet40_cls , some error happen:
2020-11-15 20:56:49.388664
Traceback (most recent call last):
File "/home/gahho/anaconda3/envs/sph3dgcn/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1334, in _do_call
return fn(*args)
File "/home/gahho/anaconda3/envs/sph3dgcn/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1319, in _run_fn
options, feed_dict, fetch_list, target_list, run_metadata)
File "/home/gahho/anaconda3/envs/sph3dgcn/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1407, in _call_tf_sessionrun
run_metadata)
tensorflow.python.framework.errors_impl.InternalError: CUB segmented reduce errorinvalid device function
[[{{node Max}} = Max[T=DT_FLOAT, Tidx=DT_INT32, keep_dims=true, _device="/job:localhost/replica:0/task:0/device:GPU:0"](Sum, ArgMax/dimension)]]
[[{{node GroupCrossDeviceControlEdges_0/Adam/value/_82}} = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device_incarnation=1, tensor_name="edge_7776_GroupCrossDeviceControlEdges_0/Adam/value", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:CPU:0"]]
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/home/gahho/SPH3D-GCN/modelnet40_cls/train_modelnet.py", line 376, in
train()
File "/home/gahho/SPH3D-GCN/modelnet40_cls/train_modelnet.py", line 247, in train
train_one_epoch(sess, ops, next_train_element, train_writer)
File "/home/gahho/SPH3D-GCN/modelnet40_cls/train_modelnet.py", line 292, in train_one_epoch
ops['train_op'], ops['loss'], ops['pred']], feed_dict=feed_dict)
File "/home/gahho/anaconda3/envs/sph3dgcn/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 929, in run
run_metadata_ptr)
File "/home/gahho/anaconda3/envs/sph3dgcn/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1152, in _run
feed_dict_tensor, options, run_metadata)
File "/home/gahho/anaconda3/envs/sph3dgcn/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1328, in _do_run
run_metadata)
File "/home/gahho/anaconda3/envs/sph3dgcn/lib/python3.5/site-packages/tensorflow/python/client/session.py", line 1348, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.InternalError: CUB segmented reduce errorinvalid device function
[[node Max (defined at /home/gahho/SPH3D-GCN/models/SPH3D_modelnet.py:13)  = Max[T=DT_FLOAT, Tidx=DT_INT32, keep_dims=true, _device="/job:localhost/replica:0/task:0/device:GPU:0"](Sum, ArgMax/dimension)]]
[[{{node GroupCrossDeviceControlEdges_0/Adam/value/_82}} = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device_incarnation=1, tensor_name="edge_7776_GroupCrossDeviceControlEdges_0/Adam/value", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:CPU:0"]]
Caused by op 'Max', defined at:
File "/home/gahho/SPH3D-GCN/modelnet40_cls/train_modelnet.py", line 376, in
train()
File "/home/gahho/SPH3D-GCN/modelnet40_cls/train_modelnet.py", line 161, in train
pred, end_points = MODEL.get_model(xyz_pl, training_pl, config=net_config)
File "/home/gahho/SPH3D-GCN/models/SPH3D_modelnet.py", line 42, in get_model
points = normalize_xyz(points)
File "/home/gahho/SPH3D-GCN/models/SPH3D_modelnet.py", line 13, in normalize_xyz
scale = tf.reduce_max(tf.reduce_sum(tf.square(points),axis=-1,keepdims=True),axis=1,keepdims=True)
File "/home/gahho/anaconda3/envs/sph3dgcn/lib/python3.5/site-packages/tensorflow/python/util/deprecation.py", line 488, in new_func
return func(*args, **kwargs)
File "/home/gahho/anaconda3/envs/sph3dgcn/lib/python3.5/site-packages/tensorflow/python/ops/math_ops.py", line 1643, in reduce_max
name=name))
File "/home/gahho/anaconda3/envs/sph3dgcn/lib/python3.5/site-packages/tensorflow/python/ops/gen_math_ops.py", line 4641, in _max
name=name)
File "/home/gahho/anaconda3/envs/sph3dgcn/lib/python3.5/site-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
op_def=op_def)
File "/home/gahho/anaconda3/envs/sph3dgcn/lib/python3.5/site-packages/tensorflow/python/util/deprecation.py", line 488, in new_func
return func(*args, **kwargs)
File "/home/gahho/anaconda3/envs/sph3dgcn/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 3274, in create_op
op_def=op_def)
File "/home/gahho/anaconda3/envs/sph3dgcn/lib/python3.5/site-packages/tensorflow/python/framework/ops.py", line 1770, in init
self._traceback = tf_stack.extract_stack()
InternalError (see above for traceback): CUB segmented reduce errorinvalid device function
[[node Max (defined at /home/gahho/SPH3D-GCN/models/SPH3D_modelnet.py:13)  = Max[T=DT_FLOAT, Tidx=DT_INT32, keep_dims=true, _device="/job:localhost/replica:0/task:0/device:GPU:0"](Sum, ArgMax/dimension)]]
[[{{node GroupCrossDeviceControlEdges_0/Adam/value/_82}} = _Recvclient_terminated=false, recv_device="/job:localhost/replica:0/task:0/device:CPU:0", send_device="/job:localhost/replica:0/task:0/device:GPU:0", send_device_incarnation=1, tensor_name="edge_7776_GroupCrossDeviceControlEdges_0/Adam/value", tensor_type=DT_FLOAT, _device="/job:localhost/replica:0/task:0/device:CPU:0"]]
when i set the batch_size=1,it doesn't cause the error,but the result is very bad
i don't know how to fix it
thanks for your reply

The problem might be caused by the tensorflow version you are using. We test the code in Tensorflow 1.12.


System Restore Point in tweak section taking way too much

Bug description
The script took more than 45 minutes to perform a system restore point (as I'm writing it is stuck on 99% for 20 minutes)
Looks like the progress bar went slower and slower the more it progressed.
Steps to reproduce the behavior:

Go to Tweaks
Click on Create Restore Point (was enabled by default)
Click Run tweaks

Expected behavior
Expected it would take same time as a system restore point created through windows UI
Additional context
Got a warning saying Microsoft.PowerShell.Management not imported because it was already imported (Snap-in?)
The tweak I selected apart from system restore was the one for disabling telemetry
OS: Windows 10, version 1909

The same behaviour on Windows 11 23H2 (build 22631.2861). The System Restore is stuck at 99% for about 30 mins now as I'm posting this.

It takes way too long

Got a warning saying Microsoft.PowerShell.Management not imported because it was already imported.

That's because of this:
        # Attempt to load the required module for Get-ComputerRestorePoint
        try {
            Import-Module Microsoft.PowerShell.Management -ErrorAction Stop
        } catch {
            Write-Host "Failed to load the Microsoft.PowerShell.Management module: $_"
            return
        }

The script took more than 45 minutes to perform a system restore point

It looks like you have an HDD or a stupidly bloated Installation, I don't know if this is slower than Windows UI.
        Checkpoint-Computer -Description $description -RestorePointType "MODIFY_SETTINGS"

Three days ago, I installed Microwin, and after the installation, I installed a few applications using Winutil and applied some tweaks for laptop. Creating a restore point also took around 40 minutes through Winutil. Spec: Dell XPS, SSD M2, 32GB RAM

I am having the same issue... It speeds through the creation of the system restore point until around 90% (about 2-3 minutes total) and then each % point after that seems to take exponentially longer to finish.
98->99% took ~10-12 minutes... and it has now been stuck at 99% for over 45 minutes at this point.

I am seeing the same issue.  the creating restore point is stuck on 98% for the last 30 min.  I will run it without creating a restore point. Hopefully this is easy catch on the bug side.


[WIP][WB-4514] implement message polling

https://wandb.atlassian.net/browse/WB-4514
Description
What does the PR do?

[x] Add protocheck circleci test
[ ] Attempt to shutdown internal process on error paths
[ ] Make windows error handling work
[x] Add injection framework for testing error paths
[x] Add new message to test process startup only
[ ] Convert wandb.init() to use poll

Merge these PRs first:

[x] https://github.com/wandb/client/pull/1814
[x] https://github.com/wandb/client/pull/1818
[x] https://github.com/wandb/client/pull/1825
[x] https://github.com/wandb/client/pull/1830
[x] https://github.com/wandb/client/pull/1833

Testing
How was this PR tested?

Closing in favor of:
https://github.com/wandb/client/pull/1871


Add new URL parameter to API

This PR aims to reduce confusion by adding a new URL parameter named address.
The idea here is to allow the exporter to clearly understand the intent of the client when it comes to authorization of the FAClient. Currently, the API is somewhat confusing as seen from the conversation in issue #143, with confusion coming from how the exporter uses tokens from tokens.yaml vs tokens from the auth header of the incoming request .
By adding a new URL parameter, the intent of the client is clear:

If the endpoint parameter is provided, the exporter should use the configured endpoint from tokens.yaml to authenticate.
If the address parameter is provided, the exporter will expect the authentication to come from the auth header in the request.

Hi @jddarby , sorry for the delay!
So my concerns with the PR is, is that I mildly agree that the endpoint param can be a little confusing, but we have to ensure backwards compatibility, and we're not currently in a position to release a new major version as that param may changing in the very near future.
With that being said, this approach as currently written isn't going to allow for that goal.
Your comment from #143 makes sense to me "My alternative idea would be that we try to authenticate with the header token and fall back to the token in tokens.yaml if that fails but I prefer the option above."
This is probably how it should of been implemented in the first place.
(if i have a token that I received just in time, use it, otherwise use what I know)
Also - I do highly recommended, if you're able to, to update README.md to document this behavior, as also suggested in #143
Please send an email to<EMAIL_ADDRESS>and I would love to meet via Zoom and give you just a little bit more context.


Windows 11 Insider Preview 10.0.23521.1000 not Supported

termsrv.zip

termsrv.zip

Your termsrv.dll version is 10.0.23506.1000. Please, check your file or confirm that you have submitted the file of the same version as indicated in the title of the issue.

termsrv.zip
It's  10.0.23521.1000

termsrv.zip

Your termsrv.dll version is 10.0.23506.1000. Please, check your file or confirm that you have submitted the file of the same version as indicated in the title of the issue.

termsrv.zip
It's 10.0.23521.1000

https://github.com/sebaxakerhtc/rdpwrap.ini/commit/cadcfaab2f84713cc773ba9a22152543432ce577


cross-validated rmsd k-medoids without hogging memory?

I was hoping to use MiniBatchKMedoids with the RMSD metric in Osprey.  Am I correct in believing that the 5fold cross-validation requires loading all of the data into memory up front?  For example, the following (non-cross validated) script seems to use lazy loading and low memory:
from msmbuilder import example_datasets, cluster, msm, featurizer, lumping, utils, dataset, decomposition
from sklearn.pipeline import make_pipeline

n_clusters = 2

trajectories = dataset.MDTrajDataset("./trajectories/*.h5")

clusterer = cluster.MiniBatchKMedoids(n_clusters=n_clusters, metric="rmsd")
clusterer.fit(trajectories)

I don't think it should be necessary to load everything into memory simultaneously if you're using the msmbuilder dataset loader in osprey. It could be accidental. Can you figure out where it's happening?

Here's the config file:
estimator:
    pickle: my-model.pkl
    entry_point: msmbuilder.decomposition.tICA
    eval: |
        Pipeline([
                ("clusterer", MiniBatchKMedoids(batch_size=5, metric="rmsd")),
                ("msm", MarkovStateModel(n_timescales=8, lag_time=400))
        ])
    eval_scope: msmbuilder

strategy:
    name: hyperopt_tpe  # or moe, hyperopt_tpe

search_space:
  clusterer__n_clusters:
    min: 10
    max: 50
    type: int

cv: 5  # the order of K-fold cross validation to use

dataset_loader:
  name: mdtraj
  params:
    trajectories: ../trajectories/*.h5

trials:
  uri: sqlite:///kmeans.db

you need to switch your dataset loader from mdtraj to msmbuilder. currently, you're going through this code path: https://github.com/pandegroup/osprey/blob/master/osprey/dataset_loaders.py#L58-L62

but you want to be up here: https://github.com/pandegroup/osprey/blob/master/osprey/dataset_loaders.py#L16-L29

OK, I updated the yaml file according to your suggestion (see edited code block above), but I still see huge memory usage (see image):

The drop at the end is when I control-C the osprey run and clear up my memory.


Implementing Background Process Manager

We created a package called subprocess, under the shared directory for process management to address the issue: #5826
Some new features

You can start a process and configure it in Go using 'NewProcess(..)'
You can manage this process using functions like Pid(), Stop(), Start(), Restart(), Reload(), Save(path string), Signal(int64), Wait() (int64, error)
You can now serialize and deserialize configurations for a process by calling 'Save()' and 'ImportProcess(..)'

This pull request didn't trigger Jenkins as its author isn't in the whitelist.
An organization member must perform one of the following:

To have this branch tested by Jenkins, use the "ok to test" command.
To have a one time test done, use the "test this please" command.

Those commands are simple Github comments of the format: "jenkins: COMMAND"

go fmt ./... has a few fixes for bgpm_test.go.
You also need to add documentation comments so golint is happy:
stgraber@castiana:~/data/code/lxc/lxd (bgpm)$ golint shared/subprocess/
shared/subprocess/manager.go:9:1: exported function NewProcess should have comment or be unexported
shared/subprocess/manager.go:20:1: exported function ImportProcess should have comment or be unexported
shared/subprocess/proc.go:13:6: exported type Process should have comment or be unexported
shared/subprocess/proc.go:22:1: exported method Process.Pid should have comment or be unexported
shared/subprocess/proc.go:25:1: exported method Process.Stop should have comment or be unexported
shared/subprocess/proc.go:39:1: exported method Process.Start should have comment or be unexported
shared/subprocess/proc.go:74:1: exported method Process.Restart should have comment or be unexported
shared/subprocess/proc.go:85:1: exported method Process.Reload should have comment or be unexported
shared/subprocess/proc.go:99:1: exported method Process.Save should have comment or be unexported
shared/subprocess/proc.go:110:1: exported method Process.Signal should have comment or be unexported
shared/subprocess/proc.go:124:1: exported method Process.Wait should have comment or be unexported

You should also cleanup your commits a tiny bit, the second commit includes the rename which makes things confusing, it'd be better to move the rename into the first commit and only have the second commit be the tests.
You'll also want to add the new package to test/suites/static_analysis.sh so golint is run automatically on it.

Thanks for this!
I just did first quick pass of static analysis and code styling checks.

Reformatted the code per your comments as well as moving around the rename to the first commit

jenkins: test this please

There's a bit of work needed to get golint to be happy with this still:
shared/subprocess/manager.go:10:1: exported function NewProcess should have comment or be unexported
shared/subprocess/manager.go:22:1: exported function ImportProcess should have comment or be unexported
shared/subprocess/proc.go:14:6: exported type Process should have comment or be unexported
shared/subprocess/proc.go:23:1: exported method Process.Pid should have comment or be unexported
shared/subprocess/proc.go:33:1: exported method Process.Stop should have comment or be unexported
shared/subprocess/proc.go:50:1: exported method Process.Start should have comment or be unexported
shared/subprocess/proc.go:90:1: exported method Process.Restart should have comment or be unexported
shared/subprocess/proc.go:104:1: exported method Process.Reload should have comment or be unexported
shared/subprocess/proc.go:119:1: exported method Process.Save should have comment or be unexported
shared/subprocess/proc.go:133:1: exported method Process.Signal should have comment or be unexported
shared/subprocess/proc.go:149:1: exported method Process.Wait should have comment or be unexported

Hi Stephane, I cleared up those golint comments

So other than those two nits, it'd be good if you could:

Combine the last commit with the first
Replace the python based tests with POSIX shell tests (as we can't guarantee that python will be present on the systems the test run on).

Converted the tests to shell as well!

jenkins: test this please

shared/subprocess/bgpm_test.go:59:20: "occured" is a misspelling of "occurred"
shared/subprocess/bgpm_test.go:170:20: "occured" is a misspelling of "occurred"

Fixed spelling

jenkins: test this please

@ribsthakkar you now have go fmt complaining. Can you run go fmt ./... and include those changes in your commits?

Any other formatting things I should run?

jenkins: test this please

We should run go fmt (or gofmt), deadcode, go vet and golint for the common ones.
We have a couple more special ones that run to detect weirder cases, you can find the whole set in test/suites/static_analysis.sh.

Failures are unrelated, so this should be good to go, will do a last review and merge.


Amazing!

First of all, fantastic work. This thing is by far the best I've used. Not sure how you were able to push specific tasks to the background and still communicate with it, but I love it!
I did want to ask about the models. I noticed you were using gpt-3.5-turbo-0301. I'm not familiar with it. Can you explain why that one and not the normal 3.5 turbo?
Also, I tried swapping to text-davinci-003 and 002, but I got a URL error. I assume it just means the wrong syntax is being used to call the API. Is there any way to set it up to utilize older models? I've found that 002 works much better with coding than the newer ones.

@thiswinex Would you mind helping me answer a question about these two models (gpt-3.5-turbo & gpt-3.5-turbo-0301)?
I chose 0301 version just because this one is newer. 🤪 Maybe it's more advanced I think.

As for choosing model, I will take a deeper look into these models and then have a discussion with you, OK?

No worries at all, and very much appreciated.

@mikes-bowden You can get more infomation about these two models from this link: https://platform.openai.com/docs/models/gpt-3-5

@mikes-bowden You can get more infomation about these two models from this link: https://platform.openai.com/docs/models/gpt-3-5

Not what I was looking for. I'd like to know why that one selected was selected. The information on that page is lackluster at best.

@thiswinex thanks for your information.
@mikes-bowden I did not analyze carefully which model to use before, but after reading the documentation above, I believe that using gpt-3.5-turbo is a more reasonable choice.

gpt-3.5-turbo-0301 is just a snapshot of gpt-3.5-turbo and it won't be updated.

Fair enough. Thanks!
Any plans to support the others?

I think it's possible to support other models. I'd like to test it in v1.3.0, and if nothing goes wrong it will be a new config key for you to select model.

AWESOME!

Hey @mikes-bowden, it's been a long time since our last conversation. I'd like to update some info about model selection:

ChatGPT models (such as GPT-3.5 Turbo, GPT-4) and other completion models (such as text-davinci-003, text-davinci-002) are two types of models in OpenAI's documentation which use different API endpoints. You can check it here: https://platform.openai.com/docs/models/model-endpoint-compatibility.
The bot now has the ability to select a model in these chat models (which use /v1/chat/completions as an API endpoint) if you have GPT-4 access. Have a try 🙂

We have released a lot of interesting new features. I hope you like them.

Awesome!
I'll get it updated today and test it out. Thanks!


Bug: FSMP v1.49 + First Person Mods + Some SMP Clothes + Wind(?) = lose fps.

Describe the bug
A bug that kills fps and introduces cloth stretching to some SMP clothes.

Seems to happen with specific outfits only (I found CoCo 2B Wedding top 01 SMP so far)
(  https://www.nexusmods.com/skyrimspecialedition/mods/26419  )
Seems to be related to showing First Person Arms instead of 3p arms with 1st person mods.
Seems to be related to SMP wind, since it's not happening with it being OFF.

Steps to reproduce the behavior:

Install Immersive First Person View / Improved Camera beta AE4 installed (Or simply have it on)
(https://github.com/ArranzCNL/ImprovedCameraSE/releases/tag/v1.0.0)
Go into first person
Raise arms/ready weapons (Hands holding a spell/weapon)
Lose fps

Expected behavior
Fps should go down, clothes should stretch once lowering hands (Sheathing weapons)
Desktop:

Windows 10

FSMP

FSMP v1.49; No Cuda; AVX.

Attachments
hdtSMP64.log
configs.log
Videos and other info
https://user-images.githubusercontent.com/108320865/201515817-bdf2dac2-f72b-4d84-8a78-785981c31981.mp4
https://user-images.githubusercontent.com/1083208

65/201515830-924a0685-1dc6-46d0-aeb1-d161acb52fe1.mp4

It looks like the armor is clipping through the - [ ] body from your video. That's likely an XML problem. It never was an issue before because there weren't really lateral forces acting on bones.


feat(pacmod_interface): update engage_sequence

Signed-off-by: tomoya.kimura<EMAIL_ADDRESS>Updated the engage sequence to match pacmod's FW

the newest firmware is stricter at implementing clear override, and disable before enable sequence. It functions as documented. From the plot you shared, it looks like you just need to break up into two messages enabling after an override.
-- Send a msg with clear override bit high and enable bit low.
-- Send a msg after that, with clear override bit low, and enable bit high.
It needs to be two distinct messages as in #1 and #2. From your plot, it looks like it is done in one message. If you do that, you'll see what happened. Enable fails, but override active is set low, then the second time you try to enable, it works.
For the second plot, to diagnose properly what happened and provide guidance, can you share the raw CAN log/data for this event ("only accel/brake switched to auto")?

TODO:

Check if the Jpntaxi can be properly engaged
Check if the Lexus can be properly engaged

@tkimura4 Is it ready to merge this PR?

No, We have not tested it yet on the Lexus/Jpntaxi

I checked that JapanTaxi vehicle can be engaged.

@TakaHoribe Could you approve this?


livereload 404s with relative css @imports

I have a CSS file at /css/main.css which imports other CSS files in subdirectories of /css, using relative @import statements. When I make edits, livereload makes a request for /lib/text.css, which should be /css/lib/text.css, resulting in a 404.

I thought maybe this was an issue with my app stack, which is complicated, so I made a new barebones repo which demonstrates the issue, using grunt-contrib-connect and grunt-contrib-watch.
The demo repo is https://github.com/pascalpp/grunt-watch-reload-issue. To replicate the issue, clone that repo locally, run npm install and grunt, and then open http://localhost:9090 and follow the instructions shown there.
If anyone has any insight on this issue, I'd really appreciate it.

I modified my demo repo to connect to the standalone LiveReload server (using the GUI Mac app) and this issue still occurs. So I guess this is an issue with livereload itself, not grunt watch. Will seek resolution elsewhere.

If anyone else is experiencing this issue, you can follow https://github.com/livereload/livereload-js/issues/37


Unhandled errors from deferred file close operations

From Ada Logics

Target
./source-controller/controllers/storage.go
./source-controller/pkg/sourceignore/sourceignore.go
./kustomize-controller/controllers/kustomization_impersonation.go
./kustomize-controller/internal/sops/pgp/keysource.go
Throughout the codebase there are places where file close operations are deferred within a function where a file is being written to, e.g:

localPath := s.LocalPath(*artifact)
f, err := os.Open(localPath)
if err != nil {
  return err
}
defer f.Close()

// untar the artifact
untarPath := filepath.Join(tmp, "unpack")
if _, err = untar.Untar(f, untarPath); err != nil {
  return err
}

This can lead to undefined behaviour since any errors returned by the f.Close() operation are ignored. This can have consequences in the event where a close operation failed and the data was not yet flushed to the file, which the rest of the code will assume it to be. For a detailed discussion on this, please see here.

Recommendation
Ensure that errors from f.Close() are handled.

I took a stab at it on kustomize-controller. Please take a look. If this looks good, I will apply the same change to source.

Closed PR for kustomize-controller file write operation is already taken care of and file read does not need this particular update.
I have also gone through ./source-controller/controllers/storage.go and ./source-controller/pkg/sourceignore/sourceignore.go and looks like the file write operations are already taken care of. I believe this issue can be closed.

@tomhuang12 thank you for reviewing. I just had another look and I think the only outstanding occurrency is:
https://github.com/fluxcd/source-controller/blob/main/controllers/storage.go#L358
Do you mind having a look at it please?

Thanks for a second look @pjbgf 😅 . Please review my PR and let me know if there's any issues. Hopefully it looks good the first try 🤞 !

@dholbach this issue can be marked as done.


Add link back to home, fixes some styles fixes #90

@zqian @pushyamig Do we need these body styles still? I think we should move them to a a global level if you want to override bootstraps styles. (Now that we have a global template)
I personally like the look better without (most of) these overrides.

The "default style" is bootstrap's reboot
http://v4-alpha.getbootstrap.com/content/reboot/


Fixing issue: #254

Fixing issue #254  “Error: Provider 'Tweenable' must return a value …from $get factory method." using angular 1.3.x + AMD loaders(requirejs, webpack)

yeah ! thanks a lot !


Feature: Support Webpack 4

So this is my first PR and there's a high chance that I'm doing something weird in terms of forks, branches, travis etc etc. If so, bear with me and let me know 🙌
Anyhoo, I got it working with webpack 4 by replacing the q-io file reading with a fs implementation. 🎉 Also I had to do 'addDepencency' on loaded scss files in order for hot reloading to work.
I'm hoping that your Travis is set up to build on PRs. Tests are passing locally but my own Travis account didn't find my fork so I'm not able to verify 100%.
Fixes #2  (At least for webpack 4)

Ok so the test for node 0.12 is failing, but I'm getting the same error on master. Looks like an incompatible transitive dependency of ava rather than a failing test

So this is my first PR and there's a high chance that I'm doing something weird in terms of forks, branches, travis etc etc.

Nah, this basically looks good to me from a mechanical perspective. Welcome!

Anyhoo, I got it working with webpack 4 by replacing the q-io file reading with a fs implementation.

Why was this necessary?

Also I had to do 'addDepencency' on loaded scss files in order for hot reloading to work.

Cool!

Ok so the test for node 0.12 is failing,

I'm fine just disabling the tests for an ancient version of Node.js. :smile: Want to update the .travis.yml to use more modern versions of Node.js? (There may be a spot in the README that's good to update too.)

Thanks for getting back quickly!
I'll look into your comments after dinner

Why was this necessary?

To tell you the truth, I have no idea why q-io was making webpack 4 fail. I got a cryptic error related to an internal webpack optimization thing. I just tried commenting things out until I got it running without errors 😛
Here is the error:
/Users/asbjornh/Documents/KODE/test/node_modules/sass.js/dist/sass.sync.js:61
var Module=typeof Module!=="undefined"?Module:{};var moduleOverrides={};var key;for(key in Module){if(Module.hasOwnProperty(key)){moduleOverrides[key]=Module[key]}}Module["arguments"]=[];Module["thisProgram"]="./this.program";Module["quit"]=(function(status,toThrow){throw toThrow});Module["preRun"]=[];Module["postRun"]=[];var ENVIRONMENT_IS_WEB=false;var ENVIRONMENT_IS_WORKER=false;var ENVIRONMENT_IS_NODE=false;var ENVIRONMENT_IS_SHELL=false;if(Module["ENVIRONMENT"]){if(Module["ENVIRONMENT"]==="WEB"){ENVIRONMENT_IS_WEB=true}else if(Module["ENVIRONMENT"]==="WORKER"){ENVIRONMENT_IS_WORKER=true}else if(Module["ENVIRONMENT"]==="NODE"){ENVIRONMENT_IS_NODE=true}else if(Module["ENVIRONMENT"]==="SHELL"){ENVIRONMENT_IS_SHELL=true}else{throw new Error("Module['ENVIRONMENT'] value is not valid. must be one of: WEB|WORKER|NODE|SHELL.")}}else{ENVIRONMENT_IS_WEB=typeof window==="object";ENVIRONMENT_IS_WORKER=typeof importScripts==="function";EN

TypeError: availableModules is not iterable
    at handler (/Users/asbjornh/Documents/KODE/test/node_modules/webpack/lib/optimize/RemoveParentModulesPlugin.js:101:23)
    at SyncBailHook.eval [as call] (eval at create (/Users/asbjornh/Documents/KODE/test/node_modules/tapable/lib/HookCodeFactory.js:17:12), <anonymous>:13:16)
    at SyncBailHook.lazyCompileHook [as _call] (/Users/asbjornh/Documents/KODE/test/node_modules/tapable/lib/Hook.js:35:21)
    at Compilation.seal (/Users/asbjornh/Documents/KODE/test/node_modules/webpack/lib/Compilation.js:897:35)
    at hooks.make.callAsync.err (/Users/asbjornh/Documents/KODE/test/node_modules/webpack/lib/Compiler.js:494:17)
    at _done (eval at create (/Users/asbjornh/Documents/KODE/test/node_modules/tapable/lib/HookCodeFactory.js:24:12), <anonymous>:9:1)
    at _err1 (eval at create (/Users/asbjornh/Documents/KODE/test/node_modules/tapable/lib/HookCodeFactory.js:24:12), <anonymous>:32:22)
    at _addModuleChain (/Users/asbjornh/Documents/KODE/test/node_modules/webpack/lib/Compilation.js:771:12)
    at processModuleDependencies.err (/Users/asbjornh/Documents/KODE/test/node_modules/webpack/lib/Compilation.js:710:9)
    at _combinedTickCallback (internal/process/next_tick.js:131:7)
    at process._tickCallback (internal/process/next_tick.js:180:9)
error Command failed with exit code 7.

It doesn't even look related to q-io, but like I said, replacing the file reading with fs the error magically disappeared 🙂

Pushed a new update.
I didn't notice this last time because I'd forgotten to upgrade the webpack dependency, but webpack 4 isn't compatible with node 4, so i upped the test target to 6. I also think testing in lts is a good idea so I added that one as well.
I had to remove q-io from assert-css-equal as it produced the same error as in index.js when running the tests with webpack 4.
I also added a lockfile, which will hopefully ensure that tests will pass in the distant future.

(There may be a spot in the README that's good to update too.)

The readme only references .travis.yml so I think it looks good the way it is 🙂

Also, I tried upgrading to the latest version of sass.js, which made the Import from node_modules test fail because the output is different (which I guess is to be expected).
I think it would be nice to upgrade to the latest version, but I'm not sure what to do about node-modules-import.css.txt which has to be updated to the new version somehow

To tell you the truth, I have no idea why q-io was making webpack 4 fail.

Wow. I pulled down the code locally and also poked around, and I have no idea either. I was amazed that the simple presence of require('q-io/fs') causes the problem, even if q-io is not actually used.
Good idea to update the sass.js version. Updating the expectations test file was easy enough; I just added a fs.writeFileSync('out.css', actualGeneratedContent) and then copied that in to place. I did a spot-check, and the new content seems reasonable, but who knows for sure? 🙃 (This is the problem with the type of test I put in where it's a massive, black-box assertion. :joy:)
This PR will be good to go once you remove the extraneous formatting changes.

Also, hopefully you aren't bothered by me pushing a few commits directly to your branch. It seemed like the fastest way to get this PR merged, which seems like something you'd be in favor of. :smile:

Great! Thanks :)
I have no objections to you committing to my fork, but do I need to review my access settings? 😛
Or do you specifically have write privileges to anything that's forked from one of your repos? Like I said, new to this whole workflow of between-repo-PRs.
I'll take a look at those formatting changes. I'll be sure to disable Prettier next time I'm contributing to something

Alright so I've undone most of the formatting done by Prettier. I left the changes to indentation here because it seems more correct to me.
Happy? 🙂

Or do you specifically have write privileges to anything that's forked from one of your repos?

Yes, I'm almost entirely certain that's correct. There may be a setting about this when you're submitting the PR.

Merged and published as 2.0.0. Thanks for your contribution!

Yay! Thanks 😃🍻


Pre-clean docstrings when running EmbedSignature transform

PEP257 documents rules for stripping inline docstrings such that they may be formatted at the REPL using help.
Broadly, this is "strip whitespace from the first line, and dedent all subsequent lines by the minimum common whitespace prefix".
This allows writing a docstring as:
def function(*args) -> result:
    """This is the first line.

    Here is a long description.
    """

and having it rendered as:
This is the first line.

Here is a long description.

Similarly to if the docstring were:
def function(*args) -> result:
    """
    This is the first line.

    Here is a long description.
    """

When compiling Cython files with embedsignature=True, the insertion of the function signature breaks the normal cleaning rules, since the raw docstring becomes:
def function(*args) -> result:
    """function(*args) -> result
This is the first line.

    Here is a long description.
    """

Which is then rendered in the REPL as:
function(*args) -> result
This is the first line.

    Here is a long description.

Note the over-indented long description.
To fix this, change signature embedding to pre-clean the input docstring (via inspect.cleandoc) before prepending the signature. The above example is then embedded as:
def function(*args) -> result:
    """function(*args) -> result
This is the first line.

Here is a long description.
"""

which is then rendered correctly.

Thanks

Amazing! Thanks for pursuing this @wence-.

Yes, thanks Lawrence!


Pi Shutdown?

Is there any way to use this program to shutdown the Pi similar to this method: https://howchoo.com/g/ymy4nza0nzb/reboot-your-raspberry-pi-using-an-amazon-dash-button ?

No, Dasher just shoots off HTTP requests. There's no other support for other types of commands at the moment.


pkg/dds/test/dap/integration/dart_test_test.dart is micro-task scheduling dependent

With
diff --git a/pkg/dartdev/lib/src/commands/run.dart b/pkg/dartdev/lib/src/commands/run.dart
index c0f53b862fe..e79248279be 100644
--- a/pkg/dartdev/lib/src/commands/run.dart
+++ b/pkg/dartdev/lib/src/commands/run.dart
@@ -300,6 +300,8 @@ class RunCommand extends DartdevCommand {
       }
     }

+    await Future.delayed(Duration(milliseconds: 100));
+
     String? nativeAssets;
     if (nativeAssetsExperimentEnabled) {
       try {

multiple tests in pkg/dds/test/dap/integration/dart_test_test.dart start failing.
For example:
00:02 +0 -1: dart test resolves modified breakpoints [E]

  Did not receive stopped event before stream closed

  pkg/dds/test/dap/integration/test_client.dart 216:15  DapTestClient.event.<fn>

I encountered this when adding some extra checks in https://dart-review.googlesource.com/c/sdk/+/314721 that use await.
cc @DanTup (edit: I'll go ahead and land the mentioned CL approving this failure on this issue.)

I'm on Windows now and seeing a number of these fail without either of those changes - some of them have additional (but perhaps concerning) output like "Could not find file test:test" (from running dart run test:test).
<== [DAP] {"seq":11,"type":"event","body":{},"event":"terminated"}

STDERR output collected before app terminated:
Could not find file `test:test`
  Did not receive stopped event before stream closed

  C:\Dev\Google\dart-sdk\sdk\pkg\dds\test\dap\integration\test_client.dart 216:15  DapTestClient.event.<fn>
  dart:async/stream_pipe.dart 11:23                                                _runUserCode

I don't know if they're the same cause, but I'll try with your changes once these are passing.

My changes make it worse. 🙃  I boiled my changes down to that single await to make it fail more often.

@dcharkes can you reproduce it locally?
I fixed my issues above and I can't repro yours (even with a longer wait), but I'm not sure if it's fixed with my fixes (I would expect them to fix some waiting for terminated events, but not stopped unless it was the same issue I had above ("Could not find test:test" which should only have occurred if you run some specific tests in isolation)).
I've uploaded those fixes here:
https://dart-review.googlesource.com/c/sdk/+/315823
If you're able to run the bots and/or pull it locally, it'd be good to know if it may have also fixed your issue. If not, can you confirm what platforms you've seen the failure on? Thanks!

It looks like this fixes the issues! ❤
$ git fetch https://dart.googlesource.com/sdk refs/changes/23/315823/1 && git checkout -b change-315823 FETCH_HEAD
remote: Total 12 (delta 11), reused 12 (delta 11)
Unpacking objects: 100% (12/12), 1.99 KiB | 291.00 KiB/s, done.
From sso://dart/sdk
 * branch                    refs/changes/23/315823/1 -> FETCH_HEAD
Switched to a new branch 'change-315823'
dacoharkes@dacoharkes-linux:~/dart-sdk/sdk$ tools/build.py -mrelease -ax64 runtime ffi_test_functions ffi_test_dynamic_library create_platform_sdk && tools/test.py -n unittest-asserts-release-linux pkg/dds/test/dap/integration/dart_test_test
Done. Made 395 targets from 115 files in 328ms
# login as<EMAIL_ADDRESS>override GOMA_PROVIDE_INFO=true
override GOMA_SEND_USER_INFO=true
override GOMA_COMPILER_PROXY_ENABLE_CRASH_DUMP=true
override GOMACTL_USE_PROXY=true
override GOMA_ARBITRARY_TOOLCHAIN_SUPPORT=true
Enable http_proxy
override GOMA_SERVER_HOST=<IP_ADDRESS>
override GOMA_SERVER_PORT=19080
override GOMA_USE_SSL=false
GOMA version 3435fce1653aa1d611c2834901561be7e6ccfab0@1686192619

server: goma.chromium.org:443 (via http_proxy)
goma is already running.

buildtools/ninja/ninja -C out/ReleaseX64 -j1000 -l64 runtime ffi_test_functions ffi_test_dynamic_library create_platform_sdk
ninja: Entering directory `out/ReleaseX64'
[1778/1778] STAMP obj/create_platform_sdk.stamp
buildtools/ninja/ninja -C out/ReleaseX64 -j1000 -l64 runtime ffi_test_functions ffi_test_dynamic_library create_platform_sdk done.
The build took 114.747 seconds
No build targets found.
Test configuration:
    unittest-asserts-release-linux(architecture: x64, compiler: dartk, mode: release, runtime: vm, system: linux, timeout: 240, enable-asserts, use-sdk)
Suites tested: pkg
[02:12 | 100% | +    1 | -    0]

And rebasing your test changes on top of my changes:
[02:09 | 100% | +    1 | -    0]

Great, thanks for confirming! :-)


Stream IO

is there interest in adding another quiche_conn_stream_send that populates an out buffer, so that the application can actually drive this IO?

was just a point of confusion. stream send fills a bucket with packets that you must then send with connection send.


CSA Miniproject Final Review

Team

Crossover Score
Runtime
Plan
Github
Analytics

N/A
Runtime Link
Plan
Frontend Backend
Frontend Backend

Individuals

Name
Individual Review Ticket

Derrick
Review Ticket

Kaiden
Review Ticket

Jishnu
Review Ticket

Video Links

Member
Video Link

Derrick
Link

Kaiden
Link

Jishnu
Link

Fibonacci and Sorting Blog
Fibonacci Blog
Sorting Blog
Growth/Accomplishments
Initial Ticket --> Mid Review Ticket --> Ticket after Review
As a team, we have definitely grown and we have reflected based on the review. Fixes such as implementing inheritance into Fibonacci was something that was required hugely by the requirements.
We also made major reflections and developed the code to fit accordingly, for example, another thing that we were largely criticized for was the fibonacci sequence not displaying the steps taken to the solution and we adapted the code to fix that issue as well as the issue that our API set up was weird, we fixed that issue as well based on the feedback provided
Short falls/Improvement
As a team, I felt like there was a lack of collaboration. Although at times we were successfully moving our way through the project, the lack of collaboration dragged us down at the end.
Another issue that was something that was a short fall was the lack of a designated leader/person to make sure that everything was being worked on as well as manging and controlling the parts that everyone works on, dragging us down.
Another short fall is the lack of emphasis on the frontend side of things, we had the backend completed and done but we didn't put enough emphasis into creating a body or working with the frontend to display what we created on the backend.
Individual Contributions

Members
Key Commits
Contribution Notes

Derrick
Backend Frontend Key commits are linked in Contribution Notes
Overall, I had some pretty strong contributions to the project. I worked on majority of the backend, with some light touches on the front end. I'm happier that I did get to fully experience the backend as I did not touch it a lot last trimester. I mainly worked on fibonacci by completing the for and while method, cleaning up the stream method, and implementing inheritance into it. I also worked on sorting too by adding bubble and insertion work. I tried to get the API working on the front end, which it does run locally, but the sorting does not merge with the API. I tried to deploy as well, but that did not go as successfully as I thought it would.

Kaiden
Sort Recursion
For this project, I mostly worked on the backend portion. I had previously done the frontend for my last project and this mini project allowed me to learn much more about the backend because I did backend code mostly. I did the recursion method for the fibonacci algorithm. Also for the sorting part of the project, I did the merge and selection sorting algorithms. I also set up the framework for the sorting as well. I also tried to do the API but there was problem due to deploying so I was unsuccessful. I could have contributed more by being more available after school and better at communication so we could work better as a team.

Jishnu
Backend (In the comment) Frontend 1 2
Contribution wise, I feel that I played a pretty big role in the backend, even though I still had some commits that show my work on the frontend as well and my development of the frontend to show the fibonacci api at work. Last trimester, I had focused primarily on working the frontend and so I tried to make myself work more on the backend. As a result, I focused primarily on fixing up any inconsistencies in the fibonacci procedures and creating the API that would connect with the frontend. Before our miniproject review, I would say that these were my biggest commits: 1 and  2. Apart from the commits to the fibonacci API, I also contributed to the actual fibonacci procedure by making it actually use stream properly as well as fixing the sorting methods and combining it with the dog API and try to make it work . After the review that we had with Mr.Mort and the crossover groups, I prioitized working on fixing the API in a way that followed the jokes api as we had been criticized for not following and also fixed the fionacci procedure and the API to make sure that the steps were being shown. I did work on fixing the chart and did manage to end up making it work but it doesn't work on the runtimes.

Team Review "Scrum Master Ryan" grading "Scrum Master Jishnu"

[x] The team should have a Web Page(s) that teach/explain: Fibonacci, Algorithm, etc.
[x] The team should have a Web Page(s) that teach/explain: Sorting, Timing, Compares Swaps, etc.

Team Review ticket containing key Team and individual contributions

[x] Growth/Accomplishments in work is according to historical Team Plan, or they show revisions to plan according to work
[x] Short falls/Improvement that could be made in Work or Team Plan, team highlights next steps or improvements that could be made
[x] Showing key accomplishments according to requirement in Java Backend such as API, Abstract Class(es), Inheritance, Polymorphic behavior, Sorting Algorithms, etc.  In Sorting, there should NOT be static classes or static methods.
[x] Showing key user interaction and learning(s).  For instance how you visualized Sorting Algorithm: Bubble, Insertion, Selection, Merge... how you captured Big O, analytical data, usage of Data Structures...  And/or, how you provided response and Feedback to user on their success in learning or experiencing your interface.

Per check.
0.55 not attempted/no check
0.7 attempted, incoomplete, but some runtime
0.8 mastery and runtime
0.9 above and beyond.
Freeform comment.
Provide positivies and growth summary.
Justify or comment on final score.
Be sure to provide extra details on anything below 0.7 average or above 0.8.


stress_line marks vowels that are silent/elided

Broke this off from #12 since the fix will be different.
Currently I can force an elision by replacing the offending vowel with an inverted comma in the original text, but that looks weird when the syllable is silent in normal speech.
Alternatively, I can add an entry to the custom dict, but that will only scale for the most common occurrences.

Changed the name of this issue following the refactoring of show_stress_line into stress_line in issue #14

My current thinking on this is two-fold:

Current code just stops marking/counting stresses once it's run out of word stresses from CMU dict or custom_dict. So all elisions from the end of words are displayed as unstressed and don't count towards the meter.
So the custom dict would only have to deal with words that elide syllables inside (e.g. the two syllable pronunciation of 'evening'.

Closing issue as the code has moved on from this point quite a bit.
There remain issues with words that have fewer voiced syllables in CMU than syllables returned by the syllabifier logic, but the approach described above is the current de-facto behavior.
Next step is really to have custom dict include no just the CMU stresses but a full description of the word as a WordBuilder instance.
Will add a new issue for that.


Datetime as coordinaets does not convert back to datetime (returns int)

What happened:
datetime was in np.datetime64 formet. When converted t datetime.datetime format it returned an int
What you expected to happen:
`to get a datetime returned
Minimal Complete Verifiable Example:
# Put your MCVE code here
import xarray as xr
import numpy as np
import datetime
date_frame = xr.DataArray(dims='time',coords={'time':pd.date_range('2000-01-01',periods=365)},data=np.zeros(365))
print('pandas date range (datetime): ',pd.date_range('2000-01-01',periods=365)[0])
print('dataframe datetime converted to datetime (int): ',date_frame.coords['time'].data[0].astype(datetime.datetime))
print("normal numpy datetime64 converted to datetime (datetime): ",np.datetime64(datetime.datetime(2000,1,1)).astype(datetime.datetime))

output:
pandas date range (datetime):  2000-01-01 00:00:00
dataframe datetime converted to datetime (int):<PHONE_NUMBER>00000000
normal numpy datetime64 converted to datetime (datetime):  2000-01-01 00:00:00

if converted to int, it also gives different lengths of int :
date_frame:
946684800000000000
946684800000000
normal datetime64^
Anything else we need to know?:
it is also mentioned in this SO thread appears to be a problem in the datetime64....
numpy version 1.20.0
pandas version 1.2.1
Environment:
Output of xr.show_versions()

INSTALLED VERSIONS
commit: None
python: 3.7.9 | packaged by conda-forge | (default, Dec  9 2020, 21:08:20)
[GCC 9.3.0]
python-bits: 64
OS: Linux
OS-release: 5.4.0-59-generic
machine: x86_64
processor: x86_64
byteorder: little
LC_ALL: None
LANG: en_US.UTF-8
LOCALE: en_US.UTF-8
libhdf5: None
libnetcdf: None
xarray: 0.16.2
pandas: 1.2.1
numpy: 1.20.0
scipy: None
netCDF4: None
pydap: None
h5netcdf: None
h5py: None
Nio: None
zarr: 2.6.1
cftime: None
nc_time_axis: None
PseudoNetCDF: None
rasterio: None
cfgrib: None
iris: None
bottleneck: None
dask: 2021.01.1
distributed: 2021.01.1
matplotlib: None
cartopy: None
seaborn: None
numbagg: None
pint: None
setuptools: 49.6.0.post20210108
pip: 21.0.1
conda: None
pytest: None
IPython: 7.20.0
sphinx: None

Thanks for clear example. As per @keewis 's label, this does look like an upstream issue.

yes, I believe this to be a numpy issue:
date_frame.coords['time'].data[0]

returns a numpy.datetime64[ns] object, which, casted to datetime.datetime using astype becomes a int. See numpy/numpy#18363 for the upstream issue.

Yes, I created that issue :), should this then be closed?

Possibly, if this doesn't require any action from us. @spencerkclark, what do you think?

Yes I think unfortunately there is not much we can do here, since as @keewis noted, data_frame.coords["time"].data is a NumPy array.
This issue in general is a little tricky since xarray and pandas use nanosecond-precision datetime values, but datetime.datetime objects only have microsecond precision.  See also discussion in https://github.com/numpy/numpy/issues/8546, https://github.com/numpy/numpy/issues/12550, and https://github.com/pydata/xarray/issues/3256.

Closing as dupe of https://github.com/pydata/xarray/issues/3256


Pipeline: use notify instead of polling for AggregateRestoreSourceOp

What problem does this PR solve?
Issue Number: ref #8869
Problem Summary:
What is changed and how it works?

Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/cc @windtalker @gengliqi


banners have been updated and the pfp doesn't allign as it's meant to.

I'll make sure to fix this in v2. Discord decided to add a mask to the banner

Alright thanks :)


Bring website online on staging server

@jwalsh and @jotajunior: For when did you have this scheduled? My thought was that it would be good to have this going before the weekend, so we can test and debug over the weekend.
Also, could you walk me through how access/updating of the website version on the staging server would work? Thanks!

I'm looking to have this online later tonight:

@DarkSymmetry will be the primary account for the deployment; a deploy key has been added
test pages will still be available
The scraper directories may need to move out of /var/www/ : since they're hosted in the same repository I'll need to confirm with @jotajunior that we've got the current files in the repository.

To deploy:
cd ~/ProjectGAMR; git pull
I'll see what I can do to push deploy notifications to Slack when we update.

Okay, thank you!


I don't understand in what circumstances I should use ng-include and when I should use ng-inline

The blog post link has broken.
As title mentioned, how should I decide? And what is the difference between ng-include and ng-inline besides speed?
Thanks for helping

Thanks for pointing out the busted link; fixed!
You would prefer ng-inline for things that might happen in tight loops -- you'll make bootstrapping slower, but linking faster.  Whereas ng-include will keep your bootstrap faster, but make your linking slower. Also, ng-inline only supports templates that are already in the template cache.


Resource not found error- Create webapp

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

[0] create a generic issue.
[1] az webapp create       : FAILURE. Ran: 1 hrs 59 mins ago.
[2] az webapp create       : FAILURE. Ran: 1 hrs 55 mins ago.
[3] az webapp create       : FAILURE. Ran: 1 hrs 55 mins ago.
[4] az webapp create       : FAILURE. Ran: 1 hrs 53 mins ago.
[5] az webapp create       : SUCCESS. Ran: 1 hrs 51 mins ago.
[6] az login               : SUCCESS. Ran: 20 mins ago.
[7] az account set         : SUCCESS. Ran: 20 mins ago.
[8] Unknown (help version) : SUCCESS. Ran: 15 mins ago.
[9] az version             : SUCCESS. Ran: 14 mins ago.
Describe the bug
"az webapp create " command returned "ResourceNotFound - The Resource 'Microsoft.Web/sites/testapp' under resource group 'mmmmm' was not found."
Here is the command:
`user@Azure:$
user@Azure:$ az webapp create --resource-group mmmmm --plan myAppServicePlan --name testapp --runtime "PHP|7.3" --deployment-local-git
ResourceNotFound - The Resource 'Microsoft.Web/sites/testapp' under resource group 'mmmmm' was not found.
user@Azure:~$`
To Reproduce
Configure deployment user
Create Resource group
Create App service plan
Create Webapp with app name
(Followed tutorial https://docs.microsoft.com/en-us/azure/app-service/app-service-web-get-started-php)
Expected behavior
Expected webapp to be created
Environment summary
"azure-cli": "2.4.0",
"azure-cli-command-modules-nspkg": "2.0.3",
"azure-cli-core": "2.4.0",
"azure-cli-nspkg": "3.0.4",
"azure-cli-telemetry": "1.0.4",
"extensions": {}
Additional context
It appears the name "testapp" is not an available name. I noticed this when I tried the same step in portal as it clearly said "name not available". I changed the name in the CLI and then it worked. The error return was confusing

webapp

We should add site name validation to create command as well and that should make this experience match portal.

@calvinsID we currently don't do CheckNameValidation on create - once we make the changes to make show the detailed error message on az webapp up - lets see if we can use the same on create as well.


RESOLVED: Cannot read property 'eachRow' of undefined

Hi,
I apply exactly the same APIs within the following code
var Excel = require('exceljs'); var inboundWorkbook = new Excel.Workbook();
  inboundWorkbook.xlsx.readFile("Data/DataSet.xls")
    .then(function() {
      var inboundWorksheet = inboundWorkbook.getWorksheet(1); //or name of the worksheet
      inboundWorksheet.eachRow({ includeEmpty: true }, function(row, rowNumber) {
        console.log("Row " + rowNumber + " = " + JSON.stringify(row.values));
      });
});

but I got the error TypeError: Cannot read property 'eachRow'
I work in version<EMAIL_ADDRESS>so I update eachrow function exactly like the documentation but still doesn't work

+1 I am getting this error from time to time as well.
Could it be the result of trying to get a worksheet that has been deleted? If so, how can I be sure to access the first worksheet that has data in it?

This just happened to me in a production environment. I have this loop, which has worked until now:
  // Find first available worksheet.
  let sheetId: number = 0
  let worksheet: Excel.Worksheet | undefined
  do {
    sheetId++
    worksheet = workbook.getWorksheet(sheetId)
  } while (!worksheet)

For this particular file which contains exactly one worksheet named simply "Sheet1", it never finds the sheet, ie. worksheet always equals undefined, and the loop runs forever. Any solutions?

Update: as noted in this issue: https://github.com/exceljs/exceljs/issues/652, this happens when the workbook is stored in an older format, in this case, it's "Excel 97-2004 Workbook" format. I'm simply not able to fetch a sheet from those :-(

@hazemsamoaa @bam-tbf  @mkock
It isn't required to worksheet has next id, so first one could have id=10, second one id=3 and the other id=1213.
To iterate through worksheets I recommend use array worksheets:
next = wb.worksheets[i++];

I got the error TypeError: Cannot read property 'eachRow'.
Please let me know the solution what should be there


Minor issue in htmlredirect.go - reservedNames in Windows - COM0 and LPT0 are also invalid on Windows 7

I just tried creating files named COM0 as well as LPT0 on my machine and they both give the error "The specified device name is invalid" (the same as the others).
See: htmlredirect.go
Line: 73 (reservedNames := []string{"CON", "PRN", "AUX", "NUL", "COM1", "COM2", "COM3", "COM4", "COM5", "COM6", "COM7", "COM8", "COM9", "LPT1", "LPT2", "LPT3", "LPT4", "LPT5", "LPT6", "LPT7", "LPT8", "LPT9"})
Tested on Windows 7 Home Premium x64 SP1 (English).
I could try on Windows XP and Windows 10 if needed.

According to this doc, the Windows reservation is relative to a "namespace."  To open the COM0 device, you open the paths \\.\COM0 or COM0.  However, it's okay to open content/post/COM0.
Microsoft recommends that you don't create files that match their namespace reservations, but we may be able to so long as we open the file relative to a path.
Original commit was by @anthonyfok.  Comments, Anthony?

I was not thinking clearly when I saw your PR yesterday.  I see what you're saying now.
My point was to question if the reserved names list was necessary.  Are you saying that you couldn't create a file named COM0 with Hugo?

Hello!  Sorry for my tardiness in reply.
It would appear that Microsoft, at times, differ in their code and their documentation.  COM0 and LPT0 are also considered invalid in their code, at least as of Windows 7, but their documentation does not yet cover those two cases.
And if you (@moorereason) hadn't reminded me, I would have totally forgotten that I committed this code in the first place.  I think one of the Hugo users suggested this to me, and then since the Microsoft documentation says so, I went ahead and did it, but quickly forgot about it.  :-)
@magikstm, thank you for catching COM0 and LPT0 that I missed.
There may be some other cases that I missed, as in while e.g. COM1 is reserved, so is COM1.txt and the likes, and I doubt that my original code checks for such reserved names with added file extensions.

My point was to question if the reserved names list was necessary.
Are you saying that you couldn't create a file named COM0 with Hugo?

I guess Hugo might be able to create such a file name under Windows, depending on the version or the shell?, but since Microsoft specifically tells us to avoid these reserved names, it would be good programming practice to avoid them, to avoid creating unforeseen troubles for the end users.

An interesting tidbit for reference:

What’s the deal with those reserved filenames like NUL and CON? (on MSDN)

There may be some other cases that I missed, as in while e.g. COM1 is reserved, so is COM1.txt and the likes, and I doubt that my original code checks for such reserved names with added file extensions.

I can't create a file named "COM0.md", "COM1.md" or any other of the reserved name on my system. My system isn't the cleanest as I do use it for development, but I believe I've not changed that behavior.
How do you handle Windows-specific tests? I believe there are no Windows server in the CI build. Are tests for this platform run "as needed"?
I did some interesting tests trying to copy files from WinSCP (from a Linux Centos 7 server) to my Windows 7 machine:

Copied a file named "COM0" -> It was copied as "COM0"
Copied a file named "COM0.md" -> It was copied as "COM0.md"
Copied a file named "COM1" -> It was copied as "COM1%00"
Copied a file named "COM1.md" -> It was copied as "COM1%00.md"

it would be good programming practice to avoid them, to avoid creating unforeseen troubles for the end users.

I agree.


A11y_AzureDataStudio_Filter_SortingAscending/Descending_ScreenReader: Orca is announcing incorrect control type for the "Search" edit control.

“Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”
GitHubTags: #A11yMAS;#A11yTCS;#A11ySev2;#SH-AzureDataStudio-Linux-Jul2021;#SQL Azure Data Studio;#Linux;#DesktopApp;#WCAG4.1.2;#Orca;#AINotInScope;#FTP;
Environment Details:
Azure Data Studio Version
Screen Reader: Orca
Steps to Reproduce:

Launch Azure Data Studio
Connect to any SQL server and write a query
TAB till results grid and press F3 so that the filters pop-up will open
Start Orca screen reader and then TAB to "Search" edit control
Observe the announcement by screen reader

Actual:
The screen reader announces "Search entry" for "Search" edit control which is incorrect as there is no information announced for the control type of the search control.
Expected:
The screen reader should announce "Search edit "
User Impact:
Screen reader users will get confused with the control type of the button.

@12-shweta does ORCA read anything as "edit"? I am afraid that is an Orca thing to read text box as entry.

After discussion with SME this seem to just be the differences in how Orca announces these elements

Github Tags: #Closed;


Roadmap of 2022

Roadmap List

[ ] use cloudflare worker as front layer
[ ] preload deps
[ ] transform jsx/ts/tsx for browsers from githubimport "https://esm.sh/~ije/ui/Button.tsx"  // Content-Type: application/javascript
import "https://esm.sh/~ije/ui/Button.tsx?jsx=preact"  // Content-Type: application/javascript

Don't forget to pin this issue instead of #99!

Does preload deps mean defining the deep imports up front? Also, have you considered putting those imports in the response header to preload as well to help reduce the waterfall?

I believe there are some issues related to imports and version pinning that would be good to get addressed. I added a few comments to this issue after attempting to upgrade from v85 to v89 of esm.sh today.
https://github.com/ije/esm.sh/issues/325

interopDefault appears to not work correctly.
esm.sh version pinning doesn't always work for dependencies.
There was a recent regression, I started getting a type issue after upgrading from v85 to v89.

Is there a way to just use this library on the command line or within a nodeJs script to convert a node package?
I don't need a server, but just the logic as part of a generic build script.

I would like to have a parameter to mark all nodejs builtins as external, because if you list them all you get an error because the file name is too long


Unable to run patch_sklearn()

Describe the bug
I updated it to the latest version of 2024.1.0, but I got an error when I tried to run patch_sklearn() after importing patch_sklearn from sklearnex.
The error message says that: AttributeError: module 'threadpoolctl' has no attribute 'ThreadpoolController'
To Reproduce
Steps to reproduce the behavior:

pip install scikit-learn-intelex
from sklearnex import patch_sklearn
run patch_sklearn()
AttributeError: module 'threadpoolctl' has no attribute 'ThreadpoolController'

Expected behavior
Able to enable patching
Output/Screenshots

Environment:

Databricks Runtime Version 13.3
Python 3.10.12

threadpoolctl is dependency of scikit-learn and scikit-learn-intelex and definitely contains ThreadpoolController.
This issue is highly likely on environment side.


Update UG and DG implementation for attendance mark and unmark command

Fixes #148
Fixes #168

Hi, I run gradle check locally but it doesn't show any checkstyle error in the DeveloperGuide.md file, I'm not so sure why it fails in github, do you know why?

The checkstyle is failing for the ubuntu os but works for windows. If you click on 'Details' beside the failed build, it shows the lines with additional white space at the end of the line that is causing the build to fail.

Hi, I run gradle check locally but it doesn't show any checkstyle error in the DeveloperGuide.md file, I'm not so sure why it fails in github, do you know why?

The checkstyle is failing for the ubuntu os but works for windows. If you click on 'Details' beside the failed build, it shows the lines with additional white space at the end of the line that is causing the build to fail.

Thanks, already updated

docs/images/ListAttendanceResult.png
i think the command is misspelled here

Thanks for the review, already updated the image


Add hh filetype to the plugin.

To help configure other tools as phpcd.vim we need a way to avoid php collision. Using hh as a new filetype has more sense since it's really not php but hack what we want to use.

This looks to remove the .php extension support. Keeping this is important -- a lot of Hack code exists with a .php extension. (Notably all of FB's code is this way, but I've seen it a lot externally too.)

Hi josh!
Thaks for answer me, just I pushed something that solves maybe this issue. It is useful for me because in my company we use too lot of code with php extension but real <?hh.
Just feel free what is best for the project, check my last commit.
Thanks! :D

@jgimeno thanks for the update, appreciate the PR. Your change is supposed to gate the behaviour on the file actually starting with <?hh, is that correct? I'm far from an expert on vimscript :)
@mxw are you still around, can you take a look at this?

Yes, that's correct @jwatzman. For me it allows to use different omnifunc engines.
The thing about setting on every .hh, .hhi or even .php as filetype=php didn't allow me to set a different omnifunc for hack files (hackcomplete) and for php (phpcd.vim). This works for me but I don't know if it's enough for all people using the plugin.
Thanks!

Ok, I'm a little late to the party ;)
Having an own filetype for hh is a good idea. My only question regarding this change is: will changing the filetype cause problems with other plugins which depend on the php filetype? Actually I don't know many hack-plugins, but...

Hi @usox ! Don't worry! Being using those changes half a year still did not have any issue, but as you said it could have some problems that maybe are not foreseen. So let's wait for some opinions! But I guess there isn't a lot of movement lately around here.
Thanks!

Resolved by #40 - setting filetype to 'hack', and syntax to 'php'


added a user email for the git config

In the middle of fixing the auto docs

seems to be successful, ll of the tests are passing


NIH: de-duplicate items collecting funding rounds

As we found multiple items with different funding amounts, these are clearly different funding rounds for the same project.
To solve this we are discussing about aggregating rounds as a new field of the item we could call "earliest subproject", so that (pseudo code/names):
[
  {id: 1, name: "foo", date_start: "2001-01-01", date_end: "2011-01-01", cost_ref: 100, year_fiscal_funding: 2010},
  {id: 2, name: "foo", date_start: "2003-03-03", date_end: "2013-03-03", cost_ref: 300, year_fiscal_funding: 2012},
  {id: 3, name: "foo", date_start: "2005-05-05", date_end: "2015-05-05", cost_ref: 500, year_fiscal_funding: 2014},
  {id: 4, name: "bar", date_start: "2002-02-02", date_end: "2004-04-04", cost_ref: 200, year_fiscal_funding: 2003},
]

would become:
[
  {id: 1, name: "foo", date_start: "2001-01-01", date_end: "2015-05-05", cost_ref: 900, year_fiscal_funding: 2010,
    cost_rounds: [
      {id: 1, date_end: "2011-01-01", year: 2010, cost_ref: 100},
      {id: 2, date_end: "2013-03-03", year: 2012, cost_ref: 300},
      {id: 3, date_end: "2015-05-05", year: 2014, cost_ref: 500},
    ],
  },
  {id: 4, name: "bar", date_start: "2002-02-02", date_end: "2004-04-04", cost_ref: 200, year_fiscal_funding: 2003},
]

where:

for aggregated items,

date_start is the date_start of the earliest subproject
date_end is the date_end of the latest subproject
year_fiscal_funding is the year_fiscal_funding of the earliest subproject

suprojects have format [{id, date_end, year, cost_ref}]

Related: https://github.com/nestauk/nesta/pull/112#issuecomment-516368072
[1] See for example top two results at: https://search-health-scanner-5cs7g52446h7qscocqmiky5dn4.eu-west-2.es.amazonaws.com/nih_v5/_search?q=Motion Compensated Brain PET Imaging for Neuroscience Research&_source=title_of_project,date_end_project,date_start_project,year_fiscal_funding

the second item is the first round, funded in 2015, project being expected to end in 2017
the first item is the second round, funded in 2016, project being expected to end in 2019

Note that booleanFlag_duplicate_abstract will be deprecated

Closed by #112


[FR] Custom buttons to call outer actions (eg. build command for SSG websites)

Cockpit is great ❤ , I'm using it to feed my static generated websites but there's one thing I miss so much:
Custom buttons to call outer actions (URLs) like: Build production site, Build preview site, whatever...
Registering these buttons should be implemented almost the same as webhooks with a form like this:

Button label (caption)
URL (to call when button is clicked)
Advanced (HTTP basic auth, custom headers...)
When clicked... (dropdown select)

Open the URL in a new window
Wait for response (requiring some predefined JSON structure, eg. alert message and/or an URL to open in a new window)

Registered buttons could be visible on the Dashboard or in the top menu with a dropdown showing the registered actions (buttons).
What do you think?
Thank you!

Hi, I am facing the same problem, my first thought was to simply use webhooks with some event like singleton.saveData.after (I am using only singletons at the moment) but I wanted to be able to call this action on demand.
Also, I am using Github Actions to build my site and it is required to send a custom field in the body (event_type) and is not possible to do that with the current webhook implementation: https://docs.github.com/en/rest/reference/repos#create-a-repository-dispatch-event
I've done an addon very tailored to my needs, it adds a button to the menu and requests the Github URL.
https://github.com/andersanmiguel/cockpit-github-webhook/
I was thinking on make a pull request to try to integrate the ability to add custom body fields to the webhooks and then modify the addon to simply trigger an event, but I don't know if it would be better to make an addon that duplicates almost all the webhook functionality and leave the core as it is.
What do you think is better?


Stealing ractions

Summary
Whenever someone uses react command, there would be small chance for it to misfire and do something else.
For example react with other person or bot itself
Why is this needed?
because it is a feature ppl would like to have on their server
Checklist

[X] I have made sure not to add ANY sensitive information (tokens, passwords, credentials, personal details, etc.).
[X] I have searched the issue tracker and have made sure it is not a duplicate. If it is a follow up of another issue, I have specified it.

Would these specifications fit your needs?
Feature: Altering a /react command usage
Given react misfire is enabled
And I am a user
And my next react invocation is predicted to misfire
When I invoke the command /react action: hug target: Kriz
Then the action and/or target are randomly chosen
Feature: Configure the probability of react misfires
Given I am the owner of the server
When I invoke the command /react misfire set action_rate: N% target_rate: M%
And I invoke the command /react action: <any action> target: <any user>
Then the chances of the action changing is N% and target changing is M%
This way server owners would be able to configure the rates or even disable the feature altogether on a per-server basis.

Yes, amazing idea

Please, use appropriate names for your issues.


[bug] Too long without yielding when downloading basalt on slow internet connection.

My internet where I am is trash (less than 500kbps).
I tried downloading Basalt with wget run https://basalt.madefor.cc/install.lua packed, and got this error:

I Think This Issue Is Caused By An Issue In The Update Script. I Am Not A Dev For This But I Think They Need To Add A Yield.

Now that I think about it, it may not even be with Basalt, but with CC's download... although I'm not sure about that (also not a dev).

I will look into this. I think it is a basalt issue, it just needs a yield somewhere when downloading. Thanks, forgot a bit about that - sorry!


Bump github.com/golangci/golangci-lint from 1.60.3 to 1.61.0 in /hack/internal/tools.

What type of PR is this?
/kind cleanup

What this PR does / why we need it:
Bump github.com/golangci/golangci-lint from 1.60.3 to 1.61.0 in /hack/internal/tools.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Fix for https://github.com/kubernetes-sigs/kueue/pull/3061.
Does this PR introduce a user-facing change?

NONE

/assign @mimowo

/lgtm
/approve


Programming Contest Sites

Title
Dataset URL - Collect Dataset from Programming Contest Sites
Does the dataset exist in a scraped format?   Exists some resources available like CodeContest  From DeepMind, APPS, LeetCode.
Description
Code Data from Competitive Programming Pages is a high-quality resource for code generation. Websites like Codeforces, AtCoder,... provided good resources about competitive programming problem and code.
Procedure

[x]  Collect CodeContest dataset available
[x]  Processing and formatting.
[x]  lm_dataformat Processing
[x] Crawling TopCoder
[ ] Crawling LeetCode

Seems like CodeContest is missing TopCoder so perhaps we should scrape TopCoder ourselves

Seems like CodeContest is missing TopCoder so perhaps we should scrape TopCoder ourselves. Also Hackerrank

Yes, I will try to crawl data if needed. Thank you, that's a good point!

Seems like CodeContest is missing TopCoder so perhaps we should scrape TopCoder ourselves. Also Hackerrank

Yes, I will try to crawl data if needed. Thank you, that's a good point!

No worries I have a topcoder crawler!

Great! Any PR?

Just check HackerRank and LeetCode, programming problem we can scrape but code solutions it's seems not available.

DeepMind CodeContest Dataset is available under EleutherAI/lm_dataformat. Each document included: the problem description, test cases, solutions, and incorrect solution code.
Please check this link.

@PhungVanDuy For LeetCode, maybe we can scrape solutions from discussions, filtering by votes.

That's a good idea. Let me check it. Thank you so much!

I just found the way to crawl data from Topcoder:
Note below:

Access this link to get all problem statements: https://www.topcoder.com/tc?module=ProblemArchive
With one problem, go down we will see the link lead to the contest that problem belongs to (see image 1)

Jump to that link we will get the source code solutions
<img width="1189" alt="Screen Shot 2022-09-23 at 14 44 16" src="https://user-images.githubusercontent.com/28798474/1919
DeepMind CodeContest does not have problems from Topcoder if we get one it is also a good one to contribute to the Programming Contest dataset.

As I estimated (by estimate counting contests) have more than 5000 problems in Topcoder, just worry about the license. Following Topcoder statement:
This problem statement is the exclusive and proprietary property of TopCoder, Inc. Any unauthorized use or reproduction of this information without the prior written consent of TopCoder, Inc. is strictly prohibited. (c)2022, TopCoder, Inc. All rights reserved.
Any comment for this @ncoop57.

It looks like we need to email them to see there full policy. @PhungVanDuy Lemme contact someone to see how we can reach out to them about this

Great! Meanwhile I still do crawl this one Louis suggest in bad cases we can index but keep data for us to finish First Version of Competitive Programming Dataset.  Thanks

I started working on LeetCode.

@ncoop57  https://github.com/CarperAI/Code-Pile/pull/24 for TopCoder scrape dataset, I need to run on batches to get full data, and integrate with CodeContest before then.

@PhungVanDuy @faraday @Dahoas please add the following information to the issue description:
Tests
Include a dummy_dataset.parquet file to test your code against. This dummy_dataset should include the columns for the data and metadata associated with the dataset, which will then be converted into the final format for language model consumption, along with an example row or rows that you can verify your code correctly collects. In addition to this file, include the unit test that evaluates your code against this dummy_dataset.
Give an example of the columns and data:

col1
col2
....

row1
row1
....

@ncoop57 Happy to share that the Competitive Programming Dataset for Code Pile is finalized the first version included CodeContest (train dataset only to prevent fairness in the later benchmark) and TopCoder, which totally we have 18267 problems with solution code, with CodeContest we also have incorrect solutions as well. I believe this is the largest CP dataset currently. I pushed the parquet file in this. Kindly check this and give me a comment if have any concerns with this dataset.

@PhungVanDuy awesome job Duy! One thing though, when you store the data in parquet files, please store the metadata as separate columns instead of in the HTML-style tags as the parquet is an intermediate storage for now until it gets converted into the final jsonl format to be fed to the large language model. So, for example, the parquet should have a format like this:

title
source
tags
difficulty
problem
hint
solutions
...

row1
row1
row1
row1
row1
row1
row1
....

Similar to https://huggingface.co/datasets/deepmind/code_contests#data-instances
This parquet will later be converted into the format you currently have with the HTML-style tags. Sorry for the confusion 😅

@ncoop57 I got it, so do you want me to make lm_dataformat for html-styles also besides parquet like above?

not quiet yet @PhungVanDuy as during our weekly meeting we are going to finalize the format, so it might change slightly

not quiet yet @PhungVanDuy as during our weekly meeting we are going to finalize the format, so it might change slightly

Got it, no worries I have the function on code to make HTML so we can easy to change.
P/S: Hint is a new field that I combined from my research before (I collected tutorials from Codeforces and write in latex form), we can note this field to explore how tutorials affect models later.

Updated files at this link, I also refactor CP Dataset (without scrape on this file, just fetch dataset from Gdrive), only Topcoder is had scraper code. I also added unittests.py file and dummy data.

@faraday  you can make some updates about LeetCode here to keep tracking.

Here are some statistics about the LeetCode data:

Questions: 2421 records (7.8 MB as JSONL, 1.2 MB bzipped)
Discussion topics (pseudo-solutions): 2351568 records (3.2 GB as JSONL, 405 MB bzipped)
Comments: 525802 records(430 MB as JSONL, 46 MB bzipped)
Comment replies: 293361 records (222 MB as JSONL, 26 MB bzipped)

I have used asyncio and aiohttp in scraping code but server-side rate limits are bottleneck in downloading topic data.
I think it's best to grab the snapshot (time-tagged with 20220928) and shape the scraper as a delta-update from there.

As an example, Two Sum has around 11k related topics, many of them people sending their solutions in different languages (probably covering most of programming languages). If we are to represent topics as solution list together with the question, I don't think databases could cope well with that. We'll discuss more about this, noting it here. We should probably aim for representing it as Forum as you previously suggested.


130.0.1 (64-Bit)

No longer works. The input field in the popup or new tab is empty, and the text is not transferred.
Fresh Firefox installation, no other add-ons installed.
Win 10; Fire Fox 130.0.1 (64-Bit)

same here Win 11 Fox 130.0.1 (64-Bit)

The problem appears to have been caused by DeepL changing the URL for translation.
Previously, passing #null/en caused it to auto-detect source language and translate to English. With the change, however, #null is not a valid language and causes the input to remain empty.

I don't know what to put instead of null, tried some other variants, but it doesn't help

@xpmn This seems to be the language of the word that should be translated:
https://www.deepl.com/de/translator#en/de/hello
https://www.deepl.com/de/translator#de/es/hallo

@no-one yes, I know, but previously I passed null to initiate autodetect. Now it stop working, so I've set to en just to make it at least not lost text

Thank you for the prompt fix. Could you please upload update to Mozilla ?

It's on review already

yes it was ¯_(ツ)_/¯
but whole this extension is a hack

FYI, I was tired of waiting for Firefox to review and installed it directly from github. It was much easier that I expected, cheers for the fix :)

I am unable to check right now – but does language detection occur client-side or server-side? Because in the latter case, we could simply send a request to DeepL server, get the language in return, and only then open the URL with the correct languages.


[Excel] (ExcelApi 1.12) Add two links to ExcelApi 1.12 release notes page

Docs Build status updates of commit 41bd7c7:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/reference/requirement-sets/excel-api-1-12-requirement-set.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Add support for persistent references

Python has the mechanism for objects in one pickle to reference objects
in another pickle. This mechanism is primarily used in ZODB object
database for one object to "persistently" reference another object(*).
I needed a way to load and understand objects from a ZODB database in
Go, and hence comes this patch for ogórek that teaches it to understand
such persistent references.
Like it is already with Call and Class we add a dedicated Ref type to
represent persistent references in pickle stream.
(*) in ZODB when one object is changed, there is no need to resave
unchanged object that is referencing it because its reference stays
valid and pointing to changed object: the "persistent ID" in ZODB is
only object ID and which revision of the referenced object it points to
is implicitly follows by current transaction number view of the database.

Looks good to me. Thanks!

Kamil Kisiel писал 2018-07-01 23:14:

Merged #37 [1].
--
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub [2], or mute the
thread [3].
Links:
[1] https://github.com/kisielk/og-rek/pull/37
[2] https://github.com/kisielk/og-rek/pull/37#event-1710417344
[3]
https://github.com/notifications/unsubscribe-auth/ABtgyJI_6zPZF5M-3M6Oz10F4pfFtivtks5uCS26gaJpZM4U8fsr

Thanks.

Thanks for merging the patches.


Public Polls that show who voted for each option

The anonymous poll is great for asking questions such as "did you like today's session?" or "did the BBEG deserve to die?" but when it comes to "who can come tomorrow for the next session?" you want to see what everyone posted.
My proposal is that a new public poll type is added. In that poll type, you have the player names or character names under each bar in the poll. A visual way to do this would be to use a tiny player or character tokens as assigned to players. This would make it very similar to polls in Facebook Messenger.

Done in v1.0.1


SaxonJS supports setting JSON as the context but SaxonJ does not

In tasks.json for SaxonJS we can set the jsonSource property that corresponds to the -json argument for the Saxon command-line-interface. We need to add the equivalent for SaxonJ (Java) also.

Issue now fixed