toil job object has no fileStore attribute

With the latest version of toil-vg (as of commit 67bf867e0044a294e1cb580ed94116a5e92cae6c) I'm failing to write input files to the toil job fileStore attribute since it appears to not exist.
Command:
toil-vg run --realTimeLogging --logInfo --logDebug --reads_per_chunk 8000 --call_chunk_size 20000 --index_mode gcsa-mem --gcsa_index_cores 2 --kmers_cores 2 --alignment_cores 2 --calling_cores 2 --vcfeval_cores 2 --fastq '/home/markellocj/test_input/small_sim_reads.fq.gz' --graphs '/home/markellocj/test_input/small.vg' --chroms x --config $CONFIG --workDir '/home/markellocj/test_work_dir' --container Singularity --force_outstore '/home/markellocj/local-testvg-test1' 'sample' ${OUT_DIR}
Where /home/markellocj/test_input/small.vg and /home/markellocj/test_input/small_sim_reads.fq.gz are the same input files as are pulled from the aws s3 bucket in the first unit test. The config file is generated straight from toil-vg generate-config
Traceback:

cn3092 2017-07-13 18:15:27,954 MainThread INFO toil.lib.bioio: Root logger is at level 'DEBUG', 'toil' logger at level 'DEBUG'.
cn3092 2017-07-13 18:15:27,959 MainThread DEBUG toil.jobStores.fileJobStore: Path to job store directory is '/home/markellocj/local-testvg-test1'.
cn3092 2017-07-13 18:15:27,961 MainThread INFO toil.jobStores.abstractJobStore: The workflow ID is: '6ef557d5-5507-4ccb-90cc-371e1109ffe5'
cn3092 2017-07-13 18:15:27,962 MainThread INFO toil_vg.vg_common: Importing /home/markellocj/test_input/small.vg
Traceback (most recent call last):
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_data_dir/bin/toil-vg", line 11, in
sys.exit(main())
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_data_dir/lib/python2.7/site-packages/toil_vg/vg_toil.py", line 286, in main
pipeline_main(context.to_options(args))
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_data_dir/lib/python2.7/site-packages/toil_vg/vg_toil.py", line 342, in pipeline_main
inputGraphFileIDs.append(import_to_store(toil, options, graph))
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_data_dir/lib/python2.7/site-packages/toil_vg/vg_common.py", line 278, in import_to_store
return  write_to_store(None, options, path, use_out_store, out_store_key)
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_data_dir/lib/python2.7/site-packages/toil_vg/vg_common.py", line 300, in write_to_store
return job.fileStore.writeGlobalFile(path)
AttributeError: 'NoneType' object has no attribute 'fileStore'

@adamnovak It looks like changes made in the write_to_store function in the vg_common.py module on commit dd60d673f706e55907f86b5ded279b13fb9fa633 broke my tests. I'm not sure how my local run differs from the unit tests.

Sorry, I missed that import_into_store is based on passing a null job around, and my changes to always put things in the file store don't work if you try and use None as a job.
I've redone import_into_store to work like the other functions in #246, so now it shouldn't crash like this.

@adamnovak Getting a new issue that's likely related to this issue. It's now no longer recognizing input files as existing.
Command (same as before):
toil-vg run --realTimeLogging --logInfo --logDebug --reads_per_chunk 8000 --call_chunk_size 20000 --index_mode gcsa-mem --gcsa_index_cores 2 --kmers_cores 2 --alignment_cores 2 --calling_cores 2 --vcfeval_cores 2 --fastq '/home/markellocj/test_input/small_sim_reads.fq' --graphs '/home/markellocj/test_input/small.vg' --chroms x --config $CONFIG --workDir '/home/markellocj/test_work_dir' --container Singularity --force_outstore '/home/markellocj/local-testvg-test1' 'sample' ${OUT_DIR}
Traceback:

cn3092 2017-07-13 20:25:18,486 MainThread INFO toil.lib.bioio: Root logger is at level 'DEBUG', 'toil' logger at level 'DEBUG'.
cn3092 2017-07-13 20:25:18,490 MainThread DEBUG toil.jobStores.fileJobStore: Path to job store directory is '/home/markellocj/local-testvg-test1'.
cn3092 2017-07-13 20:25:18,491 MainThread INFO toil.jobStores.abstractJobStore: The workflow ID is: '8ffc439f-93a7-482e-91ea-6a61024eafcc'
cn3092 2017-07-13 20:25:18,493 MainThread INFO toil_vg.vg_common: Importing /home/markellocj/test_input/small.vg
cn3092 2017-07-13 20:25:18,504 MainThread DEBUG toil.jobStores.abstractJobStore: Unable to import 'toil.jobStores.azureJobStore' as is expected if the corresponding extra was omitted at installation time.
cn3092 2017-07-13 20:25:18,617 MainThread INFO toil_vg.vg_common: Importing file:///home/markellocj/test_input/small_sim_reads.fq
Traceback (most recent call last):
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_data_dir/bin/toil-vg", line 11, in
sys.exit(main())
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_data_dir/lib/python2.7/site-packages/toil_vg/vg_toil.py", line 286, in main
pipeline_main(context.to_options(args))
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_data_dir/lib/python2.7/site-packages/toil_vg/vg_toil.py", line 346, in pipeline_main
inputReadsFileIDs.append(import_to_store(toil, options, sample_reads))
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_data_dir/lib/python2.7/site-packages/toil_vg/vg_common.py", line 274, in import_to_store
out_store.write_output_file(path, key)
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_data_dir/lib/python2.7/site-packages/toil_vg/iostore.py", line 493, in write_output_file
shutil.copy2(local_path, temp_path)
File "/usr/local/Anaconda/envs/py2.7/lib/python2.7/shutil.py", line 130, in copy2
copyfile(src, dst)
File "/usr/local/Anaconda/envs/py2.7/lib/python2.7/shutil.py", line 82, in copyfile
with open(src, 'rb') as fsrc:
IOError: [Errno 2] No such file or directory: 'file:///home/markellocj/test_input/small_sim_reads.fq'

OK, what we have here is a disagreement between the Toil way of doing
things (where you import a URL) and the IOStore way of doing things (where
you can only import a local file path).
I think the right way to do things is going to be to import to the
fileStore, and then to immediately download the file from the fileStore and
stick it in the IOStore if we're also supposed to be dumping to the
IOStore. People are going to want to be able to use input data in, say, S3,
while still dumping to the IOStore.
On Thu, Jul 13, 2017 at 5:29 PM, Charles Markello<EMAIL_ADDRESS>wrote:

@adamnovak https://github.com/adamnovak Getting a new issue that's
likely related to this issue. It's now no longer recognizing input files as
existing.
Command (same as before):
toil-vg run --realTimeLogging --logInfo --logDebug --reads_per_chunk 8000
--call_chunk_size 20000 --index_mode gcsa-mem --gcsa_index_cores 2
--kmers_cores 2 --alignment_cores 2 --calling_cores 2 --vcfeval_cores 2
--fastq '/home/markellocj/test_input/small_sim_reads.fq' --graphs
'/home/markellocj/test_input/small.vg' --chroms x --config $CONFIG
--workDir '/home/markellocj/test_work_dir' --container Singularity
--force_outstore '/home/markellocj/local-testvg-test1' 'sample' ${OUT_DIR}
Traceback:
cn3092 2017-07-13 20:25:18,486 MainThread INFO toil.lib.bioio: Root logger
is at level 'DEBUG', 'toil' logger at level 'DEBUG'.
cn3092 2017-07-13 20:25:18,490 MainThread DEBUG
toil.jobStores.fileJobStore: Path to job store directory is
'/home/markellocj/local-testvg-test1'.
cn3092 2017-07-13 20:25:18,491 MainThread INFO toil.jobStores.abstractJobStore:
The workflow ID is: '8ffc439f-93a7-482e-91ea-6a61024eafcc'
cn3092 2017-07-13 20:25:18,493 MainThread INFO toil_vg.vg_common:
Importing /home/markellocj/test_input/small.vg
cn3092 2017-07-13 20:25:18,504 MainThread DEBUG toil.jobStores.abstractJobStore:
Unable to import 'toil.jobStores.azureJobStore' as is expected if the
corresponding extra was omitted at installation time.
cn3092 2017-07-13 20:25:18,617 MainThread INFO toil_vg.vg_common:
Importing file:///home/markellocj/test_input/small_sim_reads.fq
Traceback (most recent call last):
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_data_dir/bin/toil-vg",
line 11, in
sys.exit(main())
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_
data_dir/lib/python2.7/site-packages/toil_vg/vg_toil.py", line 286, in
main
pipeline_main(context.to_options(args))
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_
data_dir/lib/python2.7/site-packages/toil_vg/vg_toil.py", line 346, in
pipeline_main
inputReadsFileIDs.append(import_to_store(toil, options, sample_reads))
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_
data_dir/lib/python2.7/site-packages/toil_vg/vg_common.py", line 274, in
import_to_store
out_store.write_output_file(path, key)
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_
data_dir/lib/python2.7/site-packages/toil_vg/iostore.py", line 493, in
write_output_file
shutil.copy2(local_path, temp_path)
File "/usr/local/Anaconda/envs/py2.7/lib/python2.7/shutil.py", line 130,
in copy2
copyfile(src, dst)
File "/usr/local/Anaconda/envs/py2.7/lib/python2.7/shutil.py", line 82,
in copyfile
with open(src, 'rb') as fsrc:
IOError: [Errno 2] No such file or directory:
'file:///home/markellocj/test_input/small_sim_reads.fq'
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BD2KGenomics/toil-vg/issues/245#issuecomment-315237310,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE0_X2TQdyZtMUwVdtLvaMYLjTeR9ABBks5sNrZsgaJpZM4OXnjY
.

OK, I have a fix PR'd (with an actual test this time). Want to see if it
works for you, @cmarkello?
On Fri, Jul 14, 2017 at 9:41 AM, Adam Novak<EMAIL_ADDRESS>wrote:

OK, what we have here is a disagreement between the Toil way of doing
things (where you import a URL) and the IOStore way of doing things (where
you can only import a local file path).
I think the right way to do things is going to be to import to the
fileStore, and then to immediately download the file from the fileStore and
stick it in the IOStore if we're also supposed to be dumping to the
IOStore. People are going to want to be able to use input data in, say, S3,
while still dumping to the IOStore.
On Thu, Jul 13, 2017 at 5:29 PM, Charles Markello <
<EMAIL_ADDRESS>wrote:

@adamnovak https://github.com/adamnovak Getting a new issue that's
likely related to this issue. It's now no longer recognizing input files as
existing.
Command (same as before):
toil-vg run --realTimeLogging --logInfo --logDebug --reads_per_chunk 8000
--call_chunk_size 20000 --index_mode gcsa-mem --gcsa_index_cores 2
--kmers_cores 2 --alignment_cores 2 --calling_cores 2 --vcfeval_cores 2
--fastq '/home/markellocj/test_input/small_sim_reads.fq' --graphs
'/home/markellocj/test_input/small.vg' --chroms x --config $CONFIG
--workDir '/home/markellocj/test_work_dir' --container Singularity
--force_outstore '/home/markellocj/local-testvg-test1' 'sample'
${OUT_DIR}
Traceback:
cn3092 2017-07-13 20:25:18,486 MainThread INFO toil.lib.bioio: Root
logger is at level 'DEBUG', 'toil' logger at level 'DEBUG'.
cn3092 2017-07-13 20:25:18,490 MainThread DEBUG
toil.jobStores.fileJobStore: Path to job store directory is
'/home/markellocj/local-testvg-test1'.
cn3092 2017-07-13 20:25:18,491 MainThread INFO
toil.jobStores.abstractJobStore: The workflow ID is:
'8ffc439f-93a7-482e-91ea-6a61024eafcc'
cn3092 2017-07-13 20:25:18,493 MainThread INFO toil_vg.vg_common:
Importing /home/markellocj/test_input/small.vg
cn3092 2017-07-13 20:25:18,504 MainThread DEBUG
toil.jobStores.abstractJobStore: Unable to import
'toil.jobStores.azureJobStore' as is expected if the corresponding extra
was omitted at installation time.
cn3092 2017-07-13 20:25:18,617 MainThread INFO toil_vg.vg_common:
Importing file:///home/markellocj/test_input/small_sim_reads.fq
Traceback (most recent call last):
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_data_dir/bin/toil-vg",
line 11, in
sys.exit(main())
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_data_
dir/lib/python2.7/site-packages/toil_vg/vg_toil.py", line 286, in main
pipeline_main(context.to_options(args))
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_data_
dir/lib/python2.7/site-packages/toil_vg/vg_toil.py", line 346, in
pipeline_main
inputReadsFileIDs.append(import_to_store(toil, options, sample_reads))
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_data_
dir/lib/python2.7/site-packages/toil_vg/vg_common.py", line 274, in
import_to_store
out_store.write_output_file(path, key)
File "/home/markellocj/toilvenv_fix_new_test_rollback_use_data_
dir/lib/python2.7/site-packages/toil_vg/iostore.py", line 493, in
write_output_file
shutil.copy2(local_path, temp_path)
File "/usr/local/Anaconda/envs/py2.7/lib/python2.7/shutil.py", line 130,
in copy2
copyfile(src, dst)
File "/usr/local/Anaconda/envs/py2.7/lib/python2.7/shutil.py", line 82,
in copyfile
with open(src, 'rb') as fsrc:
IOError: [Errno 2] No such file or directory:
'file:///home/markellocj/test_input/small_sim_reads.fq'
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BD2KGenomics/toil-vg/issues/245#issuecomment-315237310,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AE0_X2TQdyZtMUwVdtLvaMYLjTeR9ABBks5sNrZsgaJpZM4OXnjY
.


fix: Ensure that we don't crash when attempting to generate SQL for a LINQ expression which performs static field or property access

Description
This PR should fix the issue raised in #3188 in which attempts to access a static field or property (such as DateTime.UtcNow) within a LINQ-to-SQL expression will result in a NullReferenceException. It does so by correctly handling the presence of a null member expression when evaluating field and property access.
Type of change
Please delete options that are not relevant.

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

Closing issues
To automatically close an issue: closes #3188

/azp run


Found manga cant be downloaded

Describe the bug
If you search for a Manga you sometimes cant download it cuz it doesnt display any info under Downloads (after selecting the time) and then it simply throws an error manga could not be found.
Reproduction steps

Select Source
Enter Name of more niche Manga
Select Manga
Enter Time
No Info Displayed
No Download

Expected behavior
It should find all Info it needs since it can find the Manga and it exists on the site.
Additional context
I used Manganato and Game Of Familia: Kazoku Senki.

Yeah i wonder if we will ever see this fixed....

@ElryWeeb @ghost
I think it is relate to this issus cause by special caracter.
After multiple test with mangal, it doen't seems to be a problem with the extractor because I can extrat the manga with delimiter using "inline" mode or "mini" manualy.
Maybe Kaizoku don't handle the conversion when trying to create the title with what mangal return.
Anyway, for the small number of manga that have a problem, I use the "mangasee" source witch works fine.


NAV 2017 RTM clickonce issue

We had the same issue at #119 already. I am getting the same error using IE and Edge trying to install the clickonce client. What's interesting is that I am getting a warning that the container is 85 days old and that i should pull a new one. Well, it was just pulled today, so I am assuming that you didn't have a chance to update this container in a while.
Tried it also with the latest 2017 build and i am getting the same error.
Could it be that - whatever you did to 2016 also needs to be done to 2017? I might be able to look into it a bit more tonight so that I can narrow it down.

Clickonce broke for 2016, 2017 and 2018 builds with the AAD authentication from last weekend.
Will be fixed this weekend.

Images have been rebuild with generic layer version <IP_ADDRESS>.
Use docker pull and docker inspect to validate that you have the right version and clickonce should work again.


Te anomaly change chart

adding change chart to anomalies view,
moving anomaly metric query to anomalies view,
linking self-service with eventlistener

lgtm


bucket list is not updated when switching between new and existing projects

I created a video that demonstrates the problems at https://www.dropbox.com/s/97i5z1pe4f79bv4/stateless-bucket-list-issues-01.mp4?dl=0
Problems Demonstrated in Video

[x] use dashes in bucket name.
[x] existing bucket list is not updated when switching from existing project to create new project.
[x] generated bucket name should always fit the bucket name restrictions.

looks correct now. thank you


Have pendingupgrade and update queues co-process

If one queue is empty, have it check the other on each loop.

Niko added a status2 to allow for this.


Adds ECCN response statement

Fixes #9012

Resolves https://bugzilla.mozilla.org/show_bug.cgi?id=1407061


Add support for key word argument matching with respond_to

First pass at the final part of #471, probably needs more specs but WDYT in general?
(Fixes #471 so it'll autoclose)

This is nearly a year old.  Gonna close it since it's so stale.  #471 is still open so hopefully someone will tackle that at some point.


Remove themes from Text teaser

We have created a couple of themes which fail Wcag colour checking in the new Text teaser.
Could we please remove/replace:
Mid orange with Red and Mid green with Red. I would like to keep these background colour options with the default black text colour instead.

@millianapia Great, thanks approved


[Emotion] Targeting components in styled() computes to NO_COMPONENT_SELECTOR css

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
I am trying to use Emotion's targeting of components inside styled() like in steps bellow, but it computes into:
<style data-emotion="css" data-s="">NO_COMPONENT_SELECTOR:hover .css-1in7nrd{...}</style>

Expected behavior 🤔
https://emotion.sh/docs/styled#targeting-another-emotion-component
Steps to reproduce 🕹
const B = styled(Box)();
const A = styled(Box)({
  [`${B}:hover &`]: {
    ...
  },
})

Context 🔦
I am transpiling with Typescript.
Your environment 🌎
System:
OS: Linux 5.15 Ubuntu 22.04 LTS 22.04 LTS (Jammy Jellyfish)
Binaries:
Node: 16.13.0 - /usr/local/bin/node
Yarn: Not Found
npm: 8.3.1 - /usr/local/bin/npm
Browsers:
Chrome: 101.0.4951.64
Firefox: 100.0.2
npmPackages:
@emotion/react: 11.9.0 => 11.9.0
@emotion/styled: 11.8.1 => 11.8.1
@mui/core: 5.0.0-alpha.54 => 5.0.0-alpha.54
@mui/icons-material: 5.6.0 => 5.6.0
@mui/lab: 5.0.0-alpha.67 => 5.0.0-alpha.67
@mui/material: 5.6.0 => 5.6.0
@mui/styles: 5.6.0 => 5.6.0
@mui/system: 5.6.0 => 5.6.0
@mui/x-data-grid-pro: 5.8.0 => 5.8.0
@mui/x-date-pickers: 5.0.0-alpha.2 => 5.0.0-alpha.2
@mui/x-date-pickers-pro: 5.0.0-alpha.2 => 5.0.0-alpha.2
@mui/x-license-pro: 5.10.0 => 5.10.0
@types/react: 17.0.43 => 17.0.43
react: 18.0.0 => 18.0.0
react-dom: 18.0.0 => 18.0.0
typescript: 4.6.4 => 4.6.4

Thanks indeed I was answered on the emotion Slack!


tes format with max. digits, see also https://github.com/yiisoft/yii2…

see https://github.com/yiisoft/yii2/issues/17708

Q
A

Is bugfix?
✔ (test fix)

New feature?
❌

Breaks BC?
❌

Tests pass?
✔ (should hopefully)

Fixed issues
#17708

Now this is really strange?! If not just plain wrong...
1) yiiunit\framework\i18n\FormatterNumberTest::testIntlAsScientific
Failed asserting that two strings are identical.
--- Expected
+++ Actual
@@ @@
-8.76543210988E16
+8.765432109880E16


RedisCommandExecutionException: MOVED

