Mapping MIDI time to frame count

I looked up TimePosition and BarBeatTick, but they don't seem to have informations to map MIDI time to frame count.
I'd like to get following informations:

Length of tick, beat or bar in frames.
Frame count to reach next tick, beat or bar from the start of current buffer.

Or is it expected to handle MIDI time code directly?
It also looks like VstTimeInfo in vestige.h has all relevant information.

DPF follows the JACK transport API when it comes to tempo.
https://jackaudio.org/api/structjack__position__t.html
https://distrho.github.io/DPF/structTimePosition_1_1BarBeatTick.html
ticksPerBeat tells you how many ticks there are in a beat.
beatsPerBar tells you how many beats there are in a bar.
Ticks are unrelated to frames.
Frame is monotonic, increases as long as host keeps playing without reposition of timeline.
The amount of ticks in a bar varies between applications, some provide more resolution than others. for example, LMMS uses 48, while MusE uses 1920

So tempo and ticksPerBeat can be used to get the length of the tick in seconds.
auto ticksPerSecond = bbt.ticksPerBeat * bbt.beatsPerMinute / 60;
auto secondsPerTick = 1.0 / ticksPerSecond;

Thank you for the explanation.

It is best not to do things based on tick count though.
Being an integer value, it does not have full precision.
I guess I can make it a double.. JACK standalones do not have this fine precision, but LV2 and VST2 plugins do.
Let me know if you run into issues.

Hi. I have another question.
I'm trying to make a CV plugin which outputs trigger signal at the start of bar/beat. I wrote a code like following:
void run(const float **, float **outputs, uint32_t frames) override
{
  const auto timePos = getTimePosition();
  if (timePos.bbt.valid) {
    auto &bbt = timePos.bbt;

    double secondsPerTick = 60.0 / (bbt.ticksPerBeat * bbt.beatsPerMinute);

    double tickLength = sampleRate * secondsPerTick;
    double beatLength = tickLength * bbt.ticksPerBeat;
    double barLength = beatLength * bbt.beatsPerBar;

    double barStartFrame = tickLength * bbt.barStartTick;
    double beatStartFrame = barStartFrame + beatLength * (bbt.beat - 1);
    double tickStartFrame = beatStartFrame + tickLength * bbt.tick;

    double offsetToNextTick = timePos.frame - tickStartFrame;

    // ...
  }
}

I'm trying to calculate frame offset from the start of current buffer to the next tick as offsetToNextTick. The code above is assuming that bbt.tick indicates latest tick count right before the start of current buffer. However, offsetToNextTick can be both positive or negative when I tested.
The question is that where exactly is bbt.tick indicating? Is it something like the closest tick to the start of current buffer?
Following is a drawing of my understanding of relation about tick and the start of buffer. Start of buffer is more close to B. I'd like to know which of tick A or tick B becomes bbt.tick. Or could it be possible that bbt.tick indicates completely different time?

the transport information refers to the frame 0 of each process cycle. so yes, closest to the start of each buffer

Thanks for the answer.
I tested more and found that the code on previous comment breaks when tempo is changed. I looked up LV2  metronome example, and it seems like they are using barBeat which is provided by float.
Is it possible to add barBeat to BarBeatTick?

hey, sorry for the lack of news on this.
to make things clear... I am slowing down a bit on the hobby-project work, focusing on a specific project at a time. I have scheduled on my calendar time that I will dedicate to each project. First carla, then cadence, then jack2, then distrho-ports and finally DPF.
I will get back to DPF full-force in november. someone else has been asking about exactly the same thing (in regards to needing a precise timing source)

I'd like to say thanks for your works. DPF and Carla made my plugin development far easier.
Just one thing, I know a person who got mentally ill for working too hard, so please take your time and get a break when feeling tired. It's OK to procrastinate.
For anyone stumbled upon this issue, I have temporary patches to solve this issue on here (DistrhoPlugin*.patch). Example snippet is also available. Hope this helps.

In hvcc it's all puredata logic to do stuff. so having a sample accurate MTC is nice there. You have no other way to sync to anything otherwise. So yeah it's just midi paradigm you generally have to work with.


fix(loaders): always create plugin instance with new

class plugin transpile with babel contains a function '_classCallCheck',
will throw an error when a class is invoked without 'new'
see: https://babeljs.io/docs/en/babel-plugin-transform-classes#examples

@kuoruan thanks nice stuff, just some questions.
How can I make it fail? At this point hasn't been reported any issue.

Hi @juanpicado , I forked a plugin from github-oauth-ui, and compile it with rollup, the target is both ESM and CJS.
Here is my repo: https://github.com/kuoruan/verdaccio-oidc-ui
See the rollup config:
https://github.com/kuoruan/verdaccio-oidc-ui/blob/d7d85b586a421312ae006796cc7504faa316ec4d/rollup.config.js#L52-L62
If I change the exports field to auto or default, the output will be like
class Plugin {}

module.exports = Plugin;

create a config file: verdaccio.yaml
storage: ./storage
plugins: ./plugins

middlewares:
  oidc-ui:
    enabled: true

auth:
  oidc-ui:
    configuration-endpoint: https://gitlab.com/.well-known/openid-configuration
    client-id: xxx
    client-secret: xxx
    scope: "openid api"
    username-claim: nickname
    authorized-group: false
    provider-type: gitlab
  htpasswd:
    file: ./htpasswd
    max_users: -1

security:
  api:
    jwt:
      sign:
        expiresIn: 90d
  web:
    sign:
      expiresIn: 7d

packages:
  "@*/*":
    access: $all
    publish: $authenticated
    unpublish: $authenticated

  "**":
    access: $all
    publish: $authenticated
    unpublish: $authenticated

logs: { type: stdout, format: pretty, level: trace }

Then run:
yarn link:oidc && yarn start

And the error occurs.

Class plugin with rollup output.target: "auto"  will throw an error Class constructor Plugin cannot be invoked without 'new'
Class plugin with babel plugin @babel/plugin-transform-classes will have a function _classCallCheck

So the error is caused by
module.exports = class {};


work on the touch screen

platform: android, google chrome browser.
Problem: there is no way to lock each selected item from accidental changes separately (or is it somehow possible, besides grouping?)
Thank you for the attention!

As this is more general question, try press and hold a shape until you get context menu and then choose Lock


Updated splash screen

Here is a new PR in this updated branch
https://user-images.githubusercontent.com/66668114/194749217-731b6556-d604-4349-b354-80d8afe0ab1e.mov

please set target branch to splash-screen


What happens when the server receives a misformed request to update the locks?

A request to update the locks contains a Struct with the state keys to update as keys, and either a duration in seconds or None as values. Both the python and the rust servers check for misformatted a Struct (see bellow for where it is checked) but what happens in practice if there is an error? Is that behaviour the one we want? In any case, it should be documented.
In python: https://github.com/IRL2/nanover-protocol/blob/6b7f45bed2c325c85120b2a5157bc9101e493052/python-libraries/nanover-core/src/nanover/state/state_service.py#L220-L221
In rust: https://github.com/IRL2/nanover-rs/blob/cf86bf1da2ec037dc73b7ec96d169c60ec839ca0/nanover-rs/src/services/state.rs#L157-L159

In practice:
>>> client.attempt_update_multiplayer_locks({ "somekey": "non-number" })
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Users\mark\anaconda3\envs\nanover\lib\site-packages\nanover\app\client.py", line 61, in wrapper
    return func(self, *args, **kwargs)
  File "C:\Users\mark\anaconda3\envs\nanover\lib\site-packages\nanover\app\client.py", line 651, in attempt_update_multiplayer_locks
    return self._multiplayer_client.attempt_update_locks(update)  # type: ignore
  File "C:\Users\mark\anaconda3\envs\nanover\lib\site-packages\nanover\core\nanover_client.py", line 166, in attempt_update_locks
    response = self._state_stub.UpdateLocks(request)
  File "C:\Users\mark\anaconda3\envs\nanover\lib\site-packages\grpc\_channel.py", line 1176, in __call__
    return _end_unary_response_blocking(state, call, False, None)
  File "C:\Users\mark\anaconda3\envs\nanover\lib\site-packages\grpc\_channel.py", line 1005, in _end_unary_response_blocking
    raise _InactiveRpcError(state)  # pytype: disable=not-instantiable
grpc._channel._InactiveRpcError: <_InactiveRpcError of RPC that terminated with:
        status = StatusCode.UNKNOWN
        details = "Exception calling application: Lock duration must be a real number."
        debug_error_string = "UNKNOWN:Error received from peer  {grpc_message:"Exception calling application: Lock duration must be a real number.", grpc_status:2, created_time:"2024-05-06T12:38:24.0199653+00:00"}"

This is the behaviour that is documented in https://irl2.github.io/nanover/concepts/base-protocol.html#state-locks, then. I may add a note about the request failing.
Thank you.
(I keep the issue open to remember to add said note.)


Suggestions from Nexus on new feats and subs

The suggested feats I had in mind were: Charger, Defensive Duelist, Grappler (advantage on shove instead of grapple), Sharpshooter, Skilled, Shield Master (advantage on Dex saving throws while wearing a shield and take no damage on successful Dex save roll).
Got an idea for the Battlemaster subclass too. It's slightly altered to better fit with the existing classes (and to balance it at the same time)
Lvl 3: ""Commanding Presence" - Learn 2 maneuvers (these maneuvers could be learned as special battlemaster feats), gain 3 superiority dice, +2 on all Charisma checks
Lvl 7: "Student of War" - Learn 1 maneuver, +1 superiority die
Lvl 9: "Know Your Enemy" - Learn 1 maneuver +1 superiority die, +2 Survival skill checks
Maneuvers:
"Distracting Strike" - When you hit a creature with a weapon attack, you can expend one superiority die to distract the creature, giving your allies an opening.  The next attack roll against the target by an attacker other than you has advantage if the attack is made before the start of your next turn.
"Feinting Attack" - You can expend one superiority die and use a bonus action on your turn to feint, choosing one creature within 5 feet of you as your target. You have advantage on your next attack roll against that creature this turn. If that attack hits, add the superiority die to the attack's damage roll.

"Rally" - On your turn, you can use a bonus action and expend one superiority die to bolster the resolve of one of your companions. When you do so, choose a friendly creature who can see or hear you.
"Pushing Attack" - When you hit a creature with a weapon attack, you can expend one superiority die to attempt to drive the target back. You add the superiority die to the attack's damage roll, and if the target is Large or smaller, it must make a Strength saving throw. On a failed save, you push the target up to 15 feet away from you.  If the attack hits, add the superiority die to the weapon's damage roll.
Battlemaster would synergize and specialize amazingly well with all those feats, not to mention, all the other classes would benefit from even greater diversity and freedom of build choices just as well.

Is there any difference between the default battlemaster subclass and the above? it looks the same at a glance so the details are in the PHB. also the tactician fills a similar niche and it could be easier to improve that subclass rather than creating a new subclass from scratch.
I think chris mentioned that suggestions that are not actively being worked on/bugs might be more suit to being discussions rather than issues, at least until someone starts working on it. There's an open discussion about improving that subclass
https://github.com/ChrisPJohn/SolastaCommunityExpansion/discussions/385
either way it might be better to wait until the dlc is released to see how the character creation changes might impact the usefulness of a fighter subclass based on commanding the battle.

The Battlemaster as suggested is only slightly different from Battlemaster in original PHB (the differences are only meant for balance purposes, renaming or to change some things which (I think) cannot be easily implemented into current Solasta gameplay.

Commanding presence is actually a maneuver you can do, but I don't know if it's possible to implement such a thing during regular checks or how useful it would be in campaigns, so I decided to give flat +2 on all these checks instead.
An alternative name for Lvl 3 talent could also be "Combat Superiority" or "Student of War" as it is in PHB, although +2 on charisma checks doesn't really fit in those names so I decided to simply rename it to "Commanding Presence" which would suggest you are both a commander capable of doing combat maneuvers and a figure of some importance, hence the +2 bonus. Plus it is the name of one of the abilities which explains the bonus itself.

"Student of war", original PBH -
"At 3rd level, you gain proficiency with one type of artisan's tools of your choice."

I chose this for the name of lvl 7 talent instead, as I'm again unsure if it would be useful during a (custom) campaign.
On lvl 3 talent I nerfed the 4 superiority dice (original PHB is 4) to 3 dice as +2 to Cha checks is always active, so in other words to compensate. Also on lvl 3 it should give 3 maneuvers and I chose 2. This is because I'm unsure how many of the suggested maneuvers would actually make it to live version, and I didn't want to have all of them available, but would rather want players to choose the ones they want for their build (so making strategic decisions, which is the idea of the whole class).
Naturally, lvl 7 and 9 give more maneuver options, which totals to 4, as opposed to 7 in PHB by lvl 10. Moreover, I'm honestly afraid Battlemaster if implemented as is from PHB, would overshadow most other martial (sub)classes from the sheer volume of options and damage increases it's got access to in the digital game (Solasta), that's why he is "nerfed" for the most part. This is just my reasoning and of course, can be changed to whatever the developers find suitable and most balanced.

"Know Your Enemy", original PHB -
"Starting at 7th level, if you spend at least 1 minute observing or interacting with another creature outside combat, you can learn certain information about its capabilities compared to your own. The DM tells you if the creature is your equal, superior, or inferior in regard to two of the following characteristics of your choice: Str, Dex, Con, AC, HP, total levels, fighter levels."

This is just a name I took for lvl 9 (but now that I think about it, it should be level 10 talent instead). Since I have no idea if it can even be implemented in the current gameplay as stated in PHB (or it's usefulness again), I just decided for it to give flat +2 survival, as it already helps you identify monsters you fight against. I think it's a quick and simple alternative to it.
Another thing about the original PHB lvl 10 talent: "Improved Combat Superiority" - changes all superiority dice from d8 to d10.  I think it'd turn Battlemaster into a complete dominator if all of his PHB buffs and abilities were implemented as is, so I doubt Champion or Mountaineer for example would ever even be considered if Battlemaster was released in it's full original glory.

And to top and possibly disregard all I've stated so far: I've played only a tiny bit with Tactician. It's somewhat similar to Battlemaster but somehow different as they offer different abilities (original pen & paper classes comparison).
Current Tactician's abilities are good (although I will test them further and provide better feedback some time later) but I think the class as is, is lacking that oomf to make it a good and most importantly, FUN choice of subclass. A couple of abilities added and it would shine without a single doubt.
With all of that being said, I believe Tactician's and Battlemaster's abilities could be merged to create a truly great class for Solasta. An alternative could be to add the original Tactician's abilities to his current arsenal and make that class even better, which would also be a much easier approach from the technical perspective as well.
I will make a couple of suggestions on Tactician's page in case any of the CE developers ever wishes to make/upgrade either of the two subclasses.

Now the feats:
"Charger" - As part of the Dash action you can make a melee attack with a +5 bonus if you move at least 10 ft before.

It's a unique feat and offers an additional way of engaging the enemy backlines.
"Defensive Duelist" - prerequisite: Dexterity 13 or higher. You can add your proficiency bonus to your AC if you are wielding a finesse weapon, in reaction to a melee attack.
Admittedly, this is similar to original Solasta's "Twin Blade" (When you are about to get hit by a melee attack while dual-wielding (not sword + shield), you can use your reaction to get +3 AC until the end of the attacker's turn.)
Although, the intended purpose behind this feat is to offer 1 handed finesse duelists at least one more option, as I find vanilla duelists to simply underperform in comparison to all other weapon choices/specializations. 2 handed weapons deal more damage, (cross)bow users shoot freely from afar, shield users have slightly less damage but much more AC. Dual wielders deal much more damage AND have access to "Twin Blade". So I wanted to give duelists a bit more options of building (versatility).
"Grappler" - You have advantage on attack rolls when grappling, and can try to restrained a creature grappled by you.
This could be simply changed to "You have advantage on Shove attempts". That's it. Would offer a unique "grappler" style of play, which the rest of your team could exploit.
"Sharpshooter" - Your ranged attacks ignore some cover, no disavantage at long range, and possibility to take -5 to hit for +10 on ranged damage.
Essentially, it's a "power attack" for ranged users. Numbers could be tweaked a bit (e.g. -3 for +6 damage), and the "No disadvantage at long range" could be implemented just as well.
"Skilled" - You gain proliciency with three skills or tools.
Self explanatory really. Provides additional skill check options. More options = more fun.
"Shield Master" - Attack also allows to shove, shield bonus to Dex. saving throws againts spells, and no 1/2 damage on successful saving throw.
I'd just give it the "Shield bonus also provides bonus to Dexterity saving throws. Successful dexterity saving throws negate all damage, instead of 1/2.

Thanks for taking the time to detail the changes, it is appreciated. It should help if someone takes on the subclass.
Personally I'd be more likely to work on the tactician as it already exists and is inspired by the battlemaster so serves a similar purpose but best of luck to whoever might take this on. I think it's current limitations are because it was created so early and had more limitations than we have now so more abilities could be added

Yep, agreed. Tactician should be updated instead, although before writing this suggestion, I didn't want to undermine any current or future efforts for that subclass, so I rather decided to make it as a separate suggestion.
Names could definitely use a makeover, or simply use the existing ones. But frankly, they would be the least of one's worries as the subclass itself would be a colossal undertaking.
I'd try to create the class myself, had I any programming knowledge...

Should the issue be converted to a discussion as it doesn't appear to be active at the moment and tracking suggestions was why discussions were enabled?


New font scaling system

b/269358835
Android is proposing new font scaling system. We should make sure flutter is aligned with them.

@chunhtai that link isn't a valid URL.

@mhmdanas The link is still confidential, and not ready to be public yet

lowering to P3, since we are targeting Q2 2023

linking PR: https://github.com/flutter/engine/pull/44907
Mostly feature complete, a few framework widgets have not been migrated.

Migration guide https://github.com/flutter/website/pull/9425

@LongCatIsLooong This issue was assigned the team-android label, but it looks like most/all of the work was handled in the framework so I changed the team label to team-framework. If you're done with the work, is this an issue that can be closed?


Json in 'log' field not parsed/exploded after migration from 0.12 to 1.2

fluentd or td-agent version: docker image fluent/fluentd-kubernetes-daemonset:v1.2-debian-elasticsearch
Environment information, e.g. OS: kubernetes 1.9
Your configuration: default
Your problem explanation. If you have an error logs, write it together.
I'm pushing microservices logs into Elasticsearch using fluentd. Our microservices are making json logs to stdout where Kubernetes catch them and stream to json files making 'json in json'. In version 0.12 those logs were automatically 'exploded' so I had possibility to easily filter by fields in those logs in Elasticsearch. In Fluentd 1.12 I see only json string in Kubernetes 'log' field. What happend?

Thank you @repeatedly for answer!
Here is Log:
{"log":"{\"Timestamp\":\"2018-06-11T14:44:57.3591704+00:00\",\"Level\":\"Warning\",\"MessageTemplate\":\"The LINQ expression '{expression}' could not be translated and will be evaluated locally.\",\"RenderedMessage\":\"The LINQ expression '\\\"where ([x] == [dummy].something)\\\"' could not be translated and will be evaluated locally.\"", "stream":"stdout","time":"2018-06-11T14:44:57.364059249Z"}
In 0.12 I was able to easily filter in Elasticsearch using for example Level field. Now after migration I cant as in Elasticsearch I see only "log" field with whole json string which I provided

We're having the exact same issue.
On 1.1.3 our JSON logs were being parsed correctly. Using 1.2.2, which fixes a stack trace problem we were having, no JSON log parsing is occuring.
Neither of the following configurations work for us on 1.2.x:
    <source>
      @id source_kubernetes_containers
      @type tail
      path /var/log/containers/*.log
      pos_file /var/log/fluentd/kubernetes-containers.pos
      read_from_head true
      tag kubernetes.*
      <parse>
        @type json
        json_parser yajl
        time_format %Y-%m-%dT%H:%M:%S.%NZ
      </parse>
    </source>

    <source>
      @id source_kubernetes_containers
      @type tail
      path /var/log/containers/*.log
      pos_file /var/log/fluentd/kubernetes-containers.pos
      read_from_head true
      tag kubernetes.*
      <parse>
        @type json
        time_format %Y-%m-%dT%H:%M:%S.%NZ
      </parse>
    </source>

I confirm that downgrade to 1.1.3 fix the problem

https://github.com/fabric8io/fluent-plugin-kubernetes_metadata_filter/pull/128
I found the root cause is fluent-plugin-kubernetes-metadata-filter removes automatic json merging feature since latest version.

Fix for that in new version can be achieved using
https://docs.fluentd.org/v1.0/articles/filter_parser
https://docs.fluentd.org/v1.0/articles/parser_json
as mentioned in https://github.com/fabric8io/fluent-plugin-kubernetes_metadata_filter/issues/79
Probably it can look like:
<filter foo.bar>
  @type parser
  format json
  key_name log
</filter>

@jcardoso-bv I'm running version 1.1.3 for now. Could not get json_parser plugin to work in 1.2.4.  Can I see the complete conf? Are you using the filter parser with the  you have above.

We're having the exact same issue.
On 1.1.3 our JSON logs were being parsed correctly. Using 1.2.2, which fixes a stack trace problem we were having, no JSON log parsing is occuring.
Neither of the following configurations work for us on 1.2.x:
    <source>
      @id source_kubernetes_containers
      @type tail
      path /var/log/containers/*.log
      pos_file /var/log/fluentd/kubernetes-containers.pos
      read_from_head true
      tag kubernetes.*
      <parse>
        @type json
        json_parser yajl
        time_format %Y-%m-%dT%H:%M:%S.%NZ
      </parse>
    </source>

    <source>
      @id source_kubernetes_containers
      @type tail
      path /var/log/containers/*.log
      pos_file /var/log/fluentd/kubernetes-containers.pos
      read_from_head true
      tag kubernetes.*
      <parse>
        @type json
        time_format %Y-%m-%dT%H:%M:%S.%NZ
      </parse>
    </source>

Im also having the same issue and tried couple of filter parser configs suggested here, but didn't work.. Any solution ?

FIX : https://github.com/fluent/fluentd/issues/2073#issuecomment-409469227


[TinyPNG] ...

Extension
https://www.raycast.com/kawamataryo/tinypng
Description
Iam getting this error when trying to compress
Steps To Reproduce
Opening raycast, searching tiny plugin, trying to compress

Current Behaviour
No response
Expected Behaviour
No response

Did you make sure that Finder is your frontmost application? The command works for me.


Lib/locale

Localization (L10N) Support Plugin for Transloco
close #62

Thank you @Edouardbozon and @armanozak for your review!! much appreciated 🙌🙌


[Snyk] Security upgrade tornado from 3.2 to 6.4.1

This PR was automatically created by Snyk using the credentials of a real user.![snyk-top-banner](https://github.com/andygongea/OWASP-Benchmark/assets/818805/c518c423-16fe-447e-b67f-ad5a49b5d123)
Snyk has created this PR to fix 2 vulnerabilities in the pip dependencies of this project.
Snyk changed the following file(s):

requirements.txt

⚠ Warning
```
magpie 0.1.0 requires tornado, which is not installed.
magpie 0.1.0 requires wsgiref, which is not installed.
celery 5.3.0 requires kombu, which is not installed.
</details>

---

> [!IMPORTANT]
>
> - Check the changes in this PR to ensure they won't cause issues with your project.
> - Max score is 1000. Note that the real score may have changed since the PR was raised.
> - This PR was automatically created by Snyk using the credentials of a real user.
> - Some vulnerabilities couldn't be fully fixed and so Snyk will still find them when the project is tested again. This may be because the vulnerability existed within more than one direct dependency, but not all of the affected dependencies could be upgraded.

---

**Note:** _You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs._

For more information: <img src="https://api.segment.io/v1/pixel/track?data=eyJ3cml0ZUtleSI6InJyWmxZcEdHY2RyTHZsb0lYd0dUcVg4WkFRTnNCOUEwIiwiYW5vbnltb3VzSWQiOiI3ZjNlNDg2OC02YmVmLTRjNGYtODZjYi1iN2Y2NzhhNTBlYWIiLCJldmVudCI6IlBSIHZpZXdlZCIsInByb3BlcnRpZXMiOnsicHJJZCI6IjdmM2U0ODY4LTZiZWYtNGM0Zi04NmNiLWI3ZjY3OGE1MGVhYiJ9fQ==" width="0" height="0"/>
🧐 [View latest project report](https://app.snyk.io/org/fmigneault/project/4692f721-89a0-4fd1-978b-285b018fe6d2?utm_source=github&utm_medium=referral&page=fix-pr)
📜 [Customise PR templates](https://docs.snyk.io/scan-using-snyk/pull-requests/snyk-fix-pull-or-merge-requests/customize-pr-templates)
🛠 [Adjust project settings](https://app.snyk.io/org/fmigneault/project/4692f721-89a0-4fd1-978b-285b018fe6d2?utm_source=github&utm_medium=referral&page=fix-pr/settings)
📚 [Read about Snyk's upgrade logic](https://support.snyk.io/hc/en-us/articles/360003891078-Snyk-patches-to-fix-vulnerabilities)

---

**Learn how to fix vulnerabilities with free interactive lessons:**

🦉 [Improper Neutralization of CRLF Sequences ('CRLF Injection')](https://learn.snyk.io/lesson/malicious-code-injection/?loc=fix-pr)

[//]: # 'snyk:metadata:{"customTemplate":{"variablesUsed":[],"fieldsUsed":[]},"dependencies":[{"name":"tornado","from":"3.2","to":"6.4.1"}],"env":"prod","issuesToFix":[{"exploit_maturity":"Proof of Concept","id":"SNYK-PYTHON-TORNADO-7217828","priority_score":718,"priority_score_factors":[{"type":"exploit","label":"Proof of Concept","score":107},{"type":"freshness","label":true,"score":71},{"type":"fixability","label":true,"score":214},{"type":"cvssScore","label":"6.5","score":325},{"type":"scoreVersion","label":"v1","score":1}],"severity":"medium","title":"Improper Neutralization of CRLF Sequences ('CRLF Injection')"},{"exploit_maturity":"Proof of Concept","id":"SNYK-PYTHON-TORNADO-7217829","priority_score":658,"priority_score_factors":[{"type":"exploit","label":"Proof of Concept","score":107},{"type":"freshness","label":true,"score":71},{"type":"fixability","label":true,"score":214},{"type":"cvssScore","label":"5.3","score":265},{"type":"scoreVersion","label":"v1","score":1}],"severity":"medium","title":"HTTP Request Smuggling"}],"prId":"7f3e4868-6bef-4c4f-86cb-b7f678a50eab","prPublicId":"7f3e4868-6bef-4c4f-86cb-b7f678a50eab","packageManager":"pip","priorityScoreList":[718,658],"projectPublicId":"4692f721-89a0-4fd1-978b-285b018fe6d2","projectUrl":"https://app.snyk.io/org/fmigneault/project/4692f721-89a0-4fd1-978b-285b018fe6d2?utm_source=github&utm_medium=referral&page=fix-pr","prType":"fix","templateFieldSources":{"branchName":"default","commitMessage":"default","description":"default","title":"default"},"templateVariants":["updated-fix-title","pr-warning-shown","priorityScore"],"type":"auto","upgrade":[],"vulns":["SNYK-PYTHON-TORNADO-7217828","SNYK-PYTHON-TORNADO-7217829"],"patch":[],"isBreakingChange":false,"remediationStrategy":"vuln"}'

invalid magpie reference, wrong package


irods_testing_environment.test_utils.run_plugin_tests does not use plugin_branch appropriately

Currently, testing plugins on a <4.3.0 version requires use of --test-hook-path because the plugin_branch option for test_manager.run is not using the plugin_branch option. We should detect the version of iRODS installed on the host and use this to determine the branch that we want to use. Alternatively, we could provide an option to specify a branch to use or even a git URL.

This may be blocking the ability to test plugins for 4.2.12.
For now, use the --test-hook-path workaround until we can get a fix for this.


'Color' is from 'dart:ui'.                   color: theme.backgroundColor ?? Colors.white,

/E:/Andriod/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_datetime_picker-1.5.1/lib/flutter_datetime_picker.dart:311:32: Warning: Operand of null-aware operation '??' has type 'Color' which excludes null.

'Color' is from 'dart:ui'.
color: theme.backgroundColor ?? Colors.white,

I also get this error every time execute flutter run. Happen after migrating to flutter channel master.

I have this error too.
flutter_datetime_picker-1.5.1/lib/flutter_datetime_picker.dart:311:32: Warning: Operand of null-aware operation '??' has type 'Color' which excludes null.

'Color' is from 'dart:ui'.
color: theme.backgroundColor ?? Colors.white,

Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, 2.0.2, on Microsoft Windows [Version 10.0.19041.928], locale tr-TR)
[√] Android toolchain - develop for Android devices (Android SDK version 30.0.2)
[√] Chrome - develop for the web
[√] Android Studio (version 4.1.0)
[√] VS Code (version 1.55.1)
[√] Connected device (3 available)
• No issues found!

I have this error too.

I get this error too.

This problem has already been fixed, but has not been released yet. Would be super nice with a quick release, as my release builds are failing at the moment!

Same here

me too, I have some error, pub-cache/hosted/pub.dartlang.org/flutter_datetime_picker-1.5.1/lib/flutter_datetime_picker.dart:311:32: Warning: Operand of null-aware operation '??' has type 'Color' which excludes null.

'Color' is from 'dart:ui'.
color: theme.backgroundColor ?? Colors.white,

I hope the fixed package will be released soon.
/opt/homebrew/Caskroom/flutter/2.2.2/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_datetime_picker-1.5.1/lib/flutter_datetime_picker.dart:311:32: Warning: Operand of null-aware
operation '??' has type 'Color' which excludes null.
 - 'Color' is from 'dart:ui'.
                  color: theme.backgroundColor ?? Colors.white,

this issue started to be really annoying, the funny thing is that it is fixed but not deployed

It would certainly be helpful if the project owner added collaborators to this project to keep it alive.

What can we do to have this released as a hot fix / minor update?, we're wrapping up a release and this is the lone outstanding warning on our runs/builds... we've been seeing it through the whole dev process patiently waiting for the committed fix to make it to a release.


ETVR v4 PCB Wiring diagram missing

the ETVR v4 PCB Wiring diagram is missing. Currently at the step of wiring everything together and i cannot figure out how to wire in the ETVR v4 PCB.

I unfortunately do not have any pictures of the underside of my vr board, but in summery:

Solder two wires to the "VIN" + and - pins
Feed them through the little hole that goes from the ETVR pcb area, to the USB hub area (you can see these purple/black wires in the writeup)
Solder these respective + and - wires to any 5v and GND pads on the USB hub. You can share the same pins any ESPs is using for power.

let me know if this helps and i can include it in the write up.

Perfect, will try once i receive the correct hub in the mail, i will take as many images as possible for you here

Ok, so im having an issue with the et r module of when the power switch is set to on, i get an error in windows mentioning theres a power surge and it exceeds the power limit of the hub. Any fix?

That means there is a short circuit or solder bridge somewhere between VUSB and Gnd. You should inspect all your soldered connections and make sure everything is soldered to the right places. Also make sure no solder or pieces of wire is touching places it shouldn't.

Ok, so i think i got it, but im not sure how to test to make sure that the LEDs and etvr board are working. Is there any way to test these to see if they appear?

I dont seem to see any leds.
On Mon, Dec 16, 2024 at 13:59 Physics-Dude @.***> wrote:

Look at the IR LEDs with your phone camera. you should see a pink
dot/light in each led. There are some examples in the EyeTrackVR Discord
server.
—
Reply to this email directly, view it on GitHub
https://github.com/Physics-Dude/Phys-Index-EyetrackVR-HW/issues/7#issuecomment-2546403776,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AU3UKD2IYS4NVA46Y7OM4WD2F4PH5AVCNFSM6AAAAABRKCNUJCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNBWGQYDGNZXGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

Check out the ETVR Discord server. The search bar can give you a trove of past fixes and pictures. and lots of people there can help.
https://discord.gg/kkXYbVykZX


MEDJHAMSKIE/KOTLIN/LIBUpdate multiplatform-getting-started.md

Developer/library/slacks.youtuber

Hey there,
It seems that your PR doesn't add value to the Kotlin Multiplatform Development docs. Thanks for your interest; I'll close this PR.


removed aws access/secret keys

@TailorDev this is a great repo for getting started with Lambda and Terraform. I just noticed some unnecessary bits and removed them: the aws access/secret keys.
If the AWS CLI is setup properly, Terraform will pull these credentials out of the environment. At that point there is no need to have users (new to AWS/Terraform) put their credentials in a repo that has the potential to be committed.

Nice!  I jumped the shark and started my project to pass libfoo.so files over the wire and execute them on Lambda: https://github.com/chadbrewbaker/lambda-dlopen
Why bother uploading your code into AWS Lambda when you can just pass the code you want to execute on demand?

Thanks for the PR. I'll take a look next week :)

This was a really long week.

Hey @mateusfccp you know that time may expand dramatically in Open Source... There is no place for such sarcasm here. With :heart:

@jmaupetit Sorry, man, I was only joking. No intention to offend.


Support for Text Selection

I am using LinkConsumableTextView and enabled setTextIsSelectable(true) for it in RecyclerView but as soon as I long-press on the LinkConsumableTextView it highlights the text being selected and immediately fires the onClickListener and jumps to the second screen which shouldn't be the case. Can you please help me to achieve the below mentioned functionality?
Functionality:

Long Press should enable text selection but shouldn't fire onClickListener().
Click (Not Links) should jump to next screen.
Clicking on links should function as it is.

I would imagine that when you long click the link and the text selection happens, the TextView loses focus, so the click event goes off. Hmm. I am not really sure how to solve this, admittedly. I know that you can long click any non-linked text and then drag to select the linked text that way, but that is different from what you are asking about.
In my apps, I just set up a long click listener for the link that brings up a contextual menu where one of the actions is copy. I feel like if the user is long clicking a link, they will see that it is highlighted and they would not expect it to react the same way as regular text. I think it is perfectly justified - and expected - to pop up some kind of AlertDialog or bottom sheet to handle the long click actions on the link. Your users will expect a different experience here, it isn't normal text that they are clicking, it is linked text.

Does that help you with your issue? I am going to close this, if so.

Your answer was quite helpful though it didn't solve my problem. I guess I will have to try few more alternatives to achieve what is needed or go with what you suggested. Deeply appreciate you help and quick response.


Generate API reference docs

We should generate API reference docs for various aspects of Kubeflow; e.g our custom operators, the Katib API, Pipelines API.
Here are instructions for how Kubernetes generates its reference docs. Looks like its based on OpenAPI specs.
https://kubernetes.io/docs/contribute/generate-ref-docs/kubernetes-api/
Related issues:
kubeflow/katib#120 - Document Katib API
kubeflow/tf-operator#731 - Document TFJob API

Here's the existing issue for the Pipelines reference: #345 - it includes some discussion.

We need to think about how we'll include the generated reference docs in our Kubeflow website.
As an example, see Kubernetes:

Has a “Reference” page within the website framework, listing the various reference docs: https://kubernetes.io/docs/reference/
Then branches out to a separate look and feel for the generated reference: https://kubernetes.io/docs/reference/generated/kubernetes-api/v1.13/

Useful discussion happening over here: https://github.com/kubeflow/tf-operator/issues/731

Docs for TFJob are now generated; kubeflow/tf-operator#731 is closed.
@sarahmaddox Is there more work to be done here or can we close this issue?

I'm not sure - are there any other APIs / CRDs we should generate references for? If so, perhaps we should create individual issues for those, and close this generic issue.

We now have a framework in place to hold generated references.
We've the created TFJob reference.
We have a separate issue for the Pipelines reference: https://github.com/kubeflow/website/issues/345

SGTM. Lets close this issue and open up additional specific issues as needed.


--wrap-length str vs int comparison

i'm getting an error when using --wrap-length= parameter on the command line tool
isort version: 4.3.21
i'm not sure if this has already been fixed in the develop branch, but should be a very quick fix if not.
  File "/virtualenv/bin/isort", line 10, in <module>
    sys.exit(main())
  File "/virtualenv/python3.6/site-packages/isort/main.py", line 379, in main
    for sort_attempt in attempt_iterator:
  File "/virtualenv/python3.6/site-packages/isort/main.py", line 377, in <genexpr>
    attempt_iterator = (sort_imports(file_name, **arguments) for file_name in file_names)
  File "/virtualenv/python3.6/site-packages/isort/main.py", line 88, in sort_imports
    result = SortImports(file_name, **arguments)
  File "/virtualenv/python3.6/site-packages/isort/isort.py", line 207, in __init__
    self._add_formatted_imports()
  File "/virtualenv/python3.6/site-packages/isort/isort.py", line 606, in _add_formatted_imports
    self._add_from_imports(from_modules, section, section_output, sort_ignore_case)
  File "/virtualenv/python3.6/site-packages/isort/isort.py", line 465, in _add_from_imports
    import_statements.append(self._wrap(single_import_line))
  File "/virtualenv/python3.6/site-packages/isort/isort.py", line 375, in _wrap
    while (len(line) + 2) > (self.config['wrap_length'] or self.config['line_length']) and line_parts:
TypeError: '>' not supported between instances of 'int' and 'str'

I appreciate the report!
This is fixed in develop and will make it's way in the 5.0.0 release.
Thanks!
~Timothy


Performance improvements for projection

Instead of rehydrating all CFDs whenever any CFD or the quote changes we now:

Only rehydrate the specific CFD whenever it changes
Keep the hydrated CFD around (this is basically an in-memory persistent read model)
Only apply the new quote to the CFDs whenever the quote changes

As a follow-up, we can eventually take apart the projection::Cfd struct into two structs:

The aggregate hydrated from all events
The struct that is actually serialized over the wire

bors r+


Multiline po file (question)

I have a po file:
msgid ""
"%user% acaba de responder tu comentario.
Consulta que te ha "
"dicho %link%aquí."
msgstr ""
"%user% acaba de respondre el teu comentari.
Consulta que t'ha "
"dit %link%aquí."

This is the result:
array(1) {
  '%user% acaba de responder tu comentario.
Consulta que te ha dicho %link%aquí.' =>
  array(2) {
    'msgid' =>
    array(3) {
      [0] =>
      string(0) ""
      [1] =>
      string(63) "%user% acaba de responder tu comentario.
Consulta que te ha "
      [2] =>
      string(22) "dicho %link%aquí."
    }
    'msgstr' =>
    array(3) {
      [0] =>
      string(0) ""
      [1] =>
      string(65) "%user% acaba de respondre el teu comentari.
Consulta que t'ha "
      [2] =>
      string(20) "dit %link%aquí."
    }
  }
}

Only the array key is a concatenated msgid. Is it intented?
Would not be better to concatenate all the things? Is there a case when we might need this "raw" form?

The array key is just there to be able to locate translations.
Say you want to get a given translation, and you know its key is "Hello user".
You can quickly get the translation with
$entries = $poparser->parse();
$myKey = 'Hello user';
if (isset($entries[$myKey])) {
    $entries[$mykey]['msgstr'] = 'Hola usuario';
}

In your case, where the msgid is multilined, you should first "join" the msgid to get the key that will let you reach the desired translation:
$msgid = array('Hello', 'user');
$msgid_key = implode('', $mgid);

if (isset($entries[$msgid_key])) {
$entries[$msgid_key]['msgstr'] = 'Hola usuario';
}

Okay, I see, thank you.

msgid "%s post restored from the Trash."
msgid_plural ""
"%s posts restored "
"from the Trash."
msgstr[0] ""
"%s entrada restaurada "
"desde la Papelera."
msgstr[1] "%s entradas restauradas desde la Papelera."

array(1) {
  '%s post restored from the Trash.' =>
  array(4) {
    'msgid' =>
    array(1) {
      [0] =>
      string(32) "%s post restored from the Trash."
    }
    'msgid_plural' =>
    array(3) {
      [0] =>
      string(0) ""
      [1] =>
      string(18) "%s posts restored "
      [2] =>
      string(15) "from the Trash."
    }
    'msgstr' =>
    array(4) {
      [0] =>
      string(0) ""
      [1] =>
      string(22) "%s entrada restaurada "
      [2] =>
      string(18) "desde la Papelera."
      [3] =>
      string(42) "%s entradas restauradas desde la Papelera."
    }
  }
}

Okay, now I have the proper example for my original problem.
I would like to concatenate the multiline translations in my msgstrarray. You can see 4 records there. How could I do that when I don't know where ends the first multiline translation?

I don't understand your question. If you want to concatenate the 'msgstr' content just do
implode('', $entries[$key]['msgstr']);

I don't fully understand your question, but if all you want to do is concatenate a set of data in an entry just use the php function implode:
$msgstr = implode('', $entries[$yourMsgID]['msgstr'];

'msgstr' =>
    array(4) {
      [0] =>
      string(0) ""
      [1] =>
      string(22) "%s entrada restaurada "
      [2] =>
      string(18) "desde la Papelera."
      [3] =>
      string(42) "%s entradas restauradas desde la Papelera."
    }

In the example above there are two translations: 0,1,2 and 3
I'd like to do concatenating individually on each translation line and I need two separate strings.

Ok now I see, there's plurals in there...
That's a bug then because the parser is mixing together every plural translation

Could you provide me a po file example?

msgid ""
"%s post restored from "
"the Trash."
msgid_plural ""
"%s posts restored "
"from the Trash 2."
msgstr[0] ""
"%s entrada restaurada "
"desde la Papelera."
msgstr[1] "%s entradas restauradas desde la Papelera."

Fixed on master.
Will port the fix to 5.0 branch too.


GPS/GNSS does not work

Using Lilygo T-A7670E board, GNSS does not work.
Tried the test method described here: https://github.com/Xinyuan-LilyGO/T-A7670X/issues/25
Result:
08:42:42.014 -> $GNGGA,064243.00,,,,,0,00,99.99,,,,,,7F
08:42:42.048 -> $GNGSA,A,1,,,,,,,,,,,,,99.99,99.99,99.99,133
08:42:42.117 -> $GNGSA,A,1,,,,,,,,,,,,,99.99,99.99,99.99,331
08:42:42.151 -> $GNGSA,A,1,,,,,,,,,,,,,99.99,99.99,99.99,230
08:42:42.220 -> $GPGSV,1,1,03,19,,,20,24,,,30,17,,,21,06C
08:42:42.253 -> $GAGSV,1,1,00,$GNRMC,064244.00,V,,,,,,,140423,,,N,V19
So the time is correct (but perhaps that is from the GSM side, as 4G is working) but there is no location.
Tried outside, waited for >1hr so it could get almanac, makes no difference.

same here this is the result for me
OK
AT+CGNSSPWR=1
OK
+CGNSSPWR: READY!
AT+CGNSSMODE=3
OK
AT+CGNSSNMEA=1,1,1,1,1,1,0,0
OK
AT+CGPSNMEARATE=2
OK
AT+CGNSSTST=1
OK
AT+CGNSSPORTSWITCH=0,1
OK
$GNRMC,061609.50,V,,,,,,,260423,,,N,V15
$GNGGA,061609.50,,,,,0,00,99.99,,,,,,75
$GNGLL,,,,,061609.50,V,N59
$GNGSA,A,1,,,,,,,,,,,,,99.99,99.99,99.99,133
$GNGSA,A,1,,,,,,,,,,,,,99.99,99.99,99.99,331
$GNGSA,A,1,,,,,,,,,,,,,99.99,99.99,99.99,230
$GPGSV,2,1,05,06,,,27,14,,,27,19,,,27,194,,,29,05A
$GPGSV,2,2,05,195,,,28,057
$GAGSV,1,1,00,074
$GLGSV,1,1,00,079
$GNVTG,,,,,,,,,N2E
$GNRMC,061610.00,V,,,,,,,260423,,,N,V18
$GNGGA,061610.00,,,,,0,00,99.99,,,,,,78
$GNGLL,,,,,061610.00,V,N54
$GNGSA,A,1,,,,,,,,,,,,,99.99,99.99,99.99,133
$GNGSA,A,1,,,,,,,,,,,,,99.99,99.99,99.99,331
$GNGSA,A,1,,,,,,,,,,,,,99.99,99.99,99.99,230
$GPGSV,2,1,05,06,,,30,14,,,26,19,,,29,194,,,32,059
$GPGSV,2,2,05,195,,,28,057
$GAGSV,1,1,00,074
$GLGSV,1,1,00,079
$GNVTG,,,,,,,,,N2E

Please confirm that the following two commands have been run before running GPS positioning, it will turn on the power supply of the GPS antenna, otherwise GPS positioning will be invalid

AT+CGDRT=4,1
AT+CGSETV=4,0

After executing the above command, check whether there is 2.5~3V power supply at the antenna end.

Positioning must be done outdoors, and positioning cannot be performed indoors.

Yes! there is indeed voltage. I read about 3.06V but the output in the serial monitor is the same as before.

If all the conditions are met and it is still impossible to locate, you can consider whether there is an antenna issue. Do you have any other antennas to try? Being able to output valid NMEA sentences indicates that GNSS is working normally

If you still have problems, please reopen the issue


Idea of snippet feature enhancement

Code snippets are templates that make it easier to enter repeating code patterns, such as loops or conditional-statements.

Above is copied from VSCode documentation website. Yes, we do need to enter repeating code patterns from time to time. But sometime, we also want to create files in our project and this is a part of the repeating pattern. For example, when I have a snippet like this:
{
    "link_to_css_file": {
        "prefix": "csssrc",
        "scope": "html",
        "body": [
          "<link href=\"${1:main}.css\" rel=\"stylesheet\" media=\"screen\">"
        ],
        "description": "Link to a css file"
    },
}

When I type csssrc and press Tab, the above snippet is inserted. But remember this repeating pattern doesn't finished, and the next step I will do is probably manually create a file called main.css. My idea of this snippet feature enhancement just comes from this situation. So my suggestion is, we can add some special comments in the snippet like this:
{
    "link_to_css_file": {
        "prefix": "csssrc",
        "scope": "html",
        "body": [
          "<link href=\"${1:main}.css\" rel=\"stylesheet\" media=\"screen\">"
        ],
        "description": "Link to a css file"
    },
}

//<file start>, name: $1.css
// body {
//     margin: 0
// }
//<file end>

By using //<file start> and //<file end>, vscode knows that a file need to be generated and its name is $1.css. By the end of the snippet expanding process, we have the value of $1 so the file name may be something like main.css and vscode will create it at the root folder of the project. We can specify the generated file is in a specific folder too like this.
{
    "link_to_css_file": {
        "prefix": "csssrc",
        "scope": "html",
        "body": [
          "<link href=\"${1:main}.css\" rel=\"stylesheet\" media=\"screen\">"
        ],
        "description": "Link to a css file"
    },
}

//<file start>, name: stylesheets/$1.css
// body {
//     margin: 0
// }
//<file end>

vscode notice the folder stylesheets doesn't exist so it will be  created automatically and the file $1.css will be placed into it.
We can create multiple file sections in a snippet. Let's say we have this snippet:
{
    "link_to_css_file": {
        "prefix": "csssrc",
        "scope": "html",
        "body": [
          "<link href=\"${1:main}.css\" rel=\"stylesheet\" media=\"screen\">",
          "<script type=\"text/javaScript\" src=\"js\${2:main}.js\"></script>"
        ],
        "description": "Link to a css file"
    },
}

//<file start>, name: stylesheets/$1.css
// body {
//     margin: 0
// }
//<file end>

//<file start>, name: js/$2.js
// var that = this;
//<file end>

Here when we filled $1 and $2, vscode will insert the link and script tags into our text document, it will also create two folders stylesheets and js. A file named main.css with the specified content will be created in the folder stylesheets and a file named main.js will be created in the folder js.
For blob objects like an image file, we can also add them with base64 encoded text string like this.
{
    "link_to_css_file": {
        "prefix": "csssrc",
        "scope": "html",
        "body": [
          "<link href=\"${1:main}.css\" rel=\"stylesheet\" media=\"screen\">",
          "<script type=\"text/javaScript\" src=\"js\${2:main}.js\"></script>"
        ],
        "description": "Link to a css file"
    },
}

//<file start>, name: stylesheets/$1.css
// body {
//     margin: 0
// }
//<file end>

//<file start>, name: js/$2.js
// var that = this;
//<file end>

//<blob start>, name: image/background.png
// iVBORw0KGgoAAAANSUhEUgAAABQAAAAVCAYAAABG1c6oAAAACXBIWXMAAA7EAAAOxAGVKw4bAAAAB3RJTUUH4ggNAQUUMy3JOgAAAAd0RVh0QXV0aG9yAKmuzEgAAAAMdEVYdERlc2NyaXB0aW9uABMJISMAAAAKdEVYdENvcHlyaWdodACsD8w6AAAADnRFWHRDcmVhdGlvbiB0aW1lADX3DwkAAAAJdEVYdFNvZnR3YXJlAF1w/zoAAAALdEVYdERpc2NsYWltZXIAt8C0jwAAAAh0RVh0V2FybmluZwDAG+aHAAAAB3RFWHRTb3VyY2UA9f+D6wAAAAh0RVh0Q29tbWVudAD2zJa/AAAABnRFWHRUaXRsZQCo7tInAAACJklEQVQ4jc2Tz2sTURDHP0naYrKNu0mbghTlHawac7PQHAL2GKUNHirYQ6XgqV7q2Z56qYj4LyT/Q9JAIP0LjNWDNQqthx5sQVJouuYl0k3yPLQb8nNT8OIXBvbNm/kwM2/HZZqmajSaGIZOu8rlMyqV39T+1PBe86LrOn6/v3Uvq1WajQbdcimlFIBlWeRyOT4Wdinu7XH06xhpSmRNonk1tOsa4uYt7twN8+hxnFgshp0nZRWPx31BVEqpVDKlIuGIAjosNBlqWffd4kJCpdNpZcs0TXV6WlauueicKnwoACCE6GlhkA4PDwFYXEjw7v1bwvfuX7QMqEEgWZGUTkodvu5YG/xma4vXGxt4IuHIplW3+lag+TSeLi2RSDzhwewsTdWkWCwyOjJKQA9g1S2mb0wDsJ3dplFvghBCtZs9y/mH8+rb96Jq1/n5uUolU625tue15twNtGFOSqfT6nJUPbmuy4+OVvcP9pm5PeP4KPF4nHw+3zFTzavh7oYtLiSGwgCWny33+GRNdgIBgkFjKAxg3Ofr6+8Bftr9fCXgl6/Fvn6PYRib9sEwDA5+HDARnCAajQ6EyWqV5ysraD6NsbEx5wpDkyHWX62TyWQGAl+srlI6KTEVmnKuECCgBwBIppKcnZn4/eO43W6Ojn+SzWRZe7lGfmcHIQT9FqLnt2mXvVZCiI41dNp5R6AtWZEAaOPasFBGhkZcEWSr51H+Vf8/8C8zyyrm8/kxHgAAAABJRU5ErkJggg==
//<blob end>

Here when we insert the snippet, the folder image will also be created and the image file background.png will be created in the folder and will be filled using the the base64 data.
Hope you can consider my idea.  Thanks.

Closing as dupe of #10561


adding synonyms to exposure receptor

Including two related synonyms to the term exposure receptor per a request on ECTO.
https://github.com/EnvironmentOntology/environmental-exposure-ontology/issues/165
No option to select broad synonym so included 'has_related_synonym'
Hopefully this looks better @matentzn ?

This is generally ok, but arent these synonyms exact?

@diatomsRcool @matentzn Great, I've updated to exact. I do not have write privileges for ExO currently. Are either of you able to grant me write access and/or to approve and merge this PR?

I dont!

I think the synonym types are fine in this case, and I've merged the commits.


Add service account section

I want to merge this change because it implements service account section allowing users to add bots 🤖
Screenshots

Pull Request Checklist

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

Account's permissions are not loaded, I can only select "full permissions" option:


feat(staking): add default sorting for  browse pools [LW-10104]

Checklist

[X] JIRA - LW-10104
[-] Proper tests implemented
[-] Screenshots added.

Proposed solution
Utility function for default sorting exposed in BrowsePools feature.
On List view header click - if changing sort file, then using the new default sort order, otherwise reverse order.
On Sorting list (right-side) - if click on sort field - use the default sort order.
Legacy Staking supported as well
Testing
Please test legacy staking (hardware wallets)
Screenshots
N/A

coverage: 51.421%. remained the same
when pulling 71310135f5994f1d8ff788b3612bf12dc686dc53 on feat/LW-10104-staking-browse-pools-default-sort-order
into 5637f14e62d7164fda3c3ba13df39d753ba0b91f on main.


1.16.1 update

Notes:

The locate command works, no need for the mixin.
The access widener might be removable with Shadow but I couldn't make it work.
Moved the start into the feature, and the pieces to the piece to make it easier to read.
A new branch 1.16.1 should be created, so that this doesn't interfere with the 1.15 version.
Fixed issues saving, present in 1.15 for some reason.
Spawn rate of the structure is 1 in 150 and y position is 80 now. Easier to find that way.

Thank you~!


ci: update error reference docs

This PR is auto-generated by a nightly GitHub action to update the error reference docs from source code in withastro/astro.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File
Note

en/reference/error-reference.mdx
Source changed, localizations will be marked as outdated.

en/reference/errors/actions-without-server-output-error.mdx
Source changed, localizations will be marked as outdated.

en/reference/errors/adapter-support-output-mismatch.mdx
Localization added, will be marked as complete.

en/reference/errors/content-collection-type-mismatch-error.mdx
Source changed, localizations will be marked as outdated.

en/reference/errors/content-entry-data-error.mdx
Localization added, will be marked as complete.

en/reference/errors/content-loader-invalid-data-error.mdx
Localization added, will be marked as complete.

en/reference/errors/content-schema-contains-slug-error.mdx
Source changed, localizations will be marked as outdated.

en/reference/errors/env-invalid-variables.mdx
Source changed, localizations will be marked as outdated.

en/reference/errors/forbidden-rewrite.mdx
Localization added, will be marked as complete.

en/reference/errors/get-static-paths-required.mdx
Source changed, localizations will be marked as outdated.

en/reference/errors/i18n-not-enabled.mdx
Source changed, localizations will be marked as outdated.

en/reference/errors/invalid-content-entry-data-error.mdx
Localization added, will be marked as complete.

en/reference/errors/invalid-content-entry-frontmatter-error.mdx
Source changed, localizations will be marked as outdated.

en/reference/errors/invalid-content-entry-slug-error.mdx
Source changed, localizations will be marked as outdated.

en/reference/errors/locals-reassigned.mdx
Localization added, will be marked as complete.

en/reference/errors/mixed-content-data-collection-error.mdx
Source changed, localizations will be marked as outdated.

en/reference/errors/no-adapter-installed-server-islands.mdx
Localization added, will be marked as complete.

en/reference/errors/no-adapter-installed.mdx
Source changed, localizations will be marked as outdated.

en/reference/errors/only-response-can-be-returned.mdx
Source changed, localizations will be marked as outdated.

en/reference/errors/prerender-client-address-not-available.mdx
Source changed, localizations will be marked as outdated.

en/reference/errors/static-client-address-not-available.mdx
Source changed, localizations will be marked as outdated.

en/reference/errors/unknown-content-collection-error.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

Also noting that I have changed some error messages locally in the v5.0 PR that this will try to overwrite.
Since I don't expect any actual new error messages before v5 is released, and the v5 docs branch is ready for launch as-is with overwrites my plan is:

All remaining v5-beta PRs are merged, which may involve more error message overwrites if link have to change.
v5 is released, v5 docs are published. The error messages in docs will all be correct.
THEN I will run the action again so this will be updated. All the "desired changes" from the switch from next to main will no longer show up as differences and will go away from this PR.
THEN I will use this PR as my reference to make changes to make a PR to error messages at the source. Whatever is left standing in this PR should only be things I've overwritten locally, the way I want them. This PR should still not be merged, and live docs will still be fine.
THEN after that PR is merged, I will run the action again and if all is well, this PR should close itself. If not, return to Step 3.


Bug: Fix texture size limit issue in GreasedLinePluginMaterial

Previously, when the number of lines exceeded 8192, rendering issues occurred due to the 16384x1 texture size limit. This commit resolves this by dynamically adjusting the texture's dimensions based on the number of colors and the maximum texture size supported by the engine. This change ensures robust handling of large color sets by distributing colors across multiple rows in the texture when necessary, preventing overflow and maintaining correct color rendering.

Here is an example where it breaks:
https://playground.babylonjs.com/#H1LRZ3#512

Snapshot stored with reference name:
refs/pull/15031/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15031/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15031/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15031/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15031/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15031/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15031/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15031/merge#BCU1XR#0

WebGL2 visualization test reporter:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15031/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
Important - these might fail sporadically. This is an optional test.
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15031/merge/testResults/webgpuplaywright/index.html

Visualization tests for WebGL 1 have failed. If some tests failed because the snapshots do not match, the report can be found at
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/15031/merge/testResults/webgl1/index.html
If tests were successful afterwards, this report might not be available anymore.


Please update the tag with latest commit

currently go get is pulling v1.0.0. This tag excludes the recent add of the Delete function.

updated.


CustomRequestBodyValidator - return problem() does not work, we have to raise an exception - BadRequestProblem.

This is what worked for me finally.. return problem() does not work for some reason. we have to raise an exception.
class CustomRequestBodyValidator(RequestBodyValidator):
def validate_schema(self, data, url):
    if self.is_null_value_valid and is_null(data):
        return None
    try:
        self.validator.validate(data)
    except ValidationError as exception:
        LOG.error(
            "{url} validation error: {error}".format(url=url, error=exception),
            extra={"validator": "body"},
        )
        # return problem(400, "Bad Request test", str(exception))        DOES NOT WORK
        raise BadRequestProblem("Bad Request test", str(exception))    WORKS

    return None

app = connexion.App(name)
app.add_api(
"app.yaml",
strict_validation=True,
validate_responses=True,
validator_map={"body": CustomRequestBodyValidator},
)
Originally posted by @bhargaviurs in https://github.com/zalando/connexion/issue_comments/706469183

Hi @bhargaviurs I don't see why this is an issue. Feel free to reopen with more information if you think it is.


Develop 2.3.0

new features memo

add more examples

updates

updated EasyMetalShader
updated FontVertexBuilder

resolved issues

app crashes when rendering complex SVG files

You are the Man!

@TarasShu
Sorry for the late reply! Thank you for creating the issue! #135


Global flags --debug, --cache-dir not working

Discussed in https://github.com/aquasecurity/trivy/discussions/7319

Originally posted by aorakzai August  8, 2024
Description
Global flags --cache-dir, --debug not working on some platform and in docker
Desired Behavior
trivy --cache-dir "/tmp/trivy" aws

Should not fail
Actual Behavior
But it rather gives the following output
2024-08-08T12:26:09+02:00	FATAL	unknown flag: --cache-dir	<nil>

If I try to run with --debug flag then it complain about --debug flag
2024-08-08T12:32:27+02:00	FATAL	unknown flag: --debug	<nil>

Reproduction Steps
1. install using brew install trivy
2. install aws plugin trivy plugin install github.com/aquasecurity/trivy-aws
3. run trivy --cache-dir "/tmp/trivy" aws
...

Target
None
Scanner
None
Output Format
None
Mode
None
Debug Output
2024-08-08T12:32:27+02:00	FATAL	unknown flag: --debug	<nil>

Operating System
macOS Sonoma
Version
Version: 0.54.1
Vulnerability DB:
  Version: 2
  UpdatedAt: 2023-09-26 12:19:21.015897072 +0000 UTC
  NextUpdate: 2023-09-26 18:19:21.015896472 +0000 UTC
  DownloadedAt: 2023-09-26 18:06:08.173976 +0000 UTC
Check Bundle:
  Digest: sha256:24b38cdf646f0e5becf55a709ae9a3c4e819a348c28990cec0b6aabe4637d8b1
  DownloadedAt: 2023-10-06 11:07:09.944315 +0000 UTC

Checklist

[X] Run trivy clean --all
[X] Read the troubleshooting

looks like this issue is still not fixed I am getting the same issue error on version 0.56.1 and 0.55.2 have not tried other versions.

Hi @aorakzai ! I'll take a look

@aorakzai We have not yet released a new version with this fix.
/cc @simar7

Released https://github.com/aquasecurity/trivy-aws/releases/tag/v0.14.2


dosc: move glossary and add some more details about vulnerabilities and advisories

@ctron : Thanks for tagging me for a review!  I will try to review this before the end of the week.

@ritz303 thanks for the review, learned a few more things :) I pushed some changes, I hope I covered all of it.

I marked it "merge when ready", so if you're ok with it, give a +1 and it should merge, even when I'm already in :christmas_tree: mode :wink:

@ctron : I just found one more minor nit-pick, regarding capitalization, but overall this doc looks much better.  Thank you for making these updates.


png.Reader not guessing bytes

The docs says Reader.init takes bytes but fails on the isinstance(some_bytes, array) check.
Is this expected behaviour?
>>> import png
>>> png.Reader("tests/doop.png")
<png.Reader object at 0x7fd48d3cf100>
>>> with open("tests/doop.png", 'rb') as f:
...     png.Reader(f.read())
...
Traceback (most recent call last):
  File "<stdin>", line 2, in <module>
  File "/home/calum/.cache/pypoetry/virtualenvs/hsi-ZCFKj1YC-py3.10/lib/python3.10/site-packages/png.py", line 1375, in __init__
    raise ProtocolError("expecting filename, file or bytes array")
png.ProtocolError: ProtocolError: expecting filename, file or bytes array

The design predates the type bytes (it's from early Python 2, and possibly Python 1.5.2), so it only works on array.array("B").
However, i can see that png.Reader(f.read()) could be useful, so i shall consider this a bug.
png.Reader(f) should work by the way; although it does then become unclear when you should close f.


Added zookeeper logging

Added logging receiver for Zookeeper.
A single receiver that supports two log formats:

Log format for installing via apache tar
Log format for installing via Debian/Ubuntu package manager

Log formats differ by a single myid field that is not present in package manager install.
Sample log example:
{
  "insertId": "10wjfxvf48fn8v",
  "jsonPayload": {
    "thread": "NIOWorkerThread-2",
    "source": "NIOServerCnxn",
    "line": 514,
    "message": "Processing mntr command from /0:0:0:0:0:0:0:1:60632\n",
    "myid": 1
  },
  "resource": {
    "type": "gce_instance",
    "labels": {
      "zone": "us-central1-a",
      "project_id": "otel-agent-dev",
      "instance_id": "119216823488650460"
    }
  },
  "timestamp": "2022-02-09T12:33:31.711Z",
  "severity": "INFO",
  "logName": "projects/otel-agent-dev/logs/zookeeper",
  "receiveTimestamp": "2022-02-09T12:34:32.231089946Z"
}

Also changed debian install script to use package manager to cover both logging cases.

Adding @jefferbrecht for reverse shadow.


Adding GKMIMS to jetbrains college list

Providing college URL - "https://www.gkmims.net.in/b.sc.php"

@krish007S Pull request merged. Thank you!


Maintenance Windows

Doc should define what the impact is if there are now MWs configured for the endpoint.  In reference to:

The following settings must be shorter in duration than the shortest maintenance window applied to the computer:
Display a temporary notification to the user that indicates the interval before the user is logged off or the computer restarts (minutes)
Display a dialog box that the user cannot close, which displays the countdown interval before the user is logged off or the computer restarts (minutes)

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

ID: c3f43b73-de34-a8b0-13c6-c647724161e0
Version Independent ID: 39e43e5b-626b-b5b9-67de-f7f7a39950cd
Content: Device restart notifications - Configuration Manager
Content Source: sccm/core/clients/deploy/device-restart-notifications.md
Product: configuration-manager
Technology: configmgr-client
GitHub Login: @mestew
Microsoft Alias: mstewart

Thanks again for the feedback. We recently moved this content to a new GitHub repository, so this issue is no longer tracked with the original article. As part of that process, we reviewed our backlog of feedback, and made some tough decisions on items that we don't think we'll realistically ever address. If you're passionate about this specific feedback, please refile it on the current article, now under docs.microsoft.com/mem/configmgr.
These settings are explained else where in the article, this bit just calls out the necessary configuration if you're using maintenance windows.


Teams Fx Toolkit Migrated Samples

Executed Build PipeLine : https://github.com/OfficeDev/Microsoft-Teams-Samples/actions/runs/5654136544


Fix test, migrate to mocha, add travis

Please merge #154, https://github.com/uwdata/vega-lite-ui/pull/15, and publish vlui 0.7.8 first

Fixes #146, #150.
also fixes https://github.com/uwdata/vega-lite-ui/issues/14

@domoritz — FYI, this issue and #157 are now passing!

Yeah, sorry. I started fixing things but the app still didn't work for me (could not find cp.gen). Does everything work for you?

Have you tried (a) bower unlink and reinstall, or (b) pull the latest compass and build) ?

Yes. Also, I get the feeling that the order of link is important.
Still getting errors.

It should be working now.
Most of the problems are from versioning.
Compass 0.5.4 release was originally broken with missing compass.js.  I have just updated the tag.
Please try
git pull
bower uninstall viscompass
bower clean cache
bower install # to install latest version of datalib and viscompass

Then it should work now.


Chore: Migrate Nx 14 to 17, refine integrating admin-ui & ui packages

Description
For integrating Accio's admin-ui & ui in VulcanSQL, we need to refine the following issues:

installing the missing packages
fixing the Nx configurations, lint & unit test.

Since the Accio repository is using Nx 16 and VulcanSQL is using Nx 14, there are some differences between these two versions' configuration templates, I decided to upgrade VulcanSQL & Accio's Nx version, migrating the Nx version from 14 to 17.
Issue ticket number
closes N/A
Additional Context

The extension-driver-ksqldb packages also have to align the project.json configurations with other packages.

@andreashimin LGTM, thanks for your hard work!


Add license attribute

https://docs.npmjs.com/files/package.json#license
http://npm1k.org/

@benjamn Could you please accept this PR? It helps with automated dependency license checking (e.g. nlf). Thank you. :bow:


Correct link to pull request 1801 in changelog

Thanks for the PR! Someday, somebody will get around to making those changelog links automatic... :)

Thanks for the PR! Someday, somebody will get around to making those changelog links automatic... :)

yeah, I only noticed because I wanted to look at the pr and ended up on a random issue


web-sys 0.3.57 is missing bindings for MediaStreamTrackProcessor

Describe the Bug
Missing bindings for MediaStreamTrackProcessor on web-sys 0.3.57 after running cargo with
features: ["MediaStreamTrackProcessor"]
and
--cfg=web_sys_unstable_apis
Steps to Reproduce
I recorded a whole video about yew and this came up exactly at
https://youtu.be/In09Lgqxp6Y?t=940

Expected Behavior
I should be able to import MediaStreamTrackProcessor and use it in my YEW app.
Actual Behavior

Additional Context
I installed web-sys via cargo patch by cloning this repo and everything worked.

Hello!
I have faced with the same issue.

MediaStreamTrackProcessor was added with commit
Added MediaCaptureTransform API
This commit were after labling with 0.2.80 of wasm-bindgen / web-sys - 0.3.57. So will be available in 0.2.81 and is already available in main.

got it, thank you so much!!!


Add a Carousel in the Meet our Member section

It would be better to add a carousel-slider in the About Us, Meet our Members section.
The profile images in the About us section are also stretched up, so using object-fit would be a great step.

attach sample,or ur work should be up to the mark,,then i will assign to u

@Osman-Rasooli Are you a Social Summer of Code participant?

Not actually, I was involved in family business for a while, so I had to
skip programming for a while. Now, I'm back to development.

—
Reply to this email directly, view it on GitHub
https://github.com/Susmita-Dey/Sukoon/issues/357#issuecomment-1241559595,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2DZ7ODLKUNPOFEO6LVCLR3V5LLKDANCNFSM6AAAAAAQCVYDCE
.


what about the loss ?

Can you share your loss log? thanks

Hi.
Thanks for your interest in my repo.
I don't have log files for loss anymore.
I'm sorry I couldn't help.


cookbook site install : tar error on windows

tar.exe  bundled with chef-dk on windows don't support the --force-local option

STDERR: tar: Option --force-local is not supported

fixes : #4862

Yeah, we ship BSD tar in embedded/bin with the omnibus msi
Unfortunately, this undos the 'fix' in 12.6 for gnu tar which deliberately added this flag on windows.
I believe we're caught in some warfare here between people who have put embedded/bin in their $PATH and then added a GNU tar in front of that and people who have not.
I think we need another fix to path_sanity to make it a bit more assertive and fix chef-client's PATH so that embedded/bin is always first and you can't opt-out of it any more.
Of course that'll get into a fight with people who have deliberately put BSD tar first in their PATH and then expect execute resources and such to pick it up -- so we probably need to enforce in shell_out that embedded/bin comes first, while in shell_out_with_systems_locale that embedded is not added (or that we don't apply any PATH fixes at all).

I don't know if we hit merge on this one and move forwards or not, the yak here goes exceedingly deep...
The existing code, though, is particularly bad because without both:

adding embedded/bin to the path so that path_sanity will not add it first
adding gnu tar in front of that to the path so that gnu tar is picked up first

Then knife cookbook site install is broken on windows.
This is the correct fix for a vanilla chef/chef-dk install on windows.
So I think we need to merge this, and break everyone that was 'fixed' by the change in 12.6

Alternatively we could stop using an external tar binary and use the support baked in to Ruby: https://github.com/poise/poise-archive/blob/master/lib/poise_archive/archive_providers/tar.rb as an example. For the file sizes involved in cookbook sharing, it should be fine.

Do we use the included tar.exe for anything other than supermarket downloads?

Dunno.  I do think we need to sort out the embedded path issue.  But converting to pure-ruby tar where we can also makes a hell of a lot of sense to just side step all these issues...

Indeed, i think this is a good opportunity and a good idea to use native tar.rb.
However i believe this will be a lot of work to do just to make happy the people who fiddle with the $PATH too much.
From my own experience, using cygwin/babun gave me such headaches that i switched back to the old batch scripts under cmd.exe
Could it be possible to merge the fix anyway ?
Thanks in advance

The problem is that if we merge this we fix you, but we break the person who submitted the PR to 12.6.0 to add this.
So we're stuck with flipping a coin to decide which set of users to break.
In that case, probably not.  Particularly when using pure-ruby tar will fix both use cases.

I have made more research and i found a possible alternate solution.

tar.exe bundled with chef-dk on windows doesn't support the option --force-local
tar.exe in cygwin or mingw32 (in git for windows) support the option --force-local

i'm not sure, but could it be possible to "update" tar.exe bundled in chef-dk to a version compatible with --force-local ?
Using the --force-local option for fix an incorrectly escaped $path is not the right solution, but hopefully, someone already fixed escaping in a previous release.
See man tar :

--force-local
Forces tar to interpret the file name given to --file as a local file, even if it looks like a remote tape drive name.

Maybe the good solution was to remove --force-local and upgrading tar.exe

Anyone that puts git for windows in front of the chef-dk embedded path will wind up with a tar that doesn't support --force-local and will be broken.  That is likely the exact set of users I'm talking about would be broken.

And if we remove --force-local then its the other way around.   That will work with the existing chef-dk tar.exe but anyone that puts a GNU tar.exe first in the path will break.  We definitely wouldn't want to switch the tar.exe in chef-dk in that case since those changes would break it again.

I understand, but for the moment the tar.exe embedded with chef-dk DON'T support --force-local making 'knife cookbook site install' unusable without installing another tar.exe and fiddling with the windows path

As a short term fix we could remove the option and use the absolute path to the included tar executable.

Gentlemens, i have made some changes.
With this new version, tar is usable with both versions of tar.exe
Embeded (Bsd) tar

D:\Chef\chef-repo> knife cookbook site install httpd
Installing httpd to D:/Chef/chef-repo/cookbooks
Checking out the master branch.
Creating pristine copy branch chef-vendor-httpd
Downloading httpd from Supermarket at version 0.3.6 to D:/Chef/chef-repo/cookbooks/httpd.tar.gz
Cookbook saved: D:/Chef/chef-repo/cookbooks/httpd.tar.gz
Removing pre-existing version.
Uncompressing httpd version 0.3.6.
BSD tar detected
Removing downloaded tarball
1 files updated, committing changes
Creating tag cookbook-site-imported-httpd-0.3.6
Checking out the master branch.
Updating 49c1f1b..6a2a0dd
Fast-forward
cookbooks/httpd/CHANGELOG.md                       | 153 ++++
[snip]
Cookbook httpd version 0.3.6 successfully installed

Github/gitforwindows/cygwin (Gnu) tar

D:\Chef\chef-repo> knife cookbook site install java
Installing java to D:/Chef/chef-repo/cookbooks
Checking out the master branch.
Creating pristine copy branch chef-vendor-java
Downloading java from Supermarket at version 1.39.0 to D:/Chef/chef-repo/cookbooks/java.tar.gz
Cookbook saved: D:/Chef/chef-repo/cookbooks/java.tar.gz
Removing pre-existing version.
Uncompressing java version 1.39.0.
GNU tar detected
Removing downloaded tarball
1 files updated, committing changes
Creating tag cookbook-site-imported-java-1.39.0
Checking out the master branch.
Updating 6a2a0dd..31b1530
Fast-forward
cookbooks/java/.gitignore                          |  47 +++
[snip]
Cookbook java version 1.39.0 successfully installed

I think printing the current "version" of tar will be helpfull for users
feel free to test with many version of tar for windows as possible, i have only found 2 distinct versions for the moment

Not sure why github things that was an outdated diff comment, but ^^ we can't merge this as is.

@willoucom I'm the one who implemented the previous PR, and I had stumbled into this problem a few days back. You beat me to the correct implementation which handles both case, by running tar first to look if it will support that option. I support the changes made in 9856652

i have corrected the backticks

@MichaelPereira thanks for your support

👍  on the approach, just a few nits

👍 after nit eradication

Gentlemens, thanks a lot for your time

oh i guess there's no ui.debug sorry, i think you want Chef::Log.debug


#merge doesn't seem to work with pg_search_scope's

Ok, let's dive right into code!  :)
I have an Event (lots of stuff removed to hide my sins)
class Event < ActiveRecord::Base
  include PgSearch

  has_one  :location, class_name: 'Where'

  #where
  scope :search_for_location,
    ->(location) { joins(:location).merge(Where.search_for_address(location)) }

end

An Event has a who, what, when, where, why.
So here is the VERY POORLY NAMED Where
# Looking back on it, this is probably the poorest name I could have chosen
# TODO: Refactor this to be Location, or some other noun
class Where < ActiveRecord::Base
  include PgSearch
  attr_accessible :address #, :latitude, :longitude

  belongs_to :event

  geocoded_by :address

  after_validation :geocode, :if => :address_changed?
  before_validation :standardize_address, :if => :address_changed?

  validates :event_id, uniqueness: true, presence: true

  pg_search_scope :search_for_address, against: :address,
    using: {
      tsearch: {
        dictionary: "english",
        tsvector_column: 'where_tsv'
    }
  }

  private

  # This is just a place holder to make things look nice
  # at some point we will probably want to actually have
  # a real address standardizer, but for now let just be
  # happy to have the address `titleized`
  def standardize_address
    self.address = self.address.titleize
  end
end

And a Where's migration to make it really fast ;)
class AddIndexForLocationToEventForFullTextSearch < ActiveRecord::Migration
  def up
    execute "ALTER TABLE wheres ADD COLUMN where_tsv tsvector"

    execute <<-QUERY
      UPDATE wheres SET where_tsv= (to_tsvector('english', coalesce("wheres"."address"::text, '')))
    QUERY

    execute "CREATE TRIGGER where_tsvector_update BEFORE INSERT OR UPDATE
               ON wheres FOR EACH ROW EXECUTE PROCEDURE
               tsvector_update_trigger(where_tsv, 'pg_catalog.english', address);"
  end

  def down
    execute "drop trigger where_tsvector_update on wheres"
    execute "alter table wheres drop column where_tsv"
  end
end

And now for the fun stuff, console to the rescue!
>> Event.search_for_location("china")
  Event Load (1.3ms)  SELECT "wheres".*, (ts_rank(("wheres"."where_tsv"), (to_tsquery('english', ''' ' || 'china' || ' ''')), 0)) AS pg_search_rank FROM "events" INNER JOIN "wheres" ON "wheres"."event_id" = "events"."id" WHERE ((("wheres"."where_tsv") @@ (to_tsquery('english', ''' ' || 'china' || ' ''')))) ORDER BY pg_search_rank DESC, "wheres"."id" ASC
=> [#<Event id: 288, created_at: "2012-11-09 23:56:19", updated_at: "2012-11-16 17:54:15">]

YAY!, but wait!!  That's not the right id!!
Where.search_for_address("china")
  Where Load (0.6ms)  SELECT "wheres".*, (ts_rank(("wheres"."where_tsv"), (to_tsquery('english', ''' ' || 'china' || ' ''')), 0)) AS pg_search_rank FROM "wheres" WHERE ((("wheres"."where_tsv") @@ (to_tsquery('english', ''' ' || 'china' || ' ''')))) ORDER BY pg_search_rank DESC, "wheres"."id" ASC
=> [#<Where id: 288, event_id: 1824, address: "China, Tx", latitude: 30.047992, longitude: -94.3357433, created_at: "2012-11-09 23:56:19", updated_at: "2012-11-16 17:54:15", where_tsv: "'china':1 'tx':2">]

Event.search_for_location seems to be selecting wheres not events...
DANG!  So am I doing something wrong?

pg_search 1.0 changes how queries are generated and should no longer cause this issue.


Cancel requests

Reject the imageLoad promise onAbort
Show how to use cancelFn() in the demo.

Coverage decreased (-0.2%) to 3.846% when pulling 6bbeffe2fba16e1b64911cb0e0000b47f45f327d on diego0020:cancel-requests into 6a5cd3df561d05451b387ff190d6d1ec193c2632 on cornerstonejs:master.


Full toml-test compliance

[x] literal strings 'like this'
[x] multiline strings """like this"""
[x] multiline literal strings '''like this'''
[x] float parsing (?)
[x] inline tables { like = "this", and = "another" }
[ ] handle table defined multiple times
[x] local datetime, local date, and local time

Handling for all of the string types, local datetimes, and inline tables is addressed in the current state of #9, along with a bugfix regarding the handling of cases where the final line ends in EOF instead of newline.  (That bugfix was more or less addressed as part of the inline table implementation.)  For the most part, these changes had a relatively clear path forward without making significant changes to the basic parsing framework.  I may open an issue to come up with a more elegant solution to the EOF bugfix, but I wanted to wait until we resolved the duplicate table feature, as there's a potential decision point there that may affect this as well.
The crux of the problem with the duplicate tables is that the current parser completely builds out each table at a given level before merging them.  Therefore, it has no way of distinguishing the following two cases, one of which is valid and one of which isn't.
# VALID
[a.b.c]
answer = 42

[a]
better = 43

# INVALID (a simpler test without the dotted keys is explicitly in the test suite)
[a]
b.c.answer = 42

[a]
better = 43

The existing duplication check requires the entire tables to be equal?, which is why it would let something like the latter example through.  The current check is sufficient to address the case where all the tables are empty, as then they would trivially be exact duplicates.
The TOML spec doesn't include all the contents of the table as part of "defining the table."  Thus the invalid example "defines" table a twice, even though its contents are disjoint.  That you can explicitly define a parent table even though "earlier" dotted keys implicitly define it (the valid example above) is because the ordering on the same level is specified not to matter.  As such, the valid example is equivalent to the case where the a definition occurs before the a.b.c definition.
So I see a couple of possibilities here.

Pass the explicitly defined keys along, so that a modified duplication check can have a go at them.  This is probably the simplest approach, but this could get tricky around where blocks end.  I haven't thought through all the permutations.
Delay inferring parent tables, then incorporate this into the inference/merge step.  This might be trickier, but would then allow us to get rid of the clunky duplication check that occurs immediately prior to merge.
Modify the parser to be more line-oriented instead of block-oriented.  This would (in my opinion) make the parser a little bit closer to the language of the TOML spec itself, which could result in a simpler final product, but this would be a relatively big overhaul.  Further, it's not clear to me that TOML won't eventually add the explicit notion of a block, as there seems to be some discussion of that in the TOML repo issues.

I'm probably going to take a shot at option (1) for a first go and see how gnarly it gets, but I'm open to other suggestions.


📚🩹⚙ Fixing docs build.

This is being more complex than expected due to the configuration of different building flows on the same project. I'm almost convinced that I'll end up leaving the documentation in another separate repository.
I've also considered leaving a branch just for documentation, but I've found too many cons and little to no benefits.


extra items rendered oustide viewport

Without enableUnequalChildrenSizes or bufferAmount set there are sometimes 1 or 2 extra items being rendered outside the viewport.
This can be easily reproduced using the samples at https://rintoj.github.io/

This has been fixed & released with npm version 1.0.7


Liberty

@mattgreene
What's this change do?
This PR is to merge liberty into master. Once that is done, I will create a branch for liberty and mitaka based on the tip of master. Both liberty and mitaka installations of these playbooks have been tested.
Any background context?
Where should the reviewer start?
Review the license, contributing doc, and README file, just to make sure it's fit for public consumption.

All yaml files have been reviewed by @dflanigan, but feel free to peruse them if you'd like

@mattgreene: I addressed your comments. I created two new playbooks. The deployment of the agent alone and of the driver alone. I kept the original playbook to deploy both as well. For each playbook, I add a vars file. One vars file would be more confusing to just deploy the agent, but we could do that, if you'd like.

This code is fine for 'master', but when it splits into liberty and mitaka branches the vars file needs to reflect the correct package versions.  playbooks/vars/agent_deploy_vars.yaml refers to 9.1.0 (which is mitaka, though your PR origin branch is liberty).


Maps do not retain order in the config viewer

A potential solution would be to use a LinkedHashMap (insertion order preserved). Though I am just speculating for now.

This may not be true


Add Sam Capehart to Contributors list

Cool project.

Hi  @JimmyNeutron8, I'm quite elated about your pull request. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a pull request and learned that I merged it?
We’ve recently added social share  to our web app. Could you please go to https://roshanjossey.github.io/first-contributions/#social-share and share your first contribution to open source?
Also, check out projects with easy issues while you’re there.

Hi. The instructions were very easy to follow. I have some previous experience cloning, editing, and pushing repositories using github, but this is the first time I created a branch and merged it. Everything went smoothly :)

I’d urge you to continue contributing to other open source projects.
We also have a slack team to help/mentor people trying to contribute to open source projects.
You can join the team by going to https://goo.gl/PM1Jdh


Feedback requested: Page-Requested Reader Mode

I just wrote up a proposal for a standard I think you'd find interesting: https://github.com/jab/reader-mode
If you have a moment, please take a peek and let me know what you think.
Thanks for any feedback, and thanks for the great work on this extension!

This extension is aims to care the eyes, but not reducing the information noise. In my opinion bad UX  is a problem of website a owner and it should not be changed by using extra APIs, but changing website creators minds.


Getx with New FirebaseAuth

Is your feature request related to a problem? Please describe.
it can't use Getx with firebase user stream binding to go to homeSreen when authenticated or loginScreen if not logged inn.
Describe the solution you'd like
example code please.

Hi, please share your code to see what you have done. Otherwise, you can browse issues to get some example, I've written it one too.

If you are looking for sample code or something similar, please join one of our community channels, you will be answered faster, and it will certainly be the appropriate place.
This space is destined exclusively to issues and requests for resources.
As this is neither an issue nor a resource request, I am closing this.

If you are looking for sample code or something similar, please join one of our community channels, you will be answered faster, and it will certainly be the appropriate place.
This space is destined exclusively to issues and requests for resources.
As this is neither an issue nor a resource request, I am closing this.

thank you for closing my problem and thank you for the advice
Sorry : i am totally new here on GIT
what channels ? where are the links? how i can subscribe to these channels?
i was watching a video on you tube from 3h the man was struggling with the Auth control too.
i think your package is great but it is to advance for me as i count on resources and documentation .
i am sorry for asking the author for support .
i am sorry to waist your time.

https://github.com/jonataslaw/getx#communication-and-support-channels
Slack, telegram and discord

don't worry man, GetX can be quite challenging to learn at first, because it has a lot of things and it is very different of other know packages.
 Life in flutter becomes way easier with GetX

don't worry man, GetX can be quite challenging to learn at first, because it has a lot of things and it is very different of other know packages.
 Life in flutter becomes way easier with GetX

Nipodemos.
thank you to your message.
i agree with you, learning something new can be challenging  when you have no reference nor recommendation from the official author.
i was trying to get the right technique from the source like always. there is a mess in resources specially on youtube including wrong information and techniques.
i like flutter and Dart i think i should stick to the thing which has available resources and support. i can understand that beginners questions are stupid some times :)
my apologies again to the Author  of the Package


Getting "x509: certificate signed by unknown authority" with Route53 setup

Using the latest (as of 2019-05-18) version of registry.opensource.zalan.do/teapot/external-dns I'm getting the following logs:
level=error msg="RequestError: send request failed\ncaused by: Get https://route53.amazonaws.com/2013-04-01/hostedzone: x509: certificate signed by unknown authority"

It turns out that it cannot trust route53.amazonaws.com because /etc/ssl/certs directory is empty. I believe this problem is also the reason for #1035.
Possible solution: include certificate installation in the dockerfile:
RUN apk add ca-certificates && update-ca-certificates

I also see this issue using the latest tag:
time="2019-05-19T15:59:18Z" level=error msg="Get https://www.googleapis.com/dns/v1/projects/REDACTED/managedZones?alt=json&prettyPrint=false: x509: certificate signed by unknown authority"

Using the latest release of v0.5.14 resolves this issue.
For me, when I attempted this, latest resolved to docker-pullable://registry.opensource.zalan.do/teapot/external-dns@sha256:15e89230a0bbe62dac4c2af011fc3ed24384f2ec4df4899de16dda48c0087a93. I'm running on Google Kubernetes Engine, version v1.12.7-gke.10.

The same is the case for AWS Route53 with latest docker image
time="2019-05-21T10:03:48Z" level=error msg="RequestError: send request failed\ncaused by: Get https://route53.amazonaws.com/2013-04-01/hostedzone: x509: certificate signed by unknown authority"

Sorry about that! @dryewo fixed it in #1037.

New images are being built right now and should work again.
The latest released version v0.5.14 works, too.


Add independent type hints

Here it is, the annotation files, and runs mypy during CI.

What is the aliasing useful for if the names are the same?

This is actually a workaround for another problem. If type checkers only sees a single occurance of symbol, they may emit warning that the symbol is unused. 2 occurances shut them up 😄

Thank you!


Istio/SDS-logging: There is no way to change the log level for the sdsServiceLog in node-agent.

(This is used to request new product features, please visit https://discuss.istio.io for questions on using Istio)
Add a way to change the log level for the sdsServiceLog in node-agent
There are no alternatives
Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[x] Security
[ ] Test and Release
[ ] User Experience
[x] Developer Infrastructure
Additional context

@JimmyCYJ could you see if this is now supported?

Node-agent is now no longer supported in 1.5 so I think nothing is needed here

@howardjohn then how does the SDS work now? I mean who gives the certs to the pods?


Backport knative pgk with addressable fix and unit test

Fixing critical bug regarding addressable and make contrib work w/ serving v1beta1
Proposed Changes
Port a few PRs to 0.7 release branch, to get contrib working w/ serving v1beta1:

#474
#477
#476

Release Note

make contrib work with Serving `v1beta1` API

/approve

/approve


set 'filter_level' for Streaming API has an error

Set filter_level for Streaming API as below gives an error:
my_stream.filter(follow= follow_users,filter_level=['medium'],async=True)

Error says:
AttributeError: 'list' object has no attribute 'encode'

Python version: 3.5
Tweepy version: 3.6 (It also happens with 3.5 as well)

I looked at Streaming.py to see what gives an error:
 if filter_level:
            self.body['filter_level'] = filter_level.encode(encoding)

I think it has to be fixed as below, and it works. I tested it.
if filter_level:
            self.body['filter_level'] = u','.join(filter_level).encode(encoding)

Sorry, This was a mistake.


Fix #398 salt backend returning non-encoded strings

salt backend failed because local salt client returns
stdout and stderr as str objects not encoded bytes.
Taking a lead from the ansible implementation the stdout and
stderr results are both encoded and returned as kwargs through
base.result method.

Hi, thanks for this!
What version of salt are you using and using which python version (on salt side) ? I guess this does not occur always because we have a test which should fail, so maybe it's related by python2 salt ?

I can confirm that this has to be merged for Salt 3000 or newer to work.

This can be closed, as it is merged in #636.