config:set --global for common variables

From @boffbowsh on September 10, 2014 14:8
It'd be extremely useful for a Deis deployment of common applications to have the concept of cluster-wide environment variables, for example a REDIS_URL if all apps hit a common Redis server, or perhaps some email service credentials.
Regarding the action that would take place when these get updated, I think it would be best to allow the operators to re-release and thus restart apps at their own pace.
Copied from original issue: deis/deis#1804

From @bacongobbler on September 10, 2014 15:13
Not sure what name to give the option, but --cluster is already used to specify which cluster you want the app to be deployed to. Maybe something like --global?
Also, would this force a new release across all existing apps in the cluster?

From @bacongobbler on September 10, 2014 15:24

Is it possible to force a new release of an app without changing app code or config?

If we're following 12 factor to a tee, "build + config = release". It's not possible to force a new release until something directly changes the environment. Setting a cluster-wide environment variable should create a new release for each application since it would be picked up the next time the application created a release, anyways. I'm concerned about the computational power required to re-provision apps across the cluster, but that shouldn't be too much of a concern so much as doing "The Right Thing:tm:".

From @boffbowsh on September 10, 2014 15:35
It's more a case of the timing of it being in control of devs/ops. Restarting all apps on the cluster at the same time may not be desirable. If that can be controlled I don't see a problem in creating a release for each app at the time the setting is changed.
Disclaimer: we don't currently use Deis, but are very much looking forward to using it once it goes Stable, with HA, logging etc. This is one of the features that would possibly block us from using Deis without having something external that manages common config across all apps.

From @boffbowsh on September 10, 2014 15:16
Ah yes of course. Perhaps config:set:cluster then or cluster:config:set?
I think changing this should just essentially be a database setting, and subsequent releases will pick it up. Is it possible to force a new release of an app without changing app code or config?

From @sforsman on February 11, 2015 18:50
This feature would definitely be useful in our use cases as well. It would simplify management and deployment of our applications.

From @carmstrong on February 10, 2015 18:41
Renamed issue to config:set --global, as dropping the notion of clusters would have made this command confusing.

From @mboersma on May 20, 2015 3:19
A workaround for now is something like deis create && deis config:push -p ~/.env.
EDIT: changed config:set to config:push. With love from @bacongobbler :heart:

From @kikicarbonell on September 10, 2014 19:40
config:set --cluster for common variables, I need too! :+1:

From @jwaldrip on July 13, 2015 15:49
Pulled this from another request I had #4013:
As a developer I would like to set ENV variables across my cluster, so that I don't have to configure it for each application
Example Use Case:
deis config:set --global NEW_RELIC_LICENSE_KEY=abc123

Acceptance Criteria:

New builds will reflect the ENV variable.
New applications will reflect the ENV variable.
ENV variable can be overridden if set at the application level.

Nice to haves:
Aliasing. It would be great if I could set SOME_VAR=$OTHER_VAR. This would allow me to do NEW_RELIC_APP_NAME=$DEIS_APP.

This issue was moved to teamhephy/controller#39


rPackages.zlib: fix build

Description of changes
Added pkgs.zlib.dev to nativeBuildInputs.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

You can add a formal review and approve it, which can help with getting it merged.

It wasn't easy, but I found how to do this


Is it possible to change the default settings of the embedded PDF viewer?

I'd like to change some of the default settings of the integrated PDF viewer, for example the default zoom (from "Auto Zoom" to “Full Width”) or the background color of the PDF page (currently transparent, which is not suitable when a dark mode is used).
Is it possible to change these default settings on the latexcgi side?

well by default it just does
https://texlive.net/pdfjs/web/viewer.html?file=/latexcgi/document_E367_13155.pdf
so customisation is limited to whatever options PDFjs exposes as URL parameters.
You can set the zoom level there by the look of it
https://github.com/mozilla/pdf.js/wiki/Viewer-options
....#zoom=page-width (untested)

OK that does the trick, thanks. But is it possible to set this in runlatex.js? AFAICS, /pdfjs/web/viewer.html appears only in perl/latexcgi.

the background color should I think be settable on the iframe if you use a modified runlatex.js, to inject some javascript see
https://stackoverflow.com/a/50549200

Perfect, thanks!

OK that does the trick, thanks. But is it possible to set this in runlatex.js? AFAICS, /pdfjs/web/viewer.html appears only in perl/latexcgi.

er... I'll think about it...

OK that does the trick, thanks. But is it possible to set this in runlatex.js? AFAICS, /pdfjs/web/viewer.html appears only in perl/latexcgi.

er... I'll think about it...

Obviously, no pressure, but is there any news on the subject?


Freezes in QTile WM when closing a window that it's tiled with

I'm not sure if this happens on other Linux tiling WMs, but on QTile it freezes. Does not happne with any other theme.

Ahhhhh. What?

If you want to test this out, start a VM with Linux (any distribution will do, as long as it has Qtile). Then, install Qtile and log into it. Install Discord then apply the theme and you'll see the issue

I might give some evidence but I don't have my computer on rn

I'm sorry, I'm not going through all that just to fix it for 1 person.

While it may be only for one person now, imagine how many more people will have this same issue when this theme gets more popular. Also, it's never good to have buggy code unless it's a developer release

it's never good to have buggy code unless it's a developer release.

You're the only person that I've heard to have this problem.
As I said, I'm not doing all that to fix it for 1 person.
Either fix it yourself and then tell me what you did or find another theme.

I can't really fix it myself because I don't know CSS well. I think it might be a bug in the Better-discord rendering engine


[SDPA-4224] Remove section-io-id from proxied req

JIRA issue: https://digital-engagement.atlassian.net/browse/SDPA-4224
Changed

Remove section-io-id from proxy request to backend

Screenshots

@steveworley, I was wondering if you're looking at something here on the section side to limit headers sent to Origin? https://digital-engagement.atlassian.net/browse/SDPSUP-1606

@dylankelly , I don't think this makes sense in Ripple, it's too bespoke to our CDN. Take a look at this, https://digital-engagement.atlassian.net/browse/DDIDO-286

Agreed - Happy to remove this but the issue in the linked ticket is different to this. This addresses the issue that was identified on vicpol where simply adding the section-io-id to a request to BE via section causes 508 loop detected It has nothing to do with the size of the header. Please confirm with Steve what you want to do.


ifconfig -l | grep -F "tapX " will fail if tapX is the last in the list

12:56 < kerio> pr1ntf: ifconfig -l | grep -F "tapX " will fail if tapX is the last in the list
12:56 < kerio> i don't even :|
13:00 < kerio> i reckon ( printf " " ; ifconfig -l ) | grep -q -F " $tap"; would work
13:01 < kerio> prepending a space instead of appending because it's easier than deleting the newline at the end, but still
13:02 < kerio> wait nvm you have to append
13:02 < kerio> goddammit

< kerio> or replace spaces with newlines in the output of ifconfig -l and then grep for ^tapX$

17:52 < kerio> i got it
17:52 < kerio> ifconfig -l | tr '\n' ' '
17:52 < kerio> it will replace the ending newline with a space
17:52 < kerio> and then you grep for "tapX "
17:52 < kerio> (or do it viceversa)

The problem is here:
https://github.com/pr1ntf/iohyve/blob/master/iohyve#L654
local tapif="$('ifconfig' -a | grep $tap: | cut -c1-4)"
That's not supposed to be the case, it should be the ifconfig -l method.

It got changed back after the multiple tap support merge.
This needs to be changed back.

This has been fixed.


How to run bbox and mask operations only on CPU

Team,
I'm using mmdetection with mask-rcnn for bounding box and masking of objects, it works great. I'm new to mmdetection and trying to figure out how to run masking and bounding box operations only on CPUs. The training was done with GPU enabled runtime.
Thank you all for your great contributions to this project.
Mohankumar.

I figured it out at the below function. Thank you!
def init_detector(config, checkpoint=None, device='cpu'):


Add some presentations (more need adding later)

Add a list of references to presentations about Orchestrator.
I've added one I did yesterday and your previous one at Percona Live but know you've done a lot more. Perhaps they can be added later?
Either way I think it's good for users to see this list and hopefully it will give more context to people looking at Orchestrator for the first time.

Excellent! I'll add more later. FYI, all my presentations are on https://speakerdeck.com/shlominoach


Add validation to parameters in cloudwatch-metrics integration policy

Description

Add validation to variables: ExternalIdSecret, CustomAccountId
Rename variables:
ExternalId --> ExternalIdSecret
CustomerAccountId --> CustomAccountId
Change ExternalId format to be of structure  ExternalIdSecret@AWSAccount

CDS-1522

 Fixes #
How Has This Been Tested?

Checklist:

