Update Examples File

Currently the examples file generates substitutions using the old diagrammatic framework. These need to be updated before this file will be usable again.
Additionally, it would be good to find a better way of describing operations as DWDs without needing to create a closed-compile loop. This would likely require a sub-function of diag2dwd which accepts the specification of input/output signatures for the generated DWD, which wouldn't be that large of a lift from the current diag2dwd tooling.

PR #15 addresses both issues mentioned above, both updating the Examples folder and adding more options for generating the directed wiring diagram from a free diagram


Page position moved after choosing concepts

After choosing a concept, the current position of the page moves (~to the first annotated position?~).

This also happens to newly added concepts (not edited).

maybe the page move back to the position where concept is chosen just before（直前）.


Add South Korea Language

There is no exact meaning of the word 'Poll' in Korea.
And it's not all accurate, but it's mostly accurate.

Hello ! Thanks for contributing. Everything seems good for me I will merge your PR.


Fix upgrading k8s minor version error messages

Fixes #182. Cheers @fshot!

Thank you, @DevAndrewGeorge !


Autocomplete: unstable-openai provider legacyModel conditions

The diff in this PR affects only Cody debug output logs because the model ID is not used by the unstable-openai autocomplete provider. I suggest keeping whatever model name clients receive in the Cody output channel for debugging purposes. Open to suggestions 🙂 (cc @slimsag because of your comment here).

Yes, there might be a deployment name that does not clearly translate to the actual model behind it, but this is still a debugging signal that the client correctly resolved the site configuration.
It makes the code simpler by removing special cases.
I plan to go over other providers and remove similar conditions there.

No functional changes.

Test plan
CI

Going to update failing tests tomorrow 🙂

TY @arafatkatze. I'll fix the tests and merge this one as is.
I have a follow-up PR that will add tests for what matters — autocomplete request parameters. Right now, we assert the model ID assigned to the provider instance, but often, it's not used at all to make requests, which means this assertion is irrelevant to the actual functionality.

We can also iterate on output logs in follow-up PRs based on the feedback and further testing.

Here's the follow up in-progress PR: https://github.com/sourcegraph/cody/pull/5604


Add ability to write integer cell data

I think I will just extrapolate the definition of the active array in a completely consistent manner with what currently exists, with the exception that I will add an "define_active_scalar_array" (and its analogues), so that the user doesn't need to just settle for the most recently defined scalar/vector/tensor/* array being the active one (which will be the default behavior), or to be forced to add the cell data in a particular order.

I think it is cleaner to template the add_xxx_field functions on the data type, and instantiate add_xxx_field<int> and add_xxx_field<double> in the cpp. I'll try to add this when I can.
What do you mean by "active" array? current_vector_point_data_, etc? Do you think this is needed?

Meh, no worries, I can do the templating.
Yes, current_*point_data.  Its not totally clear what active arrays are used for, but I did find this at https://kitware.github.io/vtk-js/api/Common_DataModel_DataSetAttributes.html:
"This adds to vtkFieldData the ability to pick one of the arrays from the field as the currently active array for each attribute type. In other words, you pick one array to be called “THE” Scalars, and then filters down the pipeline will treat that array specially. For example vtkContourFilter will contour “THE” Scalar array unless a different array is asked for.",
as well as some other discussions, like https://discourse.paraview.org/t/have-paraview-use-the-active-scalars-as-the-display-coloring/1037.  So it seems like the active array will be the default array for a given data array type, in at least some contexts.  As such, I think yes, it makes sense to allow users to define the default scalars/tensors/whatever.


Nuke member role requirement

The _member_ role is a vestige from OpenStack days of yore and no longer exists on recent versions by default.  Do as the TODO suggests and remove it.

If it's broke then it's because of Sausage Cloud's policies that need bringing into line - let me know how you fare.

Okay so, as expected, provisioning with the new defaults results in:
{"ts":1721636294055.5776,"caller":"controller/controller.go:329","msg":"Reconciler error","controller":"openstackcluster","controllerGroup":"infrastructure.cluster.x-k8s.io","controllerKind":"OpenStackCluster","OpenStackCluster":{"name":"cluster-cd0eae95","namespace":"f8cdef7d-d5dc-41a3-94ec-2b30802b70ef"},"namespace":"f8cdef7d-d5dc-41a3-94ec-2b30802b70ef","name":"cluster-cd0eae95","reconcileID":"6e360044-5b83-4907-91c1-34f48d568cb1","err":"failed to reconcile load balancer: Request forbidden: [GET https://compute.sausage.cloud:9876/v2.0/lbaas/loadbalancers?name=k8s-clusterapi-cluster-f8cdef7d-d5dc-41a3-94ec-2b30802b70ef-cluster-cd0eae95-kubeapi], error message: {\"faultcode\": \"Client\", \"faultstring\": \"Policy does not allow this request to be performed.\", \"debuginfo\": null}"}

And trying to delete the cluster leads to a deadlock as it cannot read loadbalancers to delete loadbalancers 😸
Setting to the following restores functionality (and actually tests it works):
regions:
- name: gb-north-1
  provider: openstack
  openstack:
    endpoint: https://compute.sausage.cloud:5000
    serviceAccountSecret:
      name: gb-north-1-credentials
    identity:
      clusterRoles:
      - _member_
      - member
      - load-balancer_member


Fix "words" view; upgrade DataTables; scope last_info to room

Some screenshots...
DataTables with Scroller plugin and some other styling tweaks, plus linking between Facts/Words:

Scoping of "last_info" to room or user:

Bump?

FYI: I've added an additional commit that contains (as described) some small tweaks/fixes that I've added – from testing on our own instance – since creating this pull request.


Missing types in v5.1.0

Expected Behavior
Types should be included in the latest version
Current Behavior
Types are not included in the latest release and cause CI (and local development) to fail.
Could not find a declaration file for module '@storyblok/astro'. '<project_path>/node_modules/.pnpm/@storyblok+astro@5.1.0_astro@5.0.9_@types+node@22.10.2_jiti@1.21.6_rollup@4.28.1_typescript@5.7.2_yaml<EMAIL_ADDRESS>implicitly has an 'any' type.
  If the '@storyblok/astro' package actually exposes this module, try adding a new declaration (.d.ts) file containing `declare module '@storyblok/astro';`ts(7016)

astro info
Astro                    v5.0.9
Node                     v20.10.0
System                   macOS (x64)
Package Manager          pnpm
Output                   server
Adapter                  @astrojs/cloudflare
Integrations             @storyblok/astro
                         @astrojs/tailwind

Typescript Version: ^5.7.2
Steps to Reproduce

Upgrade to v5.1.0
Restart TS server (in visual studio code) or let it build in CI

Downgrading to v5.0.1 fixes the issue locally and in CI


tick "mark all mails" and untick one mail - delete marked mails failed

RainLoop version: <IP_ADDRESS>, Firefox 52.0.2 , OS Win7 / Server: Debian Jessie
Expected behavior and actual behavior:
Navigate to a mailfolder
Mark all mails in the folder (one site 20 mails)
Uncheck one mail
Press delete-button
Expected that would be deleted 19 mails
Actual behavior is, that the unticked is also deleted even it is unticked
Steps to reproduce the problem:
Navigate to a mailfolder
Mark all mails in the folder (one site 20 mails)
Uncheck one mail
Press delete-button
Logs or screenshots:
No logs and screenshots available

Dupe of https://github.com/RainLoop/rainloop-webmail/issues/1388, fixed in https://github.com/RainLoop/rainloop-webmail/commit/317318e1f3dd1a3381bd628903f99e59b3224af5 but not released yet.


Ui animation in navbar

The issue involves enhancing the user interface by adding animations to the navigation bar. This feature aims to improve the visual appeal and user experience of the website or application by making the navigation more dynamic and engaging. The task would typically include designing subtle animations that activate when users interact with the navbar elements, such as hovering over links or clicking on menu items, without compromising the usability or performance of the site.

hey i want to resolve this issue so could you assign it to me

hello admin I want to work on this issue under gssoc24 please assign it


Cache dependencies on github action. [skip ci]

FYI, [skip ci] should be in the commit title, not in the PR title.

@hcho3 I'm not sure why is mvn on windows latest is consistently failing/

No idea.


Some confusion about the code?

https://github.com/LTS4/hold-me-tight/blob/b893e97f0b5fe8100472ac68d715d0cb99d0c7dc/utils.py#L166
Hi, the work "Hold me tight!" is excellent and I enjoy the reading. Here, I have a small question about the code:
for inputs, targets in dataloader:
    inputs, targets = inputs.to(DEVICE), targets.to(DEVICE)
    if proc_fun:
        inputs = proc_fun(inputs)
    adv_perts = torch.zeros_like(inputs)
    for n, im in enumerate(inputs):
        adv_perts[n], _, _, _, _ = subspace_deepfool(im, model, trans, Sp=Sp)

You define adv_perts through adv_perts = torch.zeros_like(inputs), and then calculate the perturbations by adv_perts[n], _, _, _, _ = subspace_deepfool(im, model, trans, Sp=Sp). However, you re-define adv_perts in the next loop without dealing with the calculated adv_perts[n]. I am a little confused about this and look forward to your reply.

Hi @LeavesLei. Thanks for spotting this!
In fact, the sp_margin.append(adv_perts.cpu().view([-1, np.prod(inputs.shape[1:])]).norm(dim=[1])) should be indented once, in order to get inside the for inputs, targets in dataloader: loop.
I just pushed an updated version that fixes this issue. Let me know if you still think that something is problematic!

Thank your quick reply 😄. I think the issue can be closed.


Add support for Python 3.12 and enable it in CI

Pull Request check-list
Please make sure to review and check all of these items:

[X] Do tests and lints pass with this change?
[ ] Do the CI tests pass with this change (enable it first in your forked repo and wait for the github action build to finish)?
[ ] Is the new or changed code fully tested?
[ ] Is a documentation update included (if this change modifies existing APIs, or introduces new ones)?
[ ] Is there an example added to the examples folder (if applicable)?

NOTE: these things are not required to open a PR and can be done
afterwards / while the PR is open.
Description of change
This changeset finishes adding support for Python 3.12

Oh, I didn't know updating black would introduce syntax errors for 3.8.
I'm going to drop this commit then. Not a biggie.

Codecov Report
Attention: Patch coverage is 83.33333% with 1 line in your changes missing coverage. Please review.

Project coverage is 61.32%. Comparing base (e4cb9e2) to head (3417ad7).

Files
Patch %
Lines

valkey/commands/timeseries/info.py
0.00%
1 Missing :warning:

:exclamation:  There is a different number of reports uploaded between BASE (e4cb9e2) and HEAD (3417ad7). Click for more details.
HEAD has 54 uploads less than BASE

Flag
BASE (e4cb9e2)
HEAD (3417ad7)

60
6

Additional details and impacted files
@@             Coverage Diff             @@
##             main      #54       +/-   ##
===========================================
- Coverage   75.01%   61.32%   -13.69%
===========================================
  Files         132      132
  Lines       34368    34367        -1
===========================================
- Hits        25781    21077     -4704
- Misses       8587    13290     +4703

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

Oh, I didn't know updating black would introduce syntax errors for 3.8.
I'm going to drop this commit then. Not a biggie.

yeah I faced this is another PR too. I think the line size limit on Black is different from whatever checks the linting in CI

I think the line size limit on Black is different from whatever checks the linting in CI

For me it wasn't even about the line size. Apparently adding parenthesis to with statement and moving its internals to different lines wasn't valid in 3.8. Bizarre!

So excited to see this merged!


Anti Spoofing Project using DL

Deep Learning Simplified Repository (Proposing new issue)
:red_circle: Project Title : Anti Spoofing Project
:red_circle: Aim : Create a DL model which will identify the real image, replay, layout and cut out.
:red_circle: Dataset : https://www.kaggle.com/datasets/tapakah68/anti-spoofing
:red_circle: Approach : Try to use 3-4 algorithms to implement the models and compare all the algorithms to find out the best fitted algorithm for the model by checking the accuracy scores. Also do not forget to do a exploratory data analysis before creating any model.

📍 Follow the Guidelines to Contribute in the Project :

You need to create a separate folder named as the Project Title.
Inside that folder, there will be four main components.

Images - To store the required images.
Dataset - To store the dataset or, information/source about the dataset.
Model - To store the machine learning model you've created using the dataset.
requirements.txt - This file will contain the required packages/libraries to run the project in other machines.

Inside the Model folder, the README.md file must be filled up properly, with proper visualizations and conclusions.

:red_circle::yellow_circle: Points to Note :

The issues will be assigned on a first come first serve basis, 1 Issue == 1 PR.
"Issue Title" and "PR Title should be the same. Include issue number along with it.
Follow Contributing Guidelines & Code of Conduct before start Contributing.

:white_check_mark: To be Mentioned while taking the issue :

Full name :
GitHub Profile Link :
Email ID :
Participant ID (if applicable):
Approach for this Project :
What is your participant role? (Mention the Open Source program)

Happy Contributing 🚀
All the best. Enjoy your open source journey ahead. 😎

I would like to work on this, frankly I don't know much but I will learn and do. I am a newbie(first year UG student). I want to learn and grow. please give me the opportunity to learn and experience

Please do not to spam in every single issue. This might cause escalation as you are breaching Code of Conduct and Contribution Guidelines.
@rafiya2003

Full name : Gudimella Saketa Sri Ramacharyulu
GitHub Profile Link : https://github.com/SaketGudimella
Email ID<EMAIL_ADDRESS>Participant ID (if applicable): d4880732-5d70-48ba-a05e-a68cb5508a2b
Approach for this Project :
I would be implementing this Anti-Spoofing project, with 3 specific DL algorithms -  1) Convolutional Neural Networks (CNNs), 2) Recurrent Neural Networks (RNNs), and 3) Siamese Networks. I will use the CNNs to extract the discriminative features from the images and classify them into different categories,the RNNs will be applied to capture temporal dependencies in sequential data, such as videos and the Siamese networks will be used to compare input images with reference images and determine their similarity or difference.
What is your participant role? SSoC'23 (Contributor)

Issue assigned to you @SaketGudimella


removed duplicate default value

This change is

done

@OskarStark as you can see in the file footer this document is auto-generated. You need to go to the source code of this program to fix this and later on we'll import the auto-generated result from there. Otherwise your changes will be overwritten and it'll return back to its buggy state.


error imgs must be a list, but got <class 'torch.Tensor'>  from test.py

I try to validate my model with test.py as it been written https://github.com/open-mmlab/mmdetection/blob/master/docs/GETTING_STARTED.md#test-a-dataset
but get some internal not-intuitively clear error
Ubuntu 18, with gpu 2080
/home/user/anaconda3/bin/python ../mmdetection/tools/test.py configs/mobilenet_ssd512.py work_dirs/mobilenet_ssd512/latest.pth --out mobilenet_ssd512_test.pkl --eval bbox --show

[                              ] 0/2914, elapsed: 0s, ETA:Traceback (most recent call last):
  File "../mmdetection/tools/test.py", line 224, in <module>
    main()
  File "../mmdetection/tools/test.py", line 184, in main
    outputs = single_gpu_test(model, data_loader, args.show)
  File "../mmdetection/tools/test.py", line 26, in single_gpu_test
    result = model(return_loss=False, rescale=not show, **data)
  File "/home/user/anaconda3/lib/python3.7/site-packages/torch/nn/modules/module.py", line 547, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/user/anaconda3/lib/python3.7/site-packages/torch/nn/parallel/data_parallel.py", line 150, in forward
    return self.module(*inputs[0], **kwargs[0])
  File "/home/user/anaconda3/lib/python3.7/site-packages/torch/nn/modules/module.py", line 547, in __call__
    result = self.forward(*input, **kwargs)
  File "/home/user/repos/mmdetection/mmdet/core/fp16/decorators.py", line 49, in new_func
    return old_func(*args, **kwargs)
  File "/home/user/repos/mmdetection/mmdet/models/detectors/base.py", line 88, in forward
    return self.forward_test(img, img_meta, **kwargs)
  File "/home/user/repos/mmdetection/mmdet/models/detectors/base.py", line 67, in forward_test
    name, type(var)))
TypeError: imgs must be a list, but got <class 'torch.Tensor'>

I have the same problem. Please give me some advice

Is my PR not enough for you?

@lijain did it helped?

@lijain and can you reproduce the error?

I happened to meet this error/problem when I tried to use the test.py script with my own HTC configuration. However, if I use the error doesn't exist if I use the original config from mmdetection library. Is there any follow up regarding this issue?


Tapping on cell during scroll view bounce highlights cell but does not select it

When I swipe the article list, if I then try and tap on an article during the bounce, it highlights the cell while my finger is touching the screen but doesn’t actually select it, so the result of tapping is for the cell to highlight momentarily and then revert back to the previously-selected cell. I can even hold my finger down for a moment and it stays highlighted, but then reverts when I release, as long as I didn’t hold it down long enough to trigger the contextual menu.
This is confusing and annoying. Either it should select the article despite bouncing (this is my preference), or it should avoid highlighting the cell if it’s not going to go ahead and select it.
Curiously, this bug only triggers if my scroll gesture ends before the article list starts bouncing. Which is to say, it has to be in the state where it’s inertial scrolling without dragging. This makes some sense, as when it’s inertial scrolling like this a tap doesn’t select anything it just stops the scroll, but it also doesn’t highlight the cell when stopping the scroll. It’s only when it’s bouncing after the inertial scroll that the tap highlights the cell without selecting. And really, it shouldn’t matter if I dragged into the bounce or if it was inertial scrolling before bouncing, either way tapping during the bounce should select the article.
Reproduced on iOS 14.0.1 with NNW 5.0.3 (Build 50)

This is the default UITableView highlighting behavior.  You can see it in action on Notes where the selection briefly flashes gray.  Interestingly, they seem to have overridden this behavior Mail, probably because it highlights blue and is more noticeable.
Changing VibrantTableViewCell.setHighlighted to an empty implementation is all we need to do to defeat the default UITableView highlighting behavior.
@brentsimmons Should we change this or stick with the default behavior?

I just tested in Notes and tapping during the bounce actually does select the note. Press and hold for a moment shows the grey selection and doesn’t select, but a normal quick tap does.

I must be doing something wrong. I can't get either Mail or Notes to select while the scroll view is bouncing.

Mail isn’t doing it for me, but Notes is. Also neither one of them have NNW’s behavior of highlighting immediately on touch down even if I’m attempting to scroll (NNW is somewhat inconsistent here but usually when I touch down it highlights the cell (and unhighlights the selected one) immediately even if I’m about to scroll).
If it makes a difference, I’m testing on iPad. It’s entirely possible iPhone has different behavior.
Also NNW will select the article if I drag into the bounce area and let go and then tap an article, but Mail doesn’t select anything in that scenario.

I'm able to reproduce with some consistency the highlighting immediately on touch down with Mail.  See the below video.
RPReplay_Final1601267633.mov.zip
Are you sure that this isn't a standard iOS behavior that you are only noticing in NetNewsWire because of the custom selection color?  We're just using a plain old UITableView.

I'm closing this since I think it is normal iOS system behavior.  @lilyball if you have more to add, we can reopen and investigate further.


updated launchcontrol (1.26.1)

Thank you for the contribution. It was merged directly as d34f5212b3716ba89c5279ba99b9027825b89248 to keep commit history cleaner. Your contribution is still credited to you.


HAWQ-35. Handle a case when a large tuple across split boundary

For large tuples in a randomly distributed table, if one large tuple is across the split boundary, then the execution would fail and generate core.
The fix here is to handle this specific case. More explanations please read the comments.

Looks fine.

The fix looks good. +1


[SPARK-10997] [core] Add "client mode" to netty rpc env.

"Client mode" means the RPC env will not listen for incoming connections.
This allows certain processes in the Spark stack (such as Executors or
tha YARN client-mode AM) to act as pure clients when using the netty-based
RPC backend, reducing the number of sockets needed by the app and also the
number of open ports.
Client connections are also preferred when endpoints that actually have
a listening socket are involved; so, for example, if a Worker connects
to a Master and the Master needs to send a message to a Worker endpoint,
that client connection will be used, even though the Worker is also
listening for incoming connections.
With this change, the workaround for SPARK-10987 isn't necessary anymore, and
is removed. The AM connects to the driver in "client mode", and that connection
is used for all driver <-> AM communication, and so the AM is properly notified
when the connection goes down.

Merged build triggered.

Merged build started.

/cc @rxin @zsxwing
I tested this on a real YARN cluster with dynamic allocation (= also used the external shuffle service), everything looks fine.

Test build #44110 has started for PR 9210 at commit 48f3c55.

Test build #44110 has finished for PR 9210 at commit 48f3c55.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class RegisteredExecutor(hostname: String) extends CoarseGrainedClusterMessage\n

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44110/
Test FAILed.

Merged build finished. Test FAILed.

retest this please

Merged build triggered.

Merged build started.

Test build #44127 has started for PR 9210 at commit 48f3c55.

