Change "people" to "teams of people" on Pricing page

Changed link to direct to team list.  "Teams of people" sounds friendlier than just "teams" and helps people remember that teams are made of people.

teams are made of people

My worst fear has come true: teams are soylent green :scream:

:tea:

a green wafer

Obviously, I still have to watch this movie.


Update SAP Cloud SDK Tutorials

@sacnl If you find time, please take a look. I updated the rules from exact matched strings to multiple choice (which you somehow don't see here ¯_(ツ)_/¯ - Junjie told me, that I also need to change the according tutorials, so I hope that's enough).
I also adjusted the Tutorial for app deployment, because the CLI api changed slightly to account for the reported Windows issue.

Am I supposed to review the changed test sections as well? If yes, how? If not, fine :)

I guess that is not necessary, I will fix your proposed changes and merge.

@sacnl I forgot I can't merge ;) Please review again.


User friendly validation message defined in the model itself?

Why not let users define nice validation messages in the model itself, next to each validation, and keyed to say the field name - like is done in rails.
This could make it super easy to flash the messages and highlight the fields where validation failed in the UI. This should not require any further parsing and stuff like mongoose requires.

@mikermcneil Was this feature added at the latest version? Currently having problems with validation errors.

For sails v0.11.X you may use this sails-hook-validation

@lykmapipo thanks for the link!

Latest version 0.12.9 of sails doesn't seem to be working with sails-hook-validation.


python/tests/t_*: Use the "super().manifests()"-at-the end idiom

In today's Ambassador-U, Flynn explained that the proper idiom is to
include new manifests at the beginning; eg.
    return """
MY-YAML
""" + super().manifests()

and not
    return super().manifests() + """
MY-YAML
"""

pretty much because "namespaces".
But a bunch of the existing tests do it the discouraged way.  This
increases the odds that someone won't just figure-it-out looking at
whats's there.  So change it.

v2:

rebase on to master
fix t_shadow.py per Flynn's comment
also adjusted a new test in t_tls.py

make pytest KAT_RUN_MODE=envoy is running now...


add timing log to check_database

Explain your changes:
This PR adds timing log to Persistent_frontier.Instance.check_database. This is for the debugging of a long async cycle that we saw on testworld-2-0.
See: https://app.zenhub.com/workspaces/protocol-eng-64d12d77bff4a91f97f60205/issues/gh/minaprotocol/mina/14665
Explain how you tested your changes:
*
Checklist:

[ ] Dependency versions are unchanged

Notify Velocity team if dependencies must change in CI

[ ] Modified the current draft of release notes with details on what is completed or incomplete within this project
[ ] Document code purpose, how to use it

Mention expected invariants, implicit constraints

[ ] Tests were added for the new behavior

Document test purpose, significance of failures
Test names should reflect their purpose

[ ] All tests pass (CI will check this if you didn't)
[ ] Serialized types are in stable-versioned modules
[ ] Does this close issues? List them

Closes #0000

!ci-build-me

!ci-build-me

!ci-build-me

!ci-build-me

!approved-for-mainnet


Ability to exclude fields/records

Add additional optional configuration to exclude fields and possibly children records (EG where clause).  Currently this can be done by just nulling/removing them from the clone SObject instance property

Edit: there doesn't appear to be any mutate the children on a generic SObject, so I don't removing a child from the returned result set currently is possible.
One solution would be to expose a Map<string, SObject[]> which contained the children that will be updated (key is relationshipname)


Dragonflight

Get the layout ready for Dragon Flight

Thank you very much for updating this addon, I was still using it because I really like the interface.


Decouple SP4 and SP5 by reducing shared code

While initially building-out SP5 we shared functionality with SP4 using shared code projects (.shproj)
https://github.com/ScottPlot/ScottPlot/tree/main/src/Shared/ScottPlot
After doing this for a few months, I am no longer in favor of this design because:

It couples SP4 and SP5 together in a way that is not desirable
It requires one code change to require running SP4 and SP5 builds and tests
It complicates the solution files and makes it difficult to locate code
Breaking changes allowed in SP5 cannot occur in files shared by SP4

Current Functionality in Shared Code
I don't think we're getting enough benefit to justify this complexity

Palettes
Statistics
Numerical conversion helpers
Data generation
A few misc interfaces (IPalette and IOHLC)

Plan

Copy these files as needed to get SP4 and SP5 to build independently
Tactically import helpful classes from SP5 into SP4 by editing the csproj file

Generate.cs
Maybe nothing else?

Separate the CI workflows (they'll run faster now)

Related

#2408
#2096
#2028
#2027
#2018
#2009
#1967
#1963
#1956

The shared code is such a sneaky source of bugs!
Working in SP5 would produce a clean build but crash SP4, and vise-versa.
The main branch build was broken for about an hour today because of shared code file issues related to colors and data generators.
I'm happy that as of 150cd5a9b42d91d9d99d7db34b1ae3ae9212dbb5 the SP4 and SP5 code bases are now entirely separate.
"But what about code duplication?" -- It's an issue, but no where near as big of an issue as maintaining shared code was.
If we add colors to SP5 or special functions to the SP5 data generator classes that we really want in SP4, we can manually copy the files over. I think that's a small/rare/unlikely "if" though, and keeping the code bases separate optimizes for stable builds avoids unpredictable errors.


Error loading libfuse on Ubuntu runners

Hi,
Not sure if it's a problem with this action or with the github runner, but suddenly I am unable to use the action due to a dependency on FUSE, which actually looks like a dependency of AppImage (?).

Any idea about what is happening? Did github changed anything in their Ubuntu runners?

My best, thanks ;)

Hi @marcosvafilho sorry for the late response. The problem is that Github updated the default ubuntu-latest runner to 22.04 which no longer ships libefuse2 by default which you've correctly noted is an AppImage problem.
Until AppImage is updated to either support a static fuse2 or switch to fuse3 (there are various issues linked on the https://github.com/AppImage/AppImageKit/issues project) you have a couple of options (in no particular order):

Switch your workflows to use ubuntu-20.04
Manually install libfuse2 when running on ubuntu-latest or ubuntu-22.04, e.g.:- uses: mfinelli/setup-imagemagick@v2
- run: sudo apt-get install -y libfuse2
- run: magick....

I'll update the readme to mention this in the next few days


fix(material): style for selected vertical tabs

Thanks for the PR! We missed that style. Indeed it looks better that way.
Side note: Material design spec does not recommend having vertical tabs at all.
But they are implemented by libraries, like Material UI and Vuetify, so I think we should do it.


[BUGFIX] bug in contact-import and reverted tests

this bugfix is fixing issue with importing contacts by id.
reverted tests to old version (which was actually right)

ack


Bundle integration for Atlas

As mentioned in the review of #54, this adds Atlas and its dependencies as optional projects to the bundle.
By default, they are not being built, which should allow to pick up an existing Atlas installation as usual. --with-atlas will enable building eckit, fckit and atlas. By default, tests are disabled for these dependencies but can be enabled using --with-dependency-tests.
I have also enabled Github actions integration for the new variant.

Merge pending ctest PR #56
Then @sbrdar should rebase PR #54
Then this PR should be rebased.


add decodeBase64 configuration

Description
Allow Azure Blob Storage binding to decode base64 file content.
Issue reference
dapr/components-contrib#388

@mchmarny @msfussell @orizohar please review this doc pr.


adding user_undel and adding parameters to user_add

In client.py I'm adding user_undel to allow restoring of preserved users.
I am also adding the gid_number and the uid_number parameters to user_add so that the uidNumber and gidNumber can be set upon user creation

@hzgraham These changes looks sane to me. Could you please however fix linter issues so that I can merge this PR?

@AmbientLighter thanks, I pushed a fix for that issue. It seems to have passed, but let me know if there is anything else.

@AmbientLighter I wasn't exactly sure what you wanted me to remove, but I removed the "skip_errors" parameter from the doc string. Let me know if there is anything else.


Add As the world Caves in remix by Sarah Cothran

Song Name
As the world Caves in remix
Artist Name
Sarah Cothran
Source
Youtube
Youtube Link
5zhz9Qcnta5JbxPp
Direct File Link
No response
Song ID
977012
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 108242126

reject
invalid YouTube link


Ros2 communication problems

Would you please tell me the ros2 esp32 you are using the cable or wifi communications

Thanks for you question!
It's cable communications，use eps-usart

Thank you very much for your answer!
I now have been entangled with the time to use wireless communication to subscribe to the ros2 news
If you have any idea, please communicate with me
Thank you very much


I can't install using MSIX file, it says "Contact your app dev"

I tried all other ways to install after MSIX failed, and I can't even use the app.

Are you sure you have installed the certificate?

Hey, the certificate expired 3 days ago ^^

Updated the certificate, you can install now.


[Add] Animated Fluent Emojis

Site URL
https://animated-fluent-emoji.vercel.app/
Description of the above provided site
Super cool emojis which are animated and great to use for Readme files or Websites.
Here's the example of emojis.

Category
Frontend
Sub Category
"icons", "emojis"
Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)
[X] I am a GSSoC'23 contributor

can you please assign this issue to me ?

can you please assign this issue to me?

Done @dhruvdabhi101! :)


Sending request with non-string value to kafka-enabled endpoint terminates RIG

Using RIG 2.2.1, when a client sends a request to a RIG Proxy API-Endpoint that should create a Kafka event AND when that request contains a non-string value by either accident or deliberately sent by some bad user, RIG crashes.
Possible solution: when receiving requests on a Kafka-enabled target endpoint, RIG should check that all values are strings and if there is a non-string value,  either reject the request or silently toString() all values before forwarding to Kafka.
Example Proxy-Config to accept request to create a Kafka record:
[{
  "id": "kafka",
  "version_data": {
    "default": {
      "endpoints": [{
        "id": "produce",
        "method": "POST",
        "path": "/",
        "target": "kafka"
      }]
    }
  }
}]

Event including a field containing a number (statusCode)
{
  "event": {
    "statusCode": 200,
    "id": "069711bf-3946-4661-984f-c667657b8d85",
    "type": "com.example",
    "time": "2018-04-05T17:31:00Z",
    "specversion": "0.2",
    "source": "/cli",
    "contenttype": "application/json",
    "data": "{\"item\":2,\"other\":\"twelve\"}"
  },
  "partition": "0"
}

