Update teams-live-event-usage-report.md

Adding that anonymous presenters will not be included in the report.

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@HowlinWolf-92 PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


Fix crash with media (images/videos) on Android 4.x

The preview is now displayed correctly, and application does not crash anymore.
I did not find why nothing happens when I click on it...

@af-anssi FYI we decided to fix this crash by another way
see https://github.com/dinsic-pim/riot-android/pull/333


Seeding user

How is the initial ASP.NET Identity user seeded? Is it similar? What about claims?

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

ID: fc418ab9-0592-f610-e751-40cb501220da
Version Independent ID: 822e8923-2647-f573-0c74-66af903776d8
Content: Data Seeding - EF Core
Content Source: entity-framework/core/modeling/data-seeding.md
Service: unspecified
Product: entity-framework
GitHub Login: @AndriySvyryd
Microsoft Alias: divega

I too would like to know how to seed a user in EF Core 2.1.0.

Duplicate of #736


USB printing on macOS

Hi! I'm trying to use this library to print receipts from a POS application. However, I'm facing some trouble getting my printer to show up as an available device.
For development, I'm using a MacBook Pro with M1 Pro chip (so arm, not x86). Eventually, the software should run on a Raspberry Pi so I'm taking a Linux approach overall. The printer is an Epson TM-T20II with USB and serial ports, connected via USB directly to my laptop.
First, I had the printer show up as a 'normal' printer in my system preferences, and I could select it as a print target in e.g. my text processor. Since this doesn't seem like "raw" printing mode, I've used the TM utility to change from "Printer Class" to "Vendor Class". This made the printer issue a short buzz, and since then it's listed as "offline" in my printer menu.
Going by the USB usage guide in the readme, I should see the printer exposed as a file under /dev/ttyusb or a variation. Unfortunately, neither of them seem to exist on my machine after connecting the printer. Some files I've looked for: /dev/ttyusb, /dev/ttyusbserial, /dev/usb/lp*, /dev/tty.*, ...
In my application, running SerialPort.GetPortNames() only yields /dev/tty.Bluetooth-Incoming-Port and /dev/cu.Bluetooth-Incoming-Port.
Going by some of the existing issues, the "virtual COM port driver" seems to come up now and again, but that's probably not applicable here?
Am I missing something here? Is USB printing on macOS possible at all? Would using a parallel to USB cable instead work?
I'm willing to explore some options here, but am a bit clueless where to start looking.
Thanks in advance!

Hi @butskristof ,
Yes, USB printing on macOS should be possible. It was only tested in 1.x releases, but the FilePrinter implementation hasn't changed since then.  I haven't tested it on an M1, but I don't think that should make any difference, and it seems like your problem is related to getting the printer available in the first place.
The virtual COM port driver is only really useful on Windows, because Windows doesn't expose a native serial-level communication for USB devices unless a driver implements it.  On Linux and MacOS, the lower level direct serial communication to the printers is exposed as a file handle, much like many other devices on those platforms.  That's the thing that shows up in /dev/ttyusb*.  I don't think having a higher-level print drive for not having it would make any difference as far as ability to print using the file handle.
I have the same MacBook as you, as well as a TM-T20II USB - funny coincidence!  I'll give it try this weekend and see if I can reproduce your issue or point you in the direction of how this specific printer mounts on MacOS.

Thanks for the clarification @lukevp ! At least now I know I was on the right path :-).
Just to verify, I've loaded up Linux Mint on another computer and connected the printer. There actually was a /dev/usb/lp0 file handle there. Could it be that I'm just looking over the mount point in macOS? There were a lot of tty devices, just none that pointed to USB or LP.
Is there any command I could use to single out one of these as the printer?

@butskristof looked into this a little more.  One easy way to tell if the device is mounting is just to capture the device list to a file, then plug it in, and see if there are any new devices.  something like this:
ls -al /dev > ~/devices-before.txt
plug device in
ls -al /dev > ~/devices-after.txt
then you can use diff cli command or do what I do - open one file in vs code and do a Compare Active File with... command to compare to the second file.
My findings so far: the device did not mount for me either on macOS 13.1 with an M1 Pro.  I looked into this a bit and it seems that M1 compatibility is lower with receipt/label printers out of the box, so we will likely need a custom driver.  I looked on Epson' site and found a download for the TM series drivers for Mac.  the problem is that it only lists up to macOS 12, and only lists the TM-T20III.

The good news is that the M1 is listed there for the newer OSes.  It's possible that the driver is still compatible with macOS 13 and with the TM-T20II (and it's just not listing those 2 because in the case of the OS it was released before 13 came out, and in the case of the printer, they only list the latest model of that line). However, even installing the driver doesn't expose the device handle anymore and only supports USB-level printing.  That would be a bummer and a pretty major functional regression but it's definitely possible that that's just how the driver works now.
These are my suggested options for you:

use RS232 and a generic usb/serial interface that should happily mount to /dev/, probably with no drivers.
switch to a wifi or Ethernet printer (these both definitely work from Mac as this interface is not dependent on the host at all)
reach out to Epson to inquire about using a /dev/ mount with the TM-T20II on MacOS 13.1.  If you do this, feel free to CC me as well, as I'd be very interested to hear from them about this support.

After re-reading your original post, your final destination is a Raspberry Pi anyway.  Might I suggest you just connect the printer directly to your Raspberry Pi?  You can then use VS Code Remote Development to work on the scripts from your Mac but have a remote terminal on the Raspberry Pi for testing and issuing commands.

Hi Luke, thanks for the detailed research!
Bummer that the Apple Silicon support isn't there and that we're dependent on Epson driver support now. It's only to be expected Epson will stop providing these drivers now.
As I mentioned before, on another computer with a Linux installation the device seemed to show up so I'll probably just use that for developing the part of the application that actually requires the printer (and the final deployment).
Good to know that a generic RS232/USB would probably work as well on my Mac, should I ever have to go that route.
Thanks again for your help!


Have an option for atlas administrators to be emailed whenever a new user is created/registered

This should be configured in mail.properties as an option (true/false?)
Possibly allow for a configurable URL that is included in the email (ex. link to the user management page of the atlas)

@alexgao1 which one of these can configure the Subject line in the notification email? Is is configurable like the other emails inside atlasname/docs?

@alexgao1 which one of these can configure the Subject line in the notification email? Is is configurable like the other emails inside atlasname/docs?

Have you performed a nunaliit upgrade on the atlas?
The email templates should be called org.nunaliit.email_template.user_registration and org.nunaliit.email_template.user_registration, which will both have a body.txt and a subject.txt.


Demo Web-App (based on Panel)

Build a MWE of a Pyodide-based Panel web-app using Brightway.
Compare the Panel documentation:

Converting Panel Applications

https://github.com/brightway-lca/bw_webapp


Copy the .env file into the distribution directory.

Good catch. not merging this PR cuz i'm doing  a huge re-write. I've incorporated this fix into the re-write.
see https://github.com/jaws-framework/JAWS/commit/f271f17524962c0dfec5bb658bbebc06523a4939


New error values in under README.md/Notes

Are the updated results in the Notes section of README.md evaluated with depths capped at 80m, or with depths capped at 50m?

I think it is capped at 80m. I can get similar numbers using their training code.

It is capped at 80m.


Unable to user custom created theme (own style )

I have created style for panellist. remove animation . it not working

Hi @chinu13Jain,
Actually, I should not add the animation in the default style.
I will remove it on the next version, thank you.


Fix Autofac sample

Related to that tweet

StructureMap output:
Sending Ping...
- Starting Up
-- Handling Request
--- Handled Ping: Ping
-- Finished Request
- All Done
Received: Ping Pong
Sending Ping async...
- Starting Up
-- Handling Request
--- Handled Ping: Ping
-- Finished Request
- All Done
Received async: Ping Pong
Publishing Pinged...
Got notified.
Got pinged.
Got pinged also.
Got notified also async.
Publishing Pinged async...
Got notified.
Got notified also async.
Got pinged async.
Got pinged also async.

Autofac PR output:
Sending Ping...
- Starting Up
-- Handling Request
--- Handled Ping: Ping
-- Finished Request
- All Done
Received: Ping Pong
Sending Ping async...
- Starting Up
-- Handling Request
--- Handled Ping: Ping
-- Finished Request
- All Done
Received async: Ping Pong
Publishing Pinged...
Got pinged also.
Got pinged.
Publishing Pinged async...
Got pinged also async.
Got pinged async.

The contravariant source ensures that the second part is correct.
I wonder, is there a way to apply the registration source only to specific configuration?

Ah right, I focused on the first part, sorry about that.
I did manage to have Autofac not resolve a GenericPipelineBehavior<object, Pong> by putting generic type constrainsts, but we still get too many. I'll try and play with it a bit more to see if I get to something.
It's interesting, because when I think about it, Autofac seems correct, whatever that means, even though in this case it's not what we expect.

This is still missing the registration for the single generic parameter types
IRequestHandler<> / Async / ICancellable
It's easily overlooked.


Python pandas dtypes

Pandas DataFrame supports more dtypes than 'int64', 'float64' and 'bool', therefor added a bunch of extra dtypes for the data variable.
From now on the label variable can be a Pandas DataFrame with the same dtypes as the data variable or it can be a numpy.ndarray
If the label is a Pandas DataFrame, it will be converted to float.
If no feature_types is set, the data dtypes will be converted to 'int', 'float' or 'i' (bool).
The feature_names may contain every character except [, ] or <
List of possible dtypes:
'int8', 'int16', 'int32', 'int64'
'uint8', 'uint16', 'uint32', 'uint64'
'float16', 'float32', 'float64'
'bool'

An example to test various changes:
import xgboost as xgb
import pandas as pd

col_1 = [76.1, 44, 20.4, 42, 74, 73, 6, 57, 59.224, 26, 64, 28, 99, 90.8, 40, 77, 14, 2, 97, 33]
col_2 = [62, 31, 51, 19, 1.4, 80, 74, 15.22, 42, 26, 29.7, 95, 2, 99, 12.87, 58, 30, 68, 92, 45]
col_3 = [1, 1, 0, 0, 1, 1, 1, 1, 0, 0, 0, 0, 1, 1, 1, 0, 1, 0, 1, 0]
target = [1, 1, 0, 0, 1, 1, 1, 1, 0, 0, 0, 0, 1, 1, 1, 0, 1, 0, 1, 0]

# X and y both DataFrames.
# X uses column names with an underscore
X = pd.DataFrame({'COL_1':col_1,
                  'COL_2':col_2,
                  'COL_3':col_3})
y = pd.DataFrame({'TARGET':target})

# Testing different types
# All three can be 'int8', 'float16', 'uint32' etc.
X['COL_1'] = X['COL_1'].astype('uint8')
X['COL_2'] = X['COL_2'].astype('float32')
X['COL_3'] = X['COL_3'].astype('bool')

# Testing a vary strange column name
strange_column_name = 'v3Ry_$t@ng3 N@m3'
X.rename(columns={'COL_1':strange_column_name}, inplace=True)

X_train = X[0:15]
X_test = X[15:20]

# Can be a numpy.ndarray when using y_train = y[0:15].values
y_train = y[0:15]

param_dist = {'objective':'binary:logistic', 'n_estimators':2}