Test build #44127 has finished for PR 9210 at commit 48f3c55.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class RegisteredExecutor(hostname: String) extends CoarseGrainedClusterMessage\n

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44127/
Test PASSed.

Just a nit. Otherwise LGTM

Merged build triggered.

Merged build started.

Test build #44185 has started for PR 9210 at commit da4d824.

Test build #44185 has finished for PR 9210 at commit da4d824.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class RegisteredExecutor(hostname: String) extends CoarseGrainedClusterMessage\n

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44185/
Test FAILed.

retest this please

Merged build triggered.

Merged build started.

Test build #44187 has started for PR 9210 at commit da4d824.

Test build #44187 has finished for PR 9210 at commit da4d824.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class RegisteredExecutor(hostname: String) extends CoarseGrainedClusterMessage\n

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44187/
Test PASSed.

Build triggered.

Build started.

Test build #44199 has started for PR 9210 at commit 11d9092.

Merged build triggered.

Merged build started.

Test build #44200 has started for PR 9210 at commit 418b472.

Test build #44200 has finished for PR 9210 at commit 418b472.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class RegisteredExecutor(hostname: String) extends CoarseGrainedClusterMessage\n

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44200/
Test PASSed.

Merged build finished. Test PASSed.

Test build #44199 has finished for PR 9210 at commit 11d9092.

This patch passes all tests.
This patch does not merge cleanly.
This patch adds the following public classes (experimental):\n  *   case class RegisteredExecutor(hostname: String) extends CoarseGrainedClusterMessage\n

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44199/
Test PASSed.

Build finished. Test PASSed.

Merged build triggered.

Merged build started.

Test build #44298 has started for PR 9210 at commit acce31f.

Test build #44298 has finished for PR 9210 at commit acce31f.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class RegisteredExecutor(hostname: String) extends CoarseGrainedClusterMessage\n

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44298/
Test PASSed.

hi there, any remaining feedback here?

retest this please

Merged build triggered.

Merged build started.

Test build #44778 has started for PR 9210 at commit acce31f.

Test build #44778 has finished for PR 9210 at commit acce31f.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class RegisteredExecutor(hostname: String) extends CoarseGrainedClusterMessage\n

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44778/
Test FAILed.

hmm, akka test... it's failed for me in unrelated changes, also. retest this please.

Merged build triggered.

Merged build started.

Test build #44787 has started for PR 9210 at commit acce31f.

Test build #44787 has finished for PR 9210 at commit acce31f.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):\n  *   case class RegisteredExecutor(hostname: String) extends CoarseGrainedClusterMessage\n

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/44787/
Test PASSed.

Merged build finished. Test PASSed.

Merging this to master.

Just took another look. LGTM


Add missing tooltips in the report

There are some missing tooltips in the report:

Go to next version,
Go to previous version,
Previous url,
Next url.

The following tooltips are still missing:
Previous url,
Next url.
version: 3.3.1.SNAPSHOT

Tested and accepted
(during previous tests, app was only partially setted up)
version: 3.3.1.SNAPSHOT


Suppress x-enumNames in swagger json

Is there an option to suppress the x-enumNames content from the swagger model?  This content causes AWS API Gateway to have a fit.

You can implement a custom ISchemaProcessor and remove these in each generated schema

Thanks, will do.

You can implement a custom ISchemaProcessor and remove these in each generated schema

This works for runtime purposes, do you have any suggestions when using Nswag.Msbuild ?  This is how my documentation is generated.

In v12, this is the recommended way for CLI: Create nswag.json with NSwagStudio (see screenshot) and run via CLI, https://user-images.githubusercontent.com/2603405/47795762-5d143780-dd23-11e8-849a-a56d4b1dbd42.png

Does this mean NSwag.MSBuild is obsolete?  I have opted for code-first swagger.json generation.

Nope, just run the config file with NSwag.MSBuild instead of specifying everything inline

Nope, just run the config file with NSwag.MSBuild instead of specifying everything inline

You mean this config file, right? https://github.com/RSuter/NSwag/wiki/NSwag-Configuration-Document

Yes exactly this... but the sample in the wiki might be outdated, create one with studio...
This screenshot shows the most recent recommended flow (middleware + config):
https://github.com/RSuter/NSwag/pull/1655

Yes exactly this... but the sample in the wiki might be outdated, create one with studio...
This screenshot shows the most recent recommended flow (middleware + config):
#1655

I see the command line supports DocumentProcessors and OperationProcessors, but not SchemaProcessors (ISchemaProcessor) ... is the workaround to use a DocumentProcessor?

I think its just missing, but with document provider the cli just uses the config in AddSwaggerDocument()


[APIs Zoom Admin Access] Assignment

https://docs.google.com/spreadsheets/d/1bN00rsPpw54CnCjNHHdrty0bvSA5vi1ei7TDXSOf89o/edit#gid=1206744069

I have contacted every group via agenda minutes today and updated each ticket.

individually tackled by gits :)


Badges?

Maybe a single combined badge that shows the "worst" result?
FWIW here is some example badge generation code. Different language, but maybe still useful:
https://github.com/metacran/metacranweb/blob/8e930ab92c2f73c0cef4c3f64cf9ae208aa0bb9f/routes/badges.js

Good idea.  Similarly, I need to add https://github.com/ropensci/cchecksapi/issues/12 in the JSON response.
your JS code seems simple enough. thanks!

Super cool! I would just have "CRAN" as the badge label, because I think  "Cchecks" is a bit mystical.
Can we also have a badge that is just [CRAN|OK], [CRAN|NOTE], etc. in the right color? Or that's the "worst" badge maybe?
Personally,  I think the flavors  are not super important, especially because occasionally they change, and then it is hard to create a stable badge URL. Maybe some meta flavors, like "Windows", "Linux", "macOS" and "Solaris" would make sense? I.e. "Linux" would show the worst outcome of any Linux flavors, etc.
For linking, I think people could link to the main cran page, or the cran check page, maybe?
Btw. it would be great to be able to ignore some check results, because  e.g. the processx NOTE will never be fixed, and the package is perfectly fine. I am not sure what cchecksapi can do here, just bringing it up...

Okay, was thinking it should be set apart from yours but if you're okay with that, then i can do that.

Can we also have a badge that is just ...

Yeah, I think that's what I had in mind for the /badges/worst/:package route. Sound good?
I agree that meta-flavors might be useful. can do that instead
Will think more about where to link

it would be great to be able to ignore some check results

maybe via a query parameter?  e.g., ?ignore=r-oldrel-osx-x86_64 or ?ignore=osx

Okay, was thinking it should be set apart from yours but if you're okay with that, then i can do that.

Yeah, it will set apart, because one is a version number, the other is OK, NOTE, etc.

Yeah, I think that's what I had in mind for the /badges/worst/:package route. Sound good?

Yep!

maybe via a query parameter? e.g., ?ignore=r-oldrel-osx-x86_64 or ?ignore=osx

I was actually thinking about some whitelisting based on the actual failure. Because e.g. here I get a  lot of NOTEs just for the package size: https://cran.rstudio.com/web/checks/check_results_igraph.html
This is not trivial to implement of course, I think for now, maybe a query parameter to badges/worst/:package that just ignores all NOTEs would be good enough. I think a WARNING is usually not sg you would want to ignore.

Okay, updated, any other thoughts before I merge to master?

Super cool! Starting to add it to READMEs right now!


/opt/gtk/share/gir-1.0/Gst-1.0.gir file should contain the info that we have to free resources, and the gen.nim script should pick that info up by use of the gobject-introspection API functions and generate code in the gst.nim module to free resources. But that does not really work currently.

Gst-1.0.gir file should contain the info that we have to free resources

It does not because the API doesn't and can't follow the standard free()-style conventions.

Gst-1.0.gir file should contain the info that we have to free resources

It does not because the API doesn't and can't follow the standard free()-style conventions.

I think this makes the generator sad. Is there any machine-readable info that can be use to tackle the problem?

I think this makes the generator sad. Is there any machine-readable info that can be use to tackle the problem?

No, you'd have to introduce new annotations for these kind of things or bind them manually.

No, you'd have to introduce new annotations for these kind of things or bind them manually.

That seems to be really not easy.
https://gstreamer.freedesktop.org/documentation/gstreamer/gstbuffer.html?gi-language=c#gst_buffer_unmap
gst_buffer_unmap() gets as argument not only the buffer but also a MapInfo. From the API it is not really clear, but I assume that the MapInfo have to be the one which was used for the gst_buffer_map(). Handling this automatically seems to be really hard. Mr. Rumpf may have ideas, but I will not ask him, he does not like GTK that much. Of course we can export the low level function gst_buffer_unmap() somehow and allow manually calling it, but that way we loose the advantages of high level languages. I had always doubts if using gstreamer from Nim makes much sense. I added it because someone asked for it years ago. That person never came back. Maybe using gstreamer from plain C or Rust just makes more sense. When you want to continue with Nim: You can simple edit the file ~/.nimble/pkgs/gintro-#head/gintro/gst.nim, add an asterics to gst_buffer_unmap() so that you can call it from your own module and also modify other code of that module. When you have found a manually fix, we can see if we can automate it. It depends how buffer.map() is generally called. When it is always called like in your example after "var buffer: gst.Buffer = sample.getBuffer" then we may find a way, but for arbitrary calls it seems to be difficult.

That seems to be really not easy.
https://gstreamer.freedesktop.org/documentation/gstreamer/gstbuffer.html?gi-language=c#gst_buffer_unmap
gst_buffer_unmap() gets as argument not only the buffer but also a MapInfo. From the API it is not really clear, but I assume that the MapInfo have to be the one which was used for the gst_buffer_map(). Handling this automatically seems to be really hard. Mr. Rumpf may have ideas, but I will not ask him, he does not like GTK that much. Of course we can export the low level function gst_buffer_unmap() somehow and allow manually calling it, but that way we loose the advantages of high level languages. I had always doubts if using gstreamer from Nim makes much sense. I added it because someone asked for it years ago. That person never came back. Maybe using gstreamer from plain C or Rust just makes more sense. When you want to continue with Nim: You can simple edit the file ~/.nimble/pkgs/gintro-#head/gintro/gst.nim, add an asterics to gst_buffer_unmap() so that you can call it from your own module and also modify other code of that module. When you have found a manually fix, we can see if we can automate it. It depends how buffer.map() is generally called. When it is always called like in your example after "var buffer: gst.Buffer = sample.getBuffer" then we may find a way, but for arbitrary calls it seems to be difficult.

Well, just calling the available unmap() function may work:
salewski@nuc ~ $ grep -A9 "proc gst_buffer_unmap" ~/.nimble/pkgs/gintro-#head/gintro/gst.nim
proc gst_buffer_unmap(self: ptr Buffer00; info: MapInfo) {.
    importc, libprag.}

