icjx: disable interrupts in interrupt worker

Summary
Interrupts have to be disabled if interrupt worker processes them, otherwise assertion may occur as another interrupt tries to queue worker that is not available (because it processes previous interrupts). Interrupts are re-enabled after the worker leaves the loop processing previous interrupts.
Impact
Fixes potential assertion in icjx expander code if two interrupts are called shortly after each other.
Testing
Tested with icjx expander and SAMv7 based custom board.

[Experimental Bot, please feedback here]
Yes, this PR appears to meet the basic NuttX requirements, but it's missing some crucial details.  While it addresses the core components of the summary and impact, it lacks the specificity needed for a thorough review.  Here's a breakdown of what's missing and how to improve it:
Missing Information in Summary:

What functional part of the code is being changed? Be precise.  e.g., "The interrupt handling logic within the I2C driver for the ICJX expander."
How does the change exactly work?  Don't just say interrupts are disabled and re-enabled.  Where are they disabled and re-enabled?  What specific functions or lines of code are modified? What mechanism is used to disable/enable (e.g., critical section, spinlock, etc.)?
Related Issues:  If this fixes a reported bug or implements a requested feature, link the relevant issue(s).  Even if it doesn't directly address an existing issue, linking a related one can provide valuable context.

Missing Information in Impact:

Expand on "Fixes potential assertion...":  Explain why the assertion could occur. What was the race condition? This helps reviewers understand the problem and the effectiveness of the solution.
Be explicit about all impact categories:  Even if the answer is "NO," state it explicitly for all categories (user, build, hardware, documentation, security, compatibility).  If the answer is "YES," provide details.  For example, if there's no user impact, say "Impact on user: NO."
Consider backward compatibility: Does this change affect any existing users of the ICJX driver?

Missing Information in Testing:

Build Host(s):  Provide specifics.  "Linux, Ubuntu 20.04, GCC 9.4.0"
Target(s):  Be precise. "SAMV71-Xplained Ultra, config: default" (or whatever your configuration is)
Testing Logs:  The provided logs are placeholders. Include actual logs demonstrating the issue before the change and the corrected behavior after the change. Show that the assertion no longer occurs.  If the logs are very long, consider providing snippets of the relevant parts or linking to a separate file.

Example of Improved PR Description:
## Summary

This PR fixes a potential assertion in the interrupt handling logic within the I2C driver for the ICJX expander.  The assertion could occur if two interrupts were triggered in close succession. This was due to a race condition where a second interrupt could attempt to queue a worker thread while the worker was already processing a previous interrupt.  The fix disables interrupts before entering the interrupt worker's processing loop and re-enables them after the loop completes.  This is achieved by using a critical section around the processing loop.  This ensures that no other interrupts can interfere with the worker while it is handling pending interrupts.

Related Issue: [If applicable, link to a NuttX issue]

## Impact

* Is new feature added? NO
* Is existing feature changed? YES (Fixes a bug in the ICJX expander driver)
* Impact on user: NO
* Impact on build: NO
* Impact on hardware: YES (Specific to boards using the ICJX expander)
* Impact on documentation: NO
* Impact on security: YES (The assertion failure could potentially lead to unpredictable behavior.)
* Impact on compatibility: NO (Backward compatible with existing ICJX driver usage)

## Testing

I confirm that changes are verified on local setup and works as intended:
* Build Host(s): Linux, Ubuntu 20.04, GCC 9.4.0
* Target(s): SAMV71-Xplained Ultra, config: default

Testing logs before change:

[relevant log snippets showing the assertion failure]

Testing logs after change:

[relevant log snippets showing the corrected behavior, demonstrating that the assertion no longer occurs]

By providing these additional details, you make it much easier for reviewers to understand, evaluate, and ultimately merge your PR.


Hope to support code search in the future

Feeling is still very necessary, hope to adopt, thanks

Hi, this is duplicated to https://github.com/gogits/gogs/issues/13.


Module not found: Can't resolve 'vis-network/dist/types/vis-types' in '/home/jamie/programs/vis-test/src'

Hi, I seem to be unable to use the DataSet.
Steps to reproduce:
Run the commands:
yarn create react-app vis-test --typescript
cd vis-test
yarn add vis-data
yarn add vis-network

Replace the src/App.tsx file with the following:
import React from 'react';
import './App.css';
import { Network } from 'vis-network';
import { DataSet } from 'vis-network/dist/types/vis-types';

class App extends React.Component<{}, { networkElement?: HTMLDivElement }> {
  constructor(props: {}) {
    super(props);
    this.state = {};
  }

  render() {
    const instance = this;

    if (instance.state.networkElement) {
      const nodes = new DataSet([{ id: 'a' }, { id: 'b' }]);
      const edges =  new DataSet([{ from: 'a', to: 'b' }]);
      const data = { nodes, edges };
      new Network(instance.state.networkElement, data);
    }

    function refCallback(elem: HTMLDivElement | null) {
      if (elem && !instance.state.networkElement)
        instance.setState({ networkElement: elem });
    }

    return (

    );
  }
}

export default App;

run the command:
yarn start

See runtime error:
Failed to compile.

./src/App.tsx
Module not found: Can't resolve 'vis-network/dist/types/vis-types' in '/home/jamie/programs/vis-test/src'

thanks for reporting this @jamietwells, as well as the steps to reproduce.
That said, we don't current ship TypeScript types for visjs-network.  You might have luck making a custom module declaration for visjs-network in your project, and then dropping in these generated types https://github.com/visjs-community/visjs-network/issues/52#issuecomment-511128577
Like you see in #52, we're thinking about implementing more robust support for TypeScript.  If that is something that you are interested in, we welcome contributions 😄


Updating Nginx Reverse Proxy documentation

Closes #727
Staged: http://<IP_ADDRESS>:9000/staging/DOCS-727/integrations/setup-nginx-proxy-with-minio.html
Summary
With the websocket changes for MinIO Console, the existing nginx reverse proxy instructions need updating.
We also have run into a lot of users who have confusion on hosting MinIO on subpaths, which breaks the S3 signature calculation.
This PR handles both areas:

Proxying MinIO based on subdomains of a single hostname
Proxying MinIO based on subpath of a single hostname
Including the websocket upload logic

These docs make no reference on the nginx structure, because in my latest testing the whole sites-available setup has vanished entirely for a simpler single default file you modify. We are comfortable letting Users take on the responsibility of being experts at nginx.

Consider updating the NGINX link on this page to point to this topic.
https://min.io/docs/minio/linux/operations/install-deploy-manage/expand-minio-deployment.html#networking-and-firewalls

@djwfyi back to you - I have internal signoff on the engineering side.


docs(readme): fix links

thank you
please update commit message as per conventional commits
like docs(readme): fix links

made the required changes. Can you check now?

made the required changes. Can you check now?

You've updated PR name.


Fix energy calculation module, modify its kernel, add test

Fix the incomplete 'ComputePotentialEnergy.py' module and add test for this module.

Hi Hossein,
very nice work. I think it would be better to have k and impactFact as class variables rather than global scope variables (see inline comment). Could you change that, please. I'll merge as soon as that's done.
Cheers,
Dominic

i'm not sure why I changed that! Is it still okay that they're being passed to the kernel as options? or should I pass k as a scalar?

I was wondering about that, too. In principle having k and impactFact passed as kernel options can result in faster code (e.g. AMD GPUs can embedd scalars into op codes for instructions, thus saving registers). However I doubt that this leads to a noticeable performance difference in this case. An advantage of passing them as scalars to the kernel is that one can change them without having to recompile the kernel.
I would pass them as scalar arguments to the kernel but I don't have a super strong preference one way or the other. I'll leave this decision up to you.

Nice work Hossein. Thanks.


_ Prechecks _

+1 :green_heart:
dupname
0m  0s

No case conflicting files found.

+0 :ok:
codespell
0m  0s

codespell was not available.

+0 :ok:
shelldocs
0m  0s

Shelldocs was not available.

+1 :green_heart:
@author
0m  0s

The patch does not contain any @author tags.

+1 :green_heart:
test4tests
0m  0s

The patch appears to include 1 new or modified test files.

_ branch-3.3 Compile Tests _

+1 :green_heart:
mvninstall
33m 49s

branch-3.3 passed

+1 :green_heart:
mvnsite
0m 39s

branch-3.3 passed

+1 :green_heart:
shadedclient
16m 41s

branch has no errors when building and testing our client artifacts.

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
0m 33s

the patch passed

+1 :green_heart:
blanks
0m  0s

The patch has no blanks issues.

+1 :green_heart:
mvnsite
0m 31s

the patch passed

+1 :green_heart:
shellcheck
0m  0s

No new issues.

+1 :green_heart:
shadedclient
15m 44s

patch has no errors when building and testing our client artifacts.

_ Other Tests _

+1 :green_heart:
unit
0m 32s

hadoop-azure in the patch passed.

+1 :green_heart:
asflicense
0m 35s

The patch does not generate ASF License warnings.

79m 14s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3125/1/artifact/out/Dockerfile

GITHUB PR
https://github.com/apache/hadoop/pull/3125

Optional Tests
dupname asflicense mvnsite unit codespell shellcheck shelldocs

uname
Linux 68fa8e5ec581 4.15.0-112-generic #113-Ubuntu SMP Thu Jul 9 23:41:39 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/bin/hadoop.sh

git revision
branch-3.3 / 580f74282256f72ac7bee945bae3216fcef41378

Test Results
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3125/1/testReport/

Max. process+thread count
539 (vs. ulimit of 5500)

modules
C: hadoop-tools/hadoop-azure U: hadoop-tools/hadoop-azure

Console output
https://ci-hadoop.apache.org/job/hadoop-multibranch/job/PR-3125/1/console

versions
git=2.17.1 maven=3.6.0 shellcheck=0.4.6

Powered by
Apache Yetus 0.14.0-SNAPSHOT https://yetus.apache.org

This message was automatically generated.

Testing done: It was ensured that the xml gets generated with the append blob config and runs successfully.

+1


Feature Request: CornerRadius on Cursor Shape

Summary of the new feature/enhancement
When setting the cursor shape to filledBox or vintage, it would be nice if you could round the corners of that shape.
Proposed technical implementation details (optional)
If the cursor is drawn from a shape, then you could add a setting for cursorCornerRadius: "2, 2, 2, 2" and apply that to the shape.  If however the cursor image is drawn from a font in the glyph - then perhaps CascadiaCode could have rounded boxes for the cursors.

Hey, that’s clever. I like it.

I would like to take this one as a first issue!

I would like to take this one as a first issue!

Nobody else seems to be working on it, so we look forward to your pull request!

@jpbelval You still working on it?

Yes, a PR will come shortly.

@ATOMworkplace you can take it!

@jpbelval @ATOMworkplace anyone dealing with issue rn?

You can take it!


non-interactive mode for whole document

Hello guys.
Would be possible to introduce the new CLI parameter --non-interactive which will together with runme run --all execute all commands without any interaction (like standard script)?
This may be handy for testing the whole document in pipeline.
Thank you...

Thanks for raising this @ruzickap. We do have a similar ticket here: https://github.com/stateful/runme/issues/297. We'll take a look.

This was fixed and released in https://github.com/stateful/runme/issues/297#issuecomment-1675113228.
The way it will work is adding --skip-prompts to explicitly force it. However, in a C/I pipeline it will auto-detect whether or not a terminal is attached and default to --skip-prompts=false, @ruzickap. Let us know how it goes for you.