bst = xgb.XGBModel(**param_dist)
bst.fit(X_train, y_train)
y_pred = bst.predict(X_test)

print(y_pred)

Thanks. It would be better if you can add the unit test to the package instead of putting it as an example above. Also, perhaps refactor the same list of dtypes that appear repeatedly?
cc: @tqchen

@terrytangyuan I simplified the code, so there is one dict left. Where do you want the example? In the guide-python folder?

@JohanManders Thanks! You can put it [here] (https://github.com/dmlc/xgboost/blob/master/tests/python/test_basic.py). Add your test example in test_feature_names or create a separate function if you like. It would also be good if you could squash your commits a little bit to make the commit messages cleaner and more meaningful.

@terrytangyuan

Sorry for not responding sooner. I have been trying to squash the comments, but whatever I try I keep getting an error that 541580d could not be applied. Maybe you have a suggestion?
I think the two modified functions test_dmatrix_init and test_pandas are sufficient for testing the modifications.

@JohanManders Thanks for the clarification. Merged.


cleaning and minimize the includes

This seems to do more than just cleanup extraneous includes. We shouldn't remove includes because it still happens to compile. Each file should include what is uses.


SmartMeteringInstallation: operation to remove smart meter is missing

SmartMeteringInstallation IF contains operation to add a device (AddDeviceRequest), but operation to remove it is missing.
AdminDeviceManagement IF contains RemoveDevice (RemoveDeviceRequest) operation, but this operation will not remove data from osgp_adapter_protocol_dlms database.dlms_device table.

Our apologies for the late response. The operation to remove a device is indeed missing in SmartMetering. At the moment we don't have a need for this operation. So it's not on our backlog yet, but it might be added in the future. Please let us know if you still need this operation and if so, are you gonna add it yourself or will you wait for a future change from our side?


fix ben-eb/cssnano/448 don't remove fallbacks for custom css properties

@ai could you check, please.
Thanks.
fixes #448

/cc @evilebottnawi

Can you add a test where the properties are mixed up to demonstrate its correctness? E.g.:
h1{
  padding-top:10px;
  padding-right:var(--variable);
  padding-right:15px;
  padding-bottom:var(--variable);
  padding-bottom:20px;
  padding-left:25px;
  padding-top:var(--variable);
  padding-left:var(--variable)
}

@andyjansson done

@karapetyan okay, looking at the final test, I don't think the implementation is correct. padding-right and padding-bottom should be overwritten by concrete values, while the rest should not.

@andyjansson can you explain what tests we should add more?

The correct minification should be something like:
h1{
  padding: 10px 15px 20px 25px;
  padding-left:var(--variable)
  padding-top:var(--variable);
}

@andyjansson previous implementation was overcoded and awful, thank you for review. What you think about new one?

I'll be off this weekend, so I won't be able to properly review this until Monday. But at first glance, it seems good.

@andyjansson :+1: i will close other problems for cssnano@4 and Monday after review we can merge this and maybe release rc.3, when two-three week and release stable 4 version

Alright, the implementation looks good. I was a bit confused by the odd ordering of the properties in the tests until I realised I was looking at integration tests (the reordering is due to css-declaration-sorter).
I'd like some unit tests where we test the plugin in isolation as well, if you don't mind (under /packages/postcss-merge-longhand/src/__tests__). The other thing that caught my eye was isCustomProp which could possibly be written so that false-positives are less likely to happen.
Apart from that, it looks really good. Good job!

@andyjansson @evilebottnawi great review, thank you guys. It was review driven development, actually)

/cc @andyjansson

@karapetyan Thanks!

/home/stanislav/projects/cssnano/packages/postcss-merge-longhand/src/__tests__/borders.js
  316:5  error  'processCss' is not defined  no-undef
  323:5  error  'processCss' is not defined  no-undef
  330:5  error  'processCss' is not defined  no-undef

@karapetyan @evilebottnawi
I actually came for someone else, I think it's necessary to fix it, but not in my PR

@YozhikM PR welcome :+1:


Missing FOD location for the Windows 10 20H2 MSIX Drivers

Article outlines the location for the MSIX Drivers for previous versions of Windows, but not for the MSIX Driver on Windows 10 20H2.

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

ID: 39e02e6d-02b2-fc96-2ef2-ae27d1d60ae8
Version Independent ID: ce70da72-0526-53e1-f005-9f64641ed286
Content: Using the MSIX Packaging Tool in a disconnected environment - MSIX
Content Source: msix-src/packaging-tool/disconnected-environment.md
Product: msix
GitHub Login: @sharlaakers
Microsoft Alias: shakers

Documentation has been updated.


Added a new command "deleteCategory"

Added a new command "deleteCategory", allowing the user to delete a category of recorded person.
Example: deleteCategory 1 c/Email

LGTM! Nice job bro
like overall its so good bro idk
like seriously this code is so good with no bugs at all
is this enough words prof

Overall looks good to me. It achieves the desired features. Thanks for the implementation!

The code quality is good, easy to read and straightforward. Do rmb to add in javaDoc :)

Overall great code. Nice implementation of delete command. Awesome job 😄
However, try to give some description of what your code does for us to understand easily.
Nice choice of making a new class and using good coding standards like abstraction to make your code easier to understand.
I will need to do the same as you hahaha 😆.


point density feature extractor

ask Zsofi for the exact explanation
Number of points within cylinder normalized by the area.
All feature extractors need to implement the feature extractor interface from #19.

The extractor needs to use the feature extractor abstract base class.


Create script for evaluating pyhms on different benchmark problems

I created new repo and added BBOB problems: https://github.com/WojtAcht/pyhms_evaluator


cephfs: GetCephFSNetNamespaceFilePath fails if config.json is missing

Describe the bug
GetCephFSNetNamespaceFilePath fails if config.json (supplied by ceph-csi-config ConfigMap) is missing. Specifically, the code shown below expects the config to be there:
https://github.com/ceph/ceph-csi/blob/2655d2edca323c94d25bd64b44306d7997921fe0/internal/cephfs/nodeserver.go#L129-L134
https://github.com/ceph/ceph-csi/blob/2655d2edca323c94d25bd64b44306d7997921fe0/internal/util/csiconfig.go#L188-L196
In deployments where the config is missing, it makes mounting volumes impossible. NodeStageVolume always fails.
This may be true for rbd-csi too.
Steps to reproduce
apiVersion: v1
kind: PersistentVolume
metadata:
  name: my-cephfs-pv
spec:
  accessModes:
  - ReadWriteMany
  capacity:
    storage: 1Gi
  csi:
    driver: cephfs.csi.ceph.com
    volumeHandle: my-cephfs-volume
    nodeStageSecretRef:
      name: csi-cephfs-secret
      namespace: default
    volumeAttributes:
      provisionVolume: "false"
      monitors: "..."
      rootPath: "..."

      # clusterID is not specified. We're passing monitor IPs directly with `monitors` parameter.

---
apiVersion: v1
kind: PersistentVolumeClaim
metadata:
  name: my-cephfs-pvc
spec:
  accessModes:
  - ReadWriteMany
  resources:
    requests:
      storage: 1Gi
  volumeName: my-cephfs-pv
  storageClassName: ""
---
apiVersion: v1
kind: Pod
metadata:
  name: my-cephfs-pod
spec:
  containers:
    - name: web-server
      image: nginx
      imagePullPolicy: IfNotPresent
      volumeMounts:
        - name: mypvc
          mountPath: /var/lib/www
  volumes:
    - name: mypvc
      persistentVolumeClaim:
        claimName: my-cephfs-pvc
        readOnly: false

Actual results
Mounting the volume fails with:
E0515 08:52:55.398271       1 utils.go:200] ID: 10 Req-ID: ... GRPC error: rpc error: code = Internal desc = missing configuration for cluster ID ""

Expected behavior
Mounting should have succeeded despite the missing config.
Additional context

netNamespaceFilePath should be treated as an optional parameter. Skip it silently, if it cannot be retrieved.
I'm wondering why is this even part of the per-cluster config? Is the net ns file path expected to be different across different cluster configurations? Maybe we could pass it to the driver via cmd args / env variable too?

