major differences between translation time and training time

When I use translate.py the results are quite weird, if I train my model for low number of iterations I end up with blank translations(each sentence from the test is translated to eos right away).
If I train my model for high number of iterations I end up with sentences which contains single word with repetitions.
The validation examples during training time looks OK (--sampleFreq param).
Do you have any thoughts?

Hi
(Note that Nematus has a support list
https://groups.google.com/d/forum/nematus-support)
When translation goes badly wrong like this, it's usually a data
problem. Is your data sentence aligned, is it in the correct language?
How large is your training data, and where does it come from?
Does perplexity on the development set decrease (look at the training log)?
Post your command line, so we can see what settings you are using,
cheers - Barry
On 28/02/17 18:04, yftah89 wrote:

When I use translate.py the results are quite weird, if I train my
model for low number of iterations I end up with blank
translations(each sentence from the test is translated to eos right away).
If I train my model for high number of iterations I end up with
sentences which contains single word with repetitions.
The validation examples during training time looks OK (--sampleFreq
param).
Do you have any thoughts?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/rsennrich/nematus/issues/27, or mute the thread
https://github.com/notifications/unsubscribe-auth/AA8mG8ghh3qNo6Q9-XWnUMSvJqlrJ4NOks5rhGGlgaJpZM4MOukb.

Hi Barry, thanks for the quick response,
When translation goes badly wrong like this, it's usually a data
problem. Is your data sentence aligned, is it in the correct language? - yes for both questions
How large is your training data, and where does it come from? - about 50K sentences, not taken from translation dataset
Does perplexity on the development set decrease (look at the training log)? yes it does, the weird thing is that during training time I get some samples from my validation set in the following way :
source: [the source sentence]
truth: [the real translation]
sample: [the model's translation]
and it seems to be allright (I don't refer to actual translation's accuracy I rather refer to simple sanity check - different words and not just empty line or one repetitive word like translation time).
So I don't believe I have over\under fitting problem.
the training command is the following:
THEANO_FLAGS=device=gpu,floatX=float32  python nmt.py --datasets "examples-train.text" "labels-train.text" --dictionaries "examples-train.text.json" "labels-train.text.json" --model check.model --dim_word 100 --dim 64  --valid_datasets "examples-val.text" "labels-val.text"
the translating command is the following:
THEANO_FLAGS=device=gpu,floatX=float32  python translate.py --input examples-test.text --output answers.txt --model check.model.npz
I know that --dim might seems to be small but I really don't think its underfitting problem, moreover, when I try to translate my validation set (the same one I get OK samples on in training time) I'm still getting the same weird results (all sentences are eos or on repetitive word)

Hi
What exactly is your training data, if it is not translation?
Have you tried decoding with the -n (length normalisation) flag?
What about an n-best list? Do you see sensible translations anywhere in
the list?
cheers - Barry
On 28/02/17 23:35, yftah89 wrote:

Hi Barry, thanks for the quick response,
When translation goes badly wrong like this, it's usually a data
problem. Is your data sentence aligned, is it in the correct language?

/yes for both questions/
How large is your training data, and where does it come from? - /about
50K sentences, not taken from translation dataset/

Does perplexity on the development set decrease (look at the training
log)? /yes it does, the weird thing is that during training time I get
some samples from my validation set in the following way :
source: [the source sentence]
truth: [the real translation]
sample: [the model's translation]
and it seems to be allright (I don't refer to actual translation's
accuracy I rather refer to simple sanity check - different words and
not just empty line or one repetitive word like translation time).
So I don't believe I have over\under fitting problem./
/the training command is the following:/
THEANO_FLAGS=device=gpu,floatX=float32 python nmt.py --datasets
"examples-train.text" "labels-train.text" --dictionaries
"examples-train.text.json" "labels-train.text.json" --model
check.model --dim_word 100 --dim 64 --valid_datasets
"examples-val.text" "labels-val.text"
/the translating command is the following:/
THEANO_FLAGS=device=gpu,floatX=float32 python translate.py --input
examples-test.text --output answers.txt --model check.model.npz
I know that --dim might seems to be small but I really don't think its
underfitting problem, moreover, when I try to translate my validation
set (the same one I get OK samples on in training time) I'm still
getting the same weird results (all sentences are eos or on repetitive
word)
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rsennrich/nematus/issues/27#issuecomment-283196411,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA8mG3jbfxnXl2gidCPa-5raXt0Rgsqbks5rhK86gaJpZM4MOukb.

What exactly is your training data, if it is not translation?
I'm trying to translate sentences from English to English (kind of paraphrasing)
Have you tried decoding with the -n (length normalisation) flag?
What about an n-best list? Do you see sensible translations anywhere in
the list?
I just tried both and unfortunately neither helps.
What really bugs me is that in training time the translated samples looks great, what is the difference between whats going on in the training time and what going on in translation time?
Is the samples which are given at training time are not translated in the exact same way like the translations in translation time? if not then what is the difference?
Thanks in advance for all of your help

Hi
Sampling and translating use the same code, but sampling is stochastic,
whereas translating uses beam search to find the top scoring outputs.
That's why I was asking about an n-best list, maybe there are good
translations with worse model scores.
It does sound like your model is under-trained. Perhaps the data is not
sufficient, perhaps it is noisy, perhaps it is not the right model for this
task. When a translation model is under-trained it just latches onto words
or phrases from the target language and repeats them. Have a look at the
perplexities - have they converged? If yes, and the model is still
under-trained, then you probably need more data or a different architecture
cheers - Barry
On 1 March 2017 at 15:15, yftah89<EMAIL_ADDRESS>wrote:

What exactly is your training data, if it is not translation?
I'm trying to translate sentences from English to English (kind of
paraphrasing)
Have you tried decoding with the -n (length normalisation) flag?
What about an n-best list? Do you see sensible translations anywhere in
the list?
I just tried both and unfortunately neither helps.
What really bugs me is that in training time the translated samples looks
great, what is the difference between whats going on in the training time
and what going on in translation time?
Is the samples which are given at training time are not translated in the
exact same way like the translations in translation time? if not then what
is the difference?
Thanks in advance for all of your help
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rsennrich/nematus/issues/27#issuecomment-283367930,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA8mG8SKwpCFMV5qQ_78A2JHhro9rNbcks5rhYuYgaJpZM4MOukb
.

But if you are right and my model is under trained or my perplexities are not converged, why the samples that the model provides me in training time are OK? wouldn't bad model provide bad samples?
If I could get the same results I get in samples for my whole test set I will be very happy

Hi
To be honest, I never look at the samples. How do the perplexities look?
cheers - Barry
On 02/03/17 01:17, yftah89 wrote:

But if you are right and my model is under trained or my perplexities
are not converged, why the samples that the model provides me in
training time are OK? wouldn't bad model provide bad samples?
If I could get the same results I get in samples for my whole test set
I will be very happy
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rsennrich/nematus/issues/27#issuecomment-283525478,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA8mG2K7lo_6S0bWr6I4xXuFazceAOelks5rhhi7gaJpZM4MOukb.

I'm not sure that perplexity is the right way to measure my performance, the loss that nematus provides is around 20, I'm not sure if this considered to be good or not.
Can I use some interface for stochastic inference in translation time or I'm bound to write my on code in order to do so?
Again, thank for your help and efforts

Hi
It's not the absolute loss that matters, it's the convergence I was asking
about. Nematus optimises cross-entropy by default, which is closely related
to perplexity, so if it is not the right way to measure your performance,
then you have a problem.
You can could write some code to call the gen_sample() method in nmt.py, to
give stochastic samples. However this is not tackling the real problem -
beam search should be better than stochastic sampling,
cheers - Barry
On 2 March 2017 at 21:59, yftah89<EMAIL_ADDRESS>wrote:

I'm not sure that perplexity is the right way to measure my performance,
the loss that nematus provides is around 20, I'm not sure if this
considered to be good or not.
Can I use some interface for stochastic inference in translation time or
I'm bound to write my on code in order to do so?
Again, thank for your help and efforts
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/rsennrich/nematus/issues/27#issuecomment-283795884,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA8mG0LvRtVsfNT8zVgyyq-T-9g3IgIuks5rhzvJgaJpZM4MOukb
.

I think I have found the cause for the major differences between the samples in training time and the results in translation time but I want to be sure, can you please shed some light on this comments in the code
#FIXME: sampling is currently performed according to the last model only
line 724 in nmt.py

The gen_sample function supports ensembles for beam search, but for ancestral sampling, doesn't support ensembles and uses only its last component if passed an ensemble. Unless you're using ensembles at test time, this difference shouldn't affect you.

I see, so I still got no idea why I get bad results in translation time and good ones in the samples that the train time provides, the only difference I see so far is that the samples are stochastic and the translation use beam search but to the best of my knowledge beam search should be better, do you have any idea of some additional differences between the samples in training time and the translation process?
If not, can you give me some explanation\references about stochastic vs beam search methods and if stochastic can be better than beam search?


order gets changed while conversion.

Hi ,
I am trying to convert xml data to json and after some processing i want to convert back to xml. For testing purpose i am converting below data which is part of my xml file.
    <ProfAlign name="Layout">
	<PVI>0.77.906</PVI>
	<ParaCurve length="200.">247.13 91.807</ParaCurve>
	<PVI>1658.62622142<PHONE_NUMBER>4</PVI>
   </ProfAlign>

xml2json ({compact: true, spaces: 4}) converts this data as below,
"ProfAlign": [
{
"_attributes": {
"name": "Layout"
},
"PVI": [
{
"_text": "0. 77.906"
},
{
"_text": "1658.62622142<PHONE_NUMBER>4"
}
],
"ParaCurve": {
"_attributes": {
"length": "200."
},
"_text": "247.13 91.807"
}
}
And json2xml ({indentAttributes: false, spaces: '\t', fullTagEmptyElement: true, compact: true}) converts back to xml like below,
  <ProfAlign name="Layout">
	<PVI>0.77.906</PVI>
	<PVI>1658.62622142<PHONE_NUMBER>4</PVI>
	<ParaCurve length="200.">247.13 91.807</ParaCurve>
  </ProfAlign>

As you can see, order is changed. Please suggest how to handle this.

To preserve the order, use compact: false.
Using comact: true will not preserve the order and this is how most other libraries are doing.
See Compact Vs Non-Compact for more details.
Actually, this is the main reason why I created this library: to preserve the order of the elements.

Yes It worked, but made the json object very bulky. Thanks

Parsing of entities like " and &#xd' are currently not supported.
I will try to add new option, say parseEntities, to support this in the coming few days.

I have data in my xml like, (Please do not consider space after slash characters)
<collection name="arguments"/ >
<string name="connect_timeout"'></ string>
with fullTagEmptyElement: false =>
<collection name="arguments"/ >
<string name="connect_timeout"/ >
with fullTagEmptyElement: true =>
<collection name="arguments"/ ></ collectionProp>
<string name="connect_timeout"'></ string>
Can we handle these two scenarios individually ? please suggest.

Do you mean you want for certain elements to get output like  and for other elements it should output ?
If this is what you want, then how does the converter knows that a elements should be shortened and b elements should have full closing tag?

Yes this is what i want as my input is like this and i can not help here. Is it possible,  we can put some flag in converted json and use that flag to take appropriate action whether it should be  or  ?

@sandext12
Regarding encoding and decoding of html entities, it is actually tricky and and there are many edge cases. Therefore, I suggest to do this manually by utilizing he library which was created to address this thoroughly. I could have include it in my library but it would increase its size and probably not needed for most users.
Regarding generation of  and , I am still not sure how to do this in a generic way. Can you provide an example of how would you pass the options for this requirement?

@sandext12
I have published version 1.6.1 of this library which can help you to decide which empty elements can have full closing tags and which can have self closing tags. Just supply a function to fullTagEmptyElementFn option.
Example:
json2xml(json, {compact: true, fullTagEmptyElementFn: function(elementName) {
  return elementName === 'string'; // returning true forces to generate full closing tag
}});

This way, you can achieve the desired output:
<collection name="arguments"/>
<string name="connect_timeout"></string>

For other functions, see this.

Closed as no further comments received.


Late feedback/answer from AWS

Your OS (including version) where you are running AlexaPi:
Raspbian - Stretch(newest version)

Your hardware platform and model you are running on:
Raspberry Pi 2

Python release (python2 --version):
2.7

Description of problem:
I built my RPi echo last week its working, but feedback is so late. Like 10 to 15 seconds for an answer. sometimes I think its not working at all. Its freezed after multiple questions & a reboot is required.

Expected:
It should work instantaneously or maybe a acceptable delay of 2, 3 seconds

Additional info:
I have a video of working but cant upload here

After stopping the Alexa service I got this log
2017-11-05 14:31:04 DEBUG: Setting up playback handler: VlcHandler
2017-11-05 14:31:04 INFO: Checking Internet Connection ...
2017-11-05 14:31:04 DEBUG: Starting new HTTPS connection (1): api.amazon.com
2017-11-05 14:31:08 DEBUG: https://api.amazon.com:443 "GET /auth/o2/token HTTP/1                                                                                .1" 404 29
2017-11-05 14:31:08 INFO: Connection OK
2017-11-05 14:31:08 INFO: AVS token: Requesting a new one
2017-11-05 14:31:08 DEBUG: Starting new HTTPS connection (1): api.amazon.com
2017-11-05 14:31:16 DEBUG: https://api.amazon.com:443 "POST /auth/o2/token HTTP/                                                                                1.1" 200 945
2017-11-05 14:31:16 INFO: AVS token: Obtained successfully
Expression 'paInvalidSampleRate' failed in 'src/hostapi/alsa/pa_linux_alsa.c', l                                                                                ine: 2048
Expression 'PaAlsaStreamComponent_InitialConfigure( &self->capture, inParams, se                                                                                lf->primeBuffers, hwParamsCapture, &realSr )' failed in 'src/hostapi/alsa/pa_lin                                                                                ux_alsa.c', line: 2719
Expression 'PaAlsaStream_Configure( stream, inputParameters, outputParameters, s                                                                                ampleRate, framesPerBuffer, &inputLatency, &outputLatency, &hostBufferSizeMode )                                                                                ' failed in 'src/hostapi/alsa/pa_linux_alsa.c', line: 2843
2017-11-05 14:31:16 DEBUG: Stopping audio play
2017-11-05 14:31:16 DEBUG: Playing audio: /opt/AlexaPi/src/resources/hello.mp3
2017-11-05 14:31:16 DEBUG: Player State: State.Opening
Exception in thread Thread-1:
Traceback (most recent call last):
File "/usr/lib/python2.7/threading.py", line 801, in __bootstrap_inner
self.run()
File "/usr/lib/python2.7/threading.py", line 754, in run
self.__target(*self.__args, **self.__kwargs)
File "/opt/AlexaPi/src/alexapi/triggers/pocketsphinxtrigger.py", line 66, in t                                                                                hread
self._capture.handle_init(self.AUDIO_RATE, self.AUDIO_CHUNK_SIZE)
File "/opt/AlexaPi/src/alexapi/capture.py", line 113, in handle_init
frames_per_buffer=chunk_size
File "/usr/local/lib/python2.7/dist-packages/pyaudio.py", line 750, in open
stream = Stream(self, *args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/pyaudio.py", line 441, in _init                                                                                _
self._stream = pa.open(**arguments)
IOError: [Errno -9997] Invalid sample rate
2017-11-05 14:31:16 DEBUG: Player State: State.Playing
2017-11-05 14:31:16 DEBUG: Started play.
2017-11-05 14:31:19 DEBUG: Player State: State.Ended
2017-11-05 14:31:19 DEBUG: Finished play.
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.rear
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.center_lfe
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.side
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.hdmi
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.hdmi
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.modem
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.modem
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.phoneline
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.phoneline
ALSA lib confmisc.c:1281:(snd_func_refer) Unable to find definition 'defaults.bl                                                                                uealsa.device'
ALSA lib conf.c:4528:(_snd_config_evaluate) function snd_func_refer returned err                                                                                or: No such file or directory
ALSA lib conf.c:4996:(snd_config_expand) Args evaluate error: No such file or di                                                                                rectory
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM bluealsa
ALSA lib confmisc.c:1281:(snd_func_refer) Unable to find definition 'defaults.bl                                                                                uealsa.device'
ALSA lib conf.c:4528:(_snd_config_evaluate) function snd_func_refer returned err                                                                                or: No such file or directory
ALSA lib conf.c:4996:(snd_config_expand) Args evaluate error: No such file or di                                                                                rectory
ALSA lib pcm.c:2495:(snd_pcm_open_noupdate) Unknown PCM bluealsa
connect(2) call to /tmp/jack-1000/default/jack_0 failed (err=No such file or dir                                                                                ectory)
attempt to connect to server failed
INFO: pocketsphinx.c(152): Parsed model-specific feature parameters from /usr/lo                                                                                cal/lib/python2.7/dist-packages/pocketsphinx/model/en-us/feat.params
Current configuration:
[NAME]                  [DEFLT]         [VALUE]
-agc                    none            none
-agcthresh              2.0             2.000000e+00
-allphone
-allphone_ci            no              no
-alpha                  0.97            9.700000e-01
-ascale                 20.0            2.000000e+01
-aw                     1               1
-backtrace              no              no
-beam                   1e-48           1.000000e-48
-bestpath               yes             yes
-bestpathlw             9.5             9.500000e+00
-ceplen                 13              13
-cmn                    live            batch
-cmninit                40,3,-1         41.00,-5.29,-0.12,5.09,2.48,-4.07,-1.37,                                                                                -1.78,-5.08,-2.05,-6.45,-1.42,1.17
-compallsen             no              no
-debug                                  0
-dict                                   /usr/local/lib/python2.7/dist-packages/p

Post the log when running in debug mode.
Read up on the docs: https://github.com/alexa-pi/AlexaPi/wiki

@renekliment I've posted log 5 days back. i know you are busy but can you take a look at it. Please?
I want to use my AlexaPi daily and like every other human out there for each reply I dont have patience of 20 to 30 seconds.


file upload ddb

File Upload dynamo db
What user problem are we solving?
build dynamo db table for file upload
What solution does this PR provide?
dynamo db table configured in backend + terraform
Testing Methodology
tested basic CRUD operations
    file_up_ddb = FileUploadDDBUtil(FILE_UPLOAD_TABLE_NAME, AWS_REGION)
    createFileUploadData({"s3_uri": "s3://test-bucket/test.csv", "uid": "1234", "created": get_current_timestamp(), "filename": "test.csv"})
    print(getFileUploadData("s3://test-bucket/test.csv"));
    updateFileUploadData({"s3_uri": "s3://test-bucket/test.csv", "filename": "test2.csv", "created": get_current_timestamp(), "uid": "1234"});
    print(getFileUploadData("s3://test-bucket/test.csv"));
    createFileUploadData({"s3_uri": "s3://test-bucket-2/test2.csv", "uid": "5678", "filename": "test2_1.csv", "created": get_current_timestamp()})
    print(getAllUserFileUploadData("5678"))

Any other considerations
make sure we setup relevant backend api endpoints using this table

Looks good to me!

@farisdurrani you can merge it when you are ready


History: Table performance is poor with thousands of rows

When history is huge (~4000 events and large payload), current cadence client GetWorkflowHistory tasks long time (> 5 min) to return result.
As a result, UI freeze and looks like not responding before getting the result.
We need to add support to use long poll with pagination.

The real issue is that the history table is not virtualized, and rendering 1000s of rows, including the details column with many items of data is very burdesome on the browser. Bringing in a proper data grid component will solve this.


Error Textures [MC1.12.2]

Versions:

Minecraft: 1.12.2
Forge: all (last <IP_ADDRESS>37)
McJtyLib: 2.6.7
RFTools: 7.32
Error Textures. Lines: 10733-10738.
Debug Log - https://drive.google.com/open?id=17dCrv1l_taOd9Js8wWa6RZbId6PPamuF

Not harmful. You can safely ignore that

Good. Thank you very much!


Allow using RPC hostnames (wallet-cli)

I'm assuming this was tested.

I just tried using a DNS name and it worked as expected.


Update readme with latest 1.0 changes

Merge this before pull request #19 (subworkflow-task)
@jnjosh Please review.

:+1:


Ignore/Override AWS Environment Variables

Summary:
I'm trying to host an app using dynamoose on Zeit's now v2.
AWS credential environment variables are reserved: https://zeit.co/docs/v2/deployments/environment-variables-and-secrets/#reserved-variables
How can I get dynamoose to connect only using what I have defined in my config, and ignore the global vars?
Code sample:
  dynamoose.AWS.config.update({
    accessKeyId: config.aws.accessKey,
    secretAccessKey: config.aws.secretKey,
    region: config.aws.region
  });

Current output and behavior:
AccessDeniedException: User: arn:aws:sts::-------:assumed-role/cloudwatch_logs_events_putter/-------------------- is not authorized to perform: dynamodb:DescribeTable on resource: arn:aws:dynamodb:us-east-1:------:table/Corporation
    at Request.extractError (/var/task/user/src/index.ts:84720:27)
    at Request.callListeners (/var/task/user/src/index.ts:18027:20)
    at Request.emit (/var/task/user/src/index.ts:17999:10)
    at Request.emit (/var/task/user/src/index.ts:46797:14)
    at Request.transition (/var/task/user/src/index.ts:46136:10)
    at AcceptorStateMachine.runTo (/var/task/user/src/index.ts:66972:12)
    at /var/task/user/src/index.ts:66984:10
    at Request.<anonymous> (/var/task/user/src/index.ts:46152:9)
    at Request.<anonymous> (/var/task/user/src/index.ts:46799:12)
    at Request.callListeners (/var/task/user/src/index.ts:18037:18)

Expected output and behavior:
To connect and use the credentials in the config.update call. I know this is a bit of an odd case due to a specific host, but figured I would see if I could get any help.
Environment:
Operating System: macOS
Operating System Version: macOS 10.14.4
Node.js version (node -v):11.6.0
NPM version: (npm -v):6.5.0
Dynamoose version: 1.7.2
Dynamoose Plugins: N/A

Dynamoose Plugins:

[ ] Yes, I believe that one or multiple 3rd party Dynamoose plugins are effecting this issue
[ ] No, I believe this issue is independent of any 3rd party Dynamoose plugins I'm using
[ ] Unknown, I'm unsure if Dynamoose plugins are effecting this issue
[X] I am not using any Dynamoose plugins

Other information (if applicable):
Type (select 1):

[X] Bug report
[ ] Feature suggestion
[ ] Question
[ ] Other suggestion
[ ] Something not listed here

Other:

[X] I have read through the Dynamoose documentation before posting this issue
[X] I have searched through the GitHub issues (including closed issues) and pull requests to ensure this issue has not already been raised before
[X] I have searched the internet and Stack Overflow to ensure this issue hasn't been raised or answered before
[X] I have tested the code provided and am confident it doesn't work as intended
[X] I have ensured that all of my plugins that I'm using with Dynamoose are listed above
[X] I have filled out all fields above
[X] I am running the latest version of Dynamoose

@mckernanin Your code looks right. The only thing I can think of that would be a problem is if you are creating the model our sing Dynamoose in anyway before running the dynamoose.AWS.config.update method.
Or if your IAM role doesn't have the DescribeTable permission.

Closing this due to no recent activity. If this is still a problem or you have questions please feel free to comment again and we can try to help further.


Fix javadoc tables

Motivation:
Typos and wrong place of javadoc's block tag were causing issues with the rendering of the tables.
Modification:
fixed the typo.
Result:
The tables appears as expected.

Thanks very much


[microsoft-signalr] build failure

Operating system
Windows
Compiler
MSVC
Steps to reproduce the behavior
`.\vcpkg.exe install microsoft-signalr[messagepack]:x86-windows-static`

Failure logs
Could not find a package configuration file provided by "msgpack" with any
of the following names:

  msgpackConfig.cmake
  msgpack-config.cmake

Add the installation prefix of "msgpack" to CMAKE_PREFIX_PATH or set
"msgpack_DIR" to a directory containing one of the above files.  If
"msgpack" provides a separate development package or SDK, be sure it has
been installed.

Additional context
No response

@BillyONeal The port now builds successfully but when included in a project it throws the same error again.
CMakeLists.txt to reproduce:
cmake_minimum_required(VERSION 3.8)

set(VCPKG_TARGET_TRIPLET x86-windows-static)
set(CMAKE_TOOLCHAIN_FILE "C:/Users/Dev/vcpkg/scripts/buildsystems/vcpkg.cmake")

project(signalr-test)

find_package(microsoft-signalr CONFIG REQUIRED)

vcpkg.json
{
  "name": "signalr-test",
  "version": "0.1.0",
  "dependencies": [
    {
      "name": "microsoft-signalr",
      "default-features": false,
      "features": ["messagepack"]
    }
  ]
}

Sorry, this was my mistake. I forgot to modify the contents in microsoft-signalr-config.in.cmake. I have submitted a PR to fix this issue.
Sorry again.

@Cheney-W Thank you! It partially works now. I opened another issue: #31370


HWY-256: Request latest units on era start.

https://casperlabs.atlassian.net/browse/HWY-256
If there's a chance that we're starting an era after it's already finished – send the request for validators' latest panoramas.

This PR will fail tests. I'm calling Timestamp::now() inside the method and compare it with start_time which in some tests is set to 0. This will always yield true (now > 0) and we will return unnecessary effects in tests.

bors merge

bors retry


⚠ Bofrost.fr has degraded performance

In 38ee05c, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1537 ms

Resolved: Bofrost.fr performance has improved in fd9102a after 6 minutes.


Super galaxy rumble bug

Super galaxy rumble skin is bugged, because his Q fires in the right instead of forward, could anyone help with that?

#1


Inclusion reliably takes 2 attempts for a specific device, and then doesn't finish interview

Is your problem within Home Assistant (Core or Z-Wave JS Integration)?
NO, my problem is NOT within Home Assistant or the ZWave JS integration
Is your problem within ZWaveJS2MQTT?
NO, my problem is NOT within ZWaveJS2MQTT
Checklist

[X] I have checked the troubleshooting section and my problem is not described there.

[X] I have read the changelog and my problem was not mentioned there.

Describe the bug
What causes the bug?
2 bugs within same process, will describe both here but may need to bump one to another issue if they're unrelated.  I'm using zwave2mqtt UI and not sure if source of errors are in node-zwave-js or zwave2mqtt.
Try to add a node to the network, it only completes node addition after 2nd inclusion, and having been added it does not complete interview, and for 30 minutes floods the network with traffic before giving up.
What do you observe?
Part A:
To get a device included and showing up in the zwave2mqtt UI requires 2 separate inclusion attempts.
On the first attempt, device flashes LED indicating it's added but it never registers in UI, and the inclusion process times out.
On second attempt, device flashes LED indicating it's added and then confirms node as added, and shows up in UI.
Part B:
Having been added, the device never gets beyond Node Info stage.  The zwave logs spams about 25x per second with 'Node 52, Endpoint 1: Trying to access endpoint instance before Multi Channel interview' and looks to give up after 30 minutes if I let it run on.
What did you expect to happen?
Inclusion and interview process would result in a recognized new node on the network.
Steps to reproduce the behavior:
Attempt 1
ZWave2mqtt UI - start inclusion without security
On device - press 3 times to do the inclusion
Inclusion does not register and times out.  Have to close UI window manually.
Attempt 2
ZWave2mqtt UI - start inclusion without security
On device - press 3 times to do the inclusion
Inclusion registers returns with new node ID.
Watch UI for interview status - it gets to Node Info and marked as Alive but no further.
Device information
The device here in the logs is
Manufacturer: Zooz
Model name: ZEN25
Node ID in your network: 52, 53 (same device, different attempts to include on network)
I have 3 of these devices. 2 successfully added prior to this problem, the third refuses to do so.  The 2 that did add OK both suffered from the 2-attempt to add node problem but they completed the interview OK after that.
How are you using node-zwave-js?

[ ] zwavejs2mqtt Docker image (latest)
[ ] zwavejs2mqtt Docker image (dev)
[ ] zwavejs2mqtt Docker manually built (please specify branches)
[ ] ioBroker.zwave2 adapter (please specify version)
[ ] HomeAssistant zwave_js integration  (please specify version)
[ ] pkg
[ ] node-red-contrib-zwave-js (please specify version, double click node to find out)
[ ] Manually built from GitHub (please specify branch)
[X] Other (please describe)

Which branches or versions?
Deployed to HA via frenck's zwavejs2mqtt add-on and configured to use the web socket connection.
zwavejs2mqtt = 5.6.2
node-zwave-js = 8.3.1
HA = 2021.9.7
Did you change anything?
yes (please describe)
If yes, what did you change?
The device was previously added / working on a different network without error. I factory reset it and attempted to add to a new network.
Did this work before?
Yes (please describe)
If yes, where did it work?
I rebuilt network on a new stick.  Old network was Zooz ZST 10 500, new network uses ZST 10 700 (with flashed firmware to latest, 7.16.4).
Old network allows add of nodes reliably including this one device without error.
Attach Driver Logfile
zwavejs_2021-09-28.log

Your log is not on loglevel debug, let's try this again :)

Sigh I am so sorry. Meticulously read instructions, and can't see a thing!  This device has me annoyed!
Logs attached with debug. Node 54.  Of course on this time it did manage to complete the interview and it's now ready to use; all I can think of there is the 'Trying to access endpoint instance before Multi Channel interview' traffic spam was causing issue with getting the data back.
On the add node, this still took 2 attempts to add node.  First one timed out and I had to close window, 2nd attempt resulted in the add OK.
These are clean logs from a restart, let network startup finish, then add new node x2.
zwavejs_2021-09-29.log
zwavejs2mqtt.log

Some things that stand out:

After almost exactly 30 seconds, something stops the inclusion process. I'm guessing you have a command timeout configured in zwavejs2mqtt?
It takes a whopping 40 seconds from receiving the AddingSlave callback until ProtocolDone is received, which normally takes milliseconds. At this point the inclusion process was stopped already, so the callback is ignored and nothing gets included.

To piggyback on the meter reporting: Z-Wave is a very constrained protocol and not meant for real-time electricity measurement. These can easily bring a whole network to its knees (like here).
You can get pretty close to real-time though if you configure the devices to report changes and throw in a few timed reports (order of magnitude: minutes) for good measure.

To the add process - the 30s timeout is done by the zwave2mqtt UI, there's no user input to set that.
To the meter reads, the defaults are report every 30s, I had set them to be much higher but a factory reset cleared those.  No idea why those are reporting more frequently, I'll raise with Zooz.

To the add process - the 30s timeout is done by the zwave2mqtt UI, there's no user input to set that.

FYI, you can configure the timeout inclusion timeout in the zwavejs2mqtt Z-Wave settings, option Inclusion/Exclusion timeout.


Add workflow to send tweets when new release is published

...

Hi, thanks for contributing to Strawberry 🍓!
We noticed that this PR is missing a RELEASE.md file. We use that to automatically do releases here on GitHub and, most importantly, to PyPI!
So as soon as this PR is merged, a release will be made 🚀.
Here's an example of RELEASE.md:
Release type: patch

Description of the changes, ideally with some examples, if adding a new feature.

Release type can be one of patch, minor or major. We use semver, so make sure to pick the appropriate type. If in doubt feel free to ask :)

Hello
🌏

!

Hello
🌏

!

Hello
🌏

!

This is how the comment looks like:


SubmitMultipartMultilangualExample - wrong value for MAX_MULTIPART_MSG_SEGMENT_SIZE_7BIT

There is an error in SubmitMultipartMultilanguageExample which prevents successful sending of 7bit encoded Long/Multipart SM:
7bit message segment is defined as 154 chars:
private static final int MAX_MULTIPART_MSG_SEGMENT_SIZE_7BIT = 154;
It should be 153 chars! Every part of SM in 7bit encoding should be maximum UDH+153chars!
153 chars for 7-bit enc
134 chars for 8-bit enc - binary
67 chars for 16-bit enc - unicode
When using the example to send UCS2 messages (like in Arabic language), everything works good.
Ivo Parvanov

Indeed, this is not correct. I fixed it. It's time to update the examples. When splitting extended characters cannot divided into different parts, is also some which is missing here.

I just added a complete rewrite of the concatenation example, as it did to handle extended GSM characters correctly.


[T4A3][F11-A3]Hsieh Hsin Han

…rent class called Contact.

Accepting this PR without a review because another of your PRs for this tutorial was reviewed in detail.

@Tony-Hsieh  Some comments added. Please close the PR after reading comments. Sorry to bother you again. You may want to see the comments.


4.2.0  - deleteFilesForConfiguration will not delete files after crash

4.2.0 -[RLMRealm deleteFilesForConfiguration:]/Realm.deleteFiles(for:)
Issue exits with 4.3.0 as well
If the app crashes, it appears the Realm file is left 'open' so attempting to use the above function after crash (clean rebuild etc) results in an error and will not delete the file(s):

Realm file at path /Users/yyyyyyy/Library/Application Support/com.my_company.project_name/default.realm cannot be deleted because it is currently opened.

Goals
Delete the realm files and start fresh
Expected Results
Realm files to be deleted regardless of the crash
Actual Results
Files do not delete
Steps to Reproduce
Make the app crash, then attempt to delete the realm files with the deleteFilesForConfiguration function
The current resolution is the manually toss the files or in code do the following
func doDelete() {
    let realmURL = Realm.Configuration.defaultConfiguration.fileURL!
    print("deleting: \(realmURL)")
    let realmURLs = [
        realmURL,
        realmURL.appendingPathExtension("lock"),
        realmURL.appendingPathExtension("note"),
        realmURL.appendingPathExtension("management")
    ]
    for URL in realmURLs {
        do {
            try FileManager.default.removeItem(at: URL)
        } catch {
            // handle error
        }
    }
}

Code Sample
do {
    let config = Realm.Configuration.defaultConfiguration
    let isSuccess = try Realm.deleteFiles(for: config)
    if isSuccess == true {
        print("files were located and deleted")
    } else {
        print("nothing was deleted, files were not found")
    }
} catch let error as NSError {
    print(error.localizedDescription)
}

Version of Realm and Tooling
Realm framework version: 4.2.2
Realm Object Server version: n/a
Xcode version: 11.3
iOS/OSX version: macOS 10.15.2
Dependency manager + version: n/a

Oops. App running in the background had the file opened so this was a false alarm. Closing.

➤ Unito Sync Bot commented:
Transition made by Unito


Storage PartitionedUploader supports BinaryData

Client APIs that upload via PartitionedUploader accept either Stream or BinaryData as input. But the uploader itself only accepted streams, wrapping the BinaryData in one under the hood. This leads to a lot of extra buffering and copying on data that is already referenceable in memory.
This abstracts the main logic of PartitionedUploader to handle content without knowing the type. Public APIs on the type now accept either Stream or BinaryData, injecting the appropriate data partitioning and upload behaviors into the main logic.

API change check
API changes are not detected in this pull request.

Still need to replicate existing Stream tests as BinaryData versions

Binary data tests already exist extensively in blob tests. Added some low-level tests in common to asserts delegates are called as expected.


DSP: catalog/datasets/{id} endpoint is not implemented

I couldn't find the endpoint in latest main branch. According to the spec, this is a mandatory endpoint:
https://github.com/International-Data-Spaces-Association/ids-specification/blob/main/catalog/catalog.binding.https.md#24-the-catalogdatasetsid-endpoint
It is especially important in cases we already know the asset / dataset id, e.g. from a AAS registry.
--
Matthias Binzer

This is planned to be implemented during milestone 10

Can you say something about timing of milestone 10? And which product-edc (tractus-x) this will end up?
Thanks,
Matthias

The timing for Milestone 10 will be announced in the next days.
Talking about this issue, the Dataset id will be the same of the asset, and it could be used to retrieve the dataset with this endpoint.


Redesign UI of docs side nav

todo: move to design system

@aeplay It's not done but feel free to review
I'm also going to get the design system app running so that I can make a style guide like this and put SideNav component there

@aeplay On second thought, I'm not adding this to the design system because it's very specific to the docs especially the section with the API references.

Looks really nice!!
Only one issue remaining in Safari which still shows the built-in detail expander arrows:

@aeplay Thanks for the fix 🙏


chore: remove haul cli

Summary

Remove Haul CLI and create plugin for React Native CLI instead.
Possibly remove Logger (to be checked)
Test plan

Current issue:
npx react-native ram-bundle --verbose
warn Package @haul-bundler/react-native-cli-plugin has been ignored because it contains invalid configuration. Reason: Unexpected identifier
error Option "--entry-file" is missing
Error: Option "--entry-file" is missing
    at options.forEach.opt (/Users/noemirozpara/Documents/callstack/haul/fixtures/react_native_with_haul_0_60x/node_modules/@react-native-community/cli/build/tools/assertRequiredOptions.js:51:13)
    at Array.forEach (<anonymous>)
    at assertRequiredOptions (/Users/noemirozpara/Documents/callstack/haul/fixtures/react_native_with_haul_0_60x/node_modules/@react-native-community/cli/build/tools/assertRequiredOptions.js:40:11)
    at Command.handleAction (/Users/noemirozpara/Documents/callstack/haul/fixtures/react_native_with_haul_0_60x/node_modules/@react-native-community/cli/build/cliEntry.js:159:42)
    at Command.listener (/Users/noemirozpara/Documents/callstack/haul/fixtures/react_native_with_haul_0_60x/node_modules/commander/index.js:315:8)
    at Command.emit (events.js:198:13)
    at Command.parseArgs (/Users/noemirozpara/Documents/callstack/haul/fixtures/react_native_with_haul_0_60x/node_modules/commander/index.js:651:12)
    at Command.parse (/Users/noemirozpara/Documents/callstack/haul/fixtures/react_native_with_haul_0_60x/node_modules/commander/index.js:474:21)
    at setupAndRun (/Users/noemirozpara/Documents/callstack/haul/fixtures/react_native_with_haul_0_60x/node_modules/@react-native-community/cli/build/cliEntry.js:210:24)
    at Object.run (/Users/noemirozpara/Documents/callstack/haul/fixtures/react_native_with_haul_0_60x/node_modules/@react-native-community/cli/build/cliEntry.js:173:11)

Current to do:

[ ] handle required cli options (e.g. haul-ram-bundle --platform)
[ ] test commands
[ ] update fixtures and test to use new cli


Export JSON schema compliant objects

Instead of custom schema format, we should be exporting a standardized variant

This would be great!
Sharing a bit of the use cases I have in mind:
It would allow re-using the same JS objects in other libraries that can do data validation using JSON Schema, without having to manually extract the particular JSON Schema from OpenAPI.
I would imagine it could be simpler than the current approach that generates a custom schema, instead, this would be just a copy of the content of the JSON Schema inside of OpenAPI deserialized from a JSON string to an actual JS object.
I can also imagine there could be libraries that generate automatic UIs based on JSON Schema and could take this as input.

Roger that @tiangolo. @jordanshatford I'd be okay with having a superset of JSON Schema if there are fields we export that aren't part of the specs, I didn't look into the diff between them yet

@lucsky @Darhagonable @tiangolo JSON Schemas 2020-12 are supported in v0.36.0 if you're using OpenAPI 3.1 🎉 would love your feedback on this!


Not all canvas tiles are not get redraw when zoom in/out

from below picture, we can see tile(15,6,4) and (0,6,4) are not get re-draw:

Maybe they shouldn't be redrawn?
Try ToggleWireframe control
        globe.planet.addControl(new ToggleWireframe()); // from control/ToggleWireframe.js and then press X key
And check that the segments(not tiles) are really deeper than they are. You will see "the division".

However, if you wanted to see the same zoom tiles I could add an option.
This is ok:

This is also ok:

That's make sense. thanks!


Instead of "no emit requested" would be great to show the actual JS vs. TS comparison

refs https://github.com/TypeStrong/atom-typescript/pull/415

I do miss the ability to see the emitted JS in an easy manner.  While I don't care for the web essentials-style split screen, it is useful from time to time.  I wonder if it would be possible to enable some sort of keyboard shortcut to "pop" the emitted JS out in a floating window.  Perhaps a bit like a code lens preview.  This would be tricky because it would need to work with source maps.

@basarat Did you make a recent change to disable the "this file changed - do you want to reload?" prompt when looking at the emitted JS and TS in the same view as a split-screen?  Or was this a recent change in Atom?  Now it appears that the viewed JS file immediately refreshes when the TS compiles-on-save as long as you haven't made any changes in the JS file directly.
I notice we also have the same problem as Web Essentials used to where the emitted JS scrolls to the top when the file is refreshed.  If you didn't make this change then perhaps this is an Atom bug.

I do miss the ability to see the emitted JS in an easy manner

Already there : https://github.com/TypeStrong/atom-typescript#output-toggle

where the emitted JS scrolls to the top when the file is refreshed. If you didn't make this change then perhaps this is an Atom bug.

Verified that this happens now. Its a atom bug nothing to do with us :rose:

Already there : https://github.com/TypeStrong/atom-typescript#output-toggle

I do need to keep more up to date on the patch notes.

fixed with https://github.com/TypeStrong/atom-typescript/pull/422 thanks to @helarqjsc :rose:


Move to pip-tools

Looks like the improved version of pip, that's not entirely a different thing like Poetry.
Also need to look how to bump dependabot to use that - https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/about-dependabot-version-updates#pip-and-pip-compile

https://github.com/jazzband/pip-tools


Always hash job name

With k8s v1.22 the batch name doesn't allow subdomains
Removed unused method in Cluster reconciler

Job name validation in v1.21 doesn't check for DNS naming convention
Job name validation in v1.22 checks for DNS name but doesn't allow subdomain convention (dot - .)
Awaiting reply from k8s slack why validation used DNS1123 but not DNS1035 label format.
This bug seems to have regressed somewhere b/n release 2.2.0 and current main
The only side-effect I can think of duplicate jobs in currently deployed operator
Pls let me know of any changes if required (removed the getRef based on make test)

@leelavg  you push the commit before I modify pipline. please repush again to trigger new pipline

for side effect , the prepare vg job can run repeatly,  so I think there is no problem.

/lgtm


Embedded videos are not played

Hi,
I tried to read cc-shared-culture-20120130.epub but none of the embedded videos are played.
Package.nativeGetArchiveInfoSize(): no archive found for path 'OEBPS/video/shared-culture.mp4'
Package.nativeReadStreamForRelativePath(): no archive found for path 'OEBPS/video/shared-culture.mp4'

Branch : develop
Lastest commit : 7f964be716eb5e34d9ae63278f890f545b1a3c18
Android version : 4.4.2

Addressed by #50


fix(network): comment out quic - fails to use transport

Pull Request type
Please check the type of change your PR introduces:

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: N/A
What is the new behavior?

Does this introduce a breaking change?

[ ] Yes
[ ] No

Other information

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

Current dependencies on/for this PR:

#1739
#1738  👈
#1729
#1704
#1670
#1715
main

This stack of pull requests is managed by Graphite.


Add description for MEI virtual unit

This PR adds a description of the nature of MEI virtual unit to chapter 3 of the Guidelines.
Closes #782

Thank you @doerners .
Please note that there is also https://github.com/music-encoding/music-encoding/blob/374ce487fd0a8041e9f01db5a61266889362b847/source/modules/MEI.visual.xml#L1493-L1496
Maybe it's possible to harmonize the wording in all three passages to avoid diverging interpretations?

In terms of harmonisation I'd prefer the "middle" of the staffline, because then page orientation is not of importance.
If you would be so kind as to harmonise it in the other spots (visual and shared)


TypeError: AudioFeatures.init() got an unexpected keyword argument 'wakeword_models'

This looks like a great package and a good way of making hot word detection cheaper to implement so thank you to everyone putting time and effort into this project.
I'm attempting to utilize it to facilitate wake word detection for a music related AI assistant - Landing page here that explains everything if anyone is interested.
I'm running the repository inside a UTM virtual machine Ubuntu 24.0.4 on an 2020 M1 macbook pro. However I am seeing the following error.
Full terminal output
(.venv) jbflow@tlkbck:~/clones/openWakeWord/examples$ python3 detect_from_microphone.py --model_path hey_jarvis_v0.1.tflite
ALSA lib pcm.c:2721:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.rear
ALSA lib pcm.c:2721:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.center_lfe
ALSA lib pcm.c:2721:(snd_pcm_open_noupdate) Unknown PCM cards.pcm.side
ALSA lib pcm_route.c:878:(find_matching_chmap) Found no matching channel map
ALSA lib pcm_route.c:878:(find_matching_chmap) Found no matching channel map
ALSA lib pcm_route.c:878:(find_matching_chmap) Found no matching channel map
ALSA lib pcm_route.c:878:(find_matching_chmap) Found no matching channel map
Cannot connect to server socket err = No such file or directory
Cannot connect to server request channel
jack server is not running or cannot be started
JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock
JackShmReadWritePtr::~JackShmReadWritePtr - Init not done for -1, skipping unlock
Traceback (most recent call last):
  File "/home/jbflow/clones/openWakeWord/examples/detect_from_microphone.py", line 57, in <module>
    owwModel = Model(wakeword_models=[args.model_path], inference_framework=args.inference_framework)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/jbflow/projects/tbww/.venv/lib/python3.12/site-packages/openwakeword/model.py", line 139, in __init__
    self.preprocessor = AudioFeatures(**kwargs)
                        ^^^^^^^^^^^^^^^^^^^^^^^
TypeError: AudioFeatures.__init__() got an unexpected keyword argument 'wakeword_models'

Any help resolving this issue would be appreciated, please let me know if any further information is required.

It seems the issue was simply I was somehow running an older version of the example script. Making a fresh clone of the repo and running the script from the main branch worked fine so closing this issue


update context and tf .14 support

what
fixes and updates for tf 0.14 compatibility

/test all

/test all


[Caching] Add fast swift instance setup for cache replay

Add a fast path to create swift CompilerInstance when it is only used to replay output when there is a cache hit. The normal setup function is very expensive to call, especially in cache mode to setup inputs, and it needs to be called once per input file from libSwiftScan API due to the current caching granularity.
The fast path will only construct the part that is needed for output replay, including the CAS, the output backend and caching diagnostic processor.
rdar://127062609

@swift-ci please smoke test


TIme Sheet (P950), inconsistent property values

On the Time Sheet (P950), there is a collection of 7 fields, named Field1..Field7, that are filled dynamically from the Time Sheet Detail (T952).  Fields1..Field5 all have the DecimalPlaces property set to 0:2.  Field6 and Field7 do not have this property set.
The DecimalPlaces property cannot be overridden in an extension and because the source for the fields is a variable (an array item actually) I cannot simply hide the field and then add a new field with the same source and correct property setting.
Please correct this in base.

This also applies to P954 Manager Time Sheet by Job


fix(kibana): remove duplicated elasticsearch-certs volume

thanks!
i have rebased and merged it here:

https://github.com/wiremind/wiremind-helm-charts/pull/616


Add skeleton component

https://codepen.io/stevenlei/pen/NWRBYJP
https://freefrontend.com/css-skeleton-loadings/

hey! can i get more informaton regarding this issue like on which page is it to be implemented


WIP - update microsoft-defender-atp-mac.md

Added 3 new sections

What's new
What to expect in the portal
Config commands for CLI

Split up original page into 4 shorter pages, covering 3 different installation methods + 1 resources page

Original page still exists, containing some brief information as an overview

Distributed new subsections to split pages as needed

What to expect is under Resources page
Config commands are under Manual installation page

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

Re-opened since my partner Max is working on the public repo...

Looking for a content review before merging

Looks good.


feat: Support for recursive and circular references

This PR adds support for recursive and circular references in properties and additionalProperties.
Related To/May Close

#338
#466

Known Issues

Recursive/circular reference paths that are required all the way through would be impossible to use in generated clients

class Example:
    # This would be impossible to create as the `example` attribute would require infinitely
    # recursive creation of Example objects
    example: Example

@maz808 Hello! Thanks for your work and PR! Seems like the generated client doesn’t work due to circular import exceptions.
For example:
openapi: 3.0.3
info:
  title: 'Example'
  version: 0.1.0
servers:
  - url: 'http://example.com'
paths:
  '/foo':
    delete:
      responses:
        '200':
          description: OK
components:
  schemas:
    Brother:
      type: object
      properties:
        sister:
          $ref: '#/components/schemas/Sister'
    Sister:
      type: object
      properties:
        brother:
          $ref: '#/components/schemas/Brother'

When I trying to use Sister model
from example_client.models.sister import Sister

if __name__ == '__main__':
    Sister()

File “…/example_client/models/__init__.py", line 3, in <module>
    from .brother import Brother
  File "…/example_client/models/brother.py", line 5, in <module>
    from ..models.sister import Sister
  File "…/example_client/models/sister.py", line 5, in <module>
    from ..models.brother import Brother
ImportError: cannot import name 'Brother' from partially initialized module 'example_client.models.brother' (most likely due to a circular import)

Seems like the first problem should be solved in the way that supports circular references. It has several workarounds. But I like the approach to use just one module for storing all models, like module models.py with
from __future__ import annotations

@attr.s(auto_attribs=True)
class Brother:
	…
    sister: Union[Unset, Sister] = UNSET
	…

    @classmethod
    def from_dict(cls: Brother, src_dict: Dict[str, Any]) -> Brother:
        …

@attr.s(auto_attribs=True)
class Sister:
	…

@dbanty What do you think about it?

This PR lives on through @mtovt's work in #670 🥳

@dbanty I noticed the release page doesn't acknowledge my contribution to the recursive and nested schema issue. Could I be added to the list of thanks?

Weird... I remember explicitly adding you. Sorry about that, I'll fix it!!

@dbanty I thought you had too 🤔 I appreciate it though! Thank you!


⚠ SolarQuery has degraded performance

In 5d487e9, SolarQuery ($SOLARQUERY_URL) experienced degraded performance:

HTTP code: 200
Response time: 678 ms

Resolved: SolarQuery performance has improved in 3f66822 after 9 minutes.


Update ngx_brotli_module to the latest brotli

Google released 3? new Brotli releases since the release of Cloudflare's ngx_brotli_module.
Please update ngx_brotli_module to the latest Brotli.
Thanks.

@vkrasnov @cloudflare ping

@lenovouser  , @vkrasnov has updated his repo.
I've tested it and it works properly with the latest Brotli git clone.


HSEARCH-2509 & HSEARCH-2467 Upgrade to Jest 2.0.4 to fix gson serialization issues with numeric range queries

Relevant tickets:

https://hibernate.atlassian.net/browse/HSEARCH-2509
https://hibernate.atlassian.net/browse/HSEARCH-2467

merged!


Expose more time parsers

Hope this isn't too savage.
! @jystic

I'm a little bit worried about using these in revolution, they have a specific meaning which is "zebra text time format", I could change what this looks like at any time.
I think revolution should have its own parsers. If you want to use these implementations as a starting point you'd be better off copying the implementations to revolution rather than using these directly.

yeah I was tossing up between those options tbh. Revolution has the responsibility to parse ISO dates as well as enterprise dates that zebra just doesn't.


MetricsTest.java:41-43: Impediment: test for LCOM5 with...

The puzzle 18-f4ca747e from #18 has to be resolved:
https://github.com/yegor256/jpeek/blob/afff84db718691401c1a47914b387fb171db9564/src/test/java/org/jpeek/MetricsTest.java#L41-L43
The puzzle was created by George Aristy on 29-Jan-18.
Estimate: 30 minutes, role: IMP.
If you have any technical questions, don't ask me, submit new tickets instead. The task will be "done" when the problem is fixed and the text of the puzzle is removed from the source code. Here is more about PDD and about me.

The job #125 assigned to @rexim/z, here is why. The budget is 30 minutes, see §4. Please, read §8 and §9. If the task is not clear, read this and this.

The puzzle 18-f4ca747e has disappeared from the source code, that's why I closed this issue.

@ypshenychka/z please review this job, as in §30

@0crat quality good

QA review completed: +15 points just awarded to @ypshenychka/z, total is +75

Order was successfully finished: +35 points just awarded to @rexim/z, total is +170

@0pdd all puzzles are solved

@0pdd all puzzles are solved

@0pdd all puzzles are solved


Replace master/slave terminology with primary/replica

Should we keep both terms in a first pass to make the transition easier?

I’m fine with the proposed change, but 1) we need to ensure backwards compatibility (until next major version), and 2) master branch is currently a Rails 5.2 compatibility release candidate. I’d like to have that 5.2 compatibility properly released before merging this change.

It would be nice with a description.

It would be good to keep the old terms initially for compatibility, and maybe add a deprecation warning?

closing this in favor of #216


Fix mobile in dark mode

What's wrong ?
Dark mode in mobile is broken
What's inside this PR ?
A color change on the correct class in mobile, with an !important to override the light mode's !important 🙄
Screenshots
Before:

After:

Mobile searchbar is a bit broken but I think the problem may be related to docs-searchbar.js and not this plugin

compared to algolia:

Mobile searchbar is a bit broken but I think the problem may be related to docs-searchbar.js and not this plugin

compared to algolia:

Actually that's because of the layout "columns" by default, which is not a good idea in mobile.
If you change the layout to "simple", it's much more better :)
But yes, people shouldn't have to choose the "simple" layout because the columns one seems broken in mobile, I totally agree !

bors merge


PSResourceInfo should include functions, cmdlets, etc

Summary of the new feature / enhancement
Currently if you want to see which functions or cmdlets are included in a module you have to inspect the Tags property for a PSFunction_ or PSCmdlet_ or PSCommand_ and then replace the prefix to get the actual value. There should be a Functions, Cmdlets, and Command properties.
PSFunction_New-RandomString
PSCommand_New-RandomString
PSFunction_Add-VM
PSCommand_Add-VM
PSIncludes_Function

Proposed technical implementation details (optional)
No response

Missed the Includes property


Support Java 21

The FunctionInstanceInjector SPI is no longer used under Java 21. The class instantiation is simply not going through the Factory defined by this interface.

Some unknown plugin moved my SPI file under an invalid path resources/META-INF/services/dependency-injection/...
Which makes no sense. I'll have to find the culprit and re-file. Ignore this issue.


Could not create project, error ELIFECYCLE, failed at libui-node install script

Here is log:
16978 error code ELIFECYCLE
16979 error errno 1
16980 error<EMAIL_ADDRESS>install: "libui-download && autogypi && npm run build"
16980 error Exit status 1
16981 error Failed at the<EMAIL_ADDRESS>install script.
16981 error This is probably not a problem with npm. There is likely additional logging output above.
Thank you..

I am getting the same error.

same here, please help

This error appears to be related to Node v12. Can you confirm the version you are using?
I've been able to get create-proton-app working as per Quick Start guide using Node v8.16.1. It does not work on v10.16.3 LTS or beyond.
However, I was able to get the demo apps working using Node v10.16.3 LTS by following the Old Manual instructions.

Just tested and was able to get the demo notepad app working using node v11.15.0.
So it looks like the last Node versions that can be used  are:

create-proton-app: Node v8.16.1
manual method: Node v11.15.0

Hope this helps someone.

Likely also relevant libui-node issues: 132 and 142 (which links to bcoin-org/bcrypto#7 which might have the useful solution)
Solution might be to update libui-node to use v12 compatible compilation like the bcrypto issue did.


[No Review]: Collect cherry picked branches for 4.45.2 hotfix release.

API change check
API changes are not detected in this pull request.


Use truncated instead of Truncated

Just noticed that DiffEqBenchmarks uses the deprecated Truncated instead of truncated which (potentially) creates optimized truncated distributions and uses Truncated as fallback. I remember some simple models in Turing being quite slow due to deprecation warnings, so probably better to use truncated in the benchmarks.
I'm not sure what's the correct procedure for rerunning and regenerating the different output formats?

The procedure to run it is documented in https://github.com/SciML/DiffEqBenchmarks.jl#contributing .  It might be good to re-weave all 3 of those. @Vaibhavdixit02 and I have been discussing the Lorenz one, because Stan fails with those priors by just never finishing, so we are trying to find a good way to document that behavior, like cut it early or something. If you have any ideas go for it.


fix top Navigation

When using topNavigation the sidebarCollapsibleOnDesktop has to be false if it is already enabled in the panel
Before:

After:

and thank you for this amazing plugin :)

@atmonshi thanks for the PR


Fix conversion of recursive schema

Hello @fabian-hiller 👋
It seems that we forgot to test the conversion of recursive schema 😬.
It currently results in an error RangeError: Maximum call stack size exceeded.
I think I have a solution, let me know what you think of this or if you find a better solution

Thanks again for your contribution! I fixed both bugs based on your PR, but with a few changes in this commit. I have added you as the co-author.


Add MAX78002 SoC files into Zephyr system

This PR adds MAX78002 SoC files into the Zephyr system to enable Zephyr support for this SoC. Also, it includes some updates (which already merged to MSDK repository) for bare-metal drivers.

@MaureenHelm FYI.

Can you provide links to the MSDK PRs please?

Can you provide links to the MSDK PRs please?

Updated PR message with MSDK PR links

MAX78002 PR passed all twister tests. If it is okay for you, please merge this PR. @MaureenHelm


How to build the project locally?

I would like to thank you for this example which really looks intersting and exiting. But I struggle a bit how to build it locally.
Could you please just add some instructions into the readme file, what I need to do in order to build and run it locally?
Thanks 😃

Thanks for the response :)
After doing everything you have written in the docs and the running npm run start in the app folder I still get this error:

Also in the app folder I don't see any compiled C# assemblies. You mentin a postbuild script. Is this a file or just some configuration? Because I coudn't find any such file.


Migrate with RST-docs based repo

Note: I will leave current .md files for mentees to write rst based docs on doc/source folder.

(Self-merging, agreed by @ppiyakk2 )


Usage with React Context

I am considering in using mutative, but I must admit I lack experience to "safely" integrate this kind of library myself to the usage with React Context.
Is there an example I could see of how to implement a React Context using useMutativeReducer() ?
and achieve the best performance possible?
It perhaps could follow the same example as React Docs -> https://react.dev/learn/scaling-up-with-reducer-and-context

hi @IrvingArmenta , referring to the React documentation is a good choice. You should note that in useMutativeReducer, the state update logic in the reducer is 'mutative'. For example, this documentation https://github.com/mutativejs/use-mutative?tab=readme-ov-file#usemutativereducer


ui-scroll: loading data only at last scroll bar position

Hello,
if you are at page 100 and drag scroll bar to page number 20, is it possible to load data only for that last page, skipping pages in between?

That would probably mean throttling

Yes, can we adjust throttling some how?

Actual code base of ui-scroll is in https://github.com/Hill30/NGScroller. Please re-submit your issue over there if needed.

@RimvydasUrbonas I would look into _.throttle or some sort of equivalent solution. It doesn't need to be a feature of this module.

No this is not a throttle issue. Because even i put _.throttle on viewport.bind('scroll', _.throttle(resizeAndScrollHandler, 500)) on scroll.js, it comes back step by step from end to begining. It's a problem when I am e.x. at 900 to 1000 items range and I drag scrollbar at the beggining, so I expect jump directly to 0 to 100 range, but unfortunately scroll goes to 800 - 900 then 700-800 then 600-700 and when request takes few seconds to retrieve data it takes plenty of time to return to 0 - 100.


feat(*): Added Grafana and AWS CRDs

@Moep90 can you please add a link to the crd origin on github?

@eyarz sure!
Their repository is building them using Kustomize I suppose, but here is the link to it https://github.com/grafana/grafana-operator/tree/master/config/crd

@eyarz where do you want me to add the CRDs?
I did not found an example in the contribution guidelines as well as the latest PRs
However, the CRDs can be found here: https://github.com/grafana/grafana-operator/tree/master/config/crd

I just needed the link so we are set
you're right, there is no contribution guidelines and you are more then welcome to help with that 😅


Remove WorkFlow

The Python automated workflow is not working but the code is working,

Hope this issue is solved


Match and MatchAttachment Test Coverage

Not all match and match attachment related client methods are validated in the test suite. Need additional tests to ensure proper functioning.

Resolved in commit 2c41a0cbbf6f67d5a9544d2e8b2e3ed009f4c99e


Could Thales Main Page Be Updated?

Thoughts on this? Quick and easy thing; I think good for visibility and adds easy access to Twitter and the dApp.
Example of what I mean:

Oh wow you supaaaa fast! 💪
Looks amazing, great upgrade! Nice work.....!


minor updates

twitter like csv config + other updates

I am merging the PR so I can verify my updated queries. Ted, please follow up on Yi-Hong's comment in your next PR.


relationship link for belongsTo relationship

for hasMany / belongsToMany relationship, link was shown as:
               "comments": {
                    "links": {
                        "self": "/api/v1/posts/1/relationships/comments",
                        "related": "/api/v1/posts/1/comments"
                    }
                }

but for belongsTo relationship, link was shown as:
               "user": {
                    "data": {
                        "type": "users",
                        "id": "4"
                    },
                    "links": {
                        "self": "/api/v1/posts/1/relationships/user",
                        "related": "/api/v1/posts/1/user"
                    }
                },

where link of either "self" or "related"
like "/api/v1/posts/1/relationships/user" and "/api/v1/posts/1/user" were not reachable, returning 404.
is it possible to trigger the link to be reachable?

or is it possible to return like, for example:
                    "links": {
                        "self": "/api/v1/user/1",
                    }

How to make it 'reachable'? Add it to routes with the corresponding handler in the controller. For example.

How to customize links section and exclude related? You've got a few options to do that

Turn it off by default for all relationships
    protected function isShowRelatedLinkInRelationships(): bool
    {
        return false;
    }

Or exclude specific relationship
    protected function getExcludesFromDefaultShowRelatedLinkInRelationships(): array
    {
        return parent::getExcludesFromDefaultShowRelatedLinkInRelationships() +
            [
                self::REL_ROLE => true,
            ];
    }

Or remove/add default links
    public function getRelationships($model, bool $isPrimary, array $includeRelationships): ?array
    {
        $default = parent::getRelationships($model, $isPrimary, $includeRelationships);

        unset($default[self::REL_ROLE][self::LINKS][DocumentInterface::KEYWORD_RELATED]);

        return $default;
    }

Thanks!


using this gem on Windows is not working Av::CommandError

Hi, this is my first issue posting on github, so please if I'm not doing something right, please tell me :)
OK, so we have one project that is working on Linux version of server on AWS, and I use Windows as a OS. When I want to create new attachment using this gem, I get following error:
**Av::CommandError in Api::V1::DigitalProductsController#create
error while running command if command -v avprobe 2>/dev/null; then echo "true"; else echo "false"; fi: Command 'if command -v avprobe 2>/dev/null; then echo "true"; else echo "false"; fi' returned 1. Expected 0 Here is the command output: STDOUT:
STDERR:
-v was unexpected at this time.**
_I'm reporting this as an issue, as gem is working fine on Linux, but on Windows it is not good. _
I have tried to do everything that others users reported similar problems, but without any success. Also, on stack or Google, there is no search result with my issue, like nobody has this problem :(
Please can you see if there is an issue for real, or there are some things that are too obvious that I have missed. I'm sending some screens just in order to have full picture.
Thank you.

Hi, I have "fixed" av-0.9.0 gem by ignoring validation self.detect_command('ffmpeg') and have only ffmpeg command available as a choice, and now it works also on windows. This was kind of dirty method of removing the issue, but it works.
So for anybody that will face same issue, go to installed copy of av gem and find file:
av/cli.rb
then in that file remove line 7 "found << 'avconv' if self.detect_command('avprobe')"
then remove in line 8 validation "if self.detect_command('ffmpeg')"
BUT please ensure that you have ffmpeg working before you do this!
I'm writing this here only to help windows developers as this is a problem that will appear using this gem.

Hi, i have the same problem with windows.
I have do exactly what you recommend, but i don't know how to install this "fixed gem" in my project;
Can you tell  me how you do ?
P.S: just for information, i am learning rails and web-development since on month...
Thank You


Elements receive zero-th property part value for all properties

Please consider this reduced test case and check the console: https://glitch.com/edit/#!/tremendous-jeep?path=x-example.js:12:75
What I expected
Element props follow: foo="foo" bar="bar" disabled="true"

What I got
Element props follow: foo="foo" bar="foo" disabled="foo"

Fixed in #55


Angular always reports this component as untouched

You can see this in the chrome inspector even in the examples section. The dirty/pristine works fine but the component always reports untouched even after changes. This has caused some issue with validation on touched.  I hope this makes sense. Thanks.

@ryanmagits : Yes as of now only required and dirty/preistine are supported on this component. Touched and untouched are queued for next release. Also, I would appreciate if you would raise pull request for fixing this.

fixed. Fix available in v2.1.4


Highlighting for .R files not working

Hi there,
I don't seem to manage to get the highlighting to work for my .R files. What am I missing? I have already tried to go through all the settings twice and looked for answers online, unfortunately without luck.
Comments are green, logicals (true/false) are bright blue and string shows as brown whereas the rest of the code is dark blue, but apart from this, I don't get the nice (and most of all useful) color highlighting I see in your snapshots. Just as a matter of information, I work on a mac.
How can I fix this?
Would be very grateful for any help or hint about what the problem could be.
Dominique

@wdomi Thank you for using.
I could not understand how is your code.
Could you put your screenshot?
And, you can edit this file to fix highlight
https://github.com/Ikuyadeu/vscode-R/blob/master/syntax/r.json

Hi,
Thank you for attaching again the json link. I am sorry to say that I don't understand how to edit it to fix the problem.
Here a screenshot

here a snapshot of another R file (using tidyverse syntax).

Many thanks for your help.
Dominique

This doesn't seem to be related to the R extension. You just need to change your theme to something that uses more colours. Note that most light themes aren't that flashy; if you want lots of colours, try a dark theme.

Oh man, thanks so much! This was very helpful!
Sorry to bother you with such a naive issue!
Many thanks!
dominique


Expected id of elements of field nodes to be strings.

Common database id field is not allowed in Relay modern:
https://github.com/facebook/relay/issues/1682
I suggest to generating id as _id.

Try --classic-ids
https://github.com/postgraphql/postgraphql/blob/master/docs/cli.md

It works! Thank you very much.
I am working on demo based on your forum example: https://github.com/vysokyj/postgraphql-relay-demo

Excellent 👍

Try --classic-ids
https://github.com/postgraphql/postgraphql/blob/master/docs/cli.md

Sir Benjie... dead link. I'm using _id for my ids but still getting RelayResponseNormalizer: Expected id on field 'foobar' to be a string.

You need to use —classic-ids. https://www.graphile.org/postgraphile/usage-cli/


When running multiple commands error is only reported if the last command fails

When running multiple commands as part of a block, error is only reported if the last command fails.
→ cat /tmp/test.md
# Hello

```bash
doesnotexit
echo "Hello"
```
→ ie execute /tmp/test.md
Hello
1. Hello

  ✔ doesnotexit
    echo "Hello"

  Hello

As you can see doesnotexit fails but the execution is marked successful. Perhaps there should be a way to do set -e for such blocks?
Usecase:
I was testing my README.md as part of CI and only found out the issue only once I moved the failing command to independent block.

 kubectl krew install aks should fail but works fine.
kubectl krew unistall aks fails only because it is the last (only) command in code block.

This has been resolved in #155


Captcha not showing up, missing libfreetype

Hi,
I set up seafile using your docker compose setup and I noticed that after I typed wrong password oft enough I will get a captcha prompt by no captcha image.
I get this error in my seahub log:
2021-06-05 18:58:12,026 [ERROR] django.request:228 log_response Internal Server Error: /captcha/image/d1eb0a433a290465afb68dfcb2ce3fe595a36774/
Traceback (most recent call last):
  File "/opt/seafile/seafile-server-8.0.4/seahub/thirdpart/django/core/handlers/exception.py", line 34, in inner
    response = get_response(request)
  File "/opt/seafile/seafile-server-8.0.4/seahub/thirdpart/django/core/handlers/base.py", line 115, in _get_response
    response = self.process_exception_by_middleware(e, request)
  File "/opt/seafile/seafile-server-8.0.4/seahub/thirdpart/django/core/handlers/base.py", line 113, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/opt/seafile/seafile-server-8.0.4/seahub/thirdpart/captcha/views.py", line 67, in captcha_image
    font = ImageFont.truetype(fontpath, settings.CAPTCHA_FONT_SIZE * scale)
  File "/opt/seafile/seafile-server-8.0.4/seahub/thirdpart/PIL/ImageFont.py", line 853, in truetype
    return freetype(font)
  File "/opt/seafile/seafile-server-8.0.4/seahub/thirdpart/PIL/ImageFont.py", line 850, in freetype
    return FreeTypeFont(font, size, index, encoding, layout_engine)
  File "/opt/seafile/seafile-server-8.0.4/seahub/thirdpart/PIL/ImageFont.py", line 185, in __init__
    if core.HAVE_RAQM:
  File "/opt/seafile/seafile-server-8.0.4/seahub/thirdpart/PIL/ImageFont.py", line 44, in __getattr__
    raise ImportError("The _imagingft C module is not installed")
ImportError: The _imagingft C module is not installed

It looks like it just needs the libfreetype6 package. Here is an issue and the fixing commit in another seafile docker repository.

Hi,
I just pushed the image for Seafile 8.0.5 yesterday and it seems working out the box:

Try to update with:
docker-compose down
docker-compose pull seafile
docker-compose up -d

Let me know if that solved your issue.
There's always the possibility the library's missing for the armv7 image only, I'll have a look at it and fix if necessary.

Okay this is clearly a bug on armv7 only, I tried a fix and will build the images tomorrow to see if everything is ok.
Thanks for your reporting!

Patch is on the hub, update as explained before and you should be ok.
Enjoy using my project!

@ChatDeBlofeld,  thanks for fixing this so fast!


Add GxBuildHasher::with_seed

This PR adds a new method to build a GxBuildHasher with a given seed.
See https://github.com/ogxd/gxhash/issues/79 for the corresponding issue.

Hello @imartayan, that'd be a good addition. I'll merge it but then change the type from i64 to u64 afterward given this issue (in this order so that I can merge your PR now)


RE1 Biorand Error - Ran out of free slots

Version
3.0.9 (4d8d5d6)
Seed
R710-0000-ZTQA00C72Z660N9MMAM0ZZZZZZZZZZZZZLY
Scenario
RE 1 (Chris)
Player Character
Chris as Wesker and Jill as Rebecca
What happened?
I went to generate the seed, but when it finished I got an error message saying there were no more free slots left and to report this error to you.

No longer accepting reports for this issue. Work around is to try another seed.


Feature idea - Keeping main-module-browser field triplet together in package.json on defaults

Hi! First, thanks for this amazing tool. I'm about to call the first dibs in your "dependents" list on npm :)

When running on default setting, I noticed that the main key is kept sensibly towards the top, but browser and module keys are sorted alphabetically among "unrecognised" keys.
In the nearest future, more and more libs will start using module and browser keys and it would be nice if the default setting on format-package would gang them together.
For example, Rollup boilerplate is outputing them and in my libraries I set something like:
...
"main": "dist/util-nonempty.cjs.js",
"browser": "dist/util-nonempty.umd.js",
"module": "dist/util-nonempty.esm.js",
"scripts": {
  "build": "rollup -c && npm run format:pkg",
...

It would be nice to visually-grep are the paths correct, are the names the same, are extensions right – all in one place.
@camacho What do you think about this idea? Would it be sensible to put browser and module keys right under main, on the default format-package setting?
Cheers!

This sounds like a good idea to me. It's tricky to know how much opinion we should bake in versus let users set with their own config, but I don't see anything controversial about adding those keys under main

Awesome!
Implementation-wise, do you have to add two new fields, module and browser under main in https://github.com/camacho/format-package/blob/master/lib/defaults/order.json and that's it?

@revelt yup!

This should now be updated in the 3.0 release today

awesome!

By the way, even if I tried to switch to clinton which also does sorting on --fix mode, I can't use it because it would "fix" many other things that don't need fixing. I like format-package because it does one thing only. This is a very important USP of this lib.


On-premise should be on-premises

In Step 1, the architecture chart lists On-Premise as a type. This should be On-premises

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

ID: e457d022-6b68-137b-5723-098c6d301af6
Version Independent ID: df9dc070-ee8b-3cc5-8ea7-68dc2d5abb56
Content: Identify Defender for Endpoint architecture and deployment method
Content Source: microsoft-365/security/defender-endpoint/deployment-strategy.md
Service: microsoft-365-security
Sub-service: mde
GitHub Login: @siosulli
Microsoft Alias: siosulli

@MarcLaf Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/12694
Once the author approves, the changes will be live.
Thanks
Sri


chore: update readme links

Related to: https://github.com/SAP/cloud-sdk-backlog/issues/888, as the error handling page should have better visibility.
I adjusted the readme file:

add support channel link
add error handling doc
use portal release notes instead of GitHub release notes

Is this related to a BLI or just a general approvement? If there is a BLI please link it in this issue.

No, it's a general improvement.
I linked to a related ticket, though.


Request: Space from center

I'd love the ability to space a set of elements' center a fixed distance apart.
The workaround is to put these in equal width containers, but I'd love to remove that step.

This could go as an option in the distribution settings dialogue, right?

Yup, it's definitely a boolean option. 👍

Hi! I added this feature in the following PR : https://github.com/PEZ/SketchDistributor/pull/27
Hope you'll enjoy it :)

I pulled #27 now. Now trying  to get the new version out.

Now it seems to be out there. Version 1.2.1. Please let me know if it works for you guys or not.
One thing I might want to fix is that it probably should ”remember” the combination of direction and anchor. If that makes sense. But right now if you distribute vertically between centers also next horizontal distribution will default to center-spacing.


Document compatibility of json filters

I use this tool all the time to automate tasks over json APIs with shellscripts. It's the quickest way to get a working jq selector. Most of the times it works, but I take it that compatibility is not full.
It would be nice to have the limitations documented in the readme.

Hi @plainas :)
Can you give me an example of the limitations you're referring to? I don't know of any JSON documents that gron is unable to handle

Well, it has actually worked for me. I don't know if I ever found a selector that is not jq compatible. If I did I can't recall it specifically. I was under the impression, from the discussions on hacker news when this was posted a while ago, that it was not intended to be fully compatible with jq. But maybe I got it wrong. Is it? Are the paths always valid jq selectors?
Hey, thanks for such useful tool.

Ah, I think I misunderstood your original point sorry!
Compatibility with jq isn't a design goal for gron, but outputting valid JavaScript is (by default at least). I'm not familiar enough with jq's selector syntax to know for sure if it's always compatible, but gron's output grammar is fairly simple, and is defined here if you want to see if it's likely to ever spit out anything that's incompatible: http://tomnomnom.github.io/gron/
I'm glad gron fits your use-case so far! Sorry I couldn't be of more help!


Update sbt to 1.5.4

Updates org.scala-sbt:sbt from 1.5.2 to 1.5.4.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Files still referring to the old version number
The following files still refer to the old version number (1.5.2).
You might want to review and update them manually.
modules/hariko-core/src/main/scala/codes/quine/labo/hariko/Cogen.scala

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scala-sbt", artifactId = "sbt" } ]

labels: library-update, semver-patch, old-version-remains

Superseded by #57.


add contrast control button

In some folders (like Travis/contrib/W3KG233/archive/W3KG233-I3KG995) the border between the two images is a little bit difficult to see, especially at lower resolutions (4x3, 3x2, etc.). It could be interesting to have a button to display the images with more contrast (directly from the UI or, if it's really too slow, we could have an option in the server query for the thumbnails

done: https://scamqc.bdrc.io/?folder=Travis%2Fcontrib%2FW3KG233%2Farchive%2FW3KG233-I3KG995%2F

impressive, thanks a lot!


Missing steps in general usage documentation

📚 Docs or Wiki Bug Report
Description [REQUIRED]
I did the steps you suggested to resolve the error and the “make build” and “make docker” succeeded.  However, https://github.com/edgexfoundry/device-usb-camera/blob/main/docs/general-usage.md then suggests the following:
Run device-usb-camera
Docker
For non secure mode
o   make gen ds-usb-camera no-secty
For secure mode
o   make gen ds-usb-camera
Docker Compose start command
docker-compose -p edgex up -d

When I tried either of the first two, it reports:
Make: *** No rule to make target ‘gen’.  Stop.

I looked through the Makefile and there is not a target gen, so not sure if there was a step I missed that configured the Makefile to have a gen: target or if this is an error in the instructions.
🔬 Minimal Reproduction
What's the affected URL? [REQUIRED]
https://github.com/edgexfoundry/device-usb-camera/blob/main/docs/general-usage.md#run-device-usb-camera
Expected vs Actual Behavior

📷Screenshot

🔥 Exception or Error

🌍  Your Environment

@ajcasagrande , is this still an issue with latest docs moved to edgex-docs?


Added logging

Hi @nunomaduro,
I really like the idea behind this framework and wanted to contribute.
So I took up the simple tasks of adding logging to the framework :)
Design choices:

Used Monolog
Added a few processers to add context to each logging entry, using build-in Monolog processers and added a request id so you can find all log entries by looking up that request id
The location storage/logs is open for discussion

I also commit the composer.lock file, I'm not sure why this is not commited in the first place?
Let me know what you think :)
Visti

@vistik Hey buddy, are you fine of keeping this PR on hold until we define how service providers should work on narration?

@nunomaduro No I dont mind :) Let us not merge stuff unless you are happy with the design.
Is there anyway to be more included or at least read your thought on the development of the framework so I can get a better understanding of the framework and your design ideas/process?

@vistik Yes. I will be start working in the framework on March. I will keep you posted.

@nunomaduro Great I will be looking forward to that. Do you have any parts of the framework you want me to take a stab at?

@vistik @tutNichts Yes. I really want to us to start discussing components/decisions using RFC. Just like this https://github.com/narration/narration/issues/13.  Before people start coding.