Connection is not thread-safe

Version: redis==2.10.6
Platform:  Python 3.7 on  Ubuntu 18.04
Description:
I encountered the same question with the link: https://github.com/celery/celery/issues/4363
I reproduce the error by inserting time.sleep() in Connection.connect which is trying to initialize self._sock, then start two thread, make both pass self._sock existence check and run into Connection._on_connect (apprently Connection.connnect without any thread-lock).
So when one thread in on_connect() call  read_response, it maybe get another thread's response, then raise the error.
def on_connect(self):
    "Initialize the connection, authenticate and select a database"
    self._parser.on_connect(self)

    # if a password is specified, authenticate
    if self.password:
        self.send_command('AUTH', self.password)
        if nativestr(self.read_response()) != 'OK':
            raise AuthenticationError('Invalid Password')

    # if a database is specified, switch to it
    if self.db:
        self.send_command('SELECT', self.db)
        res = self.read_response()
        if nativestr(res) != 'OK':
            raise ConnectionError('Invalid Database')

Is this a bug or something I don't get?

Connections are not meant to be threadsafe. ConnectionPools are threadsafe.

@andymccurdy https://github.com/andymccurdy/redis-py/issues/906 it seems the default connectionpool is not threading safe.


analysis with "so" before qualifier in a sentence with negation gives wrong results

Thanks for picking this project up. It's a good fit for an app prototype I'm building.
Right now I'm playing with the library from Clojure REPL.
Works fine except there are some problems with "so".
moodwiz.core=> (calculate-sentiment "good")
{"negative" 0.0, "neutral" 0.0, "positive" 1.0, "compound" 0.4404}
moodwiz.core=> (calculate-sentiment "so good")
{"negative" 0.0, "neutral" 0.238, "positive" 0.762, "compound" 0.4927}
moodwiz.core=> (calculate-sentiment "I feel good")
{"negative" 0.0, "neutral" 0.256, "positive" 0.744, "compound" 0.4404}
moodwiz.core=> (calculate-sentiment "I feel so good")
{"negative" 0.0, "neutral" 0.385, "positive" 0.615, "compound" 0.4927}
moodwiz.core=> (calculate-sentiment "I don't feel good")
{"negative" 0.546, "neutral" 0.454, "positive" 0.0, "compound" -0.3412}
moodwiz.core=> (calculate-sentiment "I don't feel so good")
{"negative" 0.0, "neutral" 0.445, "positive" 0.555, "compound" 0.5777}

Hi Nek, this is not an issue in this project code. I've tested[1] it and it's an issue with the original Python implementation from Hutto and the NLTK team. I recommend that you point it out to them so we can all benefit from the improvement.
Anyway, thanks a lot for the report! I'll mark it as an improvement to be scheduled.
And yes, I also picked VADER because I'm integrating it with another project :-)
Cheers!
[1]
Python 3.5.3 (default, Jan 19 2017, 14:11:04)
[GCC 6.3.0 20170118] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from nltk.sentiment.vader import SentimentIntensityAnalyzer
>>> sid = SentimentIntensityAnalyzer()
>>> ss = sid.polarity_scores("I don't feel so good")
>>> for k in sorted(ss):
...         print('{0}: {1}\t'.format(k, ss[k]), end='')
...
compound: 0.5777	neg: 0.0	neu: 0.445	pos: 0.555	>>>
>>>

Hi,
I looked into this issue. The issue arises from this line which is the implementation of this.
If you use a statement like I don't feel completely good, you can get correct result i.e. {negative=0.466, neutral=0.534, positive=0.0, compound=-0.3865}. If you have so in place of completely,  the valence is multiplied by a 1.25, otherwise it is multiplied by -0.74.
I made a few changes here but they will break tests and also some checkstyle rules.
I basically added a rule which checks in a if the trigram has <negative word> <some word | so | this> <so | this>., if that occurs accordingly adjust the score. Previously, we were only handling <never> <some word | so | this> <so | this>. After this you'll get {negative=0.466, neutral=0.534, positive=0.0, compound=-0.3865}.

Hi Animesh, thanks for taking the time to try a fix/enhancement to this issue.
I took a look at your changes. I believe it will be better to recode it without breaking the tests... ;-) otherwise, we will have to guarantee (by extensive testing) that the implementation is indeed better than the original by Hutto!
Cheers and tell me what do you think...


docs(v2): grammar typo in migration-overview.md

Small grammatical change

Motivation
(Write your motivation here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Bonus points for screenshots and videos!)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/facebook/docusaurus, and link to your PR here.)

thanks


Ensure Transaction Uniqueness in check_next_block

Motivation
The check_next_block function, which is executed by syncing nodes, currently lacks a validation step to ensure that new transactions in a block do not already exist in the ledger. Without this check, there is a risk that duplicate transactions could be introduced into the ledger. This PR adds the necessary validation to prevent such scenarios, ensuring that each transaction is only included once.
This only affects syncing nodes.
Test Plan
Ran it locally.
TODOs:

[ ] add unit test to showcase in which scenario the missing uniqueness check could have been exploited.

Closing this draft PR, as the unit test in the branch ensure_tx_do_not_exist_unit_test showed that there are already tx uniqueness checks in check_next_block -> check_speculate -> check_transactions -> check_transaction:
    // Ensure the transaction ID is unique.
    if self.block_store().contains_transaction_id(&transaction.id())? {
        bail!("Transaction '{}' already exists in the ledger", transaction.id())
    }


Write a Java program to find lcm of two numbers

Description
Write a Java program to find lcm of two numbers
LCM (Least Common Multiple) of two numbers is the smallest number which can be divided by both numbers. For example, the LCM of 15 and 20 is 60 and LCM of 10 and 15 is 30.
Input  : 12 18
Output : 36

How to contribute

Save the solution in FindLcmOfTwoNumbers.java file
Add FindLcmOfTwoNumbers.java file in find-lcm-of-two-numbers folder

I would like to work on this, pls assign.

!assign

!assign

Hey @anshu-6537, this issue is already assigned to @Vignesh-dev07 !!!
Please choose another issue.
Thanks for your interest in contributing to this project.


Fixed #144 UseBasicParsing missing on Invoke-WebRequest

This should make OctoPosh usable by system accounts or when Internet Explorer is not available/configured. See #144

What about this pull request. This forbids anyone using a local system account to use OctoPosh :(

Hi @blemasle I was convinced I merged this already. For some reason its not letting me do it right now. Would you please branch the latest Octoposh and send back a PR? I'd do it myself but I'd like to give you the credit for it :)

Sorry, I did not see the notification about your request. I'll do it right now

@Dalmirog There you go :)

@Dalmirog Just a friendly reminder about this pull request.


Simulator export fails

I am trying to build my own app using the export target with the simulator. And I've run into few issues :
To be clear, here's the procedure I following :

./tools/configure.sh sim:nsh
make export
tar -xf nuttx-export-12.4.0.tar.gz
My CMakeList.txt file :

cmake_minimum_required(VERSION 3.16)
project(test)
include(../nuttx-export-12.4.0/scripts/toolchain.cmake)
add_executable(test main.c)

cmake . && make and here's the output
ld: unrecognized option '-Wl,--gc-sections'

Indeed in the target.cmake of the export the LDFLAGS are :
"-Wl,--gc-sections -Wl,-Ttext-segment=0x400000 -Wl,-Map=/home/nuttxspace/nuttx/nuttx.map")
Which can not work because we passing the -Wl argument directly to the linker... First issue.

To go further, I have just modified the target.cmake, removing the -Wl before the arguments.
Then the linker returns this error :
ld: read in flex scanner failed
With some research, this error is a the way of the linker to say that it can't read some arguments. Some tests showed me that the -T{LINKER_SCRIPT} argument (toolchain.cmake) was the culprit.
Here's the line where we set the LINKER_SCRIPT variable :
set(LINKER_SCRIPT ${NUTTX_PATH}/scripts/${LDNAME})
Problem, in the target.cmake the ${LDNAME} is empty:
set(LDNAME              "")
The LDNAME variable is set in the Export.mk file :

ifneq ($(ARCHSCRIPT),)
  LDPATH = $(call CONVERT_PATH,$(ARCHSCRIPT))
  LDNAME = ${notdir ${LDPATH}}
  LDDIR = ${dir ${LDPATH}}
endif

For now, I've stopped there the investigation, but I will go further in this. These errors make me think that the simulator export is bugged. What do you think ? Is it possible that it isn't really used and it is now outdated ? Or is there something I'm doing wrong ?
By the way, without the export, it works perfectly well (tested with nsh).
ping @leducp

