I want pass an async function to Result.mapErr().

Thank you for maintaining this library.
I have a question to how to writing code.
I want to call an async function after a sync function.
I want to use 'await' to the async function.
How should I write code?
I have read issues and wiki, but I haven't know it. Sorry.
Env

node v20.6.0
TypeScript 5.2.2
neverthrow 6.0.0

Source

I will find cache by sync. If not found, I will call the async api function.

import { Result, err, ok } from 'neverthrow'

import * as fs from 'fs'

const readCache = (cacheFilePath: string): Result<string, Error> => {
    if (fs.existsSync(cacheFilePath)) {
        console.log('found cache')
        const cacheContent = fs.readFileSync(cacheFilePath).toString()
        return ok(cacheContent)
    }
    console.log('not found cache')
    return err(new Error(`Not found cache: ${cacheFilePath}`))
}

const requestByApi = async (): Promise<string> => {
    console.log('calling api...')
    return Promise.resolve('new content')
}

readCache('/cache/f1.json')
    .mapErr(async (_: Error) => await requestByApi())
    .map((content: string) => {
        console.log(`got ${content}`)
    })
console.log('finish')

The following is output of result.
not found cache
calling api...
finish

However, I want other output of the following.
not found cache
calling api...
got new content
finish

I know how to use fromPromise and rewrite code, but I got compile error from TypeScript.
const requestByApi = async (): Promise<string> => {
    console.log('calling api...')
    return Promise.resolve('new content')
}

// Argument of type '() => Promise<string>' is not assignable to parameter of type 'PromiseLike<unknown>'.
const safeRequestByApi = ResultAsync.fromPromise(requestByApi, (e) => e)

Hello @mitsuru793
I’m sorry I can’t write a full response as I am away but you should replace your mapErr with orElse (see doc).
To fix the error above you need to pass a promise to fromPromise. Here you pass a function that returns a promise.

@paduc
Thank you very much.
orElse is useful, but it cannot use await inside it.
Can't neverthrow chain sync and async function?
const safeRequestByApi = Result.fromThrowable(requestByApi, (e) => e)

readCache('/cache/f1.json')
    .orElse((_: Error) => safeRequestByApi())
    .map((content: string) => {
        console.log(`got ${content}`)
    })
console.log('finish')

got output
not found cache
calling api...
got [object Promise]
finish

I expected output.
not found cache
calling api...
got new content
finish

I want to write the following.
readCache('/cache/f1.json')
    .orElse(async (_: Error) => await safeRequestByApi())
    .map((content: string) => {
        console.log(`got ${content}`)
    })

I have understand the error of fromPromise. Thank you.
However, this take the result of function.
// run requestByApi here. I want not to run here.
const safeRequestByApi = ResultAsync.fromPromise(requestByApi(), (e) => e)

@mitsuru793

@paduc Thank you very much.
orElse is useful, but it cannot use await inside it. Can't neverthrow chain sync and async function?

Yes it can. When you start with a sync Result and you want to chain with something that returns a ResultAsync, you need to use async variants (see Result.asyncMap). I see that there is no async variant for orElse (a PR would be welcome).
If you start with a ResultAsync you won't have this problem.
 okAsync() // only to start with an async object
     .andThen(() => readCache('/cache/f1.json'))
     .orElse((_: Error) => safeRequestByApi())
     .map((content: string) => {
         console.log(`got ${content}`)
     })

I want to write the following.
readCache('/cache/f1.json')
    .orElse(async (_: Error) => await safeRequestByApi())
    .map((content: string) => {
        console.log(`got ${content}`)
    })

orElse only takes functions that return a Result/ResultAsync, not a promise. Therefor, you cannot use await. It's fundamental to neverthrow.

I have understand the error of fromPromise. Thank you. However, this take the result of function.
// run requestByApi here. I want not to run here.
const safeRequestByApi = ResultAsync.fromPromise(requestByApi(), (e) => e)

You can use a function:

const safeRequestByApi  = (params) => {
   return ResultAsync.fromPromise(requestByApi(params), (e) => e)
}

@paduc
Thank you very much and your answer is amazing!
I have resolved my issue by you.
I will close with my issue.


Sidebar collapse icon issue

Reproduction link
https://stackblitz.com/run
Steps to reproduce

1. Go to the official website: https://ng.ant.design/components/me![SidebarICollapseIcon](https://user-images.githubusercontent.com/73377984/148087472-1cd9103d-7d79-4b6e-8858-988049f89a91.PNG)
2. then just click once on the collapse icon and do not move the cursor and now the double icon issue could be noticable

What is expected?
Collapse icon should appear only once when it is clicked even after page reload
What is actually happening?
Sidebar collapse icon is appearing twice when it is clicked first time after reload

Environment
Info

ng-zorro-antd
13.0.0

Browser
Chrome

this double icon issue comes only once per page successful loading
but this issue is not happening in stackblitz
this bug is presented in (ng.ant.design) official page

@wendellhu95 I think this is related to the Icon Module and caused by the perf PR #7071 , which setSVGData method is running outside the Angular zone. Could you help take a look at it?


FEATURE REQUEST: styleUrls for dialog components

Feature request: Styleable MatDialogs
@Component in a component used for MatDialogs should accept and apply styleUrls as any other component
Currently the StyleUrls get ignored..
The only way i found to style all instances of a dialog on a separate file is to add that style sheet to
angular-cli.json (so to the whole project) and then apply a class when opening the dialog
This is not a big issue but just having the dialog apply that style sheet automatically like any otehr component would be much better imo...

Closing this issue as it does not follow the issue template. Please file a new issue that follows the template, which gives the team the information needed to investigate


problem with wide_n_deep_tutorial.py on Tensorflow 1.0

Using Python 3.6.0 (Anaconda x64), Tensorflow 1.0, macOS Sierra version 10.12.4, I get the following error:

python wide_deep.py
Training data is downloaded to /var/folders/h2/727s56vx40s_6n2z9ldl6kx00000gs/T/tmp76m50o3h
Test data is downloaded to /var/folders/h2/727s56vx40s_6n2z9ldl6kx00000gs/T/tmpwzhof_zb
model directory = /var/folders/h2/727s56vx40s_6n2z9ldl6kx00000gs/T/tmpclbsc2wm
Traceback (most recent call last):
File "wide_deep.py", line 234, in
tf.app.run(main=main, argv=[sys.argv[0]] + unparsed)
File "/Users/CBrauer/anaconda/lib/python3.6/site-packages/tensorflow/python/platform/app.py", line 44, in run
_sys.exit(main(_sys.argv[:1] + flags_passthrough))
File "wide_deep.py", line 197, in main
FLAGS.train_data, FLAGS.test_data)
File "wide_deep.py", line 185, in train_and_eval
m = build_estimator(model_dir, model_type)
File "wide_deep.py", line 132, in build_estimator
fix_global_step_increment_bug=True)
TypeError: init() got an unexpected keyword argument 'fix_global_step_increment_bug'

Charles