Sending the event to the configured Proxy Endpoint will immediately crash RIG. If putting the number in quotes, it works fine.
08:08:58.111 [info]  POST /
request_id=FdZb4CkQCpiJl0QAAADS application=plug module=Plug.Logger function=call/2 file=lib/plug/logger.ex line=27 pid=<0.3312.0>
:gen_event handler Logger.Backends.Console installed in Logger terminating
** (exit) an exception was raised:
    ** (Protocol.UndefinedError) protocol String.Chars not implemented for {%ArgumentError{message: "argument error"}, [{:erlang, :iolist_to_binary, [200], []}, {:brod_utils, :bin, 1, [file: 'src/brod_utils.erl', line: 648]}, {:brod_utils, :"-unify_msg/1-fun-0-", 1, [file: 'src/brod_utils.erl', line: 657]}, {:lists, :map, 2, [file: 'lists.erl', line: 1239]}, {:lists, :map, 2, [file: 'lists.erl', line: 1239]}, {:brod_utils, :unify_msg, 1, [file: 'src/brod_utils.erl', line: 657]}, {:brod_utils, :make_batch_input, 2, [file: 'src/brod_utils.erl', line: 435]}, {:brod_producer, :produce_cb, 4, [file: 'src/brod_producer.erl', line: 199]}]}. This protocol is implemented for: Atom, BitString, Date, DateTime, Float, Integer, List, NaiveDateTime, Rig.Subscription, Time, URI, Version, Version.Requirement
        (elixir) /usr/local/src/elixir/lib/elixir/lib/string/chars.ex:3: String.Chars.impl_for!/1
        (elixir) /usr/local/src/elixir/lib/elixir/lib/string/chars.ex:22: String.Chars.to_string/1
        (logger) lib/logger/formatter.ex:166: anonymous fn/1 in Logger.Formatter.output/5
        (elixir) lib/enum.ex:1314: Enum."-map/2-lists^map/1-0-"/2
        (elixir) lib/enum.ex:1314: Enum."-map/2-lists^map/1-0-"/2
        (logger) lib/logger/formatter.ex:152: anonymous fn/6 in Logger.Formatter.format/5
        (elixir) lib/enum.ex:1925: Enum."-reduce/3-lists^foldl/2-0-"/3
        (logger) lib/logger/formatter.ex:151: Logger.Formatter.format/5

08:08:58.154 [info]  Application rig_kafka exited: shutdown
module=application_controller function=info_exited/3 file=application_controller.erl line=1941 pid=<0.1744.0>
{"Kernel pid terminated",application_controller,"{application_terminated,rig_kafka,shutdown}"}
Kernel pid terminated (application_controller) ({application_terminated,rig_kafka,shutdown})

Crash dump is being written to: erl_crash.dump...done

@actraiser thanks for the report, I'll check!

@actraiser tried both 2.1.1 and latest and was able to send the mentioned event successfully. Would it be possible for you to create some small example where it is reproduced?
What I've used:
[
  {
    "id": "my-service",
    "version_data": {
      "default": {
        "endpoints": [
          {
            "id": "my-endpoint",
            "method": "POST",
            "path": "/",
            "target": "kafka"
          }
        ]
      }
    },
    "proxy": {
      "use_env": true,
      "target_url": "API_HOST",
      "port": 3000
    }
  }
]

docker run --name rig -d \
-e PROXY_CONFIG_FILE="$config" \
-e KAFKA_BROKERS="kafka:9092" \
-e PROXY_KAFKA_REQUEST_TOPIC="req-topic" \
-e LOG_LEVEL=debug \
-p 4000:4000 \
--network kafka_default \
accenture/reactive-interaction-gateway:2.2.1

Thanks for checking! I will review my code after the weekend again and let you know how I triggered those crashes.
I actually believe that I made a wrong assumption of the underlying problem in the first place and that those crashes rather happen when using the Kafka-Endpoint in conjunction with an AVRO-Schema.
When encoding fails (because the sent data does not match the schema fetched from the schema-registry), rig seems to crash - but  I will go recheck and and provide a reproducable case.
Greets
-act

Sorry for the delay - I had to reinstall my environment (thank you Apple) but ever since, I can not reproduce the problem any more. Please consider the issue closed.

Ok, no problem. Feel free to re-open or create new issue if you tackle it again.


Support new use case: bootstrap of new streaming query with initial state

See below: actual end user asked the use case on Spark user mailing list.
<EMAIL_ADDRESS>Initial state can be either specific values or calculate from batch data.
The issue is, structured streaming doesn't have a concept of "initial state" - at least in HDFSBackedStateStore. If the version is 0, it doesn't load the state.
So either we should propose a change to Spark to allow initial state for version 0, or create a new checkpoint which has "batch 0" with empty offset/commit but initial state.
We could try latter (less coupled with upstream) and propose a change on upstream if it doesn't work.

I can't think about the approach of creating offsets/commits for empty batch - offset implementations are vary for data sources. Only one I can think it's possible is let state implementation allows version 0 of state as initial state. Unlike other stuff, it would only work if the state implementation supports version 0, as "supporting version" is not defined in the interface.


editOnInput: wrong lastChangeType

https://github.com/facebook/draft-js/blob/master/src/component/handlers/edit/editOnInput.js#L128-L137
'spellcheck-change' should be used if there's no entity under the cursor.

We may need a new change type for this case to avoid confusion.


Introduce Thunder

I went for thor and thunder but we could very well change it to lib cli or thor-lib thor-cli....

Merging now as it is hard to rebase. Taking action on the comment above in new PR.


ValidationError: Trying to set Table.Column of type 'number' to '10' of type 'string' at Function.fromWrongPropertyType

Describe the bug
With strict: true option provided, ORM is throwing a validation error on read even though I am not setting anything.
Stack trace
ValidationError: Trying to set Stock.quantity of type 'number' to '10' of type 'string'
    at Function.fromWrongPropertyType (/path/to/my_project/node_modules/mikro-orm/dist/utils/ValidationError.js:22:16)
    at EntityValidator.validateProperty (/path/to/my_project/node_modules/mikro-orm/dist/entity/EntityValidator.js:36:43)
    at /path/to/my_project/node_modules/mikro-orm/dist/entity/EntityAssigner.js:31:49
    at Array.forEach (<anonymous>)
    at Function.assign (/path/to/my_project/node_modules/mikro-orm/dist/entity/EntityAssigner.js:16:27)
    at EntityManager.merge (/path/to/my_project/node_modules/mikro-orm/dist/EntityManager.js:219:33)
    at EntityManager.find (/path/to/my_project/node_modules/mikro-orm/dist/EntityManager.js:81:33) {
  entity: undefined,
  name: 'ValidationError'
}

To Reproduce
Steps to reproduce the behavior:

Create an entity with TS type number, DB type numeric in Postgres 12.
Insert some example rows into DB
Run orm.em.find(models.SomeEntity, {});

Expected behavior
No ValidationError should be triggered on read if DB schema matches TS entity (it does in my case).
Additional context
My DB schema was in place before using Mikro-ORM and the DDL was not generated by Mikro-ORM. Still, the column type is numeric and the entity field type is number, so there should be no issue with this column value being read in.
For now, I am avoiding this by turning off strict: true, which suppresses the error.
Versions

Dependency
Version

node
12.14.1

typescript
3.7.5

mikro-orm
3.1.0

your-driver
postgresql

Create an entity with TS type number, DB type numeric in Postgres 12.

The numeric type here will be the problem I guess. PG driver is returning that as a string by default but we can easily override that.

Without knowing the details of the PG driver, one reason it could be returning it as a string is because the numeric type handles much greater precision (up to ~150,000 digits of precision) than JavaScript's number. I think your fix will work well for 95% of use cases, though it will cause issues for the edge case of very large or high precision numbers.
I can come back to help tackle this as we run into large numbers in our system (which may very possibly happen).

Also I forgot to say -- thank you for the quick fix!

Yes its exactly because of that. I was thinking about making this change optional, but it should be possible to override it back to the original behaviour with the very same way (just after the ORM init happens) so I think its fine like this:
import { types } from 'pg';

const orm = await MikroORM.init();
types.setTypeParser(1700, str => str);

Fwiw I had a similar issue in a "generated TS clients for protobuf" project where protobuf supports longs / 64 bit numbers that are bigger than number.
At first I didn't want to support "anything not a number", so did something somewhat similar to @B4nan's fix for this issue, however, I was able to do a bounds check on the protobuf number and throw a runtime error if it was higher/lower than the JS number could represent.
It would be ideal to do something like this b/c right now Mikro will silently throw away precision/something that we should assume users care about (why else are they using numeric?) without telling them, and so potentially creating silent bugs.
Granted, I'm not sure if that's easily possible to do bounds checking given the PG driver just gives you a string.
For my protobuf project, eventually someone came along who really wanted/needed the "bigger than number" support, and they added a built-time / config flag that changed to use the long npm library instead of numbers for any field marked as 64-bit in protobuf (or say numeric here in pg).
That enabled the "best" of both worlds: people would really wanted numbers could still use them, but they'd get runtime errors if the client was going to actively loose/mis-represent data, and people that knew they needed the bigger-than-number support could opt-in to the less-idiomatic-but-it-works Long types.
https://www.npmjs.com/package/long
Disclaimer I've not looked into the pg numeric type at all, but just relating what worked well on another project for a similar-ish scenario.

At first I didn't want to support "anything not a number", so did something somewhat similar to @B4nan's fix for this issue, however, I was able to do a bounds check on the protobuf number and throw a runtime error if it was higher/lower than the JS number could represent.

Well that is kinda the way how it works now (without this fix), without strict mode enabled, string numbers will get automatically converted to number and then they are converted back to string, checking that the original string is equal.
As said earlier, pg's numeric type is like a big float:

up to 131072 digits before the decimal point; up to 16383 digits after the decimal point