hello，i dont understand this codes ![image](https://user-images.githubusercontent.com/28784625/60074643-021dcd80-9756-11e9-9ad0-6cb96d1f5831.png) but ,this code have a exception ,please tell me thats reason

Please do not paste images. Please replace it with the actual stack trace.

ok，Let's look at where an exception is thrown  ; class is  LettuceFutures ,
` public static  T awaitOrCancel(RedisFuture cmd, long timeout, TimeUnit unit) {
    try {
        if (!cmd.await(timeout, unit)) {
            cmd.cancel(true);
            throw ExceptionFactory.createTimeoutException(Duration.ofNanos(unit.toNanos(timeout)));
        }
        return cmd.get(); // Command [type=SETEX, output=StatusOutput [output=null, error='MOVED 9084 <IP_ADDRESS>:7003']]
    } catch (RuntimeException e) {
        throw e;
    } catch (ExecutionException e) { //java.util.concurrent.ExecutionException: io.lettuce.core.RedisCommandExecutionException: MOVED 9084 <IP_ADDRESS>:7003

        if (e.getCause() instanceof RedisCommandExecutionException) {
            throw ExceptionFactory.createExecutionException(e.getCause().getMessage(), e.getCause());
        }

        if (e.getCause() instanceof RedisCommandTimeoutException) {
            throw new RedisCommandTimeoutException(e.getCause());
        }

        throw new RedisException(e.getCause());
    } catch (InterruptedException e) {

        Thread.currentThread().interrupt();
        throw new RedisCommandInterruptedException(e);
    } catch (Exception e) {
        throw ExceptionFactory.createExecutionException(null, e);
    }
}`

please , See the integration code for the lettuce version 5.1.6 and 2.1.8 with spring-data-redis

Looks like your configuration uses Redis Standalone while you're connected to a Redis Cluster.

My configuration uses redis sentinel while i'm connected to a redis cluster,but when my configuration uses redis cluster  , it is well

Another ,my configuration uses redis sentinel while i'm connected to a redis cluster . It's also good to save values that are objects and not characters;Conversely, the same error occurs when the value is a character type

Closing this ticket because we cannot do anything here.


LenientSslConnectionSocketFactory breaks X509 Certificate Authentication

I have the problem that mutual authentication against ngnix/1.13.5 failed with: 400 No required SSL certificate was sent
I narrowed down the problem to the LenientSslConnectionSocketFactory, it seem that overrwriting this method and using an empty string for the target, does break the authentication for me.
public Socket createLayeredSocket(Socket socket, String target, int port, HttpContext context) throws IOException {
  return super.createLayeredSocket(socket, "", port, context);
}

I am not sure why this SocketFactory is in use but if I replace it with just the SSLConnectionSocketFactory authentication works for me again.
My configuration:
   * configure ssl = { keyStore: 'classpath:files/certificates/server.jks', keyStorePassword: 'asdfasdf', trustAll: 'true' };
Karate Version: 0.7.0.RC7

guessing the solution is to use the default SSLConnectionSocketFactory when user provides cert / trustStore.

@ptrthomas yes the solution for me was to use the default SSLConnectionSocketFactory

@leozilla if you can build from source or verify if the above commit / fix works that would be great

closing this as I cannot replicate and no response from submitter

https://stackoverflow.com/questions/55354081/sni-sslhandshakeexception-unrecognized-name
Found this issue while investigating a problem, will try to continue it.

@MercierAdrien can you open a new issue (refer this one) and I'll assign you to that


Fix Step 2: Bind SSL

@t106362512 commented on Mon Apr 08 2019
If SSL Cipher doesn't set the "AES128-SHA", it can't not working.
https://i.imgur.com/xY0o5tl.png

文件詳細資料
⚠ 請勿編輯此區段。 這是 docs.microsoft.com ➟ GitHub 問題連結的必要項。

ID: 77a94e6f-a799-494f-f2f9-171c1804e5f4
Version Independent ID: 0ca5e2ae-5889-61ef-3ce7-5c447f7519a6
Content: 設定 SSL 連線能力，以安全地連線到適用於 MySQL 的 Azure 資料庫
Content Source: articles/mysql/howto-configure-ssl.md
Service: mysql
GitHub Login: @ajlam
Microsoft Alias: andrela

Hi team,
I'm moving this here since the user is making a suggestion to be added to the instructions
best

Thanks for the feedback and bringing this to our notice . At this time we are reviewing the feedback and will update the document as appropriate .

@ajlam There appears to be a note added to this document (or included in the steps) that SSL is required, assuming based upon the notes.

Cannot reproduce issue.
@t106362512  - what version of MySQL Workbench are you using?

@t106362512 - what version of MySQL Workbench are you using?

@t106362512 - I am closing the issue for now. Please re-open if you run into this issue again.
#please-close

We will now proceed to close this thread. If there are further questions regarding this matter, please comment and we will gladly continue the discussion.


Add documentation generation with Typedoc

Add generated documentation from JSDocs in code to GitHub Pages.

I am kind of confused with how the pages workflow works. Right now, the docs are generated into the docs/docs folder but I never used parcel so I do not know how to set it up to serve the docs.
I also need to fix up how the documentation is structured because the modules are kind of messed up in the docs.
Lastly, do you want me to add a Semantic-UI theme to the docs or is the built-in one fine?

Actually, I moved the output to docs/build/docs so when the user navigates to /docs, they will land on the documentation home page. I also renamed the script to build:docs

Actually, don't merge this yet. I want to figure out how to use this: https://github.com/christopherthielen/typedoc-plugin-external-module-name

This PR changes some indentation to tabs instead of spaces, which I'm personally a fan of, just felt like I should mention it.

This PR changes some indentation to tabs instead of spaces, which I'm personally a fan of, just felt like I should mention it.

Oh, sorry. Must have been the auto-formatter in my workspace. I can change it back if you want.

Well personally I'd be a big fan of using tabs everywhere, but I only just merged my first (not very valuable) PR. It's up to @ci010 to decide 😉

Actually, this is my first PR too. Let me change it back.

I think it's okay for one file one page, as we don't have that much files and I think it won't be too mess.
About the theme, I don't find the semantic ui theme for the typedoc. We can keep the default theme right now, and improve it look and feel through iterations.
If you are ready to merge, just let me know

If you are ready to merge, just let me know

Go ahead.

Hmm, there seems to be something wrong with the production. All the paths to the modules are broken. Let me see if I can fix that.


File /usr/lib64/python27/subprocess.py raise child_exception OSError:No such file or directory

These keys j,k,h,l,is not work well in using,I often confused when I want to go back or ahead.
Also,I could not play music with it,there is still some errors display on the screen.
File /usr/lib64/python27/subprocess.py   raise child_exception OSError:No such file or directory

找到解决方案了么？

Seems like that's a bug related with Python on your OS

I also encountered this problem. solved by install mpg123.


fix(gettingstarted) malformed html

Fixes https://github.com/fabric8io/fabric8-ui/issues/1247

There were the following issues with your Pull Request

Commit: 342d2a500d30c7c25fe7096f0b28b16da2c1c644
Commits must be in the following format: %{type}(%{scope}): %{description}

Guidelines are available at https://github.com/fabric8io/fabric8-ui

This message was auto-generated by https://gitcop.com

@corinnekrych snapshot fabric8-ui is deployed and available for testing at https://fabric8-ui-fabric8-ui-pr-1248-fabric8-ui.int.rdu2c.fabric8.io

we will keep the form tag, fixed in #1250


Enable Hold Mode by default on specially crafted map?

Might be useful if there are maps out there that is made specifically for this ruleset. (cough cough like my map here cough cough)
Maybe checking the map's tags if there is somethin like "soyokaze", similar to how Rush! does.
Regardless, cool stuff :)

Resolved in release 2021.1111.0 in commit fa22ce1744d827651cf5838c0e88fddfa874d9f5.
Holds are now generated by default on all maps.


  Check minimumAliasedLineWidth and maximumAliasedLineWidth.

Not sure if this is exposing a bug in our code we previously missed, or if the test is just bad.

Is this one still needing to be resolved? If so, I can take it.

@dave-b-b Please confirm if this is still an issue by running npm test -- --browsers Firefox. See the Testing Guide for more info.
The actual fix for this may be more involved depending on the result of the test.

Okay. I'll take a look and update here.

These are the tests that failed. Sorry it took so long.
Failed tests.pdf

No worries, thanks @dave-b-b! It looks like this has escalated to a larger issue. I've updated the labels accordingly, and removed the "goof first issue" label.


allowing parent webview to close weblayer

On iOS, calling cobalt.webLayer('dismiss') on parent dismiss the weblayer.
On Android it works only if called from the weblayer webview itself.
I will propose here a small change that does the job

#135 #136 #137 all have their branche but are also merged into an experimental branche issues-135-136-137 about weblayers fixes

I disapprove this commit https://github.com/cobaltians/Cobalt-Android/commit/81844158f89133dd4db6e7398476658938f34dea

Done in weblayer-experiments branch


Update repository after transfer

See https://github.com/senzing-garage/garage-roadmap/issues/3

Pull requests:

https://github.com/senzing-garage/resolver/pull/141


cc-wrapper: fortran: enable stackprotector on M1

Was failing due to https://github.com/iains/gcc-darwin-arm64/issues/54, fix picked up in https://github.com/NixOS/nixpkgs/pull/186477. Reverts https://github.com/NixOS/nixpkgs/pull/151983, which originally addressed https://github.com/NixOS/nixpkgs/issues/140041.
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Semi-disaster. I built everything overnight and it's stuck:
[1/5107/5111 built (91 failed), 524 copied (2888.8/2890.7 MiB), 1827.1 MiB DL] building singular-4.3.1 (buildPhase): unset DISPLAY

I will do more building when I can, but enough built locally for me that I am inclined to mark this PR as ready for review and to try on staging.

Let's let it cook in staging.


How to connect to different Bluetooth OBD adater (than Carly)

Hello,
My Bluetooth OBD adapter identifies itself as "OBD BLE".
What can i do to get SmartOBD2 to use it? (Currently when i press the connect button, it starts a spinner and appears to wait forever)
Do i need to change RestoreIdentifierKey, DeviceName, serviceUUID and peripheralUUID in BleConnection.swift?

Hello RestoreIdentifierKey doesn't need change just the userdevice when I get home today I will implement a scan view to be able to search for and connect to a different ble device.

That would be awesome. Thank you.

That would be awesome. Thank you.

I have add a list of the devices broadcasting the serve FFE0 because that's the service I think most obd devices communicate. if your device is not showing up change let me know or change         centralManager?.scanForPeripherals(withServices: [CBUUID(string: service)], options: scanOption)
to         centralManager?.scanForPeripherals(withServices: nil, options: scanOption) this will make it it scans for all devices

Yes i had to use withServices: nil before it saw the "OBD BLE".
It has services 1804, Battery and FFF0 (all isPrimary = YES)
I also captured the peripheralUUID.

Yes i had to use withServices: nil before it saw the "OBD BLE". It has services 1804, Battery and FFF0 (all isPrimary = YES) I also captured the peripheralUUID.

still trying the figure out how I could be more flexible with the bluetooth adapters but for now change FFE1 to FFF0
guard let characteristics = service.characteristics else {
logger.error("No characteristics found")
return
}
self.discoveredServicesAndCharacteristics.append((service, characteristics))
    for characteristic in characteristics {
        switch characteristic.uuid.uuidString {
        case "FFE1":
            ecuCharacteristic = characteristic
            peripheral.setNotifyValue(true, for: characteristic)
            logger.info("Adapter Ready")
        default:
            if debug {
                logger.info("Unhandled Characteristic UUID: \(characteristic.uuid)")
            }
            if characteristic.properties.contains(.notify) {
                peripheral.setNotifyValue(true, for: characteristic)

            }
        }
    }


KeyPerFile Configuration Provider delimiter content

Fixes #9875

@scottaddie I think we can do this based on my own test and Hao's remark on the issue at :point_right: https://github.com/aspnet/Docs/issues/9875#issuecomment-445150430


Thermostat current temperature

It appears that the current temperature of thermostats are not updated. Subscribing to the MQTT topic, I can see that the API is sending the correct value back, but no updates occur in home assistant.

Apologies, please ignore.

Apologies, please disregard


only set language when necessary