@gman0 this is only affected for the pre-provisioned cephfs volumes (a special case that doesn't have clusterID in volume context) not for the static or dynamic provisioned volume. #3154 should fix it. can you please check


Fixes #1123 - Monitor log recovery says invalid date - Temporary Fix

I have looked around the code trying to see what is causing this in the REST service and the underlying solution of why a bad value is being passed into the RecoveryStatus.runtime value is not that apparent. I am putting out this option that for the present we can just set the date to "Date.now" which is what the Date constructor gives you (only when the date is 'Invalid Date') .   I will work on a better solution to this issue.

I don't think it's a good idea to mask the error with the current date. It could be misleading.


Fix rows fix

It looks like the + and * were swapped in shift_rows and inv_shift_rows as the multiply needs to go on column size to get current row in array.

Note the way in which the 16 input bytes are represented in the AES matrix—they go down the columns, not across the rows.

Yeah, I realised that too late.  sorry.


Panel blocks content on the right

I know this is referenced in #6, but it's probably easier to deal with as an isolated issue.
It's sensible to do the tree vertically since it's large and vertically-oriented, but blocking content prevents me from using my app with this enabled, which is a bummer.
You could add a button to switch sides, but why cover anything? Could this work: Just pad in the body enough to give your panel some room.
document.getElementsByTagName('body')[0].style.paddingRight = '290px'

This is partly solved in PR #74. It does not cover the mobile emulation use-case, or even all websites I've tested, but it does improve usability on the majority of tested sites.


Repair create command

Coverage remained the same at 100.0% when pulling f75bb61d04c774d9e70295c701b0504214acd027 on repair-api-create into 87bc7d5bbc26cffe877f32398891eb9640d9b8ce on master.


schema_design

Hi.
users table is not found.

Hi @shadowd-dev,
Below is the table structure for above issue -
CREATE TABLE users
(
user_id text NOT NULL DEFAULT uuid_generate_v1(),
username character varying(25) NOT NULL,
password character varying(50) NOT NULL,
email character varying(100) NOT NULL,
CONSTRAINT users_pkey PRIMARY KEY (user_id),
CONSTRAINT users_username_key UNIQUE (username)
)
I have also updated in schema_design file.
Thanks


flutter/dart patched sdks should be bundled independent of aot/jit

https://bugs.fuchsia.dev/p/fuchsia/issues/detail?id=53418

This is needed to reduce the size of the fuchsia checkout. We should look at removing any duplicate build artifacts from being uploaded to CIPD.

Being tracked in upstream issue


Support fractional seconds

Description
Add a f64 variant to Timestamp so that it supports fractional seconds
Related Issue
Fixes #110
Testing
Tested encoding and decoding in a unit test
Checklist:

[X] I have read and followed the CONTRIBUTING guidelines for this project.
[X] I have added or updated tests and they pass.
[X] I have added or updated documentation and it is accurate.
[X] I have noted any breaking changes in this module or downstream modules.

Closed per discussion with @hansl


Request for Data Usage Permission

Hi, excellent work! I am interested in using the data from this repository Huggingface for academic research purposes. However, I noticed that there is no license information provided. Could you please clarify whether the data in this repository is available for reuse, and if so, under what terms and conditions?
Thank you for your assistance in clarifying the usage rights for this data.
Sincerely,

Hey, thanks for reaching out and sorry I forgot to fill in the license info. The dataset is 100% derived from LibriLight and we are releasing the processed data under the same terms – Public Domain.
The Encodec acoustic tokens are as good as they’ll get. For the semantic tokens, we are working on improving them right now so we’ll likely update them in a week or two.
If you use it we would love if you could give us a boost by pointing at this repo but that’s not a requirement.


How do I get the error text from Midjourney?

How do I get the error text? I can't find all the messages that are parsed from the discord - the script doesn't seem to see these messages... Very strange. Sorry for bad English

In discord it calls ephemeral messages - special message mode, most of system messages are send as ephemeral. Unfortunately it's no way to get it by API

What about captcha messages?


Snippet data in JSON

Hey :wave:,
I'm the maintainer of both https://github.com/sQVe/30s and https://github.com/sQVe/30-seconds-of-code-ultisnips, which are not unable to update due to the lack of snippet_data/snippets.json.
Do you have any recommendations on how custom clients could consume the snippets provided here? It would be a shame if the previously provided JSON format was removed.
:pray:

Unfortunately we have moved most of the logic and systems to our integration pipeline as the main objective is to be able to publish and distribute code snippets via the website and this sped up development on that immensely. I'm afraid that if you want to extract JSON data you would have to write your own extraction logic and make sure that everything is up to date whenever we change formats in the future. I know this is probably a bummer, but it made a lot of things possible, which is why we ultimately set up a centralized pipeline instead of many smaller ones to deal with the issue.

@Chalarangelo Alright. I can see the reasoning behind this but it excludes a alternative ecosystem. It also deprecates the current npm package.

The npm package is discontinued due to the fact that it was technically never updated and that we cannot maintain consistent enough performance and quality so that we are both a learning resource and a package, so it didn't make the cut. It was, like many other things, an experiment that failed to keep up due to lack of interest and manpower.
We are more focused on the website, as a narrower scope and not trying to do too many things at once will produce higher quality results. I agree that some features were nice, but unfortunately we only have so many resources and we want to keep the core experience - that is providing high quality explanatory content - intact and ensure it's always our top priority.

Good delete my duplicates accounts.


How to train custom data after creating mdb files.

I finally create data.mdb and lock.mdb with this command:
!python '/content/drive/My Drive/deep-text-recognition-benchmark-1/create_lmdb_dataset.py' --inputPath '/content/drive/My Drive/deep-text-recognition-benchmark-1/train' --gtFile '/content/drive/My Drive/deep-text-recognition-benchmark-1/gt.txt' --outputPath '/content/drive/My Drive/deep-text-recognition-benchmark-1/result'
Now I have to use another command to start training:
!python '/content/drive/My Drive/deep-text-recognition-benchmark-1/train.py' --train_data '/content/deep-text-recognition-benchmark-1/train' --valid_data '/content/deep-text-recognition-benchmark-1/val' --Transformation TPS --FeatureExtraction ResNet --SequenceModeling BiLSTM --Prediction Attn --select_data '/content/drive/My Drive/deep-text-recognition-benchmark-1/result'
But if I use this command the output will be:
Filtering the images containing characters which are not in opt.character
Filtering the images whose label is longer than opt.batch_max_length
--------------------------------------------------------------------------------
dataset_root: /content/deep-text-recognition-benchmark-1/train
opt.select_data: ['/content/drive/My Drive/deep', 'text', 'recognition', 'benchmark', '1/result']
opt.batch_ratio: ['1']
Traceback (most recent call last):
  File "/content/drive/My Drive/deep-text-recognition-benchmark-1/train.py", line 315, in <module>
    train(opt)
  File "/content/drive/My Drive/deep-text-recognition-benchmark-1/train.py", line 31, in train
    train_dataset = Batch_Balanced_Dataset(opt)
  File "/content/drive/My Drive/deep-text-recognition-benchmark-1/dataset.py", line 31, in __init__
    assert len(opt.select_data) == len(opt.batch_ratio)
AssertionError

And if I use this command without --select_data like this:
!python '/content/drive/My Drive/deep-text-recognition-benchmark-1/train.py' --train_data '/content/deep-text-recognition-benchmark-1/train' --valid_data '/content/deep-text-recognition-benchmark-1/val' --Transformation TPS --FeatureExtraction ResNet --SequenceModeling BiLSTM --Prediction Attn
The output will be:
Filtering the images containing characters which are not in opt.character
Filtering the images whose label is longer than opt.batch_max_length
--------------------------------------------------------------------------------
dataset_root: /content/deep-text-recognition-benchmark-1/train
opt.select_data: ['/']
opt.batch_ratio: ['1']
--------------------------------------------------------------------------------
dataset_root:    /content/deep-text-recognition-benchmark-1/train	 dataset: /
Traceback (most recent call last):
  File "/content/drive/My Drive/deep-text-recognition-benchmark-1/train.py", line 315, in <module>
    train(opt)
  File "/content/drive/My Drive/deep-text-recognition-benchmark-1/train.py", line 31, in train
    train_dataset = Batch_Balanced_Dataset(opt)
  File "/content/drive/My Drive/deep-text-recognition-benchmark-1/dataset.py", line 42, in __init__
    _dataset, _dataset_log = hierarchical_dataset(root=opt.train_data, opt=opt, select_data=[selected_d])
  File "/content/drive/My Drive/deep-text-recognition-benchmark-1/dataset.py", line 124, in hierarchical_dataset
    concatenated_dataset = ConcatDataset(dataset_list)
  File "/usr/local/lib/python3.6/dist-packages/torch/utils/data/dataset.py", line 188, in __init__
    assert len(datasets) > 0, 'datasets should not be an empty iterable'
AssertionError: datasets should not be an empty iterable

There is a train folder with 6 images.
There is a val folder but it's empty, is it ok? also have made things from this link What I'm doing wrong??

Sorry for the late reply,
I reply to https://github.com/clovaai/deep-text-recognition-benchmark/issues/172#issuecomment-639380782
Please check and if the problem was not solved, comment on that issue.
I will close this issue since it seems duplicate.
Best

Thanks for your reply,
opt.select_data: ['/'] means that in '/' will be the .mdb files?
Then I got the .mdb files (data.mdb and lock.mdb)
opt.select_data refers to those files?

opt.select_data: ['/'] means 'use all dataset under --train_data folder'.
default is opt.select_data: MJ-ST which means 'use MJ and ST datasets under --train_data folder'.
You can check the structure of the --train_data folder in here.
data_lmdb_release
└──training
      └──MJ
          └── MJ_train
              ├── data.mdb
              └── lock.mdb
          └── MJ_valid
              ├── data.mdb
              └── lock.mdb
          └── MJ_test
              ├── data.mdb
              └── lock.mdb

Hi, I have created same structure as discussed above:
data_lmdb_release
└──training
└──data
└── data_train
├── data.mdb
└── lock.mdb
└── data_valid
├── data.mdb
└── lock.mdb
└──data_test
├── data.mdb
└── lock.mdb
Using training command as :
python train.py --train_data data_lmdb_release/training/ data/data_train --valid_data data_lmdb_release/training/data/data_valid --select_data data --batch_ratio 0.5-0.5 --Transformation TPS --FeatureExtraction ResNet --SequenceModeling BiLSTM --Prediction Attn
But getting error:
/mnt/vol1/testing_anaconda3/envs/test_TR/lib/python3.6/site-packages/torch/cuda/init.py:52: UserWarning: CUDA initialization: The NVIDIA driver on your system is too old (found version 10010). Please update your GPU driver by downloading and installing a new version from the URL: http://www.nvidia.com/Download/index.aspx Alternatively, go to: https://pytorch.org to install a PyTorch version that has been compiled with your version of the CUDA driver. (Triggered internally at  /pytorch/c10/cuda/CUDAFunctions.cpp:100.)
return torch._C._cuda_getDeviceCount() > 0
Filtering the images containing characters which are not in opt.character
Filtering the images whose label is longer than opt.batch_max_length
dataset_root: data_lmdb_release/training/data/data_train
opt.select_data: ['data']
opt.batch_ratio: ['0.5', '0.5']
Traceback (most recent call last):
File "train.py", line 317, in
train(opt)
File "train.py", line 31, in train
train_dataset = Batch_Balanced_Dataset(opt)
File "/mnt/vol1/sakshi/deep-text-recognition-benchmark-master/dataset.py", line 31, in init
assert len(opt.select_data) == len(opt.batch_ratio)
AssertionError
please suggest

To comment on the original question:
I ran into the same problem and found out while debugging that filtering of my training data resulted into nSamples = 0.
Either you change the batch_max_length to a length lower than your minimal data length and so on.. Or you just add  --data_filtering_off to your command.

@chirico85  Thank you so much. this solved my issue


Class getting stripped out unnecessarily

I have this, roughly—
let computedClasses = cls(
  'inline-block text-base text-left font-semibold leading-tight',
  'text-xl'
);

console.log(computedClasses); // -> 'inline-block text-left font-semibold text-xl'

I can't figure out why leading-tight is getting removed.
Here is my implementation of cls:
import clsx, { type ClassValue } from 'clsx';
import { extendTailwindMerge, validators } from 'tailwind-merge';
import { spreadKeys } from '@carevoyance/calculations';

const trbl = ['t', 'r', 'b', 'l'];
const xy = ['x', 'y'];
const dirs = { trbl, xy, all: [...trbl, ...xy] };

const twMerge = extendTailwindMerge({
  classGroups: {
    shadow: [{
      shadow: [{
        ...spreadKeys(dirs.trbl, ['', 'inner', validators.isTshirtSize]),
        border: [
          '',
          validators.isNumber,
          validators.isArbitraryLength,
          spreadKeys(dirs.all, [validators.isNumber, validators.isArbitraryLength]),
        ],
      }],
    }],
  },
});

export const cls = (...inputs: ClassValue[]) => twMerge(clsx(...inputs));

Thanks for all the help, as always 🙏🏼

Hey @brandonmcconnell! 👋
This is because text-xl also sets line-height (line-height: 1.75rem; to be exact). If you want to preserve leading-tight, you need to set it after text-xl.
It's similar to how px-4 p-5 gets reduced to p-5 because you want to override the paddings set by px-4 but p-5 px-4 stays intact because tailwind-merge sees that as a refinement to p-5.

But I know that this is confusing. There were already several issues about this exact problem. Not many people know that Tailwind CSS is doing this. 🤔

@dcastil Thanks for clarifying that! I believe—though I could be wrong—if you have both a font-size utility like text-xs and a line-height utility like leading-tight, leading-_ will override the line-height regardless of the order (in pure TailwindCSS).
It would be great if tailwind-merge offered a setting to allow utilities like these and those you mentioned to not be considered conflicts and to allow TailwindCSS to handle their overrides naturally.

Also, if they're considered conflicts and I set leading-x after text-x, does that mean that leading would be prioritized and the text utility would be removed instead, making it difficult/impossible to have both?

If you put leading-x after text-x, both stay in there since you want to override the line-height of text-x but keep its font-size. The override happens through the CSS order in that case.
But e.g. think of a component that defines a line-height with leading-x inside and then you want to do <MyComponent className="text-xl" /> (which then results in twMerge('… leading-x', props.className)). The dev might want to override leading-x here with the line-height in text-xl, so tailwind-merge needs to remove that.
But you can fix this by the way! You need to remove a conflict from the config (this one).
const twMerge = extendTailwindMerge({ /* your config */ }, config => {
    delete config.conflictingClassGroups['font-size']
    return config
})

@dcastil Thanks!

@brandonmcconnell I'm closing this issue as resolved. Let me know if your issue persists. 😊

@dcastil Just wanted to confirm— using delete config.conflictingClassGroups['font-size'] will still remove conflicting font-size utilities of the same type, correct? For exmaple, twMerge('text-lg text-sm') would still compile to text-sm, right?

Yes! config.conflictingClassGroups['font-size'] only holds the information that font-size classes clash with line-height classes. Classes from the same group always conflict with each other.

@dcastil Awesome, thanks!

Mentioned this issue in https://github.com/dcastil/tailwind-merge/discussions/226#discussioncomment-5917498.

Simple transformer which spits something out on the console (process.env.NODE_ENV === "development" only) when a class is dropped.
import clsx, { type ClassValue } from "clsx";
import { twMerge } from "tailwind-merge";

const getCaller = () => {
  const getErrorObject = () => {
    try {
      throw Error("");
    } catch (err) {
      return err as Error;
    }
  };

  const err = getErrorObject();
  const caller_line = err?.stack?.split("\n")[4];
  const clean = caller_line?.trim().replace(/^at /, "from ");
  return clean;
};

const missingWords = (before: string, after: string) => {
  const afterArr = after.split(" ");
  return before
    .split(" ")
    .filter((w) => !afterArr.includes(w))
    .join(" ");
};

const twM = (...inputs: ClassValue[]) => {
  const joinedClasses = clsx(...inputs);
  const mergedClasses = twMerge(joinedClasses);
  if (
    process.env.NODE_ENV === "development" &&
    joinedClasses !== mergedClasses
  ) {
    console.log(
      `twM: "${joinedClasses}" lost "${missingWords(
        joinedClasses,
        mergedClasses
      )}"
      ${getCaller() || ""}`
    );
  }

  return mergedClasses;
};

export default twM;

Sample output:
twM: "z-10 leading-relaxed text-9xl font-bold uppercase" lost "leading-relaxed"
      from Home (webpack-internal:///./src/pages/index.tsx:72:111)


MB-9669: Ensure auth API calls are never cached

Description
Looking at the logs from the most recent occurrence (see the comments in jira), there doesn't seem to be a log for the GET /auth/login-gov before the redirect back to /auth/login-gov/callback. This is a bit of a shot in the dark, but let's make sure none of the API calls is ever cached.
Reviewer Notes
Not positive this will fix the bug, but it also seems like the right thing to do regardless. We install this middleware for all our other API endpoints.
I deployed this on the load test environment

Messages

:book:
:link: MB-9669

  Generated by :no_entry_sign: dangerJS against 8745011eea526b445468dc055e225d9f0af2f483

I am curious where we can potentially remove the no cache directive elsewhere. I think there's a security requirement around not caching PII but I could be wrong.

I don't think there's any PII in these HTTP requests, but I think, in theory, a client could currently cache the GET /auth/login-gov. I'm not positive about that, though

I am curious where we can potentially remove the no cache directive elsewhere. I think there's a security requirement around not caching PII but I could be wrong.

I don't think there's any PII in these HTTP requests, but I think, in theory, a client could currently cache the GET /auth/login-gov. I'm not positive about that, though

I agree it makes sense for the auth to never be cached and was trying to make heads or tails of why it may exist elsewhere where it could benefit our users to cache some things.

I am curious where we can potentially remove the no cache directive elsewhere. I think there's a security requirement around not caching PII but I could be wrong.

I don't think there's any PII in these HTTP requests, but I think, in theory, a client could currently cache the GET /auth/login-gov. I'm not positive about that, though

I agree it makes sense for the auth to never be cached and was trying to make heads or tails of why it may exist elsewhere where it could benefit our users to cache some things.

Yeah, a good cache strategy could be a way to reduce load on the server, but wow, I think it's complicated and challenging to do right. It's probably farther down our list of priorities. :grinning:

LGTM. At this point we use it almost everywhere. I would say we should just apply it to all using root (or maybe site, tbh I'm not sure how those two interact). But we don't have it enabled for static, and that's actually probably a good place to allow cache use so the way we're doing it now might be best.

Yeah, I didn't want to get too aggressive. I'd love to redo our routes anyway to be less confusing. Maybe someday.


[Color Picker] Pantone names

Description of the new feature / enhancement
When using the colour picker it would be nice if it showed the closest pantone colour that matches.
Scenario when this would be used?
Artists that want to get an specific pantone colour out of an image to inspire their designs.
Supporting information
Having the pantone colour show directly in powertoys would make productivity for colour accurate work to be a lot faster so we have to resort to an 3rd party table which is not guaranteed to have all the colours.

The Pantone name is a trademark and requires permission/license to use.  In fact, it's being removed from Adobe Photoshop. If Photoshop cannot get the rights to use Pantone names, I highly doubt that PowerToys can.

The Pantone name is a trademark and requires permission/license to use. In fact, it's being removed from Adobe Photoshop. If Photoshop cannot get the rights to use Pantone names, I highly doubt that PowerToys can.

That is a big bummer... i hope powertoys can get this licence in the future.

i don't think we'd be able to do this.  "closeness" is hard which is why our color names when enabled are pretty broad.  I don't think we'd implement this due to need of license and how that would interact with the open source nature.  If someone from Pantone would like to do this partnership, happy to have that discussion.


Practice Mode is Broken

if you place a checkpoint while holding and then release mouse and restart it still continues holding even when im not
its really annoying in robot wave and ship

I was about to make an issue about this. I've noticed the same thing.

Disable Practice Fix in Replay Engine settings, this fix is designed for gd bot to keep the macro from breaking

There are definitely some bugs with it even in macros

it is not broken, it is a feature


chore(deps): upgrade dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-1.x" workflow

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

Questions ?
Please refer to the Backport tool documentation


mquantiles for numpy.matrix input returning first entries of matrix

The function mquantiles in scipy.stats.mstats does not throw an error when the input a is of type numpy.matrix. However, instead of calculating the quantiles, the method returns the first len(prob) values of the sorted matrix.

I don't think think this will be fixed. 8 years without comment and both np.matrix and mstats are on their way out.
Would you agree @tirthasheshpatel?

Yup, np.matrix is long outdated and mstats is being replaced by _axis_nan_policy_factory. So, +1 for closing such issues.


Option to use Code's Insider version

Implement an option (a switch) in the settings to use Code's Insider version instead of the usual.
Therefore change the path to:
var newPath = ProgramFilesx86() + Path.DirectorySeparatorChar + "Microsoft VS Code Insiders" + Path.DirectorySeparatorChar + "bin" + Path.DirectorySeparatorChar + "code-insiders.cmd";
And refresh the "VSCode_CodePath" Editor prefs.
That would be great if you can implement that feature.

Why not manually set the path to that :) (in preferences)


Feature request for "textlint" config in package.json

The https://textlint.github.io/docs/configuring.html docs show that textlint can be configured with various types of .textlintrc* files. It would be nice if a textlint config could be used in the package.json as well to keep configuration contained to package.json for users that like to do that. For example, Prettier has the prettier key as well as all the .prettierrc* options. This seems to be common for many other npm packages (ESLint, lint-staged, husky, etc). using cosmiconfig which supports all these different configurations automatically.

Yes.
https://github.com/azu/rc-config-loader#api also support packageJSON option.

📝 I've noticed that rc-config-loader's type definition is wrong.
This fixes is released as patch https://github.com/azu/rc-config-loader/releases/tag/2.0.5

@azu Cool! I've submitted a PR for it: https://github.com/textlint/textlint/pull/644

I've released https://github.com/textlint/textlint/releases/tag/textlint%4011.5.0


Add animation for background

Our current background on all pages but the chapter pages contains four colored lines (a static SVG). The new oceanprotocol.com page contains the same lines but they are animated (changing amplitude).
I suggest the same or similar animation for the Ocean Academy. Animations add to the experience of an "interactive experience" that we want to convey.

I absolutely love it. I guess the OCEAN team would love to have it too. ;)

Merged with https://github.com/learn-ocean/ocean-academy/pull/33


Android Studio Image is incorrect in documentation

The documentation that describes how to download/compile Android app is incorrect.
It shows the wrong line number in the image, but shows the correct string.
This is confusing.

Can you provide more details?  which document, which page, and which image is incorrect?
What should it read?

I added some clarity in that document in commit e07f710 which should address your issue.  Let me know if anybody still finds it unclear.


Conflict resolution

Removed the logic for conflict resolution because I realized simply hydrating the conflicting commands on a fresh stream and retrying the commit is a pretty bad policy.
The correct thing to do is to re-run the business logic executed by the command.
To do that off the top of my head we'd have to re-run the command's handle method for the specific instance that changed the stream.. which is just not possible to do correctly either.
Will have to look at other resolution strategies.
I guess the best way would be to let the user define his own custom conflict resolvers so he can explicitly define when a version conflict can be accepted and when it can't

ref d3a6d1cc47de74e542d3cd69e6c224839f68057c and 0a8b283053e8171a456928c8ed635f45ce46671a

Better conflict resolution has been implemented - new system supports a range of different options, can be found here https://github.com/volak/Aggregates.NET/blob/master/src/Aggregates.NET.Domain/Internal/ConflictResolvers.cs


Respond with 200 Ok on successful connection

Otherwise the browser will wait forever.

Why do you need to write it yourself? The remote site would respond for you, won't it?

A CONNECT request works as follows:
Client:
CONNECT target:80 HTTP/1.0

Proxy:
HTTP/1.0 200 Ok

Client:
GET / HTTP/1.0
Host: target

Target:
HTTP/1.0 200 Ok

If the proxy doesn't respond with the initial 200 Ok, the client will never issue the subsequent request to the remote site.


Log directories being skipped on Windows

Hi, I found a pass in main function of planet.scripts.train. The code running in Win 8.1 with python 3.5. The result like as follows. Am I running in a worry way ? Thanks
def main(args):
  training.utility.set_up_logging()
  experiment = training.Experiment(
      args.logdir,
      process_fn=functools.partial(process, args=args),
      start_fn=functools.partial(start, args=args),
      resume_fn=functools.partial(resume, args=args),
      num_runs=args.num_runs,
      ping_every=args.ping_every,
      resume_runs=args.resume_runs)
  for run in experiment:
    for unused_score in run:
      pass

Hey @cfzing, the code wasn't tested on Windows and it might not run. I suggest you change train.py to not use the Experiment class and instead call start() and process() directly from the main function:
def main(args):
  config = start(args.logdir, args)
  for score in process(logdir, configs, args):
    pass

This way, you remove the feature of running a queue of multiple experiments with a smaller number of machines, but you are circumventing code that could potentially cause problems. Note if you want to continue experiments after stopping them, you could call resume() instead of start().

Closing this due to inactivity.


✨ Easy grid systems?

Description
Will we have a predefined grid systems based on variable number of --columns (both flex and grid), like these ones:

Tailwind [1] [2] / uno
960.gs
Flexbox grid
Susy

Master CSS's grid system is arbitrary and more flexible than the traditional 12-column, 16-column, or 24-column.

<div class="grid-cols:5 grid-cols:6@sm grid-cols:8@md grid-cols:16@lg gap:5x<EMAIL_ADDRESS>

    <div class="w:1/4<EMAIL_ADDRESS>    <div class="w:1/4<EMAIL_ADDRESS>    <div class="w:2/4<EMAIL_ADDRESS>

The documentation of the grid system has indeed been missing since v1. We will add a new page docs/grid-system during the 2.0 RC.
https://css.master.co/docs/grid-columns


Import Format is now automatically being fixed with no way to disable or allow mixed imports

Environment data

Language Server version: v2022.8.40
OS and version: MacOs 10.15.7 (19H1715)
Python version (& distribution if applicable, e.g. Anaconda): 3.9.10

Code Snippet

from engine import views
from . import blueprint as api

Repro Steps

Create python file with above imports
Save

Expected behavior
Relative import not automatically changed to absolute
Or allow python.analysis.importFormat to be set to both
Or allow turning this feature off
Actual behavior
relative import is being changed to engine.server.v1
Logs

XXX

I think you have codeaction on save enabled with source.fixAll: true.
you should be able to turn it off using source.fixAll.convertImportFormat: false
see here for more detail - https://github.com/microsoft/pylance-release#source-code-actions


Build: Upgrade Promu to latest

Is your proposal related to a problem?
Relates to #4988 but it's also just nice to have up-to-date packages. Currently we run 0.5, 0.13 is out.
It relates to that issue, because release 0.11 included parallelism: https://github.com/prometheus/promu/releases/tag/v0.11.0
At the moment I also tried implementing this, but I guess it hits some form of CGO / non-CGO quirks. Causing builds to fail with promu @0.13
Describe the solution you'd like
We are able to run Promu on a recent / latest release and therefor open the possibility for parallelism

The thing I'm stuck on at the moment; https://github.com/prometheus/promu/issues/228


Troubles while including template that uses extends in loops

Hi,
I've got some troubles when using include in for loops. The included files do not "view" the overrided {% block %}.
I mention that I'm using precompiled templates.
Is it a bug or am I wrong ?
Exemple :
File A.twig
{% block myBlock %}Hello A{% endblock %}

File B.twig
{% extends 'A.twig' %}
{% block myBlock %}Hello B{{ i }}{% endblock %}

File Loop.twig
{% for i in [1, 2, 3] %}
    {% include 'B.twig' %}
{% endfor %}
{% include 'B.twig' with { 'i':4 } %}

Renders :
Hello A
Hello A
Hello A
Hello B4

+1 same issue on my project, did you find a fix or hack to by-pass the bug ?
Thx

I also just encountered this problem. I thought I was screwing something up for the longest time

Okay, so in twig.logic.js:
"Block Logic Tokens"
In parse, it says don't override previous blocks unless they're imported with "use", and loops should be exempted as well (comments on lines 6052 and 6053 of the full webpack compiled twig.js file)
So on line 6054, "context.loop" checked whether or not it is a loop. Deleting this fixes the functionality we are searching for.
But my question is -- what unintended side effects will this have? Is Twig for PHP not intended to have loops around included extended files (I haven't checked)? Is this going to cause unforeseen problems with loops within extended files or something?

Okay so it seems when the for loop is called in the parent file, context.loop is set to true for when you enter the included child file... the intended effect is that if you have a block within a for loop it will overwrite the block output when you get to the next iteration, e.g.
{% for i in [1, 2, 3] %}
    {% block loop_item %}{{ i }}{% endblock %}
{% endfor %}

If you got rid of the context.loop, it will only write 1 1 1
But as it stands, the context.loop being set to true by the initial outside-file for-loop causes any blocks to pass the check of context.loop, which causes it to always print the parent loop, since it thinks the block is already set they just have to grab the block which it already has...

I attempted to delve into the context structure but realized I wasn't strong enough in Twig to know the necessities of what changing things would do -- i.e. if I had a parent -> child1 -> child2 -> use multiple files / for loops within those files or includes or whatever, how things would propagate context-wise (at least throughout the Javascript file).
The problem lies in an included file's context knowing it is in a loop.
From twig.core.js
    Twig.ChildContext = function(context) {
        var ChildContext = function ChildContext() {};
        ChildContext.prototype = context;
        return new ChildContext();
    };

Which is called in the include/extend/for block logic tokens in twig.logic.js
A workaround for now is not including a context to the child, which can be done by using with...only. For example, in your code above:
File Loop.twig
{% for i in [1, 2, 3] %}
    {% include 'B.twig' with { 'i': i } only %}
{% endfor %}
{% include 'B.twig' with { 'i':4 } %}

Renders
Hello B1
Hello B2
Hello B3
Hello B4
This fix is simple enough for me and works in my case. I hope it works in others' as well... I know it's silly writing it out like that but the child currently needs a local context to work with. If you want to bother with editing the actual library you can check out the Include block logic tokens for include and possibly expanding upon the ChildContext to let it know if it's the parent looping,..

This is definitely supposed to work:
https://twigfiddle.com/4jha45
Something is broken with twig.js support for blocks.

This is definitely supposed to work:
https://twigfiddle.com/4jha45
Something is broken with twig.js support for blocks.


Design QA - My bets tablet (1024 to 1200 breakpoint)

Last column content is touching table border.

Design:

looks fixed


pagination.page comes back as a string when ?page=1 is passed

Seems like this got fixed at some point


Same code works in Uno and not in Mega2560

Hello, I have something rather wierd.
I'm trying to join several arduinos in 1-wire network.
I wrote simple test bellow to check stuff - it just checks device presense.
It contains both code for master (BOARD_NUMBER=1) and slave(everything else).
Problem is, when I use Arduino Uno as a slave - it works fine.
However when I upload the same code to Arduino Mega 2560 and use it in the same setup master node doesn't see slave (slave initialized though - printed it's address in serial).
(Tried both with receive callback and without one. Tried both Uno and Mega as master)
Relevant part (quite basic though):
#include <OneWireSlave.h>
#define PIN 2
#define DEVICE_TYPE 0

byte address[8] = {};

void generate_address(){
    randomSeed(analogRead(A0));
    address[0] = DEVICE_TYPE;
    for (int i=0; i<7; i++){
        address[i] = random();
    }
    address[7] = OneWire::crc8( address, 7);
}

void setup_onewire(){
      //OWSlave.setReceiveCallback(&receive);
      OWSlave.begin(address, PIN);
 }

void setup() {
    Serial.begin(9600);
    generate_address();
    setup_onewire();
    Serial.print("One wire address: ");
    for( int i = 0; i < 8; i++) {
        Serial.print(address[i], HEX);
        Serial.print(" ");
    }
    Serial.println();
    pinMode(LED_BUILTIN, OUTPUT);
    digitalWrite(LED_BUILTIN, LOW);
}

Full code:
one-wire-test-code.txt

If someone makes it work on other boards that Arduino Uno (the only board I've tested), I'm interested to know ; but otherwise I can't help myself since I don't have the hardware to test, nor the motivation to work on it.
I leave the issue open so others can know which boards support are most requested.


Plus1 throws lighthouse DNS failure error.

[2020-05-22 16:55:31] ERROR: Error: Lighthouse returned error: DNS_FAILURE. DNS servers could not resolve the provided domain.
at Gaxios. (/gpsi/node_modules/gaxios/build/src/gaxios.js:72:27)
at Generator.next ()
at fulfilled (/gpsi/node_modules/gaxios/build/src/gaxios.js:16:58)
at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:97:5)
[2020-05-22 16:55:35] ERROR: Error: Lighthouse returned error: DNS_FAILURE. DNS servers could not resolve the provided domain.
at Gaxios. (/gpsi/node_modules/gaxios/build/src/gaxios.js:72:27)
at Generator.next ()
at fulfilled (/gpsi/node_modules/gaxios/build/src/gaxios.js:16:58)
at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:97:5)
[2020-05-22 16:55:38] ERROR: Error: Lighthouse returned error: DNS_FAILURE. DNS servers could not resolve the provided domain.
at Gaxios. (/gpsi/node_modules/gaxios/build/src/gaxios.js:72:27)
at Generator.next ()
at fulfilled (/gpsi/node_modules/gaxios/build/src/gaxios.js:16:58)
at runMicrotasks ()
at processTicksAndRejections (internal/process/task_queues.js:97:5)
[2020-05-22 16:55:48] INFO: https://www.bedbathandbeyond.com/?ioid=HH000660 TTFB: 246ms DOMContentLoaded: 2.44s firstPaint: 562ms FCP: 562ms LCP: 935ms Load:

Yes I got the same. Its from the GPSI server that runs Lighthouse that have some kind of problem.

I tested now and this works now.


_gen_field() method is defaulting to True value for 'null' parameter.

_gen_field() method is defaulting to True value for 'null' parameter. This causes a warning to be generated every time you try to create a field using default values.

Default value was replaced to None


cabal clean etc leave build-breaking artifacts

I just had to dig from Verifier down to determine an old abcBridge 0.13 tarball downloaded by abcBridge setup was holding up the compilation of SAWScript.  We should probably make a more effective cabal clean if possible.

We do have scripts/clean-abc.sh, and it should be straightforward to update Setup.hs to run it during cabal clean.

Tom: can you give more details on the error state you were experiencing?  At what point did the build break and what messages were you getting?
We could make cabal clean more aggressive, but I think it is a bad idea.  The same clean procedure should work for the released Hackage packages, which have the associated ABC sources bundled in; these released Hackage packages should never be fetching tarballs and such.  Maybe we could try to detect if we are building from git sources and have a more aggressive clean in that case?
I recognize the current situation is not ideal; I'm open to suggestions.

@robdockins I've lost the state of the system that errored but the message was:
Setting up ABC tree for abcBridge version 0.14-dev...

The ABC source version 0.13-dev does not match the abcBridge package version 0.14-dev.