@B4nan  Just discovered something and would appreciate your guidance on good next steps.
If you re-read the thread above, we chose to coerce Postgres’ numeric type (which can have arbitrary precision and is returned by pg as a string) into a number using parseFloat. We spoke about the fact that we could use a custom type to avoid this coercion if we wanted to retain the arbitrary precision in our JS code. I am now attempting to do exactly that, to be able to handle floats larger than the limits of JS’ number type.
This is the custom type I put together for that:
import Decimal from 'decimal.js';
import { Platform, Type } from 'mikro-orm';

export class DecimalType extends Type {

  convertToDatabaseValue(value: Decimal, _platform: Platform): string {
    return value.toFixed();
  }

  convertToJSValue(value: any, _platform: Platform): any {
    return new Decimal(value);
  }
}

This works great for JS -----> DB insertion.
However, for JS <---- DB retrieval, it does not work, because the previous fix causes the value argument of convertToJSValue to already be coerced into a number prior to the call to the custom type’s convertToJSValue.
You can test and confirm this in Node 12 by trying to retrieve a Postgres numeric of value<PHONE_NUMBER>740991001 — this is coerced to be<PHONE_NUMBER>740991000 (a different number) by the time it arrives at convertToJSValue.
I am going to make a change to my project’s version of MikroORM to patch this temporarily for the purposes of my project, but would love to understand how we can fix this more permanently within the official MikroORM project. Thanks!

@B4nan  Just discovered something and would appreciate your guidance on good next steps.
If you re-read the thread above, we chose to coerce Postgres’ numeric type (which can have arbitrary precision and is returned by pg as a string) into a number using parseFloat. We spoke about the fact that we could use a custom type to avoid this coercion if we wanted to retain the arbitrary precision in our JS code. I am now attempting to do exactly that, to be able to handle floats larger than the limits of JS’ number type.
This is the custom type I put together for that:
import Decimal from 'decimal.js';
import { Platform, Type } from 'mikro-orm';

export class DecimalType extends Type {

  convertToDatabaseValue(value: Decimal, _platform: Platform): string {
    return value.toFixed();
  }

  convertToJSValue(value: any, _platform: Platform): any {
    return new Decimal(value);
  }
}

This works great for JS -----> DB insertion.
However, for JS <---- DB retrieval, it does not work, because the previous fix causes the value argument of convertToJSValue to already be coerced into a number prior to the call to the custom type’s convertToJSValue.
You can test and confirm this in Node 12 by trying to retrieve a Postgres numeric of value<PHONE_NUMBER>740991001 — this is coerced to be<PHONE_NUMBER>740991000 (a different number) by the time it arrives at convertToJSValue.
I am going to make a change to my project’s version of MikroORM to patch this temporarily for the purposes of my project, but would love to understand how we can fix this more permanently within the official MikroORM project. Thanks!

@B4nan  Just discovered something and would appreciate your guidance on good next steps.
If you re-read the thread above, we chose to coerce Postgres’ numeric type (which can have arbitrary precision and is returned by pg as a string) into a number using parseFloat. We spoke about the fact that we could use a custom type to avoid this coercion if we wanted to retain the arbitrary precision in our JS code. I am now attempting to do exactly that, to be able to handle floats larger than the limits of JS’ number type.
This is the custom type I put together for that:
import Decimal from 'decimal.js';
import { Platform, Type } from 'mikro-orm';

export class DecimalType extends Type {

  convertToDatabaseValue(value: Decimal, _platform: Platform): string {
    return value.toFixed();
  }

  convertToJSValue(value: any, _platform: Platform): any {
    return new Decimal(value);
  }
}

This works great for JS -----> DB insertion.
However, for JS <---- DB retrieval, it does not work, because the previous fix causes the value argument of convertToJSValue to already be coerced into a number prior to the call to the custom type’s convertToJSValue.
You can test and confirm this in Node 12 by trying to retrieve a Postgres numeric of value<PHONE_NUMBER>740991001 — this is coerced to be<PHONE_NUMBER>740991000 (a different number) by the time it arrives at convertToJSValue.
I am going to make a change to my project’s version of MikroORM to patch this temporarily for the purposes of my project, but would love to understand how we can fix this more permanently within the official MikroORM project. Thanks!

I thought you can override this easily, right after you initialize the ORM, you can do:
import { types } from 'pg';
types.setTypeParser(1700, str => str);

^ Can confirm this works when called after MikroORM.init, thanks @B4nan. Avoids needing to modify my project's copy of MikroORM, which is great.
Do you want library users to be doing this sort of thing, of reaching into the underlying DB drivers? I know you have driverOptions available, but this is somewhat different to that.

Yeah converting the value to number was a bad idea, let's drop that conversion in v4 and let users do what they want with the string (probably via predefined custom type).

I think an alternative would be to expose this as a config in MikroORM.init, so that users can turn it off via an explicit configuration rather than by reaching into the driver. Could it somehow be included in driverOptions or somewhere similar?

Actually on second thought, I agree with you, let's keep it an explicit decision by the user. If they use an int4, pg returns this as a number as expected. It's only larger Postgres number types such as int8 and numeric that are returned as string, and it's good for the caller to explicitly consider the implications of this for their software.
Any reason we want to keep this type conversion in v3 versus removing it? It wasn't present in earlier versions of v3 so it's not exactly a breaking change.

Well in fact the change I made here before was a breaking change too, now I will be basically reverting it :D The change was part of 3.1.1, I feel like reverting it in late 3.6 version is not that good idea (although small BC, still a BC, and doing it at the end of life of v3 feels wrong).
Generally I think I am starting to take more things into the BC category gradually as the user base is growing. And v4 will come quite soon, so not that big deal hopefully, especially because there is that workaround mentioned above.

Sounds good @B4nan, that makes sense. I'm not personally bothered at all since I have the workaround. Thanks for thoughts here and hopefully something that can be resolved properly in v4.

Closing as fixed in dev branch, subscribe to #527 for updates.


Updates to deprecate OnlineVoiceRoutingPolicy parameter.

Docs Build status updates of commit 4803c36:
:clock10: Full build: the average full build time is 13 min(s) 51 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 4803c36:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Set-CsTenantNetworkSite.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 0651c2b:
:clock10: Full build: the average full build time is 14 min(s) 5 sec(s), this is based on the last 1000 build(s) for this repository.

Docs Build status updates of commit 0651c2b:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

skype/skype-ps/skype/Set-CsTenantNetworkSite.md
:white_check_mark:Succeeded
View (skype-ps)

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add test coverage reporting using codecov

Codecov Report

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

@@            Coverage Diff            @@
##             master      #45   +/-   ##
=========================================
  Coverage          ?   70.32%
=========================================
  Files             ?       11
  Lines             ?      364
  Branches          ?       48
=========================================
  Hits              ?      256
  Misses            ?       92
  Partials          ?       16


fix: POST requests time out

On the deployed ELIXIR Cloud Registry, requests to create resources time out with no response, but resources are actually created. There are no error messages in the log.
An example log entry looks like this:
[2023-11-01 16:32:15,293: INFO    ] Added service with id 'G1YII2'. [cloud_registry.ga4gh.registry.service]
[2023-11-01 16:32:15,295: INFO    ] Response to request "POST /ga4gh/registry/v1/services HTTP/1.1" from <IP_ADDRESS>: G1YII2 [foca.utils.logging]

So it appears that a response was sent. However, the client never received it.

Update: Also the DELETE /services/{service_id} ID times out and makes the service unresponsive, i.e., afterwards also the GET endpoints do not work any longer. Again, no log messages.

According to the logs, the responses are immediately generated after they are received, but receiving the request takes up to minutes. So this hints at some problems in the Nginx configuration.


Create ucvvirtual.txt

@EduardoPazamaya As far as I understand, this is online school, which does not have offline classroom courses. Regrettably, JetBrains does not offer free licenses to online schools. I am closing this request without merging. Please reopen and comment if I am mistaken. Thank you for the understanding.

@philipto The "Universidad César Vallejo" (UCV) school has two domains, "ucv.edu.pe" to administrative employees, and "ucvvirtual.edu.pe" to students. UCV is not an online school despite it has the "virtual" word in the domain name.


Execution failed for task ':app:compileFlutterBuildRelease'.

Duplicate Check

[X] I have searched the opened issues and there are no duplicates

