Fix discontiguous selection

The reason for this error is because when we click on span browser makes another selection, so when we are trying to create selection in a different DOM branch, Chrome throws.
toggle-selection removes selection from currently clicked element so our problem is fixed.
Before

After

I wanted to write some tests but it seems like we need real integration tests (or at least some relatively close emulation), I could not reproduce this issue with unit-testing.

The CI failed on this due to a bug in sudodoki/toggle-selection, and will clear when sudodoki/toggle-selection#1 is merged.

Also we can't merge until tests are included here or on that library.

@zenorocha tests for this are not trivial since they require actual Chrome. This bug does not appear in FF, for example (oh it does not work in FF anyway). So you need some infrastructure around it. I strongly suggest to move it to a separate issue as Integration tests in real browser or something like that.

We already have the infrastructure to run tests in real browser, it's just a matter of including Chrome to this list and karma-chrome-runner

If you don't mind me modifying that, then I'm okay adding Chrome there. I usually prefer to make as atomic PRs as possible and do not creep into other areas. That was the reason I did not add any changes in your test/build process.

Dependency updated, so phantomjs does not freak out. Tests pass.

Hey @nkbt, I think I fixed this problem with: https://github.com/zenorocha/clipboard.js/commit/9377659c9c37833b2fe27dca09ffaa486e773063
Could you test and let me know if it works? If not, please send a JSFiddle with steps to reproduce the error.

Sorry was unable to test this thing yesterday, was busy doing 120 slides for my today's talk.
I quickly checked the code and it seems like it should work fine. I'll give it a try just in case today after work though.

You're awesome, thanks your help and good luck with the talk!

Thanks, just finished it half an hour ago. It was good. I guess so =). Will upload it later since uploading 5 gigs from australia is like, several hours...

:P


chore: update issue templates (don't merge)

Signed-off-by: Navendu Pottekkat<EMAIL_ADDRESS>Don't merge
Fixes #1571
Updates issue templates to add bug report, feature request, and documentation templates.
Waiting to add a template to report performance issues.

Codecov Report

Merging #1590 (b827d5f) into master (486b46a) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1590   +/-   ##
=======================================
  Coverage   41.44%   41.44%
=======================================
  Files          87       87
  Lines        7420     7420
=======================================
  Hits         3075     3075
  Misses       3990     3990
  Partials      355      355

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

@tao12345666333 Added a template for reporting performance issues as well. Please take a look when you are available.


With rails 5.2, where clauses will no longer cast Class objects as Strings

We need to explicitly pass String objects.
Similar to https://github.com/ManageIQ/manageiq/pull/19016
Fixes errors of this variety on rails 5.2 (in a rails 5.1 backward compatible
way):
irb(main):001:0> MiqReportResult.with_chargeback
The version of PostgreSQL being connected to is incompatible with ManageIQ (10 required)
PostgreSQLAdapter#log_after_checkout, connection_pool: size: 5, connections: 1, in use: 1, waiting_in_queue: 0
Traceback (most recent call last):
       16: from activerecord (<IP_ADDRESS>) lib/active_record/relation.rb:584:in `skip_query_cache_if_necessary
       ...
        1: from activerecord (<IP_ADDRESS>) lib/active_record/connection_adapters/abstract/quoting.rb:34:in `rescue in type_cast'
TypeError (can't cast Class)

cc @kbrock

@miq-bot add_label jansa/yes?


feat: rename and modularize pair 2.0

Renamed Separated to Pair and worked for 2.0 release.

Added file block at top
Checked for missing implementations and implemented
Modified all combinator variable names
Added documentation to every export
Implemented full test coverage

A few other changes snuck into this commit.

Added a few missing instances for array, datum, map, monoid, nilable, number, option, etc.
Removed Alternative and Plus, since we don't have any real implementations for those types.
Updated coverage script to not test docs, run in parallel, avoid unstable, and supply no permissions to the test suite.

Fixes #145

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

138 of 140 (98.57%)  changed or added relevant lines in 8 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.7%) to 89.536%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

array.ts
2
4
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.7%

Covered Lines:
4610

Relevant Lines:
5165

💛  - Coveralls


WebP rendering options

I couldn't find a way to set the options listed in the ImageMagick docs, is there a wand api for setting them?
http://www.imagemagick.org/script/webp.php

You would have to include wand.api.library and integrate MagickSetOption. Example ...
from ctypes import c_void_p, c_char_p
from wand.api import library
from wand.image import Image

# Tell python about the MagickSetOption method
library.MagickSetOption.argtypes = [c_void_p,  # MagickWand * wand
                                    c_char_p,  # const char * option
                                    c_char_p]  # const char * value

with Image(filename="/path/to/source.png") as image:
    # -define webp:lossless=true
    library.MagickSetOption(image.wand,       # MagickWand
                            "webp:lossless",  # option
                            "true")           # value
    image.save(filename="/path/to/lossless.webp")

@emcconville, oh that's neat, thanks a lot for the example, and sorry for the noise. :stuck_out_tongue_winking_eye:


How can get the status of the mail?

Hey,
I am using the Sendgrid API and I want to check if sent mail status with Success, Bounce, Pending, Spam, Bounce etc.
I have already written the code with autoload.php,
Can you please help me?

Hi,
You have a couple options.
First, you can use SendGrid's Webhooks to receive information on individual emails.  Documentation is available at https://sendgrid.com/docs/API_Reference/Webhooks/index.html. However, this means you need to build functionality to be able to accept webhook calls and store information sent by each call.   I have built this functionality into iDimensionz's Communiqué email service.  We're working on the UI to display that info to users.
You're other option is to use the Stats endpoints in SendGrid's WebAPI v3 to get those types of statistics about emails.  Documentation is available at https://sendgrid.com/docs/API_Reference/Web_API_v3/Stats/index.html . These are organized by date so it is more summary information rather than detailed information about specific emails.
I don't believe SendGrid's PHP WebAPI client has implemented the Stats endpoint yet.  I have started implementing the Stats endpoints in my PHP WebAPI v3 repo and they should be available soon.  You can checkout my repo at https://github.com/idimensionz/sendgrid-webapi-v3-php and play with some examples from my repo at  https://github.com/idimensionz/sendgrid-webapi-v3-examples.  I also setup a free Hipchat chat room to answer questions and discuss my API client which you can join at https://www.hipchat.com/gFqa8nAS0 .

Thanks for your help :)
I really appreciated to your help.

@idimensionz thanks for jumping in with a solution!
@helloarpitgshah thanks for reaching out and supporting SendGrid!
Our new library, to be out of beta shortly, supports all of our v3 Web API calls including the new v3 /mail/send: https://github.com/sendgrid/sendgrid-php/tree/v3beta


Mbit is not the correct unit for "Left data" and "Initial data"

I guess the correct unit should be MB oder maybe MiB (I don't know the orange API).

It is :
https://github.com/rexave/hass-orange-internet-on-the-move/blob/0f2e7d50deeff853f89897acac920f90d31324ca/custom_components/orange_internet_on_the_move/sensor.py#L147
please explain more

As I have booked 2 Gigabytes it should be 2048 Megabytes. Megabits is a rather unusual unit for sizes. When I configure Megabytes in HA for these sensors the value is much too small, I think HA will divide the Sensor value by 1024.
That’s the reason why I think the unit is wrong. I know that is also wrong in the source code, I’ve already looked it up.
I would rather provide the value as bytes or megabytes.


Running Issues

Hi Dane,
I have troubles when running the code.
In this line:
from pridict import priority_dict
I can't find the file 'pridict'. Is it in the file folder 'cypridict'? I can only find the file pridict.so ,but my system is Windows.
Thanks,
z4z5

Hi z4z5, were you able to install the cypridict submodule, install cython and cythonize it (i.e. the "Installation" section of the readme?).

Hi Dane,
Thanks for your response. I tried to install the cypridict submodule, install cython and cythonize it.It seems that I successfully ran the install.sh, but when I tried to run 'cython table/hamming.pyx' and 'cython table/hctable.pyx', the error is below:
Error compiling Cython file:
...
from libc cimport stdint
^
table/hamming.pyx:1:0: 'VaST-master.table.hamming' is not a valid module name

Hi Dane,
Thank you for your reply, the problem has been solved:)


glTF Statistics

Merges the gltf-statistics repository.
Currently, the --stats option is used to print the statistics of the input and output onto the console.
This can be easily expanded to run on any stage of the pipeline by calling the function.
Another extension, which is a part of gltf-statistics is to write to CSV files. Again, this is very easy to expand on in the future when gltf-pipeline will run for directories.

I found that some of the JS files in lib had exec permissions. I have removed those permissions in this PR.
Also fixed trailing whitespace

Changes completed

I see that you added CesiumMan and CesiumAir with this for tests. Can you use some of the existing test models instead? Adding models tends to bloat the repo, so we try to avoid if possible.

Also, would it be possible for you to put your whitespace deletion and file permission changes in a separate pull request? It will get merged faster and it would make this one a lot easier to read.

Also, would it be possible for you to put your whitespace deletion and file permission changes in a separate pull request? It will get merged faster and it would make this one a lot easier to read.

@lasalvavida I've moved this into #224. I will clean this PR's commits and also have the suggested changes.

I see that you added CesiumMan and CesiumAir with this for tests. Can you use some of the existing test models instead? Adding models tends to bloat the repo, so we try to avoid if possible.

These were the tests from gltf-statistics for which there were reference values, hence I copied the files over. I agree it increases the size of the repo. Is there any way to get the statistics of gltf files without actually using this statistics feature? It wouldn't make sense to generate statistics with this and then run the specs for the same numbers.

Is there any way to get the statistics of gltf files without actually using this statistics feature?

Just open them up in a text editor:

buffersSizeInBytes - add up byteLength for all buffers
numberOfImages - count them
numberOfExternalRequests - how many buffers, shaders, images use external URIs?
numberOfDrawCalls - should be the same as the number of triangles (# of indices / 3)
numberOfRenderedPrimitives - count them
numberOfNodes - count
numberOfMeshes - count
numberOfMaterials - count
numberOfAnimations - count

Should be pretty straightforward; if you use specs/data/riggedSimpleUnoptimized and specs/data/boxTexturedUnoptimized, you should have full coverage for images/animations.

Cleaned and updated

@shehzan10 can you please open a pull request into gltf-statistics to update the README.md to say that project is no longer maintained and to use gltf-pipeline instead (include an example of running it from the command-line)?
There might still be some users of gltf-statistics so I don't want to fully remove the repo or the files yet.

Just those comments; this looks great.  Please verify the reference doc build.
@lasalvavida or @lilleyse can merge when ready.

@lilleyse Completed the object type and updated documentation.

@lilleyse I think I've addressed all the items.

The newline spacing could be improved:

Is this better? I reordered it a little too.
$ node ./bin/gltf-pipeline.js --stats -i ./specs/data/boxTexturedUnoptimized/CesiumTexturedBoxTest.gltf -o output.gltf

Statistics for specs/data/boxTexturedUnoptimized/CesiumTexturedBoxTest.gltf
------------------
Total size of all buffers: 840 bytes
Images: 1
Draw calls: 1
Rendered primitives (e.g., triangles): 0
Nodes: 4
Meshes: 1
Materials: 1
Animations: 0
External requests (not data uris): 4

Statistics for output.gltf
------------------
Total size of all buffers: 840 bytes
Images: 1
Draw calls: 1
Rendered primitives (e.g., triangles): 0
Nodes: 1
Meshes: 1
Materials: 1
Animations: 0
External requests (not data uris): 3

optimize: 70.616ms

Note: The text pasted above the not only the same as gltf-statistics, but is also used by agi-model-inspector in the Key Statistics drop down.

agi-model-inspector uses gltf-statistics so it explains why it also says 0.

Is this better? I reordered it a little too.

The new formatting looks great!

agi-model-inspector uses gltf-statistics so it explains why it also says 0.

Actually the model inspector seems to be up-to-date but the gltf-statistics still uses the old terminology.

Pushed the new format.

Nice - once https://github.com/AnalyticalGraphicsInc/gltf-pipeline/pull/220#discussion_r97804477 is addressed this should be good to go.


Add first pass of Z3Solver SelectOp

This is necessary to implement the SelectOp in the interpreter, and pass the collatz test case

I'll have to add these after adding the interpreter SelectOp as well


Change language dynamically

Is it possible to change the language and the keyboard layout when a specific key is pressed?

its possible in,(This is what i did)First create a function that return a keyboard with the keys of your selected language (See the implementation of the DefaultKeyboard class )  and then  go to the class   KeyboardViewController there is a function called layoutHelper using this u can changed the language of the keyboard very easily. you just have to return the keyboard you have created there. and when you click on your specific keyboard just reboot the keyboard and the rest will be done properly


Proposal: Add option to emit //go:generate annotation

What do you want to change?
I would like sqlc to provide an option to emitting //go:generate annotation on generated models file in order to let us auto-generate the extended codes using go generate command.
Proposal
I would like to add some option like go_generate_annotations in config that lead sqlc to emit go:generate annotation.
sql:
  - gen:
      go:
        go_generate_annotations:
          - generator: go-genaccessor
            emit_into_models_file: true

Expected result in models.go:
//go:generate go-genaccessor

Background
I was firstly looking for the way to add getter/setter methods to the sqlc-generated models, and found that there is no straightforward option to achieve, nor any plan in future to support it. Instead, there have been some comments to the following issues that encouraging the use of any another tool to extend the code.

https://github.com/sqlc-dev/sqlc/pull/2471
https://github.com/sqlc-dev/sqlc/pull/387

Golang already provides an official go generate subcommand (https://go.dev/blog/generate) that let us autogenerate the code, but that command requires //go:generate annotation on the subjected file in order to tell the command which file should be subjected and which plugin should be used for autogeneration.
Goal
I confirmed that I was able to add extra struct tags to the columns of the model struct can be achieved with following config:
sql:
  - gen:
      go:
        overrides:
          - column: items.id
            go_struct_tag: 'getter:"GetID" setter:"SetID"'

Which results to the following code, which seems perfect.
type Item struct {
	ID uuid.UUID `getter:"GetID" setter:"SetID"`
}

However that file doesn't obviously provide the go:generate annotation and I couldn't achieve the generation of getters and setters"
What database engines need to be changed?
No response
What programming language backends need to be changed?

Ah I see. That will surely work! Thanks!


FAC staff image cropped from around 1053-1075px width

@yvonne-liu @sofer did you manage to replicate this in the end? It would be good to know the context if so.

@yvonne-liu @sofer @rebeccaradding I've checked on all widths and only found issue with 1040px, which I've just amended. Do you mind checking?

@maxgerber I'm still seeing an issue with heights shorter than 762px. Should I open a separate issue? Unless the overlap is intentional?


Enforcing Unique IDs

Is this possible to do? Seems like redux-orm would be confused if there were multiple records with the same ID since it wouldn't know which one to fetch. Do you plan to add a check against this?
In many cases, you will be getting records from an API which is the "source of truth". You then want to add it to the orm, but it's possible to manually specify the ID and keep essentially creating the same one. Seems like it should be the responsibility of the ORM to prevent this behavior since it then cannot operate correctly.

Currently a creating a record with an id that equals another existing record would just overwrite it - that's not expected functionality. I agree that this should be added.
Ideally this will be added to the database layer Table.prototype.insert. Currently that returns a new state and the created record. It should be extended to be able to return an error value as well, if it wasn't able to create a new record. Ability to return error values from the database layer isn't used by the ORM layer yet but when the new database API introduced in 0.9.0, it accommodates use of that pattern.
For now, you can patch this easily at the ORM layer before a full feature is added (might differ slightly for TS):
class MyBaseModel extends Model {
  static create(props) {
    if (props.hasOwnProperty(this.idAttribute) && this.hasId(props[this.idAttribute])) {
      throw new Error(`Id with ${props[this.idAttribute]} already exists`);
    }
    return super.create(props);
  }
}

yup, makes sense, just checking on this. That's what I added in my basemodel.

I created the same object that u can extend to make and update if we currently have the record on our store or create if not:
import { Model } from 'redux-orm';

export default class DopeModel extends Model {
  static create(props) {
    if (props.hasOwnProperty(this.idAttribute) && this.hasId(props[this.idAttribute])) {
      const model = this.withId(props.id);
      return model.update(props);
    }
    return super.create(props);
  }
}

can we have idAttribute with 2 columns or two props of the object ?

@udayreddy This is currently not possible. And I'm not planning this myself because there's already more than enough on the roadmap. But would happily merge a PR if tested, consistent with current memoization and non-breaking. I guess a reasonable API would be Book.idAttribute = ["title", "edition"].
The obvious easy workaround would be to store them in the same field instead (book.id = `${book.title}-${book.edition}`). Note that that's likely to cause bugs as you need to do this manually each time you perform a write operation (book.id = someId, book.set("id", someId), book.update({ id: someId }), Book.create). So if you can, I would suggest to add a surrogate key to your data source for this purpose.
And for anyone passing by, the Model.upsert solution by @carlesnunez has been added in v0.9.1.

Thank you for the kudos @haveyaseen 🤟🏻😊


HttpCookie.Value should return the URL decoded cookie value

[X] You've read the Contributor Guide and Code of Conduct.
[X] You've included unit tests for your change, where applicable.
[X] You've included inline docs for your change, where applicable.
[X] There's an open issue for the PR that you are making. If you'd like to propose a new feature or change, please open an issue to discuss the change or find an existing issue.

PR Title
HttpCookie.Value should return the URL decoded cookie value
PR Description
Create a new HttpCookie object in NET Standard / NET Core URL encodes the value of of cookie, whereas in NET Framework 4.7.2, this doesn't occur. This is probably the desired behavior but makes it difficult to share the same cookie between the two apps (NET Core and NET Framework).
To reproduce this bug:
var cookie = new HttpCookie(cookieName, "a|b") { Secure = true, SameSite = SameSiteMode.None };
Console.WriteLine(cookie.Value); // prints a%7Cb

Also, here is an example that HttpCookie should just return the cookie value without URL encoding it: https://dotnetfiddle.net/eusU6N (but Values should URL encode, which I added a unit test for it in this PR).
Addresses #393

I manually tested these changes locally and confirmed that HttpCookie.Value is now correctly reflecting the URL decoded value, however, the Set-cookie header generated in the response gets URL encoded again.
I'm not quite sure why this is happening or how the Set-cookie header gets generated but I noticed that this file generates the Set-cookie header https://github.com/microsoft/referencesource/blob/51cf7850defa8a17d815b4700b67116e3fa283c2/System.Web/HttpCookie.cs#L444. Maybe we should have the same method?

I'm not quite sure why this is happening or how the Set-cookie header gets generated but I noticed that this file generates the Set-cookie header https://github.com/microsoft/referencesource/blob/51cf7850defa8a17d815b4700b67116e3fa283c2/System.Web/HttpCookie.cs#L444. Maybe we should have the same method?

So it sounds like your fix gets us partially there, but there is still some URL encoding going on? If so, would you be up for integrating that method in? There's some e2e tests that set up a test host that should allow testing this scenario

I'm not quite sure why this is happening or how the Set-cookie header gets generated but I noticed that this file generates the Set-cookie header https://github.com/microsoft/referencesource/blob/51cf7850defa8a17d815b4700b67116e3fa283c2/System.Web/HttpCookie.cs#L444. Maybe we should have the same method?

So it sounds like your fix gets us partially there, but there is still some URL encoding going on? If so, would you be up for integrating that method in? There's some e2e tests that set up a test host that should allow testing this scenario

@twsouthwick Thanks for the review.
I think I prefer to run some more e2e tests as this is not going to actually fix #393. Re e2e tests, are you referring to these Playwright tests here https://github.com/dotnet/systemweb-adapters/tree/main/test/Samples.MVCApp.Tests?

I was referring to these tests: https://github.com/dotnet/systemweb-adapters/blob/main/test/Microsoft.AspNetCore.SystemWebAdapters.CoreServices.Tests/ResponseStreamTests.cs

@twsouthwick Updated the PR and added the ToSetCookieHeaderValue method which correctly sets the Set-cookie headers. Also added some end to end tests. Let me know what you think.


Side-chain connections are not displayed in Snake mode

Steps to Reproduce

Switch to the Macro mode – the Snake mode
Draw any monomers (peptides, nucleotides, CHEMs) on the canvas
Drag side-chain connections between these monomers

Actual behavior
Side-chain connections can be dragged, but they are not displayed on the canvas

Expected behavior
Side-chain connections are displayed on the canvas
Screenshots
https://github.com/user-attachments/assets/525c8ae3-c3b8-440c-b412-3a574a045825
Desktop (please complete the following information):

OS: Windows 11 Pro (version 22H2)
Browser: Сhrome
Version: (127.0.6533.122) x64

Ketcher version 2.24.0-rc.1, Build at 2024-08-23; 12:23:08
Indigo version 1.24.0-rc.4
Issue found while testing https://github.com/epam/ketcher/issues/3532

Tested. Bug not fixed.
https://github.com/user-attachments/assets/4d610f81-c2ab-4858-933c-81075112b9a2
Desktop:

OS: Windows 11
Browser Chrome
Version 128.0.6613.120 (Official Build) (64-bit)

Ketcher version
[Version 2.26.0-rc.1]
Indigo version
[Version 1.25.0-rc.1]

Tested. Bug fixed.
https://github.com/user-attachments/assets/70e73144-ec65-44ae-b807-0b846ccaeccb
Desktop:

OS: Windows 11
Browser Chrome
Version 128.0.6613.120 (Official Build) (64-bit)

Ketcher version
[Version 2.26.0-rc.3]
Indigo version
[Version 1.25.0-rc.2]


Moe writing branch from 7f9a8c870bb42ea159d2efae6dc2f4ad89e2fb85

Merge internal changes.

A few jdk6 issues and a few maven issues... have fixes pending for each and will repush when ready.


Created Permutation-of-String.md as a contribution to Backtracking Algorithms

Hello Sir. Arun here and i am a contributor to GSSOC '23. This PR entails descriptive solution and intuition for the "permutation of String" problem in Backtracking. Please consider this PR. Thank you

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


Give all settings appropriate scopes, so that they work in multi-root workspaces

Currently no sqlfluff settings have scopes defined. I'm not sure what the default is, but when I create a multi-root workspace the sqlfluff settings no longer apply.
This makes sense, as the workspace settings docs say:

To avoid setting collisions, only resource (file, folder) settings are applied when using a multi-root workspace.

From the extension docs there are a lot of possible scopes, but in particular:

scope
A configuration setting can have one of the following possible scopes:

... snip ...
resource - Resource settings, which apply to files and folders, and can be configured in all settings levels, even folder settings.

I believe that all sqlfluff settings could be resource settings but I'm not sure. Changing this seems straightforward, for example (from current package.json)
  "configuration": {
      "properties": {
        "sqlfluff.config": {
          "type": "string",
		  "scope": "resource",  // <-- new
          "default": "",
          "description": "Specify an additional configuration file that overrides the standard configuration files."
        },

@quodlibetor I'm think the default scope is "window", will changing the scope to "resource" create any issues? I'm not very familiar with scopes.

I'm not confident, I've never done any VSCode extension development.
Intuitively, though, all of the settings that I can see should be editable on a per-directory level -- none of them affect the VSCode window itself, just how sqlfluff itself is executed.

Makes sense, doesn't seem like this change would hurt anything. Pushing the update in v3.3.0

thank you!


[cur-565] Units table and unit details in generated .docx

Description

Interleaved units table into unit details in curriculum .docx
Changes to the units, broken down into year/tier/subject
Also small changes to unit details in curriculum .docx

Issue(s)
CUR-565 & CUR-567
How to test

Go to https://deploy-preview-2448--oak-web-application.netlify.thenational.academy/teachers/curriculum/docx-poc/select
Choose a subject via the autoComplete
Download the docx template - linked on the page
Download the rendered document
Check the unit table and unit details are interleaved and contain the correct data

Screenshots

Checklist

[ ] Added or updated tests where appropriate
[ ] Manually tested across browsers / devices
[ ] Considered impact on accessibility
[ ] Design sign-off
[ ] Approved by product owner
[ ] Does this PR update a package with a breaking change

Don't worry about the sonar error for now, as we're merging into feat/cur-441/curriculum-download-docx-poc and I'll write test for the missing file in that PR.

Merged this into feat/cur-441/curriculum-download-docx-poc so it makes it easier to test together


Add Universidad Tecnologica de la Sierra Hidalguense

Website: https://utsh.edu.mx/
Educational offer related to SWE: https://utsh.edu.mx/oferta-educativa-utsh/idgs/

@stoneface919 First stage of verification passed, official domain is utsh.edu.mx as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@stoneface919 Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Support for migrated accounts

As it mentioned here: https://github.com/subutux/rmapy/issues/25 Remarkable probably release new API, and uploading files it is now done in a different way. Whether you intend to introduce support for the new protocol?

remarkable-rss uses https://github.com/benoitsepe/reMarkable-typescript under the hood, which has already an issue open for this: https://github.com/benoitsepe/reMarkable-typescript/issues/34.

Yes, but there’s been no support for a long time :(

There's a new npm library that supports 1.5: https://www.npmjs.com/package/rmapi-js
Maybe worth a try migrating?


On the fullDetails page, while saving a user's features, indicate the machine is working to save data. (Perhaps have a spinning wheel next to mouse)

Done. It usually saves too fast too notice the spinning wheel though.


stm32-rtc: Add support for the internal low speed clock (LSI)

Committed upstream: http://sourceforge.net/p/nuttx/git/ci/1923fed668f74aafc8edcb13cb9cab07e50ba97e/
Some notes:

You version of stm32_rtc.c is not the same as the upstream GIT.  As a result the patch did not apply and parts of it had to be included manually.  You might want to review.  The basic problem is that the upstream version includes a retry loop and the PX4 version does not.  Changes were made to logic that is now within a loop.

The HSE must be divided down to get the 1MHz RTC clock input.  That divider was hard-coded to 24 for a 24MHz HSE clock.  Most STM32s do not have a 24MHz HSE clock.  I added logic to calculate the divider to get a 1MHz clock based on STM32_HSE_FREQUENCY which should be provided in every STM32 board.h header file.  This change is untested.

Various stylistic changes to conform with the NuttX coding style.

Also, there were no missing changes to the Kconfig file that would allow selecting the LSE/HSE RTC clock source options.  I added these:  http://sourceforge.net/p/nuttx/git/ci/9337faee31925b60a699519c4fd0506b34f0585f/

Thanks Greg! The plan was to rework the patch for the upstream version (as we will ditch this branch as soon as we have the soak testing done on the upstream version) - its very nice from you that you did that step already for us, much appreciated!

The truth is that I I just get confused about which repository I am looking at.  I probably should get off of this mailing list and just monitor the newer repository.  Then I would pick up changes only after you have reviewed then and prep'ed then for upstream.


Request for maintainer rights

@chessai @andrewthad I have full maintainer rights here, but I don't have upload rights on Hackage. I just prepared <IP_ADDRESS>, but I can't actually publish it. Could one of you please add me to the maintainer group? My Hackage username is OliverCharles.

Done!

Thank you! :heart:


The watch command is missing from the cli docs table of contents

- Do you want to request a feature or report a bug?
Bug
- What is the current behavior?
The watch command is covered in the CLI docs, but missing from the table of contents
- If the current behavior is a bug, please provide the steps to reproduce.

Go to https://cli.netlify.com/commands/watch/ - note that this page about the watch command exists
Go to https://cli.netlify.com/commands/ - note that the watch command is included in the list of available CLI commands
Check the cli docs table of contents and note that the watch command is missing despite there being a page about it.

- What is the expected behavior?
watch is included in the table of contents
- Additional context
Reported by a community member in https://community.netlify.com/t/cli-documentation-watch-command/4161

This is by design the command might go away


BEGIN_INTERFACE_MAP and END_INTERFACE_MAP conflict with MFC macros

The Common/Platform/NMR_COM_Native.h (included via NMR_COMInterfaces.h -> NMR_COMFactory.h) defines BEGIN_INTERFACE_MAP macro.
The MFC afxwin.h defines macro with the same name.
If both headers are included, there is an obvious conflict.
Would it be possible to rename these two macros according to LIB3MF-prefixed convention used for all the other macros?

Please see #51. Native COM functionality is very likely to be removed in the future.

Thanks


OU-413: reset proxy so is re created when the datasource changes

This PR resets the proxy when the datasource changes in order to re create it

/cherry-pick release-0.1

/lgtm


cluster-assign-roles fails with Net:SSH:HostKeyMismatch when rebuilding cluster

When deleting a cluster and reusing a bootstrap node, one will get an error like the following:
Refreshed chef-vault item ssh_host_keys/iptables-bcpc-vm1.bcpc.example.com
Refreshed chef-vault item ssh_host_keys/iptables-bcpc-vm2.bcpc.example.com
iptables-bcpc-vm2.bcpc.example.com: Cheffing with runlist 'role[Basic],recipe[bcpc::default],recipe[bcpc::networking]'
bundler: failed to load command: ./cluster_assign_roles.rb (./cluster_assign_roles.rb)
Net::SSH::HostKeyMismatch: fingerprint d0:bb:35:f8:c5:15:bb:dc:89:40:67:9c:d9:71:a1:02 does not match for "<IP_ADDRESS>"
  /home/vagrant/chef-bcpc/ruby/2.4.0/gems/net-ssh-4.2.0/lib/net/ssh/verifiers/secure.rb:48:in `process_cache_miss'
  /home/vagrant/chef-bcpc/ruby/2.4.0/gems/net-ssh-4.2.0/lib/net/ssh/verifiers/secure.rb:33:in `verify'
  /home/vagrant/chef-bcpc/ruby/2.4.0/gems/net-ssh-4.2.0/lib/net/ssh/verifiers/strict.rb:16:in `verify'
  /home/vagrant/chef-bcpc/ruby/2.4.0/gems/net-ssh-4.2.0/lib/net/ssh/verifiers/lenient.rb:15:in `verify'

Unexpectedly, even if ~/.ssh/known_hosts is deleted, one gets this issue coming back. This is due to the SSH keys being stored in Knife vault and being replaced on the hosts[1] -- which are different than they were after the OS install. We should setup cluster-assign-roles to pre-load the correct host keys into the ssh known_hosts file or to ignore this issue when rebuilding a machine (maybe a rebuild/ignore known hosts flag)?
If not a production environment, one can also run the following to not overwrite the new ssh host key:
knife data bag delete -y ssh_host_keys <fqdn>
knife data bag delete -y ssh_host_keys <fqdn>_keys

The repxe-host.sh script orchestrates all of this. if you are deleting a cluster, you need to do everything that repxe-host.sh does to get it properly cleaned.

Ah this is a VM specific issue for testing.

@aespinosa Ah good idea; yes, my hang-up was this broke the idempotency of tests/automated_install.sh so it may be possible to envision it doing the necessary work in cluster_assign_roles.rb.


Recently Played icon missing

On Windows 10, Recently Played icon on the left pane is missing.

OS: Windows 10 21H1
Spotify version <IP_ADDRESS>8
Spicetify version 1.2.1
Dribbblish

Sometimes re-installing fixes everything. Have you tried re-installing Dribbblish or Spicetify or even Spotify?

Sometimes re-installing fixes everything. Have you tried re-installing Dribbblish or Spicetify or even Spotify?

@Aerozolic

Having this problem as well

@DatAres37 same issue


netns-remap breaks  functionality on LFS-based distro (or insufficient wording for the error)

Output of docker version:
# docker version
Client:
 Version:      1.10.2
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   c3959b1
 Built:        Mon Feb 22 22:37:33 2016
 OS/Arch:      linux/amd64

Server:
 Version:      1.10.2
 API version:  1.22
 Go version:   go1.5.3
 Git commit:   c3959b1
 Built:        Mon Feb 22 22:37:33 2016
 OS/Arch:      linux/amd64

Output of docker info:
# docker info
Containers: 1
 Running: 0
 Paused: 0
 Stopped: 1
Images: 1
Server Version: 1.10.2
Storage Driver: aufs
 Root Dir: /var/lib/docker/165536.165536/aufs
 Backing Filesystem: extfs
 Dirs: 4
 Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Plugins:
 Volume: local
 Network: null host bridge
Kernel Version: 4.4.2
Operating System: <unknown>
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 3.861 GiB
Name: docker_test
ID: G4SB:5QE4:GESE:NTUG:JU6A:YPDR:UTFS:3UGJ:D7SD:2MS3:LKCZ:D4LK

check-config.sh
info: reading kernel config from /proc/config.gz ...

Generally Necessary:
- cgroup hierarchy: properly mounted [/sys/fs/cgroup]
- CONFIG_NAMESPACES: enabled
- CONFIG_NET_NS: enabled
- CONFIG_PID_NS: enabled
- CONFIG_IPC_NS: enabled
- CONFIG_UTS_NS: enabled
- CONFIG_DEVPTS_MULTIPLE_INSTANCES: enabled
- CONFIG_CGROUPS: enabled
- CONFIG_CGROUP_CPUACCT: enabled
- CONFIG_CGROUP_DEVICE: enabled
- CONFIG_CGROUP_FREEZER: enabled
- CONFIG_CGROUP_SCHED: enabled
- CONFIG_CPUSETS: enabled
- CONFIG_MEMCG: enabled
- CONFIG_MACVLAN: enabled (as module)
- CONFIG_VETH: enabled (as module)
- CONFIG_BRIDGE: enabled (as module)
- CONFIG_BRIDGE_NETFILTER: enabled (as module)
- CONFIG_NF_NAT_IPV4: enabled (as module)
- CONFIG_IP_NF_FILTER: enabled (as module)
- CONFIG_IP_NF_TARGET_MASQUERADE: enabled (as module)
- CONFIG_NETFILTER_XT_MATCH_ADDRTYPE: enabled (as module)
- CONFIG_NETFILTER_XT_MATCH_CONNTRACK: enabled (as module)
- CONFIG_NF_NAT: enabled (as module)
- CONFIG_NF_NAT_NEEDED: enabled
- CONFIG_POSIX_MQUEUE: enabled

Optional Features:
- CONFIG_USER_NS: enabled
- CONFIG_SECCOMP: enabled
- CONFIG_MEMCG_KMEM: enabled
- CONFIG_MEMCG_SWAP: enabled
- CONFIG_MEMCG_SWAP_ENABLED: enabled
- CONFIG_BLK_CGROUP: enabled
- CONFIG_IOSCHED_CFQ: enabled (as module)
- CONFIG_BLK_DEV_THROTTLING: enabled
- CONFIG_CGROUP_PERF: missing
- CONFIG_CGROUP_HUGETLB: enabled
- CONFIG_NET_CLS_CGROUP: enabled
- CONFIG_CGROUP_NET_PRIO: enabled
- CONFIG_CFS_BANDWIDTH: enabled
- CONFIG_FAIR_GROUP_SCHED: enabled
- CONFIG_RT_GROUP_SCHED: enabled
- CONFIG_EXT3_FS: enabled (as module)
- CONFIG_EXT3_FS_XATTR: missing
- CONFIG_EXT3_FS_POSIX_ACL: enabled
- CONFIG_EXT3_FS_SECURITY: enabled
    (enable these ext3 configs if you are using ext3 as backing filesystem)
- CONFIG_EXT4_FS: enabled
- CONFIG_EXT4_FS_POSIX_ACL: enabled
- CONFIG_EXT4_FS_SECURITY: enabled
- Storage Drivers:
  - "aufs":
    - CONFIG_AUFS_FS: enabled
  - "btrfs":
    - CONFIG_BTRFS_FS: enabled (as module)
  - "devicemapper":
    - CONFIG_BLK_DEV_DM: enabled (as module)
    - CONFIG_DM_THIN_PROVISIONING: enabled (as module)
  - "overlay":
    - CONFIG_OVERLAY_FS: enabled (as module)
  - "zfs":
    - /dev/zfs: missing
    - zfs command: missing
    - zpool command: missing

Provide additional environment details (AWS, VirtualBox, physical, etc.):
VirtualBox, but can be reproduced on AWS.
List the steps to reproduce the issue

start daemon as "docker daemon --userns-remap=default"
start container "docker run hello-world"

Describe the results you received:
# docker run hello-world
Unable to find image 'hello-world:latest' locally
latest: Pulling from library/hello-world
03f4658f8b78: Pull complete
a3ed95caeb02: Pull complete
Digest: sha256:8be990ef2aeb16dbcb9271ddfe2610fa6658d13f6dfb8bc72074cc1ca36966a7
Status: Downloaded newer image for hello-world:latest
Timestamp: 2016-02-24 05:47:16.268287337 -0500 EST
Code: System error

Message: no such file or directory

Frames:
---
0: setupRootfs
Package: github.com/opencontainers/runc/libcontainer
File: rootfs_linux.go@51
---
1: Init
Package: github.com/opencontainers/runc/libcontainer.(*linuxStandardInit)
File: standard_init_linux.go@57
---
2: StartInitialization
Package: github.com/opencontainers/runc/libcontainer.(*LinuxFactory)
File: factory_linux.go@240
---
3: initializer
Package: github.com/docker/docker/daemon/execdriver/native
File: init.go@35
---
4: Init
Package: github.com/docker/docker/pkg/reexec
File: reexec.go@26
---
5: main
Package: main
File: docker.go@18
---
6: main
Package: runtime
File: proc.go@111
---
7: goexit
Package: runtime
File: asm_amd64.s@1721
docker: Error response from daemon: Container command not found or does not exist..

Daemon logs:
# docker daemon --userns-remap=default
INFO[0000] User namespaces: ID ranges will be mapped to subuid/subgid ranges of: dockremap:dockremap
INFO[0000] Graph migration to content-addressability took 0.00 seconds
INFO[0000] Default bridge (docker0) is assigned with an IP address <IP_ADDRESS>/16. Daemon option --bip can be used to set a preferred IP address
WARN[0000] Your kernel does not support cgroup blkio weight
WARN[0000] Your kernel does not support cgroup blkio weight_device
INFO[0000] Loading containers: start.

INFO[0000] Loading containers: done.
INFO[0000] Daemon has completed initialization
INFO[0000] Docker daemon                                 commit=c3959b1 execdriver=native-0.2 graphdriver=aufs version=1.10.2
INFO[0000] API listen on /var/run/docker.sock
ERRO[0010] Handler for POST /v1.22/containers/create returned error: No such image: hello-world:latest
WARN[0013] exit status 1
ERRO[0013] error locating sandbox id 95a5df6c0ac5bf42170a96c878bea0c56f68d412eeee345dbab614a74b3721fa: sandbox 95a5df6c0ac5bf42170a96c878bea0c56f68d412eeee345dbab614a74b3721fa not found
WARN[0013] failed to cleanup ipc mounts:
failed to umount /var/lib/docker/165536.165536/containers/6cc4ae2e6e411912eb728cd2470011341cb110a043f7375f1720e133fff9e523/shm: invalid argument
ERRO[0013] Error unmounting container 6cc4ae2e6e411912eb728cd2470011341cb110a043f7375f1720e133fff9e523: not mounted
ERRO[0013] Handler for POST /v1.22/containers/6cc4ae2e6e411912eb728cd2470011341cb110a043f7375f1720e133fff9e523/start returned error: Container command not found or does not exist.

without "--netns-remap" everything works correctly. Does docker require external binaries for netns mode?

@novsh on VirtualBox, was that running boot2docker? For reference, https://github.com/boot2docker/boot2docker/pull/1142

also /cc @estesp

@thaJeztah  - no, it's just a VB (which runs on Suse linux) and boots into "linux from scratch" system. All docker versions works fine (  1.8.3, 1.9.1, 1.10.2). 1.10.x fails only if I enable "--userns-reap". I also checked it on aws to make sure it's not hw/virtualization.

Hey @novsh, what's the easiest way for me to create the LFS system to reproduce?

@estesp creating LFS is kind of painful and time consuming.  May I give you OVA image or LiveCD (LiveUSB) or access to an aws instance? Will it be easier for you?

@novsh definitely that would give me a quicker way to investigate. Any of those options works for me.

Ok, so thanks to @novsh giving me a system to debug on, it was fairly straightforward to find the issue.  When user namespaces are enabled all the /dev/* node mounts/creation switch from mknod to a bind mount, which works on many distros because all the nodes exist on the host.  In the case of lighter-weight distros, expecting /dev/fuse to exist is obviously a bit of a stretch.  I'll figure out a PR to gracefully continue if /dev/fuse doesn't exist, as I don't think we want hard failure as that is sort of an "extra device" that claims Java relies on it.  Opinions?
@novsh: if you want to play further you should be able to mknod /dev/fuse c 10 229 on your system and containers will run with user namespaces enabled. (I created it on the system you gave me access to and verified that hello-world runs now)

@estesp Thanks a lot! As far as I understand fuse should be enabled in kernel to let udev to create this device.  for now for this system
# zgrep -i fuse /proc/config.gz
# CONFIG_FUSE_FS is not set

maybe it makes sense to add this check to check-config.sh
Thanks again!

@novsh I did add a fix for /dev/fuse not existing on the host to Docker, but yes, it might be worth us adding a check for it with a comment that Java use in a container with user namespaces enabled will be impacted if you don't have CONFIG_FUSE_FS enabled.


Add Gatsby v4 support to Plugin

[NOTE]: # Building the Gatsby v4 web app with "gatsby-plugin-newrelic": "^2.0.2" gives a WARN of Plugin gatsby-plugin-newrelic is not compatible with your gatsby version 4.3.0 - It requires<EMAIL_ADDRESS>and npm i gives a WARN of<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed.
Summary
[NOTE]: # "gatsby-plugin-newrelic": "^2.0.2" appears to work well with Gatsby v4
clear WARNs from terminal
[NOTE]: # no WARNs in terminal for building with Gatsby v4.x
[TIP]:  #
Possible Solution
[NOTE]: # change line ( https://github.com/newrelic/gatsby-plugin-newrelic/blob/main/package.json#L22 ) to be "gatsby": "^3.0.0 || ^4.0.0 to support either major version
Additional context
[TIP]:  # This can give the false impression that the plugin does not work with Gatsby v4, but in fact it does.
Acceptance Criteria

[x] Review PR and approve
[x] creating an alpha release of this package on NPM
[ ] provide that to @herecydev for testing purposes from PR #45
[ ] If testing passes, then publish this version to NPM as a new version release.

@tristandenyer thanks for reporting this, we have an open PR in review to address this enhancement.

@herecydev 👋

It doesn't look like the next version updated on NPM after this merge, so I'm going to try to sync with @devfreddy on what may have gone wrong in our workflow here.

It seems we have all the correct updates made, but we are getting a failure on authentication to NPM now, for OTP. I'll look into this as well.
'npm ERR! This operation requires a one-time password from your authenticator.\n' +
    'npm ERR! You can provide a one-time password by passing --otp=<code> to the command you ran.\n' +
    'npm ERR! If you already provided a one-time password then it is likely that you either typoed\n' +
    'npm ERR! it, or it timed out. Please try again.\n' +
    '\n' +
    'npm ERR! A complete log of this run can be found in:\n' +
    'npm ERR!     /home/runner/.npm/_logs/2022-01-06T20_11_34_361Z-debug.log',
  failed: true,
  timedOut: false,
  isCanceled: false,
  killed: false,
  pluginName: '@semantic-release/npm'

I'm working on resolving this issue with some internal teams.

The release job ran successfully, 🥳   but NPM didn't update 😭  So I'm now looking into what may be occurring with NPM.
[8:20:27 PM] [semantic-release] › ✔  Completed step "verifyConditions" of plugin "@semantic-release/npm"
[8:20:27 PM] [semantic-release] › ℹ  Start step "verifyConditions" of plugin "@semantic-release/git"
[8:20:27 PM] [semantic-release] › ✔  Completed step "verifyConditions" of plugin "@semantic-release/git"
[8:20:27 PM] [semantic-release] › ℹ  Found git tag v2.1.0 associated with version 2.1.0 on branch next
[8:20:27 PM] [semantic-release] › ℹ  Found 0 commits since last release

@tristandenyer / @herecydev  We've now published a next version of this plugin that will support Gatsby 4. if you come across any issues, please let us know by reporting an issue here.
https://www.npmjs.com/package/gatsby-plugin-newrelic/v/2.1.1
Sorry for the delays on this, we actually had to work through some internal process improvements for this repository, but now it's all sorted out.


Revert jobID on batch request

What changed?
Revert jobID on batch request

Why?
No need to pass in job id from user. The batch will generate deterministic id from request.

Breaking changes

WDYM by deterministic ID?

The idea is to hash the request to an ID so we don't ask user to supply a job_id which feels weird.

WDYM by deterministic ID?

Adding the job ID in request was for de-dup retries from clients.(Make sure no duplicate batch job created). The new approach is to hash the request so that it is deterministic from the request parameters. If those parameters are the same, we treats it as the same request and do the de-dup.
The request parameters will be hashed to a workflow ID. The workflow ID and run ID will be encoded into job ID based on base64. The describe and stop requests need to decode the job ID to get the workflow ID and run ID. The job ID length is fixed to 64.

 (Let's say the signal itself has no args.)

 (Let's say the signal itself has no args.)

If the batch job is finished, it will start a new one with a different run ID. If the batch job is still running, it will not start a new one.

If the batch job is finished, it will start a new one with a different run ID. If the batch job is still running, it will not start a new one.

Then there's a weird discontinuity: say my batch signal job takes exactly 5:00. If I want to signal again in 4:59, it de-dups and doesn't work. But if I do it in 5:01, it signals again. This seems a little surprising

If the batch job is finished, it will start a new one with a different run ID. If the batch job is still running, it will not start a new one.

It is the same behavior as you try to start same workflow.

When does the batch job list workflows? In the beginning of the job?

I'm asking because if we dedupe based on the input we prevent the same batch from being used to signal / cancel workflows as they're being created unless the batch job takes incoming workflows that match the query into account.

It is the same behavior as you try to start same workflow.

Yeah but when starting a workflow, the caller can choose whether they want deduping or not. Here, the behavior is not easily predictable. I'd be fine with making job id optional and generating it from the input if missing, but there should be a way for a caller to opt out of deduping, e.g. if they want to send a signal multiple times, or cancel new workflows that have started after a previous batch is running before that batch finishes.

When does the batch job list workflows? In the beginning of the job?

The list could result in many pages and the batch operation progress as it page through the result set. Each page is a new ListWorkflowExecutions call. If you want to use batch to send signal to workflows including workflows started after the batch operation initiated, then there is race condition and even if you could start multiple runs you risk of having some workflows receive it once and others received twice. Better option is to limit the time window of each batch (by filter on startTime to that fixed window), and you can start multiple batches each cover a disjoined time window.

I'd be fine with making job id optional
It is weird to ask for job_id as input and it is even more weird to return it as output as well.

they want to send a signal multiple times
I think you are making up weird use case here. If the signal is different, they can start different batches. Or they can scope the time range of target workflows.

I'm asking because if we dedupe based on the input we prevent the same batch from being used to signal / cancel workflows as they're being created unless the batch job takes incoming workflows that match the query into account.

As Yimin mentioned, there is race condition on the listing. It means the de-dup won't work if we takes the incoming workflows.

I'd be fine with making job id optional
What about take in an optional request id, we can hash the input including the request id?

I’m still not 100% convinced that a user will never want to send a signal multiple times.
What if they send the signal and then realize that they have some workers up running workflow code that doesn’t have the signal handler configured, they might want to restart the batch and not have to wait for the previous batch to complete. We can maybe tell them that they should stop the previous batch first but it doesn’t seem like it’ll be the most obvious behavior, I don’t think people will pay attention to the fact that we give them back the same batch ID.
If we dedupe, there should be an explicit server response saying that a batch with these parameters is already running and that no nee work was started, I’d even include the guidance given in the comments of this PR in the server message.
For example the message might say:
“A batch with these parameters is already running with ID …. To restart this batch do … if you want to target run this batch job on new workflows do …”
We could also provide restart and continue from helper APIs to help users achieve these things.

I’m still not 100% convinced that a user will never want to send a signal multiple times.

There is definitely cases that a user want to send the same signal multiple times. The rare case is to send the same signal multiple times concurrently. And I agree we need to indicate it to users if a batch job is already running. I think include an optional request id would solve your concern.


Bug report: Not treating well buffer data...

I found a critical bug when working with buffers.
What happens is that sublevel probabbly relies on Bytewise to do all the buffer encoding/decoding. However, there are cases where you want to generate the buffer by some external genrator. Like UUID generators.
Here's how to reproduce the bug:
var b = new Buffer(8);
b.writeUInt8(123,0);

db = sublevel(db);

db.put(b, "TST", {keyEncoding: 'binary'});

db.createKeyStream({keyEncoding: 'binary'}).on('data', console.log);

The above code won't even return Buffers on keys... It would generate useless strings.
A possible workaround:
var b = new Buffer(8);
b.writeUInt8(123,0);

db = sublevel(db);

db.put(bytewise.encode(b), "TST", {keyEncoding: 'binary'});

db.createKeyStream({keyEncoding: 'binary'}).on('data', function (d) {console.log(bytewise.decode(d))});

I have no idea why it works, but it shouldn't work this way. Both examples work well with raw levelup instance.
Thanks.

I found the issue...
It's here: https://github.com/dominictarr/level-sublevel/blob/master/codec/index.js#L8
Here's what happens, ByteWise overrides the toString method of each encoded buffer. So everything that is being called with ByteWise's toString without specifying encoding, would be encoded to hex. The default behavior of node encode's everything to utf8 which seems not to work well with custom buffers.
Anyway, when I override the toString method of a custom buffer to be hex as default, everything works.
What I'd suggest is that the typecasting of buffers won't rely on default methods but on hex. I can issue a pull request if you'd like.
Thanks.


How does preact compare to react-lite?

https://github.com/Lucifier129/react-lite

@andyhu I should clarify up-front: Preact is not just an attempt to perfectly emulate React's behavior in a small library size (although preact-compat probably is that attempt).  It is an implementation of the concepts of React, but it is not free of opinions.  There is a fairly complete list of differences between Preact and React on the wiki.
Here are some key differences:
Preact is faster
This standard benchmark shows Preact is more than 2x faster overall:

Preact is much smaller
Preact is 3.5Kb, react-lite is 17kb.
More Features
Preact actually has more features from React than react-lite does!  It can render to a static HTML string via preact-render-to-string. Further, Preact supports isomorphic rendering, meaning your code on the client picks up where the server rendered markup leaves off - the first render on the client is a diff, instead of a replace.
Preact is Modern
Preact is written in ES2015. It intentionally avoids bringing deprecated APIs into its core, instead leaving that work to preact-compat so that it is entirely optional.  It is pluggable via vnode hooks, includes performance features like batching right out of the box, and works in IE9.

Closing since this is answered. Please let me know if I missed anything important.


[bsc-testnet] Txs failing on contract

https://testnet.bscscan.com/address/0xce5e0d6870594f4923322d1771cf9606c2f1db7e
Constant reverts, not sure what's going on.

Blocked by Graph not syncing BSC.

New contract seems to be working: https://testnet.bscscan.com/address/0x29E81453AAe28A63aE12c7ED7b3F8BC16629A4Fd


Update harfbuzz

@hjanetzek Notice any changed behavior from this?

@matt it looks all correct with this Burmese font http://www.rfa.org/burmese/help/font_download_english.html

Ok I am not seeing the Burmese issue on android now on master. But both master and this branch shows issue on ios. Looks like @hjanetzek initial guess of wrong font being picked on ios could be the issue here.
Irrespective its good to update harfbuzz.

@tallytalwar @matteblair have you tried the ios font for Burmese on osx? I guess the same font used on ios is available there. If it works the problem could be with extracting the font header/tables in TGFontConverter.mm

@hjanetzek I can check that by explicitly bundling this font in the ios demo app. Stay tuned.

Merging this in, but investigation for burmese on ios still continues.

@hjanetzek with respect to your suggestion of trying out Myanmar Sangam MN font for osx demo.
I found the font file (ttc) sitting here: /Library/Fonts/Myanmar Sangam MN.ttc. Used https://transfonter.org/ttc-unpack to convert this ttc to ttf. And following is the result on osx demo:

So possible its the font file on ios/osx!


I can't get it working with my webm while it work as expected with the webm given in the demo

You can find my file here : http://beginfill.com/webm/out.webm
It's using an alpha background , maybe the problem come from here (?)
Any idea how to get it working as expected ?

Same problem here.. any possible solution?

@kareljuricka
I didn't find THE solution so I created an alternative.
Honnestly, it's not a good solution but it works.
I used ffmpeg to extract frames from the webm, then I created some spritesheet using these frames and created a spritesheet reader in order to be able to read the webm frame by frame.
It's a very heavy solution but it works.
If you like I can give you the code I use to do it


 It would be cool if I can only see news about the games without the comments.
Apart from comments, sometimes posts show "Hot" Comments under it (example). If possible, please consider blocking it too.
Thanks for such a great extension!

Glad it's working out for you. :) I added the site.. after you update the definitions (on the options page), let me know if you still see comments anywhere on there.


[feat] Add serve

What does this PR do?

Fixes # (issue)
Before submitting

[ ] Was this discussed/approved via a Github issue? (no need for typos and docs improvements)
[ ] Did you read the contributor guideline, Pull Request section?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes?
[ ] Did you write any new necessary tests? [not needed for typos/docs]
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] If you made a notable change (that affects users), did you update the CHANGELOG?

PR review

[ ] Is this pull request ready for review? (if not, please submit in draft mode)

Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

Hello @tchaton! Thanks for opening this PR.

In the file flash/core/serve/dag/optimization.py:

Line 598:67: W504 line break after binary operator
Line 600:25: E129 visually indented line with same indent as next logical line
Line 702:25: W503 line break before binary operator
Line 702:73: W504 line break after binary operator
Line 704:25: E129 visually indented line with same indent as next logical line
Line 879:13: W503 line break before binary operator

In the file tests/core/data/test_data_pipeline.py:

Line 85:121: E501 line too long (145 > 120 characters)

Do see the Hitchhiker's guide to code style


AV ckpt inference error

Hi there,
Thanks for releasing the new av ckpt, but it seems your codebase hasn't been updated yet. So when running inference with the sample code, it gets error of
TypeError: process_video() got an unexpected keyword argument 'va'
Do you have any timeline of av version updates?
Bests,

Thanks for your attention! You can switch to the audio_visual branch (https://github.com/DAMO-NLP-SG/VideoLLaMA2/tree/audio_visual) and clone the repository to run inference for audio_visual related tasks.

Hi there,
Thanks for releasing the new av ckpt, but it seems your codebase hasn't been updated yet. So when running inference with the sample code, it gets error of TypeError: process_video() got an unexpected keyword argument 'va' Do you have any timeline of av version updates?
Bests,

Hello,I'm a phD student from ZJU, I also use videollama2 to do my own research,we create a WeChat group to discuss some issues of videollama2 and help each other,could you join us? Please contact me: WeChat number == LiangMeng19357260600, phone number == +86<PHONE_NUMBER>0,e-mail ==<EMAIL_ADDRESS>


Bump gotip to go1.17beta1

Motivation
Hey! I'm finishing up a play-with-go guide on shuffling tests; a new Go 1.17 feature.
For this reason, I needed a more recent gotip image until Go 1.17 is officially released.
The problem
Solved, thanks Marcos!
~Unfortunately, the image cannot be currently built, just by pointing to a newer commit.~
~It either fails with gpg: keyserver receive failed: Connection timed out or Version '1:2.31.1-0ppa1~ubuntu20.04.1' for 'git' was not found.~

Hey @tpaschalis! just fixed this on tip. Please rebase and give it go!

Just rebased and checked that the image works as intended locally, including git ^^
I'll continue polishing the guide on play-with-go and use this image when it's merged. Thanks a lot for the help Marcos!

Closing this as we are now on go1.19.


Crawl German Nvidia.de Store Page

I have the product id for the german nvidia store page, is it possible to crawl that instead of the USA site?

There is a Canadian version. I don't mind putting it in if you have the URLs unless you'd like to make a PR!

I have the product id for the german nvidia store page, is it possible to crawl that instead of the USA site?

I have the product id for the german nvidia store page, is it possible to crawl that instead of the USA site?

Hey there,
thanks for the reply!
I have two links:

You can verify it is the correct product ID (at time of writing) here:
But I cant find the german link for the nvidia.ts file. In the nvidia.ts file there is a link that looks like this:
https://api.digitalriver.com/v1/shoppers/me/products/5438481700/inventory-status?apiKey=9485fa7b159e42edb08a83bde0d83dia
So we need to figure out the german api Link I guess?

Looks like we could get the inventory status from
https://in-and-ru-store-api.uk-e1.cloudhub.io/DR/products/de_DE/EUR/5438792300

Not sure how reliable that API is though.

I use it with his link https://in-and-ru-store-api.uk-e1.cloudhub.io/DR/products/de_DE/EUR/5438792300
it works but dont know how exactly this is.

Looks like we could get the inventory status from
https://in-and-ru-store-api.uk-e1.cloudhub.io/DR/products/de_DE/EUR/5438792300

Not sure how reliable that API is though.
I tried using the product id of<PHONE_NUMBER> and returned that it wasn't found in https://api.digitalriver.com/v1/shoppers/me/products/5438792300/inventory-status?apiKey=9485fa7b159e42edb08a83bde0d83dia

It tells me
"Please verify your API key and secret (if applicable) is correct." Any workaround?

The in-and-ru-store-api link is reliable, I have been using the US version of it for days in a bash loop to check the inventory. You can make it check US inventory by changing /de_DE/EUR/5438792300 to /en_us/USD/5438481700.


Custom date input does not reflect first day of the week

Right now when using HammerTime in Swedish, the calendar appears like this:

It appears with Sunday (Sö) first, when the standard for Sweden is Monday (Må). This issue is also seen with English, UK, español, italiano and so on.
Further are minor issues, such as how the weekday names starts with a capital letter, when this isn't the case in for example Windows, where it's written as "må ti on to fr lö sö". Second issue is that Saturdays are marked in red, when in Sweden, only Sunday and official holidays are red days. Marking non-red days in red just looks wrong.
Similar issue with the red days is the case for when using Hebrew (Israel) where Saturday and Sunday are still marked in red, despite the official weekend in the country is Friday and Saturday. Their work week is Sunday through Thursday.

Thank you for reporting these issues, the wrong first day of week was a coding error,trying to read the information from a locale called 'locale' instead of the current language, and as for the weekend days the date libraries do not provide which days should be marked red, so it takes the default Saturday and Sunday. I've updated the code for Swedish and Hebrew based on your suggestion and I will reach out to the translators to get this information for other languages.

Unicode has territory information regarding this: https://unicode-org.github.io/cldr-staging/charts/latest/supplemental/territory_information.html
Default for all regions it is Sat-Sun for the weekend, but that doesn't mean both days are red days.
For regions where the weekend is different:
Thu-Fri: AF
Fri-Fri: IR
Fri-Sat : BH DZ EG IL IQ JO KW LY OM QA SA SD SY YE
Sun-Sun : IN UG

So for Sweden, the weekend is Sat-Sun, but only Sun is red.
Looking for other calendars from different sources: Germany, Switzerland, Austria, but it's hard to find any good sources for this information. There's a lot of random sites ... Spain
Would be good if CLDR did have information about red days, so most probably just avoid including them.

Due to a bug I had to set the option that controls which days are red to a fixed empty list if the values are not known yet, because the library wasn't using the defaults in some cases. In effect this means that until manually specified, a locale will have no weekend days associated. I'll add the info to the locales gradually as I get feedback from the translators.


Fix tests when run via http://localhost:4200/tests

no issue

blogUrl in the mocked config was hardcoded, switching to using the current origin fixed the navigation tests
we were only pulling jquery drag simulator library in on test builds but we also need it added to test-support.js in development builds
lazy loading is disabled in the test env which previously worked fine as all of the codemirror assets are bundled into the test build, now that we also run tests using the development build we needed to update the config to import codemirror assets into the test-support.js file - this ensures that we have normal lazy-loading behaviour when developing locally and that accessing via http://localhost:4200/tests doesn't die horribly due to missing dependencies

Coverage remained the same at 74.645% when pulling 3f93f210e3cb3bbce9be913ce17582f8bf512e5d on kevinansfield:dev-tests-fix into f5fc35fb9864eba162b663c32eec9784efe3ca63 on TryGhost:master.


fix bug when splice feature for batch

if utt.shape[0] < context_width, tfkaldi will break down

@vrenkens This PR will skip the utt which is too short to expand context

Hi Gafeng,
Thanks for bringing this bug to my attention. However, I don't think your patch fixes the bug. There are also some incorrect (or unnecessary) statements. For example, you set the splice control bool and the check it in an if statement immediately. I have instead fixed the bug myself. Can you test it and confirm it has been fixed?
Cheers


When enabling drawHierarchyInKeyWindow, view's safeAreaInsets may not be set correctly

Describe the bug
For our use case, we need to enable drawHierarchyInKeyWindow to test 3rd party UI framework and generate visual diff results. As such, we're extending swift-snapshot-testing/Sources/SnapshotTesting/Snapshotting/UIViewController's image function, and passing in the boolean to enable drawHierarchyInKeyWindow:
extension Snapshotting where Value == UIViewController, Format == UIImage {
    /*
     Copied from swift-snapshot-testing/Sources/SnapshotTesting/Snapshotting/UIViewController
     Exposing drawHierarchyInKeyWindow
     */
    public static func image(
        drawHierarchyInKeyWindow: Bool,
        on config: ViewImageConfig,
        precision: Float = 1,
        perceptualPrecision: Float = 0.98,
        size: CGSize? = nil,
        traits: UITraitCollection = .init()
    )
    -> Snapshotting {
        print("config's safe area: \(config.safeArea)")
        return SimplySnapshotting.image(precision: precision, perceptualPrecision: perceptualPrecision, scale: traits.displayScale).asyncPullback { viewController in
            snapshotView(
                config: size.map { .init(safeArea: config.safeArea, size: $0, traits: config.traits) } ?? config,
                drawHierarchyInKeyWindow: drawHierarchyInKeyWindow,
                traits: traits,
                view: viewController.view,
                viewController: viewController
            )
        }
    }
}

With drawHierarchyInKeyWindow enabled, when running snapshot tests on a mix of iPhone and iPad strategies in landscape, it seems that if previous device's safe area is .zero, it breaks the the next device's safe area.
For example, from the provided project:
final class MyAppTests: XCTestCase {

    func testSnapshot() throws {
        assertSnapshots(matching: NamesTableViewController(), as: [
            "ipad-mini-landscape" : .image(drawHierarchyInKeyWindow: true, on: .iPadMini(.landscape)),
            "iphone-se-landscape" : .image(drawHierarchyInKeyWindow: true, on: .iPhoneSe(.landscape))
        ])
    }
}

If the snapshot test runs the iPad Mini strategy first (which has 20 top offset), then the test will correctly generate the ref and test result view with the proper safe area guides. However, if the iPhone strategy is run first (which has 0 top offset), the following iPad run will also inherit the 0 offset.
This seems to originate from the following place:
// in Sources/SnapshotTesting/Common/View

func prepareView(
  config: ViewImageConfig,
  drawHierarchyInKeyWindow: Bool,
  traits: UITraitCollection,
  view: UIView,
  viewController: UIViewController
  ) -> () -> Void {
  ...
  let dispose = add(traits: traits, viewController: viewController, to: window)
  ...
}

private func add(traits: UITraitCollection, viewController: UIViewController, to window: UIWindow) -> () -> Void
{
  ...
  viewController.didMove(toParent: rootViewController)

  window.rootViewController = rootViewController // <-----

  rootViewController.beginAppearanceTransition(true, animated: false)
  ...
}

In successful test run (iPad runs before iPhone), setting the window's rootViewController shows the safe area guides are applied. However, when iPhone test runs before iPad, the safe area guide seems to get zeroed out.
To Reproduce
MyApp.zip
Expected behavior
The safe area guide should adjust when running multiple strategies with different orientations.
Screenshots
This is the expected snapshot:

When run after iPhone strategy, the generated iPad Mini snapshot shows no offset:

The test then fails because the entire view is shifted due to the missing offset

Environment

swift-snapshot-testing version 1.11.0
Xcode 14.2
Swift 5
OS: 15.2

Additional context
I may be using this incorrectly to enable drawHierarchyInKeyWindow, and would love to hear any suggestion on an alternative approach 🙏

I wonder if the bug is actually here (https://github.com/pointfreeco/swift-snapshot-testing/blob/main/Sources/SnapshotTesting/Common/View.swift#L964) - since there's a special case for zero safeArea that doesn't seem to be undone (via the dispose pattern) like most of the other changes.


[CLOSED] Change repo settings to rebase and merge

 Issue by owalk
Thursday Feb 09, 2023 at 22:30 GMT
Originally opened as https://github.com/Sean-Nishi/M1Wengine/issues/25

The current settings allow for merge commits, we should disallow this in favor of rebase and merge on PR close.
There should be a setting to do so under the repository settings.
The image below shows that creating a merge commit is currently the default option.
Remove the 'create a merge commit' option entirely.

 Comment by Sean-Nishi
Saturday Feb 11, 2023 at 15:11 GMT

Setting was in my repo settings instead of project settings.


Dear @improbably-you,
Thank you for taking the time to contribute to our project. Your enthusiasm is appreciated!
In order to maintain the project's focus on improving user experience and functionality, we kindly request that you submit pull requests that address critical issues or enhance the project's detectability. While ASCII art and similar content can be entertaining, our priority is to address the most pressing concerns affecting users.
We look forward to your future contributions that align with the project's objectives.
Best regards,
Momen

I feel like that comment is misleading. I did not contribute any ASCII art. I modularized the code, removed uneccesary calls, conditionals (used arg parses to handle missed required params), useless imports, variables, etc. I also added new functionality. I made suggestions across the board and offered a number of efficiency improvements. It's funny to dismiss it as ASCII art.

Hey there!,
Sorry if you misunderstood me, but your comment was having suggestions to
use the string formatting with the ASCII art, and this comment was after
the first one in which you asked for unnecessary imports and this was
already done and also applied some OPSEC enhancements. I think there is no
additional enhancements regarding unused/necessary vars or imports.
On Sun, 26 Mar 2023 at 6:32 PM improbably-you @.***>
wrote:

I feel like that comment is misleading. I did not contribute any ASCII
art. I modularized the code, removed uneccesary calls, conditionals (used
arg parses to handle missed required params), useless imports, variables,
etc. I also added new functionality. I made suggestions across the board
and offered a number of efficiency improvements. It's funny to dismiss it
as ASCII art.
—
Reply to this email directly, view it on GitHub
https://github.com/Cyber-Guy1/WCE/pull/2#issuecomment-1484132554, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/APZZMFDIEBKZFWQXHV7SONLW6BOQXANCNFSM6AAAAAAWH3ZZD4
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

My comment was unrelated to my pull request. There are still issues with the code. If you'd like to see a full list of changes included in this pull request, please don't hesitate to open the commit and read through them yourself 😜

In response to your comment, I kindly request that you provide a detailed summary of the issues addressed in your pull request, as it will greatly assist in the review process. Please ensure that your summary includes the following information:

The origin of each issue
Your rationale for identifying these as issues
The ways in which your proposed changes will improve the code and its workflow
The specific problems your changes aim to resolve

Providing this information will enable both of us to use our time more efficiently and maintain a professional atmosphere. The purpose of pull request comments is to clearly communicate the intentions behind your code changes, facilitating a productive review process. I recognize that your time is valuable, and I appreciate your commitment to this project. In order to facilitate a productive and efficient review process, I kindly request that you provide the necessary information before proceeding with further comments. This will enable us to focus on addressing the key issues and ensure that our collaboration remains professional and effective.
In the interest of maintaining a professional atmosphere, I kindly request that you refrain from using emojis in our communication. Doing so will help ensure that our discussions remain focused and mature, allowing us to work more effectively together. Thank you for your understanding.

For commander.py
The origin of each issue:

The original code lacked the functionality to read commands from a file.
The original code had global variables, which made the code harder to read and maintain.
The original code contained an unnecessary ASCII art, which is unrelated to the main functionality of the script.
In the original code, the argparse-related code and the main functionality were mixed together, making it harder to read and maintain.
The original code was missing error handling when creating a socket, which could lead to unexpected crashes.
The original code used the same variable names for command and target IP across different lines, making it harder to follow the logic.
The original code contained unnecessary conditional statements that could be simplified.

Your rationale for identifying these as issues:

Reading commands from a file is useful when a user wants to execute multiple commands without providing them one by one.
Refactoring the code by removing global variables and organizing the code into functions improves readability and maintainability.
Removing the ASCII art maintains a professional appearance and focuses on the core functionality of the script.
Separating the argparse-related code from the main functionality improves the code's readability and maintainability.
Adding error handling for socket creation prevents unexpected crashes and provides a better user experience.
Using distinct variable names for command and target IP improves the clarity of the code and prevents confusion.
Simplifying unnecessary conditional statements can improve the code's readability.

The ways in which your proposed changes will improve the code and its workflow:

The updated code includes a new function read_commands_from_file(file_path) to read commands from a file.
The updated code organizes the code into functions, such as create_arg_parser() and main(), improving the structure and making it easier to follow.
The updated code removes the ASCII art, keeping the focus on the functionality and maintaining a professional atmosphere.
The updated code refactors the argparse-related code into a separate function called create_arg_parser().
The updated code adds error handling when creating a socket in the main() function.
The updated code uses different variable names for command and target IP, making it easier to understand the code's logic.
The updated code simplifies the conditional statements related to checking if command-line arguments are provided.

The specific problems your changes aim to resolve:

The updated code addresses the lack of functionality to read commands from a file.
The updated code resolves issues related to readability and maintainability by refactoring the code structure.
The updated code removes unnecessary ASCII art to maintain a professional appearance.
The updated code resolves issues related to readability and maintainability by separating argparse-related code from the main functionality.
The updated code addresses the lack of error handling when creating a socket, preventing unexpected crashes.
The updated code improves code clarity by using distinct variable names for command and target IP.
The updated code resolves issues related to code readability by simplifying unnecessary conditional statements.

For receiver.py
The origin of each issue

In the original code, the argument parser is created and configured directly in the global scope, making it less modular.
In the original code, the decode function reads the 'log' from the global scope, limiting its reusability.
The original code lacks a main function, causing the main part of the code to be mixed with other parts, which affects code organization.
In the original code, the KeyboardInterrupt exception handling is done within the same try block as the subprocess, making the code harder to read.
The original code includes unnecessary variable initializations in the decode function, causing inefficiency.

Your rationale for identifying these as issues

Creating and configuring the argument parser directly in the global scope can make the code harder to maintain and update in the future.
Reading the 'log' from the global scope limits the decode function's versatility and reusability in other contexts.
Mixing the main part of the code with other parts can make it harder to understand the code's structure and to perform testing.
Combining KeyboardInterrupt exception handling with the subprocess code makes the code more difficult to understand.
Unnecessary variable initializations consume resources and affect the code's overall performance.

The ways in which your proposed changes will improve the code and its workflow

By using the create_arg_parser function, the code becomes more modular, making it easier to maintain and update the argument parser.
Passing the 'log' directly to the decode function as an argument makes it more versatile and reusable.
Encapsulating the main part of the code within a main function improves the code organization and allows for easier testing.
Separating the KeyboardInterrupt exception handling into a distinct except block makes the code more readable and understandable.
Removing unnecessary variable initializations in the decode function improves the code's efficiency and resource usage.

The specific problems your changes aim to resolve

The changes improve maintainability and modularity by organizing the argument parser creation and configuration into a function.
The changes enhance the decode function's reusability by making it independent of the global scope.
The changes address the code organization issue by introducing a main function, which separates the main code logic from other parts.
The changes improve code readability by handling the KeyboardInterrupt exception in a separate except block.
The changes aim to resolve inefficiency by removing unnecessary variable initializations in the decode function.

Dear @improbably-you,
Thank you for taking the time to submit a comprehensive pull request with detailed explanations regarding the issues you've identified in the code. Your focus on improving readability, maintainability, and efficiency is commendable.
While we agree that the addition of the new function for reading commands from a file is a valuable enhancement to the code, we believe that many of the other proposed changes may not significantly impact the overall usability of the code. As a result, we will focus on integrating the new function into the codebase soon.
Regarding the other issues you've mentioned, such as the removal of ASCII art and similar changes, we appreciate your input, but we believe that these modifications may not substantially improve the user experience. Instead, we will prioritize adding comments and documentation to the code, which will facilitate easier maintenance and debugging for future contributors.
We value your contribution and encourage you to continue submitting issues or pull requests related to adding new features or addressing critical issues that may affect the code's operational usability.
Thank you once again for your efforts, and please do not hesitate to reach out with further suggestions or concerns.
Best regards

I think it's cool that you're using ChatGPT to respond to me, it's very innovative.
However, some might find it a bit impersonal and rude. While the remarks from the bot are professional in nature, they miss a number of clue-ins regarding context.
For example, you phrased your last response as an email and used the royal 'we' across the board.

Dear @improbably-you,
I think your problem is with me personally for some reason, but it's ok; nothing personally from my side except respect as you were trying to help out here. I'm not required to explain why, but I'm going just to make you understand, I'm responding with "we" as I always do in my emails and even comments, as I used to work in teams and so I used to write them whenever I remembered, but other than that, I don't know what you're talking about by "using chatGPT." If you have something personally, please discuss it with me by sending me a DM. Otherwise, please keep in mind the core usage of this function at github, which is enhancing code instead of throwing judgment. I'm sorry but I'm not going to continue this conversation as I have important things to do, best of luck.
Best regards


Need to document how to enable flowzone in non product-os repos

Figuring that out for balena-io-hardware right now : https://jel.ly.fish/thread-5bd62f66-1041-447b-a27c-fda219f6db37
Will hopefully get some screenshots to add to the readme from this exercise

Here's an example: https://github.com/balena-io/open-balena-registry-proxy/pull/48

But I imaging the bit you are missing is all the secrets at the org level, which requires an owner and someone with access to all the secrets.

But I imagine the bit you are missing is all the secrets at the org level, which requires an owner and someone with access to all the secrets.

@klutchell would I need "all" the secrets? I am not going to be using any of these -
      NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
      DOCKERHUB_USER: ${{ secrets.DOCKER_REGISTRY_USER }}
      DOCKERHUB_TOKEN: ${{ secrets.DOCKER_REGISTRY_PASS }}
      BALENA_API_KEY: ${{ secrets.BALENA_API_KEY }}

I think I'd only need
      FLOWZONE_TOKEN: ${{ secrets.FLOWZONE_TOKEN }}
      GPG_PRIVATE_KEY: ${{ secrets.GPG_PRIVATE_KEY }}
      GPG_PASSPHRASE: ${{ secrets.GPG_PASSPHRASE }}

which I will ask the ops team to set for my org

Correct, you only need the latter 3. I have the values you need, but I'm not an owner of that org to set secrets.
We have already added a GPG key to the balena-ci GH user, you just need the secrets created.

@klutchell can you share those secrets with @LPapapo please?
He's helping me out with this here

operational, resolved.


GO:0000154 : rRNA modification

should
GO:0042254 : ribosome biogenesis and assembly
be a parent?
Reported by: ValWood
Original Ticket: "geneontology/ontology-requests/1563":https://sourceforge.net/p/geneontology/ontology-requests/1563

thanks www.movied.org


Add workaround for too long test names on Android

This PR is aimed at #1725

I'd prefer to wait until it's fixed upstream (link).


Game list and message personalization

Adding a simple game list (configurable in the config.json) and adding a name field in the messages.

This is fantastic, thanks!


multi balances

Overview
Description: As a user, I want to see my total G$ balance across all chains, and my balance per chain.
Acceptance Criteria:

When in collapsed mode, balance should show chain balance instead of total balance. (No changes to existing behavior)

Notes: Button between chain balances is the Bridge flow. (Issue #????)
Assets
Preview:

Figma:
https://www.figma.com/file/ihw1PxBvLxacTHnN2aj4lC/3.-Product-💵?node-id=3870%3A78605&t=FRWORG5Tec4aWGqa-1

Updated preview:

Added request:

[ ] - tapping balance of inactive network switches chain

Verified on prod


Adding token collection to lex.py for REPL project

This pull request is for the features added for my proposed 2/5/2024 and 2/26/2024 milestones, adding a function to collect all identifiers defined by either the user (variables, functions, etc.) or the language (keywords, built-ins, etc.), with another function to filter the identifiers based on a prefix.
These additions were added into lex.py as the functions get_identifiers and get_identifiers_by_prefix
Additionally, a file was added to .gitignore (asteroid/repl_test.py), that I have been using to test out these features

feel free to merge this


When will the codes be released?

Thank you for your perfect work! When can you release the code?

+1


Toolchain: Run 'make' in a watch on the VM

Watch the /src/ folder. On changes, do a rebuild.

This might get canned, as the build process is actually quite slow.


Change aws-iam-autenticator to awscli for helmfile

Delete process for installing kubectl

base image helmfile has already kubectl
https://github.com/roboll/helmfile/blob/master/Dockerfile.helm3#L30-L33

Add awscli for eks authentication tool
Fix README

LGTM!!


Make it possible to run in virtual directory

Some minor changes to make it possible for Vorto to run on an Umbraco instance inside a virtual dir

Nice. Didn't know you could extend the server variables like that. Very cool.


s behaves differently on sizes bigger than 0x400

Structures over 0x400 aren't properly created when using "s" on the call of an allocator:
Note this is especially problematic because it's extremely difficult using the UI to turn the invalid __padding offset back into the actual padding.
(just press "s" on x0_7 and x0_6 in the attached)

sample.zip

The structure are created in the very same way. They look different in the types view because we did some UI hack to avoid the excessive padding in really large structs. You can test that if you create a structure >=0x400 directly, it is the same.
Does this behavior causes anything inconvenient?

This is just a consequence of the analysis.types.paddingThreshold setting being rather low by default. Any types >= that setting are rendered in a single line for performance reasons (previous users had trouble with structures that were gigabytes in size). The default setting value of 0x400 may be a bit too low for this, given that it can trigger with very reasonably sized structs as you've noticed here. So the crux of this issue is likely just "make that setting bigger by default," as it seems to otherwise be working as intended.

Ultimately rendering it like that is actually fine visually if we fixed the fact that "create structs referenced" feature fails to work when it's rendered like this. Don't know if that is an easy change or not though.


Don't nag about rewards approval on sign in

Some users will opt to use LBRY without rewards, so we should not nag them on every sign in to enable them.

Being taken care of and will eventually be saved into the wallet preferences.

This should now be fixed.