the ios browser doesn't seem to have .languages, as it already seems to have a language set.
it was therefore breaking inside the setLanguage() function, when trying to evaluate navigator.languages - there also doesn't appear to be any need for the setLanguage() function when using iOS

scrap that sorry this is much better


change http to https

Docs Build status updates of commit 3d36120:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Advertising/bingads-13/guides/walkthrough-web-application-python.md
:white_check_mark:Succeeded

Advertising/hotel-service/get-started.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 0fd965a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

Advertising/bingads-13/guides/walkthrough-web-application-python.md
:white_check_mark:Succeeded

Advertising/hotel-service/get-started.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Add feedback section

@rikinsk I have pushed this functionality to issue-533 branch
https://github.com/hasura/docs/tree/issue-533
Take a look at it and let me know if there are any changes!


Some questions on AccessToken, pooling and performance

There have been some ongoing discussions about how to use access tokens efficiently with EF Core (see this blog post for an example), and I'm trying to figure out the right way to use access tokens efficiently. Since I'm a beginner I wanted to try to clear up a thing or two :)

Once a physical connection has been established to SQL Server/SQL Azure, does it stay valid until physically closed, or is some sort of access token refresh/renegotiation have to occur? In other words, is the access token only important for initially creating/opening the connection, and no longer relevant afterwards?
From a cursory look at the SqlClient source code, it seems that the access token (and also SqlCredential) is part of the pooling key (SqlConnectionPoolKey). If I understand correctly, this means that if the access token expires and needs to be rotated, a new pool is created, and all the physical connections in the old pool would never get used again, is that right? If so, I'd be interested in understanding why - would it not be possible to omit the access token from the pooling key, reusing the same pool as the token changes during the lifetime of an application? Is this a sort of security measure to require code using SqlClient to always know the access token, i.e. not return an already-open, pooled connection to code which doesn't know it?
My actual concern here is with the performance implications of getting an access token each time an SqlConnection needs to be opened. Since this may be a heavy operation (i.e. with network traffic), this could be significant, especially where the whole point of pooling is to avoid heavy operations on Open.
Depending on the answers to the above, would it make sense to imagine SqlConnection accepting a callback that returns an access token, which gets called only if a new physical connection needs to be created? For pooling scenarios, this callback would never be invoked, saving the overhead of fetching a token. With the current situation, user code has no way of knowing whether calling Open will create a new physical connection or not (as pooling is an internal concern), and so must always populate the AccessToken property via what is potentially a heavy operation.

Thanks for taking the time to read and answer the above...!
/cc @ajcvickers

Note: @ErikEJ has just referred me to https://github.com/dotnet/SqlClient/pull/730#issuecomment-705307179, which may be relevant in this context - could this be used to only acquire access tokens when they are actually needed?

Hi @roji
Let me provide you answers as per my best understanding:

Once a physical connection has been established to SQL Server/SQL Azure, does it stay valid until physically closed, or is some sort of access token refresh/renegotiation have to occur? In other words, is the access token only important for initially creating/opening the connection, and no longer relevant afterwards?

The answer is No. Once a physical connection is created, as long as it is not "RESET" (happens during reactivation of pooled connection), it will stay valid. if it is being reset and access token is not valid in that case, the server may choose to "Revalidate" the access token, and reject the connection request. This is what we also did in PR #443 to renew access token if it's expired when acquiring connection from pool. If user provides access token with SqlConnection API, it is "expected" to be always valid, whether or not pooling is enabled.

From a cursory look at the SqlClient source code, it seems that the access token (and also SqlCredential) is part of the pooling key (SqlConnectionPoolKey). If I understand correctly, this means that if the access token expires and needs to be rotated, a new pool is created, and all the physical connections in the old pool would never get used again, is that right?

Yes, it is required to be part of pool key, to avoid re-using connections that were created with an "expired" access token, as Server will revalidate, and that should be "expected". The revalidation is not clearly defined yet but will likely become more strict in future. We discovered it happened every 10 hours in a span of "active" physical connection when connections were reset. But we must expect it to occur anytime a new connection is created/pooled connection is activated and when that happens "access token" must be "valid".

Depending on the answers to the above, would it make sense to imagine SqlConnection accepting a callback that returns an access token, which gets called only if a new physical connection needs to be created? For pooling scenarios, this callback would never be invoked, saving the overhead of fetching a token.

A callback would be always needed to be called, as I mentioned above the importance of acquiring access token even when activating a pooled connection. We can expect the user will update their access tokens when opening a pooled connection if they're expired or near expiry.

Also to mention, there are possibilities of accepting a function callback and acquiring access token from the callback as and when needed. But that would require driver to validate access token everytime before making a connection request which is possible using JWT APIs and if token is found expired, driver could acquire fresh from callback.
It would become similar to design of AzureKeyVaultProvider that accepts authenticationCallback in it's constructor that only provides an implementation and not an actual access token.
The new API trends are however moving ahead to a Credential based construction patterns as most APIs in Azure.Identity library are designed now. Would recommend going through those once.

Thanks for the details responses @cheenamalhotra.

The revalidation is not clearly defined yet but will likely become more strict in future. We discovered it happened every 10 hours in a span of "active" physical connection when connections were reset

So to be sure I understand... After 10 hours of a physical connection lifetime, SQL Server will trigger some sort of revalidation? Or does the revalidation happen only when SqlClient initiates a RESET as part of the pooling? In other words, say my application opens a new connection (with an access token), and doesn't return it to the pool for 2 days - is it guaranteed to stay usable and not to error because of token expiry?

A callback would be always needed to be called, as I mentioned above the importance of acquiring access token even when activating a pooled connection.

Again - just to be sure I understand: does this mean that every time a pooled connection is handed out (i.e. when SqlConnection.Open is called), the access token is sent to SQL Server as part of a RESET? Does this imply an extra database roundtrip for every pooled Open, and does this roundtrip need to happen even when access tokens are in use? I'm asking because my expectation when using pooling is for DbConnection.Open to be an extremely short operation with no networking.
The following paragraph from the SQL Server Connection Pooling docs:

If a connection exists to a server that has disappeared, this connection can be drawn from the pool even if the connection pooler has not detected the severed connection and marked it as invalid. This is the case because the overhead of checking that the connection is still valid would eliminate the benefits of having a pooler by causing another round trip to the server to occur. When this occurs, the first attempt to use the connection will detect that the connection has been severed, and an exception is thrown.

Although this paragraph talks about severed connections, it seems relevant (perf-wise) also for any roundtrips done for token revalidation etc.

After 10 hours of a physical connection lifetime, SQL Server will trigger some sort of revalidation? Or does the revalidation happen only when SqlClient initiates a RESET as part of the pooling?

Revalidation only occurs when "RESET" bit is set in header information, only when pooled connection is reactivated.
We have tested non-pooled connection lifetime too, they stay open forever, as long as they are active and not getting stale.

does this mean that every time a pooled connection is handed out (i.e. when SqlConnection.Open is called), the access token is sent to SQL Server as part of a RESET?

Let me explain more in detail.
So, let's say you created a new pooled connection and closed it, driver will pool this connection but the physical connection is already active on server side. When the driver reactivates this pooled connection, it's only done on client side by taking it out of pool stack and setting it's state Active. But when next query is executed, we ensure a "RESET" bit is set in TDS header with next command on this newly reactivated pooled connection to cleanup any stale state, as per standard behavior.
Now, Access Token information is already available with the server, since it was part of "login credential" with this which connection was instantiated. The server revalidates credentials whenever RESET bit is received after 10 hours, no matter what mode of authentication is chosen. If user specified username/password and let's say password expires after 5 hours, the server will revalidate credentials anytime 10 hours the first RESET is triggered from client. And if password is found expired, command will not execute and exception will be thrown making connection non-usable. All future commands on this connection will continue to fail.

@cheenamalhotra thank you very much, that's very helpful!
I guess I have one more question. When you say that the server "revalidates credentials" (after seeing a query with the RESET bit on), does that mean it requests the credentials again from the client (access token, user/password), and SqlClient sends it at that point? In other words, does SqlClient basically re-authenticate in that scenario (RESET bit on after 10 hours)? Or does SQL Server just check if the original credentials it received are still valid, and basically break the connection? I'm trying to understand if SqlClient has an opportunity to supply new credentials when revalidation occurs, or is the connection basically doomed 10 hours after the original credentials are no longer valid.
Thanks again for your patient and detailed responses.

SQL Server checks if the original credentials it received are still valid, and will throw exception because RESET bit is set when executing a new SqlCommand on the same physical connection.
I'm not sure if we have an opportunity after receiving exception to update login credentials on an active connection, since that would mean we need to execute Login Flow once again - same as a new connection request.
@saurabh500
Are you aware of any such possibility?

Thanks again - I think I have a clear picture, and the SqlClient behavior makes total sense (including why an access token callback would have to be called every time).
@saurabh500 I'm still interested if a re-login after expiration is at least a theoretical possibility - it could allow a single pool, without the access token being part of the key, and with the callback only being called when Login Flow is initiated again. But it really is more of an academic interest at this point.

Hello @cheenamalhotra  and @roji
The TDS protocol doesn't allow reauthentication on the same physical connection. So there is no protocol support for re-login on a physical connection. Any login today, requires a new physical TCP connection.
PS: I have been working on internal repositories only and not paying much attention to GH mentions. In case there is something that requires an answer from me, it would be best to ping me on Teams. I happened to have installed the github app and I was going through the mentions and this caught my eye.

Thanks for confirming @saurabh500
Sure, will ping you when needed :)

Thanks @saurabh500 and @cheenamalhotra, I have all I need here!
To sum it up, SQL Server revalidates the initially-provided credentials at RESET after 10 hours. Since there's no opportunity to rotate credentials with SQL Server (i.e. provide new ones), connections basically have a maximum lifetime of around 10 hours (assuming normal pooling). This means that to avoid errors, it's the user's responsibility to always provide a valid access token when calling Open, and SqlClient internally ensures this works by having separate pools for different access tokens.
For context, see https://github.com/dotnet/efcore/issues/13261#issuecomment-714923330 - it's bad perf-wise to actually fetch the token (i.e. call Azure.Identity GetTokenAsync) every single time - it's the user's responsibility to cache tokens. Following the request for SqlClient to integrate with Azure.Identity made in https://github.com/dotnet/SqlClient/pull/730#issuecomment-700566412, I opened https://github.com/dotnet/SqlClient/issues/771. We have https://github.com/dotnet/efcore/issues/13261 tracking doing the same in EF Core, but it seems that SqlClient is the better place to do this (i.e. it would be usable by non-EF users).
One last academic note: I find it really strange that SQL Server revalidates credentials only when RESET is sent by client... In other words, from a security perspective it makes little sense for a server to require pooled connections to revalidate but to never revalidate a connection as long as its not returned to the pool... But that has nothing to do with SqlClient.