This may cause problems with the build; it is recommended you cancel this build and check out matching ABC sources
into the abc-build directory and try again.  You can run scripts/clean_abc.sh to remove the current ABC sources;
this will cause latest development branch to be automatically downloaded on the next build.

Press [enter] to continue anyway, or press [^C] to abort (build will automatically abort in 15 seconds).

Strictly speaking, the old tarball's existence wasn't the issue.  I believe manually (or otherwise) deleting abc-build was the key step to get the build system to download the correct version of ABC.  Sorry this is less than exact.

@TomMD: Indeed.  I've been reluctant to make the build system auto-delete the abc-build directory, but maybe that is a better default in the version mismatch case (which should never occur for released Hackage packages).
@acfolzer: We've had a discussion on this exact topic in the past.  Have you had any additional thoughts since then?

@robdockins I understand the reluctance.  If this ticket falls into the NOTABUG or WONTFIX category I would understand.  The issue struck me as more significant earlier but I recognize that any regular user will acquire the code differently and without cruft of prior build artifacts.

After discussion with @atomb and @acfoltzer, we decided to implement the following change: during the configure phase if we detect ABC sources that do not match the abcBridge cabal package version, we delete the abc-build directory and try to fetch sources.  A version mismatch should never occur for released packages, so that addresses my major concern.   With any luck, this should make the next version bump go more smoothly.
Fixed in 3359908a657cac032fb64b279a26a20d378f8c2d