Describe the bug
i want to help
Code sample
Code
[../../../../AppData/Local/Pub/Cache/hosted/pub.dev/flet-0.25.1/lib/src/utils/theme.dart:356:28: Error: A value of type 'TabBarThemeData' can't be returned from a function with return type 'TabBarTheme?'.
        - 'TabBarThemeData' is from 'package:flutter/src/material/tab_bar_theme.dart' ('/C:/Program%20Files/flutter/packages/flutter/lib/src/material/tab_bar_theme.dart').
        - 'TabBarTheme' is from 'package:flutter/src/material/tab_bar_theme.dart' ('/C:/Program%20Files/flutter/packages/flutter/lib/src/material/tab_bar_theme.dart').
         return theme.tabBarTheme.copyWith(
                                  ^
       ../../../../AppData/Local/Pub/Cache/hosted/pub.dev/flet-0.25.1/lib/src/utils/theme.dart:438:28: Error: A value of type 'DialogThemeData' can't be returned from a function with return type 'DialogTheme?'.
        - 'DialogThemeData' is from 'package:flutter/src/material/dialog_theme.dart' ('/C:/Program%20Files/flutter/packages/flutter/lib/src/material/dialog_theme.dart').
        - 'DialogTheme' is from 'package:flutter/src/material/dialog_theme.dart' ('/C:/Program%20Files/flutter/packages/flutter/lib/src/material/dialog_theme.dart').
         return theme.dialogTheme.copyWith(
                                  ^
       ../../../../AppData/Local/Pub/Cache/hosted/pub.dev/flet-0.25.1/lib/src/utils/theme.dart:482:26: Error: A value of type 'CardThemeData' can't be returned from a function with return type 'CardTheme?'.
        - 'CardThemeData' is from 'package:flutter/src/material/card_theme.dart' ('/C:/Program%20Files/flutter/packages/flutter/lib/src/material/card_theme.dart').
        - 'CardTheme' is from 'package:flutter/src/material/card_theme.dart' ('/C:/Program%20Files/flutter/packages/flutter/lib/src/material/card_theme.dart').
         return theme.cardTheme.copyWith(
                                ^
       Target kernel_snapshot_program failed: Exception

       FAILURE: Build failed with an exception.

       * What went wrong:
       Execution failed for task ':app:compileFlutterBuildRelease'.
       > Process 'command 'C:\Program Files\flutter\bin\flutter.bat'' finished with non-zero exit value 1

       * Try:
       > Run with --stacktrace option to get the stack trace.
       > Run with --info or --debug option to get more log output.
       > Run with --scan to get full insights.
       > Get more help at https://help.gradle.org.

       BUILD FAILED in 5m 25s
       Gradle task assembleRelease failed with exit code 1

       Error building Flet app - see the log of failed command above.
![395726983-83d4d7ea-5afd-4a6a-81a1-ca1eb10f125c](https://github.com/user-attachments/assets/7a956945-9a3b-4112-b237-aff683a3c29d)
]

![395726983-83d4d7ea-5afd-4a6a-81a1-ca1eb10f125c](https://github.com/user-attachments/assets/0c666510-60dd-4f57-8e5c-2ba250a59d5d)
To reproduce
Execution failed for task ':app:compileFlutterBuildRelease'.
> Process 'command 'C:\Program Files\flutter\bin\flutter.bat'' finished with non-zero exit value 1
Expected behavior
No response
Screenshots / Videos

Captures
[Upload media here]

Operating System
Windows
Operating system details
windows 11
Flet version
flet 0.25.2
Regression
No, it isn't
Suggestions
No response
Logs
Logs
[Paste your logs here]

Additional details
No response

Duplicate of https://github.com/flet-dev/flet/issues/4548


dotenv fails to load .env file that include backslashes

A=\.
B="\."
C='\.'

Only C works. If A or B are present, the file fails to load.
Since dotenv.ok() silence the error, it's quite hard to detect. I guess it should not fail if the file is absent, so we could .expect() it and be more confident

This was confusing - I agree that using .expect() by default is a good idea, it would have drawn me the reason why I had access to every env variable except the one with a backslash in it, which was really confusing!


autocmd ColorScheme * MyCmd  NOT work

vim in windows 10, vversion 8.1, patch 1-497
Add autocmd ColorScheme * echo("color scheme test") in _vimrc file for test purpose.
Use ":colorscheme mycolor", color scheme changed ,but autocmd not invocked

Please try using :echomsg and then check using the :messages command for the output. I think the colorscheme caused a redraw that causes your :echo to be cleared.

Indeed, the redraw or clearing the echo happens some short time after the ColorScheme autocmd is fired.  You can also try autocmd ColorScheme * echo 'test' | sleep 500m which will show the echo briefly.

Find the cause of the problem, I change color scheme for difference file type, so i add "autocmd  BufWinEnter * call ChangeScheme()",  and "autocmd ColorScheme * echo("color scheme test")" for testing. but colorscheme in ChangeScheme() function not invoke another autocmd(autocmd ColorScheme * echo("color scheme test"))，so plugin like indentLine not work Incorrect.
Add "nested" fix this:   autocmd BufWinEnter * nested call SetScheme()
Thanks
@chrisbra @andymass


Add unit test code coverage for *.ipynb files with nbdev

https://github.com/Origami-TinyML/tflm_hello_world/blob/main/README.md#sprint-1
Some discussion about how to introduce unit test coverage to show with nbdev, https://forums.fast.ai/t/nbdev-code-coverage-n-tests/73993

https://pete88b.github.io/decision_tree/test_coverage/

Borna's original:

I have recently been exploring the use of nbdev and Jupyter Notebooks. I attempted to add nbdev to my repository, but I am unsure if I have done so correctly.
My fork of the repository can be found here: https://github.com/lifeofborna/tflm_hello_world.
I am now wondering which specific .ipynb files are meant by "Add unit test code coverage for *.ipynb files." I would greatly appreciate any tips or suggestions on how to properly add nbdev to my repository, and how to proceed with adding code coverage for the .ipynb files. Thank you.

https://github.com/Origami-TinyML/tflm_hello_world/issues/35#issue-1556546028

You may want to use a jupyter notebook in hello world training section, for example.

I tried to implement this :
https://pete88b.github.io/decision_tree/test_coverage/
I made a testcoverage.py script that's said in the article in the root of the repository.
I created a virtual environment where I pip-installed nbdev.
But when I try to run this testcoverage.py script it crashes because it doesn't recognize some of the functions such as the guy mentions in his article "split_flags_and_code". Even though I have the correct imports and they're recognized.
He says I should use the newest version of nbdev which I'm quite sure  I'm using (nbdev==2.3.9) since I freshly pip-installed it in the virtual environment. https://pypi.org/project/nbdev/
I'm not sure how to proceed from here. I still have time tomorrow to continue on this for this week but sadly most of my time today went to setting up the nbdev repo (task 1) since my task depended on it.

I'm not sure how to proceed from here. I still have time tomorrow to continue on this for this week but sadly most of my time today went to setting up the nbdev repo (task 1) since my task depended on it.

@lifeofborna you could share what you've done with @nellatuulikki so that she won't need to do the same thing again.
Please send a PR for @nellatuulikki 's issue, at first. @nellatuulikki could verify if this works or not.
To solve your problem, you may want to do the following:

Create a new repo which has minimal files only to verify unit test under nbdev.
Reproduce the problem in this minimal setup
Send a PR to describe what's wrong there
Ask for some help in the discord with this issue link.

nbdev2 has been released so that pip install should install version 2+ nowadays.
@lifeofborna & @nellatuulikki could work on this together tomorrow.

Probably you could use this tutorial example with adding unit test to reproduce the problem you have.
https://nbdev.fast.ai/tutorials/tutorial.html

Moved this from WIP, since we need to initialize nbdev to the repo first correctly.

Me and @lifeofborna tried to implement unit testing following https://pete88b.github.io/decision_tree/test_coverage/ instructions to https://github.com/fastai/nbdev-hello-world  but we still didn't manage to import split_flags_and_code from nbdev.
We looked into the source code and we didn't find any usage of split_flags_and_code in nbdev.export. However, we looked into the older version of nbdev  (https://github.com/fastai/nbdev1/blob/master/nbdev/export.py) and noticed that here they have function called split_flags_and_code. However, this export.py module has been completely rewritten in new nbdev (current version of export module https://github.com/fastai/nbdev/blob/master/nbdev/export.py).
So question is should keep trying to implement unit test code coverage report based on pete88b's instructions and find out if we could use other functions instead of split_flags_and_code by creating an issue to nbdev or should we try to think other way to implement coverage report.

Hi @nellatuulikki
split_flags_and_code  seems to do "Splits the source of a cell into 2 parts and returns (flags, code)". This looks just spliting 'code' and '# comment'???
Can we just copy this function in pete88b's code and make it work?
Under nbdev, unit test is just a code to run in the jupyter notebook. I'm not sure if there's other python unit test coverage could be easily applied or not. Probably just trying to run the above could be the easiest? The missing function itself doesn't seem to be too big or complicated either?

And I asked :)
https://forums.fast.ai/t/nbdev-code-coverage-n-tests/73993/26

Thanks for a quick answer. We can try to add the function from nbdev1 to pete88's code.

I'll also join this debugging tomorrow ;)

Me and @lifeofborna had a look into this https://github.com/pete88b/decision_tree/blob/master/test_nbs.py code and it seem to require a lot of rewriting of the code. We don't know how long or even if it's possible to use that code any longer.
We are exploring alternative ways to implement unit test code coverage. We found this youtube video (https://www.youtube.com/watch?v=Pf1ADyUKOrE) and we were thinking should we try to implement testing following instructions on this video.

And a follow-up question. What is the purpose of testing the Jupyter notebooks? Are we only checking if they run without crashing or are we conducting more specific unit tests?
I found that a library called nbval would be the best fitting way to conduct these tests: https://nbval.readthedocs.io/en/latest/
which utilized pytest. We should be able to add coverage for it as well.

No, the point is not to test jupyter notebook itself. As you've seen the tutorial of nbdev, nbdev prompts developers to write code, (unit)tests & doc at once. What's missing in nbdev is the coverage report of its (unit)tests. The core code is converted into a package by nbdev_export. But the test part isn't in a package. nbdev_test runs all notebook. So tests in it are run as well. But no coverage report. That's the missing one if we execute modern SW development method like SCRUM / Agile, usually they recommends unit test coverage as one of their KPI. At worst, we could skip and proceed if effort is too much :(
I'll take a look at the above video from @nellatuulikki
I didn't get the point of nbval yet.

Here's something similar which I expected above, https://forums.fast.ai/t/nbdev-code-coverage-n-tests/73993/27
IIUC, nbprocess is the previous name of nbdev2 so that https://github.com/muellerzr/nbprocess-sandbox code is still 7 months old althoug you still need to update for nbdev(2).

It looks like if you can put the removed preprocs into somewhere in the latest proc.begin the rest should work?

Update*
We managed to get muellers sandbox code to run on NBDEV2. Currently, it creates a new folder where it extracts the .ipynb files to python test file.  You can then run !pytest/foldername and it will run tests.
We will next implement it to our main GitHub repository.

Congrats! @lifeofborna & @nellatuulikki
It was one of big barrier for setting up our CI / CD.

One additional question do we wish to support unittest as well currently we have the pytest working. You can see the unit test in the below link at "Using unit test" section @doyu

Either one of them would be OK.
I have no idea which is better, pytest vs unit test.
The outcome coverage data would be fed into codecov as well as C++ coverage @ArttuLe did.
Probably Pytest is beter? You can quicly google it :D

We will next implement it to our main GitHub repository.

Cool!


Fixed missing frames when resizing component

Before this PR we were setting scene.isDirty after resizing and Renderer would render it in the next animation frame. That means that we would have to wait up to 16ms until a new frame was rendered so we ended up with a white frame after resizing.
https://model-viewer-tests.glitch.me/resize.html
https://model-viewer-tests.glitch.me/resize-horizontal.html
https://model-viewer-tests.glitch.me/resize-vertical.html
After this PR we are rendering a frame right when the component gets resized.
https://model-viewer-tests.glitch.me/resize-fixed.html
https://model-viewer-tests.glitch.me/resize-horizontal-fixed.html
https://model-viewer-tests.glitch.me/resize-vertical-fixed.html
This PR is built on top of #552.

Thanks for figuring this one out! This is definitely something that I would like to see at least one test for, as it seems ripe for regressions in future changes.

Agreed. I do not know how to create tests though and currently I do not have the time nor energy to learn.

Closing this in favor of #619.


Emergency banner doesn't handle commas in Rake arguments

Discovered during the Emergency Publishing Drill:
https://trello.com/c/iniJbqCt/321-emergency-publishing-drill-monday-7-august
https://github.com/alphagov/static/blob/master/lib/tasks/emergency_banner.rake
The link text "Norwegian journalist and statesman, dies at 71" rendered as "Norwegian journalist and statesman".
Rake arguments need to escape commas as commas are used to split arguments:
https://github.com/ruby/rake/issues/21
When typing these fields into Jenkins it's not obvious that commas must be escaped.

Can the Jenkins job quote the arguments?

We're already quoting arguments:
emergency_banner:deploy[\"$CAMPAIGN_CLASS\",\"$HEADING\",\"$SHORT_DESCRIPTION\",\"$LINK\",\"$LINK_TEXT\"]"

Adding comments to Jenkins fields:
https://github.com/alphagov/govuk-puppet/pull/6242

Looks to be fixed


Not working on windows with mingw.

Undefined posix_memalign, maybe use _aligned_malloc instead?

Ahhh, Windows...
I took a look at https://github.com/rust-lang/rust/blob/master/library/std/src/sys/windows/alloc.rs, and it seems like the "right" approach is to use HeapAlloc directly.
Alternatively, a quick fix could be to use #[cfg(windows)] and #[not(cfg(windows))] to switch between _aligned_malloc posix_memalign and respectively.
Unfortunately, I don't have a Windows development environment set up, so this isn't something I can easily test, so I'm not sure when i'll get around to fixing this issue. That said, if you're up for it, it shouldn't be too difficult to make a PR that adds windows support (using either of the two methods listed above), and it would certainly be much appreciated :)

The way to fix this problem is simple, but is it ok to use native Windows system call? After all, this crate is call libc_alloc, not native_alloc. malloc suits the name "libc" better.

Funny enough, I had originally considered calling this library malloc_alloc, as I did in-fact initually use malloc directly (until realizing that it didn't fulfill Rust's alignment guarantees). When I switched over to posix_memalign, I decided to go up an abstraction layer and call the crate libc_alloc, though in hindsignt, maybe I should have gone even more generic and called the crate system_alloc right off the bat. Ah well ¯_(ツ)_/¯
As such, I'm not too squeamish about using non portable libc functions (case in point - posix_memalign).


Whether the runv project will be maintained

I know that hyper.sh and clearcontainer are combined, so I wanna to know whether this project will continue to be maintained or will use a new project

It will be the kata runtime after yet another refactor and take kata-agent which is originated from hyperstart to replace the hyperstart.

What is the benefit or reason to replace hyperstart with kata-agent?


[feature request] Option to change image alt attribute

This would help me a lot;)
Thanks in advance
Leonard Storcks
from https://lernzettel.org/

An easy way to add custom image processing would be great too, i. e. I´d like to upload the images to Google Cloud Storage. You could for instance provide a function that receives the image data and returns a custom source.

Hi leo, thanks for your issue.
Recently I will improve the Image extension,  including your above features.

support update alt attribute
In the current version, you can pass uploadRequest to the Image extension when you instantiate it:

  new Image({
     uploadRequest (file: File): Promise<url> {
       // upload file to Google Cloud Storage

       return Promise.resolve(url); // return a Promise with your file source url
     },
  })

but now there is no uploading feedback, i will optimize it.
If you have any questions about this API, I will resolve it and deploy it with other image improvements.

Thank you very very very much!!!!

update image alt is available in version 1.17.0

thanks 👍👍 element-tiptap will help me switch to an own little CMS solution for https://lernzettel.org/ :)

[bug report] The image options are not visible in full-screen mode.

And I am getting the following warning:
vue.runtime.esm.js?2b0e:619 [Vue warn]: Invalid prop: type check failed for prop "view". Expected EditorView, got Object
found in
--->
 at packages/popover/src/main.vue

 at src/App.vue

Any ideas on this? Thanks :)

[bug report] The image options are not visible in full-screen mode.

Hey, i saw the problem and thought about it.
Using native fullscreen will just make the element fullscreen, but element-ui appends popover, tooltip etc. to the body, they're not in this element, so them not visible in fullscreen mode.
But if do not append them to the body, will also have rendering problems if the editor render in el-dialog.
So here I'm going to set position: fixed to the editor to make it "fullscreen" .
If you have a better suggestion, I would appreciate it very much.

I have simply fixed these two problems and they are available now.
I am still thinking about other better methods.
If there are any other problems, welcome to submit an issue.
Thank you.


Packaging via SetupTools

Should we package the project via SetupTools?
https://packaging.python.org/guides/distributing-packages-using-setuptools/
I'll submit a pull request once you say yes.
And also, couldn't fix the bugs in #23, I might just rewrite the code.

i dont think we should convert this to package yet, it's just few small separate games..


List Provision results in partial field population

After upgrading to 2.19.1710.1 Pnp Core, Provisioned Lists don't look correct.
One example of this is that the New Form contains only the Title field where the list has more fields that should be on the form.
Another is that creating a new item in the list doesn't set the default values.
My example list has a Yes No field with a Default value of Yes but if I create a new item the field is Null.
This happens regardless if the field is created via java script, CSOM or using the New Form with the missing fields.
This same code worked correctly in 2.17
In these cases the list was not created with a content type, the fields were added to the OTB item.
Not sure how else to document this issue,
Let me know if you need more information,
Dave
Images.zip

We closed this issue as it had not activity within last 180 days. This is a generic process we have decided to perform for issues, which have not been explicitly marked still to be "work in progress" based on tags. We are performing this cleaning to make sure that old issues that have already been solved (but not closed) or are no longer relevant are cleaned out and make the issues more manageable. If this issue still valid, we would ask you to open a new issue and follow the guidance in the issue template related on the recommended location. We do apologize any inconveniences this might cause. Please do remember that issues in the issue lists are also messages for others in the community, so you can also check if you can assist on any of them. “Sharing is caring!”


Add Jetpack navigation module

Adding support for the Navigation Graph #305 is a little tricky. The issue you run into is cause we need the navigation controller, and, so this means when restoring from process death, the fragments are restored in the activity#onCreate super before the activity has inflated its content. So, the NavigationController is not always accessible in Fragment#onCreate. When not restoring from process death, the existing ViewModel delegates method of creating the ViewModel like fragmentViewModel works fine.
For now, my current workaround is to remove the lifecycle aware lazy and use Lazy. I think it should be possible to make navigation ViewModel delegate auto-subscribe by delaying the subscription until onStart when the NavigationController is not yet accessible.
I still need to add some CI tests, but, I thought I would open the PR now to get some feedback on my current approach and if it makes sense to have navigation as an add-on module.

Personally, I like this direction. We were definitely having difficulties with restoring after process death, and if this solves that issue, it'd be very welcome.

Any idea on how to work around that?

Any idea on how to work around that?

At the time you asked no. I think I have a working prototype now.
Sorry I Accidentally deleted this PR 😅
The MvRx view needs to be subscribed to the ViewModel and there seems to be two ways to trigger this safely.

try/catch in the lifecyleOwner onCreate, and fallback to onStart if onCreate fails
Access the ViewModel from the Fragment in or after and only after the onActivityCreated, forcing the lazy initializer to fire.

I opted to use both as some fragments might only ever use invalidate.


Firefox Needed?

Thank you for your code. However I got some problems.
When I add -v tag, I can run the script. However, if I only use -q -a , there is error:
('https://class.coursera.org/algs4partII-007/', 'algs4partII-007')
Traceback (most recent call last):
File "dl_all.py", line 282, in
session = webdriver.Firefox()
File "/Users/Doodle/anaconda/lib/python2.7/site-packages/selenium/webdriver/firefox/webdriver.py", line 81, in init
self.binary, timeout)
File "/Users/Doodle/anaconda/lib/python2.7/site-packages/selenium/webdriver/firefox/extension_connection.py", line 51, in init
self.binary.launch_browser(self.profile, timeout=timeout)
File "/Users/Doodle/anaconda/lib/python2.7/site-packages/selenium/webdriver/firefox/firefox_binary.py", line 67, in launch_browser
self._start_from_profile_path(self.profile.path)
File "/Users/Doodle/anaconda/lib/python2.7/site-packages/selenium/webdriver/firefox/firefox_binary.py", line 90, in _start_from_profile_path
env=self._firefox_env)
File "/Users/Doodle/anaconda/lib/python2.7/subprocess.py", line 710, in init
errread, errwrite)
File "/Users/Doodle/anaconda/lib/python2.7/subprocess.py", line 1335, in _execute_child
raise child_exception
Could you help me?

Yes, this script needs Firefox or Phantonjs to imitate browser operation. You'd better install firefox 46, not 47.

Pretty much what Light said. You need to install firefox and (possibly) specify the file path to selenium.


custom CustomizationId

Added a way to set the customization id of the invoice.
This is to replace pull request #4 with a sensible way of changing the customization id.

Thanks @mcmihai !
Let me know when you think a new release will be useful for your use case. I guess that you still need to revise the hardcoding of currency in the invoice template.

I just uploaded version 0.2 to pypi with your PR included:
https://pypi.org/project/en16931/0.2/


Currency using ISO Code and locale

Updated branch to detect/fix edge case that Siva had found. Ready for testing again.


Arabic support for Frames

How can I add Arabic support for Frames?
I know that I can use placeholders for the field names:
localization: {
              cardNumberPlaceholder: 'Arabic string',
              expiryMonthPlaceholder: 'MM',
              expiryYearPlaceholder: 'YY',
              cvvPlaceholder: 'CVV'
            },

But I am not able to change the direction to RTL. Since inside the iFrame the lang is still set to EN-GB (default locale) even though I use Arabic placeholders. So my input fields and their placeholders are not positioned correctly
<html lang="EN-GB">

Closing this issue as this package will soon be deprecated. A new version using Checkout.com Flow is currently in the works.


Add a hash algorithm method to the KMS clients

Summary

Opened as part of a discussion on https://github.com/sigstore/timestamp-authority/pull/488, this adds a HashFunc method (open to any name suggestions) to the KMS clients. This method will return the hash algorithm used with the signing key. I opened this so we could use the signing key has function in sigstore/timestamp-authority without having to pass the hash function as a flag to the server. See this PR for how the new KMS method is used in the TSA.
Release Note

Documentation

The failure might be due to a aws mock that needs to be updated since there are different calls being made, though i haven't looked closely.

The failure might be due to a aws mock that needs to be updated since there are different calls being made, though i haven't looked closely.

Agreed, I'm going to take closer look at the aws mock.


Workspace2.0: Owner should be first column in public workspace view

So I decided not to make this change because:

We were able to copy/rename the workshop folder as a workspace called "PATRIC Workshop", and
So that the list of public workspaces has the same column ordering as the other lists of workspaces.


Don't send Postgres backups offsite

We've replaced autopostgresql backups with WAL-E, which by design backs up to S3. This job backs up almost 300GB of data to S3 daily, and ends up taking longer than 24 hours, meaning the safety of our MySQL and Mongo backups is reduced.
TODO:

[x] Finish testing the WAL-E restore in staging

So, WAL-E replaces this job in this case, for PostgreSQL backups?

Yep. I think we need a bit more discussion before turning off autopostgresqlbackup completely, but WAL-E definitely* provides our offsite needs at the moment.


About the placement place of the certificate

Hi folks,
The command of sample PowerShell is described as "cert:\localMachine\my" although "To check that the certificate is set, go to the Kudu console and issue this command dir cert:\localmachine\root in the PowerShell debug console. " is in the section of Private client certificate.
App Service (include Web App, App Service Environment) recognizes that the certificate can not be placed in localmachine\root. Is this correct?

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

ID: 5f3a8caa-796a-18ab-480c-161e6c806782
Version Independent ID: 9b8988f6-79d9-4558-5c0c-983ceb857a76
Content: Certificates and the App Service Environment - Azure
Content Source: articles/app-service/environment/certificates.md
Service: app-service
GitHub Login: @ccompy
Microsoft Alias: ccompy

@kogesaka This feature is available only for App Service Environment(ASE). The ASE is a single tenant system. Because it is single tenant, there are some features available only with an ASE that are not available in the multi-tenant App Service (web App).
You can load root certificate in ASE. Here are the steps to do the same from portal:
· Go to an app that needs the cert to be available.
· Prepare your cert as a cer file (see steps below for more detail with screenshots
· Go to SSL settings in the app.  Click Upload Certificate.  Select Public.  Select Local Machine.  Provide a name.  Browse and select your cer file.  Select upload.
· Copy the thumbprint.
· Go to Application Settings. Create an App Setting  WEBSITE_LOAD_ROOT_CERTIFICATES with the thumbprint as the value.  If you have multiple you can put them in the same setting separated by commas and no whitespace

The cert will be available by all the apps in the same app service plan as the app which configured that setting.  If you need it to be available for apps in a different App Service plan you will need to repeat the App Setting operation in an app in that App Service plan.

The last important step is you need to upload each certificate in the certificate chain. Originally, I forgot that when you install a certificate via the browser workflow, you are installing the entire certificate chain. In this scenario though, we have to importing each certificate in the chain individually to the trusted root using the steps detailed out above.
Hope this helps!

@kogesaka Thank you for the prompt response. Closing the issue now.


⚠ Bofrost.be has degraded performance

In 3fc21b2, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1375 ms

Resolved: Bofrost.be performance has improved in 0a0f484 after 8 minutes.


Regenerated builtin file, with updated crdb_interal.force_retry text

Fixes #5538.
Regenerated functions.md file, on https://github.com/cockroachdb/cockroach/releases/tag/v20.2.0-rc.3 + https://github.com/cockroachdb/cockroach/pull/55996/commits/ae19006f70604b9dc95f726498b5075dd44078ee
See https://github.com/cockroachdb/cockroach/pull/55996.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/3c39236fe33b5cfc7712211c521f96731f21c6d5/


feat: auto completion for anchors and headers

Typing any of the following will now trigger a completion:
Completes headers:
[[#

Completes existing block anchors:
[[#^

Completes all blocks:
[[^

This is working only for same file links for now (for example, [[#header]]. I'll expand it to work with other files while waiting for feedback, and add "complete all files [[^^" support in the coming days.

This looks good. let me know when you feel ready to move this out of draft status

This looks good. let me know when you feel ready to move this out of draft status

I'm trying to track down an issue where typing [[#^ completely breaks the autocomplete. It gets stuck saying Loading..., and autocomplete stops working completely (for everything, not just blocks), and there are no logs or anything to show what might be wrong. I'll un-draft this and let you know once I fix that.

roger, sounds good
On Thu, Jul 1, 2021 at 10:22 AM Kaan Genç @.***> wrote:

This looks good. let me know when you feel ready to move this out of draft
status
I'm trying to track down an issue where typing [[#^ completely breaks the
autocomplete. It gets stuck saying Loading..., and autocomplete stops
working completely (for everything, not just blocks), and there are no logs
or anything to show what might be wrong. I'll un-draft this and let you
know once I fix that.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub
https://github.com/dendronhq/dendron/pull/894#issuecomment-872421452,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADD52MYGKBKJOJXFO5T6UTTVSP4JANCNFSM47RZKALQ
.

@kevins8 Fixed the issue! Turns out using document.getWordRangeAtPosition with a complex regex effectively grinds VSCode to a halt.

We now also have support for completing anchors and headers in other files, e.g. [[note#]] for headers and [[note#^]] for anchors. Unfortunately I don't think we can support completing all blocks in other files or all files due to VSCode limitations, I wrote down my thoughts at: [[Vscode Limitations|proj.block-references.vscode-limitations]]

We now also have support for completing anchors and headers in other files, e.g. [[note#]] for headers and [[note#^]] for anchors. Unfortunately I don't think we can support completing all blocks in other files or all files due to VSCode limitations, I wrote down my thoughts at: [[Vscode Limitations|proj.block-references.vscode-limitations]]

i think you made the right call. to close off this task, can you add global auto complete as a separate task in your weekly journal so we can triage it and put it in the backlog?

@kevinslin I investigated the issues you reported:
1 and 2: Issues with Goto note with header/anchor I think were related to anchor parsing issues, and seem to be fixed now.
3: The issue with anchors getting generated twice is because we get the blocks from the engine, which has the saved version of the note. If the user gets blocks without saving the note, the blocks will be out-of-date. Some possible solutions:

For same-file completions, we can perform the get blocks operation on the client side, parsing the text of the open document instead.
We can force save the note after generating an anchor.

This doesn't solve another aspect of the out-of-date contents issue: if the user added a new block and hasn't saved yet, that block will be missing from the autocomplete

If the TextDocument.isDirty, force a save before asking the engine for completions

I think the first option is sufficient. Since it's just for same-file completions, it isn't a very expensive operation.

for the duplicate header, a few comments

we recently made a change to update the note based on text change instead of on save

note update happens here
issue is that the current updateNote call does not update the note body (even though we send the whole note to the server)
we can add an additional prop to updateNote to take an updateBody? parameter that defaults to false

note that for the above solution, we might still end up with a duplicate anchor if the completion is triggered before the update cycle takes place

there's ways to fix this but we should leave that for a phase 2 task  (add to your triage for weekly planning and we can put this on the backlog)

looks like if i create a header [[#]] without completion, it causes an error for getBlocks -> https://www.loom.com/share/23536fc01c664418b174ab94377507af

subsequent completions result in the following error  {"ctx":"provideBlockCompletionItems","error":{"isComposite":false,"status":"unknown","payload":"{}","name":"DendronError"},"msg":"Unable to get blocks for autocomplete"}

    * issue is that the current `updateNote` call does not update the note body (even though we send the whole note to the server)

That doesn't seem to be the case, I can see that the note body is also updated. I switched both findAnchors and the completions to use the cached note, and it does get the updated note body.

* looks like if i create a header `[[#]]` without completion, it causes an error for `getBlocks` -> https://www.loom.com/share/23536fc01c664418b174ab94377507af

This is because that's a broken wikilink, and the wikilink parser throws an error rather than not parsing it as a wikilink. I noticed this issue and tried a few fixes to reject broken wikilinks during parsing, but the changes I attempted were breaking other features. I'd suggest merging this feature and tackling that separately, which I'd be happy to take on.

    * issue is that the current `updateNote` call does not update the note body (even though we send the whole note to the server)

That doesn't seem to be the case, I can see that the note body is also updated. I switched both findAnchors and the completions to use the cached note, and it does get the updated note body.

* looks like if i create a header `[[#]]` without completion, it causes an error for `getBlocks` -> https://www.loom.com/share/23536fc01c664418b174ab94377507af

This is because that's a broken wikilink, and the wikilink parser throws an error rather than not parsing it as a wikilink. I noticed this issue and tried a few fixes to reject broken wikilinks during parsing, but the changes I attempted were breaking other features. I'd suggest merging this feature and tackling that separately, which I'd be happy to take on.

got it. lets take that on as a separate task then. feel free to schedule for the following week.
as for updating the body, what i mean is that it updates the note on disk but not in memory. using serialize works for now though so lets stick with that for now


error: undefined method `[]' for nil:NilClass.

Environment

OS: linux
ruby version: 2.6.0

how to reproduce
run command klipbook list --from-file "My Clippings.txt"
got error: undefined method []' for nil:NilClass. Use --trace to view backtrace
run klipbook list  --from-file "My Clippings.txt" --trace
Traceback (most recent call last):
        20: from .gem/ruby/2.6.0/bin//klipbook:23:in `<main>'
        19: from .gem/ruby/2.6.0/bin//klipbook:23:in `load'
        18: from .gem/ruby/2.6.0/gems/klipbook-4.0.0/bin/klipbook:6:in `<top (required)>'
        17: from .gem/ruby/2.6.0/gems/klipbook-4.0.0/lib/klipbook/cli.rb:53:in `execute!'
        16: from .gem/ruby/2.6.0/gems/commander-4.4.7/lib/commander/delegates.rb:15:in `run!'
        15: from .gem/ruby/2.6.0/gems/commander-4.4.7/lib/commander/runner.rb:68:in `run!'
        14: from .gem/ruby/2.6.0/gems/commander-4.4.7/lib/commander/runner.rb:446:in `run_active_command'
        13: from .gem/ruby/2.6.0/gems/commander-4.4.7/lib/commander/command.rb:153:in `run'
        12: from .gem/ruby/2.6.0/gems/commander-4.4.7/lib/commander/command.rb:182:in `call'
        11: from .gem/ruby/2.6.0/gems/klipbook-4.0.0/lib/klipbook/cli.rb:32:in `block (2 levels) in execute!'
        10: from .gem/ruby/2.6.0/gems/klipbook-4.0.0/lib/klipbook/commands/command.rb:9:in `run!'
         9: from .gem/ruby/2.6.0/gems/klipbook-4.0.0/lib/klipbook/commands/list.rb:5:in `run_command!'
         8: from .gem/ruby/2.6.0/gems/klipbook-4.0.0/lib/klipbook/sources/kindle_device/file.rb:13:in `books'
         7: from .gem/ruby/2.6.0/gems/klipbook-4.0.0/lib/klipbook/sources/kindle_device/file.rb:19:in `build_books'
         6: from .gem/ruby/2.6.0/gems/klipbook-4.0.0/lib/klipbook/sources/kindle_device/file.rb:24:in `extract_sorted_entries_from_file_text'
         5: from .gem/ruby/2.6.0/gems/klipbook-4.0.0/lib/klipbook/sources/kindle_device/file_parser.rb:14:in `extract_entries'
         4: from .gem/ruby/2.6.0/gems/klipbook-4.0.0/lib/klipbook/sources/kindle_device/file_parser.rb:20:in `build_entries'
         3: from .gem/ruby/2.6.0/gems/klipbook-4.0.0/lib/klipbook/sources/kindle_device/file_parser.rb:20:in `map'
         2: from .gem/ruby/2.6.0/gems/klipbook-4.0.0/lib/klipbook/sources/kindle_device/file_parser.rb:21:in `block in build_entries'
         1: from .gem/ruby/2.6.0/gems/klipbook-4.0.0/lib/klipbook/sources/kindle_device/entry_parser.rb:14:in `build_entry'
.gem/ruby/2.6.0/gems/klipbook-4.0.0/lib/klipbook/sources/kindle_device/entry_parser.rb:55:in `extract_type': undefined method `[]' for nil:NilClass (NoMethodError)

Do you have a sample of your clippings file? Does it look well formed?

Thanks for your reply.

No more problem now.


changes in items prop does not trigger rerender

https://github.com/orgsync/react-list/blob/master/react-list.js#L79
see the line above
since javascript uses reference pointer, it thinks the two arrays are identical because they are pointing to the same array.
to work around this, I have to clone the items to trick it into thinking that they are different. this is bad because it will lead to memory leak.
how about have another prop called version, which I can manually increment when I detect the items array has changed.

You can pass your version prop in as the code stands right now. If it changes it will cause the shouldComponentUpdate to return true.

cool. thanks.


[stable/mongodb] Add extraEnvVars

Signed-off-by: Javier J. Salmeron Garcia<EMAIL_ADDRESS>
Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
Allow custom environment variables so settings like dbPath can be changed.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] Title of the PR starts with chart name (e.g. [stable/mychartname])

I appreciate this PR!


fix(deps): update aws-amplify to v6 (major)

Update aws-amplify.  There are a number of package breaks and deprecations making this more difficult.
https://ui.docs.amplify.aws/react/getting-started/migration
https://docs.amplify.aws/react/build-a-backend/troubleshooting/migrate-from-javascript-v5-to-v6/

I'm on it!


Avoid exporting all reachable keypaths from rootmodel when using has - #2366 - RFC

One of the things that @sabob dug up in #2366 was that lots of resolution in a heavily iterated template was unnecessarily creating full RootModel exports just to check for presence of keys. This works around that by implementing the whole has method for RootModel.
I was going to make the full export opt-in only by adding a flag to RootModel.get and possibly Ractive.get, but that breaks the assumption that model.get can be called for any model anywhere. Thoughts?

Also, I tried this out on an a heavily iterative template with lots of expressions on one of my projects and it cut render time from 13s to 1s for ~1500 big funky chunks of template.

@evs-chris I've stared at your changes for 5 minutes but I'm not seeing what is getting short-cutted. How is moving the code into the RootModel different than calling super.has?
Also, AFAIK prop in obj is same as doing the crawl up the prototype chain, so can we just use that?

Oh, Model calls this.get() and Root is doing this.value

Yeah, I think we should implement a module function that does:
function hasProp( obj, key ) {
    return obj && key in obj;
}

I've read that use of in can mess with optimization, so if we put in own function it isolates that lookup from its use.

Also, AFAIK prop in obj is same as doing the crawl up the prototype chain

That actually came up twice in the original PR (#2192) :grin:

I've read that use of in can mess with optimization, so if we put it own function it isolates that lookup from its use.

I haven't seen the Chrome profiler flag any of our uses of in so far. I haven't really researched any of the other browsers optimization fences, so I have no idea about them. That said, a quick grep shows 68 instances of .*(if|for).*\w+ in \w+ in src, so if it would help performance, I'd guess 1/5 of those are in warm to hot code paths. So, should I go ahead and add a hasProp to this as well?


[BUG] Delivery count is 0

Library name and version
Azure.Messaging.ServiceBus 7.10.0
Describe the bug
I have a queue and registered handlers to that queue in my code. Everything worked perfectly, but saw a strange behavior yesterday where a message was not "delivered". I can peek the message on Servicebus explorer, but when I trigger a receive, I am getting an alert saying "there are not messages to receive". I tried purging the queue, but getting similar message that "There are no messages to purge".

App Insights also shows that there were no faults with any message sends.
Expected behavior
Message is delivered or is faulted with an exception and deadlettered.
Actual behavior
Delivery count is 0
Reproduction Steps
Not sure how to reproduce the issue.
Environment
No response

Hi @ljammala.  Thank you for reaching out and we regret that you're experiencing difficulties.  Since you're seeing this behavior with Service Bus Explorer as well as with the client library, the behavior that you're seeing is coming from the service.  Unfortunately, that's not something that we're able to assist with in this repository; your best path forward would be to open an Azure support request.
I'm going to mark this as addressed; if I've misunderstood what you're describing, please unresolve and let us know.

Could be please provide some insight as to why a message would not be "received" by a handler? Why would the delivery count be 0? I added another message on this queue and that was received and processed by my client. But this particular message is stuck and won't be "received".

The message cannot be received by a handler if the service is not sending it to the client, which appears to be the case here based on the delivery count and identical behavior in Service Bus Explorer.


Unified fork installation/launch scripts & chiadog config

Hi,
i took a deeper look into some scripts & found a lot of redundancies. So i tried to unify/simplify the configuration of [new] forks. So far it seems to work, but might need some tuning [esp for things like mmx].
But i wanted you to have a look at it & give feedback before putting more work into it.
Changes:

Only one fork_install script, which will be configured by the variable-files in /scripts/forks/${forkname}
A few new ENV Variables to configure the fork & building process
Only one chiadog-config-sample, which includes variables that will be "templated" on startup using ENV variables
Simple /scripts/build_fork_images.sh as an example of how the build process works

Neat! Reducing redundancy is always a good goal.  Avoiding regressions is also very important.  Please generate a set of Docker images from your repository so we can both test to verify this will work for every blockchain without issue.  Thanks!

Please generate a set of Docker images from your repository so we can both test to verify this will work for every blockchain without issue. Thanks!

Sure, you'll find all generated images under https://hub.docker.com/u/nold360
NOTE: The "default/chia" image is called nold360/machinaris-chia:latest

Hi, thanks so much for your interest in the project.  After a bit of review/testing, I've found regressions around a number of per-blockchain behaviors. Likely due to the loss of specialized install & launch scripts for each blockchain.
For example, this doesn't have MMX blockchain support and breaks blockchain database downloads. If you can address these concerns and incorporate recent improvements on the integration branch, I'll be happy to take another look at this PR.  Thanks again!


New project dialog workspace inputbox fixes

This makes it so that the workspace text doesn't get set until both the project name and location are selected. Also fixes the workspace text not getting updated when the location inputbox is edited directly instead of selected from the file browser.

the folder icon is cut off here.

hmm weird that again this icon problem is only on mac. Fixed it in both dialogs


Transactions not working on ios (commercial version)

Hey, I was facing a bug when trying to start transactions on iOS. Managed to fix it adding "this._transactions = [];" on line 51 to the sqlite.ios.js file.
(my package version is 1.1.5)
Decided to post here, in case it helps :)

@tcione - Thanks for the fix.   I'll get this in the latest version.  :-D

Fixed in latest version.  You should have an email to the link for it if you need it...  :grinning:


macOS Sierra Xcode 8.3 - Cross Compilation failed - Is objcopy not installed?

I'm currently trying to cross-compile a library for android.
I'm using dustup with the last target:
Default host: x86_64-apple-darwin

installed toolchains
--------------------

stable-x86_64-apple-darwin (default)
nightly-x86_64-apple-darwin

installed targets for active toolchain
--------------------------------------

aarch64-apple-ios
aarch64-linux-android
arm-linux-androideabi
armv7-apple-ios
armv7-linux-androideabi
armv7s-apple-ios
i386-apple-ios
i686-linux-android
x86_64-apple-darwin
x86_64-apple-ios

active toolchain
----------------

stable-x86_64-apple-darwin (default)
rustc 1.16.0 (30cf806ef 2017-03-10)

I have installed correctly the standalone NDK toolchain and set them up inside .cargo/config:
[target.aarch64-linux-android]
ar = "~/dev/rust/arm64/bin/aarch64-linux-android-ar"
linker = "~/dev/rust/arm64/bin/aarch64-linux-android-gcc"

[target.armv7-linux-androideabi]
ar = "/NDK/arm/bin/arm-linux-androideabi-ar"
linker = "~/dev/rust/arm/bin/arm-linux-androideabi-gcc"

[target.i686-linux-android]
ar = "~/dev/rust/x86/bin/i686-linux-android-ar"
linker = "~/dev/rust/x86/bin/i686-linux-android-gcc"

When I trigger cargo build --target=arm-linux-androideabi, the compilation failed telling me:
 Is `objcopy` not installed?

The long and complete output can be found here: https://gist.github.com/yageek/6f828bfb496d50856797cb65a453b725

This sounds like it's an issue with the backtrace-sys crate rather than Cargo itself, can you open an issue over there?


Please write details user guilde for benginer, Thank you.

Using the extensions should be pretty straightforward and uncomplicated. Make sure to read the README and the Zed docs.

It is not clear how the language support is added. Unless this extensions only adds runnables?
If you could provide some help around that, we'd be forever grateful :)

The extension has built-in syntax highlighting. Code completion, however, requires v-analyzer to be installed and visible in your system's PATH.

So all you have to do is have v-analyzer installed on your system and added to your PATH.

Had a bit of difficulty with this until I realized, this was already hosted as an extension for zed. Just have to do Ctrl+Shift+P and type extensions and search for 'V' and click install


cbmex for Mac on latest release (7.0) is still using cbSDK 6.05.

I just downloaded the latest cbmex for Mac, but looks like it's still compiled with protocol 3.10. Am I doing something wrong or has it not been updated?
Thanks,
Kian

Hi Kian,
I think there might be a bit of a naming conflict or confusion. Are you looking for Python bindings?
It seems like the original Python bindings were called 'cbmex' as they were largely based on the Matlab bindings. However that approach was abandoned. See here. The new Python bindings are in the cerebus folder and comprise a cerebus.cbpy module.
If you are looking for Python bindings then please let me know if the wheel here does not work for you.
If you are looking for Matlab bindings then I won't be able to build for you because I don't have Matlab on a Mac, but I can help you build them. I've never tested the 7.0 Matlab bindings on any platform, but I did modify cbmex.cpp when I updated to 7.0 enough so they build, so they might work.

Even without MATLAB you can always install Octave(octave-devel) and it will build cboct that is almost the same thing as cbmex

I need it for MATLAB for Mac. I downloaded it under the 7.0 release (on
release page), but it’s still compiled for 6.5 (protocol 3.10) not 7.0.
Best,
Kian
On Tue, May 15, 2018 at 8:00 PM Ehsan Azar<EMAIL_ADDRESS>wrote:

Even without MATLAB you can always install Octave(octave-devel) and it
will build cboct that is almost the same thing as cbmex
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dashesy/CereLink/issues/89#issuecomment-389370640,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJSVFz-1s7F1FE31E9ctKH0uls1HDOs6ks5ty4g0gaJpZM4UAUAH
.

--
Blackrock Microsystems | blackrockmicro.com
Cell: +1-951-254-2686
Schedule a meeting: http://calendly.com/ktorab

Hmm...
There's a chance that I accidentally compiled the mac libs while still on the master branch.
Another more likely scenario is that it is actually built against the 3.10 lib but there are some updates that need to be made to cbmex.cpp to properly wrap the new data structures.
How are you determining what version the mex lib is compiled for?
Unfortunately I won't have time to investigate this further until after June 4th.

How do I do that? Sorry I’m not very code-savvy. I don’t have Python if
that’s a Python code. This is all for MATLAB.
Best,
Kian
On Wed, May 16, 2018 at 11:59 AM Chadwick Boulay<EMAIL_ADDRESS>wrote:

Can you try running testcbsdk in the bin folder?
I'm thinking that maybe the protocol is OK, it's just that the version
identifier in the Matlab wrapper wasn't updated. I'm still searching to see
if that's even possible...
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dashesy/CereLink/issues/89#issuecomment-389610692,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJSVF2yZ-SaXleBASB8CfRw1Z5M8a2r9ks5tzGj7gaJpZM4UAUAH
.

--
Blackrock Microsystems | blackrockmicro.com
Cell: +1-951-254-2686
Schedule a meeting: http://calendly.com/ktorab

Here is where cbmex.cpp is getting the version information.
testcbsdk.cpp does essentially the same thing. So it seems unlikely that they could get different results.
Here is cbSdkGetVersion
It gets the protocol version from here.
It gets the lib version from here.
In the zip in the release page, cbhwlib.h has the new protocol version (3.11).
If testcbsdk returns 3.11 then I have no idea how they could have different results, unless I somehow copied over the mex file with one that I compiled from a previous version. Knowing my usual workflow, this seems very unlikely.
Kian, is there any chance that the old libraries/mex files are still on the path?

It's possible /usr/local/lib is not in the system path.
Try copying the dylib file into the same directory as testcbsdk and run it again.

Reading some old notes here.
Please check if you have a /usr/lib/CereLink folder.

Sigh… So frustrating. I get the “Library memory allocation error”. I
followed the instructions in the Wiki to adjust the memory allocation but
it’s still not working.
Any other ideas?
Kian
On Wed, May 16, 2018 at 1:56 PM Chadwick Boulay<EMAIL_ADDRESS>wrote:

Reading some old notes here
https://github.com/dashesy/CereLink/wiki#osx-distribution.
Please check if you have a /usr/lib/CereLink folder.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dashesy/CereLink/issues/89#issuecomment-389645568,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AJSVF_OkqcmggzsmJcX35mHw7YXmopsBks5tzIROgaJpZM4UAUAH
.

--
Blackrock Microsystems | blackrockmicro.com
Cell: +1-951-254-2686
Schedule a meeting: http://calendly.com/ktorab

Hey Ehsan, did you ever get a chance to test this? BTW, congrats on getting a Mac and welcome to the bright side. :)

Hey Kian!
No unfortunately the Mac was dead after our move

Ehsan, you need to buy a Mac then using all the open-source revenue you
have been collecting. Otherwise, I'm going to assume that you have accepted
defeat regarding the open-source argument. ;)
So?
Kian
On Tue, Nov 24, 2020 at 2:28 PM Ehsan Azar<EMAIL_ADDRESS>wrote:

Hey Kian!
No unfortunately the Mac was dead after our move
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/dashesy/CereLink/issues/89#issuecomment-733243496,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACKJKF466KUVFA5LJPSIA43SRQQPFANCNFSM4FABIADQ
.


Update api-filtering-sampling.md

@markwolff : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Using multiple on the same page error

When using multiple selects on the same page they cross-interract (only when pressing on the item, not the checkbox)
see the GIF

I fixed this issue and made a pull request.


vscode is sharing environment variables between 2 unrelated instances

If I launch 2 different vscode instances from two different command prompts on windows, the second instance of vscode will end up getting the environment variables of the first instance.

VSCode Version: 1.38.1
OS Version: Windows 10

Steps to Reproduce:

launch 2 different command prompts on windows
in the first one, type set zzz=ZZZ
launch code
from the second command prompt, launch another instance of code
go to the terminal of both instances and check the environment variables, both of them will have the environment variable zzz

I tried it also with Git Bash on windows and the same thing happens.

This is by design currently as the windows share the same "main process".
/duplicate https://github.com/microsoft/vscode/issues/15452


Node deploys

Fixes
Please link to the issue this PR addresses
Description
A few sentences describing the overall problem and goals of the pull request's commits.
Proposed changes in this PR

foo
bar
baz

Things to look at

[ ] Test coverage
[ ] Code Style
[ ] Documentation (READMEs etc)

@ChrisDufourMB
I am not confident enough to merge this PR, right now it would include the types.d.ts that I think should be a generated CI file when we build the nodeJS dist. The CI action to deploy to NPM is not there yet either
I would say this brings us like 30-40% to completion. And acts as a PoC that it can work, but this still needs a lot of work.
What are your thoughts?

This makes sense to me. If the goal of this PR is to have a completed node module that can be consumed then we still need the features that you mentioned: generated types and the NPM action.
As it stands now we have a manually created types file that we can't be sure is correctly formatted (we have export type, but maybe it should be export declare type as it is in the source code? https://github.com/GameBridgeAI/ts_serialize/blob/develop/serialize_property.ts#L16). We may want to do some experiments with tsc -d to see what it produces given certain exports.
The way we are currently deploying the package (a tar.gz file) is also not correct and needs to be refactored into a proper npm deployment to a repository somewhere.
I think we will also want to write an implementation test to verify that the produced node module can be consumed by node. Maybe it's possible to write a test that both Deno and nodejs+ts can run to confirm that it works exactly the same in both?

The declare in the source is an error on my part. declare is used for telling the compiler the type will be available at runtime in the global scope. So if a js package is included in ts without types and you know a function will exist in the global scope from the js package you can declare the type for it. I don't believe we need declare, at least where it is currently used. I am also wondering if we can find a better way to export ISODateFromJson we export it as a const but it really represents a function.
I agree with the tar.gz changes as well, I am hoping we can hook a GitHub action to deploy the dist/ from the CI. I also agree with the testing infrastructure for deploys but I am unsure of the approach.

@ChrisDufourMB node deploys are completed, just need the d.ts generator to complete that as the types are manual right now


Restrict access to admin endpoints

How can we restrict access to the provided endpoints for either GET and POST? It seems that the endpoints are accessible from anyone, even without logging in.

@MiltosD I'm sorry for having responded late.
you can define a custom view.
from django_restful_admin import RestFulModelAdmin, site
from .models improt YourModel

class YourModelAdminView(RestFulModelAdmin):
     permission_classes = [IsAutenticatedUser]

site.register(YourModel, YourModelAdminView)