Tietuku upload adapter

An adapter for chinese users, since they can't use imgur.
Here's the API: https://translate.google.com/translate?hl=it&sl=zh-CN&tl=en&u=http%3A%2F%2Fopen.tietuku.com%2Fdocv1

Yes.We desperately need tietuku.


navigation.navigate is not a function

Please provide all the information requested. Issues that do not follow this format are likely to stall.
Description
Please provide a clear and concise description of what the bug is. Include screenshots if needed.
Please test using the latest React Native release to make sure your issue has not already been fixed: https://reactnative.dev/docs/upgrading.html
React Native version:
Run react-native info in your terminal and copy the results here.
Steps To Reproduce
Provide a detailed list of steps that reproduce the issue.

Expected Results
Describe what you expected to happen.
Snack, code example, screenshot, or link to a repository:
Please provide a Snack (https://snack.expo.io/), a link to a repository on GitHub, or provide a minimal code example that reproduces the problem.
You may provide a screenshot of the application if you think it is relevant to your bug report.
Here are some tips for providing a minimal example: https://stackoverflow.com/help/mcve

Hi @shabaz-tech, please fill out the issue template. Closing this issue.
If this an error from another module, not from React Native core please file an issue with their repo.


document to get started

thank you for working on such a project. I would like to help you to write the documentation for installation and getting started. Can we chat about it? How can I contact you?

Sure<EMAIL_ADDRESS>


Update openmetadata helm chart to address issue #209 and #210

Describe your changes :
This PR addresses issue #209 and #210.

Added initContainerResources object in values.yaml for initContainer resources of OMD pod/deployment. Also updated other required files as well.
Added automountServiceAccountToken in values.yaml for OMD pod/deployment. Updated required files accordingly.

Type of change :

[ ] Bug fix
[x] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have performed a self-review of my own.
[ ] I have tagged my reviewers below.
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] My changes generate no new warnings.
[ ] I have added tests that prove my fix is effective or that my feature works.
[x] All new and existing tests passed.

Reviewers
@akash-jain-10

@preetsshah are you planning on turning that into a draft to a PR. Any pending changes here?

Thank you for you contribution @preetsshah , it looks good from our side :)
Feel free to set it ready for review and we'll merge it and issue a new release including your changes.

Many thanks @tutte! Moving the PR to ready for review.


7pm 2 update sortcarets

Before this PR, there was custom code to implement the sort carets that appeared at the top of the Count Messages By User table on the  countmessages page of the app. After this PR, that custom code has been removed, because React Bootstrap Table 2 will put these in automatically as long as the correct CSS files are in place. To check this change, go to the countmessages page in the app, and try to sort the table. You should see new sort carets and Bootstrap Table.

The code passed all the test and is approved by TA. It LGTM.

Not sure which TA gave you this verbal approval, but this doesn't really count since it's not an official review on GitHub. I wouldn't be able to normally merge with this.
I have reviewed your code though and it looks okay. Going to merge now.


.NET 8

Tool name
.NET 8
Tool license
https://github.com/dotnet/core/blob/main/LICENSE.TXT
Add or update?

[X] Add
[ ] Update

Desired version
8.0.0
Approximate size
No response
Brief description of tool
No response
URL for tool's homepage
https://dotnet.microsoft.com/en-us/download/dotnet/8.0
Provide a basic test case to validate the tool's functionality.
No response
Platforms where you need the tool

[X] Azure DevOps
[X] GitHub Actions

Runner images where you need the tool

[ ] Ubuntu 20.04
[X] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[X] macOS 13
[ ] Windows Server 2019
[X] Windows Server 2022

Can this tool be installed during the build?
No response
Tool installation time in runtime
No response
Are you willing to submit a PR?
No response

Duplicate of https://github.com/actions/runner-images/issues/8797 - we track overall effort there


Publish to PyPI

Publish aiohttp_prometheus to PyPi.

@amitsaha it's over two years now.. any plans to finally publish it to PyPI?


Demos in readthedocs have broken links

Describe the bug
Follow-up from #1279
Notebooks were added to our readthedocs documentation by using nbsphinx, but seems like quite a number of links got broken. For example in loading-saving-neo4j:

I think there's two causes here:

some of the links have nested markup, in particular code https://github.com/spatialaudio/nbsphinx/issues/301, e.g. https://stellargraph.readthedocs.io/en/v1.0.0rc1/demos/basics/loading-networkx.html#Loading-from-many-graph-formats,-via-NetworkX

A NetworkX graph can be created in many ways, including manually via  `add_... methods https://networkx.github.io/documentation/stable/tutorial.html#creating-a-graph`__, and by reading from files in many formats. In this tutorial, we’ll only use the first one.

some files don't exist in the sphinx docs (yet), like the ../README.md in the screenshot in the issue (#1298)

Now that #1394 and #1401 have landed, I think most of these are fixed. Scrolling through some random notebooks, they all seem to look good.


Principals of type Application cannot validly be used in role assignments.

Getting below error on terraform apply:
Using object ID.
2 error(s) occurred:

* azurerm_role_assignment.ra1: 1 error(s) occurred:

* azurerm_role_assignment.ra1: authorization.RoleAssignmentsClient#Create: Failure responding to request: StatusCode=400 -- Original Error: autorest/azure: Service returned an error. Status=400 Code="PrincipalTypeNotSupported" Message="Principals of type Application cannot validly be used in role assignments."
* azurerm_kubernetes_cluster.test: 1 error(s) occurred:

* azurerm_kubernetes_cluster.test: Error creating/updating Managed Kubernetes Cluster "aks-cluster1" (Resource Group "aksrg1"): containerservice.ManagedClustersClient#CreateOrUpdate: Failure sending request: StatusCode=400 -- Original Error: Code="ServicePrincipalNotFound" Message="Service principal clientID: xxxxx-xxxxx-xxxxx-xxxxx not found in Active Directory tenant xxxxx-xxxxx-xxxxx-xxxxx, Please see https://aka.ms/acs-sp-help for more details."

I also tried with using tenant ID but getting PrincipalNotFound error.

I'm getting the same error (Principals of type Application cannot validly be used in role assignments) when trying to add deploy role assignments via ARM template
I've tried parsing the appId and objectId of the service principle
Any update on this ?

Getting the same errors as well.

Same issue in ARM template

I had the same issue. Paying more attention to the name of the variable in my terraform.tfvars definition I realized I was using the wrong object_id. Indeed it is required the one related to my subscription instead of the application object id.
I hope this is helpful.

And as an additional input for everyone else stumbling into this: When you create a new "app registration" in the Azure portal, actually two objects are created: An application object and a service principal object. The object ID which appears in the Azure portal is the application object ID, not the service principal object ID. So what you can do is:
Create a SP:
az ad sp create-for-rbac -n "sample-sp"
List details for the newly created SP:
az ad sp list --filter "displayName eq 'sample-sp'"

Use the object ID from the CLI as the object ID you enter in Terraform.

And as an additional input for everyone else stumbling into this: When you create a new "app registration" in the Azure portal, actually two objects are created: An application object and a service principal object. The object ID which appears in the Azure portal is the application object ID, not the service principal object ID. So what you can do is:
Create a SP:
az ad sp create-for-rbac -n "sample-sp"
List details for the newly created SP:
az ad sp list --filter "displayName eq 'sample-sp'"

Use the object ID from the CLI as the object ID you enter in Terraform.

So why not to show in the portal the Service Principal Object ID ?
Every day many folks have the same problem not just with AKS but with many other services in Azure!

You can also find the object id through the Enterprise Application.

And as an additional input for everyone else stumbling into this: When you create a new "app registration" in the Azure portal, actually two objects are created: An application object and a service principal object. The object ID which appears in the Azure portal is the application object ID, not the service principal object ID. So what you can do is:
Create a SP: az ad sp create-for-rbac -n "sample-sp"
List details for the newly created SP: az ad sp list --filter "displayName eq 'sample-sp'"

Use the object ID from the CLI as the object ID you enter in Terraform.

I tried this, but there was no key ObjectId in my json output.

same issue

same issue

I found the answer. The json response has an id key, which is actually the objectId.

I have to agree this is missleading. Application_id i understand, but ojbect_id i don't. And things changed we this requires the object_id (id) or the service principal runnning behind the application (App registration)
https://learn.microsoft.com/en-us/cli/azure/role/assignment?view=azure-cli-latest#az-role-assignment-create-optional-parameters:
--assignee-object-id Use this parameter instead of '--assignee' to bypass Graph API invocation in case of insufficient privileges. This parameter only works with object ids for users, groups, service principals, and managed identities. For managed identities use the principal id. For service principals, use the object id and not the app id.

I don't know if it's going to be useful but after creating an app registration you can access the service principal object Id from the enterprise application and this solved it for me. The object id of the enterprise application is different from the one in the app registration.


When I open the app it kicks me out!!

Dear GitHub,
I recently opened your game blocklauncher pro and it put me back on my home screen and kicking me out I have tried both of them and they do the same exact thing!! I'm trying to experiment with a very important mod!!  And if I don't practice for gaming con what am I going to do!!! If I dont get practicing with this mod what will I do with my life!! This will cause me to fail!!!
, sincerely
Neon wolf girl

@NeonGirlWolf, this will be more important if you can also inform MCPE version, device model, BL version, etc.

Be patient, @zhuowei is busy working on updating BL to support v0.15.0, close this issue.

If you updated to 0.15:
It has not been updated for 0.15. It takes time to re-hook into all the functions. It was stated on Twitter that the author is going to TRY to get it ported for 0.15 by the end of the week. Please close this issue as it is wasting the author's time. Instead of porting the program he is spending his time closing issue like this which are just fodder. Porting the program is going to happen, so again, please close this so you do not waste the author's time.