Add tests, or whatever

It's a bit terrifying hacking on it right now and not knowing if you've broken something!

Tests... What are those?!


Feature Request: Use Bookmark Names for tab names

Feature Request: Use Bookmark names for tab names.  A tab rule would override this behavior.
Benefit #1: People already use Bookmarks extensively.  This would allow users to rename tabs by simply adding a Bookmark and naming it, without ever going into Tab Rules.
Benefit #2: This would automatically allow different tab names for different Google users, since every Google user signed into Chrome has their own Bookmarks.
Thanks.  I love Tab Modifier!

Duplicate of #229


mangle exported functions for PyPy

Continuation of #3031. This turned up in the run of PyPy 3.10 HEAD against PyO3 HEAD. Should I add a new news fragment or is the one from #3031 sufficient?

I think the existing news fragment is sufficient. Thanks!
bors r+

@mattip Could you have a look at the build failure in https://github.com/PyO3/pyo3/actions/runs/4396220835/jobs/7698525481 ? Does this need a version limit?

bors retry


[BUG] seemingly wrong up_putc usage in lcd drivers

Description / Steps to reproduce the issue
many lcd drivers have a code fragment like the following:
https://github.com/apache/nuttx/blob/db31e569579c7ec9753d5fb69d07e8f997c7eae1/boards/arm/stm32/stm3240g-eval/src/stm32_lcd.c#L519-L539
(i'm not sure why it's common for lcd drivers. maybe just a result of heavy copy-and-paste?)
it seems like mixing syslog and up_putc, assuming they output to the same console.
of course, the assumption might or might not be correct, depending on the system and configurations.
this isn't a major problem as these code fragments are often wrapped with #if 0 though.
On which OS does this issue occur?
[OS: Mac]
What is the version of your OS?
macOS 14.7
NuttX Version
master
Issue Architecture
[Arch: all]
Issue Area
[Area: Drivers]
Verification

[X] I have verified before submitting the report.

@yamt good catch, according to git blame these lines were added by @patacongo in 2012 and 2014 respectively, so I think it is not my mistake, should have some reason.

@yamt good catch, according to git blame these lines were added by @patacongo in 2012 and 2014 respectively, so I think it is not my mistake, should have some reason.

Its just old debug code that was left in.  The xyz_dumprun() functions are not referenced from anywhere in the OS and just be removed.


Replace ifdef with flag? macro method

I would like to see ifdef gone, as it feels like a  construct not consistent with the language. Let me explain why I think it's inconsistent.
Crystal has two stages at which program code is run. Macro level code runs at the compile time, everything else runs at runtime. Following that it of course has two places where decisions are made. Macro code is clearly marked with the {%, {{, }} and %} tokens, they give a clear visual indicator that the contained code is not run at runtime, but at compile time. This looks as follows:
if condition
  code
else
  code
end

{% if condition %}
  code
{% else %}
  code
{% end %}

It's fairly obvious which code is run when. But more importantly it's also fairly obvious which code is included into the resulting executable as a result of that.
Now let's compare that to ifdef:
if condition
  code
else
  code
end

ifdef condition
   code
else
  code
end

Now that visual distinction is gone, worse else, the same keyword in both places, suddenly has differing meanings, depending on what came prior it.
Okay, now we could just reintroduce the visual distinction and make the macro tokens mandatory for ifdef:
{% ifdef condition %}
  code
{% else %}
  code
{% end %}

But now we have essentially two constructs that do the same thing, branching, in the macro level. Let's go a step further and allow retrieving compile time flags via a macro method:
{% if flag?(:darwin) || !flag?(:x86_64 %}
  workaround_32bit_darwin_bug
{% elsif flag?(:linux) %}
  linux_has_shortcut
{% else %}
  regular_way_for_everybody_else
{% end %}

Yes, this is substantially more noisy, but I think it's more consistent, leads to more obvious code and should only affect low level code for the most part.

I agree, but first we should support macros inside lib. The main issue is that inside a lib the program is parsed in a different way, so expanding a macro inside it must parse it in that way too (for example one can use union and type inside a lib but not outside it).
I think what you propose is the way to do it, even if it ends up being more verbose, but it does makes things more consistent, and removes one redundant concept from the language.

I also think that the formatter could easily upgrade the code, once we make the change :-)

One problem with this change is that this stops compiling:
{% if flag?(:darwin) %}
  x = 1
{% else %}
  x = 2
{% end %}
puts x # undefined method 'x'

The "beauty" of ifdef was that code inside it had to be valid, and was replaced before semantic analysis. Now the parser never gets to see those x.
The same applies for detecting initialized instance vars.
Not a big deal, workaround is:
x =
  {% if flag?(:darwin) %}
    1
  {% else %}
    2
  {% end %}

Maybe it's even better to force this, because it's easier to see if x was initialized in all branches.

Maybe we can introduce a macro ternary for simple cases like that
x = {{flag?(:darwin) ? 1 : 2 }}

@jhass That works already :-P
The problem is that if @x is an instance variable the type won't be inferred.
Not a big deal anyway.

Hah, I never noticed.

Another issue:
puts "start"

{% if true %}
  puts "before openssl 1"
  require "openssl"
  puts "after openssl 1"
{% end %}

puts "before openssl 2"
require "openssl"
puts "after openssl 2"

Assuming "openssl" is this:
# openssl.cr
puts "In openssl"

The output is this:
start
before openssl 1
after openssl 1
before openssl 2
in openssl
after openssl 2

The problem is that requires are resolved before semantic analysis and macro expansion, and so the first require that's expanded is the second one, not the first one (in order).
This breaks HTTP::Client here, when that line is replaced with if flag?.
The solution I'm thinking to apply is to expand these macros in the same place where require is expanded, as long as they are simple (for now just a flag? check and boolean combination of those). We can later see if other macros should be evaluated as well, but maybe some require some type declarations to exist, but that happens later.
This is another reason to keep ifdef in the language, but with these small new rules we should be able to preserve the old behaviour but with a unified syntax.

Can't we just expand toplevel macro code before require? I think that would lead to the most intuitive behavior.  For the, I assume very rare, cases where you do need to require something before/at toplevel macro expansion, we could offer a {{require ...}}.

@jhass I think I'll just place the require logic where everything else is, so it "executes" in the correct order. I'm doing some tests and that seems to work fine


Fragaria crashes when certain international characters are entered

Because of the difference between "glyphs" and "characters", characters that use multiple glyphs cause Fragaria to crash.
As I understand it, a character can be composed of multiple glyphs. However, -[NSString length] returns the number of characters, while a lot of the string operations are being performed on glyphs.
The character I've been testing with is ำ. Paste it into Fragaria a couple of times and it'll soon crash.
This pull request is slightly incomplete - right now it doesn't crash, but entering these special characters will cause the line count to be incorrect (Fragaria thinks the line is wrapping if you enter a newline character after the special character). I've spent some time on trying to fix it, but don't have any more time to spend on it. At least it doesn't crash anymore, I guess?

Oh, and the example application targets failed to build because modules were disabled.

Brilliant! I'd seen this crash in Manuscripts crash logs but didn't have a clue about how to work it out.


Catching up to mainline

Please apologize for having to ask, but what would be the next step? I am quite new to GitHub and not yet familiar with the procedure regarding merge, commits, PR, etc.
As I understood before the merge shall happen in several PR's, subdivided by functionality. Or is there a swifter way to achieve the goal?


[ feature ] option to hide file type in "loader" nodes

Can we have an user option to hide the file type in "loader" nodes ?
Most of the files are safetensors anyways.

This would make it easier to read the file names, especially when there are lots of files and long names ( like loras ).
ComfyUI does not have this option, but it's a well known option, in the OS ( e.g. Windows )

Done in dev. Should apply to all combo boxes.

It works.
But I still see them in the Power Lora Loader
Can they be hidden there, too ?

done on 1.7

It works.


Add signal-cli-config to backup created by ansible backup script.

Hi @phoeinx are you on to this? Would be great to get this done asap.

I’d be careful restoring signal-cli configuration in a development environment. If you forget that you restored a configuration or if you run certain signal-cli commands, this can have an impact on the Signal account and consequently also on the production instance the account belongs to.

That's a good point I hadn't thought of.

Basically, my opinion on this is that we should probably not verify the integrity of the signal-cli backups by restoring locally. We should probably verify in regular intervals that we can restore signal-cli backups, but I’d do that using a staging backup.

Maybe, we should be testing out our backups on the staging server instead of locally since there are some key differences.

Hi,
As a short explanation - we had been talking for quite some time about also backing up the signal-cli-config in case of that something went wrong. In June it then actually did (caused by me) and we had to restore a Signal setup from a signal-cli-config and could verify that it works on a production instance. Therefore that was my follow-up PR to include storing the signal-cli-config in all our back-ups.
Otherwise, I would agree with Till about not restoring signal-cli-configs into local development setups. I think in regular periods testing if we can restore the staging signal-cli-config on staging should be a good method to verify the ability to use those backups 👍 .


CVV field stays blank

The bot works fine until it needs to complete the purchase by entering the cvv. The field stays blank and the card is not being selected. What am I doing wrong? Here are the parameters I'm entering
python3 main.py --username<EMAIL_ADDRESS>--password pass --url https://www.nike.com/launch/t/womens-air-jordan-1-silver-toe?size=9 --shoe-size 9 --cvv 123 - --shoe-type W --dont-quit --release-time "2021-02-19 10:00:00" --shipping-option STANDARD --purchase
Any help would be appreciated
J

I think a lot of people are experiencing the same issue, including myself. I ended up adding a sleep condition when it hits the CVV function so that when it pops up i can quickly enter the CVV, it does add about 10 seconds to the checkout process.

Hey @shane-war,
Thanks for letting me know. I will look into doing the same and test to see if I can find a solution on my fork. Thanks again
J

Happened to my friend too, after writing the CVV by hand & continuing,  the bot refreshes the page & Drops you out from the contest.

Does it still happen? I am going to try and cop a new release? Can the CVV field be detected as an event?


add landice gridcomp

This PR is corresponding to  the  GEOSldas PR  to add GEOSlake_GridComp

@weiyuan-jiang: I just came across the following assumption in pre-processing: the *.til file must have all of the land tiles at the beginning.  This has worked so far, but it is not a safe assumption (as noted in my comment from 2020).  You are probably already aware of it in the context of the present PR, but I'm mentioning it here just in case.
https://github.com/GEOS-ESM/GEOSldas_GridComp/blob/05bbcca97230ced6591d38e4b475c9db9915971f/GEOSldas_App/preprocess_ldas_routines.F90#L2125
    ! assume all land tiles are at the beginning
          ! UNSAFE ASSUMPTION! - reichle, 2 Aug 2020

          if (typ/=MAPL_Land .and. typ/=MAPL_Land_ExcludeFromDomain) then   ! exit if not land

             if (logit) then
                write (logunit,*) 'WARNING: Encountered first non-land tile in *.til file.'
                write (logunit,*) '         Stop reading *.til file under the assumption that'
                write (logunit,*) '           land tiles are first in *.til file.'
                write (logunit,*) '         This is NOT a safe assumption beyond Icarus-NLv[x] tile spaces!!'
             end if

             exit ! assuming land comes first in the til file

          end if

