What the iou threshold in the F1 score of adjacency relationships is?

I dont know what the iou threshold in the F1 score of adjacency relationships is.
Beacuse the iou threshold in cell detect is 0.5, I want to know if the iou threshold in the F1 score of adjacency relationships is also 0.5 .
If not 0.5，please let me know the exact threshold.

Can you open source the evaluation code for the F1 score of adjacency relationships part?


Fix sed

Mandatory Checklist
Please ensure that the following criteria are met:

[x] Title of pull request describes the changes/features
[x] Request at least 2 reviewers
[x] If new files are being added, the files are no larger than 100kB. Post the file sizes.
[x] Code coverage did not decrease. If this is a bug fix, a test should cover that bug fix. If a new feature is added, a test should be made to cover that feature.
[x] New features have appropriate documentation strings (readable by sphinx)
[x] Contributor has read and agreed with CONTRIBUTING.md and has added themselves to CONTRIBUTORS.md

As a general rule of thumb, try to follow PEP8 guidelines.
Description

I'm still getting the error:
Proteus was built using the following configuration:
Please include this information in all bug reports.
+======================================================================================================+
PROTEUS        : /home/breizh/proteus2
PROTEUS_ARCH   : linux
PROTEUS_PREFIX : /home/breizh/proteus2/linux
PROTEUS_VERSION: 68e5aeb78c5a35ba080b0e5bd33719cd4f69dcf7
+======================================================================================================+

History:
 2008  make develop
 2009  cd air-water-vv/2d/sediment/bed_sediment/
 2010  parun tank_so.py -D test
 2011  history

Error message:
breizh@XPS-13-9370:~/proteus2/air-water-vv/2d/sediment/bed_sediment$ parun tank_so.py -D test
Traceback (most recent call last):
  File "/home/breizh/proteus2/linux/bin/parun", line 7, in <module>
    exec(compile(f.read(), __file__, 'exec'))
  File "/home/breizh/proteus2/scripts/parun", line 413, in <module>
    pList.append(proteus.defaults.load_physics(pModule))
  File "/home/breizh/proteus2/proteus/defaults.py", line 104, in load_physics
    p = imp.load_source(pModule, os.path.join(path, pModule+".py"))
  File "/home/breizh/proteus2/linux/lib/python3.7/imp.py", line 171, in load_source
    module = _load(spec)
  File "<frozen importlib._bootstrap>", line 696, in _load
  File "<frozen importlib._bootstrap>", line 677, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 728, in exec_module
  File "<frozen importlib._bootstrap>", line 219, in _call_with_frames_removed
  File "./threep_navier_stokes_sed_p.py", line 9, in <module>
    LevelModelType = RANS3PSed.LevelModel
AttributeError: module 'proteus.mprans.RANS3PSed' has no attribute 'LevelModel'

Or actually, I think misunderstood and am checking prematurely.... Sorry.

Yes, just missing some imports, possibly due to some refactoring of imports (removal of import* somewhere).


mm/kmm_map: Add support to dynamically map pages into kernel virtual …

…memory
Summary
This adds functionality to map pages dynamically into kernel virtual memory. This allows implementing I/O remap for example, which is a useful (future) feature.
Now, the first target is to support mapping user pages for the kernel.
Why? There are some userspace structures that might be needed when the userspace process is not running. Semaphores are one such example. Signals and the WDT timeout both need access to the user semaphore to work properly. Even though for this only obtaining the kernel addressable page pool virtual address is needed, for completeness a procedure is provided to map several pages.
NOTE: ARM target does not implement support yet, I'm not sure how to implement it.
Impact
Add new feature for CONFIG_BUILD_KERNEL, nothing else is affected
Testing
icicle:knsh

@pussuw is there some test that could be add to ostest to verify if dynamic map pages is working? Also please consider added some basic Documentation/ it will help people discover that NuttX supports it.

ostest does not currently work in KERNEL mode.  It relies things like task_create() which don't exist in KERNEL mode.

@acassis like @patacongo said currently ostest does not work at all for kernel mode. I tried doing an intermediate step https://github.com/apache/nuttx-apps/pull/1500 by forwarding task_create to pthread_create but that is not the same thing. The correct implementation would be to go through the tests case-by-case and:

When task_create is used to make workers, they can be replaced with pthreads
When task_create is used to make other "processes", they must be replaced with posix_spawn() or similar (as we don't have fork()). Making the tests compatible to be spawned requires a bit of refactoring.

As a summary; making all ostest steps work in kernel mode is doable but not an insignificant amount of work which I simply do not have time to do now.

When task_create is used to make workers, they can be replaced with pthreads

That most likely works because NuttX uses heavyweight threads, ie., coupled tasks that masquerade as pthreads.  So IPCs like semaphores, legacy signal functions, etc. work okay because PIDs, TIDs, and pthread_t's are are aliased; semaphore shared attributes are ignored; etc.  Certainly the result would not be POSIX compliant and so not a proper solution for a POSIX compliance test.  In order to be a POSIX compliant test, the test it would have to use pthread_sigprocmask() vs. sigprocmask(), for examples.  Signals are dispatched completely differently for pthread_kill() and kill().
It might make more sense to create a separate test for unique kernel build features.  Or maybe two versions of OS test that share as much they can.
Or maybe force the existing OS test execute with a file system and all tasks as files in the file system.  Then perhaps they could be made compatible.

Certainly the result would not be POSIX compliant and so not a proper solution for a POSIX compliance test. In order to be a POSIX compliant test, the test it would have to use pthread_sigprocmask() vs. sigprocmask(), for examples. Signals are dispatched completely differently for pthread_kill() and kill().

Yes indeed, using pthreads was a kludge solution (or maybe even a dirty hack) and that is why the PR is pending.

Or maybe two versions of OS test that share as much they can.

I think this is unavoidable, some split is expected.

Or maybe force the existing OS test execute with a file system and all tasks as files in the file system. Then perhaps they could be made compatible.

I think kernel mode needs this anyway, due to not having fork() i.e. only spawning from file works. I had some sort of idea to keep the current code pretty much compatible with the flat addressing modes too with wrapping / flagging. Your solution might be better but it does force compiling the tests into loadable files -> need more memory and features enabled to test the os.

I think this is unavoidable, some split is expected.

The basic tests should be all/mostly re-usable.  It really should be a big but low risk packaging change.

I think kernel mode needs this anyway, due to not having fork() i.e. only spawning from file works.

Does vfork not work?  fork() or vfork() would ultimately be followed by exec() and so would still need to be exec'ed from a file.
This is a good discussion, but unfortunately off-topic for this PR.

The basic tests should be all/mostly re-usable. It really should be a big but low risk packaging change.
Yes agree, mostly the signal/waitpid and such tests need to be refactored.

Does vfork not work? fork() or vfork() would ultimately be followed by exec() and so would still need to be exec'ed from a file.
I was thinking task_spawn in this case is sort of just fork()'ing without exec(). vfork IMHO is superseded by posix_spawn anyway and both need a file to exec(). This should all be doable with posix_spawn and I in fact started writing a wrapper for task_create that uses posix_spawn() but then something more important came up.

This is a good discussion, but unfortunately off-topic for this PR.

I do agree with Alan, we should have a test suite for these new features. However as AFAIK there are not many users of kernel mode right now / yet, and I can personally just use my application (which uses a wide POSIX feature set) to test the features I need. So again, writing the kernel mode support for ostest is unfortunately very low priority for me right now :(

I was thinking task_spawn in this case is sort of just fork()'ing without exec().

task_spawn() is not available in KERNEL mode either:
138 #ifndef CONFIG_BUILD_KERNEL
139 /* Non-standard task_spawn interface.  This function uses the same
140  * semantics to execute a file in memory at 'entry', giving it the name
141  * 'name'.
142  */
143
144 int task_spawn(FAR const char *name, main_t entry,
145       FAR const posix_spawn_file_actions_t *file_actions,
146       FAR const posix_spawnattr_t *attr,
147       FAR char * const argv[], FAR char * const envp[]);
148 #endif

Nothing that includes the address of main() can be used to start tasks in that mode since the address of main() is not available in that address environment.  You would have to use posix_spawn() which does take a file path instead of an address.  posix_spawn() is essentially a fork+exec.

What I mean is with fork() you could do something like overload the implementation of task_spawn (in ostest) like this:
int task_spawn(FAR const char *name, main_t entry,
       FAR const posix_spawn_file_actions_t *file_actions, /* This can be ignored afaik ? */
       FAR const posix_spawnattr_t *attr, /* This can also be ignored afaik */
       FAR char * const argv[], FAR char * const envp[])
{
  int pid = fork();
  if (pid == 0)
    {
      /* Child performs the task given to task_spawn() */

      entry(argc, argv);
    }

  /* Parent returns the child pid to caller */

  return pid;
}

Unless I'm missing something this should work (might not do what we want it to do, but works), as the main thread does indeed exist in the same address environment. Maybe there is something else I'm missing.
Using posix_spawn is the best option still, just need to make at least some of the tests into loadable processes and that should do it.

task_spawn() is not available in KERNEL mode either:

My bad, I meant task_create of course.. Has been a long day...
What I mean is with fork() you could do something like overload the implementation of task_create (in ostest) like this:
int task_create(FAR const char *name, int priority,
                int stack_size, main_t entry, FAR char * const argv[])
{
  int pid = fork();
  if (pid == 0)
    {
      /* Child performs the task given to task_spawn() */
      ... calculate argc etc ...
      entry(argc, argv);
    }

  /* Parent returns the child pid to caller */

  return pid;
}

Unless I'm missing something this should work (might not do what we want it to do, but works), as the main thread does indeed exist in the same address environment. Maybe there is something else I'm missing but as far as I can tell task_create() is in fact quite close to fork() (without the obvious address environment stuff).
Using posix_spawn is the best option still, just need to make at least some of the tests into loadable processes and that should do it.

We can keep the current ostest structure even switch to posix_spawn:

Assign the test case launched by task_create an unique id(string is a good choice)
If ostest is launched with one argument, do the test identified by the unique id
Change task_create to posix_spawn with "ostest" and the unique id

posix_spawn work on all mode(FLAT, PROTECTED and KERNEL), so we can use the same code base.

For both architectures, the page allocator support is in:

arm/src/armv7-a/arm_pgalloc.c
nuttx/mm/mm_gran/mm_pgalloc.c

I implemented the riscv address environment support long ago, it implements exactly the same public API:
https://github.com/apache/nuttx/blob/master/arch/risc-v/src/common/riscv_pgalloc.c


Roll Flutter Engine from 563b8e830c69 to 11d7135c3458 (1 revision)

https://github.com/flutter/engine/compare/563b8e830c69...11d7135c3458
2022-10-18<EMAIL_ADDRESS>Roll Dart SDK from 77e83fcc14fa to f21024945abf (1 revision) (flutter/engine#36854)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Allow the client/SDK to set the message UID

I think it is better for the official client/SDK to handle generating UID for messages, IMO it is important for the sender to be aware of the message UID, in cases like he wants to send two messages to the same twin dest and then wait for both responses on the same ret queue.
now when responses return, the client has no way to distinguish which response belongs to which message.
order can not be granted, as messages are sent with multi workers, also network routes can vary between the two requests.
so if we don't want to limit the client's ability to receive multi responses on the same return queue, we need to allow the client to set the message UID, then he can check this UID on the reply.

The use case you are already discussing is already possible. Please note the following

the message ID must be unique because it's used internally for message routing. Two message with same ids on the same node will cause issues.
While the client will not set an id (rmb does) the client CAN use the same return queue for all messages. This means that all his responses for all messages that use the same retuqueu will be received on that queue.
To fix the issue about not being able to link which response belongs to which message. I suggest adding a new optional tag field on the message. The tag is transparent and has no effect on the routing but will be used to mark the message for sdk reference.


Leethub not pushing code to github.

Please review previous closed issues before filling out a new one! Duplicate issues will be closed without comment.
Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Submit a solution on leetcode
Get a RED tick mark at the bottom
Check the Leethub extension. It shows 0 submissions.
No code is pushed to github.

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Tried this on Ubuntu and Windows but didn't work on any platforms( for existing repository). On selecting the option of "Creating a new repository" it does create the repo but fails to push code to github.

Update: For Questions already pushed by the extension(~ 5 months ago on different OS) It fails to do so again now. the Tick Mark also is Red instead of Green
Its working fine for new questions!


Improve back to top UI

I have:

Improved the "Back to Top" functionality in the README by replacing the simple hyperlink with a visually appealing clickable image button.
The button now provides a better user experience and aligns with the overall UI design.
Clicking the button smoothly scrolls back to the top for a seamless navigation experience.

@phyleria done with issue #445 that i was assigned to


Decide on the chromosome IDs to use in contig, GFF,  BAM and BigWig files

[x]  put real EMBL IDs into contig files (late at night)

| test| test|

[ ]  send a message to pombelist to use the  correct names for chromosomes and contigs in their browser files

[ ]  do a one off fix for all legacy files

I've added the data from the table above to the configuration file in SVN (pombe-embl/website/pombase_v2_config.json)
And I've changed the fasta and GFF writing code to use that configuration to put the right IDs in the right places.  So it's now "II", "mitochondrial" etc. instead of "chromosome_2", "MISPCG" etc.
Tomorrow morning, JBrowse should have the right IDs.

This is the mating type region  EMBL entry
https://www.ebi.ac.uk/ena/data/view/FP565355&display=text
will add to the table above and in the json if I can....

@kimrutherford  You will need to do the json.
I'm a bit confused by things like this:
},
"AB325691": {
"display_name": "AB325691"
}
why is  "AB325691" a display name?

why is "AB325691" a display name?

I didn't what to put as the display name when I first implemented the gene pages so I put that in as a place holder.
I've now updated the config file to match your table.

I've now updated the config file to match your table.

Ah, sorry.  I see the confusion now.  I had made the changes but hadn't checked them in.

do a one off fix for all legacy data files

Do we want to keep the original files (as submitted) or just the fixed up ones.  If we keep the originals, where should the go on the FTP site?
I've just renamed the "DATASET" directory to "external_datasets" on the FTP site.

I don't think there is any pressing need to keep the older files .......

definitely not on the ftp site anyway, it would be confusing...

I don't think there is any pressing need to keep the older files .......
definitely not on the ftp site anyway, it would be confusing...

OK, I'll move the originals to oliver0 and put the fixed ones in their place.

I'm trying to fix the chromosome names in the NRhind_Transcriptome BigWig files.  To do that I've converted the .bw files to .wig files because .wig is text not binary.  I've fixed the .wig files with Perl.
But when I try to convert the .wig back to .bw I get some warnings like this for chromosome 3:
line 3413451 of Spom_all_61G9EAAXX_and_61G9UAAXX.-.sorted.wig: chromosome III has 2452883 bases, but item ends at 2452936
There aren't many warnings but it's a worry that the coordinates of the files don't match the chromosome.  Any idea what could have gone wrong?  Is there a different chr 3 sequence that they could have used?

Asked Nick, not aware of another version. We should check incase the spurious bases are at beginning rather than the end.....

Sorry, false alarm!  The EMBL IDs in the table were in the wrong order so the converter was checking the wrong chromosome sizes.  I've fixed the table and change the Perl and now it's all fine.

Sorry, false alarm!

I let Nick know.

What need to happen in this ticket still, can you list.
I put an announce label but what do we need to tell people? Just that the file names have changed? anything else?

We still need to do this bit:

put real EMBL IDs into contig files (late at night)

The nightly load starts at 7:45pm at the moment so any time after that would be good.  We need to coordinate so I can make matching changes to the loader code.

OK can do anytime you think suitable...

OK can do anytime you think suitable...

I've checked in the code changes and the contig changes.  I'm running the load on my desktop to make sure it's all OK.

I've checked in the code changes and the contig changes. I'm running the load on my desktop
to make sure it's all OK.

Last night's load was fine.
This still needs doing:

send a message to pombelist to use the correct names for chromosomes and contigs in their browser files

Do we need to tell pombelist or just document it for labs who submit data in future?

do a one off fix for all legacy data files

The data files we use in JBrowse are fixed.  If there are other files we can fix them as we find them.

Shortly after we release a beta browser we should point everyone to the format we require the data to be submitted, and point out the chromosome labelling then....

can close this then....

Antonia you are composing a list of  specification/documentation for browser files. COuld  you roll in the chromsome number with that?


feat: add keepkey settings & initialize flow

Description
Adds KeepKey settings to the V2 app, as specified in:

https://shapeshift.notion.site/KeepKey-Settings-86cce69e059346c0b14aa0547703465a
https://shapeshift.notion.site/Initialize-Recover-KeepKey-on-V2-15129a0c73fc4ace995191552721db69

Out of scope for this PR, but will come as a fast-follow:

Ability to recover an existing KeepKey wallet
PIN modal included inside the settings menu

Notice

[x] Have you followed the guidelines in our Contributing guide?
[x] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Pull Request Type

[ ] :bug: Bug fix (Non-breaking Change: Fixes an issue)
[ ] :hammer_and_wrench: Chore (Non-breaking Change: Doc updates, pkg upgrades, typos, etc..)
[x] :nail_care: New Feature (Breaking/Non-breaking Change)

Issue (if applicable)

Closes https://github.com/shapeshift/web/issues/628
Closes https://github.com/shapeshift/web/issues/1300

Risk
Minimal - mostly self-contained, with some additions to walletProvider.tsx.
Testing
Note, the functionality is behind a feature flag: REACT_APP_KEEP_KEY_SETTINGS
Connect a KeepKey to the app and use the new KeepKey menu to change the device's settings, wipe the device, and re-initialize it.
Screenshots (if applicable)

@DiggyDiggy2 @SS-FOX-McCloud @reallybeard i reckon we should be significantly more aggressive with the copy here, if this does what i think it does (wipes mnemonic from device) users will definitely lose access to funds. if this doesn't wipe the mnemonic we should still change the copy.
if it wipes the mnemonic:
"I understand that by wiping my KeepKey I will lose access to my funds if I don't have my recovery phrase.
[ ] I have backed up my recovery phrase and understand what I am doing"
need to click the checkbox to enable the button
if it doesn't wipe the mnemonic:
change title to "Reset to default settings"
change copy to "Reset your KeepKey to factory default settings. Please note this will not wipe your seed phrase" etc
thoughts? 🤷♂

back button needs a teeny bit of padding

@0xdef1cafe I agree! It will wipe the mnemonic so the more aggressive copy would work great!
"I understand that by wiping my KeepKey I will lose access to my funds if I don't have my recovery phrase.

[ ] I have backed up my recovery phrase and understand what I am doing"
need to click the checkbox to enable the button

The timeout options would probably need to be full width of their container as opposed to currently, that padding + background-color combination looks off:

Expected:

To match exactly the mockups however, the selected option should be without a background-color, only the hovered option should have it (quick fiddle, that's not exactly the hover color but DevTools rugged me on hovered state):

Made the wipe modal more aggressively worded.

 Well done Ser 🍬

Another one for that great work 🥇

Aw shucks @gomesalexandre - I'll have to get trophy shelf.

Has this been tested when unplugging/plugging in the keepkey while the settings modals are open? We need to gracefully handle disconnections.

Has this been tested when unplugging/plugging in the keepkey while the settings modals are open? We need to gracefully handle disconnections.

Yep, it works smoothly - it reconnects and fetches the latest device data.


Help with NanoVault please!

Dear friend,
I bought some Nano tokens a year ago and stored them in a Ledger Nano S with the NanoVault web app via Brave Browser (it worked fine).
A couple days ago I was checking all my tokens in the Ledger device and it was all good except from Nano...
At first, I tried the same way (web app in Brave) and it could not even connect/read the ledger... same history with Chrome...
Then I downloaded the NanoVault app in a laptop and the first step worked fine, the ledger was connected but it appears that 'Can Not Connect to the Node'... my account have no tokens and keep getting the same message: "Unable to connect to the Nano node, your balances may be inaccurate".
I tried hiding and re creating an account but keep getting the same result.
I´m sure my tokens are somewhere and this has to do with some update or something.
I really hope you can help me with this one.
Big hug from Peru,

NanoVault is not working anymore. Use www.nault.cc instead. It can access your funds on the ledger.

Or download the desktop app: https://github.com/Nault/Nault/releases/latest

Thank you!


cljs branch

was wondering if there was any development on the cljs branch.
otherwise, I'm considering writing my own cljs react chessboard component, seems like a lot of fun.

was wondering if there was any development on the cljs branch.

Not at this time.
I wouldn't mind re-writing chessboard.js using ClojureScript, but I think it works better as a small library written in JavaScript.
I would like to remove the jQuery dependency, but piece animations would be tricky.


Port Math primitives to Java

https://github.com/CategoricalData/hydra/blob/main/hydra-haskell/src/main/haskell/Hydra/Dsl/Lib/Math.hs

This is done. I consider the expansion of the language-agnostic test suite to be a separate task.


june18: Update coding style guidelines

Re: use of WINDOWS_UWP vs UNITY_UWP, etc

Per https://github.com/Microsoft/MixedRealityToolkit-Unity/pull/2136#discussion_r189693473

This might be too big a change for master, when we can easily enforce it in vNEXT.

I'm just going to add the table to CodingGuidelines.md for now.


Update zio, zio-test, zio-test-sbt to 2.0.0-RC4

Updates

dev.zio:zio
dev.zio:zio-test
dev.zio:zio-test-sbt

from 2.0.0-RC3 to 2.0.0-RC4.
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!

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

labels: library-update, early-semver-pre-release, semver-spec-pre-release, commit-count:1

Superseded by #312.


Add from_json to TableDescriptors

What is the purpose of the change
This pull request adds from_json method to TableDescriptor and relevant classes.
Brief change log

Adds from_json method to TableDescriptor and relevant classes.

Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): no
The public API: yes

Documentation

Does this pull request introduce a new feature? yes
If yes, how is the feature documented? docs

Thanks for the update! LGTM.


deno.json import maps

re recent update, deno.json becomes an import map.  However, it seems like deno.json currently fails "Import map diagnostics" when being used as an importMapURL, since it has additional keys:
Import map diagnostics:
  - Invalid top-level key "compilerOptions". Only "imports" and "scopes" can be present.
  - Invalid top-level key "tasks". Only "imports" and "scopes" can be present.
  - Invalid top-level key "lint". Only "imports" and "scopes" can be present.
  - Invalid top-level key "fmt". Only "imports" and "scopes" can be present.

Should we maybe just validate import and scope keys?

owait maybe this is just a warning? let me double-check on this to make sure I didn't break anything else at the same time...

ooooops yaya false alarm!  Just a warning! Could be slightly annoying for those compiling multiple files for multiple targets, but I guess they should just use import_map.json if it bugs them that much haha
sorry for bother


ENH: Include QA/QC of unprocessed data

Specifies the QA/QC of unprocessed data with MRIQC.
Resolves: #347.
cc/ @celprov (to be developed by the two of us)

Let's merge this for now and continue working on its development in another PR.


An Empty docs.json file results in at 500

I hit a snag while creating a new project.
Particularly this step on the https://docs.page/ page:

To get started, create an empty docs directory and a config docs.json file, both at the root of your repository.

I did the following:
$ touch docs.json
$ mkdir docs
$ echo "# Hello, World" > docs/index.mdx
$ git commit -am "init commit" && git push origin main

But when visited the docs.page site for my repo, I got a 500.
This was because my docs.json file was empty.
Once I added an empty object to  docs.json, it worked as expected:
{}

Perhaps this is because an empty .json file is not valid JSON?

Thanks! will look into this

This is a bug, it should use a default config in this situation.

Yeah I can see why this is happening. I need to move the parsing and validating of configs to the api layer.


Add algorithms to EncryptionAlgorithm for Key Vault

For Key Vault 7.2-preview, to EncryptionAlgorithm (in swagger, the KeyOperationsParameters model), add the following algorithms:

A128GCM
A192GCM
A256GCM
A128KW
A192KW
A256KW
A128CBC
A192CBC
A256CBC
A128CBCPAD
A192CBCPAD
A256CBCPAD

As many of these as possible should be supported locally without writing our own cryptography.
Also add the following properties:

Iv (model property: iv)
AdditionalAuthenticatedData (model property: aad)
Tag (model property: tag)

These will likely need to be in a new options type/parameter passed to Encrypt and Decrypt.
See the service version 7.1 -> 7.2-preview diff 🔒 for details.

As in .NET and Java, Python should implement factories (or whatever is idiomatic) to lead customers into the golden path of providing the necessary parameters, like an IV for CBC and CBCPAD. Tt's optional in .NET only because we generate one like an nonce for local encryption support; Python, who passed it only to the service, might choose to send nothing instead since the service will, and in a more secure way than perhaps possible without platform support in Python.
See https://github.com/Azure/azure-sdk-for-net/commit/5e79cd7d70ddeaf42139d596d8ec077f5513d90e for how I did this in .NET. It's bigger than what you need since it also includes local crypto support, but focus instead on the *Options classes, like EncryptOptions for shape.

As in .NET and Java, Python should implement factories (or whatever is idiomatic) to lead customers into the golden path of providing the necessary parameters, like an IV for CBC and CBCPAD. Tt's optional in .NET only because we generate one like an nonce for local encryption support; Python, who passed it only to the service, might choose to send nothing instead since the service will, and in a more secure way than perhaps possible without platform support in Python.
See https://github.com/Azure/azure-sdk-for-net/commit/5e79cd7d70ddeaf42139d596d8ec077f5513d90e for how I did this in .NET. It's bigger than what you need since it also includes local crypto support, but focus instead on the *Options classes, like EncryptOptions for shape.


feat: debug page

add debug SSG page at _debug/[[...slug]]
add handling for undefined mdx components (that are neither exported nor our presets)
adapted existing layouts and context to be used in debug page, can navigate through different debug pages on left

TODO:

type the custom remark plugins properly
handle other errors ( unclosed tags for example don't seem to be thrown explicitly by mdx-bundler, may need custom handler)
layout and design of debug page

📝Changed routes:

…er]/[name]/[[...slug]]
/_debug/[[...slug]]

(view full size)
(view full size)

Commit 7666dc7e30cf6e34b0fa12f948992d1e29aaf54b (https://docs-page-kbs2t95v9-invertase.vercel.app).

What's your thoughts on the Found column being something like green/red check marks since (as far as I can tell) it'd always either be the same data or no data displayed there otherwise compared to Looked for

What's your thoughts on the Found column being something like green/red check marks since (as far as I can tell) it'd always either be the same data or no data displayed there otherwise compared to Looked for

Is it possible to also make the sub header look different to the other rows / look like headers here:

100% this makes sense

📝 Changed routes:

…er]/[name]/[[...slug]]
/_debug/[[...slug]]

(view full size)
(view full size)

Commit 09638cd4f06102b1f609fc430d3d0dae2bec446e (https://docs-page-qslnlzzl7-invertase.vercel.app).


added emits to polyline and polygon objects

this PR avoids warnings when using events on polygon or polyline

Hello, thanks for this PR! I'll accept it as soon as possible :)

I'll create a version and upload into npm later today :)

Its up! Sorry for the delay.


JavaScript Standard Style and React Native: Unexpected token =

I'm not sure if this should work with React Native, but at the moment there is a problem with React Native and React Navigation. A correct code is displayed as incorrect.
It is about this working example:
class ChatScreen extends React.Component {
  static navigationOptions = {
    title: 'Chat with Lucy',
  };
  render() {
    return (
      <View>
        <Text>Chat with Lucy</Text>
      </View>
    );
  }
}

Is the code incorrect? Or is it not compatible with React Native?

You need to customize standard parser to https://github.com/babel/babel-eslint:
"standard.options": {
		"enable": true,
		"parser": "babel-eslint"
	},

Remember to install babel-eslint either locally under you project or globally.

Perfect, thanks for the quick and good help!


Add a new frontend option for debugging called '-disable-incremental-…

What's in this pull request?

Currently IRGen stores hashes of the bitcode generated by swift in object files.
This is then used to reduce compile time by not re-codegening if a subsequent
compilation yields a bit code with the same hash.
This is good for users and general compilation, but can result in confusion when
attempting to measure the "real" compile time of the compiler.
By default it is off.

Before merging this pull request to apple/swift repository:

[ ] Test pull request on Swift continuous integration.

Triggering Swift CI
The swift-ci is triggered by writing a comment on this PR addressed to the GitHub user @swift-ci. Different tests will run depending on the specific comment that you use. The currently available comments are:
Smoke Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please smoke test
    OS X platform               | @swift-ci Please smoke test OS X platform
    Linux platform              | @swift-ci Please smoke test Linux platform

Validation Testing
    Platform     | Comment
    ------------ | -------------
    All supported platforms     | @swift-ci Please test
    OS X platform               | @swift-ci Please test OS X platform
    Linux platform              | @swift-ci Please test Linux platform

Note: Only members of the Apple organization can trigger swift-ci.

@swift-ci Please smoke test Linux Platform

@swift-ci Please smoke test

@swift-ci Please smoke test


fix(planning_launch): rename parameter for obstacle stop planner

Signed-off-by: satoshi-ota<EMAIL_ADDRESS>PR Type

Bug Fix

Related Links

https://github.com/autowarefoundation/autoware.universe/pull/2647

Description
rename hunting_* -> chattering_*

Review Procedure

Remarks

Pre-Review Checklist for the PR Author
PR Author should check the checkboxes below when creating the PR.

[x] Code follows coding guidelines
[x] Assign PR to reviewer

Checklist for the PR Reviewer
Reviewers should check the checkboxes below before approval.

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code follows coding guidelines
[ ] (Optional) Unit tests have been written for new behavior
[ ] PR title describes the changes

Post-Review Checklist for the PR Author
PR Author should check the checkboxes below before merging.

[ ] All open points are addressed and tracked via issues or tickets
[ ] Write release notes

CI Checks

Build and test for PR: Required to pass before the merge.
Check spelling: NOT required to pass before the merge. It is up to the reviewer(s). See here if you want to add some words to the spell check dictionary.

This PR is no longer needed.


RFM - freac.portable

Current Maintainer

[x] I am the maintainer of the package and wish to pass it to someone else;

I DON'T Want To Become The Maintainer

[x] I have followed the Package Triage Process and I do NOT want to become maintainer of the package;
[x] There is no existing open maintainer request for this package;

Checklist

[x] Issue title starts with 'RFM - '

Existing Package Details
Package URL: https://chocolatey.org/packages/freac.portable
Package source URL: https://github.com/abejenaru/chocolatey-packages/tree/master/automatic/freac.portable

I will keep it!
Please add me as a maintainer and I will maintain it with pleasure!
Thanks.
Seb

Wonderful, thank you!
On Sun, 22 Mar 2020 at 04:26, Sébastien Boulianne<EMAIL_ADDRESS>wrote:

I will keep it!
Please add me as a maintainer and I will maintain it with pleasure!
Thanks.
Seb
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/chocolatey-community/chocolatey-package-requests/issues/776#issuecomment-602142123,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAM73UMWJV7AJX6DKWEKOLTRIWAN3ANCNFSM4LEKQMAA
.

Did you add me as a maintainer on your package ?
Please add boubou as a maintainer.
Thanks Abejenaru!

Did you add me as a maintainer on your package ?
Please add boubou as a maintainer.
Thanks Abejenaru!

I did! Please check the package page, I think you must confirm the change.
Thanks!

@boubou can you check your email (spam folder as well) for emails from chocolatey.org.
You will need to click on the confirmation link in that email to accept being added as a maintainer.

Hi
I checked my spam folder and I didn't find anything.
Can you send me it please ?

Hi,
I message you and I asked you to add me as a maintainer then I will keep it up to date.
I checked my spam folder but I didn't find your email.
Please resend me the email!
Thanks in advance!

I have re-added you as a maintainer, hopefully that will trigger the confirmation email!
Thank you for your interest!

Any update on this?

Moving back to available for maintainers due to lack of response.

it's been pushed by boubou, so you can assign him, and maybe close

@tunisiano187 considering that he had not responded on any of the issues that he had been assigned to, I am inclined to consider him unresponsive, and personally would like someone else to pick up the package due to this.
@pauby what are your thoughts on this?

OK, depending on the decision, i can take it if you want.
Thank you

@AdmiringWorm As we both agree, I'll just assign this to @tunisiano187 just now.

OK, so i assume i'll need to ask via admins form too ?

@tunisiano187 yes that is correct.

A newer version (1.1.3) was successfully published 2 weeks later. freac was published a few days earlier than that.

A newer version (1.1.3) was successfully published 2 weeks later. freac was published a few days earlier than that.

I've pushed the update.
in case you have another request about this package, you can ask directly on my git at : https://github.com/tunisiano187/Chocolatey-packages/issues
@AdmiringWorm you can close this issue.
Thank you

I've pushed the update.
in case you have another request about this package, you can ask directly on my git at : https://github.com/tunisiano187/Chocolatey-packages/issues
@AdmiringWorm you can close this issue.
Thank you

@tunisiano187 That's what I meant. That this issue could be closed successfully.

@tunisiano187 That's what I meant. That this issue could be closed successfully.


Default usage with Win10 (non-domain) fails with xrdp_wm_log_msg: Section "XXX" not configured

I'm running Win10 Pro in a non-domain environment.  When sending a password the Win10 client seems to include a domain (generally the local machinename) as part of the username.  This causes xrdp to fail (unless the server has been carefully prepared ahead of time to avoid this).
The user experience is a black screen with no server generated dialog.
The server xrdp.log shows: [ERROR] xrdp_wm_log_msg: Section "HPC15" not configured in xrdp.ini
It seems that using the client provided domain as a section name fails, without falling back to something else (such as showing the server-side dialog, or picking the first session section, etc).
I have been unable to coax my Win10 Pro system to NOT send a domain/machine name as part of the username.  I did finally find a workaround, namely to use the "autorun" parameter in the server xrdp.ini file.  But it would be much more end-user friendly to do something other that just showing a black screen.
I initially filed this as https://bugzilla.redhat.com/show_bug.cgi?id=1450681 but the maintainer suggested I move the request upstream.
Versions

xrdp-0.9.2-10.fc25.x86_64
Fedora-25
Windows-10 Pro

Regards
P.S. I have found XRDP extremely useful on my projects.

Could you show me your xrdp.ini and sesman.ini?

OK, I understand your situation but what is the promlem? What do you want us to do?
Do you want a fallback if the domain name did not match any session type defined in xrdp.ini?

@cbutterfield I've tried to make a fix. Could you see #758? The first section will be used so far. Do you want to specify which section to use as fallback?
For example, add new config like this to xrdp.ini and it will be used as fallback if the given domain name doesn't match any configured sections.
some_new_good_name=Xvnc

@cbutterfield can you try?

Sorry not to have responded.  Somehow I failed to notice ALL of the notifications over the last 3 weeks.  I'll try to catch up over the next day or two, and get the repo pulled and built on my system (Fedora 25).
In the meantime, thank you very much for putting time and energy into this.
Regarding your proposed fix of "the first section", that seems fine to me since I it is also consistent with what happens if "autorun=" is empty and NOT domain name is sent.

OOPS - I have cloned https://github.com/neutrinolabs/xrdp.git but don't your changes there.
I looked at the web page you indicated https://github.com/neutrinolabs/xrdp/pull/758 but don't see how to clone whatever repo holds the info.  Basically, I don't know how to deal with "pull requests" on GitHub.

Never mind, after cloning the devel tree I just scanned the first few lines of "git log" and didn't see your commit, but it did show up AFTER some commits from March.  That behavior of "git log" often trips me up, you would think I would have learned by now :-(

@cbutterfield I've created RPM with current devel head 088bd2d for you. Try this. I recommend creating another new Fedora 25 environment and test it in there.
https://xrdp.vmeta.jp/pub/tmp/xrdp-0.9.2.git088bd2d+devel-1.fc25.x86_64.rpm
https://xrdp.vmeta.jp/pub/tmp/xrdp-debuginfo-0.9.2.git088bd2d+devel-1.fc25.x86_64.rpm

I sorry it is taking me so long to test this.  It seems that you have fixed the issue I reported, but I am having some (presumably unrelated) problem getting my configuration running, probably due to the difference between the Fedora-25 downstream RPM and the more generic one you created.  I’ll try again this weekend.

Charles Butterfield, Chief Engineer
Next Century Corporation
Phone:<PHONE_NUMBER>
Direct:<PHONE_NUMBER>
Email:  Charles.Butterfield<EMAIL_ADDRESS>Web:    www.nextcentury.comhttp://www.nextcentury.com/
From: metalefty<EMAIL_ADDRESS>Sent: Monday, June 12, 2017 8:51 PM
To: neutrinolabs/xrdp<EMAIL_ADDRESS>Cc: Charles Butterfield<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [neutrinolabs/xrdp] Default usage with Win10 (non-domain) fails with xrdp_wm_log_msg: Section "XXX" not configured (#750)
@cbutterfieldhttps://github.com/cbutterfield I've created RPM with current devel head 088bd2dhttps://github.com/neutrinolabs/xrdp/commit/088bd2d81162c7ca9300a95caccb7eb52cd4f6d7 for you. Try this. I recommend creating another new Fedora 25 environment and test it in there.
https://xrdp.vmeta.jp/pub/tmp/xrdp-0.9.2.git088bd2d+devel-1.fc25.x86_64.rpm
https://xrdp.vmeta.jp/pub/tmp/xrdp-debuginfo-0.9.2.git088bd2d+devel-1.fc25.x86_64.rpm
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/neutrinolabs/xrdp/issues/750#issuecomment-307975128, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AAWiG3Wv8Bixa60kQZErPZwlD35DHmWHks5sDdzjgaJpZM4Nb8tk.

Do you want new config to specify which section to use when given domain name didn't match any section? Currently, the first section will be used and it cannot be changed. I can add it if users want.

@cbutterfield If you don't need to specify fallback session type, please close.

I think what you have done is great.  I think using the first section as a fallback (just like an empty autorun= parameter).  I will close the ticket.  Thank you!


Redundant a

Redundant a here.

Hi @silenceshell. Thanks for your PR.
I'm waiting for a istio member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
Once the patch is verified, the new status will be reflected by the ok-to-test label.
I understand the commands that are listed here.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/ok-to-test

@silenceshell: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
3ffecc71695f8ae9681fee5a97275c47e746eb4e
link
/test lint_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@silenceshell: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

lint_istio
3ffecc71695f8ae9681fee5a97275c47e746eb4e
link
/test lint_istio

integ-multicluster-k8s-tests_istio
3ffecc71695f8ae9681fee5a97275c47e746eb4e
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@silenceshell: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
3ffecc71695f8ae9681fee5a97275c47e746eb4e
link
/test integ-multicluster-k8s-tests_istio

integ-telemetry-mc-k8s-tests_istio
3ffecc71695f8ae9681fee5a97275c47e746eb4e
link
/test integ-telemetry-mc-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


Unable to use blowfish v2.79.0 as a hugo module

Describe the bug
Attempted to upgrade blowfish to the latest version in my existing Hugo project, which is set up to use Hugo modules. However, trying to build my site complains about some invalid characters in paths found in the exampleSite directory of the blowfish repo:
$ hugo
go: no module dependencies to download
go: downloading github.com/nunocoracao/blowfish/v2 v2.79.0
hugo: downloading modules …
go: github.com/nunocoracao/blowfish/v2@upgrade: create zip: exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/feature.jpg: malformed file path "exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/feature.jpg": invalid char ':'
exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/index.it.md: malformed file path "exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/index.it.md": invalid char ':'
exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/index.ja.md: malformed file path "exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/index.ja.md": invalid char ':'
exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/index.md: malformed file path "exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/index.md": invalid char ':'
exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/index.zh-cn.md: malformed file path "exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/index.zh-cn.md": invalid char ':'
exampleSite/content/users/Xeonzilla’s Note/feature.jpg: malformed file path "exampleSite/content/users/Xeonzilla’s Note/feature.jpg": invalid char '’'
exampleSite/content/users/Xeonzilla’s Note/index.it.md: malformed file path "exampleSite/content/users/Xeonzilla’s Note/index.it.md": invalid char '’'
exampleSite/content/users/Xeonzilla’s Note/index.ja.md: malformed file path "exampleSite/content/users/Xeonzilla’s Note/index.ja.md": invalid char '’'
exampleSite/content/users/Xeonzilla’s Note/index.md: malformed file path "exampleSite/content/users/Xeonzilla’s Note/index.md": invalid char '’'
exampleSite/content/users/Xeonzilla’s Note/index.zh-cn.md: malformed file path "exampleSite/content/users/Xeonzilla’s Note/index.zh-cn.md": invalid char '’'
hugo: collected modules in 2666 ms
Total in 2670 ms
Error: failed to load modules: failed to get ["github.com/nunocoracao/blowfish/v2@upgrade"]: failed to execute 'go [get github.com/nunocoracao/blowfish/v2@upgrade]': failed to execute binary "go" with args [get github.com/nunocoracao/blowfish/v2@upgrade]: go: downloading github.com/nunocoracao/blowfish/v2 v2.79.0
go: github.com/nunocoracao/blowfish/v2@upgrade: create zip: exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/feature.jpg: malformed file path "exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/feature.jpg": invalid char ':'
exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/index.it.md: malformed file path "exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/index.it.md": invalid char ':'
exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/index.ja.md: malformed file path "exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/index.ja.md": invalid char ':'
exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/index.md: malformed file path "exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/index.md": invalid char ':'
exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/index.zh-cn.md: malformed file path "exampleSite/content/users/Handbook on Teaching Empirical Software Engineering: Online Materials/index.zh-cn.md": invalid char ':'
exampleSite/content/users/Xeonzilla’s Note/feature.jpg: malformed file path "exampleSite/content/users/Xeonzilla’s Note/feature.jpg": invalid char '’'
exampleSite/content/users/Xeonzilla’s Note/index.it.md: malformed file path "exampleSite/content/users/Xeonzilla’s Note/index.it.md": invalid char '’'
exampleSite/content/users/Xeonzilla’s Note/index.ja.md: malformed file path "exampleSite/content/users/Xeonzilla’s Note/index.ja.md": invalid char '’'
exampleSite/content/users/Xeonzilla’s Note/index.md: malformed file path "exampleSite/content/users/Xeonzilla’s Note/index.md": invalid char '’'
exampleSite/content/users/Xeonzilla’s Note/index.zh-cn.md: malformed file path "exampleSite/content/users/Xeonzilla’s Note/index.zh-cn.md": invalid char '’'
 *errors.errorString

To Reproduce
Steps to reproduce the behavior:
Create a new Hugo site from scratch:
hugo new site test && cd test && hugo mod init test && hugo mod tidy

Add the Blowfish module to the Hugo configuration:
cat <<EOF >> hugo.toml
[module]
[[module.imports]]
disable = false
path = "github.com/nunocoracao/blowfish/v2"
EOF

Try to build the site:
hugo

Expected behavior
The Blowfish module downloads normally and my site builds fine.
Desktop (please complete the following information):

OS: macOS
Version: 15.1

Hugo & Blowfish versions
$ go version
go version go1.23.3 darwin/arm64

$ hugo version
hugo v0.138.0+extended+withdeploy darwin/arm64 BuildDate=2024-11-06T11:22:34Z VendorInfo=brew

Blowfish version: 2.79.0

just noticed this as well, I think we either:

find a way to exclude the exampleSite in hugo module, or
have to replace characters like : and ’ with something else
or ....just use the domain for the sites  (seems easiest)

What is the benefit of including the example site in the module at all?
Excluding it seems like a fine option for me, and would also greatly reduce the module size.

I believe there was an issue discussing about reducing repo/module size but
not sure where it is now.
I tried to fork and added dot/underscore to the example site folder see if
it will be excluded by go toolchain (read that somewhere) but none of them
works. Not really a go expert so I went to fix the issue first, but I am
all for excluding the folder.
Let see what does @nunocoracao think?
On Sat, Nov 9, 2024 at 14:25 Sander Ploegsma @.***>
wrote:

What is the benefit of including the example site in the module at all?
Excluding it seems like a fine option for me, and would also greatly
reduce the module size.
—
Reply to this email directly, view it on GitHub
https://github.com/nunocoracao/blowfish/issues/1835#issuecomment-2466075728,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AALZQXMQBNXLUL2N2FFZ5GTZ7WTGXAVCNFSM6AAAAABROFXXOCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDINRWGA3TKNZSHA
.

Just invalid both in Git & Linux system.
I think simply rename the directory name would be better:
Handbook on Teaching Empirical Software Engineering: Online Materials -> Handbook on Teaching Empirical Software Engineering - Online Materials etc.

I'm also experiencing this issue. I'm sure I can speak for us all when I say I'd greatly appreciate @nunocoracao's attention on this as it's completely blocking an upgrade to the latest version.

I have the same issue. I have a couple of websites that run on this theme, and I’ve been tidying up a little bit, by using just one module instead. Turned out, it’s not possible.
ChatGPT has returned me one more solution, to replace the module with the local version (where the exampleSite can be removed):
hugo mod edit -replace github.com/nunocoracao/blowfish/v2=../local-blowfish

but I’m not sure whether it’ll play nicely with the CI/CD pipelines that I have. I guess, the better approach would be to fork the project, and just remove all the unnecessary elements. Is there anyone who did that already?

I believe there was an issue discussing about reducing repo/module size but not sure where it is now.

I guess, you’re referring to this issue: #980

But my vote is for either forking the theme into the lean version or just removing exampleSite altogether into a separate project.

Fixing this in v2.80.0


any plans for schema-registry ?

Hi,
usually kafka setup includes schema-registry. Do you have any plans to try to develop schema-registry-native?
refs:

https://hub.docker.com/r/confluentinc/cp-schema-registry
https://docs.confluent.io/platform/current/schema-registry/index.html

Hi,
Confluent products are out of scope for this project.
You can check out the Apicurio Registry project which is an open-source schema registry implementation.
They are able to produce a native version apicurio/apicurio-registry-native:latest-snapshot but it is not released as part of the project.
I advise you to show your interest there.

I also notice that docker image does not contain default kafka scripts for creating topics and etc


remove deadcode

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.08%) to 8.573%

Totals

Change from base Build<PHONE_NUMBER>5:
0.08%

Covered Lines:
794

Relevant Lines:
9262

💛  - Coveralls


chore: Use jdk 17 for artifact publish

Welp https://github.com/Eppo-exp/android-sdk/actions/runs/9571411350/job/26388786755

@aarsilv actually this version of the android gradle plugin requires java 17
* What went wrong:
An exception occurred applying plugin request [id: 'com.android.library']
> Failed to apply plugin 'com.android.internal.library'.
   > Android Gradle plugin requires Java 17 to run. You are currently using Java 11.
      Your current JDK is located in /opt/hostedtoolcache/Java_Adopt_jdk/11.0.23-9/x64
      You can try some of the following options:
       - changing the IDE settings.
       - changing the JAVA_HOME environment variable.
       - changing `org.gradle.java.home` in `gradle.properties`.


Error executing API call (422: Unprocessable Entity):

Expected behavior
When you select Pay later as payment method it should bring you to the Pay later (klarna) payment screen.
Actual behavior
In version 4.09 and 4.08 it crashes with the follow message: Error executing API call (422: Unprocessable Entity): The 'phone' field should contain a valid phone number, in E.164 format.
Steps to reproduce the behavior
Select Pay later (Klarna) as payment method and fill in the information for it and start the payment.
Environment

PrestaShop version: <IP_ADDRESS>
Mollie module version: 4.0.9
PHP version: 7.0.33

Duplicate of 234#
Hey @Maikel726  thanks, this issue is known and on our way on fixing.
We have backloged your request about this, we will update you about the fix.
Best Regards.

Apologies for the delay. It took some time to gather all the information from the team. Please find an explanation below. When user checkouts on the webshop, the phone number by default, should be inputed with + symbol. For example, if a user enters a number with the beginning +44 (valid UK) it is transferred the same to Mollie and Prestashop all OK.
When a user enters invalid code for example +000, and finishes checkouting, the Mollie API makes a validation of the phone number and doesn’t let it proceed with a payment.  So there is no problem at all from the technical side. Problem was only the error showing itself.
We made an update, that there would not be any useless error.
Hope explanation is clear for you.
Please let me know if you have any questions.
We will release the fix this week in the 4.1.0 release.
Best Regards.

Hey @Maikel726
We have included this fix in 4.1.0 release, maybe we can close this issue and in case you will raise the new issue regarding 4.1.0?
Best Regards.

Hey @SimonasB88
When version 4.1.0 is released I will see if it is resolved, if not I will let you know and create a new issue.

Hey,
Still getting a error when trying to use Pay later,
Cart Dump: { "amount": { "currency": "EUR", "value": "126.90" }, "billingAddress": { "organizationName": " ", "streetAndNumber": "De Gruisdonk 5", "city": "Venlo", "postalCode": "5928 RT", "country": "NL",

@Maikel726 can you copy the whole error here?

@Maikel726 please reset the module after upgrade and try once again.

Hey @SimonasB88
I tried to reinstall the Mollie module ( included deleting the map ) still getting this error:
Cart Dump: { "amount": { "currency": "EUR", "value": "126.90" }, "billingAddress": { "organizationName": " ", "streetAndNumber": "De Gruisdonk 5A", "city": "Venlo", "postalCode": "5928 RT", "country": "NL", "givenName": "Maikel", "familyName": "Mollie", "email"<EMAIL_ADDRESS>"phone": "+773968147" }, "shippingAddress": { "organizationName": " ", "streetAndNumber": "De Gruisdonk 5A", "city": "Venlo", "postalCode": "5928 RT", "country": "NL", "givenName": "Maikel", "familyName": "Mollie", "email"<EMAIL_ADDRESS>"phone": "+773968147" }, "redirectUrl": "http://autoonderdelenexpres.nl/module/mollie/return?cart_id=67&utm_nooverride=1&rand=1605608696&key=52b8f9970d31b68251e928f5523e8a00&customerId=25", "webhookUrl": "http://autoonderdelenexpres.nl/module/mollie/webhook", "method": "klarnapaylater", "metadata": { "cart_id": 67, "order_reference": "1420", "secure_key": "7f957cc9d3c77d3712a959370fe2fb2a" }, "locale": "nl_NL", "orderNumber": "1420", "lines": [ { "sku": "4861\u00a40\u00a40", "name": "CAM (MAC) dakdragers aluminium Volkswagen Passat VII Alltrack 5-dr Estate 2012-2015 met open dakrailing", "productUrl": null, "imageUrl": null, "metadata": null, "quantity": 1, "vatRate": "21.00", "category": "", "unitPrice": { "currency": "EUR", "value": "119.95" }, "totalAmount": { "currency": "EUR", "value": "119.95" }, "discountAmount": [], "vatAmount": { "currency": "EUR", "value": "20.82" } }, { "sku": "", "name": "Shipping", "productUrl": null, "imageUrl": null, "metadata": null, "quantity": 1, "vatRate": "0.00", "category": null, "unitPrice": { "currency": "EUR", "value": "6.95" }, "totalAmount": { "currency": "EUR", "value": "6.95" }, "discountAmount": [], "vatAmount": { "currency": "EUR", "value": "0.00" } } ], "payment": { "webhookUrl": "http://autoonderdelenexpres.nl/module/mollie/webhook" } }

Hey @Maikel726 this is really very strange... Phone number now it is not in the spotlight, regarding this error.
Are you using One page checkout? Or have you customized your order functionalities in your Prestashop somehow?
Best Regards.

Hey @SimonasB88
I am using a 3th party one page checkout, but before this issue the Mollie module worked fine.

@Maikel726 that could make sense about the error that you are receiving. I have to check this with the developers.
I guess we could try to investigate the issue internally.
In order to investigate your issue and help you efficiently, can you send us an access to your backoffice with its url and a FTP access to your store:
URL back office:
login back office:
password back office:
FTP host:
FTP login:
FTP password:
FTP port:
Send everything to<EMAIL_ADDRESS>Thanks.
Best Regards.

Ticket closed, successfully fixed for the client, the fix will be included in the next Mollie Github release.
Best Regards.


Ability to reliably process messages?

First of all, thanks for writing and sharing this library! It looks like a promising
option for writing multi-node Kafka consumers in Go.
One possible issue I have is the ability to reliably process messages. I'd like
to be able guarantee that I don't commit an offset before I've positively
confirmed that all of the corresponded messages have been processed
successfully (i.e. not just passed into the processing code, but successfully
completed the associated application-level processing). IIUC the current
API doesn't support this. After you receive an event from the cg.Stream(),
the corresponding offset could be committed by the client at any time in the
background.
Do I understand this correctly? If so, is reliable processing something that
you'd be interested in supporting in the high-level API?

I have just arrived at this same point - realizing that the current consumergroup implementation commits the offsets independently of processing success.
As far API preferences, I was looking at #20 and it looks like you're defining a map of topics to processing functions - this would work great for us. In fact, we've implemented some code on top of consumergroup which works that way, but obviously doesn't have access to the offsets. It's even crossed my mind to use a short-term hack and set the CommitInterval to something impossibly long (basically so that consumergroup itself never commits) and move the commit logic into the processor function itself.
Are you actively working on #20 still? Do you have any thoughts on what's left before making that branch a reality? Would love to help make this happen.

Not working on it due to shifted priorities. Feel free to take it over if you're interested in this; I am available to help.
I primarily was unsure about what API would be most appropriate: using a callback function to process messages, or do something with channels.The implementation shouldn't bee too hard I think once that is figured out. I would say: pick the one you like most and see how far you get.

We built an "at least once processing" guarantee consumer over Sarama (along with other real world high level consumer features) https://github.com/stealthly/go_kafka_client if you don't need the other features maybe you could work our implementation for your needs into here or use as is.

I implemented this in #30. Any feedback would be appreciated!

This is now on master; I'd appreciate any feedback.


Resetting the form clears the list of choices

I have a form with some filters that make an ajax request to display a table. When I want to reset the filters ( $("#filter-form").trigger("reset");) all the select elements created with choices.js are empty and I have nothing left to select. I have to refresh the page to repopulate the select forms.
any ideea ?

I'm experiencing the same issue... how can we handle this scenario?

Having the same issue. Is there a fix for this?

The same issue when I clear form inputs. Still no fix?

Ran into the same problem.
My band-aid is to add a click addEventListener() for the reset button.
Then, use Choices method removeActiveItems() to clear the select elements.
This also means you'll need to add code to reset all the other form fields.
document.addEventListener('DOMContentLoaded', function () {
	const element = document.querySelector('.js-choice');

	const choicejsOptions = {
		allowHTML: false,
		removeItemButton: true,
		searchEnabled: true,
		searchChoices: true,
		searchFields: ['label', 'value'],
		placeholder: true,
	};

	const services = new Choices('.js-choice__Services', choicejsOptions);

	document.querySelector('input[type="reset"]').addEventListener('click', function (evt) {
		evt.preventDefault;

		services.removeActiveItems();
	});
});

Even the documentation shows the bug. It removes all of the options when I click the reset button

No news? I can't clean my selections... Thanks

A work-around (snippet from a Stimulus controller):
const choices = new Choices(this.element, {
  allowHTML: false,
  removeItemButton: this.element.multiple,
});

const form = this.element.closest("form");
if (!form) return;

form.addEventListener("reset", () => {
  choices.destroy();
  choices.init();
});```

Implemented as part of #1166


tolerations, nodeSelectors, etc should be set per workload

If I set a toleration to allow the node agent to work on all the nodes, the scanner process also gets that toleration which is very undesirable

Hi @treyhyde, thanks for bringing this issue to our attention! Two clarification questions for you:

Is this for the agent Helm Charts?
When you say scanner process, are you referring to the cluster mode agent?

yes and yes, sorry for the confusion

@treyhyde, awesome! We can make those changes. Thank you for raising the issue!

great thanks, I might have to disable the scanner until they are. Thanks for the followup.

This is really preventing adoption of newer lacework features (things that require the clusteragent)

This is really preventing adoption of newer lacework features (things that require the clusteragent)

Thanks for the follow up. We are working on getting this in an upcoming release.

This is really preventing adoption of newer lacework features (things that require the clusteragent)

Thanks for the follow up. We are working on getting this in an upcoming release.

@treyhyde per workload tolerations and node selectors are released as part of 6.7 version release. Please let us know if this resolves the installation issues you faced.

@rnalexlacework The syntax is a little awkward but seems to be ok to separate the tolerations of the cluster vs node agents.   I don't see a "nodeSelector" for the "deployment" available.
Just looked to fix our "proxy-scanner" config as well and there are even fewer options.

Unfortunately, it is still not possible to set tolerations for deployments of proxy-scanner and admission-controller. I can submit PR with the fix, if you don't mind


Karabiner-Elements blocking certain inputs in Rocksmith

This seems to be the only application that I run into this with. So basically, the game Rocksmith 2014 seems to have issues with taking keyboard inputs when karabiner-elements is on. I can do most keyboard inputs, but they have an in-game search field that ignores all my keyboard inputs when I try to type into it.
Before, I used to be able to just quit KE before I launched the game. But that no longer seems to work, and I basically have to remove the application from being launched on boot first.
I reazlize that the an input field within a game isn't really directly something you that would be of concern, but now that I can't just simply quit the app to have it work is kind of troublesome.

I face same issue too.
Rocksmith 2014 Guitarcade does not recognize keyboard inputs when Karabiner-Elements is running only.

Same here.
In older versions quitting Karabiner-Elements would be a temporary fix, but right now that is not enough, since the karabiner_grabber is respawned by launchctl.
Stopping the daemon and quitting Karabiner-Elements is working at the moment. Don't forget to load the daemon again once you're done so Karabiner works again!
sudo launchctl unload /Library/LaunchDaemons/org.pqrs.karabiner.karabiner_grabber.plist


feat: Python support for functions

Updating my previous PR#1063, removed the warehouse attribute from the function schema and added to the provider configuration.
Also added documentation and examples for snowflake_function

/ok-to-test sha=86a4f23


I miss FXAA :(

I used the FXAA included in this shader for H3VR because it ran much faster than MSAA and did not look much worse.
 H3VR's environments are mostly made up of very simple geometry that FXAA can anti-alias just fine. Especially if you max out the quality at 39 and leave the default settings as-is (I noticed absolutely zero difference in performance between the quality at 10 vs. the quality at 39, stayed at 2.9ms in the same spot).
I really freed up lots of performance headroom with FXAA. It still ran faster in this shader than it does when I add it afterwards as a separate shader (3.6ms compared to 2.9ms).
Can you add it back so that I can update to the latest version? If not, I would really love a more specific explanation as to why it's gone.

It was removed as it did not had the bug improvement I hoped it to have. (Simracing) The FXAA I added to the toolkit was a lower quality variant to able to work in a single pass to save peformance. I honestly did not thought that anyone would have a usecase for it. You can always add the FXAA, SMAA or DLAA as a extra shader before applying the vrtoolkit. I found DLAA was acually not bad in VR but a little more expensive. Still not as much as MSAA for sure.
I can re-add FXAA to the VRtoolkit though if you want it. (Was removed to just clean up the shader code)
But first try to use the FXAA etc. seperatly if it works for you?

I was trying other solutions, including DLAA, but they were all less efficient than the game’s MSAA and looked about the same as 2X MSAA.

I will re-add it later for you again if it has a good use. Right now you can download the old version here that stil has it:
https://github.com/retroluxfilm/reshade-vrtoolkit/releases/tag/v1.0.0
Just extraxt the shader folder and overwrite the new version.

hell yeah thanks a lot! Got version 1.0 to work

FXAA is re-added now in the repo


Error in json decoding

Hi!
I have a problem with my web-app. I can get the token when i log in but when I use the token to authenticate me it says: invalid JWT Token and shows this error:
[2018-08-31 05:40:32] security.INFO: Guard authentication failed. {"exception":"[object] (Lexik\\Bundle\\JWTAuthenticationBundle\\Exception\\InvalidTokenException(code: 0): Invalid JWT Token at /var/cryptoforecast/goemonV2/vendor/lexik/jwt-authentication-bundle/Security/Guard/JWTTokenAuthenticator.php:119, Lexik\\Bundle\\JWTAuthenticationBundle\\Exception\\JWTDecodeFailureException(code: 0): Invalid JWT Token at /var/cryptoforecast/goemonV2/vendor/lexik/jwt-authentication-bundle/Encoder/LcobucciJWTEncoder.php:55, RuntimeException(code: 0): Error while decoding to JSON: Syntax error at /var/cryptoforecast/goemonV2/vendor/lcobucci/jwt/src/Parsing/Decoder.php:36)","authenticator":"Lexik\\Bundle\\JWTAuthenticationBundle\\Security\\Guard\\JWTTokenAuthenticator"} []

On my local machine i cannot see this problem, it shows on the production machine
My configuration (DEV)

Windows 10
PHP 7.2.8 (cli) (built: Jul 18 2018 10:03:02) ( NTS MSVC15 (Visual C++ 2017) x64 )
OpenSSL 1.0.2j  26 Sep 2016

My conf (PROD)

PHP 7.2.8-1+0<PHONE_NUMBER>4257.2+stretch~1.gbp571e56 (cli) (built: Jul 25 2018 12:43:00) ( NTS )
OpenSSL 1.1.0f  25 May 2017

My Composer file:
"lexik/jwt-authentication-bundle": "^2.5.4",
"lcobucci/jwt": "3.2.4",

Could you help me?

Hey, are you using apache? Did you see https://github.com/lexik/LexikJWTAuthenticationBundle/blob/master/Resources/doc/index.md#important-note-for-apache-users?

Thank, now it works ;)

Closing as resolved


any way of getting maven artifact?

I would like to use your server for testing purposes.
I know that I can setup a local artifact, but what about artifact which can be downloaded from maven central for instance?

It's not in our plans for the near future, sorry.


Slides/2 problems found from pat

Minor fixes/changes as indicated in commit comments

 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.You have signed the CLA already but the status is still pending? Let us recheck it.


chore(paeckchen): publish one package at once

Coverage remained the same at 100.0% when pulling 616a59859db8d24663373d3ff6aa4ec483f743b9 on update-publish-script into 57d7931b6955b098a4a8db6112bc18789c0670b3 on master.

Coverage remained the same at 100.0% when pulling 83937bf7187371153f44c36a852a3c068502b564 on update-publish-script into ceb36f13bec3810704d87e2bec4b06ef506b8ea6 on master.


Reports false "plugged in"

Since the latest update, onstar2mqtt has been incorrectly reporting that the car is plugged in when it isn't.

The second time I restarted it, it correctly reported "not plugged in" so this may have been a fluke.


Missing i18n when using Fecha library to format dates (checkin and checkout inputs)

Description
Missing i18n when using Fecha library to format dates (see image attached).
Could

I please submit a PR? I have a fix for this.
[Description of the bug or feature]
Date formats within the inputs (checkin and checkout) are not using i18n.
Code sample
[ If you are submitting a bug issue, paste your code so I can reproduce it, or a link to a Codepen with it ]
Steps to Reproduce

Set the date format to MMMM D
The month is not translated.

Expected behavior: [What you expected to happen]
Date formats within the inputs (checkin and checkout) should use i18n.
Actual behavior: [What actually happened]
Date formats within the inputs (checkin and checkout) aren't using i18n, and therefore, are not being translated.
Datepicker Version
[ The version of the datepicker you have installed in your project ]

Will be released with the next version :)


Encode data

Closes #134

Codecov Report

Merging #143 (5f8a480) into develop (103d485) will increase coverage by 10.22%.
The diff coverage is 86.31%.

@@             Coverage Diff              @@
##           develop     #143       +/-   ##
============================================
+ Coverage    76.07%   86.29%   +10.22%
============================================
  Files            9       11        +2
  Lines          443      635      +192
  Branches       101      127       +26
============================================
+ Hits           337      548      +211
+ Misses          54       43       -11
+ Partials        52       44        -8

Impacted Files
Coverage Δ

src/lib/isValidSignature.ts
66.66% <66.66%> (ø)

src/lib/utils.ts
80.26% <78.99%> (+0.10%)
:arrow_up:

src/providers/ethereumProviderWrapper.ts
79.59% <80.48%> (+20.97%)
:arrow_up:

src/providers/web3ProviderWrapper.ts
84.31% <80.95%> (+19.44%)
:arrow_up:

src/index.ts
83.22% <84.32%> (+4.77%)
:arrow_up:

src/lib/provider-wrapper-utils.ts
88.88% <88.88%> (ø)

src/lib/constants.ts
100.00% <100.00%> (ø)

src/lib/encoder.ts
91.48% <100.00%> (+20.52%)
:arrow_up:

src/lib/schemaParser.ts
100.00% <100.00%> (ø)

src/schemas/index.ts
100.00% <100.00%> (ø)

... and 5 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1d63166...5f8a480. Read the comment docs.


No usage tracking

"No usage tracking"
But there are 4 cookies stored without user consent.
Is this illegal according to european GDPR law?

Ah I see, no this relates to the API. Meaning the data being sent to the API (names, emails, etc.) since the website isn't the 'product'.
In relation to GDPR, no. That's a personal data law, mostly irrelevant to cookies. It's the cookie law that requires banners.
I do see analytics on the site which I haven't checked since launch, so I'll add it to my todo to have it removed 😄 thanks for notifying

Some of the cookies are Adsense related, so instead I installed a cookie banner with option to reject cookies 👍 Still testing to see how that actually works but presumably it works as the service describes


Fixed some games. Replaced font.size with font.baseSize. Uncommented linux libs.

Maybe these are exercises for your students?

Wow! Amazing work @RDR8! It was on my TODO list reviewing all those games to latest raylib version!
Thank you very much for the effort!  The monsters in Skully are frighteningly fun. Great sound effects. Wave Collector crashes after pressing enter at the loading screen.  Going through /examples next. 😀 After I get a handle on make, will learn gdb.

Thanks again @RDR8! About Wave Collector, I'm aware of the crash, it happens randomly, I haven't touched raylib_demo since version 1.0 or so... yeah, templates folder also requires some review...

Nothing to report in /templates. (^:


baseOptions should be optional on mutation hooks

Describe the bug
The baseOptions argument on generated mutation hooks is required, when it should be optional.
To Reproduce
Generate any mutation hook.
Expected behavior
baseOptions will be optional when calling a mutation hook.
Environment:

@graphql-codegen/cli: ^1.19.1
@graphql-codegen/typescript-react-apollo: ^2.1.0

Additional context

I'll create a PR for skipping this feature for mutations.
Thank you for reporting this @daganomri

Fixed in: https://github.com/dotansimha/graphql-code-generator/pull/5046

Fix available in<EMAIL_ADDRESS>@daganomri @gynekolog

Should the baseOptions hook only be optional if there are no required variables needed for the mutation? Otherwise this undos #4964 for the mutations case

@bubba
Mutation - always optional, since you can specify it later (while executing), and I think we don't have a way to check that.
Query/Subscription - options are specified on the declaration level ,and now it will depend on the variables.

@dotansimha I didn't realise you could specify variables later. Thanks for clarifying!


opensky sensor

Description:
Report the number of flights in a given area via the OpenSky Network, and raise events for flights crossing the area boundary.
Pull request in home-assistant.github.io with documentation (if applicable): https://github.com/home-assistant/home-assistant.github.io/pull/2431
Example entry for configuration.yaml (if applicable):
sensor:
  - platform: opensky
    radius: 10   # kilometers
    name: my area  # optional
    latitude: <latitude>  # optional, defaults to home zone latitude
    longitude: <longitude>  # optional, defaults to home zone longitude

Checklist:
If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.github.io

If the code communicates with devices, web services, or third-party tools:

[x] Local tests with tox run successfully. Your PR cannot be merged unless tests pass
[x] New files were added to .coveragerc.

Addressed comments.

This is awesome. I live next door to the airport runway. I can turn up the  music/tv volume when Singapore air land. Many thanks!

i gave this a test, with this config
sensor:
  - platform: opensky
    radius: 10   # kilometers

and got this error
Traceback (most recent call last):
  File "/usr/lib/python3.5/asyncio/tasks.py", line 241, in _step
    result = coro.throw(exc)
  File "/usr/local/lib/python3.5/dist-packages/homeassistant/helpers/entity_component.py", line 359, in async_process_entity
    new_entity, self, update_before_add=update_before_add
  File "/usr/local/lib/python3.5/dist-packages/homeassistant/helpers/entity_component.py", line 189, in async_add_entity
    yield from self.hass.loop.run_in_executor(None, entity.update)
  File "/usr/lib/python3.5/asyncio/futures.py", line 361, in __iter__
    yield self  # This tells Task to wait for completion.
  File "/usr/lib/python3.5/asyncio/tasks.py", line 296, in _wakeup
    future.result()
  File "/usr/lib/python3.5/asyncio/futures.py", line 274, in result
    raise self._exception
  File "/usr/lib/python3.5/concurrent/futures/thread.py", line 55, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/home/brenda/.homeassistant/custom_components/sensor/opensky.py", line 109, in update
    if distance > self._radius:
TypeError: unorderable types: NoneType() > float()

@Br3nda do you have a lat/long set in your homeassistant config?

@happyleavesaoc yes - it is set.
It did this error first time -- and again an hour later.
Maybe you can log what the api returned when it throws an exception like this?

@Br3nda Ah ok, so it was an intermittent error? There's probably occasionally a flight with bad lat/long data. I added an extra check to prevent the error on the latest commit. Thanks for the feedback!


apis: Add condition status to SharedBackupConfig and ExternalSecretStores

Description of your changes
This adds Crossplane Status fields to two shared API's. In Spaces, we intend on exposing the status of the referenced secrets, if any, as statuses.
I have:

[x] Read and followed Upbound's contribution process.
[x] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR, as appropriate.

/assign @phisco


alertwords: Fix css in static/styles/settings.css.

https://github.com/zulip/zulip/issues/17172
Testing plan:
Consistent behavior between mobile and desktop, tested small and largest words, tested deleting functionality with no issues
GIFs or screenshots:

The boxes around each word makes them look like editable text input fields, which they are not. If we’re making this a table, we should style it identically to our other tables.

@andersk Okay I will start to put this together today. I've got a few questions here:
1.) I know there are other examples of tables such as (users) under "organization". Is there a specific component you'd like me to use so that I'm matching the styling exactly or is there a handlebars snippet of "users" I can use as a reference for styling?
2.) When I've made this update should I start a new pull request or make changes and adjust this pull request?
Thanks!

They’re all essentially the same: git grep '<table' static/templates/settings
Update the existing PR.


Handle ports already in use

if, ex, port 8080 is already in use nhost dev will crash:
╭─elimbp ~/tmp/nhost/facebook2
╰─➤  nhost dev
Error: Command failed: docker-compose up -d > /dev/null 2>&1
    at DevCommand.run (~/projects/nhost/cli/src/commands/dev.js:42:5)
    at DevCommand._run (~/projects/nhost/cli/node_modules/@oclif/command/lib/command.js:44:31)

We should handle this by either, generate a new port to use or exit and prompt with a message saying why nhost dev won't work.
I think V1 could just exit and say the ports are already in use.

Current behaviour: the CLI outputs a warning message asking the user to double check if all ports configured in config.yaml are available.

We can use detect-port to check if all ports ara available before starting the services.

We now check for ports on nhost dev.


Parse nyc output

Istanbul is being re-written as nyc, whose output is currently not being parsed by node-tdd.

@prashaantt  It works for me, are you using the text-summary reporter?
(In the Details for the package it says that is a known issue/limitation)

Hello Prashaant, do you have plan to support new nyc output in the future?

Yes, I hope to get to this someday, although the text-summary option works pretty well already.

Would you accept a PR that includes:

Setting the path to the nyc report
Setting nyc report format (support all nyc formats)
Watch report file to update coverage details


activity跳转异常

使用方式：
int flag = Intent.FLAG_ACTIVITY_CLEAR_TASK | Intent.FLAG_ACTIVITY_NEW_TASK;
ARouter.getInstance().build(UserPageConstants.IDLoginActivity).greenChannel()
.withFlags(flag)
.navigation();
部分机型跳转异常
android.util.AndroidRuntimeException
Activity could not be started for Intent { flg=0x10008000 cmp=xxx/xxx.IDLoginActivity (has extras) }
查看源码发现在Instrumentation会抛出此异常

Excuse me, has this problem been solved?

Excuse me, has this problem been solved?

Excuse me, has this problem been solved?


Label to Cosmos

Let's hold this one for next release , need to coordinate with android side , otherwise cosmos won't be able signed with Ios and Android


Short solution needed: "How to change font size" (php-gd)

Please help us write most modern and shortest code solution for this issue:
How to change font size (technology: php-gd)
Fast way
Just write the code solution in the comments.
Prefered way

Create pull request with a new code file inside inbox folder.
Don't forget to use comments explain solution.
Link to this issue in comments of pull request.

How to change font size


zephyr: Fix bogus uses of force_bytes that break on Python 3.

An expression like force_bytes(chr(...)), on Python 3 where the
force_bytes finds itself with something to do because chr returns
a text string, gives the UTF-8 encoding of the given value as a
Unicode codepoint.
Here, we don't want that -- rather we want the given value as a
single byte.  We can do that with struct.pack.
This fixes an issue where the "Link with Webathena" flow was producing
invalid credential caches when run on Python 3, breaking the Zephyr
mirror for any user who went through it anew.

Automated message from Dropbox CLA bot
@gnprice, it looks like you've already signed the Dropbox CLA. Thanks!


Fixes for Inventory packets

This fixes a lot of bugs with inventories
Source: gophertunnel

why?

Sorry I don't remember closing this at all. I wonder if the branch was deleted when it got merged. Are you able to create a new pr?

Or better open one on the parent repo as my changes were all merged.

oh okey


Fix/lazyload locales

I want to merge this change because it separates locales from bundle. Fixes #589

PR intended to be tested with API branch: master
Screenshots

Pull Request Checklist

[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Translated strings are extracted.
[ ] Number of API calls is optimized.
[ ] The changes are tested.
[ ] Data-test are added for new elements.
[ ] Type definitions are up to date.
[ ] Changes are mentioned in the changelog.

Test environment config

API_URI=https://master.staging.saleor.rocks/graphql/

Cool, thanks a lot!


Rui doc edit

Went through entire class_notebooks_best_practices.md, which is under user-guides. Rephrased and rewrote most of the items, but still needs revision in case there is a misunderstanding on my part. Left some comments in .md file.

Rewrote all of the documentations in user guide.
Out of 13 documentation, 11 of them has been reviewed by other member.
Two of them documentations (restarting_server_and_kernel.md and troubleshooting_guide.md) are complete but has not been reviewed.


Using App Studio is still recommended rather than using Developer Portal?

Hi Team,
Just to clarify...
In below section, using "App Studio" seems to be recommended to create application package.
But as far as I know now Developer Portal is GA so should Microsoft recommend using Developer Portal? (or recommend both?)
https://docs.microsoft.com/en-us/MicrosoftTeams/manage-apps?toc=%2Fmicrosoftteams%2Fplatform%2Ftoc.json&bc=%2Fmicrosoftteams%2Fplatform%2Fbreadcrumb%2Ftoc.json#upload-an-app-package
Thank you!

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

ID: fbbcabee-fa96-2878-e983-3c966f84ee15
Version Independent ID: 02fa4fd8-77e0-5cb2-a4af-576cbb68322f
Content: Manage your apps in the Microsoft Teams admin center - Microsoft Teams
Content Source: Teams/manage-apps.md
Service: msteams
GitHub Login: @KarliStites
Microsoft Alias: kastites

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


More explicit failure message when two types target the same index

https://hibernate.atlassian.net/browse/HSEARCH-4006

Perfect! Merged, thanks


Some css

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #
Test URLs:

https://carousel--birdiesforthebrave--hlxsites.hlx.page/

Let's make sure the image is full height in desktop mode, then it would be an improvement over status quo.

I just noticed that this change breaks the entire layout for smaller screen sizes, so it looks like we need to rework this here.

@cziegeler I'm merging this one, too. We'll have to make another pass at the mobile version of this page.


Can't read full type name from DartType

I have class like this
class Test {
    Map<MyEnum, Map<String, String>> animations = {};
}

Then I used source_gen to scan & read properties of this class
class MyGenerator extends Generator {
 @override
 String generate(LibraryReader library, BuildStep buildStep) {
    library.allElements.forEach((element) {
      final classElement = cast<ClassElement>(element);
      final annotations = library.annotatedWith(typeChecker);
      if (classElement != null && !classElement.isEnum) {
        for (var field in classElement.fields) {
          var type = field.type.getDisplayString(withNullability: false);
           // I expect the type is "Map<MyEnum, Map<String, String>>" but it returns "Map<dynamic, Map<dynamic, dynamic>**>"
          print(type);
        }
      }
    });
 }
}

Before I can get the full type of DartType with getDisplayString()
It returns the value Map<MyEnum, Map<String, String>>
But now it returns Map<dynamic, Map<dynamic, dynamic>**>
I did upgrade the sdk many times, so I don't remember which version that could work before.
This is my current version
Dart SDK version: 2.14.0-383.0.dev (dev) (Wed Aug 4 23:26:45 2021 -0700) on "macos_x64"

From the document, I know that I shouldn't depend on getDisplayString(), but I can't find other way to do that.
How can I get the correct type of properties like Map, List...?

FYI - @scheglov @srawlins - expected behavior or a regression?

Could you please provide the full enough subset of the generator, so that something could be run to reproduce this issue? From a first glance it should give the type. Converting it to code is a different question, but it should not be all dynamic.

FWIW, I cannot reproduce this in a standalone unit test.
  solo_test_XXX() async {
    await resolveTestCode(r'''
class Test {
    Map<MyEnum, Map<String, String>> animations = {};
}
''');
    var type = findElement.field('animations').type;
    print(type.getDisplayString(withNullability: true),);
  }

The result:
00:01 +0: ClassAliasDriverResolutionTest | solo_test_defaultConstructor
Map<dynamic, Map<String, String>>
00:01 +1: All tests passed!

We have dynamic because MyEnum is not declared.
I don 't think this is related to getDisplayString(), it simply outputs types. It must be related to type resolution, but I have hard time to imagine how to get Map resolved, but not String.

I'll close as stale. If we can get a complete reproduction, please re-open.


bugfix: fix mysql get auto-increment failure

[x] I have registered the PR changes.

Ⅰ. Describe what this PR did
mysql通过statementProxy.getGeneratedKeys()获取自增主键失败时进入catch逻辑中使用“SELECT LAST_INSERT_ID()”继续获取。当updateCount<=1时会在final逻辑中关闭rsFirstId，由于rsFirstId和genKeys引用的是同一个对象，所以genKeys也关闭了，此时再调用genKeys.next()就会报错：java.sql.SQLException: Operation not allowed after ResultSet closed
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)?
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

此时再调用genKeys.next()就会报错 指的是哪？

改动的代码块进行code formatting

OK，等下下班回去改一下

Codecov Report

Merging #4310 (4b89df0) into develop (2c6e647) will decrease coverage by 0.06%.
The diff coverage is 63.63%.

@@              Coverage Diff              @@
##             develop    #4310      +/-   ##
=============================================
- Coverage      48.52%   48.46%   -0.07%
- Complexity      3798     3802       +4
=============================================
  Files            725      725
  Lines          24306    24281      -25
  Branches        2984     2933      -51
=============================================
- Hits           11795    11767      -28
+ Misses         11261    11248      -13
- Partials        1250     1266      +16

Impacted Files
Coverage Δ

.../rm/datasource/exec/mysql/MySQLInsertExecutor.java
69.14% <63.63%> (-0.86%)
:arrow_down:

...obuf/convertor/BranchRegisterRequestConvertor.java
90.47% <0.00%> (-9.53%)
:arrow_down:

...otobuf/convertor/BranchReportRequestConvertor.java
90.90% <0.00%> (-9.10%)
:arrow_down:

...obuf/convertor/BranchRollbackRequestConvertor.java
92.00% <0.00%> (-8.00%)
:arrow_down:

...otobuf/convertor/GlobalBeginResponseConvertor.java
92.59% <0.00%> (-7.41%)
:arrow_down:

...otobuf/convertor/GlobalCommitRequestConvertor.java
93.75% <0.00%> (-6.25%)
:arrow_down:

...otobuf/convertor/GlobalStatusRequestConvertor.java
93.75% <0.00%> (-6.25%)
:arrow_down:

...obuf/convertor/GlobalRollbackRequestConvertor.java
93.75% <0.00%> (-6.25%)
:arrow_down:

...protobuf/convertor/RegisterTMRequestConvertor.java
94.73% <0.00%> (-5.27%)
:arrow_down:

...protobuf/convertor/RegisterRMRequestConvertor.java
90.47% <0.00%> (-4.77%)
:arrow_down:

... and 33 more


Fix testListenerSubscriptionOnSingleServerRestart

IssueTest.testListenerSubscriptionOnSingleServerRestart fails on some GA runs:

https://github.com/hazelcast/hazelcast-cpp-client/runs/2103618189?check_suite_focus=true
https://github.com/hazelcast/hazelcast-cpp-client/runs/2103618154?check_suite_focus=true

I was able to reproduce the failure on a VM running Ubuntu 18. It fails because sometimes the test body ends before the detached thread.
I removed the thread as the map operations are already asynchronous and happen on background.

Linux test PASSed.

Windows test PASSed.

Windows test PASSed.

Linux test PASSed.


Support https://api.github.com/user?

I need to get information about the signed-in user, and I can't find a an API to do it. If I write a PR, will it be accepted?
It seems there are more information in the response to this endpoint, then User currently have. Should I create a new structure, or extend the existing one somehow? How should I name them?

Pulls are absolutely welcome. In looked at https://developer.github.com/v3/users/#get-the-authenticated-user and I can see the structure returned has some private info not available in the user structure returned in other apis.
The pattern is typically to define a new method associated with the github struct https://github.com/softprops/hubcaps/blob/master/src/lib.rs for an exported interface in another module.
This might look something like.
let user = github.users().authenticated().unwrap()

A users mod currently exists but lacks query interfaces https://github.com/softprops/hubcaps/blob/master/src/users/mod.rs but you can infer what one might look like based on the other mods.
As for type name maybe AuthenicatedUser?
Let me know if you get stuck.

Fixed in #90 and #91