proc unmap*(self: Buffer; info: MapInfo) =
  gst_buffer_unmap(cast[ptr Buffer00](self.impl), info)

But of course having to know that one has to call it is not really nice.

Well, just calling the available unmap() function may work:
salewski@nuc ~ $ grep -A9 "proc gst_buffer_unmap" ~/.nimble/pkgs/gintro-#head/gintro/gst.nim
proc gst_buffer_unmap(self: ptr Buffer00; info: MapInfo) {.
    importc, libprag.}

proc unmap*(self: Buffer; info: MapInfo) =
  gst_buffer_unmap(cast[ptr Buffer00](self.impl), info)

But of course having to know that one has to call it is not really nice.

Well, just calling the available unmap() function may work:

yeah, see the update of my previous https://github.com/StefanSalewski/gintro/issues/108#issuecomment-761927767, I already measured that that fixed the leak, no need to edit generated nim files.
As there's no possibility to add automatic memory management for these special cases, it would still be ok as long as it follows gstreamer documentation. Problem is just separating what will be automatic and what will not, but now I know that I can look at free() convention in gir file that means gintro generator can use it (wonder if they are all ref/unref cases).
gstreamer is already working in nim, pipeline can be launched and data can be passed in and out gstreamer within nim process, and that covers a lot of cases! Also users can now handle multimedia inside of their gtk apps sharing glib main loop.

Well, just calling the available unmap() function may work:

yeah, see the update of my previous https://github.com/StefanSalewski/gintro/issues/108#issuecomment-761927767, I already measured that that fixed the leak, no need to edit generated nim files.
As there's no possibility to add automatic memory management for these special cases, it would still be ok as long as it follows gstreamer documentation. Problem is just separating what will be automatic and what will not, but now I know that I can look at free() convention in gir file that means gintro generator can use it (wonder if they are all ref/unref cases).
gstreamer is already working in nim, pipeline can be launched and data can be passed in and out gstreamer within nim process, and that covers a lot of cases! Also users can now handle multimedia inside of their gtk apps sharing glib main loop.

I have just added more gst modules:
gstallocators.nim
gstapp.nim
gstaudio.nim
gstbase.nim
gstcheck.nim
gstcontroller.nim
gstgl.nim
gstinsertbin.nim
gstmpegts.nim
gstnet.nim
gst.nim
gstpbutils.nim
gstplayer.nim
gstrtp.nim
gstrtsp.nim
gstsdp.nim
gsttag.nim
gstvideo.nim
gstwebrtc.nim

You may try with
nimble install gintro@#head

Was not too easy, but now they compile at least. And we have added the gst_init() call which seems to be necessary before we access gst from gobject-introspection, see https://discourse.gnome.org/t/glib-critical-for-use-of-gobject-intrspection/5402

I have just added more gst modules:
gstallocators.nim
gstapp.nim
gstaudio.nim
gstbase.nim
gstcheck.nim
gstcontroller.nim
gstgl.nim
gstinsertbin.nim
gstmpegts.nim
gstnet.nim
gst.nim
gstpbutils.nim
gstplayer.nim
gstrtp.nim
gstrtsp.nim
gstsdp.nim
gsttag.nim
gstvideo.nim
gstwebrtc.nim

You may try with
nimble install gintro@#head

Was not too easy, but now they compile at least. And we have added the gst_init() call which seems to be necessary before we access gst from gobject-introspection, see https://discourse.gnome.org/t/glib-critical-for-use-of-gobject-intrspection/5402

Great! Been away for a while as I've decided to stop learning nim while testing gintro+gstreamer to dive into nim book/manual first. I'll come back testing this asap.


Batch Edit - Adding to project doesn't work

From https://forum.inaturalist.org/t/batch-edit-add-to-project-doesnt-work/26947 I can replicate.

