Feature to pass organization ID into MQTT topic

As discussed in this topic would it be possible to add organization ID into the topic to subscribe only to topics belonging to this organization.
May be something like that?
  # MQTT integration backend.
  [application_server.integration.mqtt]
  # Event topic template.
  event_topic_template="organization/{{ .OrganizationID }}/device/{{ .DevEUI }}/event/{{ .EventType }}"

  # Command topic template.
  command_topic_template="organization/{{ .OrganizationID }}/device/{{ .DevEUI }}/command/{{ .CommandType }}"

and of course still have application if needed
  # MQTT integration backend.
  [application_server.integration.mqtt]
  # Event topic template.
  event_topic_template="organization/{{ .OrganizationID }}/application/{{ .ApplicationID }}/device/{{ .DevEUI }}/event/{{ .EventType }}"

  # Command topic template.
  command_topic_template="organization/{{ .OrganizationID }}/application/{{ .ApplicationID }}/device/{{ .DevEUI }}/command/{{ .CommandType }}"

Anyway, another option could be to have tag or variable to application that will be pass into json, like that we could filter by tags other side, don't know which one is simpler to you.

We can keep the application level topic by default, and support {{ .OrganizationID }} for template.
@brocaar  What do you think?
I want to try it if possible.

I'm planning to do a bit refactoring on this configuration soon, to align it with the topic_prefix, which is used in other configuration parts (ChirpStack v4). Then I will also look at this item.

I suppose the organization ID being referenced here is the tenant ID in v4?
Next to being able to use the tenant ID, it would also be useful to have the following parameters available:

Tenant name
Application name
Device name

in order to be able to control the topic through the configuration. For example, I want to integrate with an existing tenant on my external MQTT broker and I can't control the tenant ID or application ID as these are automatically generated so there's no way for me to publish to the topic of my existing broker tenant.
Has there been any progress on enabling this? If not, any suggestion on the implementation approach so we can consider contributing? We're new with the project and not experienced with Rust.

With that amount of logic, you may be better off writing a slim service to consume MQTT data, convert it as needed for your use case, and then forward it on.

I'm not sure where the additional logic is in what I would like to achieve. I'm just trying to adhere to existing topic naming conventions. All data is available in ChirpStack on the same objects as other fields that can already be used in the topic templates (application ID and device EUID). The template rendering engine has just been hardcoded to only support these fields and not others. Some more flexibility sounds like a good improvement to me.
Specifically in my case, if I could use the application name instead of its ID I would already be helped. The other field suggestions just seemed appropriate to also be supported.
While a topic translation layer could indeed also realize what I want, it would introduce an additional potential point of failure in the chain and would thus require additional monitoring. Improving the flexibility of the topic template rendering engine therefore seems like a nicer solution.

Please note that other than the identifiers (UUID / EUIs), there is no guarantee that these fields are valid for MQTT topic usage.

Indeed. That would be the responsibility of the configurator to ensure that proper governance is in place for admins/provisioning when using these fields in the templates.


User input stop words

Do you think a /n separated file read in by megbot with custom stop words to be added to the stopword array makes sense for me to implement this as?

I shall make user config file.

No need, added a config file with "addedStops". Add your custom stop words there!


chore: Trademarks keeping spelling consistencies same. GitHub and YouTube

What does this PR do?
Improve repo
For resources
Description
Why is this valuable (or not)?
Solves trademarks typos using the text provided by the company
Checklist:

[x] Read our contributing guidelines
[x] Search for duplicates. #5940
[x] Add needed indications (PDF, access notes, under construction)

Follow-up

Check the status of GitHub Actions and resolve any reported warnings!

If a title at the source spells it "Github" we shouldn't be changing it to "GitHub'

If a title at the source spells it "Github" we shouldn't be changing it to "GitHub'

Thanks for comment in a draft. It was replaced by regex. A deep review for all those...
:ok: * How to Collaborate on Github - André J
:no_entry: * Introduction to Git and GitHub - Launch School
:ok: * Introduction to Git and GitHub - Tutorial - Dr. Chris Bourke (PDF)
:ok: * Unix System Programming in OCaml GitHub Repo
:no_entry: * GitHub帮助文档
:no_entry: * Jhan Carlos Silva - Git and Github Tutorial (screencast)
:no_entry: * Git and GitHub - Elzero Web School
:no_entry: * Git and GitHub in One Video (Theory + Practical) | A 2 Z in Bangla - SHAJ.T3CH
:arrows_clockwise: * Machine Learning with Python - YouTube Playlist - Krish Naik (YouTube)
:arrows_clockwise: * Git & GitHub - Sandhika Galih, Web Programming UNPAS (YouTube)
:no_entry: * Cách sử dụng Git trong THỰC TẾ | GitHub - Ông Dev
:no_entry: * Từ gà tới pro Git và GitHub trong 20 phút - Tự học Git siêu tốc - Phạm Huy Hoàng (toidicodedao)
where:

:ok: => needing change
:arrows_clockwise: => title change / rewording
:no_entry: => revert


Packaging: make available on Pypi/conda

Hi,
Would it be possible to make this package available in Pypi and/or conda? This would greatly improve the accessibility of your tool.
We're interested in integrating AfterQC in our QC pipeline, but this depends on the package being available from Pypi or Conda.
Thanks!
M

Thanks for this good suggestion.
How about add it to BioConda first? PyPy is suggested for AfterQC due to performance consideration, but is not compatible with pypi/conda distribution.

No problem, I'd have packaged it for bioconda myself if your suggestion would have been Pypi.
Thanks!
M

Conda recipe:
bioconda/bioconda-recipes#6037

@matthdsm thanks:)

Hi, I installed Bioconda and gave the command conda install afterqc. It generates the error that there is no such package. What should I do to fix it?

Good idea on bioconda.
afterqc is present as v. 0.9.6. Check you have your channels sorted correctly @coolmak32

Hi Colin,
Hope you are doing great man. I wanted to ask you that how to select right
file for the generation of an index using STAR. I am trying to get genome
as well gene annotation data from ensembl but there are numerous files in
every directory. Also as I have told you earlier that my samples had
polyribomes associated with mRNA transcripts mostly, so should I create
index using cDNA data file or whole Genome data file.
Thanks,
Kuldeep Makwana
(Ph.D. candidate regulatory biology)
SR 271,BGES
Cleveland State University.
216-262-9156
On 31 October 2017 at 16:38, Colin Davenport<EMAIL_ADDRESS>wrote:

Good idea on bioconda.
afterqc is present as v. 0.9.6. Check you have your channels sorted
correctly @coolmak32 https://github.com/coolmak32
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/OpenGene/AfterQC/issues/23#issuecomment-340899348,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Afd3lrLg-JjFzAjH3rDmMObGFSPI3rTEks5sx4U3gaJpZM4PN8bJ
.


WIP MacOS / Linux support

I finally got something running:

xdelta detection is kind of borked right now, and I haven't tried packaging this with pyinstaller yet but it "works"!
Currently running np21.exe under wine because I can't figure out how to get the Retroarch version running.
I've also added a Pipfile for deterministically managing dependencies.

I also abstracted out the ndc code into its own package: https://github.com/meunierd/ndcpy/blob/master/ndc/init.py -- will publish once I've written a README.
Getting this across the finish line is probably gonna be slow, I'm at home on pat leave so this is being done in the like 20 minute stints that my son decides to sleep.

Great! I'll be excited to check this out. Looks like a ton of QOL improvements in there.
I was about to ask how to install ndcpy, will that just be a pip install ndcpy thing once you've published it?

It'll be something like:
pip install pipenv
pipenv install
pipenv shell --fancy

ndcpy is now published!

Hmm. Inside or outside the pipenv shell, after doing pip install ndcpy I haven't been able to import it with from ndc import NDC.
ImportError: No module named 'ndc'
I followed the steps here and in ndcpy's readme. Is there anything else you or I need to do?

Have you pulled since I rebased? I forgot to commit it before, it should be
okay now. Let me try from a fresh environment.
On Sat, 11 Nov 2017 at 23:54 Max<EMAIL_ADDRESS>wrote:

Hmm. Inside or outside the pipenv shell, after doing pip install ndcpy I
haven't been able to import it with from ndc import NDC.
ImportError: No module named 'ndc'
I followed the steps here and in ndcpy's readme. Is there anything else
you or I need to do?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/46OkuMen/romtools/pull/3#issuecomment-343713609, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACT2aAhgrdT4vO_4YRZY9VZePaQgfDwks5s1noJgaJpZM4QXCjc
.

I forgot the module in the package, will repackage 😬

OK. It installs fine now, but once it tries to run NDC, it gets into trouble parsing the output.
Traceback (most recent call last):   File "pachy98.py", line 469, in <module>     disks_in_dir = [Disk(f, ndc_dir=bin_dir) for f in image_paths_in_dir]   File "pachy98.py", line 469, in <listcomp>     disks_in_dir = [Disk(f, ndc_dir=bin_dir) for f in image_paths_in_dir]   File "C:\Users\hollowaytape\romtools\disk.py", line 118, in __init__     self.ndc = NDC(self.ndc_path)   File "C:\Users\hollowaytape\.virtualenvs\romtools-5uL6u8ZP\lib\site-packages\ndc\__init__.py", line 65, in __init__     self.__validate_bin_version()   File "C:\Users\hollowaytape\.virtualenvs\romtools-5uL6u8ZP\lib\site-packages\ndc\__init__.py", line 69, in __validate_bin_version     .check_output(self.bin) UnicodeDecodeError: 'utf-8' codec can't decode byte 0x8e in position 76: invalid start byte
I get this in two of my Windows VMs, one using the Shift JIS code page and one using the default English one.
Any idea how to address this? I think I ran into the same thing with check_output() before. I think my non-solution was to avoid using check_output() to parse output at all, but only to run commands and catch CalledProcessErrors...

My guess is that on nix platforms, the output defaults to UTF-8 and on Windows it's probably shift-jis. If you run:
pipenv open ndcpy

it should drop you into an editor for the module and you can replace the decode line with decode('shift-jis'). If that works, I can add platform detection and use the right decoder. I initially tried decoding from shift-jis on my end but ran into similar errors.

Yep! Replacing the three decodes with decode('shift-jis') works.
It looks like the file timestamp formula could be platform-dependent as well:
Traceback (most recent call last):
  File "pachy98.py", line 474, in <module>
    if all([d.find_file(filename) for filename in cfg.all_filenames]):
  File "pachy98.py", line 474, in <listcomp>
    if all([d.find_file(filename) for filename in cfg.all_filenames]):
  File "C:\Users\hollowaytape\code\roms\romtools\disk.py", line 155, in find_file
    results = self.ndc.find_all(self.filename, target_file)
  File "C:\Users\hollowaytape\.virtualenvs\romtools-OgGzTF9v\lib\site-packages\ndc\__init__.py", line 126, in find_all
    return [self.__parse(line) for line in self.__run(cmd)]
  File "C:\Users\hollowaytape\.virtualenvs\romtools-OgGzTF9v\lib\site-packages\ndc\__init__.py", line 126, in <listcomp>
    return [self.__parse(line) for line in self.__run(cmd)]
  File "C:\Users\hollowaytape\.virtualenvs\romtools-OgGzTF9v\lib\site-packages\ndc\__init__.py", line 90, in __parse
    args[-1] = datetime.strptime(args[-1], self.TIMESTAMP_FMT)
  File "c:\python35\Lib\_strptime.py", line 510, in _strptime_datetime
    tt, fraction = _strptime(data_string, format)
  File "c:\python35\Lib\_strptime.py", line 343, in _strptime
    (data_string, format))
ValueError: time data '1990/03/21 18:21:54' does not match format '%a %b %d %H:%M:%S %Y'

Which patch/disk(s) are you testing with? I'll try to repro that.
On Mon, 13 Nov 2017 at 20:19 Max<EMAIL_ADDRESS>wrote:

Yep! Replacing the three decodes with decode('shift-jis') works.
It looks like the file timestamp formula could be platform-dependent as
well:
Traceback (most recent call last):
File "pachy98.py", line 474, in
if all([d.find_file(filename) for filename in cfg.all_filenames]):
File "pachy98.py", line 474, in
if all([d.find_file(filename) for filename in cfg.all_filenames]):
File "C:\Users\hollowaytape\code\roms\romtools\disk.py", line 155, in find_file
results = self.ndc.find_all(self.filename, target_file)
File "C:\Users\hollowaytape.virtualenvs\romtools-OgGzTF9v\lib\site-packages\ndc_init_.py", line 126, in find_all
return [self.__parse(line) for line in self._run(cmd)]
File "C:\Users\hollowaytape.virtualenvs\romtools-OgGzTF9v\lib\site-packages\ndc_init.py", line 126, in
return [self.__parse(line) for line in self._run(cmd)]
File "C:\Users\hollowaytape.virtualenvs\romtools-OgGzTF9v\lib\site-packages\ndc_init.py", line 90, in __parse
args[-1] = datetime.strptime(args[-1], self.TIMESTAMP_FMT)
File "c:\python35\Lib_strptime.py", line 510, in _strptime_datetime
tt, fraction = _strptime(data_string, format)
File "c:\python35\Lib_strptime.py", line 343, in _strptime
(data_string, format))
ValueError: time data '1990/03/21 18:21:54' does not match format '%a %b %d %H:%M:%S %Y'
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/46OkuMen/romtools/pull/3#issuecomment-344114082, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AACT2ZEOEZXlRSHO2xrHhpWbI8ARhir_ks5s2OqigaJpZM4QXCjc
.

I'm testing with 46 Okunen Monogatari - The Sinkaron (J) A user.FDI, though I'll try some others and see what happens...

pipenv install again after pulling and see if this unpublished ndcpy works

Pulled and pipenv installed again, and it breaks in the same decode() spot as before. It looks like the latest ndcpy is unchanged except for the version number?

If you pipenv open ndc do you see an import platform? Maybe the platform detection isn't working and it's still hitting the unix codepath. If it's identical to the old module, maybe just paste in the content of this file

Ah, figured it out. There's still one decode() in a Windows branch left, and I opened a PR in the windows-support branch of ndcpy which fixes it. Just patched my copy of E.V.O. with no issues!
I'll continue testing tomorrow to make sure ndcpy works in my current project.

Good catch! I refactored things to get down to a single branch, and updated the pipenv. If everything's good on your end, I'll package a 0.2.0 release. 👍

Alright! That works, you can package it up.
One thing I found was that ndcpy can't delete or insert files into a disk's subdirectories. I can take a closer look at that tonight and see what I can implement.


Show public campaigns as well as opportunities in the the calendar

This updates the calendar at https://www.bikebrigade.ca/deliveries to show public Campaigns as well as Opportunities.
The idea is that dispatchers will no longer need to update the opportunities with signup links - any public campaigns will just show up on https://www.bikebrigade.ca/deliveries with the correct link within the portal.
We're demoing the portal to dispatchers on Tuesday so I'm pushing this now, but there are two outstanding tasks for later:

[x] Some programs we hide the pickup address for since it's a residential unit (Maggies is one I believe). We should move that flag from Opportunities to Programs so we can maintain that feature.
[x] Add some copy / prevent dispatchers from updating opportunities for campaigns that are launched with the portal (e.g. the program is public)

@teesloane I fixed this issue (looks like we had it in prod too 🙃 ) and did the testing you were trying to do!

I added some alerting when editing opportunities:
When you go to the opportunity page:

If you try to edit an opportunity for a public campaign:

This is great! Small suggestion - I would replace Alert with Warning if it takes less than 5 min

How did you know that this is in 20 different files 😁 https://github.com/bikebrigade/dispatch/pull/330/commits/67c73ac6e7e29c98b6c1b181bf960a0f74b65561
anyways will do :)


src: direct serialization of Function metadata as func.yaml

Changes
:broom: direct serialization of Function metadata as func.yaml
This PR recognizes the logical equivalence of the Function and Config objects, merging the two and in so doing reduces some complexity, and frees up the concept of config for use by non-source-controlled function-local/user-local settings and transient state.
/kind cleanup
Details
The Config object was created after the initial Function draft.  Its purpose was to enable manipulation of metadata about a Function.  This separation between the Function and Config objects was partly to help keep the update minimally invasive, and possibly to draw a distinction between lifecycle functions such as "build" and "deploy" from metadata accessors such as "envs" and "labels".   The Config object and its accessors have proved to be stable, well developed and useful.  The distinction now needs to be removed.
The impetus for making this change now is that forthcoming features require the more idiomatic usage of the term "config".  To summarize, "config" denotes aspects of the system which are configurable, but are not part of the Function itself.  This includes things like installed repositories, user settings (preferred language/repository), build stamps/hashes, running function PIDs, etc.  Taking the venerable git as an example: upon checking out a repository, the items configurable via the config subcommand are not included, but are rather populated based off local global presets or user-initiated action.  They are either repo-local or user-local.  They are not actually part of the repository insofar as being source controlled themselves.  This is the purpose of "dotfiles".  In the very-similar case of Functions, this currently includes Repositories, but is soon to be expanded to include metadata about settings such as preferred Function language, and state to enable func invoke, etc.
Looking from the top down, we can understand the necessity of this change by realizing that func.yaml is the serialized form of Function metadata, meaning that (when combined with source code), it is the Function itself.  Support for this position is made clear by the existence of the following mapping:
func fromConfig(c Config) (f Function) {
	return Function{
		Name:            c.Name,
		Namespace:       c.Namespace,
		Runtime:         c.Runtime,
		Image:           c.Image,
		ImageDigest:     c.ImageDigest,
		Builder:         c.Builder,
		Builders:        c.Builders,
		Buildpacks:      c.Buildpacks,
		HealthEndpoints: c.HealthEndpoints,
		Volumes:         c.Volumes,
		BuildEnvs:       c.BuildEnvs,
		Envs:            c.Envs,
		Annotations:     c.Annotations,
		Options:         c.Options,
		Labels:          c.Labels,
	}
}

// toConfig serializes a Function to a config object.
func toConfig(f Function) Config {
	return Config{
		Name:            f.Name,
		Namespace:       f.Namespace,
		Runtime:         f.Runtime,
		Image:           f.Image,
		ImageDigest:     f.ImageDigest,
		Builder:         f.Builder,
		Builders:        f.Builders,
		Buildpacks:      f.Buildpacks,
		HealthEndpoints: f.HealthEndpoints,
		Volumes:         f.Volumes,
		BuildEnvs:       f.BuildEnvs,
		Envs:            f.Envs,
		Annotations:     f.Annotations,
		Options:         f.Options,
		Labels:          f.Labels,
	}
}

As can be seen above, since almost every field of a "Config" is mapped directly to a "Function" instance, it is a strong indication they are actually the same concept.
It is worth discussing a potential point in opposition to this change:  it impies the need to flatten our command structure.  Previously, at the root level were only lifecycle commands such as build and deploy.  Let's call these "Verbs".  Metadata-manipulation subcommands were held beneath config such as envs and labels .  Let's call these "Nouns".
At first blush it may seem wrong or confusing to commingle these as root subcommands.   The problem here, however, is not that this is linguistically incorrect, but rather that our chosen CLI library does not to my knowledge support grouping in the help text.   Again taking 'git' as our example:  at the root are both verbs such as git clone and git init, as well as nouns such as git branch and git tag.  The distinction comes in the form of properly formatted help text, which groups these subcommands under appropriate headings (this can be seen by running the command git with no further arguments).
If the team agrees, we could emulate this user experience by providing a somewhat customized help text which groups our commands under appropriate headings as well.  It will require a bit of customization, however, as it does not appear that the Cobra CLI library supports this directly.  This change should also move the repositories subcommand in the opposite direction:  from the root to be a subcommand of config, because these settings fit the description of being non-source-controled, local system configuration.
To enumerate the set of challenges which necessitate this change now:

The repositories management subcommand is not modifying the Function metadata, nor are its changes source controlled, and therefore should be func config repos ....  However this is not currently possible while the config subcommand represents the Function's actual metadata.
The transient, local-only state of the hash of the container created on the last build should not be part of the function's serialized state, but it currently has nowhere to go except in func.yaml.  Creating a config to encapsulate transient, local-only state like this is needed in order to avoid triggering an unstated git commit (modified func.yaml) when only building a function locally.
In addition to the transient build hash explained in #2, it is an upcoming need to track metadata to support the func invoke subcommand.  This state is part of config, but not part of the function's serialized state, so it is dependent on this PRs change as well.
In order to provide a configurable default function language (a request identified during user interviews), it is necessary to have a git-like config with the ability to add a --global option, keeping the (again, local-only non-source-controlled configuration) in either ./.func/config.yaml or ~/.config/func/config.yaml.
The forthcoming on-cluster builds will require being able to configure settings for building a function (func config build ...) which are local-only, and not part of the source-controlled Function metadata.  This is another example of a setting that should go in a config file, but not in func.yaml.

/lgtm

Thanks for the lgtm @zroubalik .  I made a few changes since then, and would greatly appreciate a re-stamp if you have time,

/unhold
/lgtm


validation loosing focus bug

if you click a bunch of times on a drop down, and sometimes click out of
the dropdown (while its showing) the error you will get is the .call on
the rule will be null as the dropdown is the item that no longer has
focus and therefor you will get a javascript error

Sorry this is my first ever bug fix, so I am trying to figure out how to make the CI build pass so I have to figure out how to build this thing locally (which I dont know how to yet as I havent used npm and other stuff)


loading issues

My application is taking long to show the playlist.
did l make some mistakes when doing the adaptive authentication part. am stack

When filing an issue, please provide some or all of the information requested for in the issue template.
Given that there's no actionable information in this issue, I'll be closing this.
Thank you


Empty metadata time values

When using inara to create a draft, published_at metadata will have no value, and sometimes submitted_at will also be empty.
Now when any of these values are null or an empty string this error is happening in the time filter:
[INFO] Running filter /usr/local/share/openjournals/data/filters/time.lua
Error running filter /usr/local/share/openjournals/data/filters/time.lua:
/usr/local/share/openjournals/data/filters/time.lua:8: bad argument #1 to 'for iterator' (table expected, got string)
stack traceback:
	[C]: in function 'next'
	/usr/local/share/openjournals/data/filters/time.lua:8: in upvalue 'copy_string_values'
	/usr/local/share/openjournals/data/filters/time.lua:21: in function 'Meta'

Inara should accept nil published_at/submitted_at values and compile the paper with an empty value there.

Made a view changes that should fix this. We still require the values to be set, but now provide default values.


Sequential validation after onBlur on required fields

After the 'OnBlur' event in any mandatory field ... all other fields with any validation are triggered as invalid.
One of my approaches was to customize my fields for these types of forms...
...
onBlur={() => helpers.setTouched(true)}
...

But I really don't know what the best approach would be!.
https://codesandbox.io/s/polished-dew-8ns0u?file=/src/pages/index.tsx
Such an exception occurs only in subsequent steps.
Thanks!!

Any news regarding this one? :)

@bmvantunes i got a solution by resetting the "touched" to go to the next step...
https://codesandbox.io/s/polished-dew-8ns0u?file=/src/pages/index.tsx:4143-4176


Minimum required Node.js version

package.json has:
https://github.com/hashgraph/hedera-sdk-js/blob/7a72ed9168d86a97be6ac371031582357f07d777/package.json#L47-L49
I'm relatively sure that's not correct, i.e. I doubt that dependencies work on that.
And anyway, is there any value in supporting unmaintained Node.js versions that can have known security issues?
CI seems to use 10.x,12.x,13.x (btw: should be updated to 14.x perhaps?):
https://github.com/hashgraph/hedera-sdk-js/blob/7a72ed9168d86a97be6ac371031582357f07d777/.github/workflows/node.yml#L17-L19
Perhaps the required version in package.json should be updated to what is actually being tested?

I'm relatively sure that's not correct, i.e. I doubt that dependencies work on that.
And anyway, is there any value in supporting unmaintained Node.js versions that can have known security issues?

When we first wrote the SDK, that was the target. You're right though, that it definitely doesn't work now.

gprc-js has "node": "^8.13.0 || >=10.10.0" in engines but recommends 12.x.
Refs: #261.

Seems to be fixed in master? Thanks.
I'll go ahead and close this unless this is needed for some reason.


Refactor codebase to async/await - m365 spo contenttype and m365 spo contenttypehub

Closes #5022

🙏, we'll have a 👀 ASAP!

Thank you. Manually merged


Fix vague tests for "Convert JSON Data to HTML" challenge

Pre-Submission Checklist

[x] Your pull request targets the staging branch of freeCodeCamp.
[x] Branch starts with either fix/, feature/, or translate/ (e.g. fix/signin-issue)
[x] You have only one commit (if not, squash them into one commit).
[x] All new and existing tests pass the command npm test. Use git commit --amend to amend any fixes.

Type of Change

[x] Small bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would change existing functionality)
[ ] Add new translation (feature adding new translations)

Checklist:

[x] Tested changes locally.
[x] Closes currently open issue (replace XXXX with an issue no):

Relates #15726
Description

I added two more regex assertions.
Firstly, data must be stored into the previously declared variable html at some point.
Secondly, it checks whether an opening and closing strong tag can be found with some content in it

I am marking this relates; as we would still be needing the fix on production.


Clarify weekends usage in example_search.yaml

Description
Clarify comment for --weekends flag in example yaml config
The --weekends flag is intended to mean "weekends only" and will exclude weekdays from the search
Has This Been Tested?
No. But it's just a minor clarification to the documentation.
Checklist:

[x] I've read the contributing guidelines of this project
[ ] I've installed and used .pre_commit on all my code
[x] I have documented my code, particularly in hard-to-understand areas
[x] I have made any necessary corresponding changes to the documentation

Hello! Thank you for such an amazing tool!
For some backstory, this PR is mostly a result of my inability to read :)  So please feel free to close if you don't think it's helpful.
I was attempting to use the yaml config to configure my query for a few sites I was interested. I started mostly by looking through the options in example_search.yaml and tweaking and tuning for my needs.
For whatever reason, when I got to weekends, I assumed it meant "include weekends too".  Weekends at popular sites are certainly much harder to grab, so for whatever reason, I was assuming the defaults excluded these and the defaults would funnel you towards settings more likely to yield success.
Obviously that isn't the case.
The command line help (which I admittedly didn't read) makes this quite clear:
https://github.com/juftin/camply/blob/d9f3e9a0761d9370841c1e64fcb373d3d4a72491/camply/cli.py#L371
So I figured it might be worth adding similar text to the comment in the example yaml to help avoid confusion for folks (like me) who assume intent instead taking the time to properly read the help documentation.
Anyways, thank you again!

Another approach I considered, was seeing if you'd be willing to entertain a renaming of --weekends to --weekends-only.
Internally, it looks like we use weekends_only ... so that naming at least seemed consistent and unambiguous.
But it doesn't look like Click currently has support for deprecating a parameter easily:
https://github.com/pallets/click/issues/2263
However, they do have it added to a milestone for a future release. So if they add formal support for deprecating options and you'd be willing to entertain the change, I could certainly work to offer up that approach in a future PR.

This looks great - thanks for the documentation improvement!
As far as changing --weekends to --weekends-only, I agree this makes the intention more clear - but I'd like to do everything I can to avoid breaking changes as the CLI API has been incredibly stable.
Introducing a command line alias could be kinda neat, I don't think I'd want to issue a deprecation warning though. An cool solution might be something like the below which would just clarify in the documentation that you can also use the --weekends-only alias too.
@click.option("--weekends", "--weekends-only", "weekends", is_flag=True, show_default=True, default=False)
def cli(weekends: bool) -> None:
    ...

Introducing too many options might be confusing as a user though and I don't want to get too fancy with Click so I'd probably lean towards leaving the CLI as is.
Again, thanks for the help here. This change will make things clearer for lots of people using YAML searches.

:tada: This PR is included in version 0.32.3 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_faosnx9mvf/clone/recipe/meta.yaml:15: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_faosnx9mvf/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_faosnx9mvf/clone/recipe/meta.yaml:26: missing_pip_check: For pypi packages, pip check should be present in the test commands
WARNING: /tmp/abs_faosnx9mvf/clone/recipe/meta.yaml:35: missing_description: The recipe is missing a description
Errors were found


getopts rewrite of AAXtoMP3, merges, --targetdir, Readme updates.

Comments
Complete rewrite of options processing.   Much more flexible for future changes.  Now supports both short and long option formats.  Also options are now unordered.
Changes:

Merged in the -aac -opus encodings
Created a --targetdir option to all for output to go to another directory.
Lots of Readme.md updates
Lots of variable validation code added
Changed the .authcode handling again.  It's now a option as it is optional on the command line.  I've also added support for local and home directory .authcode support.

Outstanding Issues:
flac file does not want to split into multiple parts in my testing so I adjusted the option to force single for the time being.

Comments
Primary changes revolve around DEBUG output.  We now have a functional -d --debug option.  The effort is to enable others to enhance the code a little more cleanly.
Changed

added debug_file function now dumps the contents of a text file to output
modified debug function to only output if -d or --debug flag is set.
added log function to replace old debug function
moved variable computation in ffmpeg commands to resolve in variable prior to ffmpeg.
removed AAXtoMP3 as it is redundant now that --aac exists
added variou comments in code.
updated Readme.md

Issues

--flac option will only output a single file.  The split file throughs an error.
Specific container options are something that should be explored.

@upuv Can you do a rebase? thank you.

Apologize
Sorry about that.  My bad. Sorry for the lack of rebase.  <hand slap>
Comments.
I recommend testing on your platform with the -d and and short form options for most things.  Please refer to the Readme.md for details.  The -d or --debug options are extremely verbose.
Also note I have create a code structure the makes options un ordered.  Removing the previous MUST BE IN ORDER requirement.

I'll test it in the following days, Thanks again.

Changes
I added m4b support to the mp3 script
I removed the m4b script.
I updated Readme.md file.

Great, I was about to remove the m4b script myself, following the upstream update.
I'll take more time to test thoroughly.

hey @upuv , the intention of my fork was to add artwork cover to the m4b files converted, for which I did:
brew install mp4v2

then invoke it at https://github.com/jostyee/AAXtoM4B/blob/master/AAXtoM4B#L108
Since you removed this line, do you have a better alternative?

Good Spot that was a genuine regression on my part.
I will fix this and add check code to make sure the command is present.  I will not fail mp4 creation I will only WARN if command not present.

Changes

m4a and m4b now have cover art added to output file.  Fixing a regresion I introduced.
m4a and m4b better documented in Readme.
cover art inclusion will only occur if mp4art is installed.  If not the process will still succeed just with out the cover added to the output.
added install instructions to code for mp4art on MacOS and Ubuntu.  Probably should move this to Readme.

Changes

Added --chaptered option
Chaptered m4a and m4b support added.  Added cover file to each chapter
Fixed --target_dir bug
Modified log out.   Chaptering output now human readable
Modified chaptered file naming removed excess whitespace
Updated README.md

Tag

upuv_v2.1.0

Bravo, Thank you @upuv for contributing.


[CHORE]: script seed dev test

CHORE
Objective:

update script to run on dev/test db's

renaming branch to match Jira


⚠ Processors has degraded performance

In d6e850f, Processors ($STATUS_URL) experienced degraded performance:

HTTP code: 200
Response time: 53 ms

Resolved: Processors performance has improved in 5986b80.


chore(infra-jobs): remove githubCheckNames

This PR removes the GitHub check names, now useless since the use of https://github.com/jenkinsci/github-scm-trait-notification-context-plugin:

Ref:

https://github.com/jenkins-infra/helpdesk/issues/2778#issuecomment-1959565465

FWIW those status checks names are incredible long =/

I'd suggest removing "jenkins" prefix (does not add meaningful information), the job name (as there isn't any case where two jobs are running in the same folder AFAIK, and as it can easily be read from the URL on link hover), and maybe the "pr-merge/branch" suffix.
From:
jenkins/infra.ci.jenkins.io/kubernetes-jobs/kubernetes-management/pr-merge
jenkins/infra.ci.jenkins.io/updatecli/kubernetes-management/pr-merge

To:
infra.ci.jenkins.io/kubernetes-jobs
infra.ci.jenkins.io/updatecli

WDYT @timja @dduportal @smerle33?

I would think about what is important.
to me that's the check name and it should be first.
There's no need to use / to make it harder to read.
to me its:
kubernetes-management (infra.ci) or even jenkins or build for that folder.
do you even need the jenkins instance for updatecli?
updatecli or updatecli (infra.ci)
it'll be in the actor's name of who creates the check anyway

Agree, the thing is there are some constraints to get it automatically named AFAIU cf @dduportal's explanations at https://github.com/jenkins-infra/helm-charts/pull/1081


Add Amazon Linux support

Signed-off-by: Nathan Dines<EMAIL_ADDRESS>Description
This change introduces support for the Amazon Linux platform family since it diverged from the rhel platform family as of Chef/Ohai 13
Issues Resolved

N/A

Check List

[ ] All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
[ ] New functionality includes testing.
[x] New functionality has been documented in the README if applicable
[x] All commits have been signed for the Developer Certificate of Origin. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

Thanks!


design system: Convert to CSS variables

Description
To change any spacing/color/etc, it's currently needed to go into the component themselves. With the introduction of the templates/themes, the only viable way would be to have CSS variables.
Note: This might be broken down into several PRs/Issues.
Example:
.bubble {
  border: 1px solid var(--light-grey);
}

should become
.bubble {
  border: var(--bubble-border);
}

with
:root {
  /* Main colors first*/
  --color-700: rgb(94, 90, 94);

  --bubble-border: 1px solid var(--color-700);
}

(Color variables naming will change with this issue)
Acceptance criteria

[ ] Every hardcoded values that could be part of a design system (borders, gaps, padding, colors, etc.. almost everything minus the alignments like flex) should be converted to CSS variables

I can work on this starting on Monday.


Added Benue State University

Added Benue State University Makurdi by Paul-Ben Ajene

@nebluap First stage of verification passed, official domain is bsum.edu.ng as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

Hi Philip,
Thanks for your email, looking forward to a favorable final decision.
Regards,
Paul-Ben Ajene
Benue State University
Directorate of ICT
Principal Network Administrator
Disclaimer: This email is my official mail and messages are only meant for whom they are addressed to or copied, content shall not be used in any legal way neither should i accept any claim based on this message except otherwise confirmed by me.

From: Philip Torchinsky @.>
Sent: Tuesday, May 9, 2023 10:01 AM
To: JetBrains/swot @.>
Cc: ajene @.>; Mention @.>
Subject: Re: [JetBrains/swot] Added Benue State University (PR #17459)
@nebluaphttps://github.com/nebluap First stage of verification passed, official domain is bsum.edu.nghttps://bsum.edu.ng as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.
—
Reply to this email directly, view it on GitHubhttps://github.com/JetBrains/swot/pull/17459#issuecomment-1539337405, or unsubscribehttps://github.com/notifications/unsubscribe-auth/A7V45D2EECKHIT2EGEY34ATXFIBVFANCNFSM6AAAAAAXZ25K2E.
You are receiving this because you were mentioned.Message ID: @.***>

@nebluap Thank you for your patience while waiting for a review and processing your request. Pull request merged.


70-test_tls13hrr.t: Use P-521 instead of X25519 for invalid group test

Otherwise this testcase won't work with no-ecx.
This fixes the no-ecx run-checker CI on push -> urgent.

Ok with urgent

Why not just test for no-ecx - That would be clearer..

I prefer @t8m's way. It means the test still gets run in a no-ecx build.

Merged to master branch. Thank you for the reviews.


Updated documentation regarding not implemented information_schema co…

…lumns

Codecov Report

Merging #5682 into master will decrease coverage by 0.17%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #5682      +/-   ##
============================================
- Coverage        83%   82.83%   -0.18%
+ Complexity    11834    11797      -37
============================================
  Files          1391     1391
  Lines         45508    45508
  Branches       5627     5627
============================================
- Hits          37776    37696      -80
- Misses         5123     5177      +54
- Partials       2609     2635      +26

Impacted Files
Coverage Δ
Complexity Δ

...arch/indices/recovery/BlobStartPrefixResponse.java
66.66% <0%> (-33.34%)
3% <0%> (-2%)

...in/java/io/crate/lucene/match/CrateRegexQuery.java
47.82% <0%> (-26.09%)
5% <0%> (-3%)

...a/io/crate/exceptions/ContextMissingException.java
81.81% <0%> (-18.19%)
1% <0%> (-1%)

...o/crate/analyze/SafeExpressionToStringVisitor.java
83.33% <0%> (-16.67%)
7% <0%> (-1%)

...in/java/io/crate/data/CollectingBatchConsumer.java
69.23% <0%> (-15.39%)
4% <0%> (-1%)

...ate/planner/node/dql/PlanWithFetchDescription.java
80% <0%> (-10%)
12% <0%> (-2%)

...va/io/crate/planner/projection/EvalProjection.java
54.16% <0%> (-8.34%)
9% <0%> (-1%)

...peration/collect/BatchIteratorCollectorBridge.java
65.78% <0%> (-7.9%)
2% <0%> (ø)

core/src/main/java/io/crate/types/ArrayType.java
73.84% <0%> (-7.7%)
24% <0%> (-2%)

.../java/io/crate/blob/TransportDeleteBlobAction.java
75% <0%> (-5%)
4% <0%> (-1%)

... and 25 more

Continue to review full report at Codecov.

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


support email mfa

Environment information
System:
  OS: macOS 14.7
  CPU: (10) arm64 Apple M1 Pro
  Memory: 72.84 MB / 32.00 GB
  Shell: /opt/homebrew/bin/fish
Binaries:
  Node: 22.8.0 - ~/.local/state/fnm_multishells/43849_1730131892294/bin/node
  Yarn: undefined - undefined
  npm: 10.8.2 - ~/.local/state/fnm_multishells/43849_1730131892294/bin/npm
  pnpm: 9.12.0 - ~/.local/state/fnm_multishells/43849_1730131892294/bin/pnpm
NPM Packages:
  @aws-amplify/auth-construct: Not Found
  @aws-amplify/backend: 1.5.1
  @aws-amplify/backend-auth: Not Found
  @aws-amplify/backend-cli: 1.3.0
  @aws-amplify/backend-data: Not Found
  @aws-amplify/backend-deployer: Not Found
  @aws-amplify/backend-function: Not Found
  @aws-amplify/backend-output-schemas: Not Found
  @aws-amplify/backend-output-storage: Not Found
  @aws-amplify/backend-secret: Not Found
  @aws-amplify/backend-storage: Not Found
  @aws-amplify/cli-core: Not Found
  @aws-amplify/client-config: Not Found
  @aws-amplify/deployed-backend-client: Not Found
  @aws-amplify/form-generator: Not Found
  @aws-amplify/model-generator: Not Found
  @aws-amplify/platform-core: Not Found
  @aws-amplify/plugin-types: Not Found
  @aws-amplify/sandbox: Not Found
  @aws-amplify/schema-generator: Not Found
  aws-amplify: 6.6.7
  aws-cdk: 2.164.1
  aws-cdk-lib: 2.164.1
  typescript: 5.6.3
AWS environment variables:
  AWS_PROFILE = josef
  AWS_REGION = us-east-1
  AWS_STS_REGIONAL_ENDPOINTS = regional
  AWS_NODEJS_CONNECTION_REUSE_ENABLED = 1
  AWS_SDK_LOAD_CONFIG = 1
No CDK environment variables

Describe the feature
Amazon Cognito announced support for email-based multifactor with one-time passwords
// amplify/auth/resource.ts
import { defineAuth } from "@aws-amplify/backend"

export const auth = defineAuth({
  loginWith: {
    email: true,
  },
  multifactor: {
    mode: "REQUIRED",
    email: true,
    // or
    // email: {
    //   subject: "verify yourself",
    //   message: (code) => `here's your code: ${code}`
    // }
  },
})

Use case
n/a

Is there any documentation available for Amplify Gen1 customers?
We would like to enable Email OTP in our Amplify Gen1 App possibly by using an override?
Also will this feature be supported in Amplify Gen1 or just Gen2 planned?


I'm going to close this soon :3

Look at my next pull request :+1: It's a bit buggy, but still works!
https://gyazo.com/f2f800be92e1e584066f8045d49c6d10

Nice! Very clean and beautiful, keep going :)

Fixed the long text bug, every 10 seconds it will clean the console.
https://gyazo.com/8513ba6573acd27439460eff9656130b
Adding the pull request now :3

Amazing! :)

Jeej, Cr4xy is here :D cant wait to test it^^

@Cr4xy Can I also remove node.exe or is it needed? >.>

remove it, remove it, remove it! :D what do you think @Cr4xy ? Lol

#51!!!


Bug: published render version is a static image instead of being a movie

Is there an existing issue for this?

[X] I have searched the existing issues

Current Behavior:
the published render version is a static image instead of being a movie
can test on asset cube task shading in TEST_OP2PUB_23_7
https://fixstudio.ftrackapp.com/#slideEntityId=4c3f8460-dd09-11ed-baa1-06178ac4037d&slideEntityType=task&view=tasks&itemId=projects&entityId=acb580f4-bf93-4a84-b10c-e14e8e666947&entityType=show
the  color space is wrong also
Expected Behavior:

this but as a movie (this is a turnaround)
Version
3.15.9-nightly.1
What platform you are running OpenPype on?
Linux / Centos
Steps To Reproduce:
can test on asset cube task shading in TEST_OP2PUB_23_7
Are there any labels you wish to add?

[X] I have added the relevant labels to the bug report.

Relevant log output:
No response
Additional context:
No response

Hello Olivier, I had investigated your issue, in order to create movie in ftrack, you should use in Maya, 'Create>Review', it will be create a Review Node where you put your camera into (Drag And Drop the camera node), at publishing time, you should see something like that.

Once it will be published, it will create 2 jobs in Deadline, if you met some errors with the Maya Jobs, in Deadline you can remove in the environment Variable: OPENPYPE_VERSION in the 'environment' tabs of the 'Job Properties' (Job > Right Click > Modify Job Properties) and set to 'False' the 'Enable Strict Error Checking' field in Maya Batch Settings.
You can also met some errors with the Publish Job, you can remov the environement Variable: OPENPYPE_VERSION, IS_TEST in the 'environment' tabs and it will works.

Thanks for the investigations
But i dont understand, is it instructions to make a playblast review or to have a review version of the exr ?
I m not in front of my computer right now will test tomorrow

Oh, i already know how to make opengl review with openpype

@OlivierOsotimehin Is that what you’re waiting for? https://fixstudio.ftrackapp.com/player/tempdata/9f2bd456-7198-4962-9280-90a70aa36888

We are nearly there !
What's missing :
the static frame publish is straight linear :

the movie playing is not the right color space (probably simple srgb/rec709 instead of odt srgb (the exr is acescg)) :

the right colorspace of the image is visible in the first post of the thread (Expected behavior)

i put it again here :

The colorspace bug will be taken care of in a new ticket : #466


My Profile section displays all reserved rockets and space missions.

1- My Profile section displays all reserved rockets and space missions.

Hello @akbarsazish
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations! 🎉

Highlights

The app is working as expected :heavy_check_mark:

Status: APPROVED :fire:  :fire:


Doesn't work with Bootswatch and LitElement

Hi.
I am trying to use Bootstrap styles overrided by Bootswatch styles in LitElement
import {html, LitElement} from "lit";
import {customElement, property} from 'lit/decorators.js';
import styles from '../../../../../../../example/client/s.scss';

@customElement('sign-up-view')
export class LitSignUpView extends LitElement
{
    @property()
    private name?: string = 'World';

    public static override styles = [
        styles
    ];

....

s.scss
@import "_variables.scss";
@import "bootstrap/scss/bootstrap.scss";
@import "_bootswatch.scss";

build.ts
...
build({
                plugins: [
                    NodeModulesPolyfills(),
                    GlobalsPolyfills({process: true, buffer: true}),
                    // eslint-disable-next-line @typescript-eslint/ban-ts-comment
                    // @ts-ignore
                    sassPlugin({type: "lit-css", cssImports: true})
                ],
...

Bootstrap styles successfully loaded and applied, however they are not overrided by Bootswatch styles.
In console there is a warning:
@import rules are not allowed here. See https://github.com/WICG/construct-stylesheets/issues/119#issuecomment-588352418.

Can you help me please, what I am doing wrong?
Thanks

Clearly an @import has been found in  the resulting css text.
That should be treated as a css @import but constructable stylesheets don't allow those as the message is explaining.
unfortunately I am not familiar with Bootswatch to know where that @import might come from to help...
Such an import would mean that a part of the css was meant to be shared but constructable stylesheet cannot share this way it should be the developer that lists all the shared styles (in the array you pass to styles) to apply to the shadow dom.
I have to close this issue because I don't see anything I can do to help


[20.03] ocaml-ng.ocamlPackages_4_10.ocaml: 4.10.0+β2 → 4.10.0

Motivation for this change
Bring the latest released version of OCaml to the latest release of nixpkgs.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

@GrahamcOfBorg build ocaml-ng.ocamlPackages_latest.utop


Use Height instead of uint64 for latestHeight in ClientState

The tendermint light client proto file to be used on icon has a field, ClientState which uses int64 for the latestHeight. It should be using the height as defined in this proto file.
https://github.com/cosmos/ibc-go/blob/main/proto/ibc/core/client/v1/client.proto

We did find a easy workaround on the proto generated file to avoid getters, which would solve the issues we were having with the interface in the relay. So, closing this issue.


perimeter of rectangle java

Why
Closes #435

https://codinasion.web.app/programme/perimeter-of-rectangle/


Add ability to get state file from webserver using GET /status

Implements #106

Is there something else I could to do facilitate this PR?

Thanks for the pull request, I'll try to review it soon.  Last weekend + a few days around it we were celebrating midsummer here in Finland and the PGHoard maintainers were busy with festivities in the country side.

@melor, Thank you for your suggestions.  I have implemented them.  I had explicitly called close based on a stack exchange article that implied that doing so helps with debugging (see comments here: http://stackoverflow.com/questions/8369219/how-do-i-read-a-text-file-into-a-string-variable-in-python/8369345#8369345).  I'm happy to follow the coding conventions of the project.  I hardly know enough to be dangerous much else develop good coding practices on my own.

Updated error handling and tests a bit, squashed the commits and pushed.  Thanks!


AppMaps from remote recordings should be stored in tmp/appmap by default instead of the project's root directory

When stopping a remote recording in intellij, the prompt for saving the appmap suggests saving in the root directory:

Instead, it should suggest saving it in the tmp/appmap directory:

Remarks by the team:

The first choice is to use the appmap_dir from the appmap.yml file
Second choice is tmp/appmap
In either case, create or use the subdirectory “remote”


⚠ ESPN has degraded performance

In c1acfa7, ESPN (https://espn.com) experienced degraded performance:

HTTP code: 200
Response time: 612 ms

Resolved: ESPN performance has improved in f03f741.


Support displaying multiple announcements with NewAnnouncementBanner

Closes #140

@K-Phoen thanks for doing this work, is there a particular cadence you plan to do releases with this? I would like to use this feature but I think the latest release is from March


Refactor get_api to expose intermediate steps

Give access to user to re analyze data from a previous question

Refactor get_api to expose intermediate steps


[Enhancement] Cost based on location

Any chance that cost can be based on location instead of generalized?

(Please and thank you)


Add field conversion for Patient - name

Hey Paul, the name field is added to the model class now!


Update "Manage Streams" more dynamically

If you have the "Manage Streams" page open and somebody else is doing stream activity, your UI will not update immediately for all things, such as the list of subscribers in streams and stream counts.  For counts, it's probably not disruptive, but for redrawing list of subscribers, it may be a bit jolting to have rows appear/disappear.  We may want to refine this after the re-design.

I did the piece for stream counts in 0f7c8c6d34046641fbe7c75706ca36a123403fec.  It'd be nice to do the subscribers list as well.

This was fixed in #2126.


refactor [twitter] service

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @chris48s!

  Generated by :no_entry_sign: dangerJS against 43cbc473176b4b513dc62fa96604a694e1d8185f


Improve: Change state name and error wording when LegacyArgumentLength

In session-manager-plugin, it is in the state of IsAwsCliUpgradeNeeded with 4 arguments as LegacyArgumentLength.
https://github.com/aws/session-manager-plugin/blob/e4ad4017df969b298c3e263f0c8ce9881c0b0a79/src/sessionmanagerplugin/session/session.go#L136
At this time, ProcessKMSEncryptionHandshakeAction will display an error, but please consider changing the error wording and state name at this time.
https://github.com/aws/session-manager-plugin/blob/65933d1adf368d1efde7380380a19a7a691340c1/src/datachannel/streaming.go#L832
This is because session-manager-plugin is used in addition to awscli such as copilot-cli and ecspresso, and the error wording "Please upgrade to latest version of AWS CLI" can be misleading.
https://github.com/aws/copilot-cli/blob/ada23de30b1dab0b0d0659a66d45bdfc6646954e/internal/pkg/exec/ssm_plugin.go#L56
https://github.com/kayac/ecspresso/blob/v1/exec.go#L93
Thank you for your consideration.

Hi somen440@,
Thank you for reporting the issue here. Session Manager service team is looking into this request and will update the error message to be more precise in upcoming releases.
Regards,
Yuting

Hi somen440$,
The error message was updated as per your suggestion. Please try to upgrade to the latest version <IP_ADDRESS> and try it out.
https://docs.aws.amazon.com/systems-manager/latest/userguide/session-manager-working-with-install-plugin.html#plugin-version-history
Regards,
Yuting


Implement LM-Eval for language model evals

Use lm_eval to run evals. Replace current code in utils.py::test.

Dupe of #10


Doesn't work

ERROR in   TypeError: callback is not a function

  - index.js:13 compilation.plugin
    [nowork]/[html-webpack-inline-style-plugin]/index.js:13:4

Hello, what version of Node.js are you using?

It's probably more a webpack 4 thing.

Bump npm registry plz


Declared License

Type: Incomplete
Summary:
Declared License
Details:
Add BSD-2-Clause
Resolution:
https://github.com/isaacs/node-glob/blob/73f57e99510582b2024b762305970ebcf9b70aa2/LICENSE
Affected definitions:

glob 3.2.11

@geneh - This one has conflicts, too. If you want to send me instructions on how to resolve these conflicts, or we can setup a 'training' call, please let me know. Thank you!


Extension No Longer Working

Line 66 of content.js returns null
This should be fixed by simply changing the query CSS selector to just "[role=contentinfo]" ?
The original line of code: document.querySelector("div.now-playing-widget[role=contentinfo]"); is broken by the div class no longer being of the "now-playing-widget" class

Thanks for reporting, I'll have a look later.

Thanks, issue is fixed. v0.1.2 is pending Chrome Web Store review.


Linear-time Christoffel word traversal to find brightest MOS mode; extended Euclid to find gen class

The algorithm is described in Definition 3.1 (page 6) in this paper: https://arxiv.org/abs/2012.15818

This makes Björklund an orphan so it could be moved to xen-dev-utils so that it won't get lost. I can handle this piece of housekeeping after completing the review process.

I want to first get the extended Euclidean algorithm part to work, and then we'll see if my Bresenham function or the existing Bjorklund function runs faster or if we can optimize the Bjorklund function.

I want to first get the extended Euclidean algorithm part to work, and then we'll see if my Bresenham function or the existing Bjorklund function runs faster or if we can optimize the Bjorklund function.

See how xen-dev-utils does benchmarking. We don't need to integrate this all the way to scale-workshop yet to have confidence in the speed.

bresenham is completely unreferenced now. Can you add tests to verify that it works (also documenting its use in the process)?

Issues minor. Will fix on main.


[Honeybadger] Improve error handling for Project Creation when Redis Failures are encountered

Please see the following example: https://app.honeybadger.io/projects/113559/faults/104746997#notice-trace
In cases such as these, even if the Redis outage is not related to the Ruby on Rails codebase architecture, there might still be a need to improve error handling for end-users. For example:
def create
    new_project
    project_metadata = ProjectMetadata.new( current_user:, project: new_project)
    new_project_params = params.dup
    metadata_params = new_project_params.merge({
      status: Project::PENDING_STATUS
    })
    project_metadata.create(params: metadata_params)
    if new_project.save
      TigerdataMailer.with(project: @project).project_creation.deliver_later
      redirect_to project_confirmation_path(@project)
    else
      render :new
    end
  rescue RedisClient::CannotConnectError => redis_client_error
    error_message = "Failed to create the new project: Failed to connect to the Redis client: #{redis_client_error}"
    Rails.logger.error(error_message)
    flash[:notice] = error_message
    render :new
  end

for https://github.com/pulibrary/tiger-data-app/blob/main/app/controllers/projects_controller.rb#L14
Acceptance Criteria:

[x] Ensure that errors of the type RedisClient::CannotConnectError are properly handled from within ProjectsController#create
[x] This error must be logged
[x] The user should be aware that an error occurred when trying to create the new project

Perhaps this needs to be restructured to reflect that users will need to repeatedly retry to create a new project (and if Redis is down, second attempts will still fail).

Make sure that errors also go to Honeybadger.  Otherwise, this looks good.


samples/subsys/tracing/ sample.tracing.gpio fails on stm32 boards with gpio_loopback fixture

Describe the bug
Running the    samples/subsys/tracing/sample.tracing.gpio   on stm32 boards where the fixture gpio_loopback is set
will fail.
--> the gpio_loopback is set for the target board in order to pass tests/drivers/i2s/i2s_api or test/drivers/gpio/gpio_basic_api/ or tests/drivers/uart/uart_async_api
When executing the samples/subsys/tracing/sample.tracing.gpio, the console gives the following output which is not declared as "PASSED" :
...
sys_trace_idle_user
sys_trace_idle_user
thread_a: Hello World from nucleo_g474re!
sys_trace_idle_user
sys_trace_idle_user
sys_trace_idle_user
sys_trace_idle_user
sys_trace_idle_user
sys_trace_idle_user
thread_b: Hello World from nucleo_g474re!
sys_trace_idle_user
...

To Reproduce
Steps to reproduce the behavior:

west build -p auto -b nucleo_g474re samples/subsys/tracing  -DCONF_FILE="prj_gpio.conf" -DEXTRA_DTC_OVERLAY_FILE="gpio.overlay"
west flash

Expected behavior
Should the testcase be executed on that board ?
Impact
Test campaign fails on each target board where the gpio_loopback fixture is set
Environment (please complete the following information):

OS: (e.g. Linux, MacOS, Windows)   ubuntu
Toolchain (e.g Zephyr SDK, ...)  SDK 0.16.8
Commit SHA or Version used  ef6f804d8f2648935afadaab09b511913ffd9f4c

Incorrect assignment, this bug is STM specific.

The same issue is visible on:

nrf52dk/nrf52832
nrf52840dk/nrf52840
nrf5340dk/nrf5340/cpuapp
<EMAIL_ADDRESS>nrf54l15dk/nrf54l15/cpuapp
nrf54l15dk/nrf54l05/cpuapp
nrf54l15dk/nrf54l10/cpuapp
nrf54l15dk/nrf54l15/cpuflpr

should be fixed by https://github.com/zephyrproject-rtos/zephyr/pull/82648, tentatively closing

The same issue is visible on:
nrf52dk/nrf52832
nrf52840dk/nrf52840
nrf5340dk/nrf5340/cpuapp
<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>
The issue still occurs on the all nordic platforms.

@ydamigos can you take a look?

@fabiobaltieri  It seems to be a different issue. The output prints continuously:
sys_trace_isr_enter_user: 0
sys_trace_isr_exit_user: 0
...

It look likes it stays in an IRQ loop.

thanks @ydamigos, @katgiadla can you reopen a different one? (just to be clear I'm just triaging here)


About keys.html

I made the GitHub gist (https://gist.github.com/thepwrtank18/4456b1a4676a26c6ef25b8e8b70e26d7) that was copied to https://thorium.rocks/winxp/keys.html.
Not mad or anything, just want to say that a lot of keys on there aren't reliable at all (some are just trial keys, some probably cause a WPA bootloop where you can't actually activate Windows, etc).

@thepwrtank18 oooo, I see. I just ran the .md through an "md to html" converter, then added some CSS, and tweaked the grid and notes at the bottom.
I didn't edit any of the keys, however, I have used them on XP Retail, XP VL, and server 2003 with no issues.
Cool new projects following the deprecation notice.
I mainly made this for myself, as I spin up XP machines and install it on real hardware pretty often. Usually for development or testing of my XP compatible software, or just for fun/nostalgia, and was frustrated oftentimes having to track down the same key again (because I often don't use a physical CD where I could just write the key on it, as we used to do back in the XP days).
Also to test out converting an .md file into a decent html version of it. I have used it since then to help converting my .md version of docs to html versions on the thorium.rocks/docs page.
Thanks, and yeah I hope you're not mad. I considered it open source since it was on GitHub, even though technically the keys arent supposed to be open source, so I just copied it.


Allow user to set category even for top-level projects. [OSF-5833]

Product team has requested that this be merged alongside #5617
Purpose
Top level projects should be allowed to have any category. All parts of the OSF should follow these rules consistently.
From now on, a "project" is defined as something that has no parent. A "component" is something that does have a parent (even if the category is "project"- the category field is misleading).
Most parts of the OSF (incl APIv2) should already be following these rules following last year's node refactor. This PR is intended to catch stragglers. It adds internal changes, and one small UI improvement.
Changes

The project settings page now allows you to set category for top-level projects (consistent with how APIv2 works). Top-level projects no longer have to be of category "project".

Consolidated how a project was defined into a single place. This means that different parts of the OSF should give more consistent counts. (eg the profile page "# of public projects" count should match the number actually shown)

Updated project queries to exclude new things that are not projects (is_collection, is_bookmark_collection, etc)

Altered metrics scripts to use new project definition

Performance improvement for profile page: len(list(queryset)) --> queryset.count()

Side effects
The list of projects shown- and counts- will be subtly different than before. I have performed an initial audit on staging DB, before and after the code change. See spreadsheet link in JIRA ticket for some sample things that will be affected.

Search results / category breakdown
Recent activity page: /explore/activity/ (also used by new and noteworthy script)
Profile page projects list (counts, as well as what shows up in the projects vs components panels)
Benchmarks run nightly
Admin app metrics tool

Ticket
https://openscience.atlassian.net/browse/OSF-5833

Okay, pass done. 🐧 You should make maintenance tickets for the todo items and link them to the todo lines as well.

[ ] Rename the one test to something slightly more descriptive
[ ] Remove check for is_bookmark_collection in the one query

Pushed requested changes; I'll check CI in the AM.

Changes are good. When tests pass, then we are ready to merge. 🎉 🐧 🎉

Restarted build for unrelated CI failures that crept in to develop last night.
Ran flake8, jshint, and OSF unit tests locally, without failures. Fingers crossed.

Restarted second branch which failed again.


[Feature]: Allow passing bounding box colors

Description
This is important to visually distinguish gt and pred detections without having every time to use the "eye" control.
Colors often can carry semantic meaning: 1) detection score, 2) IoU test success or failure etc
It also makes sense to allow to pass different colors for the box itself and for the box caption frame.
Suggested Solution
N/A -> outlined abvoe
Alternatives
No response
Additional Context
No response

Hi @vadimkantorov,
Thanks for your feature request! I see an internal ticket similar to this already, I'll bump the priority on that for you.
Thanks,
Ramit

stale bump. I don't think it's very useful to close standing feature request as stale :(

Hi @vadimkantorov,
I'm sorry about this! This is one of our automations which closed the issue since there had been no activity here. The feature request is still planned, and this automation closing the ticket does not change that.
I'll look into the automation to try to prevent this behavior for feature requests we have.
Thanks,
Ramit

Indeed, this would be quite useful. It's a bit unfortunate, that it has been stale for over 1.5 years now 🥲
Any updates on this?

Hey @MaxSchambach! Thank you for checking in! I checked internally and it looks like the feature request got backlogged because we were working on another potential solution at the time. If you would like to I would love to submit a feature request for you to bump this ticket and potentially raise the priority on it :)

@ArtsiomWB that would be great, thank you!


Remove manual require_once calls

the zf1-future fork is using composer for autoloading.
still it contains a lot of require_once calls in its codebase, e.g.
https://github.com/Shardj/zf1-future/blob/7fa9c5f50f0ce8ceecafb13e107eebe4c3d04051/library/Zend/Controller/Front.php#L23-L30
these require once calls are harmful for static analysis.
for best performance and most accurate results PHPStan suggests to use regular composer autoloading instead of manual require/include calls.
since the composer autoloader is configured, I would like to propose dropping require calls where possible for a more relieable static analysis experience.
whats your opinion on this? PR welcome?

our concrete problem at hand: we run into fatal errors while running phpstan because of this require_once calls:
Internal error: Internal error: Class "Zend_Controller_Plugin_Abstract" not found in file C:\dvl\daiber\application\B2B\controllers\ArticleController.php

     Post the following stack trace to https://github.com/phpstan/phpstan/issues/new?template=Bug_report.md:
     #0 C:\dvl\daiber\vendor\shardj\zf1-future\library\Zend\Controller\Front.php(30): require_once()
     #1 C:\dvl\daiber\vendor\shardj\zf1-future\library\Zend\Controller\Action.php(35): require_once('...')
     #2 C:\dvl\daiber\library\clxProductNet\library\Clx\Controller\Action.php(8): require_once('...')
     #3 phar://C:/dvl/daiber/vendor/phpstan/phpstan/phpstan.phar/vendor/composer/ClassLoader.php(582): include('...')
     #4 phar://C:/dvl/daiber/vendor/phpstan/phpstan/phpstan.phar/vendor/composer/ClassLoader.php(433): Composer\Autoload\{closure}()
     #5 C:\dvl\daiber\application\B2B\controllers\ArticleController.php(16): Composer\Autoload\ClassLoader->loadClass()
     #6 C:\dvl\daiber\vendor\plugins\rocket\lib\Rocket.php(411): require_once('...')
     #7 phar://C:/dvl/daiber/vendor/phpstan/phpstan/phpstan.phar/src/Reflection/BetterReflection/SourceLocator/AutoloadFunctionsSourceLocator.php(41): Rocket::autoload()
     #8 phar://C:/dvl/daiber/vendor/phpstan/phpstan/phpstan.phar/vendor/ondrejmirtes/better-reflection/src/SourceLocator/Type/AggregateSourceLocator.php(26): PHPStan\Reflection\BetterReflection\SourceLocator\AutoloadFunctionsSourceLocator->locateIdentifier()
     #9 phar://C:/dvl/daiber/vendor/phpstan/phpstan/phpstan.phar/vendor/ondrejmirtes/better-reflection/src/SourceLocator/Type/MemoizingSourceLocator.php(33): PHPStan\BetterReflection\SourceLocator\Type\AggregateSourceLocator->locateIdentifier()
     #10 phar://C:/dvl/daiber/vendor/phpstan/phpstan/phpstan.phar/vendor/ondrejmirtes/better-reflection/src/Reflector/DefaultReflector.php(32): PHPStan\BetterReflection\SourceLocator\Type\MemoizingSourceLocator->locateIdentifier()
     #11 phar://C:/dvl/daiber/vendor/phpstan/phpstan/phpstan.phar/src/Reflection/BetterReflection/Reflector/MemoizingReflector.php(45): PHPStan\BetterReflection\Reflector\DefaultReflector->reflectClass()
     #12 phar://C:/dvl/daiber/vendor/phpstan/phpstan/phpstan.phar/src/Reflection/BetterReflection/BetterReflectionProvider.php(163): PHPStan\Reflection\BetterReflection\Reflector\MemoizingReflector->reflectClass()
     #13 phar://C:/dvl/daiber/vendor/phpstan/phpstan/phpstan.phar/src/Reflection/ReflectionProvider/MemoizingReflectionProvider.php(43): PHPStan\Reflection\BetterReflection\BetterReflectionProvider->getClass()
     #14 C:\dvl\daiber\vendor\phpstan\phpstan-deprecation-rules\src\Rules\Deprecations\ImplementationOfDeprecatedInterfaceRule.php(45): PHPStan\Reflection\ReflectionProvider\MemoizingReflectionProvider->getClass()
     #15 phar://C:/dvl/daiber/vendor/phpstan/phpstan/phpstan.phar/src/Analyser/FileAnalyser.php(106): PHPStan\Rules\Deprecations\ImplementationOfDeprecatedInterfaceRule->processNode()
     #16 phar://C:/dvl/daiber/vendor/phpstan/phpstan/phpstan.phar/src/Analyser/NodeScopeResolver.php(417): PHPStan\Analyser\FileAnalyser->PHPStan\Analyser\{closure}()
     #29 phar://C:/dvl/daiber/vendor/phpstan/phpstan/phpstan.phar/vendor/symfony/console/Command/Command.php(259): PHPStan\Command\WorkerCommand->execute()
     #30 phar://C:/dvl/daiber/vendor/phpstan/phpstan/phpstan.phar/vendor/symfony/console/Application.php(870): _PHPStan_532094bc1\Symfony\Component\Console\Command\Command->run()
     #31 phar://C:/dvl/daiber/vendor/phpstan/phpstan/phpstan.phar/vendor/symfony/console/Application.php(261): _PHPStan_532094bc1\Symfony\Component\Console\Application->doRunCommand()
     #32 phar://C:/dvl/daiber/vendor/phpstan/phpstan/phpstan.phar/vendor/symfony/console/Application.php(157): _PHPStan_532094bc1\Symfony\Component\Console\Application->doRun()
     #33 phar://C:/dvl/daiber/vendor/phpstan/phpstan/phpstan.phar/bin/phpstan(122): _PHPStan_532094bc1\Symfony\Component\Console\Application->run()
     #34 phar://C:/dvl/daiber/vendor/phpstan/phpstan/phpstan.phar/bin/phpstan(123): _PHPStan_532094bc1\{closure}()
     #35 C:\dvl\daiber\vendor\phpstan\phpstan\phpstan(8): require('...')
     #36 C:\dvl\daiber\vendor\bin\phpstan(120): include('...')
     #37 {main}
     Child process error (exit code 1):
     ```

as soon as we drop the require calls, these errors do no longer appear

I think zf-future claims to be compatible with origina zendframework, and using that loader, not composer.
i think this has been discussed already before in the issues.
if you want composer only, use zf1s:

https://github.com/zf1s/zf1

my point was not about dropping the zend loaders or loader classes but only dropping the manual require calls

and my reply point was that original zendframework expected the require_once calls to be in the code.

https://github.com/Shardj/zf1-future/issues/37

This would break "drop-in" compatibility quite badly - for example: by requiring Zend_Config & Zend_Db you can easily factory Zend_Db_Adapter based on Zend_Config without any autoloading. I can only guess how many legacy applications there are relaying on this functionality around the world.

Imo - no way. I have HUGE ZF1 codebase and last thing we need to rework in company is reworking the class names.
@staabm any possibility to suppress STAN by annotations? I'm not really using STAN, so can't say.

Yeah, the whole point of this project is to not break BC. It's a no for me.

Thanks for your input

Used that script to create a patch.
https://gist.github.com/sreichel/635d0c9858bec04a6dd46fc1f85bc3e3
Maybe add it or link it in README.


Parts of Speech vocabulary

Parts of Speech vocabulary should be re-imported as a hierarchical vocabulary (as the current one is)
BASIC:
Verb, Noun, Pronoun, Adjective, Adverb, Preposition, Conjunction, Interjection, Particle
ADVANCED:
All others

I think it belongs to FV, not BCDevExchange-app


Remove auth token management from Satellite client

Fixes VAX-660.
See individual commit messages for more details.

@kevin-dp Check out the two new commits I've just pushed.
You'll notice that authConfig is optional in some methods defined in src/satellite/registry.ts. That's because if I change any of those instances of authConfig? to authConfig, type issues appear in registry.test.ts in all instances of await mockRegistry.ensureStarted(. I'm not sure if I need to add a stub authConfig argument to all those calls. The tests pass fine without it and I don't have sufficient understanding of what's going on there to judge.


add ci converage check

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@395238f). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Current head 94843ed differs from pull request most recent head 16094e6. Consider uploading reports for the commit 16094e6 to get more accurate results

@@           Coverage Diff           @@
##             main      #68   +/-   ##
=======================================
  Coverage        ?   28.45%
=======================================
  Files           ?       30
  Lines           ?     2277
  Branches        ?        0
=======================================
  Hits            ?      648
  Misses          ?     1523
  Partials        ?      106

Flag
Coverage Δ

unittests
28.45% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.


Add flags to use client certs for cli.

This allows grpc_cli to act with the specific client identity when using SSL. It does not however set the cert when using Google Default Credentials, as it does not have the necessary options to provide a client cert and key.

Partially fixes #16059.

/cc @yang-g

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

[microbenchmarks] No significant performance differences

If not @yang-g, then perhaps @yashykt can review?

Mergeable is still failing because it needs release notes and/or language labels.

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Adding @yang-g as second reviewer

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,952,970      Total (=)      1,952,970

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,665,044      Total (<)     10,665,045

 No significant differences in binary sizes

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,952,970      Total (=)      1,952,970

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,665,042      Total (=)     10,665,042

 No significant differences in binary sizes

[microbenchmarks] No significant performance differences

Ping. @yang-g

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,956,097      Total (=)      1,956,097

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,843,882      Total (<)     10,843,883

 No significant differences in binary sizes

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,956,097      Total (=)      1,956,097

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,843,887      Total (>)     10,843,882

 No significant differences in binary sizes

[microbenchmarks] No significant performance differences

****************************************************************

libgrpc.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

****************************************************************

libgrpc++.so

     VM SIZE        FILE SIZE
 ++++++++++++++  ++++++++++++++

  [ = ]       0        0  [ = ]

[trickle] No significant performance differences

Objective-C binary sizes
*****************STATIC******************
  New size                      Old size
 1,960,882      Total (=)      1,960,882

 No significant differences in binary sizes

***************FRAMEWORKS****************
  New size                      Old size
10,881,534      Total (=)     10,881,534

 No significant differences in binary sizes

[microbenchmarks] No significant performance differences


ProvidePlugin can't provide function for TS __awaiter call

Compiled code:
    /***/ 405:
    /***/ function(module, exports, __webpack_require__) {

	/* WEBPACK VAR INJECTION */(function(__decorate, __metadata) {"use strict";
	const core_1 = __webpack_require__(4);
	const router_1 = __webpack_require__(367);
	__webpack_require__(121);
	let RedirectService = class RedirectService {
	    constructor(router) {
	        this.router = router;
	    }
	    redirect(link, delay = 0) {
	        return __awaiter(this, void 0, Promise, function* () {
	            yield this.delay(delay);
	            this.router.navigate([link]);
	        });
	    }
	    delay(delay) {
	        return new Promise((resolve) => setTimeout(resolve, delay));
	    }
	};
	RedirectService = __decorate([
	    core_1.Injectable(),
	    __metadata('design:paramtypes', [router_1.Router])
	], RedirectService);
	exports.RedirectService = RedirectService;

	/* WEBPACK VAR INJECTION */}.call(exports, __webpack_require__(2), __webpack_require__(3)))

/***/ },

Config:
 plugins: [
            new webpack.ProvidePlugin({
                '__awaiter': 'typescript-awaiter',
                '__decorate': 'typescript-decorate',
                '__metadata': 'typescript-metadata'
            })
        ]

Problem: ProvidePlugin can't provide function for TS __awaiter call.

This may be because typescript-decorate exports its function as a named, rather than default, export.
Relevant source code: https://github.com/sanex3339/typescript-decorate

Hi, no. I changed all these modules to
module.exports = function (thisArg, _arguments, P, generator) { //... }
but same problem.
15.04.2016, 13:12, "Dan"<EMAIL_ADDRESS>
This may be because typescript-decorate exports its function as a named, rather than default, export.
Relevant source code: https://github.com/sanex3339/typescript-decorate
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub

still looking for solution

I am looking for solution when import JS files from node_modules. Seems that filed does not processed at all


Add support for AEAD

Need to add constants and methods to support AEAD.

Currently working on this.  This will be in the next release within the next couple of days.

Resolved.


[Suggestion] Need to clearly specify the format to use gpu resources

Note: This repository is ONLY used to solve issues related to DOCS.
For other issues, please move to other repositories.

Is there anything that's missing or inappropriate in the docs? Please describe.
https://github.com/milvus-io/docs/blob/v0.7.0/site/en/reference/milvus_config.md
Describe your suggestion
Need to clearly describe how to add multiple GPUs. It will be great to have an example. We also need to specify both search computation and index building support multiple GPUs.

Fixed


PyTorch Server starts the # processes as the # cores on the machine ?

/kind bug
What steps did you take and what happened:
Create a PyTorch service which defaults to 1 CPU, however it started 40 processes which is the number of cores on the node.
apiVersion: "serving.kubeflow.org/v1alpha2"
kind: "InferenceService"
metadata:
  name: "pytorch-cifar10"
spec:
  default:
    predictor:
      pytorch:
        storageUri: "gs://kfserving-samples/models/pytorch/cifar10/"
        modelClassName: "Net"

kubectl logs -f  cifar10-predictor-default-gkb4r-deployment-587b7bfbf7-stv8l  -c user-container
INFO:root:Copying contents of /mnt/models to local
INFO:root:Registering model:cifar10
INFO:root:Listening on port 8080
INFO:tornado.general:Starting 40 processes

What did you expect to happen:
The KFServer should have # processes created same as the number of CPUs specified on spec
Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:

Istio Version: 1.1.7
Knative Version: 0.8.0
KFServing Version: 0.2.0
Kubeflow version:
Minikube version:
Kubernetes version: (use kubectl version):
OS (e.g. from /etc/os-release):

Currently, this just maxes out the number of cores available. We can't use the spec since partial cores are used. What's the problem with the current impl?

If the machine has 40 cores then I am a bit worried about each pod spawning 40 processes. What do you think make this configurable ?

I feel like these are details the user shouldn't be aware of. Processes are pretty lightweight, but I'd also be happy to cap this at something like 8. Did we find a performance issue? Is this something we can address in the future?


If a download task is already in progress, no need to start a new one

If a download task for a particular resource is already in progress, there is no need to start a new download task. This is particularly noticeable in the product tool when downloading updates. Long downloads can easily be started multiple times.

There could be a general download manager or the code that started the download task would need to manage that.


Mobile Responsive

Changes to allow users to access most functionality on mobile including a simple fix for Nav Bar issue #2

It may take me some time to review this; I'll have to ask you about some of these changes later on, either here or in the slack.

Sure! I probably should have broken it down into smaller chunks.

Sure! I probably should have broken it down into smaller chunks.

Actually, would it be a lot of trouble for you to break out the navbar related changes into a different PR? I put that together pretty fast and loose on the first pass, it could use a deep refactor. I'd like for the mobile view of the navbar to have an icon link to the home page, or something, so the sprites may need to be regenerated.

Looks good to me, nice work, and thanks for accommodating me re: the splitting and revisions.


document snags in reproducing results

To try to make things more reproducible, I will do a fresh rebuild of the pipeline and document snags as they come up.

I rebuilt and with the changes in #111 I didn't find any snags (at least for now).


tilt: make hud the default view

tilt up now opens the hud instead of the stream
l now opens a modal with the stream
esc now closes the active modal instead of the hud, and q now quits tilt

does the place that we capture sigint/sigterm clean up the hud well?

Yes, buuuuut, apparently tcell intercepts sigint/sigterm and just makes them do nothing? I've added ^C to the key switch.


Replication Optimization: Treating deleted put record's size as a delete record

This is an optimization for replication.
In replication, when replicator sends a ReplicaMetadataRequest to remote server, remote server would scan through the index segment files. It will stop when the number of scanned bytes are larger than the max number of byte in the ReplicaMetadataRequest. And the max number of bytes is usually 4MB (one PutBlob size).
In this case, when the remote server sees a Put record in the index segment file, it will stop scanning right away. However, if this Put record is already deleted (there is a Delete record after this Put), the size in the MessageInfo would be changed to size of Delete.  When the replicator sees this id, it knows that this id is deleted in the remote server so it does nothing. This is a terrible waste of roundtrip since we are spending one round trip to deal with one blob id.
In this PR, we would keep scanning if this put Record is already deleted. In this way, we can return more ids back to the replicator in one request. We only do this for IndexBased replication token.

Codecov Report
Base: 0.00% // Head: 14.99% // Increases project coverage by +14.99% :tada:

Coverage data is based on head (11e47d1) compared to base (0792149).
Patch coverage: 80.95% of modified lines in pull request are covered.

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2297       +/-   ##
=============================================
+ Coverage          0   14.99%   +14.99%
- Complexity        0     2389     +2389
=============================================
  Files             0      754      +754
  Lines             0    57874    +57874
  Branches          0     7113     +7113
=============================================
+ Hits              0     8681     +8681
- Misses            0    48675    +48675
- Partials          0      518      +518

Impacted Files
Coverage Δ

.../messageformat/DeleteMessageFormatInputStream.java
58.33% <57.14%> (ø)

...n/java/com/github/ambry/store/PersistentIndex.java
88.39% <92.30%> (ø)

...main/java/com/github/ambry/config/StoreConfig.java
97.47% <100.00%> (ø)

...ava/com/github/ambry/clustermap/MockReplicaId.java
56.41% <0.00%> (ø)

...n/java/com/github/ambry/clustermap/Datacenter.java
0.00% <0.00%> (ø)

...ava/com/github/ambry/frontend/UndeleteHandler.java
0.00% <0.00%> (ø)

...in/java/com/github/ambry/store/LogSegmentName.java
96.82% <0.00%> (ø)

...ava/com/github/ambry/protocol/UndeleteRequest.java
0.00% <0.00%> (ø)

...ub/ambry/network/SocketRequestResponseChannel.java
0.00% <0.00%> (ø)

...n/java/com/github/ambry/utils/CrcOutputStream.java
83.33% <0.00%> (ø)

... and 747 more

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

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Question: Windows Opened Feature enabled by default for TZE200_hue3yfsn?

Hey, this is only a question, is the sensor to detect if a windows is open enabled by default? In the original ZHA there was a temperature slider to say when it should detect if a window might be open. If it's enabled by default, how does it work? At a certain temperature or like how the manufacturer implemented it (5 °C dropoff in 5 minutes)?
The device is use is ("_TZE200_hue3yfsn", "TS0601") # Zonnsmart TV02-ZG

There were few tests of that function for different TRV's and most users agreed that this function works best if you put a TRV in the freezer. I did that myself, because opened window couldn't trigger that function. Fridge didn't trigger that either.
I was just curious if that was even implemented.
You have the TRV - check how Chinese engineers made it in the firmware. If you just open a window you can observe the behavior.
I'm almost sure nothing will happen. Temp drop is too low or too small, or both.


Will this project be developed?

as subject :)

@nicraMarcin not very positive right now as the documentation/site is gone nowadays. Too bad, as I invested quite some time getting things working to my liking.

I would like to apologise for this issue. I am working on it right now.
We have been extremely busy working on an important platform which brings revenue to our company based in London. We could not free any resources for Kubegres until today.
In regards to the future of this project, it is actively used and it is also used on the new platform that we are working, which has over 30 microservices. Each service has at least 3 instances of Postgres running with Kubegres.
An update is planned for June where we will be upgrading Kubegres with the latest code of kubebuilder v3.4.1 which is compliant with the libraries of for Kubernetes 1.23.5
I will let you know once the websites are available.

@joes me too :/ I wanted to change max_connections for server but I don't rememter config structure :( and site diesn't work :( Maybe do you have example Kubegres manifest with custem postgres.conf settings?

https://github.com/reactive-tech/kubegres-website/blob/main/website/doc/getting-started.html

https://www.kubegres.io/ is back online


Merge the “feature/body” branch into the “develop”

Merged the “feature/body” branch into the “develop”


https://docs.microsoft.com/en-us/powershell/module/appv/convertfrom-appvlegacypackage?view=win-mdop2-ps

Closing as there is no description on the issue.


Compatible to InfluxDB v2.3?

Hi
Is it compatible to InfluxDB v2.3?
Best regards, Patrick

Hi @jayjupdhig,
It should be compatible (based on the docs https://docs.influxdata.com/influxdb/v2.3/query-data/influxql/)  but it wasn't ever tested.
Please note that only InfluxQL queries are supported.


use mocha for testing

Tests with mocha instead of gradle/kotlin
Added probe config spp-probe.host, spp-probe.grpc_port, spp-probe.rest_host

will change other probes to use this config via https://github.com/sourceplusplus/live-platform/issues/452

@MrMineO5, do you know how to fix SyntaxError: Cannot use import statement outside a module? I'm trying to get this to work: https://github.com/sourceplusplus/probe-nodejs/blob/bfdc62eaf4035372b408c9ff10174219e56d9f33/test/test.js#L108

You can't require the typescript files in a javascript file, a functional, but suboptimal solution is to use ../dist/SourcePlusPlus.js

Thanks, that worked. Can you please review? I'm new to testing with mocha but I think this style of testing is more suitable for this project. There's a lot of baggage in include Kotlin/Gradle.

The test JWT tokens don't have an expiration date, but yes, ideally they should be generating the JWT token.


Add custom schema to response object

If there is schema in response object, just use it.

Coverage increased (+2.4%) to 100.0% when pulling fb62e9f95dc067b81cd0dbb310c76df6356fbef2 on nihsmik:add-schema-to-response-object into d2b9f679bbd473386858883023fd82e9d7378ed2 on ruby-grape:master.

Coverage increased (+2.4%) to 100.0% when pulling fb62e9f95dc067b81cd0dbb310c76df6356fbef2 on nihsmik:add-schema-to-response-object into d2b9f679bbd473386858883023fd82e9d7378ed2 on ruby-grape:master.

Coverage increased (+2.4%) to 100.0% when pulling fb62e9f95dc067b81cd0dbb310c76df6356fbef2 on nihsmik:add-schema-to-response-object into d2b9f679bbd473386858883023fd82e9d7378ed2 on ruby-grape:master.


How to set the Accelerometer range and increase the sampling speed? Thx

Hi, I need to sample the accelerometer at 1000 Hz, may I know how to the I2C speed?
For the accelerometer, I need +/- 4g. How to use the set_acel_range function?
Thanks

Hello AK51, the I2C speed can be configured by using the device tree.
Any linux device has a i2c clock speed fixed.
Raspberry Pi (3.18 kernel or later)
Fortunetly for the raspberry kernel, it is easy to implement by using the dtparam in the config.txt file.
sudo nano /boot/config.txt
INSERT
dtparam=i2c_baudrate=400000
THEN
sudo reboot
Notes for the Raspberry Pi 3B, 3B+ and Zero W
The I2C controller on the Raspberry Pi 3B, 3B+ and Zero W has its clock linked to the VPU core so as the VPU frequency changes depending on processor load so does the clock rate of the I2C bus.  This can cause problems with any devices that expect a constant clock rate during communication.
To solve this issue the VPU core frequency must be set to a fixed frequency by editing /boot/config.txt.  Open the file with nano using the command:
sudo nano /boot/config.txt
Add the following text to the bottom of the file.
core_freq=250
Save your changes, exit the nano editor and reboot
sudo reboot
For beaglebone in the other hand. You need to create a new overlay for the device tree
https://stackoverflow.com/questions/33549211/how-to-add-i2c-devices-on-the-beaglebone-black-using-device-tree-overlays

About the accelerometer range, indeed can be:
+/- 2 , +/- 4, +/-8 and +/- 16.
To change the scale. after you create the object you can set the accell range.
from imu_mpu6050 import MPU6050
i2c = busio.I2C(SCL,SDA)
IMU = MPU6050(i2c)
IMU.set_accel_range(IMU.ACCEL_RANGE_4G)

Let me know if you solve it.

Hi, I need to sample the accelerometer at 1000 Hz, may I know how to the I2C speed?
For the accelerometer, I need +/- 4g. How to use the set_acel_range function?
Thanks

Since no feedback was provided I will close this issue .


Request: Documentation of Karmada components

What documentation would you like to be added:
I'd like to have a document to explain how many components in Karmada and what each component does.
Just like the document of Kubernetes Components.
Why do you think this document is needed:
An overview document about the components would help people understand the architecture of Karmada quickly.
Where do you think the new document should be placed:
I think the document could be placed under /Documentation/Core Concepts/Components.
More precisely, the document should be placed in this code directory:
https://github.com/karmada-io/website/tree/main/docs/core-concepts/components.md

you can preview the document at here:  karmada components
can you help me to confirm if there are any major issues in the document ? I will continue to improve on this basis.

/assign @chaosi-zju
in favor of #420