Hi @nicolas71640 do you need to use CMakefile? Maybe the export building using Makefile still working. There is an old Documentation (that didn't migrate to our new Documentation/) here: https://cwiki.apache.org/confluence/display/NUTTX/Building+NuttX+with+Applications+Outside+of+the+Source+Tree

@xuxin930 since you are fixing the CMake, could you please take a look?

hi @acassis @nicolas71640
Thank you for your reminder 😊
The current CMake base does not support the import export feat.
We considered using cmake install during the design, but it is not perfect yet.
You can first consider using the Makefile base compilation method.
I will mark an action and try to add this feature of CMake as soon as possible.

I think there is a quiproquo here: we are using CMake as our project tool, but we build NuttX with Make file. The exported CMake toolchain file is based on the export.sh script and I imagine that the variables are coming from the Makefile mechanism since it is the one we are using to build NuttX.

Thank you all for your help.
As I understand it, @acassis and @xuxin930  you suggest to use the export AND Import script to us the Makefile to build the final binary. I actually didn't know that an import.sh and import target existed... I've tried this morning. Works perfectly with a random arch/board, but doesn't work with the simulator. First, same issue with the linker flag (-Wl,--gc-section instead of --gcc-section in the LDFLAGS), then this issue :
ld: cannot open linker script file -o: No such file or directory
Am I doing something wrong ?
Just followed a readme that I've found in the source :
    $ cd nuttx
    $ ./tools/configure.sh sim:nsh
    $ make V=1 -j7
    $ make export V=1
    $ cd ../apps
    $ ./tools/mkimport.sh -z -x ../nuttx/nuttx-export-*.tar.gz
    $ make import V=1

@nicolas71640 I got same errors you got:
LD: nuttx
ld --entry=__start -Wl,--gc-sections -Wl,-Ttext-segment=0x400000 -Wl,-Map=/home/alan/nuttxspace/nuttx/nuttx.map -L/home/alan/nuttxspace/apps/import/libs -L "/usr/lib/gcc/x86_64-linux-gnu/11"  \
  -L/home/alan/nuttxspace/apps/import/scripts -T \
  -o nuttx /home/alan/nuttxspace/apps/import/startup/sim_head.o /home/alan/nuttxspace/apps/import/startup/sim_hostfs.o /home/alan/nuttxspace/apps/import/startup/sim_hostirq.o /home/alan/nuttxspace/apps/import/startup/sim_hostmemory.o /home/alan/nuttxspace/apps/import/startup/sim_hostmisc.o /home/alan/nuttxspace/apps/import/startup/sim_hosttime.o /home/alan/nuttxspace/apps/import/startup/sim_hostuart.o /home/alan/nuttxspace/apps/builtin/builtin_list.c.home.alan.nuttxspace.apps.builtin_1.o /home/alan/nuttxspace/apps/builtin/builtin_list.c.home.alan.nuttxspace.apps.builtin.o /home/alan/nuttxspace/apps/builtin/exec_builtin.c.home.alan.nuttxspace.apps.builtin_1.o /home/alan/nuttxspace/apps/builtin/exec_builtin.c.home.alan.nuttxspace.apps.builtin.o  --start-group \
  /home/alan/nuttxspace/apps/libapps.a -lsched -ldrivers -lboards -lc -lmm -larch -lapps -lfs -lbinfmt -lboard  -lgcc  --end-group
ld: unrecognized option '-Wl,--gc-sections'
ld: use the --help option for usage information
make[1]: *** [Makefile:124: .import] Error 1
make[1]: Leaving directory '/home/alan/nuttxspace/apps'
make: *** [Makefile:142: import] Error 2

Probably it is happening because normally people use it mainly with real boards to integrating NuttX kernel as library and avoid using our building system in production.
@xiaoxiang781216 did you or your team use make export/import with simulator?

Yes, but only for Makefile

@xiaoxiang781216 What do you mean by "only for Makefile" ?
as said I said here : https://github.com/apache/nuttx/issues/11863#issuecomment-1991042732 without any cmake help, the export/import thing fails... Am I misunderstanding something here ?

we export and import only with make, not try export with make and build with cmake. Your usage doesn't coverage by daily ci or even mention in document.

I'm not using cmake. I'm using make targets "export" and "import".

Hi everyone !
After a bit of investigation, I'm beginning to understand how the simulator work, and therefore how it compiled/linked.
If you allow me, I'll sum up what I've understood and I haven't...
As the simulator runs on the host, it must replace some of its symbols by nuttx symbol. For instance, the method puts, shouldn't print something on the output but use the "mocked" serial console of nuttx (passing through the UART for instance). This map of symbols is defined in nuttx-names.in.
So the first step of the linking, is to make a "partially linked" lib (nuttx.rel) containing nuttx code, replacing the libc symbols by its own. Then the second step is to take this nuttx.rel lib and link it to the interface with the host (all the HOSTOBJS). An other object is linked in this final binary is the sim_head.o, containing the entry point of the simulator.
Did I understand correctly what's happening here ?
Now, for the export. My first though was that we needed this nuttx.rel to be linked on the final target. So put that nuttx.rel in the archive. Apparently this is not the strategy. I see that we export the nuttx-names.dat in the archive (in libs/ ). Why not, but I can't find  where this file is used to replace the symbols in the import scripts... An idea ?
Therefore, I have absolutely no idea how the simulator export could work...

@nicolas71640 I think your assumptions/findings are mostly corrects.
@xiaoxiang781216 @anchao any idea how to make the simulator to export the symbols correctly?

@nicolas71640 I think your assumptions/findings are mostly corrects.

Yes, sim has verry special linking process, which isn't fit well with the current import/export

@xiaoxiang781216 @anchao any idea how to make the simulator to export the symbols correctly?

We didn't try export/import sim before because:

Simulator doesn't support protected and kernel mode
All sim arch specific code is open source

Both make the usage of sim export/import is very seldom.
@nicolas71640 which functionality drive you want this feature?

We wanted to use the simulator for test purposes. We thought it would be a simple way to have our application packaged with nuttx in a very testable/instrumentable environment as linux. We are also planning to have tests on the board, but these tests would be slower and less instrumentable, so we wanted to have both.
The other option was a qemu obviously, but we thought the simulator would be lighter and faster, and maybe simpler to set up in our CI. Now, if you have advices, we are willing to rethink our strategy.
Exporting the nuttx.rel (and nuttx.ld), I have actually succeed to link the final binary, BUT not only with ld, but passing via gcc (as we do it the sim Makefile). I assume it uses standard libs that I don't put with ld (but I can't figure out which one I'm missing with ld). So I f I want to make the export/import stuff works for the sim, I'll have to add some var, and a specific strategy only for the simulator...

Qemu may better than sim in this case, because both toolchain and export/import is same as the real device.
Most arch (Cortex-A/M/R, ARM64, RISCV32/64 and XTENSA) has the demo board, so NuttX qemu can be setup quickly.
BTW, since toolchain/arch is same as your final product, the library built with exported SDK can be used with your hardware directly, which is impossible for simulator.

All right !
Well, thank you @xiaoxiang781216. I think we are going to follow your advice !
Closing this issue then ;)


1.2.0 and 1.3.0+ problems

Switching from 0.45.0 to 1.1.0 was successful, however, afterwards there are some issues that we encountered:

1.2.0: all modules are seen as affected and built even though there are no changes. Before 1.2.0 everything was working as expected. The only change is package.json for switching the plugin version. Both package.json and package-lock.json are nx ignored. Possibly .nxignore is no longer correctly taken into account?
1.3.0+: plugin no longer works:

Hi @MrSinisterX
I think, the problem is du to a breaking change in NX 19.2.0, they remove projectGraphCacheDirectory and they change it to workspaceDataDirectory.
If you use NX 19.2.0 or higher version, you can use the last version of nx-gradle.
If your version is less that NX 19.2.0. keep 1.2.0 version of nx-gradle.

In version 1.2.0 we changed the cache folder from .nx/cache to .nx/workspace-data. Maybe this made the cache not valid.

same error in nx repo: https://github.com/nrwl/nx/issues/26470

Hi @MrSinisterX
I think, the problem is du to a breaking change in NX 19.2.0, they remove projectGraphCacheDirectory and they change it to workspaceDataDirectory.
If you use NX 19.2.0 or higher version, you can use the last version of nx-gradle.
If your version is less that NX 19.2.0. keep 1.2.0 version of nx-gradle.

Thank you for the clarification. Indeed upgrading NX to 19.2.0 or higher works with the latest plugin version.
Also thank you for the awesome work!


Transparent Background

When i add the stats overlay to obs, on the transparent background setting, the background is white. Is there a fix to this?

This is not a bug. To use it on OBS you will need to set your background to something other than transparent (on the Fall Guys Stats settings) and set that color as transparent on your OBS settings.


[typespec-python] Calling a generated model property with the getter returns the wrong/str value

When a generated model has a property that takes an enum, it does not return the enum when the getter is used. It only returns the enum when dict notation is used.
For example, in Schema Registry , this generated SchemaProperties model is returned by the register_schema operation. Calling .format on that object will return a different value than calling ["format"].
print(type(schema_properties.format)) # is a str, but should be a SchemaFormat enum type
print(schema_properties["format"]) # correctly shows the SchemaFormat enum type

To reproduce:

Create the necessary Schema Registry JSON group/resources either through the portal, or by running the New-TestResources.ps1 script on the schemaregistry/test-resources.json file (in devbox for the necessary auth).

Command: azure-sdk-for-python> .\eng\common\TestResources\New-TestResources.ps1 -ServiceDirectory "schemaregistry"
Copy paste envvars into terminal

On this branch: azure-schemaregistry>pip install .
Add these lines under line 106 and run the sample:

print(type(schema_properties.format)) # is a str, but should be a SchemaFormat enum type
print(schema_properties["format"]) # correctly shows the SchemaFormat enum type

I can reproduce it with the following code:
from azure.schemaregistry.models._models import SchemaProperties
from azure.schemaregistry.models._enums import SchemaFormat
from azure.schemaregistry._model_base import _deserialize

# all enum type
raw_response = {"format": SchemaFormat("Avro")}
reponse = _deserialize(SchemaProperties, raw_response)
print(reponse.format + " : " + str(type(reponse.format)))
print(reponse["format"] + " : " + str(type(reponse["format"])))


Is it possible to set schema's configurable options?

Mongoose's schema has some configurable options and you can use them like:
new Schema({...}, options)
Is it possible to set them in Typegoose?

Never mind. Found out that it can be done in getModelForClass. So close the issue.


Remove pubsub

As the TestView was the last view depending on the pub sub system, modify it so it uses program.events. Then gradually remove dependencies on the publish and subscribe procs from the inheritance chains and from the code.

cc: @dmajda @demmer

+1
@bkutil The CI failure seems to be because of decreased coverage, which seems odd given this PR mostly removes code. Please investigate and adjust the numbers if necessary before merging.

Rebased

I'll move the channel allocation to a method on program in a separate PR.


Add Nomensa carousel and three fake events to demonstrate it

Self-merging on account of it just being an integration of Nomensa's code and @jsharkey13 being happy with it.


jsonparse & permanent in CS 7.0

With this code:
$_dataRaw
$data = ^jsonparse($_dataRaw)
$data.firstName

the Output in CS 7.0 is:
{ "firstName": "André" }  André
(which looks ok)
When I just add "permanent":
$_dataRaw
$data = ^jsonparse(permanent $_dataRaw)
$data.firstName

the Output in CS 7.0 becomes:
{ "firstName": "André" }
(whichs looks wrong)

This example works perfectly for me:
topic: ~introductions  keep repeat()
t: hi there
u: LABEL(test ) $_dataraw = ^' { "firstName": "André" } '
$_x =  ^jsonparse(permanent $_dataraw)
$_x.firstName
$_dataX = "hello"
$_x.firstName
email me a topic in which this fails.

Here is the minimalist code to produce the issue:
topic: ~debut repeat []

t: choose Andre ^keep()

u: (Andre)
  $_dataRaw = ^'
  {
    "civilite": "M.",
    "firstName": "André"
  }
  '

u: ($_dataRaw)

  $_dataRaw
  $data = ^jsonparse(permanent $_dataRaw)
  $data.firstName

Without "permanent", $data.firstName produces "André". With permanent, it does not.

Here is the minimalist code to produce the issue:
topic: ~debut repeat []

t: choose Andre ^keep()

u: (Andre)
  $_dataRaw = ^'
  {
    "civilite": "M.",
    "firstName": "André"
  }
  '

u: ($_dataRaw)

  $_dataRaw
  $data = ^jsonparse(permanent $_dataRaw)
  $data.firstName

Without "permanent", $data.firstName produces "André". With permanent, it does not.

fixed (see prior note on other bug)

fixed (see prior note on other bug)


Contacts search

User should be able to type in the payment/request "to/from" field any of his
contacts, autocomplete.
https://xd.adobe.com/view/f195a710-6a9c-4365-68e5-5644b73d1e36-87a4/
Send:

once contact is chosen check if exists as a user
gun.get('users/bymobile').get(mobile).get('profile').get('walletAddress')
if exists then send payment directly over blockchain without payment link
if doesnt exists then send SMS with payment link

Receive (request)

TBD

@LiavGut can you post the design please

@nicastelo updated description and added design guidelines

@sirpy
https://xd.adobe.com/view/f195a710-6a9c-4365-68e5-5644b73d1e36-87a4/

@sirpy Recent contacts should be saved in AsyncStorage, right? Or do you prefer any other place?

@sirpy @nicastelo
By the way, recent contacts don't have to mean that they are from the user contact list.
They could also be from last transactions to users with wallets (via QR code, for example).

I think they should be fetched from the user tx feed


fix(users): Duplicate Provider Accounts

Fixes an issue where a provider account could be added to the logged in
user's additionalProviderData, when another user has already connected
that same account.
Refactors the saveOAuthUserProfile method to consider the results of the
provider search query, whether or not the current user is logged in.
Steps to reproduce:

Signup for a new MEANJS account using a social provider account.
Signup for a new MEANJS account using local provider.
While still logged in with the User created in step 2, add the same social provider account used in step 1.