I can understand your point, but this happens 'cause the app IS effective and popular. I'm pretty sure the dev is proud (I hope so) of his creation and how popular it became. He can easily focus on what is top priority for him (even if it's not updating the app). He made "issues" available, so let them roll.

@charlescanato I do not understand what you mean.  He did not make the issues.  MCPE was updated.
This does NOT happen because it is popular.  It happens because of the culture we have provided everyone.  As a comparison, among the issue trackers I frequent, the Mojang one is most comparable to this one from a content and size.  The issues posted there are quite different than these.  These seem very childish, where the posts on the Mojang Jira tracker are helpful.  I will say they also have more moderators, so that helps, but the point is, that we should be respectful, not childish.
As such, I am trying to let people know that all they are doing is making the delay longer by "let them roll", now instead of working on BL, he is spending time going through comments that have not only been reported, but some have been rather nasty with threats...which I do not understand since most people that seem to be making the threats are using the FREE version.  I'm not saying ALL are threats, so please do not take that away.  I'm just saying if people had some restraint, it will come out as quickly as it can.  Follow Zhuowei on Twitter and you will be aware of what is going on and be able to understand that he is not sitting here waiting for someone to complain before updating.  He has been working on it for a while and with the drastic changes to the game between 0.14 and 0.15, there has been a LOT of additional effort that is not usual for updating BL betwixt versions.

@zedman2000 I meant Zhuowei has made THIS "issues" page available. ;-) I totally get your point, as well as I perfectly understand that this can be a disturbance for Zhuowei. But it's also understandable that if you provide a contact channel for a product whose users are not really mature into issue reporting, you can't expect everyone to be perfectly in line with these goals or not to disturb you. To be true, if I had any product designed for Minecraft, I'd expect to see what we have here.
But I'm not trying to win a debate here, just exposing my point. As I said, I do understand yours and agree with it to a certain extent.
Nothing will ever be a good replacement for respect AND wisdom. What we see here is typical Facebook teenager conversation style. For Mojang, moderation is KEY. What we see here is very similar to what you can see on StackExchange (Arqade), you end up quitting following it 'cause even as a user, it's tiresome to check that 95% of the "Issues" are useless content.


migration: enable migrating from non-root to root

What this PR does / why we need it:
The migration code allows VMIs to migrate from root to non-root, but not the other way around.
Sounds like an oversight to me, this PR addresses that.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes https://bugzilla.redhat.com/show_bug.cgi?id=2091911
Special notes for your reviewer:
Will write unit/functional test(s) if preliminary reviews are positive.
Release note:

Non-root VMs will now migrate to root VMs after a cluster disables non-root.

/approve
Other than the minor nit mentioned by acardace, this looks to be in good shape.

/retest

/cherrypick release-0.53


Transformers model as Judge

#153
Transformers library added to llm as judge.
Only need is when the JudgeLLM class called, change judge model with transformers.
Ex.
JudgeLLM(
            judge_model_name="microsoft/Phi-3-mini-128k-instruct",
            template_path="src/lighteval/tasks/extended/mt_bench/judge_prompts.jsonl",
            multi_turn=True,
        )

Hi! Thanks for this PR! Can you fix your PR so that tests are passing?

Yes, I'll fix on a short time but Run tests gives error for
ERROR tests/test_main.py - huggingface_hub.utils._errors.HfHubHTTPError: 500 Server Error: Internal Server Error for url: https://huggingface.co/api/datasets/gsm8k/paths-info/e53f048856ff4f594e959d75785d2c2d37b678ee (Request ID: Root=1-6630cc27-222f10cf5f5b028e1ffcebcc;677b1d40-c6e5-4fc5-9718-6441f20a365c)
Is it about huggingface hub?

Thanks, I fixed code quality and pushed. We can re-run whenever you want

cc @NathanHB if you have the time to do a more in depth review

hey ! thanks for the fix, I will have the bandwitdh to test next week and will merge asap :)

Hi ! Closing for inactivity. But this PR was used as the base for #223 thanks !


Permission denied when trying to use image built by repo2docker

I've built a repository using repo2docker and the image that launches automatically works.
I want to use the built image again (with different flags passed to docker run for example).  When I try I see this
$ docker run -it --rm -p 8888:8888 r2d-2fusers-2fawatters-2frepos-2fcanine-5fbc-2f1513701909:latest
s6-mkdir: warning: unable to mkdir /var/run/s6: Permission denied

How can I use the image with the image without rebuilding it?
PS: This works:
$ docker run -it --rm -p 8888:8888 r2d-2fusers-2fawatters-2frepos-2fcanine-5fbc-2f1513701909:latest bash

for what it's worth (I'm still pretty much a novice with Docker and related concepts).

I figured this out myself.  Apparently the default run command in the image is not the jupyter command.  When I give the jupyter command explicitly
$ docker run -it --rm -p 8888:8888 \
    r2d-2fusers-2fawatters-2frepos-2fcanine-5fbc-2f1513701909:latest \
    jupyter notebook --ip <IP_ADDRESS>

it works.  So I guess I would like the default run command to be specified
to be jupyter notebook by the docker build.

There doesn't seem to be a good reason for not having a default command like you proposed.
Do you want to make a PR that adds a CMD to the dockerfile template we have?

@betatim : #176

Closing this. Thank you @AaronWatters for the PR and for closing this issue. 👍


individual neuron pruning (similar to tfkerassurgeon)

Just wanted to bring up that existing pruning packages (tfkerassurgeon and keras-surgeon https://github.com/Raukk/tf-keras-surgeon) are no longer supported or compatible with Tensorflow, so it'd be great if Tensorflow could implement their own simple individual neuron pruning capability. I'm aware of the existing Tensorflow APIs that focus on automated low-magnitude pruning(https://www.tensorflow.org/model_optimization/guide/pruning/comprehensive_guide), but being able to simply delete specific neurons/channels from layers or delete specific layers would be useful, and aid in the study of neural networks. For example, something like:
# delete layer_1 from a model
model = delete_layer(model, layer_1)
# delete channels 0, 4 and 67 from layer_2 in model
model = delete_channels(model, layer_2, [0,4,67])

This allows researchers and scientists much more flexibility in deciding how exactly they want to prune their model, and foster development of new pruning techniques

Any update on this?

Hi,
Thank you for opening this issue. Since this issue has been open for a long time, the code/debug information for this issue may not be relevant with the current state of the code base.
The Tensorflow team is constantly improving the framework by fixing bugs and adding new features. We suggest you try the latest TensorFlow version with the latest compatible hardware configuration which could potentially resolve the issue. If you are still facing the issue, please create a new GitHub issue with your latest findings, with all the debugging information which could help us investigate.
Please follow the release notes to stay up to date with the latest developments which are happening in the Tensorflow space.


Add automatic versioning and release process

@sschnabe As talked about yesterday, this adds a label-based automatic versioning process:

patch: 0.0.x+1
minor: 0.x+1.0
major. x+1.0.0
It also automatically releases(using the process already in place) the plugin and create a new github release. The release notes will be generated out of the commit-messages, thus squash-merging makes the notes more readable;)

Wrong branch....


Add Polish salutation

Dzięki! :)

@jhuleatt Make sure to DM or email me so I can make sure to give you credit for helping. I put my contact info in #7 when I closed that PR.


License

I'd like to use this chart internally. Can you provide me with a License so I can use it?

Done ✅


synced ltp and blktests to 20240902

Going to merge it as no exception found in our nightly runs. @libhe @linl-rh


why you still use the MSE loss before the NNFM loss in the style transfer stage

Hi Kai,
Thanks for your great work! I m quite confused that why you still use the MSE loss before the NNFM loss in the style transfer stage in opt_style.py.

This is for finetuning the appearance after we discard the view-dependent component.

Where are you discarding the view-dependent component?

Where is the discardment of the view-dependent component is happening in opt_style.py?


Refactoring Owner getPets function to use lambda functions

@RafaelFantinel Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@RafaelFantinel Thank you for signing the Contributor License Agreement!

We've had similar proposals in the past and decided not to merge them. This one has limited scope so it is a good pull request. Not sure we really need it though, still.

I understand, I see it as a good modification to discourage the use of the traditional "for" and encourage the use of lambdas functions, bringing greater clarity and evolution in code writing.


Add make shim to Linux super env

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[ ] Have you successfully run brew tests with your changes locally?

Add a shim for make that has been missing from the super env on Linux.
CC @sjackman and @iMichka

It looks to me as though shims/linux/super/make would call itself in an infinite loop (unconfirmed).

A question for the Homebrew developers. shims/mac/super/make adds O to HOMEBREW_CCCFG to enabled ENV.refurbish_args by default for any formula that uses make, but not for ./configure or any non-make build system. Does someone know why that is, rather than enabling ENV.refurbish_args by default for all build systems?

It looks to me as though shims/linux/super/make would call itself in an infinite loop (unconfirmed).

Ha! interesting observation. It might be the case. I guess we have to change line 7 to`
MAKE=$(type -p make)

or
MAKE=$(command -v make)

?

Perhaps, but you'll need to first exclude …/shims/linux/super from PATH.

@sjackman done

A question for the Homebrew developers. shims/mac/super/make adds O to HOMEBREW_CCCFG to enabled ENV.refurbish_args by default for any formula that uses make, but not for ./configure or any non-make build system. Does someone know why that is, rather than enabling ENV.refurbish_args by default for all build systems?

Wouldn't enabling refurbish_args make most checks in ./configure pointless since e.g. autotools will call cc with different flags to determine what kind of compiler, etc. it's running on?

Yes, ENV.refurbish_args would probably break ./configure. There's a check to prevent specifically that:
https://github.com/Homebrew/brew/blob/0d33aba0b7d8448e815a2eecc92417b46cb905f3/Library/Homebrew/shims/super/cc#L104
I'm more curious why ENV.refurbish_args is not use for other non-make build systems, or phrased another way, why it's used specifically for make.

Also interesting that ENV.refurbish_args is not used whenld is used for linking, but is used when cc is used for linking.

Presumably because make usually is separated from configure.

Why is/does this need to be different from the macOS shim?

we don't have xcrun

I see. The path removal feels a little cludgy; is there no easier way of doing that?

@maxim-belkin Ping?

How can I improve this PR?  Rewrite pathremove function?
I can make it specific to PATH var instead of any variable but I don't know if it will be an improvement.

@maxim-belkin It's fine with me, just a question. Merging, thanks again!

Thanks, Mike!


Support key aliases / hyper

I like binding all wm related commands to a hyper mod (cmd + alt + ctrl + shift) so I find myself having to type that out for every new hotkey:
...
cmd + alt + ctrl + shift - h : chunkc tiling::window --focus west
cmd + alt + ctrl + shift - j : chunkc tiling::window --focus south
cmd + alt + ctrl + shift - k : chunkc tiling::window --focus north
cmd + alt + ctrl + shift - l : chunkc tiling::window --focus east
...

It would be nice to have a way to define aliases like:
hyper := cmd + alt + ctrl + shift
hyper - h : chunkc tiling::window --focus west

Alternatively having hyper available as a mod key would probably also work...

It would be nice to have a way to define aliases

Agreed, no ETA for now though.

I've been thinking some more about this, and adding a built-in hyper keyword seems like the more appropriate solution.

It seems a curious decision to hard code the definition of hyper. Not every hyper key is the same.
For example, I prefer to use cmd + alt + ctrl - but not shift - to be my hyper key, which I have mapped in karabiner.
Why?
Because if hyper is matched to all available modifier keys, I have excluded them from being used with they hyper key, much limiting its usefulness.
An example from my config:
cmd + ctrl + alt - m : open -a whatsapp
cmd + ctrl + alt + shift - m : open -a signal

As you can see I have the same problem as the original post. Wouldn't it be nice to be able to define an alias, and write:
hyper - m : open -a whatsapp
hyper + shift - m : open -a signal

Obviously I could no longer use hyper as the name of my alias in the above example as that is now hardcoded to mean something specific. But allowing an alias to be defined in the config, without disturbing the current meaning of hyper in the code, would preserve backward compatibility while allowing for some flexibility.
Not all hyper keys are the same!

Agreed with @tigger04.

I don't have time to do support for this project. Feel free to implement said change on your own,


Upload CSVs

Problem
The UI can download CSVs, but there's no way for a user to upload a CSV directly onto the app without having using the CLI.
Solution
Build a UI CSV Uploader

Closed with https://github.com/influxdata/ui/pull/421


Dyn reconfig start angle max points scan

For https://github.com/PepperlFuchs/ROS_driver/issues/35#issuecomment-754384255
In the current code, the dynamic reconfiguration for start_angle & max_num_points_scan has no effect because these changes are not communicated to R2300 by set_scanoutput_config.
This commit has been tested for R2300 only, and no changes have been made for R2000.

@wsaihopfsg thanks for the contribution! Could you please resolve the conflicts?

@wsaihopfsg thanks for the contribution! Could you please resolve the conflicts?

@wsaihopfsg thanks for the contribution! Could you please resolve the conflicts?

I am done with this conflict resolution too. Thanks

@wsaihopfsg thanks for the contribution! Could you please resolve the conflicts?

I am done with this conflict resolution too. Thanks

@wsaihopfsg this PR is behind https://github.com/PepperlFuchs/ROS_driver/pull/51. I have pushed couple of commits to this PR.

@wsaihopfsg this PR is behind https://github.com/PepperlFuchs/ROS_driver/pull/51. I have pushed couple of commits to this PR.


Low delay playback of fragmented MP4

Other browsers, like. Chrome, contains heuristics to reduce playback delay/latency when encountering fragmented MP4 files with unknown duration. This is typically encountered in live video streaming. Opening a such video will cause "Video rendering in low delay mode." to be displayed in the chrome://media-internals/ page. Edge does not seem to support anything similar, and instead buffers extensively before starting playback.
Chrome implementation:

Detection logic: https://cs.chromium.org/chromium/src/media/formats/mp4/mp4_stream_parser.cc?type=cs&l=616
Usage: https://cs.chromium.org/chromium/src/media/renderers/video_renderer_impl.cc?type=cs&l=239

Steps to reproduce:

Build & run https://github.com/forderud/AppWebStream with "8080" as argument. The FFMPEG-based back-end currently works best.
Open localhost:8080 in web browser.
Observe video latency.

Hi @forderud, thanks for the feedback! However, this is not a Chakra issue but an Edge issue- please open an issue at https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/ and the Edge team members should be able to help you.

Thanks.
The issue have now been moved to https://developer.microsoft.com/en-us/microsoft-edge/platform/issues/17410767/


Create 0399-evaluate-division.java

File(s) Modified: 0399-evaluate-division.java
Language(s) Used: Java
Submission URL: https://leetcode.com/problems/evaluate-division/submissions/1275308294/

Important
Please make sure the file name is lowercase and a duplicate file does not already exist before merging.
Adapted this solution to Java

Hi. It will be great if you can please look at this at your convenience! @a93a

Hi! @felivalencia3 if possible, can you please take a look at this at your convenience Thank you so much.


NOT A PR

trying to find out which test causes memory leaks in azure because on my machine I don't have any issues

Lol .... remember you might maybe also use docker to get the same environment...did that a few times in the past to reproduce such errors

True.. I think it's the static initialisation of maps that can cause some odd behaviour in certain compilers. I also noticed that we use gcc-6.3 in CI, might be worth upgrading to 6.5 no?

The issue does seem to be the initialisation of the maps. I will fix this with a proper PR!


Add user to group not working.

Can not reproduce bug, and tried different ways to add the user to the group

Still doesn't work atm. Tried clicking in many different places nothing seems to do anything. Only time I can add users is during initial group creation, then after that it fails.

Just tested and I can add users on staging


I wonder something

Hi, when i write "npm install" on the cmd, i received a few console errors. Also i want to ask you a question more, can i install other a music npm? or  if i paste music name on the chat can i play music?

Hi @EnginZZZ,
could you describe the error that you get on the cmd/terminal ? Before typing npm install you should install ffmpeg or the bot will not work.

"Also i want to ask you a question more, can i install other a music npm?"

If you need to install another node package yes it is possible, you just need to remove the opusscript package from the moment this is used to streaming and receiving audio and replace it with another.

"or if i paste music name on the chat can i play music?"

This is only possible if you trigger the bot with a specific command with this bot, you can't paste a song name without a command.
For example:
$ytsearch [SongName]

@Cramenorn
First, thanks a lot i will try it.  :)


Fix potential overflow condition on 32-bit architectures.

Trivial fix for the fuzzers which can overflow on 32-bit architectures depending on the input data.

thanks for the fix


Add mus_f_part3 by Toby Fox

Song Name
mus_f_part3
Artist Name
Toby Fox
Source
Youtube
Youtube Link
RAC5sRU9ZmMc1B4N
Direct File Link
No response
Song ID
1
Start Offset [ms]
0
Extra
Created from Auto Nong.
Created for level: 107394763

reject
invalid YouTube link


network

could you release your prototxt of network  before the all code？
thx

I want to learn about the architecture of SGA、LGA and Guidance subnet.Thanks!


It takes several sets of "double clicks" to get OnlyM to come up on our Kingdom Hall computer

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):

Windows version
Other important information about your environment

@BroReeves I assume you are double-clicking a desktop icon to launch OnlyM. Once is all that is required unless you are not able to double-click reliably, in which case click once to highlight the icon and then press the RETURN key. If it doesn't load after a few seconds open the task manager and see if the OnlyM process has started. The application should start eventually (without the need to attempt to launch it multiple times), but if it takes a long time, review other processes in Task Manager to see what else may be happening to impede it.

Thank you brother.  We double click and the little blue circle shows then
stops and we wait and wait.  Then we try it again.  It usually starts in
three attempts but sometimes it takes another one or two.  I will check out
what you suggested.
Bob Reeves
Elko, NV  USA
On Thu, Nov 7, 2019 at 3:17 PM Antony Corbett<EMAIL_ADDRESS>wrote:

@BroReeves https://github.com/BroReeves I assume you are
double-clicking a desktop icon to launch OnlyM. Once is all that is
required unless you are not able to double-click reliably, in which case
click once to highlight the icon and then press the RETURN key. If it
doesn't load after a few seconds open the task manager and see if the OnlyM
process has started. The application should start eventually (without the
need to attempt to launch it multiple times), but if it takes a long time,
review other processes in Task Manager to see what else may be happening to
impede it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AntonyCorbett/OnlyM/issues/297?email_source=notifications&email_token=ANWYC3QFUHTQVWSILMQWFKTQSSOXZA5CNFSM4JKPB5UKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDOFJDY#issuecomment-551310479,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANWYC3XFOVJPBAVT7BZCKALQSSOXZANCNFSM4JKPB5UA
.

@BroReeves Hi. Is there anything further to report on this issue please?

Thank you brother.  We still have to try several times before the app comes
up.  I have started the app and quickly gone to task manager and see that
it is working but then suddenly it disappears and I try again.  I have done
this several times.
Bob Reeves
On Thu, Nov 28, 2019, 9:37 AM Antony Corbett<EMAIL_ADDRESS>wrote:

@BroReeves https://github.com/BroReeves Hi. Is there anything further
to report on this issue please?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AntonyCorbett/OnlyM/issues/297?email_source=notifications&email_token=ANWYC3W4KWKSM7Z4AHGMRY3QV76XHA5CNFSM4JKPB5UKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFNFZTQ#issuecomment-559570126,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANWYC3XJKLSNWXJA3FZH5L3QV76XHANCNFSM4JKPB5UA
.

@BroReeves Please would you send the log file?
OnlyM maintains a log (a simple text file) of activity which can be useful if you run into problems. The log is stored in the "Documents\OnlyM\Logs" folder.

Will do brother.  Tomorrow.
Thank you very much
On Thu, Nov 28, 2019, 1:42 PM Antony Corbett<EMAIL_ADDRESS>wrote:

@BroReeves https://github.com/BroReeves Please would you send the log
file?
OnlyM maintains a log (a simple text file) of activity which can be useful
if you run into problems. The log is stored in the "Documents\OnlyM\Logs"
folder.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AntonyCorbett/OnlyM/issues/297?email_source=notifications&email_token=ANWYC3UQ2ZIZDBQS6MRUBJTQWA3NJA5CNFSM4JKPB5UKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFNO6II#issuecomment-559607585,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANWYC3XS3E6SNHLJDGGUXL3QWA3NJANCNFSM4JKPB5UA
.

Thank you for your patience my brother.  I found the problem.  Someone
changed the mouse action from double click to single click and didn't
inform us.  So we were stopping the app with the second click.
Bob Reeves
Elko, Nevada
On Thu, Nov 28, 2019 at 2:16 PM Robert Reeves<EMAIL_ADDRESS>wrote:

Will do brother.  Tomorrow.
Thank you very much
On Thu, Nov 28, 2019, 1:42 PM Antony Corbett<EMAIL_ADDRESS>wrote:

@BroReeves https://github.com/BroReeves Please would you send the log
file?
OnlyM maintains a log (a simple text file) of activity which can be
useful if you run into problems. The log is stored in the
"Documents\OnlyM\Logs" folder.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AntonyCorbett/OnlyM/issues/297?email_source=notifications&email_token=ANWYC3UQ2ZIZDBQS6MRUBJTQWA3NJA5CNFSM4JKPB5UKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEFNO6II#issuecomment-559607585,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANWYC3XS3E6SNHLJDGGUXL3QWA3NJANCNFSM4JKPB5UA
.

Thanks for clearing this up.


Split quotes with @next/@prev are not rendering correctly

See quotes split across lines in emdPer_Appolonius; the desired output is a single opening quote and a single closing quote. This should be handled by the XSLT template on l. 244 of xhtml5_templates_tei_module.xsl, but it's not happening.

We have decided to transcribe quotation marks and tag them with <pc>. I have made a to-do item in Teams/LEMDO/Documenters to document this.


[SP-2605] - Backport of PDI-13789 - Intermittent ConcurrentModificationException in Database.getValueFromSQLType (5.4 Suite)

@brosander, @mbatchelor, review it please. This is a backport of  https://github.com/pentaho/pentaho-kettle/pull/1376. Also I backported a fix for incorrect test.

http://<IP_ADDRESS>:8080/job/wingman/29/

Build Commands:
Build:

ant -Dpentaho.coding.standards.version=1.0.4 -Djunit.jvmargs=-XX:PermSize=256m -f core/build.xml clean-all resolve jacoco publish-local

Cleanup:
rm -r ~/.ivy2/local || echo no publish local to remove
Changed files:
core/src/org/pentaho/di/core/database/Database.java
core/test-src/org/pentaho/di/core/exception/LookupReferencesExceptionTest.java
Newly Fixed Tests:
org.pentaho.di.cluster.SlaveConnectionManagerTest.shouldOverrideDefaultSSLContextByDefault:
junit.framework.AssertionFailedError
junit.framework.AssertionFailedError:
	at org.pentaho.di.cluster.SlaveConnectionManagerTest.shouldOverrideDefaultSSLContextByDefault(SlaveConnectionManagerTest.java:45)

Still Broken Tests:
org.pentaho.di.core.database.DatabaseConnectingTest.connect_GroupIsEqual_InParallel:
junit.framework.AssertionFailedError
junit.framework.AssertionFailedError: Each instance should have a unique 'copy' value
	at org.pentaho.di.core.database.DatabaseConnectingTest.connect_GroupIsEqual_InParallel(DatabaseConnectingTest.java:132)

Unit test coverage change:
org.pentaho.di.cluster.SlaveConnectionManager:

Complexity Change :  + 11.11%
Instruction Change :  + 34.78%
Line Change :  + 26.32%
Method Change :  + 16.67%

org.pentaho.di.cluster.SlaveConnectionManager.new X509TrustManager() {...}:

Complexity Change :  + 20.00%
Instruction Change :  + 12.00%
Line Change :  + 16.67%
Method Change :  + 25.00%

org.pentaho.di.core.database.Database:

Branch Change :  + .02%
Complexity Change :  + .01%
Instruction Change :  + .08%
Line Change :  + .19%

org.pentaho.di.core.database.Database.new Comparator() {...}:

Complexity Change :  + 100.00%
Instruction Change :  + 100.00%
Line Change :  + 100.00%
Method Change :  + 100.00%

org.pentaho.di.core.row.value.ValueMetaFactory:

Branch Change :  + 4.55%
Complexity Change :  + 5.71%
Instruction Change :  + 7.50%
Line Change :  + 7.78%
Method Change :  + 7.69%

org.pentaho.di.core.row.value.ValueMetaInternetAddress:

Complexity Change :  + 2.13%
Instruction Change :  + 1.00%
Line Change :  + 2.34%
Method Change :  + 9.52%

org.pentaho.di.core.row.value.ValueMetaNone:

Complexity Change :  + 66.67%
Instruction Change :  + 81.82%
Line Change :  + 80.00%
Method Change :  + 66.67%

org.pentaho.di.core.vfs.KettleVFS.new Runnable() {...}:

Branch Change :  + 50.00%
Complexity Change :  + 33.33%
Instruction Change :  + 60.00%
Line Change :  + 75.00%
Method Change :  + 50.00%


Replace temurin20 with openjdk20 as former is not available on Windows

Description
Replace temurin20 with openjdk20 as former is not available on Windows
Issues Resolved
https://github.com/opensearch-project/opensearch-build/issues/3425
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

https://build.ci.opensearch.org/job/packer-build/28/console
https://build.ci.opensearch.org/job/packer-build/29/console


travis: fail fast if --with-marvin fails with nose

This fixes issue with recent Travis runs which gave incorrect results
around smoketests with simulator where each test run failed with an
error like "nosetests: error: no such option: --with-marvin".
Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[x] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)

We can merge this is Travis actually runs the tests and passes. See all recent travis test runs, they all silently fail.

@rhtyd I don't see what you are trying to do here. The description doesn't seem related to the changes in code. Is the pip upgrade the cause of the marvin failure?

Yes @DaanHoogland looks like latest pip does not install marvin well. Distro provided pip v9.0 works okay. There is one failure that I need to look at.

Travis passes now, issue was a missing dependency and use of upgraded pip. Please review (no other tests needed as changes are only in travis scripts) - @wido @GabrielBrascher @rafaelweingartner and others

Thanks all, now on PRs travis can actually run smoke tests. Previously they all fail to run, but travis would still stay green.


Appstudio update hacbs-test-project-i0qa

Pipelines as Code configuration proposal
To start the PipelineRun, add a new comment with content /ok-to-test
For more detailed information about running a PipelineRun, please refer to Pipelines as Code documentation Running the PipelineRun
To customize the proposed PipelineRuns after merge, please refer to Build Pipeline customization

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-i0qa-on-pull-request-
creating pipelinerun hacbs-test-project-i0qa-on-pull-request- in namespace rhtap-demo-oltd-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-i0qa-on-pull-request-
creating pipelinerun hacbs-test-project-i0qa-on-pull-request- in namespace rhtap-demo-ilih-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request

Pipelines as Code CI has failed.There was an error creating the PipelineRun: hacbs-test-project-i0qa-on-pull-request-
creating pipelinerun hacbs-test-project-i0qa-on-pull-request- in namespace rhtap-demo-luoi-tenant has failed.
Tekton Controller has reported this error: the namespace of the provided object does not match the namespace sent on the request


From Wav

Is it easy to change the code so it just creates png of wav files not mp3? If it is, which part, method, function should I change in order to achieve this goal?

Greetings, @OrhanALC :)
There is a method Waveform2Png::process, which has the $wavList variable.
You can simply change the code to put files directly to this array, without calling Waveform2Png::generateWavFiles.
That's how you can achieve your goal :)


Error loading data...

Trying to load the sample from this demo, connection is made, data is read/loaded and product class is created.
However the following error surfaces:
File "/home/ubuntu/weaviate-apps/healthcare-demo/import_data_to_weaviate.py", line 175, in main
"name": data[d].get("name", "Productname"),
Data is a json object.  Do I have to convert this to a dictionary before importing?  Not super facile with Python (10.x) and grubbing around google didn't raise anything.
TIA

Addendum: Oddly, the data wouldn't load from / on a remote docker instance but I was able to send the data from a local machine to the remote instance.

Interesting! Could you explain which approach you used to set up the demo? I think the issue could've been that dockerized applications have their own network, so access from outside the container might not work as expected.

Let me work thru it again in the next 2-3 days and return with comments.

Issue resolved here https://github.com/weaviate/healthsearch-demo/issues/10


Point marker setVisible not working

Point marker visibility can't be updated from the API.
Those markers are not using the style draw method to be drawn, and this is where visibility is used to skip render https://github.com/tangrams/tangram-es/blob/master/core/src/style/style.cpp#L400. It doesn't seem that point labels use that marker property anywhere else.
cc @sarahlensing

Resolved by #1058


Examples standalone build

Update examples build cmake file and release script based on 3.6.2 release feedback.

Test PASSed.


unshard vs merge

hi,
documentation expose "unshard" mode.  -m, --mode MODE                  mode to perform (verify, compact, unshard)
but there is no mention of it in cli code  https://github.com/liquidm/druid-dumbo/blob/master/lib/dumbo/cli.rb#L31-L58
the feature "unshard" has been removed ? or just rename has "merge" ?
thanks by advance

Looks like I missed that. Yes merge has been removed, but not from the read me. The current version is designed for druid 0.17 and there is no difference between compact and unshard, so it got dropped.


Add support for defining caveats in schema, and associated type checks

This change adds lexing, parsing, compilation and generation support for caveat definitions in schema, and fixes type checking to make use of the new definitions throughout the code base
Fixes #879
Fixes #914

Updated and responded to all comments


Fix missing Text Area header in metadata config.

This got missed in the Spotlight upgrade.

Coverage remained the same at 100.0% when pulling f622b89a3f92bbb1684219463596f2ecd0a69fd1 on fix_text_area_label into 5391afd160da28ee51b10aded30d1560568db15b on master.

before this fix:


Add Metrics to Encrypt Calls

Why
Currently we have no way of knowing how many fields are getting encrypted using the Evervault
How
Adds a call to the metrics endpoint with a Data Encrypted event. This is done by creating a thread which sends off requests that are reported to a queue. Note that if the program ends before all of the request is complete, the queue will not necessarily be empty.

Closing, feel free to reopen if needed


word2vec -negative-classes Segmentation fault

I'm trying to train a model using part-of-speech tags as word classes. When I supply even a very small file of size ~1000 lines with word classes, word2vec causes Segmentation fault. The same setup (train file of 100 lines) but with no -negative-classes argument finishes just fine.
Can anybody suggest how to debug this?
txt100.txt
nc100.txt
Exact command: ./word2vec -train txt100.txt -output model10.txt -hs 0 -size 20 -window 3 -type 3 -threads 1 -negative-classes nc100.txt
P.S. text data and pos tags are taken from the Brown corpus.

I'm also getting a Segmentation fault. I'm trying to train cwindow vectors with a file of size 10^9 bytes (the English wikipedia dump).
./word2vec -train /path/to/trainingfile -output /save/path/file -type 2 -size 20 -window 1 -negative 0 -nce 10 -hs 0 -sample 1e-4 -threads 1 -binary 1 -iter 15 -cap 1
I've also tried to use -cap 0, 4 threads, or use -negative rather than nce.

I have same situation with @felicialiu
My backtrace is
Program received signal SIGSEGV, Segmentation fault.
[Switching to Thread 0x7fffd7578700 (LWP 10260)]
0x0000000000409418 in TrainModelThread () at word2vec.c:928
928	          for (c = 0; c < layer1_size; c++) syn1[c + l2 + window_offset] += g * syn0[c + l1];
(gdb) bt
#0  0x0000000000409418 in TrainModelThread () at word2vec.c:928
#1  0x00007ffff7943aa1 in start_thread (arg=0x7fffd7578700) at pthread_create.c:301
#2  0x00007ffff7690aad in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:115

Hi @jetsnguns
I looked at the nc100.txt and the problem seems that there are words with a single class like:
BE be
So if we wish to predict the word be, there are no other negative samples to choose from. I would just write a remove these lines.
Wang Ling

Hi @papower1 ,
Thanks for sending me the backtrack, there seems to be a bug indeed in that line as I am using the wrong set of parameters in the update for hierarchical softmax. I corrected it and checked it works in text8, so can you pull and try again.
Wang Ling

Hi @felicialiu,
I tried running your line with text8, and it seems to work.
Do you get to the point where it prints the progress info? If it does not it might be because the program is running out of memory for the parameters.
Wang Ling

@wlin12 btw, when I disable 'hs', It's good to go. Hope it helps.


refactor: handle validation exception correctly

Get rid of the throw statement and handle it correctly with the helper provided by the Option monad.

Not sure I have enough context to comment, as I float further away from code and more into product land I need screenshots or error messages that were fixed or something less fp-ish to comment on. :)