[ ] I have updated the relevant component changelog(s)
[ ] This change does not affect any particular component (e.g. it's readme or docs change)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[July Release] Get, search, create and publish via new API

This is the mega epic for the July release. It also already holds the release tasks.
I took the liberty of putting off some of the original items to the next sprint since there is a lot and the next sprint being about just files would accommodate work on other things since we can't all work on files. In any case, we will review this, so feel free to make suggestions.

[x] [EPIC] Implement GET /records/:id (created by invenio-cli demo) via new API  #213

resolve PID
implement JSON record serializer [crosscutting]
error handling [crosscutting] [errors]
check permission [optional]

[x] [EPIC] Implement the GET /records (Search a record) #212

pagination URL query strings [separate]
serialization format for search [separate]

body links
pagination
header links [optional]

Error handling [crosscutting] [errors]
optional section
Permissions filtering [crosscutting] [optional]
Implement a query translator API [optional]

[x] [EPIC] Implement the POST /records/:id (Create a draft record) via new API #211

Deserilization of request data. [crosscutting] https://github.com/inveniosoftware/flask-resources/issues/62
Mint a PID (recid)
One DB table for records, and one DB table for drafts [data] [internal]
Indexing of the draft and record (using one mapping/JSONSchema) [data] [internal]
Mapping of service errors to HTTP errors. [crosscutting] [errors] [separate]
Data validation with Marshmallow [crosscutting] [errors] [marshmallow] [separate]
optional section
Permission checks and integration of Invenio-Records-Permissions [crosscutting] [optional]
Finding a solution for system fields (pids, access control, ...) [optional]
Port versioning from Zenodo [optional]

[x] [EPIC] Implement the POST /records/:id/actions/publish (publish record) via new API #210
[x] Connect API to frontend in invenio-app-rdm / react-invenio-deposit #209

release react-invenio-deposit if need be

[x] Map + serialize general Exception to HTTP exceptions #214
[x] [EPIC] Translate frontpage, Record landing page, deposit page, search page

Releases

[x] Update flask-resources to 0.2.X

[x] Update invenio-records-resources to 0.3.X

[x] Update invenio-drafts-resources to 0.1.X

[x] Update invenio-rdm-records to 0.17.X

[x] Update invenio-app-rdm 0.11.X

[x] Update cookiecutter to 0.11.X

[x] Update invenio-cli to 0.15.X

[x] Update and release docs

Achieved if: Achieved if with the July release, you can get, search, create and publish a record with the new API (meaning that the hard problems in the new API have all been fixed).

Closing as complete :tada:


[Event Request] codeunit 362 PeriodPageManagement - CreatePeriodFormat

Please extend the return value for function CreatePeriodFormat to 20 Chars.
By subscribing to the Event OnAfterCreatePeriodFormat longer return texts might be returned.
For this purpose the current function CreatePeriodFormat should probably be obsoleted.
Background:
Subscrbing to eveb OnAfterCreatePeriodFormat might be used in order to change how the date will be displayed in matrix pages ie.:
Mon, 22.08.22

This is no longer needed - therefore closing


Unable to use Static Membership feature

I couldn't use any client configuration that is mentioned in apache/confluent docs.
I'm using nodemon in development mode and everytime it restarts the server takes 3-4 minutes to join the group.
I found this blog https://www.confluent.io/blog/kafka-rebalance-protocol-static-membership/

Static Membership introduces a new consumer configuration called group.instance.id, which is configured by users to uniquely identify their consumer instances. Although the coordinator-assigned member ID gets lost during restart, the coordinator will still recognize this member based on the provided group instance ID in the join request. Therefore, the same assignment is guaranteed.

I want to set the group.instance.id configuration to a unique ID for the consumer but I don't know if it's doable to pass this shape of configuration to KafkaJS client or if there is any other way that I could achieve static membership.

hmmm 🤔 https://github.com/edenhill/librdkafka/pull/2638

I believe you are confused about how the Kafka client works. The configurations you mention are not KafkaJS configuration options. I don't know what client the apache/confluent docs are talking about - possibly librdkafka.

I want to set the group.instance.id configuration to a unique ID for the consumer but I don't know if it's doable to pass this shape of configuration to KafkaJS client or if there is any other way that I could achieve static membership.

Static membership is not supported by KafkaJS yet. If it's taking 3-4 minutes for your consumer to join the group, you should look into configuring rebalanceTimeout and sessionTimeout on the consumer. Whenever you restart the process, Kafka has to figure out that the old instance is dead and remove it from the group, as well as let the new process join the group, before the group is considered stable and can start to consume again. In development you could have much lower values for both sessionTimeout and rebalanceTimeout than you would likely want to have in production.

I know this issue was closed due to it's relationship to librdkafka, but I wanted to follow up as it is something that I believe would be a valuable addition to kafkajs as the reduction in rebalancing is significant. We've made the changes in our fork of kafkajs but have also targeted it at kafka 2.4 and above so not sure it's a great implementation for the library (hence why we haven't submitted a pull request with the change). We have been using it in production for a few months and it's been a great improvement in terms of the rebalancing "dance" we used to see with Kafka.
Here is the changes if it's helpful:
https://github.com/ambyint/kafkajs/tree/feature/group-instance-id-support

Here is the changes if it's helpful: https://github.com/ambyint/kafkajs/tree/feature/group-instance-id-support

I've tried to use this and it works pretty good!
Steps to start using improved package coupled with NestJS:

Install package using:

yarn add kafkajs@https://github.com/ambyint/kafkajs.git#feature/group-instance-id-support

Provide “group instance ID” to connectMicroservice():

  app.connectMicroservice({
    transport: Transport.KAFKA,
    options: {
      consumer: {
        groupInstanceId: 'test',
        ...
      }
    }
  })


export-sqllogin 1.0 update

whys that, @niphlod ? it does not output to file, only screen.

sorry, was on the run and on a mobile phone. forgot that it is "informational" only.

yay thank you merging 👍


feat(ui): clarified new from image menu items

Summary

Split up the New from Image submenu into two submenus.
Add actions to create a new canvas from an image without resize.
Update labels.

Related Issues / Discussions
Many discussions on discord.
QA Instructions
Does this division of menu items & the labeling of them make sense?
Merge Plan
n/a
Checklist

[x] The PR has a short but descriptive title, suitable for a changelog

Looks very clean


Introducing KServe Guru on Gurubase.io

Hello team,
I'm the maintainer of Anteon. We have created Gurubase.io with the mission of building a centralized, open-source tool-focused knowledge base. Essentially, each "guru" is equipped with custom knowledge to answer user questions based on collected data related to that tool.
I wanted to update you that I've manually added the KServe Guru to Gurubase. KServe Guru uses the data from this repo and data from the docs to answer questions by leveraging the LLM.
In this PR, I showcased the "KServe Guru", which highlights that KServe now has an AI assistant available to help users with their questions. Please let me know your thoughts on this contribution.
Additionally, if you want me to disable KServe Guru in Gurubase, just let me know that's totally fine.

just tried it, it looks very nice :)

Hi everyone,
I’d like to update you on the release of the Maintainer Panel feature on Gurubase. With this panel, you can add, remove, or update data sources, as well as change the logo and more. You can find the details here.
In the near future, I’m planning to include analytics insights in this panel, such as the number of questions asked, the most frequently asked ones, and more. I’ll be sure to update this thread once it’s available. However, in case I miss it, I highly recommend joining our Discord channel to stay updated.
If you’d prefer that I don’t update this thread anymore, please let me know.


[WIP][SPARK-40663][SQL] Migrate execution errors onto error classes: _LEGACY_ERROR_TEMP_2151-2175

What changes were proposed in this pull request?

This PR proposes to migrate 25 execution errors onto temporary error classes with the prefix _LEGACY_ERROR_TEMP_2151 to _LEGACY_ERROR_TEMP_2175.
The error classes are prefixed with _LEGACY_ERROR_TEMP_ indicates the dev-facing error messages, and won't be exposed to end users.
Why are the changes needed?

To speed-up the error class migration.
The migration on temporary error classes allow us to analyze the errors, so we can detect the most popular error classes.
Does this PR introduce any user-facing change?

No
How was this patch tested?
$ build/sbt "sql/testOnly org.apache.spark.sql.SQLQueryTestSuite"
$ build/sbt "test:testOnly *SQLQuerySuite"
$ build/sbt -Phive-thriftserver "hive-thriftserver/testOnly org.apache.spark.sql.hive.thriftserver.ThriftServerQueryTestSuite"

@itholic Please, resolve conflicts.

Resolved!

@itholic Is it ready for review?

Oh, I forgot switching to ready for review.
Yes, It's ready!

+1, LGTM. Merging to master.
Thank you, @itholic.


Update page title when route changes

Currently the title is fixed, just 'Wealth Economy', update it based on the current view

Generated views (list + edit) are not perfect, since they use 'entity' route params. userelementcell route param becomes, Userelementcell as a title. But the rest is okay for now.


Automated cherry pick of #4784: dnsrecords: add helper methods

Cherry pick of #4784 on release/2.14.
#4784: dnsrecords: add helper methods

/lgtm

/approve


fix(tag prop): tag ignored when there is animation prop

fix #1985

This feels a little weird, how does it fix the tag going away? Doesn't it just set the role? I would imagine with CSS driver the tag shouldn't go away, do you know why?

@natew That's because createComponent uses an AnimatedView and react-native-web only translate role and accessibilityRole to corresponding tags

interesting! with CSS driver tag goes away and role also has no effect

With CSS driver we control the final element completely so tag should just work no?


Add auto pruning in stores

Issue
In order to keep stores with low overhead, we need to prune some stores of the Aggregator/Signers (depending on business rules)
Tasks
In Aggregator:

[ ] Protocol Parameters (5 last epochs)
[ ] Single Signatures (5 last epochs)
[ ] Verification Keys (5 last epochs)
[ ] Stake (5 last epochs)

In Signer:

[ ] Protocol Initializer (5 last epochs)
[ ] Stake (5 last epochs)

The retention period should be configurable per environment so dev & testing env could keep data for debug purposes.


Loosing connection to server when using more than 5 Modbus client

Hi, I need to get data from 10 devices which are with different servers. I am using Modbus getter nodes. Pulling Qty: 50 registers from each device. Have 5 different modbus clients. When i run them individually, i am getting data. But, when i have them in one flow and hit Deploy,  the status of the nodes switch between "active" and "waiting" and later looses connection to server.  I have to remove the .json file and restart the whole process to come out of this error. Can anyone help me if there is any other way to get data from more than 5 modbus clients which are at different server addresses.

a) Hi, do you have some log and version information?
b) Is it possible to upload the flow to have a look? (file of clipboard export from node-red)
c) What does the queue info node tell you for each client when you add the devices one by one?
d) is the v1.0.12 installed?
@yaacov is there a restriction for multiple TCP connections? I mean no.

Thanks for looking into it.
a. Node-red version v0.16.2
Node.js version v6.10.1
i will try to record the log this time.
b. I attached my flow in the txt file below.
c. I am not using queue info.
d. Does Modbus nodes have version? Sorry. I am new to this. Can you please let me know how to check.
Tracker combined data.txt

Please, install again and you will get v1.0.13. Now it has some bug fixes from the core library for TCP transaction id handling.

so do i have to run this command below in node-red directory? Will it upgrade itself?
npm install node-red-contrib-modbus

Thank you.

I got this error.

it has some bug fixes from the core library for TCP transaction id handling.

@biancode hi,
I do not know if this is related to this problem ... but I juse updated modbus-serial to v5.2.13, it should fix the transactionId == NaN you found.

@yaacov v5.2.12 is now in v1.0.13
@Kartheek1512 what is the command? You got v1.0.9, which is an old one.

@yaacov v5.2.12 is now in v1.0.13

I added more fixes in v5.2.13 , you should update to get the fix for the NaN thing, but I do not know if it is related to this problem ...

i am using npm install node-red-contrib-modbus

Where or how was it installed before? Maybe you have to uninstall before and I mean you have to install in node-red folder, not in node_modules. It will use node_modules automatically.

Oh.. I forgot. Thank you. It worked when i did it in node-red folder. i will run the flow again and let you know how it did.

[{"id":"40c11ea2.6ac08","type":"modbus-queue-info","z":"129c512d.e72b7f","name":"Arcadia NCU1","unitid":"","lowLowLevel":25,"lowLevel":75,"highLevel":150,"highHighLevel":300,"server":"f75b60f1.d5b12","errorOnHighLevel":false,"x":580,"y":900,"wires":[[]]},{"id":"f75b60f1.d5b12","type":"modbus-client","z":"","name":"Arcadia NCU1","clienttype":"tcp","bufferCommands":true,"stateLogEnabled":false,"tcpHost":"<IP_ADDRESS>","tcpPort":"502","tcpType":"DEFAULT","serialPort":"/dev/ttyUSB","serialType":"RTU-BUFFERD","serialBaudrate":"9600","serialDatabits":"8","serialStopbits":"1","serialParity":"none","serialConnectionDelay":"100","unit_id":1,"commandDelay":1,"clientTimeout":1000,"reconnectTimeout":2000}]
do that for all eight connections - see your configuration - some hint every Modbus connection is working in the background, it is used or not in nodes