Results will be that the local provider User will be connected to the social provider account that was created in step 1.

@mleanos  Seems that test pass but I'm not sure our tests actually go through the flow impacted by this code.

@lirantal I was thinking the same thing. I'll look at adding some unit tests. Some of the logic in this flow is handling edge cases, and we don't have unit tests written for the Social Provider feature :/

Right. It might be quite overwhelming to add some tests for that so if you could at least manually test some of the flows related to login with non-social as well as with social to make sure things are playing nice would be good as well at this point.

@lirantal Sorry for my lack of response. I was putting this off so I could seriously consider adding Social Provider tests to the coverage. At this point, it doesn't look feasible to add tests for the various social providers since it would involve setting up developer accounts for each 3rd party API we'd like to test. We'd need access to these credentials during the build process which could pose some additional considerations with security & infrastructure.
As for these changes, I've definitely manually tested this with the many scenarios that I could think of. Everything is behaving as expected & fixes the bug that this PR is intended for. I'm fine with merging this, and starting a discussion on test coverage for the Social Provider feature.
WDYT?

We could probably mock in a way the social logins but yeah, that's a big undertaking at this point with the project.
@mleanos tthanks for validating the manual testing for it, please merge.

@lirantal I thought about mocking the social providers as well. I'll try working on that over the next few days.


chore: bdd-spring-1573221285 to 0.0.1

chore: Promote bdd-spring-1573221285 to version 0.0.1

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approvers:
If they are not already assigned, you can assign the PR to them by writing /assign  in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment


When creating playlists, how do you know which videos are for the learning levels?

Hi all,
I am working with a customer to setup learning pathways and have stumbled across an issue:
When Adding Content (Different Audience & Learning Levels : End User or Admin…, Intermediate, Advanced, etc… )
https://docs.microsoft.com/en-us/office365/customlearning/custom_addassets
Once its created Subcategories & Playlists… how do you differentiate the Level of “Assets” ? As it appears all videos come up and you have to add the videos in, come out of it and go in to view it. Then click each category and see what levels they are.
Is there an easier way to do this?
( See screenshot below )

Also when adding our own Training content…
Does anyone have any guidance on how this is done, where does the content need to reside, Best practices, etc…
Thank you!

We are currently reviewing the behavior and we will get back to you with our findings

I recommend saving training content on the sites where the content creator has access to edit and update. One of the big benefits of Learning Pathways is centralizing the content that is decentralized to give content owners direct access. The Category, Level, and Audience are all related to the playlist only. You cannot relate this information to each asset in the playlist.

Emily is correct.  The content currently does not relate to the levels, only the playlists.  Customizing the playlists with your own design that works for your users is the flexibility M365LP offers.  Most content that comes from our web experience at support.microsoft.com is not deeply advanced content.  From my history training I would not say any content there is beyond level 200 at this time.

User information was delivered
No further questions from submitter
Closing ticket - if new information is requested, this can always be done in a fresh issue


Survey builders should be able to send test surveys to a specified email address

Removing from milestone, as this is currently being covered by a combination between inviting editors to a survey and the new survey preview


Fix repository link

Opening repo from zed editor leads to nonexisting repo since repo name has _ instead of -

I renamed the repo and then forgot...
Thank you!


Added "className" property to admin + documented existing "style" property

Admin has an undocumented property "style" for customizing the field. However this controls only the root DOM element of the field, and so it's not able to style inner DOM elements, or to style the root element differently from inner elements (e.g. when setting font weight).
This PR adds an additional "className" property, which can be used in conjunction with custom SCSS to achieve a highly-granular CSS design for any field.
Type of change

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

Checklist:

[ ] I have added tests that prove my fix is effective or that my feature works
[x] Existing test suite passes locally with my changes
[x] I have made corresponding changes to the documentation

This is a beautiful idea! I love it!
Merged and will be deployed as beta 0.14.4-beta.0.


Cannot install 'AdvancedSorters' with PlugInstall on Macosx, it says 'No plugin to install'

Hi,
Inside Vim, I type :PlugInstall AdvancedSorters and Vim answers No plugin to install.
My .vimrc has the following lines :
call plug#begin('~/.vim/plugged')
" Initialize plugin system
call plug#end()

Do I have to load AdvancedSorters before installing it  ?
If so, how do I do that ?

:version
VIM - Vi IMproved 8.0 (2016 Sep 12, compiled Jun  6 2017 17:53:37)
MacOS X (unix) version
Included patches: 1-627
Compiled by Homebrew
Huge version without GUI.  Features included (+) or not (-):
+acl             +cscope          -footer          +menu            +num64           +startuptime     +virtualedit
+arabic          +cursorbind      +fork()          +mksession       +packages        +statusline      +visual
+autocmd         +cursorshape     -gettext         +modify_fname    +path_extra      -sun_workshop    +visualextra
-balloon_eval    +dialog_con      -hangul_input    +mouse           +perl            +syntax          +viminfo
-browse          +diff            +iconv           -mouseshape      +persistent_undo +tag_binary      +vreplace
++builtin_terms  +digraphs        +insert_expand   +mouse_dec       +postscript      +tag_old_static  +wildignore
+byte_offset     -dnd             +job             -mouse_gpm       +printer         -tag_any_white   +wildmenu
+channel         -ebcdic          +jumplist        -mouse_jsbterm   +profile         -tcl             +windows
+cindent         +emacs_tags      +keymap          +mouse_netterm   +python          +termguicolors   +writebackup
-clientserver    +eval            +lambda          +mouse_sgr       -python3         +terminfo        -X11
+clipboard       +ex_extra        +langmap         -mouse_sysmouse  +quickfix        +termresponse    -xfontset
+cmdline_compl   +extra_search    +libcall         +mouse_urxvt     +reltime         +textobjects     -xim
+cmdline_hist    +farsi           +linebreak       +mouse_xterm     +rightleft       +timers          -xpm
+cmdline_info    +file_in_path    +lispindent      +multi_byte      +ruby            +title           -xsmp
+comments        +find_in_path    +listcmds        +multi_lang      +scrollbind      -toolbar         -xterm_clipboard
+conceal         +float           +localmap        -mzscheme        +signs           +user_commands   -xterm_save
+cryptv          +folding         -lua             +netbeans_intg   +smartindent     +vertsplit
   system vimrc file: "$VIM/vimrc"
     user vimrc file: "$HOME/.vimrc"
 2nd user vimrc file: "~/.vim/vimrc"
      user exrc file: "$HOME/.exrc"
       defaults file: "$VIMRUNTIME/defaults.vim"
  fall-back for $VIM: "/usr/local/share/vim"
Compilation: clang -c -I. -Iproto -DHAVE_CONFIG_H   -DMACOS_X_UNIX  -g -O2 -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
Linking: clang   -L. -fstack-protector -L/usr/local/lib -L/usr/local/opt/libyaml/lib -L/usr/local/opt/openssl/lib -L/usr/local/o
pt/readline/lib  -L/usr/local/lib -o vim        -lncurses -liconv -framework Cocoa   -mmacosx-version-min=10.12 -fstack-protecto
r-strong -L/usr/local/lib  -L/usr/local/Cellar/perl/5.24.1/lib/perl5/5.24.1/darwin-thread-multi-2level/CORE -lperl -lm -lutil -l
c -F/usr/local/opt/python/Frameworks -framework Python   -lruby.2.4.1 -lobjc

Type:

[ ] Bug
[ ] Enhancement
[ ] Feature Request
[x] Question

OS:

[ ] All/Other
[ ] Linux
[x] OS X
[ ] Windows

Vim:

[x] Terminal Vim
[ ] GVim
[ ] Neovim

Yes, it should be:

Does it install the AdvancedSorters dependencies ?

Does it install the AdvancedSorters dependencies also ?

Does it have any dependencies? If the repo lists any dependencies (check the README), you'll have to install those as additional plugins.

Yes it depends on vim-scripts/ingo-library.
I thought vim-plug was able to resolve and install dependencies.

Unfortunately there's no standard dependency specification format for Vim plugins.

Hi,
Trying to install matchit with this line :
Plug 'vim-scripts/matchit'
but :PlugInstall says:
remote: Invalid username or password
fatal: Authentication failed for<EMAIL_ADDRESS>

Can you help me ?

@sebma it should be:
Plug 'vim-scripts/matchit.zip'
Plug 'vim-scripts/python_match.vim'

You need to check the actual repo name before adding them to your vimrc.

@nfischer Thanks a lot !


Add jsDelivr hits badge

Hey, I noticed that snowplow-javascript-tracker is the 36th most popular project on jsDelivr (congrats! :tada:), so I thought you might like this badge.

Nice thanks! :+1:

Hey @LukasDrgon - thanks for this! Are you involved with the jsDelivr project? We are thinking of moving away from hosting this ourselves and "blessing" a third-party CDN to do this for us - potentially that could be jsDelivr?

Check out over here how Algolia uses jsDelivr for the same thing https://blog.algolia.com/serving-one-billion-javascript-library-downloads/

Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).
📝 Please visit https://github.com/snowplow/snowplow/wiki/CLA to learn more and sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

I signed it!

Confirmed! @LukasDrgon has signed the Individual Contributor License Agreement. Thanks so much


--max-memory does not work - as expected

Done.

Not Done - No Time.

Version: 1.31.1 (user setup)
Commit: 1b8e8302e405050205e69b59abb3559592bb9e60
Date: 2019-02-12T02:20:54.427Z
Electron: 3.1.2
Chrome: 66.0.3359.181
Node.js: 10.2.0
V8: 6.6.346.32
OS: Windows_NT x64 10.0.17134
16 GB memory installed
Steps to Reproduce:

Open a really large file. I tried to open a 1.6 GB xml (don't ask ;-).
VS-code reports it need more memory (excellent - nothing wrong with this).
Click restart with 4096MB.

nothing happens. I would expect it would restart with the new setting.

Start code again and retry 1 get 2 again. Now configure settings. Ok so it says it is set to 4096 MB.

close window and retry loading (1) -> get 2 again.

Goto command prompt, (navigate to folder that contains file.)
code --max-memory 4096
open file => get 2 again.

Does this issue occur when all extensions are disabled?: Yes/No
Not Done - No Time - would not expect extensions to have a say in this...?

Possible duplicate of #68279

It's fixed in Insiders and will be released next week in 1.32


Sync pull requests is not reentrant

Getting new pull requests doesn't work, because the code adds a predicate on updated_at to the results of the pulls() function, and there's no way to push down this predicate if it's not an argument of the function.
To make this work, either:

add another argument to the function - this will complicate the function API
don't use the function but read from the table so that predicates get pushed down - this will fetch all PRs in one query, performing multiple HTTP requests; if any requests fail, the whole query will need to be retried, but this might be ok because there's caching enabled by default and usually there's fewer then 10k PRs in a repo

Also looks like predicates on updated_at are not pushed down at all.

And it looks like predicates on updated_at are not being pushed down.

Done in 4ee22489ea8b6aad1cab72a78c623689462aae12


现已更新版本2.2

Translated: Please update Mod Loader for BR v2.2.0

We're currently waiting for a few days to see if the hotfixes stop. We'll have a new version out soon.

我们目前正在等待几天，看看热修复是否停止。我们很快就会推出新版本。

Okay, there's another issue. I hope to add support for multiplayer mode.

There is already support for multiplayer mode. You need to run your own servers using the provided brml3_server executable. We don't allow access to Steam multiplayer for ethical reasons.

We're currently waiting for a few days to see if the hotfixes stop. We'll have a new version out soon.

Ok, thank you


current date showing wrong day

here the date and actual day is not in sync. for example 2016-0-6-13 is monday. and its showing Tuesday.

Finally able fix this by changing the abstract class DPCalender.java
int dayOfWeek = c.get(Calendar.DAY_OF_WEEK) - Calendar.SUNDAY;
to
int dayOfWeek = c.get(Calendar.DAY_OF_WEEK) - Calendar.MONDAY;


re-enabled main PROD community daily build

Thank you for submitting this pull request
JIRA: (please edit the JIRA link if it exists)
link
referenced Pull Requests: (please edit the URLs of referenced pullrequests if they exist)

referenced pull request 1
referenced pull request 2
referenced pull request 2
etc.

How to retest a PR or trigger a specific build:

* a pull request please add comment: Jenkins retest this

Jenkins retest this

@mareknovotny once this was approved and merged we should have the community upstream daily build with profile -Dproductized again


how to fix?

root@DanyaJsov:~/NotCoin-Clicker# python3 clicker.py
Request caused struct.error: 'i' format requires -2147483648 <= number <=<PHONE_NUMBER>: InvokeWithLayerRequest(layer=167, query=InitConnectionRequest(api_id=6837111937, device_model='PC 64bit', system_version='5.4.0', app_version='1.33.1', system_lang_code='en', lang_pack='', lang_code='en', query=GetConfigRequest(), proxy=None, params=None))
Traceback (most recent call last):
  File "clicker.py", line 24, in <module>
    client.start()
  File "/usr/local/lib/python3.8/dist-packages/telethon/client/auth.py", line 128, in start
    else self.loop.run_until_complete(coro)
  File "/usr/lib/python3.8/asyncio/base_events.py", line 616, in run_until_complete
    return future.result()
  File "/usr/local/lib/python3.8/dist-packages/telethon/client/auth.py", line 135, in _start
    await self.connect()
  File "/usr/local/lib/python3.8/dist-packages/telethon/client/telegrambaseclient.py", line 593, in connect
    await self._sender.send(functions.InvokeWithLayerRequest(LAYER, req))
  File "/usr/local/lib/python3.8/dist-packages/telethon/network/mtprotosender.py", line 183, in send
    state = RequestState(request)
  File "/usr/local/lib/python3.8/dist-packages/telethon/network/requeststate.py", line 17, in __init__
    self.data = bytes(request)
  File "/usr/local/lib/python3.8/dist-packages/telethon/tl/tlobject.py", line 194, in __bytes__
    return self._bytes()
  File "/usr/local/lib/python3.8/dist-packages/telethon/tl/functions/__init__.py", line 253, in _bytes
    self.query._bytes(),
  File "/usr/local/lib/python3.8/dist-packages/telethon/tl/functions/__init__.py", line 127, in _bytes
    struct.pack('<i', self.api_id),
struct.error: 'i' format requires -2147483648 <= number <=<PHONE_NUMBER>
Task was destroyed but it is pending!
task: <Task pending name='Task-5' coro=<MTProtoSender._send_loop() running at /usr/local/lib/python3.8/dist-packages/telethon/network/mtprotosender.py:464> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x7ff07aeae370>()]>>
Task was destroyed but it is pending!
task: <Task pending name='Task-6' coro=<MTProtoSender._recv_loop() running at /usr/local/lib/python3.8/dist-packages/telethon/network/mtprotosender.py:507> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x7ff07aeae070>()]>>
Task was destroyed but it is pending!
task: <Task pending name='Task-3' coro=<Connection._send_loop() running at /usr/local/lib/python3.8/dist-packages/telethon/network/connection/connection.py:316> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x7ff07aeae250>()]>>
Task was destroyed but it is pending!
task: <Task pending name='Task-4' coro=<Connection._recv_loop() running at /usr/local/lib/python3.8/dist-packages/telethon/network/connection/connection.py:335> wait_for=<Future pending cb=[<TaskWakeupMethWrapper object at 0x7ff07aeae2e0>()]>>
Exception ignored in: <coroutine object Connection._recv_loop at 0x7ff07aef6ec0>
RuntimeError: coroutine ignored GeneratorExit

you cannot pass your id as the api id. they are different things
the api id and hash comes from my.telegram.org


Bug with request to /v2/apps/.

For whatever reason, we've seen a user run dcos marathon app update . <properties>, where the . is understood to be an app ID.  So we call http://localhost:5080/marathon/v2/apps/.
Rather than interpreting this as a request for an app with ID ., marathon seems to rewrite this to /v2/apps, which serves a different JSON format, therefore crashing the CLI.

This is probably done by the HTTP framework that Marathon uses and not under our direct control. Wouldn't it make sense to validate that in the dcos CLI?

If "." is an invalid app ID, marathon should reply with an appropriate error message so that all clients can see that.  A CLI check would only fix the CLI, and in general result in duplicated effort across several clients.

Reopen if this is still relevant.


Fix sharing on iOS

Fixed

Fixed sharing to iOS. Changed file extension to ig and change UTI to com.instagram.photo.

This works, can we have someone with write access to the repo to review & approve PR?

So the igo extension is not registered by Instagram on iOS anymore ?

If igo is used Instragram is no longer shown on the Share Sheet.

@LucWollants
question: doesnt this break for older iOS versions?
or actually for older Instagram versions? meaning seems Intagram changed their "package id" and only those who updated Instagram will have the new "package id"

question: doesnt this break for older iOS versions?
or actually for older Instagram versions? meaning seems Intagram changed their "package id" and only those who updated Instagram will have the new "package id"

I did not test this, shame on me. But if that would be the case it makes sense to make this configurable?