Thanks for pointing this out. It seems that the tutorial code on the website is linking to the latest version instead of the version compatible with release 1.0.
Could you try with the tutorial version from the 1.0 branch?
https://github.com/tensorflow/tensorflow/blob/r1.0/tensorflow/examples/learn/wide_n_deep_tutorial.py
I'll try to look into how to make sure that the links in the website point to the appropriate release branch.


chore: prepare release 2024-07-02

Summary
This PR has been created using the apic release script. Once merged, the clients will try to release their new version if their version has changed.
Version Changes

csharp: 7.0.0-beta.8 -> prerelease (e.g. 7.0.0-beta.9)
dart: 1.16.0 -> prerelease (e.g. 1.16.1-0)
go: 4.0.0-beta.22 -> prerelease (e.g. 4.0.0-beta.23)
java: 4.0.0-beta.39 -> prerelease (e.g. 4.0.0-beta.40)
javascript: 5.0.0-beta.9 -> prerelease (e.g. 5.0.0-beta.10)
kotlin: 3.0.0-beta.34 -> prerelease (e.g. 3.0.0-beta.35)
php: 4.0.0-beta.8 -> prerelease (e.g. 4.0.0-beta.9)
python: 4.0.0b23 -> prerelease (e.g. 4.0.0b24)
ruby: 3.0.0.beta.8 -> prerelease (e.g. 3.0.0.beta.9)
scala: 2.0.0-beta.8 -> prerelease (e.g. 2.0.0-beta.9)
swift: 9.0.0-beta.9 -> prerelease (e.g. 9.0.0-beta.10)

Skipped Commits
It doesn't mean these commits are being excluded from the release. It means they're not taken into account when the release process figured out the next version number, and updated the changelog.

Commits without language scope:

docs: add common test suite schema for snippets (#3302)
chore: CI generated commit hash and no gen comment (#3301)
chore: notification links (#3299)

Commits with unknown language scope:

fix(scripts): do not wait release for ruby (#3303)

this was a test to make sure we can force release


Datapoint error handling

Refactor datapoint insertion to use the new CogniteError/CogniteResult system.

Checkmarx scan completed with the following findings

Lines
Severity
Category
File
Link

68
Medium
Heap_Inspection
Cognite.Metrics/MetricsConfig.cs
Checkmarx


bbox_derive: New command to derive modules for blackboxes

In certain uses cases you want to derive modules for blackbox instances, which are otherwise ignored by the hierarchy process. Here we add a new command, bbox_derive, by which one can ask to have modules derived for a selected set of blackbox instances.
I can think of two use cases:

Functional modelling of blackboxes (the abc9 integration has an abc9_ops -prep_hier command to a similar end, we can merge those commands eventually)
Deriving of one of the possible implementations for a blackbox (where the blackbox is a blackbox because an implementation hasn't been selected yet)

An incomplete test is attached.

    bbox_derive [-base <base_module>] [-naming_attr <attr>] [selection]

As part of the assembly of the design hierarchy done by the 'hierarchy' command,
specializations of parametric modules are derived on demand: for each choice of
parameter values appearing in the design, a copy of the parametric module is
derived which is specialized to that choice.

This derivation process ignores blackboxes and whiteboxes. To supplement, this
'bbox_derive' command can be used to request the derivation of modules based on
blackbox or whitebox instances appearing in the design, which is desirable in
certain use cases. Only the selected cells are considered as the instances that
steer the derivation process.

    -base <base_module>
        instead of deriving the module that directly corresponds to each blackbox
        instance, derive a specialization of <base_module> (this option applies
        to all selected blackbox cells)

    -naming_attr <attr>
        once a specialization is derived, use the value of the module attribute
        <attr> for a name which should be used for the derived module (this
        replaces the internal Yosys naming scheme in which the names of derived
        modules start with '$paramod$')

box_derive seems nice, considering you also handle whiteboxes.


Handling 1 trigger(s) that missed their scheduled fire-time.

Avoid error handlers
Handling 1 trigger(s) that missed their scheduled fire-time.s

This issue was moved to OpenQuartz/quartz-scheduler#5

Hi, I am not sure I understand what your request is. If you have a problem with quartz, please try to reproduce with a standalone Java program, and provide stacktrace and log if possible.

Close as not an issue


Support outputting unsigned provenance

In case a user wants to create GitHub workflow provenance and then attach that to a container image using cosign CLI. By doing this, the attached container image would include rekor bundle and certificate in the cosign simple signing schema.
e.g.
$ ./slsa-github-generator attest --unsigned --output predicate.json

$ ./cosign attest --predicate predicate.json --type intoto gcr.io/asra-ali/hello-ko

The generic provenance builder should only output the predicate
cc @laurentsimon

Would --unsigned imply --predicate should be set?
What would happen in this case where --output is not given? Would the unwrapped intoto statement be generated into the default --signature path?
$ slsa-github-generator attest --unsigned

I'm kind of thinking that maybe something like this makes more sense?
$ slsa-github-generator attest --signature="" --predicate predicate.json

or maybe get rid of the default signature path and require folks to specify --signature or --predicate/--output or both?


EDU-1508 Support PNG export for whiteboards

Closes https://educandu.atlassian.net/browse/EDU-1508

Looks good! 😁
One small thing I've noticed, we now use 2 different icons for download. The one in media player is:
Funny enough, the one in my Brave browser is the same as the one in our media player:
Honestly I don't have any preference, but I think we should use just one of them in both places. Feel free to pick which if you agree.

Good catch, thanks :+1: I sticked to the pre-existing icon!


Update default RACK_ENV to production

Just asking: is there a reason to force production environment? I'm trying to deploy a staging deployment and I had to create a custom runtime, only to change this (env_variables in app.yaml didn't change it).

@mitoyarzun env_variables should let you change RACK_ENV and/or RAILS_ENV. At least, it seemed to work when I just tried testing it myself. If you're still having trouble with it, can you file an issue with repo steps?
In the meantime, you can also set environment variables as part of your entrypoint in the app.yaml, in order to set the environment. For example:
entrypoint: RACK_ENV=staging bundle exec rackup -p $PORT config.ru

Many entrypoints also support command line switches to set the environment. Here's a Sinatra example:
entrypoint: bundle exec ruby myapp.rb -p $PORT -e staging

Also, this particular PR is being superseded by https://github.com/GoogleCloudPlatform/ruby-docker/pull/38 so I'm going to close it.


Switch from deprecated checkCode to update_document from the language service in basics.js.

Fixes #469

Do we still have coverage for that API anywhere, being that's what QCOM is still using currently. If we are removing it, we'll need to document migration steps for them. I believe the plan was they should move first, then we'll remove it after.

Do we still have coverage for that API anywhere, being that's what QCOM is still using currently. If we are removing it, we'll need to document migration steps for them. I believe the plan was they should move first, then we'll remove it after.

The API hasn't been removed, so we aren't breaking anyone yet. The checkCode implementation already just wraps calls into the language server, so the only special thing it does is map positions in the returned diagnostics with mapUtf8UnitsToUtf16Units. Do we want coverage specifically for that? Because the previous tests really didn't verify any corner cases of that mapping.

If QCOM is still using checkCode, (which arguably gets a lot more usage than our VS Code extension currently), then I think we should still have the tests exercise checkCode.

In that case, I'll drop this for now.


Is there a way to clear all health checks manually

We have 2 health checks that we disabled - but they both still hit our end-points each minute?

@cberlandier ,
This bug is fixed on the new version of Konga. Right now on the next branch but will be commited to muster sometime today or tomorrow. The caveat is that due to the nature of the problem, you will have to remove the previously orphaned health checks manually from Konga's db,
Thanks for the feedback!


[tool_crash] RPCError

Command
flutter run

Steps to Reproduce

...
...
...

Logs
RPCError
#0      new _OutstandingRequest (package:vm_service/src/vm_service.dart:1928:45)
#1      VmService._call (package:vm_service/src/vm_service.dart:2493:21)
#2      VmService.callServiceExtension (package:

Closing this as a duplicate of https://github.com/flutter/flutter/issues/102994


array_merge issue in Client authRequest

We are having the following error affecting a small portion of our users:
array_merge(): Argument #1 is not an array {"exception":"[object] (ErrorException(code: 0): array_merge(): Argument #1 is not an array at /var/www/app/vendor/nexmo/client/src/Client.php:205)
[stacktrace]
#0 [internal function]: Illuminate\\Foundation\\Bootstrap\\HandleExceptions->handleError(2, 'array_merge(): ...', '/var/www/app/ve...', 205, Array)
#1 /var/www/app/vendor/nexmo/client/src/Client.php(205): array_merge(NULL, Array)
#2 /var/www/app/vendor/nexmo/client/src/Client.php(359): Nexmo\\Client::authRequest(Object(Zend\\Diactoros\\Request), Object(Nexmo\\Client\\Credentials\\Basic))
#3 /var/www/app/vendor/nexmo/client/src/Message/Client.php(48): Nexmo\\Client->send(Object(Zend\\Diactoros\\Request))

Seems like something is going wrong with the authentication request, but only in some specific cases. The majority of times it works nicely, so I would discard an issue with our credentials. Are there any know causes to this problem? Could it be a regional issue? (the reports we've heard are from users in UAE and China).

It's not a lot of information but we appreciate the bug report regardless! I couldn't reproduce which isn't a surprise as you say it works the majority of the time. We'll keep trying and if you can keep us posted with anything more at your end, that would be helpful too.

Closing due to inactivity, but please re-open if you have more information relating to this issue.


Update poltergeist to 1.8.1

Older versions of poltergeist do not always return the correct exit code and can lead to test suites failing but CI passing, this behaviour has been seen in travel advice publisher branch builds.
See https://github.com/alphagov/travel-advice-publisher/pull/98 for detail.

Thanks ✨

:+1: :100:


syntax highlighting gets confused

Trying out Zed with nu highlighting I found this in default_env.nu. Notice that after the pipe, it stops highlighting.

Looking at the Syntax Tree windows in Zed, there are a few ERRORs.

It would be nice to get this fixed in order to have a more pleasant experience in Zed.

@fdncred
could you share the source code of this so that i can dump it inside Neovim and see if i get the same? :innocent:
i think i've seen this before and suspect it's getting confused around the strings / quotes :thinking:

@amtoine The file is the default_env.nu.


Rbac spike

A short explanation of the proposed change:
Write a migration to create a space operator table in ccdb

An explanation of the use cases your change solves
We will be creating a new space operator role, and cc will need to store this somewhere to recognize space operator users.

Links to any other associated PRs
[link to main issue]

[ ] I have reviewed the contributing guide

[ ] I have viewed, signed, and submitted the Contributor License Agreement

[ ] I have made this pull request to the main branch

[ ] I have run all the unit tests using bundle exec rake

[ ] I have run CF Acceptance Tests

This was a test


Adding custom field to HiveTable Entity

Hi Cloudera,
I want to add a custom field of type **_"INT"_** to every hive table so that I can later call something like _**setCustomField(value)**_
Can you please guide me how it can be done?

If you want the values to be int instead of string, you have to create a managed metadata field. This can be done by:

extending the existing Hive model
Annotate the new field with @MProperty(register=true, fieldType = CustomPropertyType.INTEGER)

Note that this requires Nav-2.6+

Thanks for your input. Currently our cluster has Nav - 2.4.4.


Propose BEP026 MER

Add new proposal for Microelectrode Recordings.

Thanks! See https://github.com/bids-standard/bids-specification/blob/master/CONTRIBUTING.md#writing-in-markdown to figure out how to fix the markdown syntax errors.

Well that was easy... sorry for all the commits.

Thanks! I recommend using prettier next time.


migrate to NHibirnate Core

just proposal - I can this, need only an agree

#167 related

The idea to migrate NH is based on the fact I discover that EF Core does not support group by(even EF 2.2 #167 and in further plans, you can investigate this starting from this thread for example https://stackoverflow.com/questions/48082277/entityframework-core-group-by-gives-me-an-order-by)
Also while the investigation I've found that EF Core has also troubles with joins, count, order by. I was very surprised by these facts because EF 6 does it perfectly. From other side I see a lot of new features such as Symbol Server or  and taking into consideration fact that primary NuGet server is used to search/retrieve the results of NuGet by various criteria I do not think that EF Core is right ORM(at least at future several years). NH much more flexible and powerful(quality of sql primary) than EF and to avoid further troubles and while we have only two tables in DB I think we need to migrate to NH.

I would like to avoid changing our database provider until we absolutely need to. If anything, I would rather move to EF6 than NHibernate, and EF6 will be supported on .NET Core 3, which is currently in preview. Does that seem reasonable? I'll close off this issue for now, but feel free to add any comments.

I would like to do not agree that moving back to EF6 is reasonable ( EF6 in compare NHibernate is much more weaker ORM.
From other side NH supports such features as shards, search, caching which may help in cases when 'Pacakges' table contains a thousands of packages.
So migrating now to NH will resolve a lot of future issues.
also for Azure lovers(maybe interesting fact):
https://ayende.com/blog/4178/sql-azure-sharding-and-nhibernate-a-call-for-volunteers

NuGet.org uses entity framework and scales to 2 million records without any sharding. I’ve also indexed all of NuGet.org using BaGet on a single database. I don’t think there’s a need to replace the current solution at this time. I’ll read up more about the caching and searching though, that could be useful!


Error when freeze the graph

Information:

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Ubuntu 16.04
TensorFlow installed from (source or binary): binary 1.5
Python version: 3.5
CUDA/cuDNN version: No

Hello,
I use a model, in my code I save the model with that command:
                writer = tf.summary.FileWriter("/tmp/model4/MTCNN/", self.sess.graph)                 saver = tf.train.Saver() #saver load pretrain model                 save_path = tf.train.Saver().save(self.sess, "/tmp/model4/MTCNN/model.ckpt")
It generates me these files:
/tmp/model4/MTCNN/checkpoint
/tmp/model4/MTCNN/events.out.tfevents.1526300275.xavier-Inspiron
/tmp/model4/MTCNN/model.ckpt.data-00000-of-00001
/tmp/model4/MTCNN/model.ckpt.index
/tmp/model4/MTCNN/model.ckpt.meta

I reload this saved model and graph with that code:
               self.sess = tf.Session()
                model_path = "/tmp/model4/MTCNN/model.ckpt"
                model_path_p = "/tmp/model4/MTCNN/"
                self.sess.run(tf.global_variables_initializer())
                new_saver = tf.train.import_meta_graph(model_path + '.meta')
                new_saver.restore(self.sess,tf.train.latest_checkpoint(model_path_p))

And this works properly. Now I want to freeze my model in pb file with that command:
python3 ./tensorflow/python/tools/freeze_graph.py --input_checkpoint=/tmp/model4/MTCNN/model.ckpt.data-00000-of-00001 --input_meta_graph_def=/tmp/model4/MTCNN/model.ckpt.meta  --output_graph=frozen_graph.pb --output_node_names='pnet/conv4-2/BiasAdd'
And I get that errors:
2018-05-14 14:35:20.597234: I tensorflow/core/platform/cpu_feature_guard.cc:137] Your CPU supports instructions that this TensorFlow binary was not compiled to use: SSE4.1 SSE4.2 AVX AVX2 FMA 2018-05-14 14:35:20.598060: W tensorflow/core/util/tensor_slice_reader.cc:95] Could not open /tmp/model4/FaceFeature/model.ckpt.data-00000-of-00001: Data loss: not an sstable (bad magic number): perhaps your file is in a different file format and you need to use a different restore operator? Traceback (most recent call last):   File "./tensorflow/python/tools/freeze_graph.py", line 360, in <module>     app.run(main=main, argv=[sys.argv[0]] + unparsed)   File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/platform/app.py", line 48, in run     _sys.exit(main(_sys.argv[:1] + flags_passthrough))   File "./tensorflow/python/tools/freeze_graph.py", line 254, in main     FLAGS.saved_model_tags, FLAGS.checkpoint_version)   File "./tensorflow/python/tools/freeze_graph.py", line 244, in freeze_graph     saved_model_tags.split(","), checkpoint_version=checkpoint_version)   File "./tensorflow/python/tools/freeze_graph.py", line 119, in freeze_graph_with_def_protos     reader = pywrap_tensorflow.NewCheckpointReader(input_checkpoint)   File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/pywrap_tensorflow_internal.py", line 150, in NewCheckpointReader     return CheckpointReader(compat.as_bytes(filepattern), status)   File "/usr/local/lib/python3.5/dist-packages/tensorflow/python/framework/errors_impl.py", line 473, in __exit__     c_api.TF_GetCode(self.status.status)) tensorflow.python.framework.errors_impl.DataLossError: Unable to open table file /tmp/model4/FaceFeature/model.ckpt.data-00000-of-00001: Data loss: not an sstable (bad magic number): perhaps your file is in a different file format and you need to use a different restore operator?

Nagging Assignee @tatatodd: It has been 21 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 36 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 51 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 66 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 81 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 96 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 111 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Nagging Assignee @tatatodd: It has been 126 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Add: remove award functionality

This fixes #434.
It's now possible to remove Award from person's details page or from
badge's details page. Both removals should end up with correct
redirects.
Tests are included.

:+1:


Add Windows CI jobs

only python 3.7 and cuda 10.0
on cpu and gpu
[x] need to fix tests\chainer_tests\test_init_docstring.py fail

How can we test the PR?

chainer.win.{gpu,cpu} project has already created on pfnci, so add these project to webhook and write comment like "test please", then the jobs will start. But I don't know a correct way to add webhook on this project so have not added webhook yet. CC: @kmaehashi

Jenkins, test this please

Jenkins CI test (for commit 72e6c2f789d8c3964159a30d4b87ccf3eb396dbd, target branch master) succeeded!

thank you for your revies

can I expect that the tests\chainer_tests\test_init_docstring.py issue will be solved in this PR?

I don't have a plan to fix it.

can I expect that the tests\chainer_tests\test_init_docstring.py issue will be solved in this PR?

I don't have a plan to fix it.

It seems this error is only occurring in this PR.
I think I can't merge the PR unless it's fixed.

It seems this error is only occurring in this PR.

I thought the error causes only Windows env.
I'd like to know how to stop the error on xpytest? CC: @imos @kmaehashi

Ah, "`I thought the error causes only Windows env." it true, (misunderstanding is misunderstanding). I could reproduce the error without xpytest nor this PR, I reported an issue #7523
As you said, this PR cannot merge until the error is resolve, I'll wait it. @niboshi

sorry for unattended, tests on windows os are failed, issued #7768. If need some time to fix the issue, CI can skip the target test.

Blocked by https://github.com/chainer/chainer/pull/8465


 Nothing to do.
**********************************************************************
1 items had failures:
   1 of   7 in pythran.toolchain.compile_pythranfile
***Test Failed*** 1 failures.
 Nothing to do.
**********************************************************************
1 items had failures:
   1 of  14 in pythran
***Test Failed*** 1 failures.

Build log:
dev-python:pythran-0.12.0:20220928-175414.log

No 'requirements-files' section was found. Nothing to do.

That's quite new to me, let's dive in!

Of, this comes from this package: https://github.com/s0undt3ch/setuptools-declarative-requirements/blob/a92708703592490e5bb3fe1e8d8726c4d57bdde1/declarative_requirements/integration.py#L43

It's not explicitly listed in the dependencies, could it be something specific to Gentoo's setuptools?

Probably specific to my test environment — I have lots of packages installed there. Unfortunately, setuptools plugin architecture involves loading all installed plugins unconditionally which can lead to all kinds of misfortune (and I'm not aware of any way to disable that). Is there any chance you could manage to get the test suite to pass independently of plugins installed?

The best thing I can think of is to disable testing the doc, something like py.test --ignore-glob='*xdoc*' pythran/tests

I suppose it's time to start the witch hunt on pytest-declarative-requirements.

Well, pytest-declarative-requirements is masked for removal from Gentoo and everything looks good, at least at a first glance. Still waiting for the test suite to finish with all impls.

@mgorny can you confirm I can close the issue now?

Yes, the tests passed. I'm sorry for forgetting to reply.


MetodoJacoobi.c

Title/Heading
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ v] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ v] This change requires a documentation update

Checklist:

[v ] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[v ] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[ ] My changes generates no new warnings
[v ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

Metodo Jacoobi

PR Rejected:

Template unfilled
No proper formating, even the file has no extension!
Not followed contribution Guidelines


Italian translation

Hi, here is the Italian translation, as for #2
I have added a link to the italian page in the main page.
Cheers 🍻

Give stars my friend.

Just to be clear, are you implying that you are not accepting the PR as long as I dont star your project?

Not at all my friend @giovannipessiva


error while compiling miniCaffe using cuDNN

I compile the minicaffe as below:
mkdir build
cd build
cmake .. -DCMAKE_BUILD_TYPE=Release -DUSE_CUDA=ON -DUSE_CUDNN=ON
make
And I have already intsalled the cudnn.But a problem was occured:
[  0%] Building NVCC (Device) object CMakeFiles/cuda_compile.dir/src/layers/cudnn/cuda_compile_generated_cudnn_bn_layer.cu.o
/home/lgz/mini-caffe/src/layers/cudnn/././cudnn.hpp(97): error: too few arguments in function call
1 error detected in the compilation of "/tmp/tmpxft_00003fb4_00000000-7_cudnn_bn_layer.cpp1.ii".
CMake Error at cuda_compile_generated_cudnn_bn_layer.cu.o.cmake:266 (message):
Error generating file
/home/lgz/mini-caffe/build/CMakeFiles/cuda_compile.dir/src/layers/cudnn/./cuda_compile_generated_cudnn_bn_layer.cu.o
What's the matter ?

My cudnn is newest one and my OS is ubuntu 16.04.Maybe the version of cuDNN cause this problem ?

I find a  post of NVIDIA which has the same question and the solution is work.
https://devtalk.nvidia.com/default/topic/1002826/gpu-accelerated-libraries/question-about-cudnnsetconvolution2ddescriptor/

What's your cudnn version? I think the newest version is cudnn6, but I didn't test with it. Only cudnn5.1 is tested.

Adding the below will solver it.
-DCUDA_NVCC_FLAGS='-std=c++11'
eg:
cmake .. -DCMAKE_BUILD_TYPE=Release -DUSE_CUDA=ON -DUSE_CUDNN=ON -DCUDA_NVCC_FLAGS='-std=c++11'


Microsoft.ApplicationInsights.AspNetCore vs Microsoft.Extensions.Logging.ApplicationInsights

There needs to be clearer guidance and mention of the two packages I mentioned in the title.

Microsoft.ApplicationInsights.AspNetCore
Microsoft.Extensions.Logging.ApplicationInsights

e.g. Differences? One When to use one or the other ? Can one be used without the other? I only discovered the second package Microsoft.Extensions.Logging.ApplicationInsights by trial and error; but the documentation should mention the existance of both, and use cases of each separately, in combination, etc.

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

ID: 4b88d8c4-0013-70aa-7997-a2203afd9743
Version Independent ID: 0ab99d5e-4ebd-8cb9-3d27-efe58def249f
Content: Azure Application Insights for ASP.NET Core applications - Azure Monitor
Content Source: articles/azure-monitor/app/asp-net-core.md
Service: azure-monitor
Sub-service: application-insights
GitHub Login: @AaronMaxwell
Microsoft Alias: aaronmax

@joe-oli
Thanks for your feedback! We will investigate and update as appropriate.

@joe-oli, thank you for this feedback.
@AaronMaxwell, can you please review this fedback and share your thoughts. Thank you

Thanks for your dedication to our documentation. This issue is being triaged by multiple product groups before we comment, and we apologize for the further delay that is expected. Thanks in advance for your understanding as we investigate to provide the most accurate answer and documentation updates.

#assign:v-jbasden
https://dev.azure.com/msft-skilling/Content/_workitems/edit/51097

Work item 51097 created for this issue.


Upgrade TeXstudio.app to v2.9.4

Upgraded version number and checksum to reflect version 2.9.4

Sorry for closing the previous request and reopening another one. It's probably not the most elegant way to solve the problem, but I'm still new on GitHub and the closing of the previous happened by accident.


Move Exec and ExecSession to embulk-api

Finally! It should be the last API for plugins. I believe that embulk-api and embulk-spi should be ready for plugins to build only with depending on them without embulk-core. Some more supplemental classes in embulk-api and utility libraries (embulk-util-*) may be needed later, though.

Thanks!


test mail form

👷 Deploy Preview for gladia processing.

Name
Link

🔨 Latest commit
7775bdfb1af49093b5bdb61ac5abfe71aad66c5f

🔍 Latest deploy log
https://app.netlify.com/sites/gladia/deploys/64295d8d5f485d0008b96fb2


Dev

Update deps
Update tests
Update python-vs-app.yml

Codecov Report

Merging #19 (1a5ee5d) into main (5591ddd) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #19   +/-   ##
=======================================
  Coverage   78.22%   78.22%
=======================================
  Files          14       14
  Lines         248      248
=======================================
  Hits          194      194
  Misses         54       54

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Change UI for submission to synbiohub

Remove purpose and chassis.
Add radio button:
If existing:
Prevent submission
Overwrite
Merge
should encode as '0', '1', '2', respectively, and strings not integers.
Note that Merge should disallow name/description to be entered as will be used from first submission with same id/version.

Names and operations for overwrite/merge, changed:
'0': Prevent if submission exists
'1': Overwrite entire submission
'2': Merge and prevent if existing objects
'3': Merge and replace if existing objects
This is not implemented yet, but basically the difference is merge needed to be split into two separate behaviors and general merged had to be removed.
We should probably discuss primary use cases by designer.

I think SBOLDesigner would primarily use the two Merge options.  Namely, in SBOLDesigner, you are usually working on one part of a submission.  You either want to "Overwrite" or create a "New Version" as per save.  The "Merge and Prevent" is basically you do not want to overwrite, you want to ensure that what is saved is uniquely identified and does not replace anything else.  The "Merge and Replace" option says you want to overwrite the existing part in the submission, if one already exists.
This is more parallel with how you treat documents.  Namely, you really only consider the individual object that you are working on within the document.  Let me know if this does not make sense.  Note I have not implemented this style of merges yet.  Right now, you can only overwrite an entire submission or merge with existing (not replace).  Will let you know when it is added.

I have added basic support for these various options with my latest commit.  This still needs to be tested/refined, but it should give basic functionality when it comes to these options.  I have also added many TODOs for improvements to the UploadDialog.  We can discuss details during our meeting on Thursday.

I tweaked the upload window a bit.

On Jan 25, 2017, at 5:03 AM, Michael Zhang<EMAIL_ADDRESS>wrote:
I have added basic support for these various options with my latest commit. This still needs to be tested/refined, but it should give basic functionality when it comes to these options. I have also added many TODOs for improvements to the UploadDialog. We can discuss details during our meeting on Thursday.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub https://github.com/SynBioDex/SBOLDesigner/issues/127#issuecomment-275020027, or mute the thread https://github.com/notifications/unsubscribe-auth/ADWD92suIgwR_H2jes_RSYyneMbI4t19ks5rVteegaJpZM4LsLBA.

Chris:
Merge should now work on submit.  Please change the names to:
Prevent Submission
Overwrite Submission
Merge and Prevent, if existing
Merge and Replace, if existing
Also, please make it a combo box like iBioSim.
Thanks,
Chris

I've updated UploadDialog and tested the submission options a bit.  The info message about the different options could probably be changed a bit.

Change ui to match: http://synbiohub.utah.edu/submit
Specifically, collections replace keywords and options names have changed.

The options have been updated and collections has replaced keywords.  However, when trying to get all collections from the java api, it crashes.  My SearchCriteria might not be correct.

Can you provide more detail about the crash and how you setup your search criteria?
Chris

On Apr 5, 2017, at 6:35 PM, Michael Zhang<EMAIL_ADDRESS>wrote:
The options have been updated and collections has replaced keywords. However, when trying to get all collections from the java api, it crashes. My SearchCriteria might not be correct.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub https://github.com/SynBioDex/SBOLDesigner/issues/127#issuecomment-291937714, or mute the thread https://github.com/notifications/unsubscribe-auth/ADWD93GWw8EJqOyYi-5_zTrLY5VOsBY4ks5rs9DJgaJpZM4LsLBA.

http://wiki.synbiohub.org/wiki/HTTP_API specifies how to search for specific collections, but not all collections.  I put "key: collection, value:", which I'm pretty sure is why it's incorrect.

http://synbiohub.org/remoteSearch/objectType%3DCollection&?limit=10000

On Apr 5, 2017, at 8:25 PM, Michael Zhang<EMAIL_ADDRESS>wrote:
http://wiki.synbiohub.org/wiki/HTTP_API http://wiki.synbiohub.org/wiki/HTTP_API specifies how to search for specific collections, but not all collections. I put "key: collection, value:", which I'm pretty sure is why it's incorrect.
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub https://github.com/SynBioDex/SBOLDesigner/issues/127#issuecomment-291969472, or mute the thread https://github.com/notifications/unsubscribe-auth/ADWD96kQxvtY-QAIKY1Wug4sM_WjYkiUks5rs-qYgaJpZM4LsLBA.

I see.
				SearchQuery query = new SearchQuery(); 				SearchCriteria crit = new SearchCriteria(); 				crit.setKey("objectType"); 				crit.setValue("Collection"); 				query.addCriteria(crit); 				List<IdentifiedMetadata> results = stack.search(query);
This is how I am constructing the query, and it's throwing:
com.google.gson.JsonSyntaxException: java.lang.IllegalStateException: Expected BEGIN_ARRAY but was STRING at line 1 column 1 path $
Do you see what I'm doing wrong?

I think you might need to set the offset and limit.

On Apr 5, 2017, at 10:08 PM, Michael Zhang<EMAIL_ADDRESS>wrote:
I see.
SearchQuery query = new SearchQuery(); SearchCriteria crit = new SearchCriteria(); crit.setKey("objectType"); crit.setValue("Collection"); query.addCriteria(crit); List results = stack.search(query);
This is how I am constructing the query, and it's throwing:
com.google.gson.JsonSyntaxException: java.lang.IllegalStateException: Expected BEGIN_ARRAY but was STRING at line 1 column 1 path $
Do you see what I'm doing wrong?
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub https://github.com/SynBioDex/SBOLDesigner/issues/127#issuecomment-291997682, or mute the thread https://github.com/notifications/unsubscribe-auth/ADWD974pDtMF4OSevx2SNFgmVT0QS3pWks5rtALWgaJpZM4LsLBA.

Yup, that fixed it.  I've pushed a working (maybe) collections browser.

I've added a better-looking way to select collections, but the submit call now hangs.  I've checked that I'm forming the comma separated collections string correctly, and it appears that I am.

I have updated libSBOLj and submit now works, but I can't tell if it's being put in the correct collections (given I choose extra collections).  Is there an easy way to tell from the SynBioHub submissions/part page?

I think it works!  Collections show up properly after making the parts public in synbiohub/submissions.


fix: safety iterating #2935

Motivation
ConcurrentHashMap didn't guarantee the iterator consistency with entries...
Make sure that:

[x] You have read the contribution guidelines.
[ ] You have created a feature request first to discuss your contribution intent. Please reference the feature request ticket number in the pull request.
[x] You applied code formatting rules using the mvn formatter:format target. Don’t submit any formatting related changes.
[ ] You submit test cases (unit or integration tests) that back your changes.

Hey @Roiocam
is this related to #2935?


[super-agent] Improve helm chart UX (NR-281093)

Is this a new chart
No
What this PR does / why we need it:
super-agent chart got the config from a field called content that added any arbitrary data to a config map. This is supposed to control this configMap and render it using values from Helm's values.yaml.
It also adds some protections so a customer is not able to use it on unsupported environments.
Special notes for your reviewer:
This PR is blocked until #1463 is merged.
Checklist

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

I had to remove from the chart the test for cluster's Kubernetes version as it was colliding with the CI so I moved it to the PR #1471

The upgrade path is failing with this error:

The upgrade path is failing with this error:
Error: UPGRADE FAILED: template: super-agent/charts/super-agent-deployment/templates/deployment-superagent.yaml:24:34: executing "super-agent/charts/super-agent-deployment/templates/deployment-superagent.yaml" at <include (print $.Template.BasePath "/configmap-superagent-config.yaml") .>: error calling include: template: super-agent/charts/super-agent-deployment/templates/configmap-superagent-config.yaml:12:8: executing "super-agent/charts/super-agent-deployment/templates/configmap-superagent-config.yaml" at <include "newrelic-super-agent.config.content" .>: error calling include: template: super-agent/charts/super-agent-deployment/templates/_helpers.tpl:137:33: executing "newrelic-super-agent.config.content" at <include "newrelic-super-agent.config.agents.yaml" .>: error calling include: template: super-agent/charts/super-agent-deployment/templates/_helpers.tpl:18:8: executing "newrelic-super-agent.config.agents.yaml" at <fail (printf "Agent %s does not have agent type" $subAgentName)>: error calling fail: Agent open-telemetry does not have agent type

The previous values.yaml had this hardcoded: https://github.com/newrelic/helm-charts/pull/1466/files#diff-3bd49393f47bdfcb35b90c33237f384109da730bc892fa317ba8db58d29d7975L149-L158
Now, instead of having it in the values we have it as a helper: https://github.com/newrelic/helm-charts/pull/1466/files#diff-f752c985c9b04f49290d4a210d58d0981e8197c5825b7747defdc701fa4e1652R25-R34
The schema changed from this:
    superAgent:
      content:
        agents:
          open-telemetry:
            agent_type: newrelic/io.opentelemetry.collector:0.2.0
    subAgents:
      open-telemetry:
        content:
          chart_values:
            licenseKey: ${nr-env:NR_LICENSE_KEY}
            cluster: ${nr-env:NR_CLUSTER_NAME}

To this:
    subAgents:
      open-telemetry:
        type: newrelic/io.opentelemetry.collector:0.2.0
        content:
          chart_values:
            licenseKey: ${nr-env:NR_LICENSE_KEY}
            cluster: ${nr-env:NR_CLUSTER_NAME}

So the new chart is complaining that the old chart does not have type on the subagent.
The test is ok to fail as this is a breaking change. It should not have any repercussion to any customer as we do not support upgrades with --reuse-values.


chore: upgrade to supabase-js v2

What kind of change does this PR introduce?
Upgrades to the rc release of gotrue-js
Additional context
At this stage, this can be treated as a proof-of-concept of a technically different approach to the auth-helpers.
Refreshing behaviour is now handled by gotrue-js, and the session is synced by using shared cookies.
The nextjs example has been updated to work with this new approach. If we decide this is the direction we want to take, we can look at upgrading the other libraries/examples too.

Looks promising but we really need a better way to manage auth state. OAuth is one weak point of Supabase auth, especially when it comes to redirecting to index page and you immediately need the session state.

Will this fix the "getting logged out with a stale token" issue I have ?


Add Video Tutorials section to about page

Closes #253.
Adds video tutorials noted in linked issue to a new About page Video Tutorials section, with VimeoEmbed component. Video titles are listed in and linked in side menu bar (src/screens/About/contents.js).
Section description currently placeholder text, to be edited or deleted.

Here's a screenshot showing the end of the data export section, start of the new "Video tutorials" section, and related links in side nav menu:


Breaking lines with 80+ chars not working in all cases

This piece of code:
		iebaltab weight price   , grpvar(tmt)      pairoutput("p") ftestoutput("p")  replace savecsv("${ietoolkit}/run/output/iebaltab/iebt_csv")  savexlsx("${ietoolkit}/run/output/iebaltab/iebt_xlsx") ///
    browse  onerow  ftest feqtest  ///
     starlevels(.35 .222 .05)   grpcodes fix(rep78)  covar(mpg) ///
      totallabel("TOOOT") ///
    rowlabels("price Price in dollar @ weight vikt")  tblnonote

is corrected to:
    iebaltab weight price   , grpvar(tmt)      pairoutput("p") ftestoutput("p")  replace savecsv("${ietoolkit}/run/output/iebaltab/iebt_csv")  savexlsx("${ietoolkit}/run/output/iebaltab/iebt_xlsx") ///
    browse  onerow  ftest feqtest  ///
     starlevels(.35 .222 .05)   grpcodes fix(rep78)  covar(mpg) ///
      totallabel("TOOOT") ///
    rowlabels("price Price in dollar @ weight vikt")  tblnonote

which still contains a line with more than 80 chars. The correction feature for long lines doesn't work in this case

I think it is the same issue as described in #55 and in #55 I think it is more succinctly expressed. But maybe still keep this open and test if this is still is the case when #55 is solved

I think it is the same issue as described in #55 and in #55 I think it is more succinctly expressed. But maybe still keep this open and test if this is still is the case when #55 is solved

I'm checking this issue now. Can it be that you actually meant #54?


Feature/gh62 l1 videoport testprofile improvement

Added changes for L1 test code related to test profile improvement

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.1 out of 2 committers have signed the CLA.:white_check_mark: hari22yuva:x: thanushree-rYou have signed the CLA already but the status is still pending? Let us recheck it.


API docs for creating a repository using a template don't indicate that template can also be public

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
https://docs.github.com/en/rest/repos/repos#create-a-repository-using-a-template
What part(s) of the article would you like to see updated?
Currently, the docs say:

The authenticated user must own or be a member of an organization that owns the repository.

Ideally, docs would say, e.g.:

Either the repository must be public or the authenticated user must own or be a member of an organization that owns the repository.

Additional information
Confirmed with curl that a repo can indeed be created from a public template repository, even when authenticated user does not own that repository or its parent organization.

@dmalan Thanks so much for opening an issue! I'll triage this for the team to take a look :eyes:

Thank you, @cmwilson21!

I don't like grammar and rows , but I learned it with my favorite teacher, @ dmalan


View size

@naoyashiga How can I change the size of the view that pops up? To make it almost fill the whole screen?

https://github.com/naoyashiga/RPModalGestureTransition/blob/master/RPModalGestureTransition/BackgroundPresentationController.swift#L11
struct ModalSizeRatio {
    static let width: CGFloat = 1.0
    static let height: CGFloat = 1.0
}

Please try! 🙏

@naoyashiga Worked! Thanks 😃


Elaborate (hypothetically) what the problem is if one does not have a proper HSM

HSM = Hardware Security Module (https://en.wikipedia.org/wiki/Hardware_security_module).
Right now digital keys for authentication are stored probably in root?

Keys are typically not stored in the raw.  Typically a cryptographic hash
is stored of the key itself with some algorithm identifiers and possible
salf/seed/initialization information.  On linux for example /etc/.passwd is
the file that contains users and passwords.  However not even a hash is
stored in this file.  An x is indicated to inform that the cryptographic
hash and salt information is stored in the shadow file i believe is in
/etc/shadow.  So when a process attempts to check user login information,
the application must be allowed to call the the kernel cryptographic
functions and access the shadow and password file.  Linux makes available a
standard API do perform this method.
In a case in which an application is managing all its own secret
information i.e. no key chain or API call to standard kernel operations the
secret material is likely to reside in a static memory location; albeit
encrypted until needed (i.e. it is recommended that when an application
receives key material that it encrypts it immediately to provide a level of
obfuscation to a process that may happen to be snooping on the memory that
the process is operating on.
HSMs can provide the same, or operate on the keys in the plain as long as
there is no software that allows exfiltration of the key material itself
but this is dependent on the scenario.
On Tue, Jul 18, 2017 at 2:28 PM, Dejan Pangercic<EMAIL_ADDRESS>wrote:

Assigned #4 https://github.com/dejanpan/sros2/issues/4 to @akonecki
https://github.com/akonecki.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dejanpan/sros2/issues/4#event-1169181343, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACW8J13b9ypCeK2W8MrpqnDPTHjQM0G6ks5sPSNsgaJpZM4Ob8oF
.

An HSM allows the application to pose questions to the HSM without revealing the secret material to the querying application (it acts like an oracle where data and provided and acted upon by the HSM; HSM in the end will provide a result.  Depending on implementation it may or may not reveal success or failure and would be dependent on the application to catch by process a resultant answer.
HSM only provides a place to keep secrets and possibly operate on the secret material within a Trusted Execution Environment.  The access to the HSM is assumed trust-worthy and is typically managed by the Kernel that would resemble something similar to access to the passwd/shadow files in linux.


add record type for communities

A brainstorming idea to add a communities record type to the map?

 add record type for communities


Remove acronyms & abbreviations

Remove the acronyms and abbreviations clause.

The links verification failure will be fixed by the tool when this is merged.


Dates is undefined error when visiting course detail page that has a project without deadlines

I think this was fixed by a pr


Also consider string interpolation

The above example only works when logging a string without using string interpolation. For example the following log statement: log.debug("This is a log from {}", "David"). This would result in event.getMessage() returning "This is a log from {}" while you would like to see the processed form "This is a log from David". The getMessage().toString() only returns the former while toString() on the event will return the latter. Furthermore the toString() method is redundant on getMessage() as it already is a string.

Thanks @GoossensMichael I've merged the PR.


Local env | Not Streaming to Kinesis Stream

Hello,
Am a newbie to Kinesis. Have tried to setup the project in all three mentioned ways in the readme but cannot make it to work.
Local Env: I have setup all required versions and did intall the project.

Did you first deploy the Cloudformation stack via the command line using ./deploy_example.sh?
Testing in local environment will only work once the stack deployed to AWS via command line.
Please follow all instructions in the previous section of the README ("Command Line Deployment"), and install all dependencies needed for development and testing.

Thanks for responding.
Yes I tried that but it fails

Seems like you'll need to sign-in to the AWS console, delete the existing CloudFormation stack (because it's stuck in a bad state), and start fresh.


Chat Notification popup duration

I have attempted to change the duration both privately and administratively. The setting does not seem to be working. Is there a configuration file I can set the duration in? 3 seconds pop-up notification is ridiculous.

here the same, i set to 0, to never dissapear the notification until the user clicked on it, but not working.

I did some research and it looks like Windows notification duration can be only controlled through Windows registry, i.e. not by applications directly, and can be set only to up to 5 minutes. In order to prevent chat messages from being overlooked (which happens constantly because users not always sitting in front of their PC just don't see the small number in the tray) the only possibility I can think of is to make the Rocket Chat client blink in the taskbar (and optionally in the tray), even when minimized to the tray. Blinking the tray only is also not clearly visible.

This is a huge problem.  Once in the tray the message is missed 90% of the time.  The icon doesn't even blink or anything, nothing pops up, and having a tiny counter in the corner of the screen doesn't call your attention.

We planned to use RocketChat for internal communication in our small company but now I have completely paused its implementation here until this issue is solved. However nobody at the development team seems to care. For me this is not an enhancement request, it is something that needs an urgent fix! If users don't see the messages the whole thing is useless. I just don't understand how others are using it reliably. Again, please add an option to make the program blink in Windows task bar so people can clearly see that there is a new message, or make the program output its own notifications without having to rely on Windows as this is not configurable.

Desktop notification duration (or blinking taskbarsymbol) is a very important feature! If users don't see the messages, they are going to use emails with lots of cc's again :(

Hello, we will be tracking the blinking icon feature request here: https://github.com/RocketChat/Rocket.Chat.Electron/issues/2525
I will be closing this issue because there are duplicated ones.


Link presets on guns page and item pages

On the guns page, presets are listed as subrows for each gun:

On an item page, when an item has presets, a select box of all the presets is displayed:

Selecting a preset will navigate to that item page.
If an item doesn't have presets, the shortName displays instead:

.deploy to development

.deploy to development

.deploy to development

.deploy


Fee calculator formula in Salesforce is not correct

I think maybe we should just send the fee value from Python as a field, instead of trying to get Salesforce to calculate with its weird formulas.
Salesforce appears to calculate the fees wrong. That is, they don't always match what Stripe is actually charging.

We would need to check this against old data if we try to use the old field in Salesforce to reflect data from the form.

Tested with:

[ ] Visa one-time
[ ] Amex one-time
[ ] ACH one-time
[ ] Visa recurring
[ ] Amex recurring
[ ] ACH recurring

This works now, both in the master branch and the refactor branch. Going to close it.


! WARNING ! WARNING ! WARNING ! WARNING !

BlockDestructionService is VERY VERY BAD!!!!! you CANT have more active blocks than players around you!!!! if theres 1 player and he mines 10 blocks they are going to stay... why??


doesn't work when using patch: true

Hi there - I just ran into an issue where my password was not being hashed and instead saved in clear text. I tracked the issue down to patch: true being set in the options.
For reference, I use your wonderful bookshelf-modelbase plugin as well. Through that, I was calling:
model.update(...)
which resulted in the password not being hashed.
I'm not sure if this is a plugin issue or an underlying Bookshelf issue, but wanted to report it.
Thanks!

@jamesdixon to clarify you were using Model.update(data) and the data was not being encrypted?


Fix Sass not building in last release

Fixes #672
This moves the V-Select styles to the component level to allow it's import to be resolved without breaking anyone's custom Sass build system.

@xaksis For you to  review/merge/build/release.


Db2 is not listed when trying to create new database connection

Once the SQLAlchemy dialect ibm_db_sa and driver (ibm_db) for Db2 installed, it should list Db2 as available database.
Expected results
When creating a new database, It should list Db2 as an available database.
Actual results
Although both ibm_db_sa and ibm_db is installed, it doesn't list Db2 as available database.
Screenshots

How to reproduce the bug

Go to Menu 'Data'
Click on 'Databases'
Click the button '+ DATABASE'
In the popup modal, select 'SUPPORTED DATABASES'
IBM Db2 should be listed but actually not

Environment
(please complete the following information):

browser type and version:
superset version: 1.3.0
python version: 3.8.12
node.js version: v14.17.6
any feature flags active:

Checklist
Make sure to follow these steps before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context
It works in Superset 1.2.0 but only appear in v1.3.0. So I found two causes:

https://github.com/apache/superset/pull/15587
https://github.com/apache/superset/pull/14295

Debugging
I've located the problem to these lines of code:
https://github.com/apache/superset/blob/63aadd3fe492c39d8949fe2510bd521669e6869d/superset/db_engine_specs/__init__.py#L132-L146
The problem is it only recognize driver by engine name, not engine alias. The fact is the SQLAlchemy dialect for Db2 has engine name ibm_db_sa while the latest name db2 is listed in its entrypoints. So when loading installed drivers, it will populates drivers with key ibm_db_sa, but when populating available_engines, it only use the engine name db2.
If I revert https://github.com/apache/superset/pull/14295 it starts to work.
Or adding the below logic for function get_available_engine_specs():
    for engine_spec in load_engine_specs():
        driver = drivers[engine_spec.engine]
        # support lookup driver via engine aliases.
        if not driver and engine_spec.engine_aliases:
            for alias in engine_spec.engine_aliases:
                driver = drivers[alias]
                if driver:
                    break

        available_engines[engine_spec] = driver

So either option works for me in my dev environment.

@shawnzhu Thanks so much for detailed repro steps and debugging. Please do open a PR with your proposed solution to fix forward. 🙏

I can verify the fix from #16800 works locally from master branch.
@junlincc @villebro thanks for the attention and fast response!

Thanks for the fix @shawnzhu! Closing


Feature request: Export InboxId from index

Is your feature request related to a problem?
Export InboxId from index.ts as well
Describe the solution to the problem
No response
Describe the uses cases for the feature
No response
Additional details
No response

Hey semi-related to this it would also be nice to expose a static version of the rust getInboxIdForAddress method but maybe I should just create a separate issue?

@peterferguson I made a separate issue for this https://github.com/xmtp/xmtp-react-native/issues/426


docs: issue template

let me know if you think we can add/remove stuff from this...

I'm all for this. We should probably add similar templates for lipgloss and bubbles, too.