Go to batch edit screen and turn on batch editing
Select some observations and add them to a project (eg https://www.inaturalist.org/projects/organisms-on-or-near-appropriate-signs)
See a brief spinner next to to the project title and then you're taken to your dashboard
Observations are never added to the project

Fix ready to review and merge. Apologies to those whose workflow I broke for the last couple of weeks.


Any non elastic/linear easing's do not work

Any of the easing functions do not work.
<AnimatedNumber value={20} easing={'easeInQuart'} formatValue={(value) => {
    console.log(value)
    return value;
}}/>

Received value on formatValue is "0", during the animation it is "NaN" and at the complete it is "20".

Does anyone have the answer?

This is because the author is using a static version of anime.js which at the time only had
'Quad', 'Cubic', 'Quart', 'Quint', 'Sine', 'Expo', 'Circ', 'Back', 'Elastic'
rather than the modern versions that contain much more than that.

Seems like only following works as of now linear, easeInElastic, easeOutElastic, easeInOutElastic.


how to control lv_indev_t *mouse without touchpad to move and click?

my ili9341 tft is not touch version.
i want to control mouse by two adc as intput x,y.
this is my mouse init code, it will show x icon as cursor .
lv_indev_t *mouse = lv_indev_get_next(NULL);
cursor = lv_label_create(lv_scr_act(), NULL);
lv_label_set_recolor(cursor, true);
lv_label_set_text(cursor, "x");
lv_indev_set_cursor(mouse, cursor);
the lcd show x cursor icon.
when i use this api, it looks just move the x cursor icon not move mouse last position?
lv_obj_set_pos(cursor, adc1, adc2);
and then call click api. the lvgl just crash and reboot.
const lv_point_t points_array[] =  { {40, 40}};
lv_indev_set_button_points(mouse, points_array);
does anyone know how to control mouse without touchpad?

LVGL should move the cursor itself if you supply the correct input data in the input device's read_cb function. Please note that you need to use an input device of type LV_INDEV_TYPE_POINTER for the cursor to function correctly.

We use GitHub issues for development related discussions.
Please use the forum to ask questions.
It's possible to sign in to the forum with your GitHub account... you don't need to make a separate account.

LVGL should move the cursor itself if you supply the correct input data in the input device's read_cb function. Please note that you need to use an input device of type LV_INDEV_TYPE_POINTER for the cursor to function correctly.

We use GitHub issues for development related discussions.
Please use the forum to ask questions.
It's possible to sign in to the forum with your GitHub account... you don't need to make a separate account.


Safari 10.0 doesn’t support minlength validation

We should probably not expose the native minlength property, as Safari 10.0 doesn’t natively produce validation errors for that. We should instead encourage users to use pattern instead.
This has been fixed already in Safari Technology Preview.
Related: maxlength works in Safari 10.0 also, it prevents the user from inputting more characters.
Also, it seems that minlength and maxlength are only evaluated for user input, and they don’t affect the validation unless the user has touched modified the input. The pattern property, on the other hand, is validated already when the element is loaded/rendered.

Looks like it's fixed in 10.3, closing.


reconstructmesh problem

Hi everybody,
I am wondering that, is this repository is up to date? I mean;
-Was the framework binary file in [StandardCyborgCocoa] repository built from [StandardCyborgSDK]?
because in my projects when I used the framework binary file in [StandardCyborgCocoa] repository, All goes well. But when I use the framework binary which is archived and built by me from [StandardCyborgSDK] my application commonly crashes at reconstrouctmesh phase.
framework from [StandardCyborgCocoa]
ios-arm64(from standardcyborgcocoa).zip
framework from built by me
ios-arm64(built by me).zip

The framework binary in -Cocoa is indeed built from -SDK, and it is up to date.
Can you plase share the crash you’re seeing?

hi aaptho thanks for your reply. here are the errors with from truedepthfusion app with SDK

When I change the framework to in -Cocoa  repository. all goes well as I said

ps:
Also I wanna take your attention to another point.
when I checked the header files of framework in Cocoa. there are 2 files;
Only in ios-arm64/StandardCyborgFusion.framework/Headers: SCFaceLandmarking3D.h
Only in ios-arm64/StandardCyborgFusion.framework/Headers: SCFaceTracking.h
these files are not in -SDK repository.

This is failing an assertion within igl in igl::min_quad_with_fixed_precompute.

This assertion within MeshTopology::compute also fails
SCASSERT(_edges[edgeCA].face1 == -1, "Non-manifold edge found in mesh topology”);
Repros 100% when using TrueDepthFusion to take a scan, then tapping Mesh.
@erkaman any ideas?

when I checked the header files of framework in Cocoa. there are 2 files;
Only in ios-arm64/StandardCyborgFusion.framework/Headers: SCFaceLandmarking3D.h Only in ios-arm64/StandardCyborgFusion.framework/Headers: SCFaceTracking.h
these files are not in -SDK repository.

These files should have been removed earlier. They are now deleted from StandardCyborgFusion, and will be removed from the published framework with the next build.

any progress or idea to solve the issue?

Hello,
Exactly the same problem here.
What xCode version are you using to build the SDK framework?
For my part, I use xCode 14.3, and I have the same crash on mesh reconstruction.
But, I build the SDK the last year, with xCode 14.1 and it worked fine.
Maybe the C++ compiler was changed between xCode version?

Hi,
I build with 14.3, and I think you may get the point. in 14.3 relase notes clang changed
Apple Clang Compiler
New Features
Clang and the build system support a new mode for building module dependencies called explicit modules which improves build performance, reliability, and correctness. The new mode is opt-in, and can be enabled by setting _EXPERIMENTAL_CLANG_EXPLICIT_MODULES as a user-defined build setting in C and Objective-C projects which build with modules enabled. (104438594)
The following C++23 features have been implemented:
New support for multidimensional subscript operator. (P2128R6)
Relaxed restrictions on the presence of non-literal variables (and labels and gotos) in constexpr functions (P2242R3).
Introduce auto(x) for decay-copy in the language (P0849R8). (104887755)

now I tried to bulid with xcode 14.1 and yes it works. it is clear that problem is xcode 14.3 Apple Clang Compiler. needs a configuration or code update.

I have just updated the SCCocoa pod that includes the new version of SCFusion 2.3.3 and managed to build successfully but then when installed on device I get what seems to be a similar crash/bug with SCMeshReconstruction or SCMeshingHelper. I have built and installed previously in June using Xcode 14.3 I believe and it worked fine. I am still using 14.3 to install this version of the app.

Oh, I see what you mean. You think when the framework was updated and compiled it was with a different version of Xcode and thus won’t work?

From: Termoraq @.>
Sent: Tuesday, August 15, 2023 2:00 pm
To: StandardCyborg/StandardCyborgSDK @.>
Cc: Joshua Shires @.>; Mention @.>
Subject: Re: [StandardCyborg/StandardCyborgSDK] Assertion failure when reconstructing mesh (Issue #12)
@scratchthatitchhttps://github.com/scratchthatitch
it is not about your xcode version.
2 weeks ago, framework of StandardCyborgCocoa is updated. I guess it was built with xcode 14.3, that is why you are getting this error.
—
Reply to this email directly, view it on GitHubhttps://github.com/StandardCyborg/StandardCyborgSDK/issues/12#issuecomment-1678897135, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANS2IXZYRBZHYD7MKP2MKUDXVNXGLANCNFSM6AAAAAAZGMXA5M.
You are receiving this because you were mentioned.Message ID: @.***>

Oh, I see what you mean. You think when the framework was updated and compiled it was with a different version of Xcode and thus won’t work?
…
________________________________ From: Termoraq @.> Sent: Tuesday, August 15, 2023 2:00 pm To: StandardCyborg/StandardCyborgSDK @.> Cc: Joshua Shires @.>; Mention @.> Subject: Re: [StandardCyborg/StandardCyborgSDK] Assertion failure when reconstructing mesh (Issue #12) @scratchthatitchhttps://github.com/scratchthatitch it is not about your xcode version. 2 weeks ago, framework of StandardCyborgCocoa is updated. I guess it was built with xcode 14.3, that is why you are getting this error. — Reply to this email directly, view it on GitHub<#12 (comment)>, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ANS2IXZYRBZHYD7MKP2MKUDXVNXGLANCNFSM6AAAAAAZGMXA5M. You are receiving this because you were mentioned.Message ID: @.***>

yes

@aaptho would we be right in assuming this?

@termoraq2 Just to confirm. You have been able to build and install and test on device the latest SCSDK 2.3.3 using Xcode 14.1 but not 14.3? Is that correct?

@termoraq2 Just to confirm. You have been able to build and install and test on device the latest SCSDK 2.3.3 using Xcode 14.1 but not 14.3? Is that correct?

Yes I build standardcyborgsdk using xcode 14.1, I build and install my own app with this sdk and it works. Also I build standardcyborgsdk using xcode 14.3.1, I build and install my own app with this sdk and it crashed. I think problem is xcode 14.3 c++ compiler.

Great news—this is now fixed with https://github.com/StandardCyborg/StandardCyborgSDK/pull/15!
With this fix, it shouldn’t matter anymore what version of Xcode you are using to build either the SDK or your app.
The fix is in StandardCyborgFusion version 2.3.6, now published to CocoaPods.


Build of IC3 failed

Hi,
i have followed instructions from http://siis.cse.psu.edu/ic3/source.html in order to build IC3, but I got a following exception while building:
[INFO] ------------------------------------------------------------------------ [INFO] BUILD FAILURE [INFO] ------------------------------------------------------------------------ [INFO] Total time: 16.739s [INFO] Finished at: Mon Jun 15 16:39:45 CEST 2015 [INFO] Final Memory: 6M/240M [INFO] ------------------------------------------------------------------------ [ERROR] Failed to execute goal on project ic3: Could not resolve dependencies for project edu.psu.cse.siis:ic3:jar:0.1.1-SNAPSHOT: Could not find artifact edu.psu.cse.siis:coal:jar:0.1.7-SNAPSHOT in soot-repo (https://raw.github.com/siis/soot/mvn-repo/) -> [Help 1]
Can you help me with this?
Thank you in advance,
Vitalii

Hi,
The current version of the IC3 code is meant to work with a development version of COAL. You can checkout the latest tag (currently v0.1.0) of IC3 using git before building (git checkout v0.1.0).
Alternatively, at any other time you could checkout projects coal-strings and coal from our repository and run mvn install on them before trying to build IC3, but I am in the middle of a big update and versions are not currently compatible. That should be fixed in the next couple of days when I push an update of IC3 compatible with the latest COAL, in case you want to check the latest revision.

Hi,
I have moved the latest commits to a new "develop" branch. You should now be able to just follow the instructions from the IC3 webpage to build the source code.

Hi Damien,
thanks a lot!
Now everything works fine.
Best Regards,
Vitalii


fix: remove hardcoded texts

Reported hard coded texts have been replaced. I've spent a couple of hours checking for more hard coded texts but I couldn't find them.
I've also reviewed some italian and german translations and proposed new translations. I've used google translate so I hope they're good to go. I'll change any wrong translation after your review.
Closes #5210
### Definition of Ready
- [X] I am happy with the code
- [X] Short description of the feature/issue is added in the pr description
- [X] PR is linked to the corresponding user story
- [X] Acceptance criteria are met
- [ ] All open todos and follow ups are defined in a new ticket and justified
- [ ] Deviations from the acceptance criteria and design are agreed with the PO and documented.
- [ ] No debug or dead code
- [ ] Critical parts are tested automatically
- [ ] Where possible E2E tests are implemented
- [ ] Documentation/examples are up-to-date
- [ ] All non-functional requirements are met
- [X] Functionality of the acceptance criteria is checked manually on the dev system.

@doncicuto Thanks for the contibution.
@peintnermax can you have a look at it?

I think its ok to use the english terms in some use cases. For example it does not make sense to translate Events and Views as these are strongly bound to our eventstore and documentation. Same for Personal Access Token @eliobischof do your have other opinions?

I second that.

Thanks for your review @peintnermax and @eliobischof, as stated earlier I use Google Translate so I apologize for incoherent texts, hope that this PR is still useful after all

of course its useful.. LGTM right now.. there are just some conflicts 👍

@doncicuto can you have a look at the conflicts?

Hi @peintnermax and @adlerhurst I've fixed pending conflicts, hope that PR is ready to merge


misc: add build matrix for pytorch and cuda version

Checklist

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

detectron2 is tightly coupled to both cuda and pytorch versions. An overview of compatibility is provided by detectron2 in the documentation. The official distribution of detectron2 builds wheels for a combination of versions for these packages. This PR adds a build matrix for all supported pytorch and cuda versions.
With this PR this should also be possible for the conda distribution of detectron2. This allows detectron2 to be used with a wide(er) variety of pytorch and cuda versions within Conda.

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

@baszalmstra Thank you!


Large file size Video not compressing

Current Behavior

The library is working fine for smaller size of videos. If I try to compress a larger size video ( 317 MB ) in my case.
It doesn't respond.

Expected Behavior

Should compress the larger files, that is where the compression is needed the most.

Your Environment

software
version

react-native-video-processing
Bunhouth/react-native-video-processing

react-native
^0.61.2

node
v12.4.0

@ahmadzahidglixen
Getting the same error, did you find any solution or root cause?

@walvekarnikhil no, I haven't found any root cause of behaviour.
Had to switch change library for compressing videos.
Now i'm using:
https://www.npmjs.com/package/react-native-audio-video-tools
that uses ffmpeg at the core.


Laravel vite with vite-pwa-plugin

Laravel Vite Plugin Version: 0.7.3
Laravel Version: 9.19
Node Version: 18.12.1
NPM Version: 9.2.0
Host operating system:  Ubuntu 22.04.1 LTS
Web Browser & Version: All
Running in Sail / Docker: No

Description:
I just finished converting a PWA from laravel mix to the vite plugin - everything was pretty straightforward, but the pwa isn't caching for offline use.
you can see the vite setup here https://github.com/el-coop/app/blob/master/vite.config.js
the reason I'm making it build into the public directory and not into the build directory is because the pwa plugin doesn't include the "build" prefix in the cache manifest  (so it shows:   re([{"revision":null,"url":"assets/_commonjsHelpers-edff4021.js"},{"revision":null,"url":"assets/_plugin-vue_export-helper-9d2f2293.js"},
anyway, now the service worker works, but it never caches for offline use - I recently updated a different app similarly (but non-laravel app) and there the caching works...
Steps To Reproduce:
Create a laravel/vue app and copy the next files
viteconfig:
https://github.com/el-coop/app/blob/master/vite.config.js
serviceWorker:
https://github.com/el-coop/app/blob/master/resources/js/serviceWorker/serviceWorker.js
registerServiceWorker:
https://github.com/el-coop/app/blob/master/resources/js/serviceWorker/registerServiceWorker.js
view:
https://github.com/el-coop/app/blob/master/resources/views/layouts/plain.blade.php
make some entry point for the app and include the registerServiceWorker file in it
You will see there is no "Content has been cached for offline use." console log - and no way to load the app when offline...
Also you can see this console log doesnt appear on the following url app.elcoop.io
thanks!
EDIT
never mind - got it working.  its been a long day haha

@LCD344 what was the solution for the issue?

@LCD344 what was the solution for the issue?

I disabled the build folder - you can see in the files I linked - specifically in thee vite.config.js - for a working version.

@LCD344 what was the solution for the issue?

I disabled the build folder - you can see in the files I linked - specifically in thee vite.config.js - for a working version.

Thanks


fix: more precise wildcard to match wal file

close: #4267

the e2e from fork https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/8682774868

https://github.com/EnterpriseDB/cloudnative-pg/actions/runs/8682774868 e2e is run here

/test depth=push feature_type=backup-restore

/ok-to-merge test failed but not related to the touch that was touched


Support interactive environment in PyCharm where there is no source file

For my workflow, the greatest problem with this package at the moment is that it throws an error when running handout.Handout() in an interactive shell:
handout.Handout('/tmp')
Traceback (most recent call last):
  File "/h/hannes/anaconda3/envs/microexon-code-tf2/lib/python3.6/site-packages/IPython/core/interactiveshell.py", line 3325, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-4-d909ee1709cf>", line 1, in <module>
    handout.Handout('/tmp')
  File "/h/hannes/anaconda3/envs/microexon-code-tf2/lib/python3.6/site-packages/handout/handout.py", line 25, in __init__
    self._source_text = inspect.getsource(module)
  File "/h/hannes/anaconda3/envs/microexon-code-tf2/lib/python3.6/inspect.py", line 973, in getsource
    lines, lnum = getsourcelines(object)
  File "/h/hannes/anaconda3/envs/microexon-code-tf2/lib/python3.6/inspect.py", line 955, in getsourcelines
    lines, lnum = findsource(object)
  File "/h/hannes/anaconda3/envs/microexon-code-tf2/lib/python3.6/inspect.py", line 768, in findsource
    file = getsourcefile(object)
  File "/h/hannes/anaconda3/envs/microexon-code-tf2/lib/python3.6/inspect.py", line 684, in getsourcefile
    filename = getfile(object)
  File "/h/hannes/anaconda3/envs/microexon-code-tf2/lib/python3.6/inspect.py", line 666, in getfile
    'function, traceback, frame, or code object'.format(object))
TypeError: None is not a module, class, method, function, traceback, frame, or code object

This is a problem for my (Jupyter-less) workflow as I like to use #%% in PyCharm to create run-able cells and send them to a shell with Ctrl+Enter. This way I can create my figures and do my analysis interactively and simply run the script at the end to obtain a rendered report. However, the error above throws a wrench into that workflow.
I think a workable simple solution would be for handout to simply do nothing when it detects an interactive shell.

Thanks for reporting! I'm wondering what source file you'd expect the handout to be built from? If there is no source file, there won't be any comment cells or code cells. In principle, it would still be possible to allow to add media to a doc if this is a reasonable use case.

Not for immediate implementation, but just to consider a workflow:
# a user creates an `InteractiveHandout` instance
idoc = InteractiveHandout('/tmp')
# alternative:
idoc = Handout('/tmp', interactive=True)
# user adds text, graphs, video, etc in interactive session calls
idoc.add_text("Print this text")
# a user creates an an artefact
idoc.show()

This way there is no source to build from, but a user can try different things in console before writing a script to persist the report. Is this something close to your workflow, @hannes-brt?
I like the idea because it gives more food for thought for #9 on structuring the classes for different sources and outputs. This workflow suggests there may be no source as text file, just calls to handout instance.

I don't necessarily think the package needs to work in terms of producing a report in a shell, but at least it shouldn't raise errors. I think the closest model from existing packages is pypublish from Pweave (http://mpastell.com/pweave/pypublish.html). Since it only uses special comment syntax to define markdown text, the code can run in a shell, as a script (python myscript.py), or in pypublish (pypublish myscript.py). The first two don't produce a report, but still run the code which you sometimes want to do. Pypublish has some problems of its own and Pweave is no longer being developed, so I'd hope handout can step into that gap, but I think that it's important running the code in a shell doesn't break things.
@epogrebnyak I don't think it's good to introduce different options or objects for interactive mode, because that again requires changing the code when running in different context. I think the simplest and easiest solution is for handout to simply do nothing when an interactive shell is detected.
@danijar Since you're local - I'd be happy to swap some ideas if you're ever at the Vector Institute.

There a lot of good points here. Instead of an interactive mode, we could have a flag to not include any source code. That would work in all environments and would also cover an additional use case people have asked for. Would that be suitable here?
@epogrebnyak Yes, let's think about making the main class independent of the input and output types. Can you think of any inputs other than "current file" and "nothing"?
@hannes-brt I'm happy to chat, could you shoot me an email so we can coordinate a time, please?

Under the hood in Handout class we merge a chain of blocks derived from static analysis of a source file and a chain of blocks derived from add_x() сalls, in principle they can be separated. Answering @danijar question - for the source file there is either current file or nothing that can be used.

@epogrebnyak Do you think they we should separate them?


project logo

Eventual plan is to create a new org to host these packages, models, and we want a logo!

Cool!

cataloguing some general concepts:

something cutesy involving a camel since "CheML" kind of sounds like that if you try to pronounce it 🐫 🐪
Anant had a cool idea about a typical Bohrian atom diagram (see e.g. this) but with the electrons as the Julia colors (if we go with this concept we should be careful to be quite distinct from the Juno logo...
Perhaps we could combine the above two with a stylized "CheML" in the shape of a two-humped camel (the Ch in on and ML in the other) as the "nucleus" of the atom?

@cormullion had shared some concepts on the Slack that are nice starts but I worry they look a bit too "experimental chemistry" focused...

Yes, it's going to need steering away from familiar chemistry iconography, but at least I've got it out of my head now! 😀

I've not yet managed to make a convincing camel...
Meanwhile, some kind of machine learning-y vibe with Julia coloration... I thought adding the name would be useful in the early days - perhaps there is room for sub-project names there...

Pity about losing the flasks -  such a strong shape!

Something to be aware of:
https://github.com/hachmannlab/chemml
https://hachmannlab.github.io/chemml/

Yes, too good an idea to be original... :(

Interestingly, its author was our software fellow at MolSSI as well.

Ahh thanks for catching that, Sina, good to know. I'll consult with the other folks on the project again and see if there's a consensus "second-choice" name...

What does 'flux' mean, in this context?

Ahh thanks for catching that, Sina, good to know (and funny coincidence!). I'll consult with the other folks on the project again and see if there's a consensus "second-choice" name...my current leaning is toward something like ChemFlux, for the parallelism with DiffEqFlux, etc.

If I'm not wrong, DiffEqFlux  is named so because it "utilizes DifferentialEquations.jl and Flux.jl as its building blocks".
Wouldn't something as significant as the entire project's name sound better if it wasn't a nod of the dependencies and frameworks it uses?
Just my two cents.

Naming things is hard! It's always nice if the name (and iconography) can convey something the project, but in reality there's a limited number of ways you can twist groups letters to get completely original yet meaningful names. Me, I quite like the idea of names that have nothing much to do with what they're about; like Kubernetes or Hadoop.
Being original might not matter too much - as long as there's some kind of limited uniqueness in a domain. So for example you could probably call a shampoo and a software framework the same thing, but probably  shouldn't use the same name for a physics-related project and a chemistry-related one.
Searching github for project names is a quick way to check out the possible competition. There are >4000 repos with "alchemy" in the name, but if none of them are in the same field, it sounds like a nice place to start from.

camelia and camellia have quite a few repo hits, chamomile has a few, camomile a few more. chamelia, chemelia, and chemalia don't, though. Perhaps there's some flowery image in there, too...

Okay, do people think AIChemy (with an upper-case i) is mainly confusing or mainly clever, especially given the prevalence of sans serif fonts...?

Another idea: Chemeleon

It was just pointed out that one would rarely if ever actually have to type AIChemy since it's the name of an org, not a package, so the AI could be called out by a different color in a logo...I'm now leaning somewhat towards that, though Chemelon does hold some charm too...

Zomg that chameleon is cute but the team seems to be leaning towards Alchemy spelled with an L but some logo that calls out that it looks like "AI" (if you're in a sans-serif font)

I think I see what you mean:

It could be worth working up...

Oooh this is a cool concept!

Assuming you made this in Luxor, would I be able to get the script to play with? I'd of course still credit you in whatever final version we come up with.
Alternatively, here are some variations I'm contemplating:

while obviously there should be some gold in the logo, I'm not sure I necessarily like this much...I wonder if there's a way to tone it down a bit, maybe a little "token"-like image somewhere, or even a gold-colored atom cartoon...
I'm curious also about doing a sans-serif font and calling out the "AI" with color instead (could even by something like AI in red, "chem" in green, and "y" in purple perhaps?)

Final question: I'm about to submit a JuliaCon proposal about this work, would you mind if I put one of these as a tentative session image? I've been assured it can be changed later in the final program assuming the talk is accepted and we've iterated on the logo by then...

Okay, so turns out there are in fact a few packages/datasets with the name Alchemy and/or Alchemist that are close enough to our application that I'm worried about the ambiguity...so it's back to the drawing board...some other ideas, some pedestrian and others a little more "out-there"...

ChemFlux, as we discussed before (I still think this is a solid option, as do several collaborators)
CaraMeLize or similar (because onions, like ML models, have layers...)
Chemeleon (also discussed above)
Another broad concept is something to do with differentiability, since differentiating through the featurization is something we eventually plan to have as a distinguishing feature. To that end, one sort of idea is playing on the nabla symbol for the vector derivative coming from the word for harp and maybe some name related to that...but I don't have any specific ideas on that front.

I'm bummed about this because I spent a bunch of time yesterday playing around in Luxor and came up with a concept I really liked for a logo...

Ah well.

I like(d) it! Add it to your "great ideas whose time is yet to come" folder! 😄

Playing around with camellia-type motifs, mainly inspired by this image: http://verdego.com/product/camellia-japonica-pink-perfection/
Here's what I've come up with so far:

Code:
@draw begin
    origin()
    background("green")
    local rad = 80
    local counter = 1
    for pt in reverse(spiral(15, 1.3, period=10pi, stepby=0.5))
        blendstart = between(O,pt,sqrt(rad/160))
        blendend = between(O,pt,1.2+counter^0.8/90)
        #println(round.([rad/160, 1.3+counter/90], digits=2))
        setblend(blend(blendstart, blendend, "violet","lightgrey"))
        if rad > 30
            rad = rad-1.3
        end
        circle(pt, rad, :fill)
        counter = counter + 1
    end
end

I'm not totally satisfied with the middle part yet (I think I probably need shapes other than circles), nor am I completely sure how/if I would want to integrate any lettering, but a proof-of-concept anyway...

Other onion-related musings:

vidalia (a few packages, but seem pretty far afield)
cipollini
Other random ideas:
Almiraj, a mythical creature which is a rabbit with a unicorn horn (and would allow the same playing with "AL/AI" as above

okay actually going to compile all the options still "in the running" in one post because I've just been dropping things throughout the day...

chemellia or similar, design concept for a logo:

chemelion, see logo concept above
ChemFlux (kind of vanilla, but still a reasonable option)
something to do with onions (because they have layers, like models...): e.g. caraMeLize, vidalia, cipollini, ...
chemoMiLe or similar (I like tea...)
Almiraj, a mythical creature which is a rabbit with a unicorn horn (which is just fun, and would allow the same playing with "AL/AI" as above
something alluding to differentiability, maybe playing with nablas and harps...

I like almiraj! AI, Mirage.  Nice wordplay and mythical creatures are great!

@cormullion, if I were to play with that, how would you suggest proceeding? e.g. for the chameleon above, did you "draw" it yourself in Luxor or just import some clipart?

For something non-geometric like the chameleon I'd usually draw it on my iPad then digitize it, then the points can live as raw  path coordinates in a Julia file, or, sometimes, just place an SVG on the drawing (readsvg() ... placeimage()).

I hope your Almiraj fares better than my Julyax](https://discourse.julialang.org/t/an-animal-for-julia/6322)! 😄

Okay, the newest iterates: both large-logo and little-box versions, on both white and black backgrounds... (I'm currently leaning towards the first font)

Nice work! I like the Arima version!
Will someone be asking "where are the Julia colors/dots"? :) Not sure they'll really fit in very well...

Thanks for everyone's help and input!


Push Kyuubi docker image

1. Describe the feature
Push Kyuubi docker image to dockerhub

LGTM

created a ticket https://issues.apache.org/jira/browse/INFRA-22212

Now the docker repo is ready, but we need secrets for github action.
Created a new ticket https://issues.apache.org/jira/browse/INFRA-22341


Cloning repo from GitLab

Hi there.
When I try to clone a repo hosted on GitLab, the app throws an error message:
Clone error: could not find appropriate mechanism for credentials
Greets
Nils

I found the solution to this issue in the FAQ. After providing credentials via the settings, everything worked fine.
A standard case of RTFM. :)
May be you could provide a bit more descriptive error message. That would improve the UX.
Greets
Nils


Update Generic Pages “about-counselling”

Automatically generated by Netlify CMS

👷 Deploy Preview for shimmering-marigold-4f5b4a processing.

Name
Link

🔨 Latest commit
97e9f8ed222cc860eb11b3631136c8c180fc7005

🔍 Latest deploy log
https://app.netlify.com/sites/shimmering-marigold-4f5b4a/deploys/633d986609ac6e0008e4ee23


ApiError: Problems resolving hue bridges, HTTP error status: 429

I have a CLI built using node-hue-api that has been working fine for months. Suddenly I'm getting rate limits now. I got a rate limit today and I haven't even run the CLI once today before that. The rate limits make no sense.
    at /opt/myapp/node_modules/node-hue-api/dist/cjs/api/discovery/nupnp.js:15:15
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async main (/opt/myapp/cli.js:44:21) {
  _hueError: undefine

Same for me, identical error message.

It is not the library; https://developers.meethue.com/rate-limiting-status-code/

I have a complete lack of visibility to the use case here and the full manifestation of the error, can you provide some more context and a reproducible way for this error to be created?

This is the code: https://github.com/lexicalunit/busylight/blob/main/cli.js
Essentially I'm just doing:
    const results = await discovery.nupnpSearch()
    const host = results[0].ipaddress
    const secure = await api.createLocal(host).connect(settings.username)
    const light = (await secure.lights.getLightByName(options.light))[0]
    const state = stateFromOptions(options)
    await secure.lights.setLightState(light.id, state)

I'm experiencing the same issue. This is a major issue that prevents basic usage.

From the Hue documentation it looks to be active as of now:

NOTE 1 – UPnP (or more specifically the UPnP subset Simple Service Discovery Protocol (SSDP)) used to be available as another discovery method but has been deprecated in favor of mDNS and will be disabled in Q2 2022.

That deprecation warning should no impact this particular endpoint, as the nupnp search is reaching out to the Hue cloud to get the data back for the known register bridges, https://discovery.meethue.com
They will have some level of rate limiting on this endpoint to stop abuse, but I cannot see any details on a rate limit documented anywhere. Do you know how many calls you are making before this starts to trigger?

I managed to reproduce the rate limiting by hitting just the end point around 10 times in less than a minute. Hue are clearly protecting this endpoint now more than they used to, and it is technically an unauthenticated endpoint, so is a reasonable safety measure I guess to stop abuse...
That said, nothing that I am going to be able to remedy here, it is enforced by Hue themselves, I will take a look into the headers to see if there is any extra data about the rate limits, but nothing that I can change from the library side, Hue own the endpoint and can do whatever they like here.

I just did a quick test and the limit seems to be 12. The thirteenth request fails. I don't know what the timeframe is.

Nothing useful in the returned headers indicating a window, number of api calls remaining or reset time:

I found this snippet buried in the Hue docs

The rate limit of using the discovery endpoint is maximum one request per 15 minutes per client. Typically it should be much less than that because the discovered IP can be stored and only rechecked when a request times out (as the ip might have changed). More regular ip scanning can always be done with mDNS.

With the deprecation of the UPnP and the addition of mDNS, it looks like I will need to add that to the library as an alternative approach for discovery now.

https://github.com/peter-murray/node-hue-api/issues/221#issuecomment-1214181471

Works great on my end! Discovery takes about 4s for me, every time. So maybe the default timeout could be a little more than 5s.

Works great for me too!


N21: puffin_egui

Part of https://github.com/rust-gamedev/rust-gamedev.github.io/issues/589
Sorry for the late submission!

Thanks for the low framerate gif!

Thank you for the PRs!


Add the no-cursor class to the editor through attributes

Hello there 👋
While adding this awesome plugin to Twist's new editor (currently in beta), I've come across an issue that I wasn't able to reproduce with a minimal example in a CodeSandbox, but it was clearly a problem in our code base.
If you're interested to know more about the issue in particular, please refer to this discussion at discuss.ProseMirror. That's also where I got the solution you see on this PR (provided by Marijn Haverbeke himself), which fixes the issue for me, and doesn't break anything, win-win.
I'm hoping you can merge this soon, and publish a new release of prosemirror-codemark.

This is great, thank for the PR! Super cool to see Twist coming out with a new editor, and glad it will be better than slacks! :)
I will test this out now and release.

This is published as 0.3.6!

Awesome, thank you so much @rowanc1 🙏


tofi: init

Added stylix target for tofi, a wayland dmenu alternative.
Based off the existing rofi target.

Locally, the failed CI tests are working for me. The tofi HM module is only two weeks old, could the stylix flake input for it be older than that?

Locally, the failed CI tests are working for me.

All tests are failing for this reason:
Run nix -L build github:danth/stylix/5d6eadb3edcd8bcb64090fa27a1527eeeb87d738#packages.x86_64-darwin.docs
error:
       … while calling the 'derivationStrict' builtin
         at <nix/derivation-internal.nix>:9:12:
            8|
            9|   strict = derivationStrict drvAttrs;
             |            ^
           10|

       … while evaluating derivation 'stylix-book'
         whose name attribute is located at /nix/store/fszphhr0kc59brk510m3a579vjbjdy2s-source/pkgs/stdenv/generic/make-derivation.nix:331:7

       … while evaluating attribute 'patchPhase' of derivation 'stylix-book'
         at /nix/store/ljy1ym11j0rrnc3s3if80darc5gi1msn-source/docs/default.nix:44:3:
           43|
           44|   patchPhase = ''
             |   ^
           45|     cp ${../README.md} src/README.md

       (stack trace truncated; use '--show-trace' to show the full trace)

       error: The option `programs.tofi' does not exist. Definition values:
       - In `/nix/store/whm360hyl2wcjbj5kal4wxibpb1hnr5q-modules/tofi/hm.nix':
           {
             _type = "if";
             condition = true;
             content = {
               settings = {
           ...

The tofi HM module is only two weeks old, could the stylix flake input for it be older than that?

You can compare your minimum Home Manager revision requirement with the one in flake.lock:
https://github.com/danth/stylix/blob/2f29ecd3e4c0ebf75f46647b4c4fb7261b978aac/flake.lock#L166-L185
However, based on the error, it seems like the Home Manager input is too old. In that case, update the Home Manager input in this PR.

I've changed to withHashtagand removed an unnecessary string interpolation

It was being used as the background, i've updated from base00 -> base02

Apologies, I meant selection (which is still base03), not background. base00 is correct as a general background color.

selection is the foreground, currently the selection background is set to the same as the default background (base00). The selection is base03

In that case, I think it's fine. We don't really have an agreed color for selection foregrounds right now.


Adding CLI rows per batch optional parameter

When we have a large number of rows, inserting them in one statement can lead to a poor performance issue,
I think it's a good idea to optionally configure how many rows per insert we want the script to generate.

Good job on this! 🎉 Fixed with https://github.com/gitdagray/csv-to-sql-insert/pull/32


Add link to download notebooks on docs page

Rendered notebooks are in the same path, i.e. the workforce example notebook can be downloaded from here
npshinx-prolog could be used to point to this file

This seems workable, the only issue is that data files need to be separately downloaded from the correct place.

This is done on latest. There is a link to download notebooks in the header of each example:

The main examples page provides this link as well.


Framework for supporting more languages in LanguageParser

Description
I am submitting this for a school project as part of a team of 5. Other team members are @LeilaChr, @maazh10, @Megabear137, @jelalalamy. This PR also has contributions from community members @Harrolee and @Mario928.
Initial context is in the issue we opened (#11229).
This pull request adds:

Generic framework for expanding the languages that LanguageParser can handle, using the tree-sitter parsing library and existing language-specific parsers written for it
Support for the following additional languages in LanguageParser:

C
C++
C#
Go
Java (contributed by @Mario928 https://github.com/ThatsJustCheesy/langchain/pull/2)
Kotlin
Lua
Perl
Ruby
Rust
Scala
TypeScript (contributed by @Harrolee https://github.com/ThatsJustCheesy/langchain/pull/1)

Here is the design document if curious, but no need to read it.
Issues

Closes #11229
Closes #10996
Closes #8405

Dependencies
tree_sitter and tree_sitter_languages on PyPI. We have tried to add these as optional dependencies.
Documentation
We have updated the list of supported languages, and also added a section to source_code.ipynb detailing how to add support for additional languages using our framework.
Maintainer

@hwchase17 (previously reviewed https://github.com/langchain-ai/langchain/pull/6486)

Thanks!!
Git commits
We will gladly squash any/all of our commits (esp merge commits) if necessary. Let us know if this is desirable, or if you will be squash-merging anyway.

Hi @eyurtsev I noticed that you reviewed the COBOL parser PR. Pinging if you have time to review this, which is similar :)

Hi @eyurtsev , is there a chance to move this one forward? It would be nice to get a couple of languages more covered! Thanks in advance!

I've been working from this branch but it's falling behind. I hesitate to spend time solving merge conflicts. Is this likely to be merge or are you going in a different direction to support language parsers?

@rlancemartin can you give us feedback on this PR?

Is there any update on this PR? Is it going to be merged or closed?

Hi @baskaryan I fixed the lint issues and our tests pass locally, we would appreciate if you could take another look at our PR.

@baskaryan It would be great if can check the PR :-)

@Microsvuln it's merged now :)

@ThatsJustCheesy That's awesome, thanks!


Feature Request: add Status to the main column

the current table column are: API 	Description 	Auth 	HTTPS 	CORS
it would be nice to have a "Status" column to check availability of the link in the API column (maybe ping to check every hour?) and display them as Online/Offline text in the main table (reason being some site replaced api doc, or even discontinued the service itself)

That's a good idea, but I believe I column for that isn't necessary.
Instead, we just remove the API if it's not available anymore.
At the moment, users already report when something is not working, so I keep an eye on it for a few days, and if it's still broken, it's removed altogether.
A script to do this check automatically would be a good addition to the project.
Would you be up to create a PR for that?

Can I work upon that?

@vrajjbhatt sure!
I recommend using a GitHub action that triggers daily in a given time and creates a new issue in case a link is broken, but feel free to put your spin on it.


draftsman.env.update fails, path mangled on windows

EDIT: I found an easy workaround: unzip the mod files! Hope this helps anyone else with this problem.
First, this package seems really wonderful, I'm very keen to a script that generates blueprints for my game. I'm having trouble at step 0 however, getting the environment to work with my mod list. I'm using Windows, and it seems like draftsman is mangling paths to use both nix and windows style paths.
Describe the bug
I am calling draftsman.env.update(verbose=True) in Python. I have one mod (Industrial Revolution, 2.3.2) in the "C:\Users\danie.conda\envs\PrettySolar\Lib\site-packages\draftsman\factorio-mods" folder. When draftsman searches for sub-code files (i.e. code/data/globals in this case) it looks for a file at  'C:\Users\danie.conda\envs\PrettySolar\lib\site-packages\draftsman\factorio-mods/IndustrialRevolution/code/data/code/data/globals.lua' (checked with custom loader) , which doesn't exist because both forward and backwards slashes are getting mangled together.
Current Behavior
Call draftsman.env.update() within a script  (or alternatively, call "dratsman-update --verbose" at the command line).
Expected behavior
Draftsman should stick to a single forward/backward slash convention when searching for files.
Additional context
Add any other context about the issue here, if seemingly relevant.
Please also include the following:
Draftsman version: 1.0.6
Python version: 3.9.7
Factorio version: '1.1.87'
If your issue is related to mod-loading with draftsman-update, in addition to the above information, please also include:

The exact draftsman-update command that fails/produces incorrect behavior
import draftsman.env draftsman.env.update(verbose=True)

The error produced by the command (if the command does fail)

Discovering mods...

(zip) IndustrialRevolution 2.3.2

Determining dependency tree...

IndustrialRevolution 2.3.2
archive? True
dependencies:
	   base >= 1.1.37
	 ? DeadlockLargerLamp >= 1.5.0
	 ? DeadlockBlackRubberBelts >= 1.0.0
	 ? IndustrialRevolutionExtraComponents
	 ? IndustrialRevolutionGas
	 ? Dectorio
	 ? textplates
	 ? rso-mod

Load order:
['base', 'IndustrialRevolution']

SETTINGS.LUA:
	mod: IndustrialRevolution
SETTINGS-UPDATES.LUA:
SETTINGS-FINAL-FIXES.LUA:
DATA.LUA:
	mod: base
	mod: IndustrialRevolution
---------------------------------------------------------------------------
LuaError                                  Traceback (most recent call last)
~\AppData\Local\Temp/ipykernel_17140/1255214403.py in <module>
      9 import draftsman.env
     10
---> 11 draftsman.env.update(verbose=True)
     12 # draftsman.env.update(verbose=True,path = r'C:/Users/danie/AppData/Roaming/Factorio/mods')

~\.conda\envs\PrettySolar\lib\site-packages\draftsman\env.py in update(verbose, path, show_logs, no_mods, report)
   1828                     print("\tmod:", mod_name)
   1829
-> 1830                 load_stage(lua, mods, mod, stage)
   1831
   1832                 # Reset the included modules

~\.conda\envs\PrettySolar\lib\site-packages\draftsman\env.py in load_stage(lua, mod_list, mod, stage)
    277     lua.globals().lua_add_path(mod.location + "/?.lua")
    278
--> 279     lua.execute(mod.data[stage])
    280
    281

lupa\lua52.pyx in lupa.lua52.LuaRuntime.execute()

lupa\lua52.pyx in lupa.lua52.run_lua()

lupa\lua52.pyx in lupa.lua52.call_lua()

lupa\lua52.pyx in lupa.lua52.execute_lua_call()

lupa\lua52.pyx in lupa.lua52.raise_lua_error()

LuaError: [string "<python>"]:260: module 'code.data.globals' not found:no file 'code/data/globals' found in 'IndustrialRevolution' archive
	no field package.preload['code.data.globals']
	no file 'C:\Users\danie\.conda\envs\PrettySolar\lib\site-packages\draftsman\factorio-mods/IndustrialRevolution/code/data/code/data/globals.lua' (checked with custom loader)
	no file 'C:\Users\danie\.conda\envs\PrettySolar\lib\site-packages\draftsman\factorio-mods/IndustrialRevolution/code/data/globals.lua' (checked with custom loader)
	no file 'code/data/globals.lua' (checked with custom loader)
	no file 'C:\Users\danie\.conda\envs\PrettySolar\lib\site-packages\draftsman\factorio-data\core/code/data/globals.lua' (checked with custom loader)
	no file 'C:\Users\danie\.conda\envs\PrettySolar\lib\site-packages\draftsman\factorio-data\core\lualib\code/data/globals.lua' (checked with custom loader)
	no file 'C:\Users\danie\.conda\envs\PrettySolar\lib\site-packages\draftsman\code/data/globals.lua' (checked with custom loader)
	no file 'C:\Users\danie\.conda\envs\PrettySolar\lua\code/data/globals.lua' (checked with custom loader)
	no file 'C:\Users\danie\.conda\envs\PrettySolar\lua\code/data/globals\init.lua' (checked with custom loader)
	no file 'C:\Users\danie\.conda\envs\PrettySolar\code/data/globals.lua' (checked with custom loader)
	no file 'C:\Users\danie\.conda\envs\PrettySolar\code/data/globals\init.lua' (checked with custom loader)
	no file '.\code/data/globals.lua' (checked with custom loader)
	no file 'C:\Users\danie\.conda\envs\PrettySolar\code\data\globals.dll'
	no file 'C:\Users\danie\.conda\envs\PrettySolar\loadall.dll'
	no file '.\code\data\globals.dll'
	no file 'C:\Users\danie\.conda\envs\PrettySolar\code.dll'
	no file 'C:\Users\danie\.conda\envs\PrettySolar\loadall.dll'
	no file '.\code.dll'
stack traceback:
	[string "<python>"]:9: in main chunk
	[string "<python>"]:260: in function 'require'
	[C]: in function 'old_require'

The mod list with their versions used with the command
Industrial revolution 2.3.2. Can be found here. Mod is 120MB due to graphics, so I won't attach it to this github issue.

Thanks for the report, I'll take a look at this in a bit. The inconsistent path slashing is a culprit of 3 different conditions:

Python's os.path utils will default to the current OS's preferred path, forward-slash on unix and back-slash on windows
zipfile (the module used for loading the mod archives) doesn't work with backslashes, so everything has to be coerced to forward slashes for interacting with zip archive internals
Finally, the process for finding Lua files uses Lua's package.path variable, which uses system-specific slashes; and these paths are combined with the previously normalized forward-slash paths to comply with zipfile, which is almost certainly the cause of the mixed pathing.

Up to this point, this mixed pathing has worked alright in terms of file resolution (if you could verify with a different set of mods/modpack that would be great), so if I had to guess there's probably something specific with Industrial Revolution that causes the breakage, or when I redid the pathing code I introduced a regression. I'll install the mods and see if I can reproduce.

Fixed:
Load order:
['base', 'IndustrialRevolution3Assets1', 'IndustrialRevolution3Assets2', 'IndustrialRevolution3Assets3', 'IndustrialRevolution3Assets4', 'IndustrialRevolution3']

SETTINGS.LUA:
        mod: IndustrialRevolution3
SETTINGS-UPDATES.LUA:
        mod: IndustrialRevolution3
SETTINGS-FINAL-FIXES.LUA:
DATA.LUA:
        mod: base
        mod: IndustrialRevolution3
DATA-UPDATES.LUA:
        mod: base
        mod: IndustrialRevolution3
DATA-FINAL-FIXES.LUA:
        mod: IndustrialRevolution3

Extracted mods...
Extracted entities...
Extracted instruments...
Extracted items...
Extracted modules...
Extracted recipes...
Extracted signals...
Extracted tiles...

Update finished.
hella slick; nothing broke!

(factorio-draftsman) D:\SourceCode\repos\Python\factorio-draftsman>python
Python 3.10.0 (tags/v3.10.0:b494f59, Oct  4 2021, 19:00:18) [MSC v.1929 64 bit (AMD64)] on win32
Type "help", "copyright", "credits" or "license" for more information.
>>> from draftsman.data import entities
>>> entities.containers
['wood-pallet', 'tin-pallet', 'wooden-chest', 'tin-chest', 'bronze-chest', 'iron-chest', 'steel-chest', 'big-ship-wreck-1', 'big-ship-wreck-2', 'big-ship-wreck-3', 'blue-chest', 'red-chest', 'factorio-logo-11tiles', 'factorio-logo-16tiles', 'factorio-logo-22tiles', 'starter-crate-medium-1', 'starter-crate-medium-2', 'starter-crate-medium-3', 'starter-crate-medium-4', 'starter-crate-small-1', 'starter-crate-small-2', 'starter-crate-small-3', 'starter-crate-small-4']

Problem was that the mod location (which ever since folder loading got added is now important for archive resolution) wasn't being converted to forward slashes, meaning it would never be replaced with the internal archive folder. This is why archives in an external directory would work, and why extracting the files out of the archive also solved the problem. This will be fixed in 1.0.7 or 2.0, whichever comes first.


Missed items in riak-cs.conf [JIRA: RCS-162]

Missed 2.0.0

max_buckets_per_user
gc.batch_size (gc_batch_size in app.config)

Lacking access log handler is not working :sob:


Is it possible to take full resolution images from ov2640?

OV2640 is 2MP module, but currently in this example maximum frame size is 800x600. Is it possible to take full resolution photo? I need only one shot image.
Thanks.

I think it's possible if you use an esp32-wrover module. It has a 4M PSRAM module.

Take a look at https://github.com/tmrttmrt/esp32-cam-demo/tree/tmrt. With enough compression it is possible to obtain jpg images with sizes of about 100 kB.

I am new to esp32 so have no idea.

@tmrttmrt I downloaded your repo and tried to compile but get this error:
xclk.c:31:12: error: 'ledc_channel_config_t {aka struct }' has no member named 'hpoint'
ch_conf.hpoint = 0;
I understand this happened as the result of the new version of ESP-IDF. I commented out the line but not sure if it will break something else

I am building with IDF3.2 latest release. I think if you update the version, it will compile.
And I am also wondering if we can take the snapshots with 1600*1200 resolution.
Memory allocation of the frame buffer is done using calloc function which allocates memory from internal SRAM, does not use PSRAM. In addition to that, even if the buffer is allocated in PSRAM using MALLOC_CAP_SPIRAM flag, this memory buffer could not be read from or written into by DMA.
I2S uses DMA to write the data to buffer, so PSRAM seems impossible to use for frame buffer to me.
Ping-pong buffering may be a solution but I am not sure if we can change the buffer address while the I2S is reading data from the camera.
@igrr do you have any comments on this issue? If you can inform us about the issue, it would be great.
Thanks in advance.

Short answer: Yes it is possible. No problem at all with the 4M PSRAM module.

Do you have the long answer? How to shoot at 1600*1200 resolution??