I wouldn't say configurable but a test for the version would be nice.
I know Apple is better than Android keeping devices updated I just dont know how many will install the apps in older iOS versions and it wont work for them.
(I'm not in the Apple ecosystem/stuff)
According to this "random site": https://david-smith.org/iosversionstats/
iOS 13 is at 92% overall so I guess thats very good. (I wish Android was like this :cry:)
I will accept this PR and release a patch version of this plugin. Ok?

I wouldn't say configurable but a test for the version would be nice.

You mean testing the iOS version or the Instagram version (not sure this can be done)?

I wouldn't say configurable but a test for the version would be nice.

You mean testing the iOS version or the Instagram version (not sure this can be done)?

I dont know enoguht iOS to answer but I read that as of iOS 13 that changed so I'd test for the iOS version but from what I know from Android thats not a OS thing, its the instagram app that is registering a "endpoint name" that you can use right?

Ooops I acidentally closed the PR.... now doesnt let me reopen.... says the original repo was deleted....
Can you re open @LucWollants ? sorry.

Can you re open
I don't have the rights to do this on this repo. The original PR is from a deleted repo I neither have access rights on.
So best to create a new fork and re-apply the changes and point to this PR.

Ok. I will make a new PR and link to this one to give credits.
but can you answer my previous question?
was this change (igo to ig and etc) an iOS 13 change or Instagram one?

Actually seems the answer is here:
https://github.com/vstirbu/InstagramPlugin/pull/106/commits/8988811d617c7ad3a2a8fa1ad7bb8dfe9a685b79 and
https://github.com/vstirbu/InstagramPlugin/pull/106/commits/4ce60cc3199e1b00bb2a1543994d94b8104b0ba8 and https://github.com/vstirbu/InstagramPlugin/pull/106/commits/b2f66119720b152869ea8aeec764f4100724570f
do you guys agree that I should apply these 3 commits?

Im coming back to this issue now.
I will manually perform this changes and push to a new branch.


Windows: Refactor volumes

Signed-off-by: John Howard<EMAIL_ADDRESS>@swernli
This PR is part of the proposal described in issue 10662 to port the docker daemon to Windows. This factors out copyOwnership() on Windows where it not supported.

LGTM (assuming CI is green).

LGTM. Merging! :fireworks:


discrepancy between Detectron2 and Boxal

Hello Pieter, I have another inquiry that would require your expertise.
Recently, I conducted an experiment comparing the performance of two programs: the orginal Detectron2 (without AL) and Boxal. For Boxal, I set the initial_training_set to include all the annotated data and  Loops=0. Basically, I tried to set all the parameters identical between the two programs.
However, despite my effort, I observed a significant discrepancy in the results obtained from Detectron2 and Boxal. The results from Detectron2 turned out to be significantly better than those from Boxal.
I would greatly appreciate your insights on the potential reasons behind this discrepancy. To be more clear, I have included a summary of the parameters used and the corresponding loss outputs below.
Parameters:
Train set size: 360
Validation set size: 45
Test set size: 45
use_initial_train_dir: True # only for Boxal
network_config: "COCO-Detection/faster_rcnn_R_50_FPN_3x.yaml"
pretrained_weights: "COCO-Detection/faster_rcnn_R_50_FPN_3x.yaml"
classes: ['damaged']
transfer_learning_on_previous_models: True
learning_rate: 0.0005
warmup_iterations: 1000
train_iterations_base: 10000
train_iterations_step_size: 1000
step_image_number: 500
eval_period: 100
checkpoint_period: -1
weight_decay: 0.0001
learning_policy: 'steps_with_decay'
step_ratios: [0.5, 0.8]
gamma: 0.1
train_batch_size: 2
roi_heads_batch_size_per_img: 128
confidence_threshold: 0.5
nms_threshold: 0.05
strategy: 'uncertainty' # only for Boxal
mode: 'mean' # only for Boxal
initial_datasize: 360 # only for Boxal
pool_size: 1 # only for Boxal
loops: 0 # only for Boxal
dropout_probability: 0.25 # only for Boxal
mcd_iterations: 10 # only for Boxal
iou_thres: 0.95 # only for Boxal
incremental_learning: False # only for Boxal
sampling_percentage_per_subset: 10 # only for Boxal

Detectron2:

Boxal:

I set the same LR for both programs. I hope it is not changed in some hidden layers in the code.
That's right. I added a few classes to both programs to handle the validation loss output.
The goal of this test was to check if both programs have a similar behavior with the same setting or not. There must be still some differences in their implementation, I mean for the case of boxal running over one loop. For example, boxal has dropout probability and detectron2 does not. I decreased the dropout probability to almost zero and the train loss for boxal got closer to that of detectron2. See below:

If you could give me some hint, about the modifications implemented in your code and any possible parameters to tweak, that would be very helpful.

Is there a way to turn off that "automatic evaluation method during training which automatically saves the best weights"? Or is it possible to deactivate it by commenting those lines out?

Here is an example of running boxal with the same paramaters, except for the number of loops (=5), number of initial_training images (=20) and pool_size (=68).

And the AP plots for the same setting:

The AP plots for detectron2 with the above-mentioned settings:

I was wondering if I read the evaluation values on the same dataset for both programs, I mean both results in the metrics files should be based on validation dataset.

@pieterblok Any comment on this? I would really appreciate your help to solve the issue. If you don't have the time to go in details, any hints on where to check for the problem would be incredibly helpful.

@pieterblok Any comment on this? I would really appreciate your help to solve the issue. If you don't have the time to go in details, any hints on where to check for the problem would be incredibly helpful.

I have to dive deeper into this, because at the moment I don't have a clue. Because I'm relocating at the moment, I don't have a computer and time to do this.
As a general answer: there have been so many things I have changed. Remember BoxAL was forked from MaskAL and if you look at the commits there, there has been a lot of small tweaks and changes. Adding dropout layers is definitely something that could have caused differences:
https://github.com/pieterblok/boxal/blob/ed4b8915547b8ec7ea7e33d0b947f1136277071a/active_learning/strategies/dropout.py#L176
I have used the default data augmentation of Detectron2 v0.4, so that should be similar. For the weights, it's the easiest way to put a checkpoint number in the yaml file, for example 500 (this means that every 500 iterations a weights-file is stored).  https://github.com/pieterblok/boxal/blob/ed4b8915547b8ec7ea7e33d0b947f1136277071a/boxal.yaml#L33
By looking at your validation loss, you can select another weights file stored in your directory.
That's all what I currently can think of.

Hi Pieter, Thanks very much for making time despite your current situation.

I disabled the dropout layers by using:
FastRCNNConvFCHead, StandardROIHeads, Res5ROIHeads
instead of:
FastRCNNConvFCHeadDropout, StandardROIHeadsDropout, Res5ROIHeadsDropout
No change was observed in the results, compared to the time I set the dropout probability to zero.

I also tried an older version of detectron2 (dated back to March 2021, when you forked MaskAL from detectron2). I thought maybe, because I am using a newer version of detectron2, I get a better results with it. But, the old version works as good as the new version.


az DesktopVirtualization update to 2021-07-12

This checklist is used to make sure that common guidelines for a pull request are followed.

About Extension Publish
There is a pipeline to automatically build, upload and publish extension wheels.
Once your PR is merged into master branch, a new PR will be created to update src/index.json automatically.
The precondition is to put your code inside this repo and upgrade the version in the PR but do not modify src/index.json.

DesktopVirtualization

Hi sayandaw,
We are going to release a new version of desktopvirtualization at 2022/04/06.
It will use the latest stable version (2021-07-12) of desktopvirtualization sdk.


Does JS have an API for finding surrogate pairs in a String?

The suggested segmentation API seems to walk through codepoints of a string and reports different things for different surrogates etc. But does JavaScript actually have an API yet for testing whether a character in a string is a high/low surrogate or not? Is there a way to determine where the different codepoints are in a String? I know I'm not always up-to-date on the latest spec changes, but every time I look up MDN, I'm not seeing any API for that, and such an API is needed to properly use the suggested segmentation API.

You asked a good question but that belong to https://github.com/tc39/ecma402/issues not here.


set default mask (like 'ocean_mask')

currently, the default mask is low resolution 'land_mask'.
It's possible to define another mask with xsar.Sentinel1Meta.set_mask_feature, but this has to be done for each openned product.
We should provide a classmethod to set the default mask at a class level, so the user can set a default mask.
For example, the default land mask doesn't mask lakes. A good ocean mask can be found here : https://osmdata.openstreetmap.de/download/water-polygons-split-4326.zip

so, to set a new default ocean mask (lakes not included):
xsar.Sentinel1Meta.set_mask_feature('ocean', os.path.join(xsar.get_test_file('water-polygons-split-4326'), 'water_polygons.shp'))


define dataset variables for Dataset Metadata Check against Define XML

currently for rules that are Dataset Metadata Check against Define XML, the ContentsDefineDatasetBuilder does not report define_dataset_variables.  Having this produced as one of the pieces of metadata would be useful for rule development

see also: https://github.com/cdisc-org/cdisc-rules-engine/issues/784

I look forward to this enhancement.


Exposing Variant Filtering Params

[x] Tested

unclear what the distinction between varscan_min_var_freq and min_var_freq for now ...

immuno.wdl takes in varscan_min_var_freq & min_var_freq then passes them to somatic_exome.wdl which passes them to detectVariants.wdl which passes min_var_freq for both subworkflows/mutect.wdl and subworkflows/strelka_and_post_processing.wdl and respectively pass that value to subworkflows.fp_filter.wdl which ultimately calls tools/fp_filters with that specified value.

I think we should reconsider why we are setting defaults for these cutoffs at various places in the chain from immuno -> somatic -> detect variants -> variant caller sub workflow -> variant caller tool or false positive filter itself.
I think we want the user running analysis from each of these points to be able override the default, but unless we are explicitly doing it on purpose for a good reason, why should we define a default value anywhere other than the final place it is about to be used (in this case either in varscan or the false positive filter)?

Yeah, agree with Malachi. This seems like an opportunity to clean up some of this parameter passing//logic (see my comment inline upthread).  It's gotten that way over time, through changes by lots of different folks, but we can draw a line in the sand and say "the mess stops now" :)

Working on testing but if all goes well this PR should be deprecated by:
https://github.com/wustl-oncology/analysis-wdls/pull/136

Testing of #136 worked out. Closing this one.


Create dependabot.yml

I have also added dependabot here so we can track the updates of the Go modules

I need an approval of @a


Support for instances without a title

If we want to run multiple tasks concurrently we must wrap them in a new Listr instance, and that instance needs a title, but maybe all we care about is concurrency and the title and the extra nesting level get in the way.

Actually this is already supported, there was no need to wrap my tasks in:
{
  title: 'foo',
  task: () => tasks
}


Bugfix in ci/docker_cache.py

Description
Fixing a small issue with an argument which had been refactored in https://github.com/apache/incubator-mxnet/pull/13182

@mxnet-label-bot update [pr-awaiting-merge]


Create should allow setting public and publishing with one call

using public and published kwargs

Deciding against this for now...it's not too difficult to ask users to do it, and compared with having to figure out error handling for a three-in-one request makes it less worth it


fix(clients): Update clients go sum file

What this PR does / why we need it:
It seems like the management service build was failing because one of the go sum files wasn't updated in clients. This PR updates that go sum file.
Which issue(s) this PR fixes:

Fixes #

Thanks for the quick review once again!


Column stacking breaks when passing in data outside of temporal range

When plotting values on stacked bar chart with a datetime x axis, using a min and max x axis range, stacking of series breaks when the data passed in falls outside the min/max date range.
You can see the issue on this Codepen: https://codepen.io/danb1/pen/JjbbXaj — it works fine initially but when you uncomment the first data object you see that the columns no longer stack correctly.

What is the behavior you expect?

I expect that if a data object is passed to Apex which falls outside of the min/max x axis range it should not impact the rendering of the data that falls within range - effectively it should be silently ignored.

What is happening instead?

When an object is passed into the data object which contains a date that is earlier than the min x axis range, it breaks the stacking of data that falls within range.

What error message are you getting?

There are no error messages, just the stacking of the columns breaks.

I have same problem. Have you solve this problem ?


feat: update to tree-sitter 0.24

LGTM 👍🏼
Sidenote: Having a command/script to run tree-sitter/parse-action locally would be nice.


'Could not find definition for module' on compile, fine otherwise

Hi all,
I've started getting the "Could not find definition for module 'UnrealEnginePython' (referenced via default plugins -> UnrealEnginePython.uplugin)" error.
I've read through the other tickets regarding this error. However, my UEPython installation is actually working as expected - I only get the error when attempting to rebuild my C++ code.
The error occurs when building from VS as well as from UE4.
Using UEPython version 20190508_4_22_python36_win64 (binary version)
Thanks,
Jasper

I also encountered this problem.Do you have a solution to this problem?

same problem here


Report.SAVEAS is slow compared to equivalent Report.Execute

There was a previous issue opened with similar behavior, but it is closed.
https://github.com/Microsoft/AL/issues/3424
When I am trying to use the method Report.SAVEAS(), it works but it is really slow.
When I am running this code:
TempBlob.Blob.CreateOutStream(MyOutstream);
report.saveas(, ReportParam, ReportFormat::Pdf, MyOutstream);
=Any report you are trying.
This takes about a minute to return the result back to the TempBlob.
If I use this call instead, the report data is returned within 2 seconds.
report.Execute(, ReportParam);
I don’t believe the issue is with the ReportParam that I am using since the Report.Execute is using the same Report Param.
I wanted to know why this call is so slow.
Versions:

AL Langauge: 2.1.79379

Business Central: CA Dynamics NAV 13.3 (27233)

I can agree with that behavior as well. The execution is also much slower here on a super fast SSD system.

This is by design. It is caused by the NetFx40_LegacySecurityPolicy server configuration being set to false in the server.config.
The AL language server in VSCode needs this setting in order that publishing, and debugging works.
The good news is that for the Spring release of BC this has been fixed.  This setting is not needed and both the language server and reporting will work as expected.

Sorry, just want to be clear...
You are saying that there is a performance issue with using Report.SaveAs and RDLC in Business Central (Fall '18). We are still seeing this issue (terrible SaveAs PDF from RDLC performance, saving from Word layout is fine) in CU6. Is it right that the fix in Spring '19 release has not been ported to Fall '18? If so, will we get the fix in CU7 or are we going to have to upgrade to Spring?
Thanks.

You can mitigate the issue in the following way:
Set NetFx40_LegacySecurityPolicy server configuration to true in production. This will get rid of the perf issue, but you cannot publish/debug AL.
If you need to publish/debug, set NetFx40_LegacySecurityPolicy server configuration to false while you do it. Reports will run slow while you debug/publish. When you are done, switch the setting back to true.
The server setting can be changed dynamically without the need for a server restart

Thanks for the reply Kennie. We're not seeing a performance improvement though. To confirm, we've

Added <NetFx40_LegacySecurityPolicy enabled="true" /> to Nav.Server.exe.config
Restarted the service tier that we're connected to

Our last throw of the dice is that all service tiers sharing that config need to be stopped and restarted, although you said that wasn't necessary. It is still taking approx. 1 minute to generate a PDF attachment
Are we missing anything?

We still experience poor performance regarding generating a PDF in BC14 CU4.
The problem occurs only with Custom Layout in the report layout selection.
When pressing the button "Email Confirmation" on a sales order the email popup takes 25 seconds to generate.
I've done some testing, RDLC (Build-in) Layout takes less then 2 seconds to generate the email popup.

Same issue on BC15 RTM when we generating Pdf using Report.SaveAs and Custom Layout.
30-60 sec to generate (with Build-in layout - 1-2 seconds)

we're also experiencing this on CU16 (Build 44656).

Hi! Please take a minute to read the guidelines for this repository https://github.com/microsoft/AL and our collection of frequently asked questions https://github.com/Microsoft/AL/wiki/frequently-Asked-Questions .
As this repository is only for issues related to the AL compiler in latest developer preview environment for Dynamics 365 Business Central. We suggest that you open a support case or file a bug in Collaborate, to ensure that all Business Central users benefit from your catch as soon as possible.
To open a support case, you can:

Open a support request to CSS through the PartnerSource portal
Contact your Service Account Manager (SAM) in your local subsidiary to understand what is included in your contract as of support incident and PAH (Partner Advisory Hours). Your SAM might also direct you "step by step" how to open a support request or how to get credentials, if this is the first time for you or your company

If you file the bug in Collaborate, remember to include steps to reproduce the issue, and the Business Central build number and country version you're using.


Fix unet perf and functional CI pipelines

PR: https://github.com/tenstorrent/tt-metal/pull/8884

merged with: 628e5aef6cb7e8e2546bb9e1c7a54144cce7bf49


Route coordinates should be latLng objects instead of arrays

In order to be consistent with the Leaflet Routing Machine API
cfr. http://www.liedman.net/leaflet-routing-machine/api/

Thank you @alex-outdoor


fix pluralization

Replace 'lv' to locale

Hi @nik-zp,
Thank you very much for catching this. It was indeed a very hot day today and this somewhat completely got by me. I will push an update to npm shortly.

Thanks for a great library :)


fix(opa_executor): remove unnecessary mutability on opa executor trait

Signed-off-by: Joseph Livesey<EMAIL_ADDRESS>

Assists CHRON-211.


Mhank gk bisa jalan

Authenticate to continue
✓ Authenticate to continue
✓ Loading QR
✓ WAPI Reinjected
✓ Client is ready
✖ !!!BROKEN METHODS DETECTED!!!

 Please update to the latest version: 3.0.8
✓ Client loaded in 76.53s
✓ Patches Installed

Gak bisa jalan bang

Coba "npm update" Atau "{updateCommand}"
(Tanpa tanda ")
Pada tanggal Sel, 3 Nov 2020 07.15, andra12lol<EMAIL_ADDRESS>menulis:

Authenticate to continue
✓ Authenticate to continue
✓ Loading QR
✓ WAPI Reinjected
✓ Client is ready
✖ !!!BROKEN METHODS DETECTED!!!
Please update to the latest version: 3.0.8
✓ Client loaded in 76.53s
✓ Patches Installed
Gak bisa jalan bang
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/MhankBarBar/whatsapp-bot/issues/103, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AN6UGWT6ANJ3LJ76WFIKHI3SN5DTNANCNFSM4TIDFXNA
.


SVIGP implementation

The SVIGP implementation does not work with the currect version of GPy. Recommend removing and reinstating in a future release.

This works now. See GPy.core.SVGP.
We probably need a notebook to show how it's supposed to work.

Alan produced a notebook at https://github.com/SheffieldML/notebook/blob/master/GPy/SVI.ipynb, this works, is tested, notebooked. Closing.


Loosen bounds on MathProgBase

It's safe to support MathProgBase 0.6, no relevant changes.

Changes Unknown when pulling 4b1b27e243e4011041f1c063a30d31e2b72a28fd on mlubin:patch-1 into ** on MPF-Optimization-Laboratory:master**.


Profile upload

@jpcunningh for docs, you can target master :) directly. Thanks!

Ended up moving this down to own section, and adding a picture example: https://github.com/openaps/docs/commit/141680307776fd34c4ae6b1f78a6099c729d9f6e
Reason for own section is now it can be linked directly to: https://openaps.readthedocs.io/en/latest/docs/While You Wait For Gear/nightscout-setup.html#how-to-automatically-sync-your-profile-from-autotune
Thank you @jpcunningh!  :)