@yaacov v1.0.14 includes now v5.0.13 - Thank you!

hi Biancode.. Thank you. But how to i send "Address" , "Quantity" and "FC" info into this node.

never - just select the connection - see info on the right side of Node-RED

watch YouTube to learn more

You created mostly two connections to the same IP - hint: you can use only one per IP

I am pulling registers 20600 to 20650 on one node and from 20800 to 20850 on another node. i dont need values from 20650 to 20800. So, i used two. I will look into your videos and see if I can use one node instead of using two. Thanks.

Hi., The new modbus version is working without any issues. Thank you. I have another question. I want to pull registers 30100, 30200,30300,30400...39900.  As you see the registers are in increments of 100. Can you please guide me which modbus node to use and function program.


Universal Robots

Improvements from the URDF:

[x] Improved Meshes.
[x] Improved bounding-objects.
[x] Fixed masses enad center of masses.
[x] Doc:

https://www.cyberbotics.com/doc/guide/actuators?version=universal-robot
https://www.cyberbotics.com/doc/guide/gripper-actuators?version=universal-robot#robotiq-3f-gripper
https://www.cyberbotics.com/doc/guide/robots?version=universal-robot
https://www.cyberbotics.com/doc/guide/ur5e?version=universal-robot

[x] Sample.

(with a ROBOTIQ arm to test)

Current status of the sample (still need some improvements):

New version with performance issue fixed and scene improvements:

test me

Build Passed :ok_hand:
You can safely merge 'universal-robot' into 'develop'.

test me

test me

Build Passed :ok_hand:
You can safely merge 'universal-robot' into 'develop'.

Build Passed :ok_hand:
You can safely merge 'universal-robot' into 'develop'.

:+1:

:+1:

👀

test me

test me

Build Passed :ok_hand:
You can safely merge 'universal-robot' into 'develop'.

Build Passed :ok_hand:
You can safely merge 'universal-robot' into 'develop'.

:+1:

test me

test me

Build Failed :confused:
https://cyberbotics2.cyberbotics.com:8443/job/Webots/467/consoleText

Build Passed :ok_hand:
You can safely merge 'universal-robot' into 'develop'.

I tested to use a normal map instead of geometry for the logo and it works fine as long as we don't zoom too much:
(normal map: left | geometry: right):

When zooming in because of the texture resolution it is not so nice compared to the geometry one (but since you have to zoom a lot I think it fine):
 (normal map)
 (geometry)

It looks better to me with this normal map. Adding the ambient occlusion map might be even better.

test me

test me

Build Passed :ok_hand:
You can safely merge 'universal-robot' into 'develop'.

Build Passed :ok_hand:
You can safely merge 'universal-robot' into 'develop'.

:+1:


A teacher can ask a student to edit the whiteboard

Dependant on #7 (Whiteboard).
Should communicate with #19 (Calling pupils).
The teacher can give one or more pupils the permission to edit the blackboard. He or she can also take the permission back.

Zusammenhang mit Issue #54


Fix for Photo Stream crash in iOS 8.1

As referenced in #41, on iOS 8.1, all the Photo Stream assets are returned nil, therefore causing a crash. I applied the solution here http://stackoverflow.com/a/26526199 to workaround it and find another way to get the selected assets.

Fixed in 2.2.4.


Support more fractions

This PR adds support for \genfrac, \brace, \brack, \above, \atopwithdelims, \overwithdelims, and \abovewithdelims.
It deletes file genfrac.js and returns groupTypes.genfrac to files buildHTML.js and buildMathML.js. I did not want to do that, but I had to expose groupTypes.genfrac to several functions that have varying numbers of arguments.

Can’t this be done like [functions/accent.js](https://github.com/Khan/KaTeX/blob/master/src/functions/accent.js)?

@ylemkimon That might work. I'll look into it tomorrow.

@ylemkimon I've followed your suggestion and placed the code into file genfrac.js. Thank you.
@kevinbarabash I've squashed the commits, or at least I've tried. Let me know if you want any further rearrangement.

@ronkok I'm going to hold off on generating screenshots for this until #1143 is merged.

Firefox

BinomTest-firefox.png

FractionTest-firefox.png
Chrome

BinomTest-chrome.png

FractionTest-chrome.png

@ronkok any idea of why some of the fraction bars are wider?

Yes, this PR supports \genfrac and \abovewithdelims, both of which give the author control over the fraction bar thickness. So I wrote screenshotter tests to demonstrate that capability.

Yes, this PR supports \genfrac and \abovewithdelims, both of which give the author control over the fraction bar thickness. So I wrote screenshotter tests to demonstrate that capability.

Cool!  Sorry... I haven't actually look at the code yet.

@kevinbarabash, If you plan to do some PR reviews, I think that PR #1249 should take precedence over this one. After #1249 is merged, I'll circle back to this one and deal with the merge conflicts.

I'm closing this PR temporarily. I'll reopen after I've dealt with a number of merge conflicts.


Add .editorconfig with two-space indents and Unix newlines

This will help avoid inconsistent indentation like the tab in my last commit. (Sorry about that!)
Most editors support .editorconfig either natively or with a plugin.
Sad but true: I meant to add a .editorconfig before fixing that bug. I always start a project with a .gitignore and .editorconfig before committing anything else. Live and learn, I guess. :-)

Thanks! :+1:


Failed to run yarn package-linux with Node.js >= 16

First of all, thank you for the package, it's really great!
I am using ArchLinux with Node.js version 17.3.0 and I noticed the same problem as described here: https://github.com/electron-userland/electron-builder/issues/5668 i.e. that the yarn package-linux command fails. Running yarn upgrade before that solves the issue by bumping electron-builder to version 22.14.5.
Could it be possible to already bump the electron-builder version in the repo, or at least add a mention in the README.md?

Thanks so much for feedback @VGuenat. It is updated now

Thank you for the quick fix, it is solved now. Closing.


Great job man! Have you thought about integrating this with boxstarter?

Have you thought about integrating this with boxstarter and chocolatey? https://github.com/Microsoft/windows-dev-box-setup-scripts
I actually might take my current boxstarter script and adjust my settings etc to fit your formatting etc so that it's easier to enable and disable comments using the commenting syntax you've used. Might put that up as a fork or something to yours though not sure yet.

There were already some discussions about Chocolatey in #23 and #52, plus the FAQ question "I'm using a tweak for <feature> on my installation, can you add it?" explicitly mentions it. I have never heard about Boxstarter, but from the documentation it seems that the parts where our projects are overlapping are marginal.
The most effortless way to integrate Win10-Initial-Setup-Script with virtually any package or automation manager is to simply download it along with a preset from your own repository and run. Something like
Invoke-WebRequest https://raw.githubusercontent.com/Disassembler0/Win10-Initial-Setup-Script/master/Win10.ps1
Invoke-WebRequest https://some.path.to/my.preset
powershell.exe -NoProfile -ExecutionPolicy Bypass -File Win10.ps1 -Preset my.preset

Other than that, I don't see how the script can easily be integrated with anything, especially when it's aimed to be initially run long before any other software is installed (can even be as part of sysprep).
So if you wish, feel free to fork and adjust to your liking, but I'm not planning to add support for any 3rd party management tools.

@Disassembler0 Totally reasonable. The purpose of Boxstarter is also sysprep but it adds additional functionality to plain powershell from my understanding, such as the ability to bootstrap and skip through program installation stuff.
One quick question though about your script.
You have the tweaks set up like this:
/#EnableFeature,        /#DisableFeature,
If I uncomment the Disable it is still line commented by the EnableFeature. Is the intention that in order to disable features one should uncomment both? So that it runs the Enable and then Disables it right afterwards? I could guess that you might do that because it would create a consistent toggling between the settings etc? Just wanted to check though.

If you're referring to the $tweaks array in the beginning of the script, then the #DisableFeature (and basically the whole second and for some tweaks also the third column) is there just to have a quick overview about the possibilities and to see what tweak needs to be run if one wishes to revert whatever EnableFeature did, without having to scroll several hundreds lines down to see how's the reversal tweak actually called and if there is even one.
Usually you don't need to revert or reapply all the tweaks, so instead of rerunning the whole script with DisableFeature uncommented, you pass it as command line argument
powershell.exe -NoProfile -ExecutionPolicy Bypass -File Win10.ps1 DisableFeature

For more info about command line arguments, refer to chapter Advanced usage in README. There will be some changes in near future when I free myself from other duties.

And yes, your assumption about consistent toggling is correct, so you could use both tweaks at the same time. But like I wrote above, there are cleaner ways to revert something.

Cool, thanks!


[ARI] Add Core Deps CI to Buildpacks Area

We missed this repo during the original team allocation
This repo contains configuration for a Concourse release pipeline

Per RFC 0007, this requires TOC approval.
As working group lead, I approve this addition.


Root of v188 has empty nodes

While trying to load the v188 chebi.obo file I discovered that the hierarchy could not be properly displayed in my local application.  I examined the raw data file and discovered there were two empty concepts at the root level and a handful of other populated concepts at the root level.  I believe this is an error.
See CHEBI_43262 and CHEBI_45394 for the empty concepts
See the following for possibly misplaced root concepts:
CHEBI_149697
CHEBI_150011
CHEBI_149732
CHEBI_149734

@safrant  Thanks for bring our attention, we will fix ASAP.

Everything seems to be fixed except CHEBI_150011.  That is still hanging out as a weird root node with no children. I am going to assume that is on purpose and close this ticket.

I have now fixed CHEBI:150011. Hopefully the issue will be resolved in ChEBI's next months release.


docs(repeat): Remove scheduler as param.

Description:
The RxJS 5.0 repeat operator does not take a scheduler as a param but is documented as having it as a parameter. I deleted this line. Where is the ReactiveX website version control hosted so I can change the documentation for the RxJS implementation of repeat?

reactivex.io/rxjs is built from this repository

Coverage remained the same at 97.677% when pulling 5b1ee5626f2e6453facca90ac915974e5b7b685a on jdetle:doc-fix into de6a8f4c9bf6026255319193edb923022237b3cd on ReactiveX:master.

http://reactivex.io/documentation/operators/ doesn't seem to be built here, and provides some erroneous documentation for the behavior of repeat.

@jdetle thank you! can you also remove it from the description too? https://github.com/jdetle/rxjs/blob/5b1ee5626f2e6453facca90ac915974e5b7b685a/src/operator/repeat.ts#L9

Returns an Observable that repeats the stream of items emitted by the source Observable at most count times, on a particular IScheduler.

Why don't we just make repeat use a scheduler? Seems like it should.

http://reactivex.io/documentation/operators/ doesn't seem to be built here, and provides some erroneous documentation for the behavior of repeat.

(incorrect link) http://reactivex.io/documentation/operators.html
Those docs are still RxJS v4 docs. We still need to coordinate how to get them updated and keep them in sync... 😢

@trxcllnt

Why don't we just make repeat use a scheduler? Seems like it should.

Indeed, but until someone steps up with the PR I'm inclined to remove it from the docs. Especially because we're short handed and it's unknown when it would get it. If you have the cycles, feel free--but I know you (and I) have tons of other things on our plate.

I can tackle it, I'm unemployed at the moment so all I have all the cycles
:)
On Mon, Feb 13, 2017 at 2:31 PM Jay Phelps<EMAIL_ADDRESS>wrote:

@trxcllnt https://github.com/trxcllnt
Why don't we just make repeat use a scheduler? Seems like it should.
Indeed, but until someone steps up with the PR I'm inclined to remove it
from the docs. Especially because we're short handed and it's unknown when
it would get it. If you have the cycles, feel free--but I know you (and I)
have tons of other things on our plate.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ReactiveX/rxjs/pull/2368#issuecomment-279544951, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AKcARtSZK0K3_RsXIcbjPokJXGfszFpbks5rcNnPgaJpZM4L_ru6
.

--
Have a wonderful day,
John J. Detlefs

Well, I think it should be step 1: PR to remove it from the docs... then step 2: PR to add it (it's a new feature, so it's a minor revision change)


Various small fixes

This fixes a number of small issues found while working on #9347

Documentation preview available at: https://linuxcontainers.org/lxd/docs/pr.10106/

@tomponline good to go


Rock: 发送文件，视频特别慢

Question

If you only want to apply for token,please contact Juzi.Bot(wechat:juzibot).

Important：Please file the issue follow the template, or we won't help you to solve the problem.

0. Report Issue Guide

Please run the following command and check whether the problem has been fixed：

rm -rf package-lock.json
rm -rf node_modules
npm install

Please search in FAQ List first, and make sure your problem has not been solved before.

Please search in the issue first, and make sure your problem had not been reported before

1. Versions

What is your wechaty version?
Answer:0.49.8

What is your wechaty-puppet-hostie version?
Answer:0.11.5

What is your node version? (run node --version)
Answer:10.16

What os are you using
Answer:MacOS

2. Describe the bug
Give a clear and concise description of what the bug is.
测试案例

发送2M mp3文件
发送500K mp4文件
发送10M pdf文件

3. To Reproduce
This part is very important: if you can not provide any reproduce steps, then the problem will be very hard to be recognized.
Steps to reproduce the behavior:

run '...'
...
...

4. Expected behavior
Give a clear and concise description of what you expected to happen.
5. Actual behavior
If applicable, add screenshots to help explain your problem. But do not paste log screenshots here.
6. Full Output Logs
Set env WECHATY_LOG=silly in order to set log level to silly, then we can get the full log (If you dosen't set log env, log level is info as default, we cannot get the full log)
We need full log instead of log screenshot or log fragments!

Show Logs

$ WECHATY_LOG=silly node yourbot.js

Question: Paste your FULL(DO NOT ONLY PROVIDE FRAGMENTS) log messages
Answer:

7. Additional context
Add any other context about the problem here.
[bug]

发送时间点：
16:19:06 SILL Contact ready() isReady() true

      ============================================
      on message : {"domain":null,"_events":{},"_eventsCount":0,"id":"1136019486","payload":{"filename":"","fromId":"Soul001001","id":"1136019486","mentionIdList":[],"roomId":"","text":"file","timestamp":1607069944,"toId":"wxid_8kg1wdu3jvk322","type":7}}, message type: 7
      text: file
      from: Contact<苏畅732>
      room: null
      ============================================

16:19:06 VERB Puppet selfId()
16:19:06 VERB Puppet selfId()
16:19:06 VERB Message mentionList()
16:19:06 VERB UrlLink constructor()
16:19:06 VERB Message say(FileBox#Base64<file.pdf>)
16:19:06 VERB PuppetHostie messageSend(Soul001001, FileBox#Base64<file.pdf>)

实际发送：


Tasks improvements

Can one of the admins verify this patch?

Current coverage is 94.95% (diff: 100%)

Merging #84 into master will increase coverage by 0.04%

@@             master        #84   diff @@
==========================================
  Files            28         28
  Lines           984        992     +8
  Methods           0          0
  Messages          0          0
  Branches         41         43     +2
==========================================
+ Hits            934        942     +8
  Misses           42         42
  Partials          8          8

Powered by Codecov. Last update 236b4b7...901e11c


Update response caching doc and sample

I've updated the sample from RC1 to RTM. Seems like the sample was based off of a web application template so I removed a lot of code that is irrelevant to response caching. I also updated the docs to use code from the sample instead of inline code blocks.
cc @Rick-Anderson @danroth27

Hi @JunTaoLuo, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

@rachelappel  please build and do a VTP (Visual Test Pass). Make sure the literalinclude looks OK.

:shipit: after addressing some minor comments.

@BrennanConroy Since I'm leaving for vacation, can you keep an eye on this PR and update it if needed?

@JunTaoLuo @BrennanConroy  it's complete and merged.

I have already addressed the comments listed here


manipulation.approach_depart_planner.py

Line 119: if len(conf_list) == 0 should be if conf_list is None
Line 121: should return None, None

Thank you! Updated!


There is a bug in the TTS function module.

There is a bug in the TTS function module. The answer can only read the sentence before the punctuation mark, and sometimes it will crash. Also, can you add a voice conversation function?

Thank you for bringing this problem to our attention. Could you give me more details about which watch you are using and which language is causing the error? I have just discovered that there can be problems if the correct language for TTS is not downloaded in the watch settings. Does your watch use the Samsung or Google text-to-speech engine?

What do you mean by "voice conversation function"? I already have voice output built in and if you use the Google or Samsung keyboard, you can also dictate. Or do you have something else in mind? :)

My watch is TICWATCH PRO 3. It uses the built-in keyboard of the watch. The voice will stop every time it reaches a punctuation mark.

Hm, that sounds very strange.
For voice input, try installing the official Google GBoard, which is much better than the pre-installed one:
https://play.google.com/store/apps/details?id=com.google.android.inputmethod.latin
Unfortunately, I can't help you with the TTS, as I can't reproduce the error (which is obviously caused by the system). Please check the settings of the watch to see which languages you have downloaded for the TTS and which engine is used.
Here is an APK based on the latest commit. Try to see if the error still occurs there, so that we are at least on the same level.
https://workupload.com/file/mXGeKgJm8MF

Thank you for the new version. I encountered a new problem. When I clicked New chat, I exited directly.

Hm, that shouldn't happen, of course. When you click on New chat, the keyboard is retrieved and the app crashes on you. Have you downloaded the GBoard and activated it as a keyboard in the settings? It's probably because the keyboard can't be extended and not because of the new app.

I downloaded and installed the latest version of GBoard, and installed the latest GOOGLE TTS as you requested. This problem occurred when clicking NEW CHAT on the new version.

Thank you for your reply. My watch system is too old and is still stuck in the W OS2 version. TTS is still not working properly and can only play the sentence before the punctuation mark. .

Hm, that's what I thought. The code to open the keyboard only seems to work for Wear OS 3. Unfortunately, Google's documentation is very very poor and since I have a lot to do in the next few days, it will probably take until next week to fix the bug.
As for the TTS, I'll see if there's anything I can do.
Thanks for the suggestion for the settings, I will implement that as well.

Hey again,
I just released WristAssist 2.1.0 which includes your feature request for Auto TTS. I also managed to update the code for the keyboard. On Versions < Wear OS 3.0 (Android 11) it uses the old code, on all newer version it uses the code that didn't work for your watch, but does regular.
For the TTS system I cannot find a better approach. Please test whether this feature works now for you with Google TTS.
Thanks again for pointing to all these issues. :)

Thank you very much 😊


update docs

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@f0df3f5). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master      #68   +/-   ##
=========================================
  Coverage          ?   23.69%
=========================================
  Files             ?       11
  Lines             ?      249
  Branches          ?        0
=========================================
  Hits              ?       59
  Misses            ?      190
  Partials          ?        0

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f0df3f5...ff89fde. Read the comment docs.


Dark Mode does not seem to work?

Hi, I downloaded the latest version 0.2.1 and followed the guide of importing and then setting up the Dark Mode, however it fails to swap the view over/stays light colours. Is there anything else?
I import following:
import JsonView from 'react18-json-view'
import 'react18-json-view/src/style.css'
import 'react18-json-view/src/dark.css'

And use as such:
<div class="item rightpane">
<JsonView src={myFormData} collapsed={false} dark={true} />
</div>

Also wanted to say thank you so much for making this, it works fantastic and is a great help to me!

The dark property just helps you keep component in dark mode. To make it flexible, you only need to import the dark.css file. Then, you can set the dark class on any parent component (such as html).
You can visit the website https://jv.yysuni.com/ and learn about how it works by examining this file.

Don't forget to give me a star if it's helpful for you~~ 🥰😁

The dark property just helps you keep component in dark mode. To make it flexible, you only need to import the dark.css file. Then, you can set the dark class on any parent component (such as html).
You can visit the website https://jv.yysuni.com/ and learn about how it works by examining this file.

So if I understand it, I need to set a dark-mode on my overall website/app for it to reflect in the JSON View?

Yes..

Got it, thank you! (I did star project by the way!) <3


Error Installing

Description
Hello
When trying to install, i get an error that says package name does not match expected package name. The rest of the message is cutted. See the capture of the installer.
Thanks and have a nice day.
Steps To Reproduce
Install with installer.
Requirements

e.g. Fix crash that occurs when...
Install with installer

Files Version
<IP_ADDRESS>
Windows Version
22631.3958
Log File
debug.log

@yayator do any of the alternative methods listed on our docs page work on your device?

Hello
Thanks for the quick response. I tried other methods, but don't work. Same error.
Attached screenshot of powershell

De: Yair @.>
Enviado: miércoles, 7 de agosto de 2024 5:14
Para: files-community/Files @.>
Cc: yayator @.>; Mention @.>
Asunto: Re: [files-community/Files] Error Installing (Issue #15967)
@yayatorhttps://github.com/yayator do any of the alternative methodshttps://files.community/docs/install#:~:text=Install via PowerShell listed on our docs page work on your device?
—
Reply to this email directly, view it on GitHubhttps://github.com/files-community/Files/issues/15967#issuecomment-2272534619, or unsubscribehttps://github.com/notifications/unsubscribe-auth/BKLYYVZ6BUVTAVWTJ3IUIFTZQGGJJAVCNFSM6AAAAABMDP2Y5KVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDENZSGUZTINRRHE.
You are receiving this because you were mentioned.Message ID: @.***>

Sending message from email doesn't work with images. Please send it via GitHub directly

Can you try

Make sure Files is closed from the system tray
Open an admin PowerShell window (Windows default not PowerShell 7)
Run the below

Get-AppXPackage Files* | Remove-AppXPackage

Check the Start menu > All apps for Files


Complete tagging for OKE

Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Description

The following tagging features are missing:
       persistent_volume_config {

            defined_tags = {"Operations.CostCenter"= "42"}
            freeform_tags = {"Department"= "Finance"}
        }
        service_lb_config {

            defined_tags = {"Operations.CostCenter"= "42"}
        }

In node pools, the tagging is also not yet added.
For defined tags, we also need to add instructions for how to create policies
New or Affected Resource(s)

Potential Terraform Configuration

# Copy-paste any Terraform configurations for how the requested feature may be used.

References

Done in #552

The linked PR adds freeform tags handling for pvc and nodepools but doesn't introduce defined tags for any resource.
The latter would be very useful, should I open a separate issue?

Yes please go ahead. I wanted to close this PR and cut a new release because of the helm bug.


Detect duplicate requests and raise error

This change is somewhat debatable. Carefree users may prefer duplicate texts to just be handled, perhaps with a warning instead of an error.
However, efficiency and correctness is the only reason to choose to use texttunnel over the openai package in the first place, so I think typical users of the package would hate wasting money on duplicate requests.

This change is somewhat debatable. Carefree users may prefer duplicate texts to just be handled, perhaps with a warning instead of an error.
However, efficiency and correctness is the only reason to choose to use texttunnel over the openai package in the first place, so I think typical users of the package would hate wasting money on duplicate requests.

Hmm just a question on the general approach: what if I wanted to take the 'boot-strapping' approach where I send the same request e.g. 20 times with a higher temperature. Would that get treated as a duplicate request? (as in, could I do it, if I so wished?) @psimm

Hmm just a question on the general approach: what if I wanted to take the 'boot-strapping' approach where I send the same request e.g. 20 times with a higher temperature. Would that get treated as a duplicate request? (as in, could I do it, if I so wished?) @psimm

No, that wouldn't be possible after this PR is merged. The requests would all have the same hash.

Hmm just a question on the general approach: what if I wanted to take the 'boot-strapping' approach where I send the same request e.g. 20 times with a higher temperature. Would that get treated as a duplicate request? (as in, could I do it, if I so wished?) @psimm

No, that wouldn't be possible after this PR is merged. The requests would all have the same hash. You can run requests with different temperatures, say 0, 0.1, 0.2... but not multiple requests with identical text and parameters.

But I guess it would be quite a niche use. Just trying to think if there might be any other cases where sending a duplicate request would not be an error..

Ok, let's go with this then.
If you're reading this comment in the future and the changes introduced here are disrupting your use of texttunnel, please let me know. There are options, such as having an argument allow_duplicate_texts that defaults to False.


Is there a way to merge this model with MS COCO dataset somehow?

Not really an issue, but more of a question

You can train your own data with the origin author's implementation.


writeimageinfo without redownloading already downloaded images

for a significant part of my downloads I have not set writeimageinfo  to true.
is there a way to download this metadata info without redownloading all the images?
thank you in advance

Assuming the downloaded file still in the target location and you are using download from list (opt 4) or member id, or if you have the list of the images, you can try to setup the config ini:
daylastupdated = 0
overwrite = True
alwaysCheckFileSize = True
writeimageinfo = True

It will go thru the list you provide and will only download if the image have different file size.

i followed your instructions, and it works. then i tried to do it with
overwrite= false and it still works, this was much faster too as the
overwrite=true redownloaded the image it seems, this made it much slower.
thanks for your help! great software.
On Fri, Oct 18, 2019, 00:44 Nandaka<EMAIL_ADDRESS>wrote:

Assuming the downloaded file still in the target location and you are
using download from list (opt 4) or member id, or if you have the list of
the images, you can try to setup the config ini:
daylastupdated = 0
overwrite = True
alwaysCheckFileSize = True
writeimageinfo = True
It will go thru the list you provide and will only download if the image
have different file size.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Nandaka/PixivUtil2/issues/538?email_source=notifications&email_token=AABOMA7RRE4G4DG52CXKOLTQPDTGDA5CNFSM4I5AA222YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBRYGAA#issuecomment-543392512,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AABOMA4QKU7LZMNTTO5XPSTQPDTGDANCNFSM4I5AA22Q
.


Update superannotate-tutorial.md

We need to update the SSO setup section on the SuperAnnotate side.

@jeevansd
Can you review the proposed changes?
When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@annak-superannotate, creating a separate PR in our private repo for better content validation, the changes will update within 2-3 working days.
#please-close


refactor: correctly type utils

Summary
Improves public facing types from nextcord.utils to be fully annotated.
Sets pyright to strict (apart from rules which cause errors from array.array).
pyright --verifytypes nextcord is now improved by .9% :sweat_smile:

Checklist

[ ] If code changes were made then they have been tested.

[ ] I have updated the documentation to reflect the changes.
[x] I have run task pyright and fixed the relevant issues.

[x] This PR fixes an issue.
[ ] This PR adds something new (e.g. new method or parameters).
[ ] This PR is a breaking change (e.g. methods or parameters removed/renamed)
[ ] This PR is not a code change (e.g. documentation, README, ...)

Does not also have the side effect of marking some of these utils now as public API? At-least per the documented reasons

Yeah, probably, they were made public since utils._* outside of utils, and from utils import _* are both using private members outside of where it is defined.

Fair enough, I don't see much of an issue with making them public


HTTP 500 on staging website

Please fix so the staging website can be used again for AppCie-developers!

Fixed!


Lots of warnings when installing angular 9.0.0-next.7

🐞 bug report
Is this a regression?

 Most likely, although it has been a few preview versions since I tried this.
Description
 A clear and concise description of the problem...
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
warning<EMAIL_ADDRESS>The engine "pnpm" appears to be invalid.
[3/4] 🔗  Linking dependencies...
warning " ><EMAIL_ADDRESS>has incorrect peer dependency "<EMAIL_ADDRESS>|| ^8.1.0-beta.0 || ^8.2.0-beta.0 || ^8.3.0-beta.0 || ^8.4.0-beta.0 || >=9.0.0-beta < 9".
warning "@angular-devkit/build-angular ><EMAIL_ADDRESS>has incorrect peer dependency "<EMAIL_ADDRESS>|| ^8.1.0-beta.0 || ^8.2.0-beta.0 || ^8.3.0-beta.0 || ^8.4.0-beta.0 || >=9.0.0-beta < 9".

🔬 Minimal Reproduction

https://github.com/fr0/angular-test/tree/ivy
🌍  Your Environment
Angular Version:

Angular CLI: 9.0.0-next.5
Node: 12.6.0
OS: darwin x64
Angular: 9.0.0-next.7
... animations, common, compiler, compiler-cli, core, forms
... language-service, platform-browser, platform-browser-dynamic
... router

Package                           Version
-----------------------------------------------------------
@angular-devkit/architect         0.900.0-next.5
@angular-devkit/build-angular     0.900.0-next.5
@angular-devkit/build-optimizer   0.900.0-next.5
@angular-devkit/build-webpack     0.900.0-next.5
@angular-devkit/core              9.0.0-next.5
@angular-devkit/schematics        9.0.0-next.5
@angular/cli                      9.0.0-next.5
@ngtools/webpack                  9.0.0-next.5
@schematics/angular               9.0.0-next.5
@schematics/update                0.900.0-next.5
ng-packagr                        5.5.1
rxjs                              6.5.3
typescript                        3.5.3
webpack                           4.40.2

That's unfortunately a defect of yarn itself.  It should not be displaying warnings for pnpm as an engine field.   It is a valid field which the pnpm package manager checks.

Tracking here: https://github.com/yarnpkg/yarn/issues/7560


aetcd3: use aiofiles >= 0.6

Signed-off-by: Joao Eduardo Luis<EMAIL_ADDRESS>

The library uses only basic aiofiles API, so I've just relaxed the supported version range to include 0.6.x in 864cd19360a7b361b7869e8d116ce169229bc3d1. Closing the merge.


Make README.md More Robust and Pretty

The README.md is not that pretty so if you would like to help get a PR in feel free to try and beautify the README with more info, images, gifs, and whatever you deem nice to see on a professional project's README.

I can do this!


update provider example with Alchemy provider

Your ENS/address: thatguyintech.eth

@thatguyintech thanks! Let's add these type of changes to the next branch.


Use same "moduleName" throughout docs

I tripped over the different "moduleName" in the documentation. This change should help people try out this package more easily.

:+1:


DEBUG CHECK: Missing HeadEnt table

I modified advdup2 to Rev4 instead of 5, because the server I use only accepts this version of files, but when I try to save a duplicate, it gives this error. The file is created, but when I try to open it, it says: File could not be decoded. (Missing HeadEnt table) Upload Canceled.
What should I do to solve this?

Can you give more info about the modification?

I just modified this line, because from what I understood, it was possible to change the advdup Rev this way. But when I use it, I can't save anything, the created file is corrupted.

I believe this doesn't work—AdvDupe2 only contains deserializer (loading) revisions. For the serializer (saving), you're still using revision 5, but it's now being mislabeled as revision 4. You'd need to install an older version of AdvDupe2 or modify the serializer to conform to the revision (I could be wrong but I think the only difference is how the data is formatted).

Yeah, I think you're right. Why you even doing this in the first place?

As I said, the server I play on uses this Rev4 version of AdvDup, so if I want to build a single player build and transfer it to the server, I can't, and the tool even crashes if I try to transfer a Rev5 file to advdup Rev4.
Could I have a version prior to Rev5?

It's 6 years old. The server should update.
https://github.com/wiremod/advdupe2/archive/d06b7e1f7f19ebb0edfd2fb1be94e2b79654febf.zip

Thanks for help!


Uncatched table formatting case

| Foo  | Bar  |
| --- | --- |
| Alfa | cosi |

Create easy table (example above):
If you format, it works.
Add space after last hyphen collumn (space or space and other text) and press format shortcut.
Get undefined before text and console error.

Offtopic:
Also i noticed there is some exception when you delete hyphen

vscode: 1.24-insiders
yzhang.markdown-all-in-one: 1.4.0

Thanks. You can get the latest build here.


New feature to generate a xml document from XML::Accessor

SwiftyXMLParser only supports parsing XML.
But it may be helpful to support reproducing raw XML.
Have the discussion here.
https://github.com/yahoojapan/SwiftyXMLParser/issues/28

Hello I am using SwiftyXMLParser and need help in generating RawXML from Accessor. But i am not finding any solution to achieve it. Can you please suggest how to start on this.
Thanks in advance.


does not respect "editor.tabSize": 2

I have an issue with vscode-zig extension. I don't want to disable zig autoformating/formatOnSave but I want it to respect my vs code settings "editor.tabSize": 2. But it always format using 4 spaces/tab-size irrespective of my setting. And I can't seem to find an option to tell vscode-zig extension to set tab size to 2 spaces. Please help me how to change vscode-zig formatter behavior to respect my vscode tabSize setting.

The formatting is provided by zig fmt which is not configurable by design.

I understand. But having such a setting really make a difference for people with small screen size such as myself using only half of the screen size. I hope such a feature appears in the future.


docs: updated doc site and json schema

Demo

Latest doc site from https://github.com/GoogleCloudPlatform/gcloud-common/pull/34
Latest JSON from https://github.com/GoogleCloudPlatform/gcloud-node/pull/1096

Looks like there are some conflicts, which isn't hard to believe when there are over 1000 changes files! Demo looks awesome (as awesome as always).
I just noticed that http://callmehiphop.github.io/gcloud-node/#/docs/v0.27.0/datastore/transaction has a @linkcode{ snippet that doesn't seem to fit in. Did we forget something in our parsing situation?

:clap: x 1,094
Found some hopefully small things!
Some {@linkcodes are here as well: http://callmehiphop.github.io/gcloud-node/#/docs/v0.27.0/pubsub/subscription
--
The Subscription constructor has some malformed examples. See the Once you have obtained a subscription object, you may begin to register listeners section.
--
Links like these: http://callmehiphop.github.io/gcloud-node/#/docs/v0.27.0/dns

Seem to be pointing to a dead page now (http://callmehiphop.github.io/gcloud-node/#/docs/?method=gcloud).
I think they just need the ?method=gcloud to be removed?
--
http://callmehiphop.github.io/gcloud-node/#/docs/v0.27.0/storage/bucket?method=acl.acl.delete
acl.acl.delete? I think there's an extra acl. in there.
--
Any reason initial-width=1 was removed? It makes my phone (Note 5) start scaled down (.8):

*this is using the Note 3 emulator*
I tried with others using the emulator, and they seem to be affected as well.
The inner docs pages work for my physical device, but the simulator shows a different story for example:
Note 3:

iPhone 5:

--
Small one!:
new site:

old site:

https://github.com/callmehiphop/gcloud-node/tree/08afcbb5a135677d66e3cffd8aed19c23b5b94c5/scripts seems to have extra files in there. Also, can you point me to where the script is that gives them the unique hashes?

I just noticed that http://callmehiphop.github.io/gcloud-node/#/docs/v0.27.0/datastore/transaction has a @linkcode{ snippet that doesn't seem to fit in. Did we forget something in our parsing situation?

Yeah, slipped my mind when generating the JSON for previous versions, should be fixed now though!

I think they just need the ?method=gcloud to be removed?

Good catch! Fixed!

acl.acl.delete? I think there's an extra acl. in there.

This might have been a side effect from all the @alias tags, I ended up removing them from master.. which is probably why I missed it!

Any reason initial-width=1 was removed?

Accident! Went with a yeoman scaffold and didn't realize their meta tag would be different.

Small one!

Should be fixed!

seems to have extra files in there. Also, can you point me to where the script is that gives them the unique hashes

Ah, yeah, I suppose when we update the site we'll need to do a little clean up. The hashes are made via gulp script when running gulp build.

The Subscription constructor has some malformed examples. See the Once you have obtained a subscription object, you may begin to register listeners section.

Eek, going to look into this one, must be the example parsing code.

Ran into an issue where certain versions of dox changed where the source line no. was, (I had to rollback versions to avoid parse errors in previous releases), so I need to go through and tidy up those.

Sorry :(

Don't be!

@stephenplusplus if you want to take another look, I think I've tackled everything

LG :+1:, but the Small one! is still there:

Just to clarify, I mean the space between "Version" and the dropdown. Previously, they had some breathing room, but now they're best friends.

Small one! is still there:

Ahh, I think that might have resurfaced from minifying the HTML. I added a


Add support for RN platform configuration

Add support to configure React Native platform configuration.
This will enable us to switch between React Native projects, currently supporting default and tvos platforms.
By default configured to use "default" RN project platform.
Steps for switching to RN TVOS project or vice versa

In existing workspace, delete react-native and node_modules folder
gclient setdep --var=react_native_platform=tvos
gclient setdep --var=react_native_platform_revision=react_native_tvos_revision
gclient setdep --var=react_native_platform_git=react_native_tvos_git
gclient sync --with_branch_heads
yarn
Build commands

@Kudo  @Taym95  Have updated changes based on your feedback.
Some additional changes are also added now to support building with the different configurations.

@munezbn
Currently only the change in view component BUILD.gn was required under configuration here.


Figure out what to do with rbi.yaml

rubocop-sorbet ships with this config for rbi files and I'm unsure what to do about it.
We can't just merge it into the config for RBI files b/c it reconfigures a bunch of base rubocop rules (which is disallowed by Standard).
The one idea I have is for a plugin setting like this:
plugins:
  - standard-sorbet:
      ignore_unnecessary_rules_for_rbi_files: true

And then given that configuration, add in a nested ignore somehow (off the top of my head I'm not sure this would work), similar to how you can ignore rules for entire globs in standard.yml
ignore:
  - '*.rbi':
    - Layout/AlignHash
    # … etc etc

Unsure what else might make sense or be workable

Is there a way to "boot" Rubocop with a different configuration that only targets *.rbi files? That way, we could have two different rulesets configured and enforce a standard, non-configurable set of rules specifically for RBI files.
We normally ignore linting on RBI files in our Rubocop configs, so I admit I haven't thought too much about how they should be linted/formatted.

That is possible but I'm not sure it's worth it.
I think the right solution is an ignore glob that effectively disables the same rules as that rbi config file, but a plugin can't provide that (and I'm not sure it should, since in this case we'd be disabling rules that are owned by other standard plugins). Flip side is that it'd be a lot to paste into every .standard.yml file


Migrate web conditional import

When I migrated to web library in https://github.com/CodingWithTashi/simple_barcode_scanner/pull/52, I forgot to migrate the conditional import. This will enable support to wasm compilation.
Migrate to package:web

@pedromneto97,  Thanks for your contribution.


Failed to start the visualizer(Internal error  in function fgOpenWindow)

I'm trying to run a simple code, by connecting to a remote linux server. And I have done X11 forwarding, but I still fail to start the visualizer when "env = RunEnv(visualize=True)".
(opensim-rl) xuanyang@machine10:~$ python ./py2/testpy2.py
Updating Model file from 30000 to latest format...
Loaded model gait9dof18musc_Thelen_BigSpheres.osim from file /home/xuanyang/anaconda2/envs/opensim-rl/lib/python2.7/site-packages/osim/env/../models/gait9dof18musc.osim
freeglut (simbody-visualizer):  ERROR:  Internal error <FBConfig with necessary capabilities not found> in function fgOpenWindow
Traceback (most recent call last):
  File "./py2/testpy2.py", line 3, in <module>
    env = RunEnv(visualize=True)
  File "/home/xuanyang/anaconda2/envs/opensim-rl/lib/python2.7/site-packages/osim/env/run.py", line 43, in __init__
    manager.integrate(state)
  File "/home/xuanyang/anaconda2/envs/opensim-rl/lib/python2.7/site-packages/opensim/simulation.py", line 38364, in integrate
    return _simulation.Manager_integrate(self, *args)
RuntimeError: std::exception in 'bool OpenSim::Manager::integrate(SimTK::State &)': SimTK Exception thrown at VisualizerProtocol.cpp:378:
  Error detected by Simbody method VisualizerProtocol: An attempt to write() 1 bytes to pipe 4 failed with errno=32 (Broken pipe).
  (Required condition 'status!=-1' was not met.)

It might be the same problem as here https://github.com/simbody/simbody/issues/550 (and https://github.com/simbody/simbody/issues/563), which are not resolved yet. Normally it is enough to restart the script. We had a similar problem with generating a video. @spMohanty, do you have some solution here?

Yeah, this looks like the same issue.
The only makeshift fix we have for now, is to both restart the environment, and the script.
Just restarting the script also keeps throwing the same broken pipe error.
Ugly, true, but thats the only working solution we have for now 😢


SwiftData Best Practices

I'm moving my app to using SwiftData to store the data a user needs on device so that the user spends less time waiting for Supabase queries to return. I'm not an expert, but I suspect it may be more inline with the way Apple wants developers to use SwiftUI and if I've learned anything it is that the more you align with what Apple is trying to get you to do the less pain.
I saw this discussion, but it didn't go very far.
So, assuming it is a good idea (you can stop me right here if not), I would think that this repo could benefit from having an example of how to do that that the community can use for inspiration or improve on. That said, I'm also assuming the maintainers are plenty busy with everything else that is not this. If it actually has a reasonable likelihood of being useful I could try to put together a stripped down version of what I'm doing as a sample app.
The plus side would be that it is at least something for people to consider and work from. The negative would be that I may be doing horrible things that would waste maintainer time to even review and shake their heads at.
If I don't hear back I'll just assume this offer would do more harm than good, but if you think it might help I'll work on contributing and example.

I think it'd be great to have more examples and documentation on how to integrate with SwiftData, I tried it a bit but it was painful so I gave up.

Hi all,
Supabase works with Codable types, and it makes no assumption on any local storage, one approach would be to make your @Model classes conform to Codable (https://www.donnywals.com/making-your-swiftdata-models-codable/), and use it directly as Supabase response/request objects.
Other than that, there's no best practices to make Supabase work with SwiftData, as they are unrelated frameworks, how data is structured it will depend on the application structure.
For now, it is up to the developer to implement a persistence layer that coordinates the data between Supabase <-> Local Storage.
@im-murilo I'd be glad helping you with the issues you had setting up SwiftData for storing the Supabase's objects if I'm able to.
Thanks.

Fair enough! I'll close this. Thanks for all the hard work


racer doesn't discover variables inside llvm_sys files

Completion works in this case:
extern crate llvm_sys;

use llvm_sys::

However, it doesn't work here:
extern crate llvm_sys;

use llvm_sys::core::

Hi @Wilfred. I think this is because all the functions in llvm_sys::core:: are extern "C", and racer doesn't support extern yet.
@varding opened #347 with the same problem. I will try and get this functionality added soon

Aha, that looks like the issue. Should I close this issue in favour of #347?

Yeah, that would be good thanks. If after I've fixed that this still
doesn't work then reopen.
Thanks!
On Sun, Aug 23, 2015 at 11:20 AM, Wilfred Hughes<EMAIL_ADDRESS>wrote:

Aha, that looks like the issue. Should I close this issue in favour of
#347 https://github.com/phildawes/racer/issues/347?
—
Reply to this email directly or view it on GitHub
https://github.com/phildawes/racer/issues/346#issuecomment-133813230.


How to change height of the bottom line to 1 pt?

How to change height of the bottom line to 1 pt?

Hi,
This is not yet planned. Thanks for point out this, I will add this feature in a coming version.
I will keep you in touch

Thanks.


add option to dump/load database to/from file

It could be a nice addon that the data dumped with .dump() can be saved to a file and later restored using FakeFirebaseFirestore(data: {}) or something like that

Imo that's outside the scope


Request for training examples using PipeStage and PipeSchedule

Would it be possible to provide a few examples on how to train a network using pipestage and pipeschedule? all the examples i have gone through so far are dedicated for inference.

We are working on consolidating PipelineStage's runtime code with that from PipelineSchedule. An outcome of that would be training support. We will create a few examples when it is done. If you'd like an early example, here is a unit test for backward for the consolidated code:
https://github.com/pytorch/PiPPy/pull/980/files#diff-a99c7fe997a1aef6c41fadf81ff7fb13e27172dd7f97b87dd81ca7d38cf833a9

Here is a simple training example:
https://github.com/pytorch/PiPPy/blob/main/examples/basic/example_train.py

@kwen2501 Hope you can provide more complicated e2e examples. I noticed pytorch will add HSDP + TP/SP as new feature. It's great if you could create a 3D(FSDP + TP/SP + PiPPy) or even 4D(HSDP + TP/SP + PiPPy) parallel HF training demo. It seems that all these features do not need to modify the LLM's source code.


Alternative labels type

Alternative label placement

Alternative label placement with an optional step

Reference: Type of steppers.

Thx for your jobs ahlechandre ! It's stepper awesome  :p


Enable more characters in commands for other languages.

Hello! When using the streamdeck plugin for fivem and placing, for example, a command (/me) with tildes typical of other languages (á)such as Spanish, or the character "ñ" and even the question mark (¿) do not work when placing them. Could something be done to include them? Thanks in advance.

I have two commands written in this case with these characters and as you can see, they don't work.
[COMMANDS]
me Le coloca el dedo índice y el dedo medio sobre el cuello bajo el ángulo del mentón y aplica presión muy ligeramente en busca de pulso carotídeo.
do ¿Tendría pulso? ¿Cómo sería este?
[IMAGE]
https://imgur.com/qC68xD9.png

Do these work if you do the command in console manually?

Do these work if you do the command in console manually?

Yes. If I put them manually in the game console it works fine. I attach an image.
[IMAGE]
https://imgur.com/uoImKEc.png

The thing is, that the characters that it does not identify are replaced by "?"
As are the tildes, the word "ñ" and also the question mark backwards "¿"

Hello again! Can you tell me if this only happens to me or is it something that happens to everyone with these characters and letters?

Hi, I haven't had a chance to test or look in to it yet but I assume it would affect everyone - it may be a limitation of the protocol, fivem or may be a simple change required in the plugin

Hi, I haven't had a chance to test or look in to it yet but I assume it would affect everyone - it may be a limitation of the protocol, fivem or may be a simple change required in the plugin

Okeyy. That change in the plugin should be done by me? If so, I don't know how to do it.

Hello, this is supported in the latest update:
https://github.com/EggRP/fxcommands/commit/1743f0630b6ff8f855379e138eb6583423d2907c

You can download the latest release from the releases page, I will have it updated in the Streamdeck plugins store shortly:
https://github.com/EggRP/fxcommands/releases/tag/v1.1.0

Hello, this is supported in the latest update:
1743f06

You can download the latest release from the releases page, I will have it updated in the Streamdeck plugins store shortly:
https://github.com/EggRP/fxcommands/releases/tag/v1.1.0

It works perfectly. Thank you very much for updating it, fixing it and everything!! <3

Awesome, you're welcome!
I will close this - if you have any new ones come up please open a new issue


Client:  1.0.18-20161220170039+7435b49

Thanks for doing all of the hard work!
I did a quick pass to bring the args up to current for inclusion in my docker-keybase project. They seem to work properly for me now but I always welcome a second set of eyes.
Cheers!

Thanks for the update and sorry for the long wait.


feat: support transformer replaceRegex

Proposed Changes:
Add transformer replaceRegex action.
like:  https://grafana.com/docs/loki/latest/clients/promtail/stages/replace/
Which issue(s) this PR fixes:

Fixes #630
Additional documentation:
interceptors:
  - type: transformer
    actions:
      - action: replaceRegex(body)
        expression: <string> // The RE2 regular expression. Each named capture group will be replaced with the value given in `replace`
        replace: <string> //
      ...

expression needs to be a Go RE2 regex string.
replace is a replace value.
use:
    actions:
      - action: replaceRegex(body)  // SSN
        expression: '([0-9]{3}-[0-9]{2}-[0-9]{4})'
        replace: '*SSN*'
      - action: replaceRegex(body)  // IP4
        expression: '(\d{1,3}[.]\d{1,3}[.]\d{1,3}[.]\d{1,3})'
        replace: '*IP4*'
      - action: replaceRegex(body)  // email
        expression: '([\w\.=-]+@[\w\.-]+\.[\w]{2,64})'
        replace: '*email*'

/cc @ethfoo PTAL


feat: reorganize repo, fix some issues and v1.0.0

Hi! 👋
Firstly, thanks for your work on this project! 🙂
Today I used patch-package to patch<EMAIL_ADDRESS>for the project I'm working on.
I notice in my terminal that I was receiving this warning:
Server-0        | remix-seo: A Twitter image should use alt text that describes the image. This is important for users who are visually impaired. Please add a text value to the `alt` key of the `twitter.image` config option to dismiss this warning.

And when I saw the browser was assigning the URL to twitter:image:alt instead of the provided alt.
Additional to this fix, this PR introduces:

Use native types from @remix-run package
A better organization of the files of the project
A fix that comes from this PR https://github.com/chaance/remix-seo/pull/3
Utilities for warn function that receives an array of strings and print the proper console logs with jump lines
Update the docs for mobileAlternate and languageAlternate to use Google documentation about SEO
Updated dependencies of all the project

Here is the diff that solved my problem with patch-package:
diff --git a/node_modules/remix-seo/dist/index.js b/node_modules/remix-seo/dist/index.js
index eeefa22..905f632 100644
--- a/node_modules/remix-seo/dist/index.js
+++ b/node_modules/remix-seo/dist/index.js
@@ -122,7 +122,7 @@ function getMeta(config, arg) {
       warnIfInvalidUrl(twitter.image.url, `The twitter:image tag must be a valid, absolute URL. Relative paths will not work as expected. Check the config's \`twitter.image.url\` value.`);
       meta["twitter:image"] = twitter.image.url;
       if (twitter.image.alt) {
-        meta["twitter:image:alt"] = twitter.image.url;
+        meta["twitter:image:alt"] = twitter.image.alt;
       } else {
         warn("A Twitter image should use alt text that describes the image. This is important for users who are visually impaired. Please add a text value to the `alt` key of the `twitter.image` config option to dismiss this warning.");
       }
diff --git a/node_modules/remix-seo/dist/index.mjs b/node_modules/remix-seo/dist/index.mjs
index a323502..06d209c 100644
--- a/node_modules/remix-seo/dist/index.mjs
+++ b/node_modules/remix-seo/dist/index.mjs
@@ -95,7 +95,7 @@ function getMeta(config, arg) {
       warnIfInvalidUrl(twitter.image.url, `The twitter:image tag must be a valid, absolute URL. Relative paths will not work as expected. Check the config's \`twitter.image.url\` value.`);
       meta["twitter:image"] = twitter.image.url;
       if (twitter.image.alt) {
-        meta["twitter:image:alt"] = twitter.image.url;
+        meta["twitter:image:alt"] = twitter.image.alt;
       } else {
         warn("A Twitter image should use alt text that describes the image. This is important for users who are visually impaired. Please add a text value to the `alt` key of the `twitter.image` config option to dismiss this warning.");
       }

This issue body was partially generated by patch-package.

Hi @danestves, thanks for the PR!
I'm happy to take a look, but as it stands this PR tries to do quite a lot that I think would be better split into chunks so that it's easier to review and merge the important stuff (like bug fixes) up front.
In general I'd prefer to keep this as a single file for now. I don't see any reason to add the complexity of separate files when Remix is going to code-split everything for you anyway.

Well you have right on that @chaance! I'm going to close this PR and only going to open the one with the fix for the Twitter image ☺


add badges with (plausible) counter

I like to have one of several badges with counters, based upon the plausible.io data.
Please start with the viewer-statistics page on arc42.org, and put the badges in there.
We decide later if we use these on other pages too...
plausible.io
They offer an API:
https://plausible.io/docs/stats-api
( I suggest to try out Postman for experiments, they have nice docu on that)
Especially interesting:
visitors - The number of unique visitors.
pageviews - The number of pageview events.
that would make TWO badges for each of the following sites:

arc42.org
arc42.de
docs.arc42.org
faq.arc42.org
quality.arc42.org

Badges via shields.io
See their documentation: https://shields.io/endpoint

imho needs to be done with JavaScript (e.g. fetch() method), but is a rather geeky enhancement.
Currently, let's leave it open


Why does JNI code require armeabi-v7a-hard-softfp and not armeabi-v7a-hard- ABI?

Reference: https://github.com/sh1r0/android-cmake/commit/20adf29a4ae6960f87f395989768935e54b1f529

https://groups.google.com/forum/#!msg/android-ndk/NbUq9FDDZOo/TJJsAS6nM7wJ

Thanks @sh1r0 as per the link above JNI functions should be compiled with softfp but armeabi-v7a-hard-softfp still sets the ABI to armeabi-v7a-hard and CXXFLAGS to -mhard-float. The only difference being the flag -D_NDK_MATH_NO_SOFTFP=1, which I am not sure how it helps in this case?

Please refer to tests/device/hard-float/jni/Android.mk in Android NDK.

I did and I understand the meaning of D_NDK_MATH_NO_SOFTFP=1 but I am not sure why is that needed to compile JNI code. Why can't armeabi-v7a-hard be used to compile JNI code. It seems like someone was able to compile the JNI code with armeabi-v7a-hard with NEON at https://github.com/sh1r0/caffe-android-lib/issues/27#issuecomment-167722576. So my original question still remains, why armeabi-v7a-hard-softfp and not armeabi-v7a-hard?

Sorry, per my previous tests, the current setup is the only way I can successfully make it. I guessed the key point is that JNI somehow needs softfp instead of hardfp. But if you succeed in making everything work by simply using armeabi-v7a-hard, please kindly let me know (PR is also welcome). Thanks.

But again armeabi-v7a-hard-softfp that are using is still using hardfp with the exception on using libm for soft abi math functions. If JNI needs softfp then we should compile JNI code with softfp and not with armeabi-v7a-hard-softfp or armeabi-v7a-hard.
Thanks anyways. My setup is a little different where I don't have any JNI code but was trying to use openblas with softfp on ARM7 as rest of my system is softfp but OpenBLAS crashes on me. I have other libraries that are dependent on OpenCV for example so I can't compile opencv with hardfloat without breaking these other modules which would be required if I am compiling Caffe and OpenBLAS with hardfloat. I don't want to compile two copies of opencv one with softfp for other softfp module and hardffp for caffe and OpenBLAS.
Do you know of a way to call from softfp based library using floating point to call into Caffe compiled with  hardfp? The link you mentioned says that adding
__attribute__((pcs("aapcs"))) is to tell to the compiler how a function expect to receive its arguments so it won't help unless I change the Caffe code to add this attribute to all the functions taking or returnign floats.

If you hope have everything with softfp, you can build openblas with ARMv5 as a workaround.

Do you know if ARMv5 OpenBLAS is compatible with ARMv7 based chipsets?

Yes, please refer to https://github.com/sh1r0/caffe-android-lib/blob/dev/scripts/build_openblas.sh.

Thank you, will give that a try. Appreciate all your help. Your project was immensely helpful to me.


release v0.4.3

@adrian-kong
hum.. the crate ("serde" feature definition i think) is faulty : the crate does not build without any features activated.
Any cargo publication does not work at the moment

@adrian-kong
I released using "cargo publish --no verify", probably what's being hosted is corrupt (at least if no feature activated).
Anyways, i'm tired of cargo and cargo induced problems. I'd rather be working on interesting stuff than pointless topics like these


chore: Update models for release 0.22.0

Issue #

Description of changes

New/existing dependencies impact assessment, if applicable

Conventional Commits

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/test


to process batch images

bro, thanks a lot for this tool. If i want to do OCR for a batch of images, what kind of command should be used?

Well, there is no built-in batch processing tool. I might add.  But in what form?
Text document with a list? A folder with pictures?
The question is also in what format to give them to you....
In fact, the library itself can be used from the Python API. And the same GPT, if you feed it documentation, will understand what and how to use, and will also write batch processing for you. Decide for yourself, I can add, but I will have to think for a long time in what form to give the data

Thank you very much for the reply, Bro,
I think, simple text files will be ok. For example, there will be an input folder which has a batch of image files of jpg or jpeg formats, the output folder has text files with the same name as the input image files (one  output text file for each of an input image file).

I will release an update by the evening

I will release an update by the evening

thank you very much, bro

@truthsayer969
Go Test
pip install -U chrome-lens-py

thank bro, i tested cli method, when i just testes with a batch of a few image, it works. But when i add such as 50 images, it does not work, resulting a blank txt file

thank bro, i tested cli method, when i just tested with a batch of a few image, it works. But when i add such as 50 images, it does not work, resulting a blank txt file

Is there a log? It always outputs logs

thank bro, i tested cli method, when i just tested with a batch of a few image, it works. But when i add such as 50 images, it does not work, resulting a blank txt file

Is there a log? It always outputs logs

Dear Bro, now i can do OCR for a batch of 48 files, but it does not give logs, so i am unable to show you.
By the way, it takes 4-5 min to do OCR for these 48 images.

thank bro, i tested cli method, when i just tested with a batch of a few image, it works. But when i add such as 50 images, it does not work, resulting a blank txt file

Is there a log? It always outputs logs

Dear Bro, now i can do OCR for a batch of 48 files, but it does not produce log file, so i am unable to show you. By the way, it takes 4-5 min to do OCR for these 48 images how can i speed it up? Would you kindly make the tool to produce a separate text file for each image? text file should have the same name as the input image file.

The log is issued in the console, and if something went wrong, it will report it directly in the console.
Yes, I will update the code so that you can create a text document for each file for each file. But if you need to use the recognition of 10-1000 images, then I suggest you stupidly put https://github.com/dmMaze/BallonsTranslator in which you can both recognize and translate....
The speed of recognition depends on 2 factors. Your Internet, which transmits requests (goes to lens) and the delay itself is sleep-time, which you can set manually, but you can not set less than 300ms

thank bro, i tested cli method, when i just tested with a batch of a few image, it works. But when i add such as 50 images, it does not work, resulting a blank txt file

Is there a log? It always outputs logs

Dear Bro, now i can do OCR for a batch of 48 files, but it does not produce log file, so i am unable to show you. By the way, it takes 4-5 min to do OCR for these 48 images how can i speed it up? Would you kindly make the tool to produce a separate text file for each image? text file should have the same name as the input image file.

1. The log is issued in the console, and if something went wrong, it will report it directly in the console.

2. Yes, I will update the code so that you can create a text document for each file for each file. But if you need to use the recognition of 10-1000 images, then I suggest you stupidly put  [https://github.com/dmMaze/BallonsTranslator ](https://github.com/dmMaze/BallonsTranslator) in which you can both recognize and translate....

3. The speed of recognition depends on 2 factors. Your Internet, which transmits requests (goes to lens) and the delay itself is
   `--sleep-time`, which you can set manually, but you can not set less than 300ms

Thank you very much, bro, for the explanation. Yes, I admit that my internet is not fast. I will wait for the update.

Done. Test it.

@truthsayer969 work?

@truthsayer969 work?

Dear Bro, It works. I tested with scan images of books. I already finished 2 books, each has 200-300 pages and it works perfectly.
Thank you very much for this wonderful tool.

Dear Bro, It works. I tested with scan images of books. I already finished 2 books, each has 200-300 pages and it works perfectly.
Thank you very much for this wonderful tool.

Maybe you can use the program that I dropped above? It also contains the same library, and will allow you to work with text faster.

And I think I'll close the ISSUE

Dear Bro, It works. I tested with scan images of books. I already finished 2 books, each has 200-300 pages and it works perfectly.
Thank you very much for this wonderful tool.

Maybe you can use the program that I dropped above? It also contains the same library, and will allow you to work with text faster.

thanks bro, for the advice, I will try it also.

And I think I'll close the ISSUE

ok, bro, Thank you very much for you kindness to explain me and update of the tool.


CL_INVALID_BUILD_OPTIONS (-43) when running tests on mesa

https://gist.github.com/anadon/94f6857692b302473250
More mesa-clBLAS issues.  Started fishing on the #dri-devel channel.  Could you guys start testing on mesa?

It is very likely a mesa bug rather than a clBLAS bug. We (arrayfire) important clBLAS functionality on intel, AMD, nvidia, apple, pocl and beignet. They pass our tests without a hitch.

Ok.  I'll talk more to the mesa dev's.  But adding mesa as a build target would be nice.

I want to test arrayfire with mesa, but this bug is holding us back.

I wonder if what I'm seeing is related or different.  I also have a bug where OpenCL is only detecting 1GB of my card's memory.

I'm seeing CAPS_DEBUG and DEBUG_2 as compile time guards for debug code.  What should I be using in src/library/blas/generic/commin2.cc between line 87 and 88 for checking makeKernel error states set by "storeErrorCode()"?

Is there a parallel to getErrorString proposed here http://stackoverflow.com/questions/24326432/convenient-way-to-show-opencl-error-codes ?  Grep isn't leading me to anything offhand.


Ozyugoo

Forked the repo, created my readme in the/2023 directory, added my name to the list of contributors, added my introduction to my readme, added my social handle, completed some of the checklists, copied the contributor's guide to the readme, and the some of the research I made.

Well done @Ozyugoo for making your contribution. However, it seems your branch has conflicts. Kindly review and fix it before it can be merged.

Well done @Ozyugoo for making your contribution. However, it seems your branch has conflicts. Kindly review and fix it before it can be merged.

Thank you @codewithmide  I will see to it now.


why epoch-Train-perplexity is nan in the mxnet lstm training, but batch-Train-perplexity is normal and keep decline?

in mxnet 0.9.3

use perplexity function:
def perplexity(label, pred):
    label = label.T.reshape((-1,))
    loss = 0.
    for i in range(pred.shape[0]):
        loss += -np.log(max(1e-10, pred[i][int(label[i])]))
    return np.exp(loss / label.size)

But I don't see the Resetting Data Iterator process. Why?

Hi there,
Did you figure this out? I am getting the same issue. Thanks!

Here is my guess:
I dig into the code for Speedometer, and there is a thrid parameter auto-reset which is true by default. It seems it would reset your metric when you call this callback. Then your number of instances becomes zero, and if that is exactly the end of the epoch, when you get the metric value, you would divide by 0, which returns NaN.
So, are you using auto-reset by default?

This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this is still an active issue. Thanks!


Messages not showing up once inside ui grid

Messages aren't showing up inside ui grid. This is because ui grid applies font-size to 0 and ui.message is using font-size:1em. I'm assuming the font-size should be 1rem for ui.message?

There's a font size reset inside ui grid, nothing can show up as a direct descendent of grid except column or row.
This is due to whitespace caused by consecutive inline-block
Will be fixed in 2.0 when all grids use flexbox


analytics: Don't log on missing X-Region-Name header

Geo header X-Region-Name is not available in all regions, therefore we should treat it as optional.

I would also change this line
to produce a warning without err as that misleads that's some kind of error...
cannot parse geo info from analytics log request header, missing geo headers ...

instead of:
cannot parse geo info from analytics log request header, err=missing geo headers:

this line

Updated in f84ec5cf509a2d4cae9820d15db24bab3d4f4d33


[bump] client: 2.0.0-internal.8.0.0 => 2.0.0-internal.9.0.0 (major)

Bumped client from 2.0.0-internal.8.0.0 to 2.0.0-internal.9.0.0.
This version is not going to be released, but https://github.com/microsoft/FluidFramework/pull/18811 is running into a lot of issues, so if those aren't resolved soon this PR can be merged as a backup to get the main branch unlocked.

/azp run Build - client packages


  Please see CONTRIBUTING.md#Commit 消息规范

Creating a pull request requires a purposeful goal. Don't use misc.