@weiyuan-jiang: I just came across the following assumption in pre-processing...

PS: The exact same code appears again a couple of hundred lines later:
https://github.com/GEOS-ESM/GEOSldas_GridComp/blob/05bbcca97230ced6591d38e4b475c9db9915971f/GEOSldas_App/preprocess_ldas_routines.F90#L2378


[EWT-115][EWT-118]dag = None and fix for DagModel.fileloc(missed in EWT-16)

Can we change the commit message to adhere to the last section in the link I sent you in Slack? Looks good to me after that (you can feel free to merge it).
These forking changes are getting a bit scary since we don't run unit tests for them internally at the moment :(...
I believe you and @msumit a long time ago came up with some plan to remove this forking change (when you were investigating why DAGs were only appearing in the UI sometimes and not others due to them only appearing on webservers where they were last parsed), or am I confusing it with another change?

@aoen I think it's the same issue, where we've to store only relative dag location in DB instead of the complete path. We've not picked up this task yet, but once we are able to fix it upstream, we don't need to carry forward any of these commits.

Got it, thanks or the reply!


Threaded conversations

Orbit should have threaded conversations. This means that you should be able to "reply" to a message.
I imagine this to work so that we add a "reply" button to each message in the UI and display an indicator in the "send message" field that you're replying to a message (as opposed to a new message).
If a message in the chat has replies to it, the UI (ie. a single message line) should display a counter of how many replies are in this thread and button to expand and collapse the thread. If expanded, the replies should be displayed indented under the original message. This could be done similar to the Directory view component.
This will also require a new field in the Post.Message data structure. I think it would suffice to add "replyto" field which contains the has of the message this message is a reply to. In the UI, one can then check if the "replyto" field is set and render the message accordingly.
I'd be happy to hear other suggestions as to how this would be displayed in the UI.

Flowdock has the best implementation I have seen so far on this: https://www.flowdock.com/features
Screenshot:

Did a simple prototype here: https://github.com/haadcode/orbit/tree/threads

Wow that flowdock UX is really good. The color + popout window adds so much
Thanks @haadcode! can't wait to try it
On Thu, Aug 4, 2016 at 06:12 Haad<EMAIL_ADDRESS>wrote:

Did a simple prototype here:
https://github.com/haadcode/orbit/tree/threads
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/haadcode/orbit/issues/60#issuecomment-237512000, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAIcocrZIkYOfXFOLwmuDmb4rQYSBlFbks5qcbsVgaJpZM4JcZ0m
.


WHEP support in egress endpoint library

Add a WHEPEndpoint that supports the protocol https://www.ietf.org/id/draft-murillo-whep-00.html

We could limit the initial version with server-side initiated offer which is similar to WHPP


Wrong relative CSS paths when using @import

I have a CSS file located at /theme/css/style.css:
/* fonts and tomorrow theme */
@import url(http://fonts.googleapis.com/css?family=Gentium+Book+Basic:400,700&subset=latin,latin-ext);
@import url(http://fonts.googleapis.com/css?family=Open+Sans:400,800&subset=latin,latin-ext);
@import url("./tomorrow.css");

@import url("http://cdnjs.cloudflare.com/ajax/libs/animate.css/3.1.0/animate.min.css");
@import url("http://netdna.bootstrapcdn.com/font-awesome/4.1.0/css/font-awesome.css");

/* markdown css */
@import url("./markdown.css");

/* ... */

However the relative paths ./tomorrow.css and ./markdown.css are returned as absolute paths (e.g. http://localhost:9000/tomorrow.css).
My code is:
// Dependencies
var Browser = require("zombie");

// Load localhost:9000
Browser.localhost("localhost:9000");

// Load the page from localhost,
// including js, css, images and iframes
var browser = new Browser({
  features: "scripts css img iframe"
});

// Open the page and list the resources
browser.visit("/", function(error) {
    console.log(browser.resources.map(function (c) {
        return c.request.url;
    }));
});

Is this a bug or I'm doing something wrong?

Probably a bug

@assaf If you told me where I should start debugging, I would be happy to contribute.

Stylesheet parsing handled by CSSOM, which gets the page URL from JSDOM. When loading images and scripts, JSDOM uses loader.resolver to resolve relative URLs, so maybe it's doing the same thing with CSS.

@assaf If you could debug and fix it I would be so thankful...!

I'm not going to have any time to work on this, sorry

FYI, I think this is a bug in jsdom, I have a fix/pull request.

FYI, second pull request with test accepted, jsdom 5.2.0 has the fix.

@dbo Awesome! Thanks a lot! :+1:

@dbo thanks for the fix!
Just releases Zombie 4.0.10 which uses JSDOM 5.2.0


Add shimmer effect on posts

This PR closes #1035
I have added shimmer effects on post of home page. While the post are loading the shimmer is shown and after being loaded the posts are shown.

No, I will add also on profile page. It somehow get slipped from my mind or I don't know there was some error. But, okay I will do changes for that too.


Resolves issue #282: Created and added iheartradio svg

Created and added iheart radio svg using official art files provided by iheart radio - converted eps to svg

Please follow the contributing guide (file must be under 1 KiB, edit README.md, etc.).

@romain-dartigues apologies.
I uploaded the wrong file but I also admit I neglected to proper ready the contributing guide.
Small concern: My smallest possible SVG created for this was 2,225 bytes (2.2kb).
Upon further research other people who have attempted the same for this logo have only been able to reduce to the same exact size. With gzip compression it can only be brought to about 1.18kb.
I know this does not align with the goal of this project so I'm afraid I'm unable to complete this.
If it's possible at all.

How about just the red heart logo? Remove the text at the bottom. I reckon that will fit into the size limit.

@edent that sounds great.
I’ll do that and update the remaining files as per the contribution guidelines today!
Thank you.

Alright @edent that reduced it by a tad over 50% and then added to reference and read me as per contribution guidelines.

This seems to have removed the actual icon you're trying to submit.

I had a little go at this by chopping it in half and using a transform to mirror... Down to 1385 bytes... https://github.com/jmb/SuperTinyIcons/blob/iheartradio/images/reference/iheartradio.svg
I thought I'd made if fit the guidelines too, but that still needs tweaking it seems!

Fixed the transform, tried rounding and now down to 948 bytes... :) Will do a PR to your repo @HenrySabio then you can PR to the master if you like!

That sounds good.
I was able to get it down to a similar size but it would not longer open in illustrator when I tried rounding the numbers. It would open  and preview fine everywhere else but would open as a blank canvas otherwise in illustrator.
Hopefully this version works fine.

Yeah it's a bit tricky - I'm using Inkscape 1.0 and ended up just copy and pasting the SVG data as required!

@jmb this looks perfect! If you could just also update the needed associated files in the reference & readme and we should be good to go with a final commit

I thought you could do that in your branch! ;)
Can do it in a bit if you need me to though!

@jmb I can! I just won't be able to until later tonight. I figured the issue/pr has been open for long enough I'm sure they'd like this resolved asap. But No problem! I'll have this taken care of.
@edent Looks like We have the solution - I'll tag you when the final commit has been made and ready to merge!

Ok, no prob will take a look shortly.

Teamwork makes the dream work :-)
Tag me when it's ready to go!


Should we pass go context.Context to all the components?

In golang, context.Context is widely used and it's always be the first argument of go method and passed everywhere.
In dapr current implementation, for example, pkg/grpc/api.go, all the public method called by gRPC have the context object, and some them (service incation, actor ) pass context in the next step (always calling the component):
func (a *api) CallLocal(ctx context.Context, in *internalv1pb.InternalInvokeRequest) (*internalv1pb.InternalInvokeResponse, error) {
	callAllowed, errMsg := a.applyAccessControlPolicies(ctx, operation, httpVerb, a.appProtocol)
	resp, err := a.appChannel.InvokeMethod(ctx, req)
}

func (a *api) CallActor(ctx context.Context, in *internalv1pb.InternalInvokeRequest) (*internalv1pb.InternalInvokeResponse, error) {
	resp, err := a.actor.Call(ctx, req)
}
func (a *api) InvokeService(ctx context.Context, in *runtimev1pb.InvokeServiceRequest) (*commonv1pb.InvokeResponse, error) {
	resp, err := a.directMessaging.Invoke(ctx, in.Id, req)
}
func (a *api) GetActorState(ctx context.Context, in *runtimev1pb.GetActorStateRequest) (*runtimev1pb.GetActorStateResponse, error) {
	resp, err := a.actor.GetState(ctx, &req)
}

And some of them (pub-sub / binding / state ) don't pass context:
func (a *api) PublishEvent(ctx context.Context, in *runtimev1pb.PublishEventRequest) (*emptypb.Empty, error) {
	err = a.pubsubAdapter.Publish(&req)
}
func (a *api) SubscribeEvent(srv runtimev1pb.Dapr_SubscribeEventServer) error {
	err := curPubsub.Subscribe(req, func(msg *pubsub.NewMessage) error
}
func (a *api) InvokeBinding(ctx context.Context, in *runtimev1pb.InvokeBindingRequest) (*runtimev1pb.InvokeBindingResponse, error) {
	resp, err := a.sendToOutputBindingFn(in.Name, req)
}
func (a *api) GetState(ctx context.Context, in *runtimev1pb.GetStateRequest) (*runtimev1pb.GetStateResponse, error) {
	getResponse, err := store.Get(&req)
}

So, there are big difference between components.
For example, actor api, all the method start with context argument :
type Actors interface {
	Call(ctx context.Context, req *invokev1.InvokeMethodRequest) (*invokev1.InvokeMethodResponse, error)
	Init() error
	Stop()
	GetState(ctx context.Context, req *GetStateRequest) (*StateResponse, error)
	TransactionalStateOperation(ctx context.Context, req *TransactionalRequest) error
	GetReminder(ctx context.Context, req *GetReminderRequest) (*Reminder, error)
	CreateReminder(ctx context.Context, req *CreateReminderRequest) error
	DeleteReminder(ctx context.Context, req *DeleteReminderRequest) error
	CreateTimer(ctx context.Context, req *CreateTimerRequest) error
	DeleteTimer(ctx context.Context, req *DeleteTimerRequest) error
	IsActorHosted(ctx context.Context, req *ActorHostedRequest) bool
	GetActiveActorsCount(ctx context.Context) []ActiveActorsCount
}

But state api, no context at all:
type Store interface {
	BulkStore
	Init(metadata Metadata) error
	Delete(req *DeleteRequest) error
	Get(req *GetRequest) (*GetResponse, error)
	Set(req *SetRequest) error
}

type BulkStore interface {
	BulkDelete(req []DeleteRequest) error
	BulkGet(req []GetRequest) (bool, []BulkGetResponse, error)
	BulkSet(req []SetRequest) error
}

I suggest to pass the context to all the components in golang style.
In fact, now I'm working to add tracting  support in our state and bindings component, and I find that I can't get the context from the public API method of dapr gRPC service, I have to find other way (using metadata, again, we use metadata to do almost everything that dapr don't support) to pass the tracing information like traceid.

In service invacation, the context is passthrough the process and finally used for tracing:
	if IsGRPCProtocol(internalMD) {
		processGRPCToGRPCTraceHeader(ctx, md, grpctracebinValue)
	} else {
		// if httpProtocol, then pass HTTP traceparent and HTTP tracestate header values, attach it in grpc-trace-bin header
		processHTTPToGRPCTraceHeader(ctx, md, traceparentValue, tracestateValue)
	}

I think we should do similar in other components such state / bindings.

In service invacation, the context is passthrough the process and finally used for tracing:
	if IsGRPCProtocol(internalMD) {
		processGRPCToGRPCTraceHeader(ctx, md, grpctracebinValue)
	} else {
		// if httpProtocol, then pass HTTP traceparent and HTTP tracestate header values, attach it in grpc-trace-bin header
		processHTTPToGRPCTraceHeader(ctx, md, traceparentValue, tracestateValue)
	}

I think we should do similar in other components such state / bindings.

@yaron2 How about this ?

@yaron2 How about this ?

@yaron2 How about this ?

Yes, that would be a good thing to do. With gRPC at least, we will be able to cancel ongoing calls.
There is a PR in place to add a context to Pub Sub.

@yaron2 How about this ?

Yes, that would be a good thing to do. With gRPC at least, we will be able to cancel ongoing calls.
There is a PR in place to add a context to Pub Sub.

@yaron2 How about this ?

Yes, that would be a good thing to do. With gRPC at least, we will be able to cancel ongoing calls.
There is a PR in place to add a context to Pub Sub.

OK, I will PR for bindings and state.

Is anyone working on this issue?
/cc @skyao @yaron2

@yaron2  @artursouza  Is this still open. If it is can you help me with what all areas should I look into?

@ShaileshSurya  Look at the interfaces for the building blocks in dapr/components-contrib repo.

@artursouza I see that the component-contrib/state is being consumed by the Dapr repository.
We might need to open two different PR's on component-contrib and dapr. This can cause both the repo's going out of sync for a while.
How can we go around this?

@artursouza I see that the component-contrib/state is being consumed by the Dapr repository.
We might need to open two different PR's on component-contrib and dapr. This can cause both the repo's going out of sync for a while.
How can we go around this?

This is common. You work on both repos at the same time and submit the PR for both. After we merge the PR for contrib, the PR for dapr/dapr is updated with the latest contrib version in master and we merge that one too.

@skyao @artursouza I'd like to contribute to the binding part.

@pigletfly I reviewed your PR. I left instructions on your PR for next steps you must complete once the contrib PR is merged:

open PR against dapr/dapr runtime.go to set the context for the bindings methods, and also make sure to import latest components-contrib into dapr/dapr
Once the dapr/dapr PR is merged, open PR to import the latest dapr/dapr into the certification test framework (the folder tests/certification within dapr/components-contrib) and fix all bindings tests there to also include the context as necessary.

@berndverst as for the pubsub component, should be change the interface into
type PubSub interface {
	Init(metadata Metadata) error
	Features() []Feature
	Publish(ctx context.Context, req *PublishRequest) error
	Subscribe(ctx context.Context, req SubscribeRequest, handler Handler) error
	Close(ctx context.Context) error
}


Changing itemPermalink: '/:slug',

How do I change itemPermalink to '/:slug'?
Where to put this option? How do I change other theme options using javascript?

Documentation does not tell you where (what file) file should I put these options. I changed the example to be a function in .vuepress/config.js, but modifyBlogPluginOptions is undefined.
module.exports = (themeConfig) => {
	const { modifyBlogPluginOptions } = themeConfig
	console.log('modifyBlogPluginOptions', modifyBlogPluginOptions)

It looks kile modifyBlogPluginOptions is deprecated, see https://github.com/vuepressjs/vuepress-theme-blog/pull/57.
Setting directories -tag inside themeConfig does not work. This is how I think it should work.
module.exports = {
  themeConfig: {
    title: 'myblog',
    directories: [...]
  }
}

I found a correct way to use this:


Core/URI.php: Parse_url() fails if a segment contains "foo:digit"

== Details ==
Type: Function stops script
File: core/URI.php:176
== Description ==
Parse_url will fail on URIs with segments that look like "foo:number". It is the colon that appears to be causing it (probably because parse_url thinks that it is a port number, and thus malformed).
== Replication ==
Go to a URL like this: http://localhost/ci/foo/bar/test:2/
== Proposed fix ==
Substitute ":" with a tag, parse the URI, and then reinstate the ":" again.
== Original code ==
{{{
#!php
$uri = parse_url($uri, PHP_URL_PATH);
}}}
== Workaround ==
{{{
#!php
if (preg_match('/:\d/', $uri))
{
$uri = str_replace('{C0L0N}',':',parse_url(str_replace(':','{C0L0N}',$uri),PHP_URL_PATH));
}
else $uri = parse_url($uri, PHP_URL_PATH);
}}}

Wouldn't this fix replace a segment thar contains the exact text {COLON} with :?


Add how to specify a virtual environment for Python Applications

Ref: https://www.nginx.com/blog/unit-0-3-beta-release-available-now/

a sample:
https://github.com/nginx/unit/issues/151#issuecomment-405249583

Hello,
Thank you for your contribution!


Text with ellipsis overflow with HTML web-renderer overlaps itself 3.7.10

The closed issue  3.7.10 3.7.10 on 3.7.9 appears again after updating to  (Channel stable, 3.7.10)

It's strange, the text is displaying well from my end.

Can you confirm there is no specific theme/font on your setup? What is the display language of your Chrome browser?
Please also try below complete sample code to see if it still persists.

Sample code
import 'package:flutter/material.dart';

void main() {
  runApp(const MaterialApp(home: TestLayout()));
}

class TestLayout extends StatefulWidget {
  const TestLayout({Key? key}) : super(key: key);

  @override
  State<TestLayout> createState() => _TestLayoutState();
}

class _TestLayoutState extends State<TestLayout> {
  @override
  Widget build(BuildContext context) {
    return Material(
      child:  Column(
        mainAxisSize: MainAxisSize.max,
        children: [
          const Expanded(
            child: Center(
              child: Text("test test \n\n XMLHttpRequest error.", textAlign: TextAlign.center,),
            ),
          ),
          Divider(
            thickness: 2.0,
            color: Theme.of(context).colorScheme.secondary.withOpacity(0.2),
          ),

        ],
      ),
    );
  }
}

i must apologize here. i had activated a flutter web preloader which apparently overwrote the body with a certain font, which then led to the strange behavior.


Animating wheel

How would I approach making the wheel spin to a random monster (so to speak) on tap?

iCarousel does the job


Ensure address is not null pointer before use.

Fixes related issues #115 and #113

Did the project support die?

Just checked. Sounds good.


BUG: cluster: fix leaves_color_list issue

Reference issue
Closes #16558
What does this implement/fix?
This fixes scipy/clustering/hierarchy.py dendrogram function from returning improper color values for leaves representing clusters with 0 distance between them.
Additional information

Can do, I'll add that test tomorrow

Hi @AtsushiSakai and @tupui, thank you for fixing the issues I left in the code, I am still not used to all the checks. When I get the time this weekend, I will look into solutions resolving the failed tests.

@njj67229 Hi. Would you like to finish this PR?. If you are busy, I can take over this PR.

This is still marked for the next release. While not critical, it'd be nice to get it in. @AtsushiSakai maybe you'd want to get this one over the finish line?

I rebased on the latest main locally and confirmed that the full test suite passes on my Linux box.
I can't actually see any of the logs that were failing back in July, so I've force pushed the rebase to see what, if anything, still remains an issue in the CI.
The linter and mypy checks had a few small complaints locally, but I don't fully trust that those will be replicated in CI since they aren't related to the diff here. So, let's see where we stand when CI flushes through.
I don't think I actually see any outstanding comments apart from fixing up the old CI failures.

Ah, I'll clean up the whitespace the linter is complaining about, probably a bit later

Ok, CI is passing now and the small code change + new tests seem pretty safe. It looks like the objective here is to make related points have consistent leaf colors instead of different ones, similar to conventional phylogenetic plots.


[prettier-plugin-sh]: TypeError [Error]: Cannot set property crypto of # which has only a getter

Setting NODE_OPTIONS="--experimental-global-webcrypto" causes a crash:
TypeError [Error]: Cannot set property crypto of #<Object> which has only a getter

The --experimental-global-webcrypto enables the Web Crypto API from the browser in NodeJS as the global crypto property.
While I’ve disabled experimental NODE_OPTIONS while running prettier, this issue will likely occur in NodeJS v18.x, which become the LTS version in October 2022.

Occurs due to node_modules/sh-syntax/lib/shim/js:
import { randomFillSync } from 'node:crypto';
import _fs from 'node:fs';
import { performance } from 'node:perf_hooks';
globalThis.fs = _fs;
globalThis.crypto = {
    // @ts-expect-error
    getRandomValues: randomFillSync,
};
// @ts-expect-error
globalThis.performance = performance;
//# sourceMappingURL=shim.js.map

Specifically:
globalThis.crypto = {
    // @ts-expect-error
    getRandomValues: randomFillSync,
};

Perhaps a workaround would be:
if (!globalThis.crypto) {
    globalThis.crypto = {
        // @ts-expect-error
        getRandomValues: randomFillSync,
    };
}

Would you like to raise a PR to sh-syntax instead?

And also, please close issues on mvdan/sh and gopherjs, they're not related.

Sure!  Sorry about that.  Resolved in https://github.com/un-ts/sh-syntax/pull/52


Add Wibowo2021 paleofeces - libs

Pull Request
This PR is for a

[x] New Publication(s)
[ ] Correction

For the following list(s):

[ ] ancientmetagenome-anthropogenic (README)
[ ] ancientmetagenome-environmental (README, ask @thedir-team-dirt for advice)
[x] ancientmetagenome-hostassociated (README, ask @thedir-team-bugparty for advice)
[ ] ancientsinglegenome-hostassociated (README, ask @thedir-team-pathopeeps for advice)

New Publication
Publication Information
This pull request is to add samples from the following publication(s): 10.1038/s41586-021-03532-0
This is to close #485
Checklist

[x] Publication is published (preprints currently not accepted)?
[x] Checked the publication is not already in the database?
[x] Checked samples in this publication are not previously published data (newly re-sequenced metagenomes are OK!)?
[x] Samples are shotgun metagenomes (hostassociated-singlegenome may also contain whole-genome enriched data)?
[x] Checked the list follows conventions as described in the corresponding sample type's README file (e.g. using ERS/SRS accession codes for ENA/SRA)?
[ ] Changelog is updated to include the publication under 'Added'?
[ ] Pull request has passed validation checks (see automated comment from github-bot)?
[ ] Review requested from corresponding team (see list of lists above for each team)?
[ ] (If applicable) Made a separate PR to add new categories to controlled lists (stored under /assets/enums, e.g. material or archive)? If so the corresponding PR is here:

If you do not know how to check errors in failed validation checks, expand here

#669 reports KAPA for the sediment samples because they were with a different kit. For this PR we can go just with the Meyer and Kircher polymerase 👍