Thanks for helping review and clean them up some, @danamlewis!


Use typescript

Start using typescript for the web client codebase rather than JS as it TS brings better typesafety.

Already done in https://github.com/reconmap/web-client/commit/3de504e6f6c68a05dfb2684b4098d5c3ab96c43a


Laravel 11 Compatibility

Thanks for this great package.
Looks like it depends on illuminate/support v10, can we bump that dependency up to v11 for Laravel 11 support?
Thanks!

@olivermbs can you open a PR for this ?


previous-version - Show on mobile

Fixes https://github.com/refined-github/refined-github/issues/7540

Test URLs
https://github.com/refined-github/refined-github/blob/main/source/background.ts
Screenshot

The code required is so ugly 🥲👹


Operations response status is always "not started"

Hi,
I'm trying to enroll an audio into an identification profile , the audio format is correct and when i try to get the operation status, it always return "not started". What's happening? I can't continue what I'm doing because of this. Thanks!

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

ID: 3604fe7e-46a3-6f5b-41e0-d3e69e01c8ae
Version Independent ID: 9415c8a5-177e-fe11-7198-3d97ed438dc8
Content: What is the Speaker Recognition API? - Azure Cognitive Services
Content Source: articles/cognitive-services/Speaker-recognition/Home.md
Service: cognitive-services
Sub-service: speaker-recognition
GitHub Login: @dwlin
Microsoft Alias: nitinme

@sirjun05 Thanks for the feedback! We are currently investigating and will update you shortly.

@sirjun05 We will now proceed to close this thread since this is not a document isssue. If there are further questions regarding this matter, please respond here and @YutongTie-MSFT and we will gladly continue the discussion.
@GiftA-MSFT Could you please check the status of escalation if there is a bug filed? Thanks.

@sirjun05 Could you please share the snapshots of the same. I have used the following sample  created the profile, upload a audio file and enroll a user. After successfully enrolling the user I tried to identify the user using the identify and it was able to successfully identify the enrolled user. Here is a screen shot of the same:

Get Operations status with operation id:
https://westus.dev.cognitive.microsoft.com/docs/services/563309b6778daf02acc0a508/operations/5645c725ca73070ee8845bd6/console

it's happening again in enrollment, same thing with identification . It's working fine yesterday.

@sirjun05 We would recommend to raise a Azure support desk ticket from Help+Support blade from Azure portal.  This will help you to share the details securely and work with an engineer who can provide more insights about the issue. We have forwarded the same to the product team to check.


Added RDP Support to Command line

This change adds a  option "RDP" in the cli.
You can use the command line option "-rdp"

Thanks for the Effort!


🎃🍂 Make this website configurable to be able to suit many other definitions!

Description
Welcome Hacktoberfest 2016! The goal of this issue is to make the webpage content configurable. Currently, the webpage reminds users that this (.) is not a period, but a full stop. By doing magic with listening to URLs, how about making the words configurable? For example, if user inputs:
https://afulls.top/?incorrect=Chips?correct=Crisps?name=Stranger
It will show them This is a bag of crisps, not a bag of chips!
You can format the URL however you like, the above is only an example. The goal is just to make it easily modifiable and configurable.
Examples

This is not a tire, it’s a tyre.
These are not pajamas, it’s pyjamas.
This is not a cookie, it’s a biscuit.

Contribute?
Please read CONTRIBUTION.MD before creating your first pull request.
Hacktoberfest 2016
This is part of a series of issues designed for Hacktoberfest 2016. New to this? Try #1

It would be possible to change to a react implementation (nodejs based: webpack workflow, create-react-app, ...), which would allow defining the different designs as components. Then you could just pass the url parameters as properties down to the different design components. This would complicate the workflow at first but could have value in terms of maintenance and flexibility. The whole project might then be minifed to one single .html file.

@Vailfire I would like it to be able to be hosted on Github Pages so I don't need an extra host though. Any other ideas?

@choyiny Yeah, sorry I didn't explained it exactly enough. The node.js workflow (required on your dev-machine) generates a minified file from all the sources. The minifed file would contain all the neccessary html and js, so it does not require any additional backend. The page could still be deployed to gh-pages.
But perhaps we don't need any additional workflow if we just add the relevant scripts by hand and renounce any special development environment and the included benefits. That would made it easier, but does no longer automatically generate a combined file of all designs.

@Vailfire I am still trying to think of a good way of combining all the designs made by contributors. Perhaps visitors can be given a random page every time they access the website. What do you think?
With regards to automatic generation, it seems like a good idea. However, I am definitely not experienced with it. I have opened a new branch js so we can play around with it.

Yeah, I like that idea and it seems definitely possible. I have some work for now with reinstalling my computer but I'll play around with it and I will open a pull request later this day.

@Vailfire has implemented a /js branch.


Can't bind to 'options' since it isn't a known property of 'chart'

Hi, i just run into this template parse error:
`Can't bind to 'options' since it isn't a known property of 'chart'. ("<chart [ERROR ->][options]="options">"): ChartBlock@0:7
'chart' is not a known element:

If 'chart' is an Angular component, then verify that it is part of this module.
If 'chart' is a Web Component then add "CUSTOM_ELEMENTS_SCHEMA" to the<EMAIL_ADDRESS>of this component to suppress this message. ("[ERROR ->]<chart [options]="options">"): ChartBlock@0:0`

I used the plunker sample to recreate this.
import { Component, Input } from '@angular/core';
import { ChartModule } from 'angular2-highcharts';

@Component({
  selector: 'chart-block',
  template: '<chart [options]="options"></chart>'
})

export class ChartBlock {
  options: Object;
  constructor() {
    this.options = {
      title: { text: 'simple chart' },
      series: [{
        data: [29.9, 71.5, 106.4, 129.2],
      }]
    };
  }
}

any help would be appreciated :)
with best regards,
max

solved: I forgot to import with @NgModule...


Remove tools feature flag

Summary
Removes the tools feature flag. Depends on #748, #730, #734, #751.

Looks good to me on the preview! and all the checks pass:)


use getvalue() for str obj

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 85.143%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sis_provisioner/models.py
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
149

Relevant Lines:
175

💛  - Coveralls


Integrate bash

I'm finding VS Code is fairly hard to use efficiently without an integrated terminal.
I have to type make repeatedly in a separate bash window (remembering to hit ctrl-s every time!), and from there, I can't double-click on compile errors to go to the line+file.
At a minimum, it would be good to have an integrated shell, and support clicking errors to go to the source.
Ideally, once integrated, a macro could: save all -> execute make, and productivity would increase substantially.

Have you tried to configure a task https://code.visualstudio.com/docs/editor/tasks, then you can trigger make using a short cut, and navigate from errors shown in the output window inside Code, back to your code

Awesome, thanks for the headsup.

@TurkeyMan If you need to run more rare commands, I've just released an extension which allows you to run commands from vscode.
See https://marketplace.visualstudio.com/items?itemName=bbenoist.shell for more info

Integrated terminal issue: https://github.com/Microsoft/vscode/issues/143

+1 https://atom.io/packages/terminal-panel

I'm going to close this as a duplicate of #143


couldnt cancel old booking

[03/13/2023 10:36:32] Carrollton Mega Center is not avaliable in around 0-7 days from today!
[03/13/2023 10:36:32] Plano is not avaliable in around 0-7 days from today!
[03/13/2023 10:36:34] Lewisville is avaliable on 3/14/2023 12:45 PM
[03/13/2023 10:36:35] Slot hold successfully
[03/13/2023 10:36:35] Booking slot....
[03/13/2023 10:36:35] Canceling existing booking 163603313
03/13/2023 10:36:35 [ERROR]: Failed to cancel booking. Please cancel it manually

This may occur when the DPS server is unresponsive. I will make a "retry" function to retry if this keep happening


Merge

About the PR

Merges upstream, DONT YELL AT ME SKIPS!
Why / Balance

Technical details

Media

Requirements

[ ] I have read and am following the Pull Request and Changelog Guidelines.
[ ] I have added media to this PR or it does not require an ingame showcase.

Breaking changes

Changelog

Ugh, happened again.


HttpClientTest.TestRandomURLWithProxy test failure on ubuntu 20.04

Describe the bug
failed tests of HttpClientTest.TestRandomURLWithProxy with message:
SSL peer certificate or SSH remote key was not OK
SDK version number
1.9.40
Platform/OS/Hardware/Device
ubuntu 20.04
host machine: windows 10, docker desktop using WSL 2
To Reproduce (observed behavior)
command ran was: docker build -t  .
docker file:
RUN git clone --recurse-submodules --branch 1.9.40 https://github.com/aws/aws-sdk-cpp
RUN mkdir sdk_build
RUN cd sdk_build
RUN cmake ../aws-sdk-cpp -DCMAKE_BUILD_TYPE=Release -D BUILD_ONLY="s3"
RUN make
RUN make install
Expected behavior
A clear and concise description of what you expected to happen.
Logs/output
#15 401.7 [----------] 6 tests from HttpClientTest
#15 401.7 [ RUN      ] HttpClientTest.TestRandomURLWithProxy
#15 402.0 /aws-sdk-cpp/aws-cpp-sdk-core-tests/http/HttpClientTest.cpp:64: Failure
#15 402.0 Failed
#15 402.0 The curl error code has to be 28 in [curlCode: 60, SSL peer certificate or SSH remote key was not OK]
#15 402.0 [  FAILED  ] HttpClientTest.TestRandomURLWithProxy (259 ms)
#15 402.0 [ RUN      ] HttpClientTest.TestRandomURLWithProxyAndOtherDeclaredAsNonProxyHost
#15 402.2 /aws-sdk-cpp/aws-cpp-sdk-core-tests/http/HttpClientTest.cpp:64: Failure
#15 402.2 Failed
#15 402.2 The curl error code has to be 28 in [curlCode: 60, SSL peer certificate or SSH remote key was not OK]
#15 402.2 [  FAILED  ] HttpClientTest.TestRandomURLWithProxyAndOtherDeclaredAsNonProxyHost (255 ms)
#15 402.2 [ RUN      ] HttpClientTest.TestRandomURLWithProxyAndDeclaredParentDomainAsNonProxyHost
#15 402.2 [       OK ] HttpClientTest.TestRandomURLWithProxyAndDeclaredParentDomainAsNonProxyHost (25 ms)
#15 402.2 [ RUN      ] HttpClientTest.TestRandomURLMultiThreaded
#15 403.4 /aws-sdk-cpp/aws-cpp-sdk-core-tests/http/HttpClientTest.cpp:64: Failure
#15 403.4 Failed
#15 403.4 The curl error code has to be 6 in [curlCode: 28, Timeout was reached]
#15 403.4 /aws-sdk-cpp/aws-cpp-sdk-core-tests/http/HttpClientTest.cpp:64: Failure
#15 403.4 Failed
#15 403.4 The curl error code has to be 6 in [curlCode: 28, Timeout was reached]
#15 403.4 /aws-sdk-cpp/aws-cpp-sdk-core-tests/http/HttpClientTest.cpp:64: Failure
Additional context
used default settings for proxy and network on docker desktop

Hi @n242 ,
This error has been fixed as of versions 1.9.56, could you update and try again?
Also, as a side note you could also make your build faster and avoid getting blocked by adding the -DENABLE_TESTING=OFF argument to your cmake command.


Fix secretname for the apiService

This has to be a known value that we can watch for in the operator, this
doesnt require to be unique per deployment as several processes can
watch the same secret
Signed-off-by: Itxaka<EMAIL_ADDRESS>

This name is used here https://github.com/rancher/elemental-operator/blob/main/pkg/operator/operator.go#L103 to have the steve aggregator watching the secret. So we need to set the name in advance


Support of keyspace field for BATCH message

This PR adds support for the keyspace field for the BATCH message as part of Native Protocol v5 Support.
This implementation is based on the current implementation for QUERY and EXECUTE messages in gocql.

@lukasz-antoniak Thanks a lot for your feedback!
I added some enhancements according to your review.
Could you please take a look at this?

Can this be closed in favor of #1822 ?

Hello👋. I don' see any problem of closing this one and others PRs as well 👍


Install problem on Python 3.11

Hi,
When trying to install 0.0.4 on Debian (12, Bookwork) with Python 3.11, I get something like:
install.log
When cloning project and doing python3 setup.py build, I get:
running build
running build_py
running build_ext
building 'gtfspy.routing.label' extension
x86_64-linux-gnu-gcc -Wsign-compare -DNDEBUG -g -fwrapv -O2 -Wall -g -fstack-protector-strong -Wformat -Werror=format-security -g -fwrapv -O2 -fPIC -I/usr/include/python3.11 -c gtfspy/routing/label.c -o build/temp.linux-x86_64-cpython-311/gtfspy/routing/label.o
cc1: fatal error: gtfspy/routing/label.c: No such file or directory
compilation terminated.
error: command '/usr/bin/x86_64-linux-gnu-gcc' failed with exit code 1

To workaround this, I need to remove routing extension and only then I can build it. Any idea how to "fix" routing extension to be able to tell people to just pip install it (I have tool that depends on gtfspy)

Hi @stalker314314,
Great to see that there is interest towards using the gtfspy-package.
Unfortunately, I have not had the time in the past (years) to maintain/support this package, and I don't have a quick answer to provide you with. @rkdarst or @jweckstr: Do you have any top-of-mind idea on this?
In any case, if you (@stalker314314) can up with a fix that solves the issue, we're happy to receive a PR so that others can benefit from your work as well.


pinpoint data extraction and processing from hbase

pinpoint version: 2.5.3
hbase version: 1.2.7
I want to access hbase from a server other than pinpoint to query and visualize the server status data collected through the agent in pinpoint.

Can I get a rough summary of what information is stored in each table where the data is stored?
Is a separate character encoding required to use the data additionally?

The data confirmed with the shell is as follows:
\x02\x00\x16foapi#_\x7F\xFF\xFEm_\x0Ej\x7F    column=C:\x03\xF2\x7F\xFC\x00\x0CFOAPI, timestamp=1729277080007, value=\x00\x00\x00\x00\x00\x00\x00\x16

The data in hBase is binary format, so a decoding process is required.
Please refer to the relevant sources.
https://github.com/pinpoint-apm/pinpoint/blob/master/web/src/main/java/com/navercorp/pinpoint/web/dao/hbase/HbaseTraceDaoV2.java#L99


Adding TASKTORRENT_MPI_CHECK macro

Using that macro instead of assert(..) to preserve checks when NDEBUG is defined. The runtime cost is minimal and MPI failures are difficult bugs to solve.

Make sure to make a mention in the README.
Eric Darve
(650) 918 6407, bldg 520, room 125
Professor of Mechanical Engineering and Computational Mathematics
Stanford University
On Mar 10, 2020, 4:46 PM -0700, Léopold Cambier<EMAIL_ADDRESS>wrote:
Merged #4https://github.com/leopoldcambier/tasktorrent/pull/4 into master.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/leopoldcambier/tasktorrent/pull/4?email_source=notifications&email_token=ABDN5VRDF6MKR4UOQW547FDRG3GL7A5CNFSM4LFKDLM2YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOXHDSEVQ#event-3116835414, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ABDN5VX7TYEQBCFTZB3GLHTRG3GL7ANCNFSM4LFKDLMQ.


Added Documentation

The first commit was to remove hw6 from the repository, in case the code is used in an assignment later.
The rest of the commits are documentation to explain what the code is doing, for other people that want to contribute to it.

Thanks for the documentation changes! Could I ask that you stick to the code style already in the repository? I'd be happy to merge the pull request at that point.

Sure. No problem.


XFCE auto mounting stopped working

Plugged in USB3.0 32GB stick formatted with ext4 and it wasn't auto-mounted.
$ ps aux | grep udisk
499       2408  0.0  0.0 297120  3852 ?        Sl   Dec17   0:00 /nix/store/nlpdr7k0mkvja3jpkf5w0wwhfwycbjvd-gvfs-1.14.2/libexec/gvfs-udisks2-volume-monitor
root      2416  0.0  0.0 419396  4200 ?        Ssl  Dec17   0:01 /nix/store/fw8b3hwmm8zqljnhw463rbfzgdcikq4n-udisks-2.1.0/lib/udisks2/udisksd --no-debug
499       2637  0.0  0.0 296992  4288 ?        Sl   Dec18   0:00 /nix/store/nlpdr7k0mkvja3jpkf5w0wwhfwycbjvd-gvfs-1.14.2/libexec/gvfs-udisks2-volume-monitor
root     26317  0.0  0.0 110104  1092 pts/2    S+   14:34   0:00 grep udisk

$ nixos-version
14.02pre35786.9fc5753 (Baboon)

Auto-mounting does not work for me, and this is important to me.
version: 15.06pre64030.e1af50c (Dingo)
My nix config.
$ ps aux | grep udisk
andy      1194  0.0  0.0 370224  6500 ?        Sl   23:38   0:00 /nix/store/ny19ab5sz0x13a809kz61nr5y5wjmbbz-gvfs-1.22.2/libexec/gvfs-udisks2-volume-monitor
root      1197  0.0  0.0 351112  6812 ?        Ssl  23:38   0:00 /nix/store/4vzpbr7fi0p7h3qzg92s5xv4094jc1ql-udisks-2.1.4/lib/udisks2/udisksd --no-debug
andy      1834  0.0  0.0 112716  2272 pts/1    S+   23:43   0:00 grep --color=auto udisk

I followed the steps here: https://nixos.org/wiki/Xfce#Volume_mounting_does_not_work
Still does not work. Thunar has this on stderr when I plug in my android phone over USB:
thunar-volman: Unsupported USB device type "usb".
thunar-volman: Unsupported USB device type "(null)".

I tried this in KDE2 as well as XFCE and was not able to mount the media device.

Can this issue be re-opened? Should I start a new one?

Let's keep the same thread. Do I understand correctly that neither auto-mount nor click-to-mount works for you?

Yes that is correct. And know the device is working because I was able to transfer files by manually mounting with jmtpfs.

(triage) does it work again?

I don't know if I am experiencing the same problem.
I'm using nixos unstable with i3 and pcmanfm and trying udisk2 service for auto mounting usb but the daemon doesn't start automatically for me and manually (without the no debug flag) detects the usb and udisk2 adds a uevents for sdb and sdb1 but doesn't show more information neither load it.

01:19:45.388:[12002]:[DEBUG]: uevent add /sys/devices/pci0000:00/0000:00:1d.0/usb1/1-1/1-1.3/1-1.3:1.0/host4/target4:0:0/4:0:0:0/block/sdb [udiskslinuxprovider.c:902, udisks_linux_provider_handle_uevent()]
01:19:45.436:[12002]:[DEBUG]: uevent add /sys/devices/pci0000:00/0000:00:1d.0/usb1/1-1/1-1.3/1-1.3:1.0/host4/target4:0:0/4:0:0:0/block/sdb/sdb1 [udiskslinuxprovider.c:902, udisks_linux_provider_]

I would be very grateful for any solution.

My issue is that NixOS automounted the other disk drives but is unable to open them because of permissions:
https://i.imgur.com/s9gIItd.png

This happened to me again

Problem solved by rebooting the computer.......


Fix docopt short option "since": -e -> -s.

docs.go
 58     -s --since <date>
 59       Show triggers that have changed their state after the given time.
 60       [default: 7 days ago]
...
154     -e --since <date>    [default: 7 days ago]

thanks


add bypasses

Fixes:

Takes an URL such as: https://anonyviet.com/tieptucdentrangmoi/?url=https%3A%2F%2Fsignup.microsoft.com%2Fsignup%3Fsku%3DEducation and returns and opens https://signup.microsoft.com/signup?sku=Education
added li.web.tr bypass

Closes (in order):

closes #1104
closes #1087

[x] I made sure there are no unnecessary changes in the code;
[ ] Tested on Chromium (Includes Opera, Brave, Vivaldi, Edge, etc);
[ ] Tested on Firefox.

Forgot there aren't new bypasses being accepted, I'll close for now :facepalm: