comment.raw is not correct - seems almost the same as comment.value

In the readme is shown that that .raw property contain the full comment including starting and ending parts /* and */.
The example from the readme isn't correct.
var str = '/**\n * this is\n *\n * a comment\n*/\n\n\nvar foo = "bar";\n';
var comments = extract(str);
console.log(comments[0].raw)
// => '*\n * this is\n *\n * a comment\n'
// and not '/**\n * this is\n *\n * a comment\n*/'

causing errors in strip-comments lib, because it tries to str.replace(comment.raw, '').
So we need some real raw, or at least useful character location position. I mean, we can do nothing with that loc currently:
loc: { start: { line: 1, column: 0 }, end: { line: 5, column: 2 } }

It's almost useless, because is on per line basis. But we need something like loc.pos which would be character position from the beginning of the string/file.

But okey, actually i have idea, to tweak the strip-comments as i do in acorn-strip-comments/index.js#L142-L147

causing errors in strip-comments lib

none of the comments libs have any errors for me.

So we need some real raw, or at least useful character location position.

Why is it a problem? everything is correct. All you need to do is prefix raw with /* and append */, then use -2 and +2 in slice.

none of the comments libs have any errors for me.

when bump to latest extract-comments, the strip-comments tests starts to fail.
Nevermind, tweaked.

k thx


Add API to generate batch id for segment upload if batch upload is within quota.

This PR:

Add API to generate batch id for segment upload if batch upload is within quota.
Modify upload segment API to include batch info to ZK metadata.

Details of the new API:
POST /batch/{tableName}
Body:
{
  "segments": [
    {
      "name": "string",
      "size": 0
    }
  ]
}

Response:
{
  "status": "Successfully initialized batch upload segments of table: testTable",
  "batchId": "b2483e32-342b-44ad-b1bb-0a40c067176b-1",
  "currentSizeAcrossReplicas": 500,
  "storageQuotaAcrossReplicas": 6000
}

Codecov Report

Merging #3443 into master will decrease coverage by 0.14%.
The diff coverage is 21.27%.

@@             Coverage Diff              @@
##             master    #3443      +/-   ##
============================================
- Coverage     70.52%   70.38%   -0.15%
  Complexity        4        4
============================================
  Files           998     1001       +3
  Lines         45674    45826     +152
  Branches       6117     6133      +16
============================================
+ Hits          32213    32256      +43
- Misses        11348    11454     +106
- Partials       2113     2116       +3

Impacted Files
Coverage Δ
Complexity Δ

...m/linkedin/pinot/common/utils/CommonConstants.java
39.58% <ø> (ø)
0 <0> (ø)
:arrow_down:

...t/controller/api/upload/batch/SegmentsInBatch.java
0% <0%> (ø)
0 <0> (?)

...upload/batch/SuccessBatchIdGenerationResponse.java
0% <0%> (ø)
0 <0> (?)

...inot/controller/api/upload/batch/SegmentEntry.java
0% <0%> (ø)
0 <0> (?)

...i/resources/PinotSegmentUploadRestletResource.java
57.26% <12.12%> (-8.06%)
0 <0> (ø)

...not/controller/validation/StorageQuotaChecker.java
42.2% <19.35%> (-32.12%)
0 <0> (ø)

...ntroller/helix/core/PinotHelixResourceManager.java
59.5% <20%> (-0.25%)
0 <0> (ø)

.../pinot/controller/api/upload/SegmentValidator.java
58.69% <33.33%> (-1.31%)
0 <0> (ø)

...not/common/metadata/segment/SegmentZKMetadata.java
76.16% <50%> (-2.82%)
0 <0> (ø)

...nkedin/pinot/controller/api/upload/ZKOperator.java
65.16% <55.55%> (-3.07%)
0 <0> (ø)

... and 26 more

Continue to review full report at Codecov.

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

Can we come up with the better api path.

@kishoreg Changed the path to /tables/{tableName}/batch.
Basically it mainly checks storage quota and should be put under the table size resource.

This PR is no more needed. I'm closing this PR.


Changes search pattern for Storm version

Fixes issue 254

Thanks for the fix!

was this supposed to be fixed?
vagrant@vagrant-ubuntu-trusty-64:~/wordcount$ sparse --version
sparse 3.11.0
vagrant@vagrant-ubuntu-trusty-64:~/wordcount$ sparse run
Traceback (most recent call last):
  File "/usr/local/bin/sparse", line 11, in <module>
    sys.exit(main())
  File "/usr/local/lib/python2.7/dist-packages/streamparse/cli/sparse.py", line 79, in main
    args.func(args)
  File "/usr/local/lib/python2.7/dist-packages/streamparse/cli/run.py", line 102, in main
    env_name=args.environment)
  File "/usr/local/lib/python2.7/dist-packages/streamparse/cli/run.py", line 46, in run_local_topology
    local_version = local_storm_version()
  File "/usr/local/lib/python2.7/dist-packages/streamparse/util.py", line 358, in local_storm_version
    return parse_version(re.findall(pattern, res.stdout, flags=re.MULTILINE)[0])
IndexError: list index out of range
vagrant@vagrant-ubuntu-trusty-64:~/wordcount$ storm version
0.9.5


Multiple include dirs

Hi,
I added the functionality to have multiple include sections in logrotate.conf
This is useful to implement the nail pattern.

Hi @noqqe, we recently migrated this module to Vox Pupli. Can you please rebase against master and add a spec test for the change?

I'll close this because i misunderstood how this module works.


upgrade new rails-settings-cached (~2.0)

@pnghai Hi, thanks for you PR. I fixed tests and removed old rails version here https://github.com/artofhuman/activeadmin_settings_cached/pull/52/files. Can you update you pr?


BertForNextSentencePrediction labels

Hi everyone!
I was reading trough the documentation, and, according to https://huggingface.co/pytorch-transformers/model_doc/bert.html#bertforsequenceclassification, it expects that next_sentence_label is 1 if B is not a next sequence for A, and 0 if B is a sequence for B.
That's somewhat counterintuitive, since most of the datasets (I believe) will assume this problem to be a binary classification.
Is my assumption correct? Should I flip my dataset before fine-tuning the model?

You should supply your own labels when using the BertForSequenceClassification class (labels input to the forward method). You can choose the labels you like.
The BertForSequenceClassification class is not related to the Next Sentence Classification task used during Bert pretraining. You can use the BertForNextSentencePrediction class if you want to do next sentence prediction (classification).

Yes, I'm sorry I meant the  BertForNextSentencePrediction class.

Yes, in this case, you should follow the docstring: 0 indicates sequence B is a continuation of sequence A, 1 indicates sequence B is a random sequence.


[+] MO : Ease backoffice. Make the select show only customers having at least a wishlist

make the module config page to populate the select only with customers having a wishilist

Hi,
Thank you for your contribution,
Best regards.


Convert characteristic value to string.

I am reading characteristic from nordic device and able to read is successfully.
Issue I am facing is to convert it to string or actual value. According to firmware developer nordic is broadcasting values as unsigned short(2 bytes) and at the react native side we will need to covert it to hexadecimal and from hexadecimal to decimal. Now I am bit confused with how does it read value properly in nRF Connect app but not with this library.
Here is response I am getting while reading characteristic.
deviceID: "D9:C2:B1:BA:85:6C"
id: 10
isIndicatable: false
isNotifiable: true
isNotifying: true
isReadable: true
isWritableWithResponse: true
isWritableWithoutResponse: false
serviceID: 9
serviceUUID: "0000180a-0000-1000-8000-00805f9b34fb"
uuid: "00002ac4-0000-1000-8000-00805f9b34fb"
value: "8wI="

@Matheuschn thanks, also faced similar need, your method worked fine

Hi!
@Matheuschn thanks a lot for the solution! 💪
@ravirupareliya Is Matheuschn answer sufficient for you and does it solve your problem?

Yes, that worked, thank you @Matheuschn


Auth on my custom api

Have any sample (nodejs) with login in my custom api and send after the bearer token inside each request?
like Asana App (tabs) makes login (open url in iframe) after this, can list itens in cards.

https://github.com/microsoft/BotBuilder-Samples/blob/main/samples/javascript_nodejs/18.bot-authentication/dialogs/mainDialog.js

Yes, like this, but without using oauth, my api use a jwt token auth based, on browser iam using a localstorage to save a token and use it on axios default header

Hi @brunokunace
The bot framework team prefers that How-To questions be asked on StackOverflow using the botframework tag.
I would start with this doc here: https://docs.microsoft.com/en-us/azure/bot-service/rest-api/bot-framework-rest-connector-authentication?view=azure-bot-service-4.0


no soundcards available to choose, only default...

... but on system/hardware there is my usb-soundcard listed. (This is with Synology DSM, yes, you wont support it, its ok)
audio:

    0:
        name: USB-Audio - Jabra SOLEMATE v1.34.0
        type: v1340
        devices:
            [object Object]
            [object Object]

But on an normal Home Assistant installation on Debian Buster with Docker, the same issue exists.
audio:

    0:
        name: HDA-Intel - HDA Intel
        type: Intel
        devices:
            [object Object]
            [object Object]
            [object Object]
            [object Object]
            [object Object]

What exactly changed in the last week?

Maybe this is part of the issue:
2020/03/04 23:13:03 socat[369] E connect(5, AF=1 "/data/internal/pulse.sock", 27): No such file or directory

Because PulseAudio did not detect you sound card. With new CLI >= 4.1 you can use ha audio logs to see what is going on. Maybe you can find on google what you need exactly for files to make it running on Linux/Pulse. This changes can be applay to this repository and we can destribute it 👍

By the way, not support synology did not mean that we not help troubleshooting and merge PR they fix issue on that platform. It means only that we not active fix bug and this part need comes from the maintainer of that community installation.

Hi @pvizeli, Supervisor Version 207 / Update Core 106.4 breaks it.
Here are the logs: https://gist.github.com/raph2i/e74b0948970a9bb1ae2d8fe6db58cd49
It worked like a charm before the update.
I don't understand what has changed, and how the "chain" works?
/dev/snd is mounted via a bind mount into hassio_supervisor and then to hassio_audio?
Has pulseaudio to be running in the underlying OS?

I think in my case ( https://github.com/home-assistant/audio/issues/7 ) it was also 106.4 upade which made the problem.

https://github.com/home-assistant/supervisor/issues/1582
I have a error  like this~

Should be solve with latest container


Fix icon changes in connection dialog

My changes for task view impacted the connection dialog; and possibly some other places. To keep this from happening I just revert my changes and moved them to the specific task location.

I'll merge now so this can be in tomorrow's insider build.


Updated Upstream (Paper)

Upstream has released updates that appear to apply and compile correctly
Paper Changes:
6733d7875 [Auto] Updated Upstream (Bukkit)

That update is just a javadoc correction.
https://hub.spigotmc.org/stash/projects/SPIGOT/repos/bukkit/commits/5eece9f8580658aff45ba851b79059c0e16b921a
Not worth Purpur having its own dedicated build number to.


[#233] npm run styleguide errors

Fixes #233
The following changes are contained in this pull request:

Updates sc-5 styleguide dependency — version 1.3.3 had parsing issues, it’s now specified at minimum 1.4.1

Removes the already-unused dependency on the deprecated eslint-config-bitcrowd.

[x] npm run checks script has been run without errors

@tessi The problem you saw seems to be limited to the version of the styleguide generator (1.3.3) that was current at the point you installed. This PR specifies a newer version which doesn't display that problem for me — if you have time could you pull this branch and try npm install again please?


Render markdown for description in software templates

Feature Suggestion
The description field in Software Template should be rendered via Markdown.
Context
We would like to provide as much guidance to users as possible and a formatted description provide good visualisation on the important points. Additionally, we can provide links to more info.
      properties:
        entity:
          title: Entity
          description: |
            Select the type of entity to create:

            - Resource (e.g. SQL Database, S3 Bucket, ...)
            - Component (e.g. backend service, data pipeline)
            - API

            Find more details in the (System Model)[https://backstage.io/docs/features/software-catalog/system-model] documentation.
          type: string
          enum:
            - component
            - resource
            - API

Hey @estahn is this something you want to contribute?
There's some good issues here from the upstream library that could help you implement this in backstage for rendering markdown as the description.
https://github.com/rjsf-team/react-jsonschema-form/issues/1010
https://github.com/rjsf-team/react-jsonschema-form/issues/1146

@benjdlambert seems like this issue was resolved with the referenced PR and can be closed

Argh, I forgot to hit send on this yesterday:
Looks like the issue for rendering markdown in the description of TemplateCard's is fixed, but it looks like this is the description field on the form components. So not stale and still up for grabs, worth looking at rjsf to see if there's any movement on this.

where exactly do we need markdown? if someone can specify that I can help!

This is to support markdown in the description fields that are rendered underneath the component. Looks like we just need to provide a DescriptionField template that uses the one from react-jsonschema-form but passes through MarkdownContent too.

I would like to see this so we could link to guidance on specific questions in forms.
I am happy to contribute but I'm not really sure where to start with this. Any pointers?

Unstale, please


help me :(

can help me with this error https://s28.postimg.org/dgyspaea5/234.png

Duplicate of #492

@imnotNxhz1 Look in #492 for the answer.


"Use system language" is not translated

Storage Explorer Version: 1.11.0
Build Number: 20191105.2
Platform/OS: All
Architecture: All
Regression: 1.11.0 (dev)
Bug Description
"Use system language" in Settings > Application > Regional Settings is not translated. While the specific languages should remain untranslated (the same strings regardless of current language setting), the system language option should be.
Additional Context
File: Shell.json
ID: LabelLocaleUseSystemLanguage
Note that there is no "_LabelLocalUseSystemLanguage.comment", so I'm not sure why the translators "untranslated" this string.

@weeteckt Do you have an update on this? Has this been fixed?

@janbaltus - I believe this is fixed but could you confirm once again?

Untranslated string  Use system language fixed in Storage Explorer repo.

Verified this issue on build master/20191123.2. It has been fixed.


Remove all whitespace related to comments

Remove all whitespace related to comments
Source code:
const obj = {
  // Here are some code comments
  name: `js`,
  /**
   * Here are some code comments
   */
  v: `es6`,
}

Conversion options:

<EMAIL_ADDRESS>
const outStr = strip(str, {
  preserveNewlines: false,
});

Actual output:
const obj = {

  name: `js`,
    v: `es6`,
}

Note: v There is a blank character before the key, but it is not in the source code. Is this a bug?
Expected output:
const obj = {
  name: `js`,
  v: `es6`,
}

I know this is really old, but in case anyone else comes across it.  I couldn't get the preserveNewlines: false to work so added this:
const cleanScript = strip(script).replace(/[\r\n]+/gm, '\n');
More on the regex here: https://www.geeksforgeeks.org/how-to-remove-all-line-breaks-from-a-string-using-javascript/

Try this tool: remove-comments

Supports batch processing of files using the glob method
Delete single-line multi-line comments from html, js, json, css/text
Keep the layout/format of the original document
Delete lines with only comments
Compatible with css3, json5, es6
Support mixed content, such as js/css in html/vue


Research NCEP Data Repository "FTPPRD"

There was a NCEP NOMADS data outage that lasted for a few days and affected the real-time data acquisition for several products hosted on Cumulus. One of the NCEP operational messages (below) suggested switching to 'FTPPRD'. This issue is intended as a "hey, we should look into FTPPRD sometime to understand what's there and how we should be using it for redundancy".
SENIOR DUTY METEOROLOGIST NWS ALERT ADMINISTRATIVE MESSAGE
NWS NCEP CENTRAL OPERATIONS COLLEGE PARK MD
0016Z TUE AUG 16 2022

...NOMADS SITE PARTIALLY RESTORED...

The NOMADS website was partially restored, though users may
continue to experience slower than normal downloads at high
traffic times and Hysplit data remains unavailable. We
anticipate having the hardware issue impacting part of the
Boulder data center corrected by late Wednesday. In the
meantime, users are encouraged to switch to FTPPRD
https/ftp data access when possible.

Handel/SDM/NCO/NCEP

^ Copied From https://www.nco.ncep.noaa.gov/status/messages/

related: https://www.lstsrv.ncep.noaa.gov/mailman/listinfo/ncep.list.nomads-ftpprd

Closed as it's kind of a note, not an issue...


v1.12 Backports 2024-01-10

[x] #29310 (@julianwiedmann)
[ ] #29971 (@renovate[bot])

Once this PR is merged, a GitHub action will update the labels of these PRs:
 29310 29971

/test-backport-1.12


Do not perform dependency verification on the poms we resolve

I can't get the test to fail (yet?) with the fix commented out.
Closes #78

The test still doesn't work. Tried all kinds of things. I want to make a release so I'm going to land the fix with the test that doesn't actually validate the behavior for now.


Twitter link not available on scorelab website footer

On the scorelab official website https://scorelab.org/ in the footer, a Twitter icon is present as visible -

But, it is taking users to the official Twitter page, not on scorelab Twitter profile.
It might be due to scorelab currently do not have a Twitter account or the link for it was not pasted. Thus, I would request @charithccmc and @wdevon99 to have a look at this issue and maybe we could remove the icon for now if we do not have a Twitter account for scorelab currently.

@Grumpyyash hello sir , can you plz assign this task to me . If  the redirected link are not added or any other issue which let it stop to redirected to scorelab Twitter page , I will find it out and solve it .

Hi @saif-malik-01 thanks for showing interest but this is not an issue with the existing project repo but it has to do with props passed in the actual website and mentors will take care of this as we do not have its access

Okay , Thanks for info :D


Feature request: Zettelkasten

I really like the idea of this plugin. I use the Zettelkasten system, and the problem i am facing is with organizing MOCs (maps of content) which are themselves notes. These maps of content more or less take the job of the folder in that all notes with a specific subject link to that note instead of being buried inside a folder.
The use case I would like is the automatic organization of notes by linking them to their respective note. For example, if I have a MOC called "AI" and another one called "LLM", then LLM would link to AI, and everything linking to LLM would also link to AI etc.
So in other words, I would like file organization through notes instead of through directories.

To be more clear, I don't use tags and directories as the primary way to organize my vault. Rather, I have one directory called "Zettelkasten" and two tags (#idea and #MOC). The way my vault is organized is by linking notes to other notes. The issue I have is that it becomes very unclear which notes should be #idea notes and which should be #MOCs, and which MOCS should link to other MOCS and Zettel notes to MOCS etc. Utilizing AI to help with this would be very useful, and seems like an interesting extension to your plugin.

I have one more note type actually which is #literature. It would be awesome if, for example, the moment I create a literature note with a given title, that note automatically links to one to many MOCS the title of the literature note is based on. The same thing with Zettels linking to MOCS as well.

sounds awesome
i'd love to have my "automatic" things going on but one of the main challenge is that it's extremely heavy on my openai credits.
are you self-hosting the product?

Yes self hosting works for me. I agree that it would cost a lot of credits.


Custom data: registerGettersAndSetters called too late

I am currently running

SpongeVanilla version: 1.12.2-7.0.0-BETA-1

Issue Description
I would like to use custom data. This issue was introduced by a recent commit in SpongeCommon. This issue is more like a question.
Here is a test repo for this issue: https://github.com/randombyte-developer/data-issue (execute the /test command)
This plugin simply gets any entity and tries to offer some sample data to it. How it is done in the plugin, it fails with this: https://gist.github.com/randombyte-developer/0e1096495581764b594775bc7d8bffc2
So, the registerGettersAndSetters() method is called too late. Okay, then I will call it earlier, but I need the instance of the manipulator for that, which isn't created yet. It think the only solution to this would be creating all possible manipulator instances in the pre init, that will be used later. But this doesn't seem like a solution. What am I doing wrong?
When should registerGettersAndSetters() be called?

@felixoi suggest "as work-around SampleKeys.BOOL.getQuery() will work, because this loads the class.. (In GameInit Event) "

It's not a workaround in these words. I think everything works, there is no issue. Since API 7 you have to register the keys before data registration is done. If you outsource the key class now, you have to load the class before data registration (static fields will be initialized which calls the key builder). The other option would be registering the keys like in this example:

This is not an issue. You need to register your keys in the same init stage as your data registrations happen.


Dev to Main Sync

Date: 15-10-2024

Developer Name: vinit

Issue Ticket Number

#822
#880
#876
#890

PR's going in this sync

#850
#881
#878
#889

Here is screenshot it is working properly over staging.


Fix the hrefs of the referenced users

Sorry, there were some typos on the links

No worries, never was really sure how to link to profile either :)


Ignore drifting velocity

GPS drift may cause an incorrect velocity to be shown - validate that the user is moving before displaying a velocity (ex. either with multiple velocity readings, GPS position changes, or accelerometer).

If velocity is derived from GPS one would probably need to handle the case that GPS is off:

one could derive the velocity from the pedometer (envisioned in suggestion #142)
add some brackets/color/symbol that makes clear this is a reading that is ancient/is not updated because of ...
switch it off

Maybe I'm the only one switching off GPS all the time to conserve battery ;)

I agree that something should be done if the GPS is off - I think for this issue, I'm going to just hide it from the UI. If I choose to implement the pedometer, then I'll revisit it.

This may already be fixed in the latest release - I improved the GPS accuracy by waiting for a specific number of satellites to be achieved or a timeout to have been reached before reporting a valid reading. That should also improve speed accuracy.
I can add one more check here to see if the GPS has a reported speed accuracy, and if it does and the variation is greater than the current speed then it will ignore that speed (I'll have to do some research with this).


[3.2] Add interpolation parameter to resize_to_po2()

Add an optional p_interpolation parameter to Image::resize_to_po2() and call it in GLES2 rasterizer storage with either INTERPOLATE_BILINEAR or INTERPOLATE_NEAREST depending on TEXTURE_FLAG_FILTER.
This is equivalent of #44445 that closes #44379 in 3.2.

I'm sure the commits that add the function and the doc update respectively could be unified.
And thanks for your first contribution to Godot!

@YeldhamDev Done! And it's my pleasure. :smiley:

Should be good to go once you squash the commits together. For simple changes like this we prefer one commit.

@clayjohn Squashed.

Thanks!


The main difference between CTTrack and Mixformer or Stark?

The main difference is in the train phrase, we add the correlative decoder based on the MAE.

Hello, I received an error message 'Network output is NAN, stop training!' during the training process. Do you know what could be the cause? My PyTorch version is 1.10.1 and my GPU is 4090


List view -> "layout option" bug

In List View -> Options(step 4/4) -> Layout Options
when "Transpose" is checked, the "Omit header" option has no effect.

fixed in #1966


Proposal: bring back post-release-replacements

I’m automating my release process and there are a few steps that cargo-release doesn’t support. One of them is updating the change log after the release with a new “Unreleased” section. (I would prefer not to have that section in the release itself.)
post-release-replacements would be perfect for my use case. It’s actually the same use case that was described when it was added in PR #228.
It was removed in PR #553 along with dev-version, but I think most of the problems there were actually with dev-version and the default behavior. Re-adding post-release-replacements wouldn’t introduce any new default behavior, so I suspect it won’t suffer from the same problems.
I’d be happy to submit a PR.
I‘m unsure if it’s worth making it work nicely with --unpublished(#551), but I’d be happy to take a crack at that.
Alternatively, the “{Pre,Post}-release Replacements” heading in reference.md should be updated. (Again, happy to submit a PR.)
Older issue: #599.

I'd like to see a more solid use case than this as this feature can make it so there is always a change after a release, breaking change detection. I do not want to add "but ignore these changes" support. I also feel covering footguns with documentation is a non-solution as it will most likely be missed.
As for PRs to clean up stale docs, sure!

The only other usage I can imagine off-hand is updating links to binary packages or some other release product, but absent somebody actually wanting to do that, it doesn’t seem worth it.
PR #632 submitted.


Poochify tests

Remove our reliance on Git LFS by using pooch to fetch files from data.kitware.com.
Link to the folder where test images can be found.

@naglepuff I don't know why the CI on OSX failed a few times.  It passes on my local OSX machine.


Further simplify int terraform files

Checked for duplicates
Yes - I've already checked
Alternatives considered
Yes - and alternatives don't suffice
Related problems
I'm frustrated when [...] happens as documented in issue-XYZ
Describe the feature request
There are still some redundancies in terraform files and while we're deploying R2 ER2 to int-pop1 we should clean it up:

We should use dev-e2e variables.tf instead of int having its own. There's really no use-case where int would have a different variables file than really any other venues on the system. In fact, I think there should only be one variables.tf for ALL venues but that would be a drastic change and needs to be discussed w the rest of the PCM team.
Clean out redundancies in int_override.tf with variables file. We should use values in variables for the AMIs and PGE releases.

Also do this for PST and OPS


Add toledoprudente college

@theylkszaszin I have to close this request without merging, because you requested to add a non-existent domain to this repository: Host toldedoprudente.edu.br not found: 3(NXDOMAIN) Please ensure that you are familiar with this repository rules located at the bottom of this page and submit a new request with the correct information. Thank you for your understanding!


AVPro option not apearing.

Describe the issue
I have AVPro installed. I am currently using it to stream from another service. I tried using the Vimeo Player and it is supposed to auto detect AVPro and give options for it. I do not get the options.
Steps to reproduce the problem
Install AVPro (Pro) SDK, install Video SDK.
Unity version
2019.4.5f
Operating system
OSX Catalina

bump.
i am also seeing the same issue. any solution plz.

Hi, from memory, you have to added the avpro to your custom defines in the player setting for it to know to choose teh vimeo option.

thanks, i already figure it out. :)


Enhanced subtractions fixture

Background
These are FASTA format files that are used for eliminating host and background information from the analysis. Currently, only one subtraction per analysis is allowed. We want to support multiple subtractions or zero in the near future.
Fixture

named subtractions
it is an iterable of subtraction objects
each subtraction has a path attribute
using the subtractions fixture triggers the download of the subtraction data to the analysis working tree
the subtraction can currently be found at job["args"]["subtraction_id"] for analysis jobs; in the future this will be a list of IDs

@igboyes Where are the subtractions downloaded from?

using the subtractions fixture triggers the download of the subtraction data to the analysis working tree
@igboyes Where are the subtractions downloaded from?

I'm thinking having a dataclass which has all of the fields from the subtraction document.
@dataclass
class Subtraction:
    name: str
    nickname: str
    path: Path
    gc: Dict[str, float]
    count: int

Data is in folder at <data_path>/subtractions/<subtraction_id> and contains a number of files:

subtraction.fa.gz
reference.1.bt2
reference.3.bt2
reference.2.bt2
reference.4.bt2
reference.rev.1.bt2
reference.rev.2.bt2

I think we should have three path attributes:

path: folder path
fasta_path: path to compressed FASTA file
bowtie2_index_path: Bowtie requires a path to the index file prefix without any file extension

Output of tree FYI:
subtractions/
└── g79r54x8
    ├── reference.1.bt2
    ├── reference.2.bt2
    ├── reference.3.bt2
    ├── reference.4.bt2
    ├── reference.rev.1.bt2
    ├── reference.rev.2.bt2
    └── subtraction.fa.gz

Data is in folder at <data_path>/subtractions/<subtraction_id> and contains a number of files:

subtraction.fa.gz
reference.1.bt2
reference.3.bt2
reference.2.bt2
reference.4.bt2
reference.rev.1.bt2
reference.rev.2.bt2

I think we should have three path attributes:

path: folder path
fasta_path: path to compressed FASTA file
bowtie2_index_path: Bowtie requires a path to the index file prefix without any file extension

Ok so to clarify;

path in this case would be <data_path>/subtractions/<subtraction_id>
fasta_path would be path/"subtraction.fa.gz"

And I'm not quite understanding what bowtie2_index_path would be here. What exactly is the index in this context?

Output of tree FYI:
subtractions/
└── g79r54x8
    ├── reference.1.bt2
    ├── reference.2.bt2
    ├── reference.3.bt2
    ├── reference.4.bt2
    ├── reference.rev.1.bt2
    ├── reference.rev.2.bt2
    └── subtraction.fa.gz

I can go ahead and write the fixture assuming that these files are present, but what is the plan for loading these files into the workflow container?
I'm assuming we will need to download them from somewhere either before the workflow runs, or when the subtractions fixture is instantiated.
I might be missing something here.


walk readers through everyday tasks

[ ] perform/commit an edit
[ ] update all targets
[ ] add a makefile target
[ ] add R and non-R dependencies
[ ] Convert an existing project
[ ] create a Docker image for archiving

I think this should be in a/multiple vignette(s), not in the main paper - because it's too specific and might need to be updated

Agree with Caspar.

OK, we are all thinking the same. Anyway, great suggestion by the reviewer.

Will be fixed over here: https://github.com/aaronpeikert/repro/issues/89


units

our simulation is 2d, but users will expect quantities and reaction parameters to be in 3d units

need to decide how to implement this / present this to the user
should be some consistency checks
and automatic rescaling where possible
could say our 2d is a slice of 3d of unit depth

(but does the depth change when the units change?)

SBML doc contains units but they are only cosmetic
DUNE doesn't care about units

discussion moved to a wiki page


Incompatibility with Airflow 2.2.4

Describe the bug
If you try to install airflow==2.2.4 with the astronomer-providers==1.0.0 it will run into a dependency version conflict.
To Reproduce
Steps to reproduce the behavior:

Create a fresh virtualenv with Python 3.9: pyenv virtualenv 3.9.7 airflow-test
Activate the virtualenv: pyenv virtualenv activate airlfow-test
Install Airflow 2.2.4 with S3 extras and astronomer-provider 1.0.0: pip install "apache-airflow[s3]==2.2.4" --constraint "https://raw.githubusercontent.com/apache/airflow/constraints-2.2.4/constraints-3.9.txt" "astronomer-providers==1.0.0"
You get an error halfway through:

ERROR: Cannot install astronomer-providers because these package versions have conflicting dependencies.

The conflict is caused by:
    aiobotocore 2.1.1 depends on botocore<1.23.25 and >=1.23.24
    The user requested (constraint) botocore==1.24.2

To fix this you could try to:
1. loosen the range of package versions you've specified
2. remove package versions to allow pip attempt to solve the dependency conflict

Expected behavior
It should install cleanly.
Additional context
Airflow 2.2.4 constraints: botocore==1.24.2
astronomer-providers 1.0.0 requirements: aiobotocore>=2.1.1
aiobotocore 2.1.1 requirements: botocore>=1.23.24,<1.23.25
There might be similar issues with the other boto dependencies... I have tried with 2.2.3 and 2.2.2 but there are similar issues with a different packages:
e.g. 2.2.3:
The conflict is caused by:
    astronomer-providers 1.0.0 depends on apache-airflow-providers-cncf-kubernetes>=3
    The user requested (constraint) apache-airflow-providers-cncf-kubernetes==2.2.0

e.g. 2.2.2:
The conflict is caused by:
    apache-airflow[s3] 2.2.2 depends on apache-airflow-providers-amazon; extra == "s3"
    astronomer-providers 1.0.0 depends on apache-airflow-providers-amazon>=3.0.0
    The user requested (constraint) apache-airflow-providers-amazon==2.4.0

Thanks for raising the issue @fj-sanchez  . This is a known issue, I have created a GitHub issue on aio-botocore here.
For now, you can install it as follows:
pip install "apache-airflow[s3]==2.2.4" --constraint "https://raw.githubusercontent.com/apache/airflow/constraints-2.2.4/constraints-3.9.txt"
pip install "astronomer-providers==1.0.0"

OR
pip install "apache-airflow[s3]==2.2.4" --constraint https://github.com/apache/airflow/blob/constraints-2.2.4/constraints-no-providers-3.9.txt pip install "astronomer-providers==1.0.0"

We publish constraints in Airflow, mainly to have a set of "known-working" dependencies but in this case that is causing conflict with aiobotocore. This will happen from time to time, however as long as the dependencies from astronomer-providers don't conflict with the ones in Airflow repo (mentioned in setup.py or setup.cfg) we are fine. As long as we remain compatible to Constraints without providers we are good. Astronomer Providers repo will also publish it's own set of constraints fine from 1.1.0 onwards


External serializer does not take existing instance into account when using BindTo

This is related to #1376
It looks like the the Nancy.Serializers.* does not take into account that when you call BindTo then the BindingContext can contain an existing instance that should be bound to, they will always try to create a new instance
https://github.com/NancyFx/Nancy.Serialization.JsonNet/blob/master/src/Nancy.Serialization.JsonNet/JsonNetBodyDeserializer.cs#L81
The same holds true for the ServiceStack serialization package
https://github.com/NancyFx/Nancy.Serialization.ServiceStack/blob/master/src/Nancy.Serialization.ServiceStack/Nancy.Serialization.ServiceStack/ServiceStackBodyDeserializer.cs#L46
We need to make sure we can distinguish an existing instance from  a created one, because the Model is set by the DefaultBinder
https://github.com/NancyFx/Nancy/blob/master/src/Nancy/ModelBinding/DefaultBinder.cs#L334

Closed by #1431


Fix v2 bump victim id for sim

Bump victim ID was being passed as a rocket-sim id to the GameState object, which is not usable. This change makes it the corresponding agent ID so that it can be mapped to the car in the GameState.
Below is a patch to example.py that reproduces the issue.
bump-victim.patch

Thanks for the pr


Update scalafmt-cli_2.13, scalafmt-core to 3.8.2

About this PR
📦 Updates

org.scalameta:scalafmt-cli_2.13
org.scalameta:scalafmt-core

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

🔍 Files still referring to the old version number
The following files still refer to the old version number (3.7.17).
You might want to review and update them manually.
website/docs/reference/cli-options.md
website/docs/reference/scala-command/cli-options.md
website/docs/reference/scala-command/runner-specification.md
website/docs/release_notes.md

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scalameta" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "org.scalameta" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, old-version-remains, commit-count:n:3

https://github.com/VirtusLab/scalafmt-native-image/releases/tag/v3.8.2


chore(main): release 2.116.3

:robot: I have created a release beep boop
2.116.3 (2024-05-22)
Bug Fixes

versioning: webhooks not triggered when new version published (#3953) (fb2191b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/Flagsmith/flagsmith/releases/tag/v2.116.3 :sunflower:


[Scheduled Maintenance] Site down for Maintenance

Additional context
Automatic maintenance triggered, expect downtime.

Incident Report: Cascading failure on KeyDB server
We have found out that our services are failing due to failure to truncate AOF (Append-only-file) logs causing the KeyDB to get stuck in a crash loop, combined with unfortunate timing caused by our automated maintenance system, suddenly shutting the entire cluster down while the issue has not been resolved.
The crash loop seemed to happen due to KeyDB's restart mechanism trying to load the untruncated AOF log which had a size of 8.9GB, which was too large for the server itself to handle, timing out the pod before it could complete the startup and causing the Kubernetes scheduler to put KeyDB into a crash loop, and not spawning redundant fail-over nodes due to failed checks.
Even worse, after rebooting the cluster the K3s nodes' network started dropping out, causing containers to refuse to start due to pull failures, prompting manual intervention and a second cluster-wide reboot.
Fortunately, the KeyDB snapshot dumps feature was working as intended, and saved the database state moments before the actual failure happened, so nothing of value was lost (even though any data loss on KeyDB will not result in any significant loss nonetheless, as we only use as a temporary cache for misc. token cookies). You may have to re-login to your services due to this.
Services Affected: all services, due to scheduled reboot not working as intended.
By the time of publishing this, all services should be working as intended.
We are sorry for the inconvenience.


💬 Feedback - Improve Documentation with step by step guide to create component

First of all, congratulations for this great work on this template! The dev experience is very pleasant!
My first attempt was to create a basic component that I could use on the home page.
Despite the steps described in the README I took some time to understand what steps to follow to create a component.
It would be good to add a step by step guide to create a component from Contentful to React code.
Maybe this guide already exists and I just couldn't find it?
I wrote down the steps I followed to create my first component
Goal
Create a component with a "Title" field that can be inserted on the home page
On Contentful backoffice

Go to Content model → +Add content type
On name field  → 💎 Demo component
On Api Identifier → componentDemo
( !!! This will be used in the code and it may be difficult to change it later, make sure you use a correct identifier)
Click “Create”
Click “Add a field” → Text (Name: title, field ID: title)
Check “This field represents the Entry title” —> Confirm —> Save
On “Content model” → 📜 Landing page → Top section → click on “Settings”
Add “💎 Demo component” to list bellow Accept only specified entry type → confirm → Save
On “Content” → Add entry → 💎 Demo component
On title → Demo Component Example
Click on Publish
On Content → Homepage → Top Section → Add existing content → Demo Component Example → Insert 1 Entry → “Publish changes”

On Code repository
Run following command
yarn run graphql-codegen:generate

Create component graphql query
src/components/features/ctf-components/ctf-demo/ctf-demo.tsx
Note that componentDemo in the query represent your Api Identifier previously defined
fragment DemoFields on ComponentDemo {
  __typename
  sys {
    id
  }
  title
}

query CtfDemo($id: String!, $locale: String, $preview: Boolean) {
  componentDemo(id: $id, locale: $locale, preview: $preview) {
    ...DemoFields
  }
}

Create Functional components
More detail here
src/components/features/ctf-components/ctf-demo/ctf-demo.tsx
the React component which is actually rendering the content type.
import { makeStyles } from '@mui/styles';
import { Theme } from '@mui/material';
import { DemoFieldsFragment } from './__generated/ctf-demo.generated';

const useStyles = makeStyles((theme: Theme) => ({
  root: {
    marginTop: theme.spacing(8),
    marginBottom: theme.spacing(8),
    textAlign: 'center',
  },
}));

export const CtfDemo = (props: DemoFieldsFragment) => {
  const { title } = props;
  const classes = useStyles();
  return
{title}
;
};

src/components/features/ctf-components/ctf-demo/ctf-demo-gql.tsx
React component which executes the GraphQL query and passes the result to a component for rendering.
import { useContentfulLiveUpdates } from '@contentful/live-preview/react';
import React from 'react';

import { useCtfDemoQuery } from './__generated/ctf-demo.generated';
import { CtfDemo } from './ctf-demo';

interface CtfDemoGqlPropsInterface {
  id: string;
  locale: string;
  preview: boolean;
}

export const CtfDemoGql = ({ id, locale, preview }: CtfDemoGqlPropsInterface) => {
  const { data, isLoading } = useCtfDemoQuery({
    id,
    locale,
    preview,
  });

  const componentDemo = useContentfulLiveUpdates(data?.componentDemo, locale);

  if (isLoading || !componentDemo) {
    return null;
  }

  return <CtfDemo {...componentDemo} />;
};

Add component to mapping
on src/mappings.ts
add import to demo component into componetMap and componentGqlMap constants
export const componentMap = {
  ComponentDemo: dynamic(() =>
    import('@src/components/features/ctf-components/ctf-demo/ctf-demo').then(module => module.CtfDemo),
  ),
  ...
}

export const componentGqlMap = {
  ...pageTopicMap,
  ComponentDemo: dynamic(() =>
    import('@src/components/features/ctf-components/ctf-demo/ctf-demo-gql').then(
      module => module.CtfDemoGql,
    ),
  ),
  ...
}

Run your code
yarn dev

You should see your component display on http://localhost:3000/

Hi @zecka,
Thank you so much for the great feedback and for sharing your process to achieve your goal of creating a new Contentful component ❤.
We actually dedicate some sections of the README to the Contentful Components and how to create them.
That said, we focus entirely on the code side of things and don't touch the Contentful Web App part.
We have a tutorial which guides one into what you basically had to do.
We mention this guide in the main README but maybe it deserves more visibility/clarity?


Fix Downloads Links to new ASP.Net 4.x Site

I changed the download links to the new ASP.NET 4.x  docs t site on GitHub.

Thank you for fixing this, @Avaruz!


ENH: Methods previous / next for RegularGridInterpolator

Is your feature request related to a problem? Please describe.
The function interp1d is marked as legacy. Although the documentation is not specific as to why and what should be used as replacement. After some digging, the reason appears to be that the function is based on FITPACK, which is considered legacy (mostly resulting from controversy around the interp2d function). The recommended replacement for interp2d (and by extension interp1d) is RegularGridInterpolator. In comparison to interp1d it seems to lack functionality however, especially methods previous and next. The former is e.g. frequently used for zero-order hold computations.
Describe the solution you'd like.
While there's open questions like how previous and next are to be interpreted in higher dimensions, it seems that functionality is lost should interp1d be eventually removed without a suitable replacement for the previous and next interpolation modes. I suggest to think about adding these modes to RegularGridInterpolator - possibly restricted to the 1d case.
Describe alternatives you've considered.
Rewrite interp1d without dependency on legacy FITPACK code.
Additional context (e.g. screenshots, GIFs)
No response

TL;DR : there is planned functionality loss, there's no need to replace anything.
Let's clear some background first. interp1d is declared legacy not deprecated. This is to say

there is no plan to remove interp1d;
we are not very interested in adding new features to interp1d (but this is not prohibited either, we can consider a well-motivated proposal, just the bar is quite high);
we believe that for almost everything which interp1d does, there's a better way;
there is no plan to remove interp1d.

nearest, previous and next modes are one exception to the last point. They do not have a dedicated replacement in the scipy.interpolate namespace. That said, they are not rocket science either: https://github.com/scipy/scipy/blob/v1.11.1/scipy/interpolate/_interpolate.py#L725

The recommended replacement for interp2d (and by extension interp1d) is ...

There is no extension here. interp2d is deprecated and will be removed, and there are at least two ways to mitigate the removal: use RGI or bug-for-bug compatible FITPACK replacements, as documented at
https://docs.scipy.org/doc/scipy-1.11.1/notebooks/interp_transition_guide.html

the reason  appears to be that the function is based on FITPACK,

Not really. interp1d is not fitpack based.
Now, for the RGI:  we definitely will not add anything d=1 only to RGI; just as you say, generalization to d > 1 is not obvious and so far there is no clear need to even think about it IMO.
So, the gist of this issue seems to be based on misunderstanding.
The solution is probably to improve the documentation, and we would greatly appreciate suggestions and especially pull requests with doc improvements.
(and you are probably best positioned for such improvements @orangelynx).

So, the gist of this issue seems to be based on misunderstanding. The solution is probably to improve the documentation
I agree, but there's at least one implementation point worth discussing (see below).

interp1d is declared legacy not deprecated. This is to say there is no plan to remove interp1d;
In terms of documentation, this is not conveyed by the current text of the legacy note:
This class is considered legacy and will no longer receive updates. This could also mean it will be removed in future SciPy versions.
The mere warning that it may be removed is enough for many developers to not use it in new code in the first place.

nearest, previous and next modes are one exception to the last point. They do not have a dedicated replacement in the scipy.interpolate namespace. That said, they are not rocket science either.
This can be said for many functions in numpy / scipy, but code becomes cleaner if even simple computations are nicely tucked away in the responsible library. The question then is whether the use case is frequent enough to warrant implementing & maintaining it? Given the use for zero-order hold upsampling I'd welcome a discussion on this. Two preliminary thoughts on this:

I see why removing the legacy status from interp1d is not desirable, as the function still has functionality that is best taken from other implementations.
A suitable replacement is perhaps best suited to the scipy.signal namespace due to the nature of the use case and the 1d restriction.

I'll think about suitable ways to rephrase the doc on interp1d.

Yes please, doc improvements would be most welcome, happy to review a PR.

No activity for a month, so I'm going to close it. Still happy to review a doc improvement PR if you come back to sending one @orangelynx !


chore: bump karpenter core 37

Fixes #N/A
Description
Bump Karpenter core
How was this change tested?
make verify
Does this change impact docs?

[ ] Yes, PR includes docs updates
[ ] Yes, issue opened: #
[x] No

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

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

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.01%) to 78.13%

Files with Coverage Reduction
New Missed Lines
%

pkg/providers/amifamily/ami.go
1
90.5%

Totals

Change from base Build<PHONE_NUMBER>7:
-0.01%

Covered Lines:
5841

Relevant Lines:
7476

💛  - Coveralls


Is there a easy way to rotate the cube without moving the world

Hi Lars, This is such a great project!
So right now when you rotate the cube (using the key events) CSXD you are moving the camera and thus the position of your hands stay the same. So in terms of the algorithm U becomes R and then B etc.
It would be nice to rotate the cube and update your hand positions too. Is there an easy way you would recommend that?

Yeah, the keyboard controls are useless right now. Just a proof of concept, really.
Are you saying you'd want to try making it better?

Ya I forked it and remapped MLRU to align with the general algorithms. But got stuck on rotating and was just looking for a clean way forward. I was envisioning a interactive mode. Where I can reset back to F2L solved and keep re-testing a random the OLL algorithms. But you could imagine the same thing for practicing solving F2L or PLL.

So one year later - does unhand() help us here?

That seems related to what you're doing, but probably not the full solution. The underlying code could make it a lot easier though.
One reason it's hard for me to answer is that I'm not sure exactly what you're trying to do.


Feature Request: Specify an AccountID instead of just "str"

In a couple of places you could specify that a field should contain a stellar account id, e.g. PaymentOperation.from_ and PaymentOperation.to:
from pydantic import ConstrainedStr

class AccountId(ConstrainedStr):
    regex = "G[A-Za-z0-9]{55}"

Hello, thank you for your suggestion, but I assume that the data provided by Horizon is reliable, so I currently have no plans to add additional verification.


[TABLE] Grid Edit and tree control

We need clarity around what this would entail and If this is needed by APM.
Any clarification and eventual UX design should be added to this thread.

this item is stale. Closing but please reopen if necessary


Compress CUDA binary code

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ The recipe is not parsable by parser conda-souschef (grayskull). Your recipe may not receive automatic updates and/or may not be compatible with conda-forge's infrastructure. Please check the logs for more information and ensure your recipe can be parsed.
ℹ The recipe is not parsable by parser conda-recipe-manager. Your recipe may not receive automatic updates and/or may not be compatible with conda-forge's infrastructure. Please check the logs for more information and ensure your recipe can be parsed.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/12267618958. Examine the logs at this URL for more detail.

Did you measure the difference between the two approaches? Would be interested to know how they compare

2GB -> 620 MB

This is unexpected to me since compression is enabled by default, but I guess compression has overhead for small files.
https://docs.nvidia.com/cuda/nvfatbin/index.html#supported-options

@conda-forge-admin, please rerender

I forgot the bot doesn't have permission to build on the CI.

Thanks, @isuruf


C# is missing in the Programming Languages list

Title. How come?

No one submitted a list for it.

Thanks, I will!
Il giorno mar 8 ott 2019 alle ore 17:11 Timothy Sturm <
<EMAIL_ADDRESS>ha scritto:

You might want to look at the .NET list item instead
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sindresorhus/awesome/issues/1625?email_source=notifications&email_token=ABVZIGJ664BYUITPOQJH4HDQNSPI5A5CNFSM4I5XAXQKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEAUPOJY#issuecomment-539555623,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABVZIGPG57VET7UNAZUMS73QNSPI5ANCNFSM4I5XAXQA
.


fisher ls error

Hi,
fisher ls returns:
test returned eval errors:
	invalid integer ''

fisher ls-remote is working ok. fisher up updated plugins.
fish -v - fish, version 2.5.0
fisher -v - fisherman version 2.11.0 ~/.config/fish/functions/fisher.fish
I have some customizations of the theme I'm using (budspencer), some symlinked directories, but it would be kinda hard to give you info on all that :)
If I can provide something specific to help you help me with this, please ask :)

@predmijat Fixed! Thanks for reporting.


Demo takes you to Countup repo

Hi,
Just wanted to let you know that the demo page takes you to the Countup repo instead of this repo.
Thanks

Thank you! Fixed.


Correctly load KeyValue for both relations and resources

Fixes #486
When using KeyValue on a relation, value at init (Alpine init) will be a Proxy to Livewire, therefore there's no content to be loaded. I found that a simple solution is to adapt the $watch of value and, if it's an object (the user values, because sometimes it can be also a Proxy instance) update the rows.
I added a condition to check if that row exists or create it to be backward compatible also with KeyValue inside a Resource.
My first solution was to recreate the rows array and populate it with the new values, but if you clicked "Add row" and then edited another row, the new (empty) row would disappear.
Repo to reproduce the issue: https://github.com/danilopolani/filament-demo

Thank you!!


Import through plugin created new workspaces instead importing in current one

Expected Behavior
Import JSON content through plugin context.data into the current workspace or given workspace id (which was possible in the past and seems to be gone now).
Actual Behavior
With every import Insomnia creates a new workspace instead using the current one.,
Reproduction Steps

Create a plugin with minimal code in index.js eg:

module.exports.workspaceActions = [
    {
        label: 'Pull from GIT',
        icon: 'fa-arrow-down',
        action: async (context, data) => {
            const impFilename = 'example.json';
            if (!fs.existsSync(impFilename)) {
                context.app.alert('Error importing',
                    'Seems example.json is not existing!');
                return false;
            }

            fs.readFile(impFilename, "utf8", function (err, fileContent) {
                context.data.import.raw(fileContent, {
                    workspaceId: data.workspace._id,
                });
            });
        },
    }
];

Use example.json eg. with the following content
{
  "_type": "export",
  "__export_format": 4,
  "__export_date": "2023-07-09T17:27:43.046Z",
  "__export_source": "insomnia.desktop.app:v2023.4.0",
  "resources": [
    {
      "_id": "req_44dcc4a6c60b42b4b91cf8888a2d9b1b",
      "parentId": "__WORKSPACE_ID__",
      "modified": "1637671845661",
      "created":<PHONE_NUMBER>875,
      "url": "https://google.de",
      "name": "New Request",
      "description": "",
      "method": "GET",
      "body": {},
      "parameters": [],
      "headers": [],
      "authentication": {},
      "metaSortKey": -1688992190875,
      "isPrivate": false,
      "settingStoreCookies": true,
      "settingSendCookies": true,
      "settingDisableRenderRequestBody": false,
      "settingEncodeUrl": true,
      "settingRebuildPath": true,
      "settingFollowRedirects": "global",
      "_type": "request"
    },
    {
      "_id": "__WORKSPACE_ID__",
      "parentId": null,
      "modified": "1637671845661",
      "created":<PHONE_NUMBER>630,
      "name": "My Collection",
      "description": "",
      "scope": "collection",
      "_type": "workspace"
    },
    {
      "_id": "__BASE_ENVIRONMENT_ID__",
      "parentId": "__WORKSPACE_ID__",
      "modified": "1637671845661",
      "created":<PHONE_NUMBER>636,
      "name": "Base Environment",
      "data": {
        "url": "https://google.de"
      },
      "dataPropertyOrder": {
        "&": [
          "url"
        ]
      },
      "color": null,
      "isPrivate": false,
      "metaSortKey":<PHONE_NUMBER>636,
      "_type": "environment"
    }
  ]
}

Try to import it. The ImportOptions were removed so it is not possible anymore to send a workspaceId.  This basically switches the import of workspaces to import of projects and breaks many plugins: https://github.com/Kong/insomnia/pull/6044
Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
After this feature: https://github.com/Kong/insomnia/pull/5869, the requests were shown multiple times with every import - reported here: https://github.com/Kong/insomnia/issues/5946
I don't see this behavior within the newer versions anymore, but with every import, a new request collection is created instead of updating the current one.
The issue might be related to the mentioned above one.
Insomnia Version
2023.4.0
What operating system are you using?
macOS
Operating System Version
macOS Ventura 13.3.1
Installation method
download from insomnia.rest
Last Known Working Insomnia version
2023.1.0

@jackkav As far as I see there is no check if the workspace already exists (in importResourcesToProject) and it is always imported as a new workspace:
const r = await Promise.all(resources.filter(isWorkspace)
    .map(resource => importResourcesToNewWorkspace(projectId, resource)));

shouldn't the method to import resources into a project check if the workspace already exists and then not import into a new workspace?
something like this:
const r = await Promise.all(resources.filter(isWorkspace)
    .map(async (resource) => {
      let isExistingWorkspace = false;
      if (resource && resource._id) {
        const existingWorkspace = await models.workspace.getById(resource._id);
        if (existingWorkspace) {
          isExistingWorkspace = true;
        }
      }
      if (isExistingWorkspace) {
        await importResourcesToWorkspace({
          workspaceId: resource._id,
        })
      } else {
        await importResourcesToNewWorkspace(projectId, resource)
      }
    }));

@sebastiande thanks for you contributions, there will be improvements to the git sync feature in our next release.

@jackkav thanks for the answer, but it is not about git sync it is about the functionality for plugins to import or export workspaces. This could be used for git sync but as well for other use cases.
Anyways what does the git sync mean? As far as I know it is only for design documents and not for request collections?

I've suggested sync method for plugins some time ago
https://github.com/Kong/insomnia/pull/6078

any news on this issue?

Anyways what does the git sync mean? As far as I know it is only for design documents and not for request collections?

team sync and git sync should now be working on both design docs and collections.
Import and export is for backing up and single instance sharing.
Sync is an important feature to us but we only have plans to support team sync and git sync going forward. I'm sure you can all appreciate that in order to provide better quality features we have to cut scope. We would greatly appreciate ideas on improving the above mentioned sync methods.
Thanks for your patience and understanding.

I created a PR to fix this issue https://github.com/Kong/insomnia/pull/7081

Hi @jackkav, I just faced the same problem on a plugin that I am working on. I didn't understand this part of your response:

Import and export is for backing up and single instance sharing.

This feature, following the documentation the documentation, is very clear that we can use it to import something on a specific (optional) workspaceId.
What does that mean, actually? Since this is a known issue and perhaps a feature that does not reflects what the plugins are capable of, I think that the correct is that the documentation should be updated.
Thanks.


Support 3D pose estimation

lightweight-human-pose-estimation-3d-demo.pytorch
repo: https://github.com/Daniil-Osokin/lightweight-human-pose-estimation-3d-demo.pytorch
Models

pytorch model: human-pose-estimation-3d.pth.zip
onnx model: human-pose-estimation-3d.onnx.zip
mlmodel: human-pose-estimation-3d.mlmodel.zip

Metadata

input: a 256x448 color image
outputs:

features:
heatmaps:
pafs:

Parsing Reference

https://github.com/Daniil-Osokin/lightweight-human-pose-estimation-3d-demo.pytorch/blob/master/modules/parse_poses.py
https://github.com/Daniil-Osokin/lightweight-human-pose-estimation-3d-demo.pytorch/blob/master/modules/legacy_pose_extractor.py

Hi @tucan9389 how about using ARKit's 3D motion tracking: https://developer.apple.com/documentation/arkit/capturing_body_motion_in_3d ? Or do you think your proposed 3D pose estimation is more accurate?

@sebo361
Thanks for your suggestion. I agree with that If the case prefers more.
But I think there is some other side of benefits when using Core ML rather than AR Kit.

When you want to inference person's 3D keypoints with a single image
When you want to inference not only a person's but also an object's 3D keypoints which was trained by yourself

@tucan9389
True, these are interesting benefits to find out! However I am more interested on comparing both approaches regarding the inference speed and accuracy of 3D human pose keypoints. Now as the latest iPad Pro (and iPhone 12) have the LIDAR sensor, i expect to have more precision in 3D human motion tracking, but I am note sure if ARKit's 3D motion tracking uses LIDAR Depth API by default - do you have any insights of that?
Maybe I should setup a project to compare ARKit's 3D motion tracking with your proposed CoreML 3D pose estimation.

@sebo361
Also, Apple's 3D body tracking ARBodyTrackingConfiguration can't work simultaneously with the world ARWorldTrackingConfiguration. So, unfortunately, you can't use Apple's 3D body tracking if you also need world tracking

https://github.com/tucan9389/PoseEstimation-TFLiteSwift
Here is 3D pose estimation demo by using TFLiteSwift. I implemented softargmax with pure Swift and Accelerate framework. In Numpy or Pytorch(or TF), there support dimension summation to use easily, but in swift there is no softargmax function in multi-dimension matrix(tensor). So the implementation of softargmax is a little bit hard.


Refs #RHIROS-1312 - handle inventory and hosts admin

Why do we need this change? :thought_balloon:
When user has 'inventory:hosts:' or 'inventory:hosts:read' or 'inventory::' or 'inventory::read' permissions
and resource definition does not have any groups it means user can see all the systems on ROS
Documentation update? :memo:

[ ] Yes
[ ] No

Security Checklist :lock:
Upon raising this PR please go through RedHatInsights/secure-coding-checklist
:guardsman: Checklist :dart:

[ ] Bugfix
[ ] New Feature
[ ] Refactor
[ ] Unittests Added
[ ] DRY code
[ ] Dependency Added
[ ] DB Migration Added

Additional :mega:
Feel free to add any other relevant details such as links, notes, screenshots, here.

Codecov Report
Patch coverage is 100.00% of modified lines.

Files Changed
Coverage

ros/api/common/add_group_filter.py
100.00%

ros/lib/rbac_interface.py
100.00%

:loudspeaker: Thoughts on this report? Let us know!.

Appreciate all the reviews! thanks @kgaikwad @PreetiW and @kruai ! Merging this now!


Possible to remove hard dependency on torch 1.10.0 ?

Would it be possible to remove the hard dependency on torch == 1.10.0 (this line) to something softer like torch >= 1.10.0 (this line) ?

torch==1.10.0 is required to fully reproduce our experiment results.
Updating torch to newest version does not break the training. But we did observe differences in performance. See issue (https://github.com/salesforce/LAVIS/issues/40) for reference.
Thanks.

This is now relaxed to torch>=1.10 in https://github.com/salesforce/LAVIS/commit/cd4db98c93d87abe926c9c0e21093f4c56f856de.
However, please be aware the for best reproducing the training for ALBEF and BLIP models, torch=1.10 is recommended.


Fix helm install not working out of the box for baserow chart

Hi,
I was attempting to install the Baserow chart locally and hit two issues (i think).

It looks like when Values.backend.config.email.password is not set the resulting email-secret.yaml ends up with a nil email-password value instead of the default "".

> helm install 1.5.3 christianknell/baserow
Error: INSTALLATION FAILED: unable to build kubernetes objects from release manifest: error validating "": error validating data: unknown object type "nil" in Secret.data.email-password

Second issue was

Error: INSTALLATION FAILED: unable to build kubernetes objects from release manifest: error validating "": error validating data: unknown object type "nil" in ConfigMap.data.BASEROW_IMPORT_TOLERATED_TYPE_ERROR_THRESHOLD

This env variable is incorrectly mentioned in Baserow's docs and no longer exists inside of Baserow. By removing it entirely I fixed this error.

One extra change I just pushed is to make the commands suggested by the notes forward the asgi container port to the host instead of the wsgi. This way the websocket connections and realtime collab will work out of the box.
Please let me know if you'd prefer these changes to be split into separate PRs very happy to do so.


API and internals improvements

I've changed it to use a Dynamic AppState, instead of an update event. I'm pretty sure this is part of what fixed my weird performEvent issue
Removed the MVar that takes reflex events to brick events. Instead, reflex events are sent over the BChan
Updated the examples to compile
Make snake a little nicer, by gating the direction inputs :)

I'm wondering if the second argument to runReflexBrickApp is redundant now.


[SPARK-40081][PYTHON][DOCS] Add document parameters for pyspark.sql.streaming.query

What changes were proposed in this pull request?

As mentioned https://github.com/apache/spark/pull/37482#issuecomment-1212245579,  this PR proposes to add document parameters for pyspark.sql.streaming.query.
Why are the changes needed?

To make the documentation more readable.
Does this PR introduce any user-facing change?

Yes, it changes the documentation.
How was this patch tested?

Manually ran each doctest.

cc @HyukjinKwon @xinrong-meng
It would be good if you could take a look when you have time, thanks!

Can one of the admins verify this patch?

cc @HeartSaVioR if you find some time to take a look

Looks pretty good otherwise.

Merged to master.

Thanks for working on this @dcoliversun


feat(cloudfront): define lambda@edge as resolvable resource

This declaration is required for deploying custom resources as lamdba
association, which by itself is required to deploy a lambda@edge for a
stack which is in a different region as 'us-east-1'.
Relates to #1575

Pull Request Checklist

[x] Testing

Unit test added (prefer not to modify an existing test, otherwise, it's probably a breaking change)
CLI change?: coordinate update of integration tests with team
cdk-init template change?: coordinated update of integration tests with team

[x] Docs

jsdocs: All public APIs documented
README: README and/or documentation topic updated
Design: For significant features, design document added to design folder

[x] Title and Description

Change type: title prefixed with fix, feat and module name in parens, which will appear in changelog
Title: use lower-case and doesn't end with a period
Breaking?: last paragraph: "BREAKING CHANGE: <describe what changed + link for details>"
Issues: Indicate issues fixed via: "Fixes #xxx" or "Closes #xxx"

[ ] Sensitive Modules (requires 2 PR approvers)

IAM Policy Document (in @aws-cdk/aws-iam)
EC2 Security Groups and ACLs (in @aws-cdk/aws-ec2)
Grant APIs (only if not based on official documentation with a reference)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license.

@rix0rrr Would you mind to review the second part of the lambda@edge support?

@rix0rrr I've integrated all your required changes

@rix0rrr At least everything compiles now and the test look fine. I'm not sure if this is working, since using this in our project with the current CDK build and link-all.sh script result in compile error in our project from the CDK. I'm not able to create a working example right now.

@rix0rrr Do you have an option to get around the open issue/change? I have no idea to solve this. Could you please rereview?

@rix0rrr I've rebased this to the latest release.

Temporarily not merging while we are releasing.


Empty performance rows on spreadsheet causes script to fail

While the code should address this, there should also be validation on the spreadsheet.


Add built-in support for automatic metrics on REST client outbound requests

Environment Details

Helidon Version: 3.x
Helidon SE or Helidon MP MP
JDK version:
OS:
Docker version (if applicable):

Problem Description
The MicroProfile REST Client spec https://download.eclipse.org/microprofile/microprofile-rest-client-3.0/microprofile-rest-client-spec-3.0.html#_other_microprofile_technologies briefly refers to possible support from metrics wrt REST clients, but defers to the MP Metrics spec which says nothing about this topic.
We have had (at least) one user request for automatic metrics on outgoing MP REST client calls and we should decide whether to implement it. This would involve deciding what specific metrics to measure, what naming convention to use for MetricIDs, etc.
Helidon already supports two types of automatic metrics on inbound requests:

All requests together are counted and metered (SE and MP).
MP requests can be measured using a separate SimpleTimer for each RESTful web service endpoint.

Related to but different from #6057

now there are two users requesting this :)

Please add support for annotated metrics on REST Client interfaces. My teams uses a shared library for our REST Client and this feature would help instrument metrics consistently across all of our services. Currently, we use a filter to handle this but it doesn't allow us to measure the individual requests without a lot of parsing. Here's what we are hoping to see...
 @Timed     @GET     @Path("/somepath")     Response getSomething(@PathParam("id") String id);


[RemoteMirror] Bump swift_reflection_libraryVersion to 3.

This indicates the presence of the fix for the async task slab size calculation in f3493bf1491c5a145b9945e048397fe76f50b8d4.
rdar://87607280

@swift-ci please smoke test and merge


how to run Armatus using hic file as input

Hi, dear all,
I am new to use Armutas. I am wondering how to run Armatus using hic file as input. Thanks!

Hi,
Thank you for your interest. You can read the Armatus README file from GitHub as the starting point, it tells you how to run Armatus from several hic input files. Feel free to contact us if you have further questions.

For the first question:
You can download via git clone https://github.com/kingsfordgroup/armatus.git and go to the directory ./armatus/binaries to see if you can directly run those executive files, otherwise you may need to compile it by yourself.
For the second question,
For the first two columns, you need to divide them by the resolution.
Hope it helps.

Hi, Yihang,
I cloned the git and in the /armatus/binaries directory there are three files:
armatus-linux-x64*
armatus-v2.3-macos-10.12*
armatus-v2.3-ubuntu-x86_64
I just run the file name in linux like this 'armatus-linux-x64', it said "armatus-linux-x64: command not found". Am I using that correctly?
Thanks

try to use ./armatus-linux-x64 instead of armatus-linux-x64

try to use ./armatus-linux-x64 instead of armatus-linux-x64

Hi, Yihang,
When I run it, it said "./armatus-linux-x64: error while loading shared libraries: libboost_iostreams.so.1.55.0: cannot open shared object file: No such file or directory".
It seems to be Boost problem, I have installed Boost followed this link: https://www.linuxfromscratch.org/blfs/view/7.5/general/boost.html. But the installation is not successful since I am not the sudo user. Do you have any idea to install this?
Thanks.

Yeah installing the libraries is always annoying. I think the best way is to contact the person that helps maintain your cluster to see if he/she can help.


Moz/refactor/dockerfiles

splits up dockerfiles and compresses their builds to minimize size:
otomi/core:
1.7Gi before
1.2Gi after

you may inspect the Dockerfiles? It is not complex. As the description mentions: it saves 500MB size when leaving out unnecesary tools, but it adds this setup to maintain. Will be beneficial when we get traction

yes, we will do it in this week.

We should consider using this tool: https://github.com/docker-slim/docker-slim

Tnx @diabhey for the suggestions. I rejected them in favor of shellcheck exclusions for our Dockerfile(s). String split should only warn.

I prefer not to. I think we should use minimal base images that are secure.
We are not in need of creating base images ourselves
On Tue, 17 May 2022 at 08:17, Abhimanyu Selvan @.***>
wrote:

We should consider using this tool:
https://github.com/docker-slim/docker-slim
—
Reply to this email directly, view it on GitHub
https://github.com/redkubes/otomi-core/pull/706#issuecomment-1128457251,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHF6KMVPGTGQSQQQNV7E33VKM2XTANCNFSM5LIHYOHA
.
You are receiving this because you authored the thread.Message ID:
@.***>

you may inspect the Dockerfiles? It is not complex. As the description mentions: it saves 500MB size when leaving out unnecesary tools, but it adds this setup to maintain. Will be beneficial when we get traction

I inspected them but it is a bit convoluted, because they are released in many places, so it is difficult to grasp an idea and check if order of build pipeline is correct.
So far I see that there are four container images now:

core
base
tools
cli
How do they depend on each other?

Btw, it does not sound like high priority now, doesn't it?

you may inspect the Dockerfiles? It is not complex. As the description mentions: it saves 500MB size when leaving out unnecesary tools, but it adds this setup to maintain. Will be beneficial when we get traction

I inspected them but it is a bit convoluted, because they are released in many places, so it is difficult to grasp an idea and check if order of build pipeline is correct. So far I see that there are four container images now:

core
base
tools
cli
How do they depend on each other?

C'mon, those dockerfiles are easy to read. What do you not get? Only core is split to also deliver cli

Btw, it does not sound like high priority now, doesn't it?

Not really, but saving 500Mb is quite a lot, so it would be beneficial for many reasons. Having all nodes pulling the otomi container when assigned a drone runner is costly.


CI pipeline for building and publishing the EiriniX helm chart

Is your feature request related to a problem? Please describe.
Concourse pipeline for building the EiriniX helm chart
Should publish the helm chart tar.gz as an asset on the github releases page.
Additional context
#578 #652

Blocked on #652 (because there's nothing to make a CI pipeline for yet).

Note to self: need to make sure we tag the images correctly (as in, not just latest).

@mook-as #652 is done - is this unblocked now?

Yep, this is unblocked (and I'm working on it).

Ooops, the CI built the chart and pushed to S3… and didn't update any index.yaml anywhere, so it wasn't published in any way that we could consume from kubecf.
Reading the original description, I guess I should put it on github releases somewhere, instead. Will figure out the requisite credentials.  In that case, I'll also make GitHub (pages) be a helm repository.

The helm chart updating is having issues: when the chart is updated (because the docker images changes, for example), the GitHub release is updated but it's not triggering a helm repo index rebuild.  Hopefully fixed in https://github.com/SUSE/eirinix-helm-release/pull/8.


[8.x] Change session to get a value from given key

Tests now can define a value by sending an array to session and get the value by sending the key string.
$this->session(['name' => 'john']);
$this->session('name'); // john

Thanks for your pull request to Laravel!
Unfortunately, I'm going to delay merging this code for now. To preserve our ability to adequately maintain the framework, we need to be very careful regarding the amount of code we include.
If possible, please consider releasing your code as a package so that the community can still take advantage of your contributions!
If you feel absolutely certain that this code corrects a bug in the framework, please "@" mention me in a follow-up comment with further explanation so that GitHub will send me a notification of your response.


Publish v0.4.0

v0.3.0 is the latest published version, v0.4.0 patches a memory leak

@Matt-Esch I'll review all the code and publish a new version this week. Thank you for your patience!

@Venemo next week was some weeks ago now. i'd really like to reference this module in my package.json from npm instead of from git. would you please be so kind and npm publish now?

Sorry guys. I published it now!

New version number is 0.4.1 :)

yay, thanks!


Issue on upgrading to 0.9.10

Hi, previously when I'm using version 0.7.4. I use to get the Item object from onItemAdded() as shown below , But when I upgraded the version to 0.9.10, I'm getting only two properties ie. ItemId and display value as mentioned below. I need the whole object to be returned similar to 0.7.4 in 0.9.10 version.
version 0.7.4:
apexUniqueId:"BRAND-1122"
confidence:"UNSCORED"
core:"BRAND"
displayHTML:"Refresh Your car"
displayValue:"Refresh Your Car"
itemId:1111
pla:"RYC"
productLine:"Refresh Your Car"
score:1.1094004
version 0.9.10:
displayValue:"Refresh Your Car"
itemId:1111

Hi @Gbuomprisco ,
I'm just wondering!, How soon it will get fixed?, because our project need to go in production

Hi @omprakash9,
sorry - not before the weekend

Okay @Gbuomprisco ,
Please let me know once it get fixed. Thankyou!

Hi @omprakash9,
if you could pull https://github.com/Gbuomprisco/ng2-tag-input/pull/293 and try it out to see if it fixes you issues (this and IE related), it would be great! I can't test IE/Spark unfortunately

@Gbuomprisco ,
Can you please publish it with new version, So that I will update my plugin and will check it in my local

@omprakash9
I'll probably publish under the tag beta, in case there are any bugs

Hi @omprakash9,
I release under the tag beta, so you need to specify the version (1.0.0 -  🎉 ). Let me know! :)

@Gbuomprisco
It is now fixed, returning the whole object, Thanks


Can't run this challenge :(

Challenge http://www.freecodecamp.com/challenges/bonfire-return-largest-numbers-in-arrays has an issue. Please describe how to reproduce it, and include links to screenshots if possible.
It doesn't matter what code you write, your get the same error:
Cannot read property 'eql' of undefined
Even chopping the function call to a straight text string gives the same error.

@dalebulbrook thanks for posting this issue!
Note that your issue is a duplicate:
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/2876
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/2874
There is a PR https://github.com/FreeCodeCamp/FreeCodeCamp/pull/2879 which is waiting for approvement.

Please review the Guidelines for Contributing, thank you!.

Fixed


Add console command example + docs

As per https://github.com/doublesecretagency/craft-businesslogic/issues/3, adding example for console controller + documentation

Merged, thanks! 👍


Variables in test steps

I include variables in my test.step to make it even more intuitive and easy to follow but how would you something like the below using magic steps?
await test.step(`Enter search term '${searchTerm}' in to search bar`, async () => {

Yes, you can use variables in step text like in regular template literals, e.g.:
const searchTerm = 'foo';
// step: Enter search term '${searchTerm}' in to search bar

Report:

Updated README.


No output when using AppleIntelInfo.kext in Tools tab?

I don't know when this bug begins, but it worked well some days ago.

same to #25


evscript is not installed with setuid

Describe the bug
evscript is not installed with setuid, so one is forced to either launch it with sudo or su, or to allow the user to access /dev/input/event*.
Steps To Reproduce
Steps to reproduce the behavior:

Install evscript with nixos-rebuild switch or alike.
Launch evscript -d /dev/input/event1. This will crash with evdev open(): Sys(EACCES).
Launch sudo evscript -d /dev/input/event1. It works (and hangs).

Expected behavior
evscript -d /dev/input/event1 should work as the sudo version. According to the README, you need to install the binary with the appropriate rights.
Indeed, the following works
$ sudo install -Ss -o root -m 4755 `which evscript` here
$ ./here -d /dev/input/event1

Notify maintainers
@MilesBreslin
Metadata
$ nix run nixpkgs#nix-info -- -m
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.68, NixOS, 22.11 (Raccoon), 22.11.20220918.f677051`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.11.0`
 - channels(root): `"nixos-22.05pre338578.a2e281f5770"`
 - channels(user): `"home-manager"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`
$

Closing because I cannot fix. This is a limitation of the nix store. To get the setuid bits on NixOS, use security.wrappers to install. I personally workaround this issue by running as a systemd service under the root user. See #99101 as an example of a similar issue.


Ability to access the client SSL Certificate

I did find any way to access the clients SSL certificate when used for client authentication. From your article on ranch it seems that access to the socket is needed and adding a function like this to cowboy_req:
cert(#http_req{socket = Socket}) ->
    ssl:peercert(Socket).

might already do the trick?

cowboy_req:get(socket, Req)
Don't think we need anything special, this covers it. And while currently undocumented/internal, this isn't going anywhere and will be documented in 2.0 with a big warning about using this (and other) functions.

That works wonderful :) :+1: thanks!

It looks like this is no longer supported in version 2.0? Is there another way to get this information?
Thanks,
Willem

Open a separate ticket, sounds like this was completely forgotten. It's not currently impossible, but it does involve having a custom protocol that then calls cowboy_tls so it's not user friendly enough.


Multiple root types in schema

The c++ flatbuffers code generated by the rlbot.fbs schema does not contain the helper function GetGameTickPacket to cast a const void* to a const GameTickPacket*. From what I can gather this has to do with flatbuffer schemas only allowing one root_type for each schema, rlbot.fbs defines two:
https://github.com/RLBot/flatbuffers-schema/blob/34c2871b46942b2f851100d53d663af06c727073/rlbot.fbs#L224
https://github.com/RLBot/flatbuffers-schema/blob/34c2871b46942b2f851100d53d663af06c727073/rlbot.fbs#L247
Generating the c++ flatbuffers code again after removing FieldInfo as a root type does create the GetGameTickPacket function. Which does indicate that this assumption is correct.
A workaround in c++ is to use the flatbuffers::GetRoot<T> function instead. However, it is probably better to split the GameTickPacket and FieldInfo tables into two schemas to avoid potential issues with supporting other languages.
Note: I did not check the other schemas, perhaps these have the same issue.

Seems like this scenario is just not really supported very well by flatbuffers. The solution to the issue I explained above is to create separate files for each root type. However, you also need to run flatc for each root type, which just creates more issues than the problem I was trying to solve.
Long story short, running flatc multiple times creates multiple implementations of any tables shared between root types (e.g. Vector3). This results in compiler errors due to having multiple definitions. You can get around this by defining different namespace for each root type but than you would still have multiple Vector3 types, for example: foo::Vector3, bar::Vector, etc..
For now I will close this issue because the solution is worse than the original problem.


Add missing peer dep axios.

Axios is specified as peer dep in cms-admin (https://github.com/vivid-planet/comet/blob/main/packages/admin/cms-admin/package.json#L152C25-L152C25)

Two thoughts:

NPM installs peer dependencies automatically since v7, why do we need to add it? I doubt we added all other peer dependencies
Why is Axios a peer dependency at all (probably my fault)? Maybe we could convert it to a normal dependency instead

we use axios for uploading dam files only, that could be easily replaced by fetch

NPM installs peer dependencies automatically since v7, why do we need to add it? I doubt we added all other peer dependencies

I prefer doing this explicitly

Why is Axios a peer dependency at all (probably my fault)? Maybe we could convert it to a normal dependency instead

in it's current form of usage (where createHttpClient is in @comet) we usually don't have an axios import in the application. But having such a factory method that is only configurable by the apiUrl is weird anyway.
And the name httpClient (and not axios) is even more weird.

I prefer doing this explicitly

same

in it's current form of usage (where createHttpClient is in @comet) we usually don't have an axios import in the application.

Yes. I found the problem because I needed to extend createHttpClient (PR in comet will follow), because I needed to add another header.


Local dependency path resolution can fail on *nix

Steps to reproduce

Create new virtual env and activate
Install a local pollination-dsl dependency
Run pollination dsl translate <name of recipe using local dependency> <some local path>

Expect:
Command fails with:
urllib.error.URLError: <urlopen error [Errno 2] No such file or directory: '//<current working directory>/<absolute path to index.json>'>

This appears to come from this line where the assumption is made that, if the path starts with file://, then it is relative to the current working directory.

Hi @tymokvo, thank you for documenting this. How important is resolving this issue for finishing the other PR for updating the docs?

Medium important, I guess. I can't really use the example that you put together until the translate command works. But I can merge the PR without the example and update the docs once I get it working if that is ok.

So I've been digging into this a bit today and I think the issue stems from the use of .split('file:///') on source URLs in multiple places in the code base. I think the best option might be to replace this pattern with urlparse(source) and then handle the case the parse result has file as its scheme property. What do you think?

If that works then I have nothing against it. urlparse was not great in handling local files and that why the .split('file:///') started to show up as an alternative solution to handle the case for local registry. I doubt this will resolve the cwd issue that you pointed above but if it does then even better! 😀

Ugh, windows. Ok, well I will try a different solution, then.

It worked fine for a plugin:

but I could create the error for a recipe. I'll give it a try later today and will report back.

Hi @tymokvo, this PR should resolve the issue on your side: (https://github.com/pollination/queenbee/pull/268). I asked for your review. 🤞


Update browserify

Summary

Addresses #229
Update browserify so that it doesn't give us security warnings anymore.

Acceptance

[x] Verify that everything is still working

Things seem to still be working. @fabulousduck going to go ahead and merge this one in as well.


d4tools-based images to use GitHub branch master when cargo installs d4tools

Description
Changed

Modified d4tools-based images to use GitHub branch main when cargo installs d4tools  (closes #29)

Fixed

"GLIBC_2.x not found" error in d4tools image (fix #32 )

How to test

[x] Build the images and make sure they have they can be executed with the latest feature (stat perc_cov)

Review

[x] Tests executed by CR
[ ] "Merge and deploy" approved by

This version is a

[ ] MAJOR - when you make incompatible API changes
[x] MINOR - when you add functionality in a backwards compatible manner
[ ] PATCH - when you make backwards compatible bug fixes or documentation/instructions

Docker build OK:

Test running commands

Running d4tools interactively from terminal the new perc_cov command is available:

d4tools image pushed to Docker Hub:

python3.11-venv-d4tools images pushed to Docker Hub

python3.11-venv-pyd4 pushed to Docker Hub


Language support: autocomplete does not decrease indent

Issue Type: Bug
Create support for a new language
In language-configuration.json add:
"indentationRules": {
"increaseIndentPattern": "begin",
"decreaseIndentPattern": "end"
}
When typing "end", indent is decreased fine, but when an auto-suggestion of "end" is accepted, the indent is not decreased.
I would expect accepting auto-suggestion to behave the same as if I typed the full word
VS Code version: Code 1.46.1 (cd9ea6488829f560dc949a8b2fb789f3cdc05f5d, 2020-06-17T21:17:14.222Z)
OS version: Darwin x64 19.5.0

Indentation rules are only applied to user gesture, e.g typing text, and not to computed modifications. Those should be already computed with the right indentation.

Except they are not
On Mon, 6 Jul 2020 at 09:03, Johannes Rieken<EMAIL_ADDRESS>wrote:

Indentation rules are only applied to user gesture, e.g typing text, and
not to computed modifications. Those should be already computed with the
right indentation.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/101711#issuecomment-654053516,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGACJ577IUSBLL5LZSBWMD3R2FZLTANCNFSM4OQJLHPA
.

Are you developing or using the language extension?

I'm developing it with the aim to use it.
It is when it is used that the reported action happens. I suppose the
auto-suggestion is some vscode magic because I certainly haven't gotten
much beyond the simple language-configuration.json.
On Mon, 6 Jul 2020 at 09:49, Johannes Rieken<EMAIL_ADDRESS>wrote:

Are you developing or using the language extension?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/microsoft/vscode/issues/101711#issuecomment-654074862,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGACJ52FX4JNW5W5QMK5YL3R2F6ZJANCNFSM4OQJLHPA
.

This is likely simple, word-based completions. Once you have a "real" completion provider this will go away


Add close methode to close socket

Hello,
Please add a methode to close socket like this :
def close(self):
        """ Close the connection
        """
        if isinstance(self._session, requests.Session):
            self._session.close()

@aviau READY TO CLOSE (closed on #465)


add Xcode9 fileheader

Changes in this pull request
This adds the new default header option added in Xcode 9 B3. This works except for // being appended at the beginning which is referenced as a radar in the link (rdar://33451838). I used this as the default header. Let me know if you would like something else.
 * Copyright (c) 2016-present, Facebook, Inc.
 * All rights reserved.
 *
 * This source code is licensed under the BSD-style license found in the
 * LICENSE file in the root directory of this source tree. An additional grant
 * of patent rights can be found in the PATENTS file in the same directory.
 */

Issue fixed: #875
Checklist

[x] All tests pass. Demo project builds and runs.
[x] I added tests, an experiment, or detailed why my change isn't tested.
[ ] I added an entry to the CHANGELOG.md for any breaking changes, enhancements, or bug fixes.
[x] I have reviewed the contributing guide

This looks great. Quick question, do we want to do a codemod now or later to get the header consistent? Also I need to check internally to make sure a linter wont flag these files for having the wrong header format.

@rnystrom yeah probably good to codemod now 👍

I'll follow up w/ a codemod


Added HeroPhotoView class which is more suitable for using with Hero …

This PR fixes issue #15

Thanks for your contribution, I will review this in the next few days. Kudos!

Thanks for your contribution. After reviewing your code, I used the idea of storing imageInfo as a state at #22 and posted a possible solution to the hero bug in #15.


Release 1.0.2

@rultor deploy

@rultor deploy

@candrews OK, I'll try to deploy now. You can check the progress here

@rultor deploy

@candrews Done! FYI, the full log is here (took me 7min)

@rultor release, tag=1.0.2

@rultor release, tag=1.0.2

@candrews OK, I will release it now. Please check the progress here

@rultor release, tag=1.0.2

@candrews Done! FYI, the full log is here (took me 13min)


Calling prev and next when the ordering model attribute is enumerable creates an invalid SQL where clause

The model:
class Address < ApplicationRecord
  include OrderQuery
  belongs_to :network
  validates :inet, presence: true
  order_query :by_inet, [ :inet ]

  def next loop_around: false
    self.by_inet.next(loop_around)
  end

  def prev loop_around: false
    self.by_inet.previous(loop_around)
  end

end

The model field named inet is an IPAddress.
IPAddress includes Enumerable (a network IP address could be said to cover multiple IP addresses).
Calling @address.prev from a view fails with:
  Rendering addresses/show.html.erb
  Address Load (0.9ms)  SELECT "addresses".* FROM "addresses" WHERE ("addresses"."inet" <= inet '<IP_ADDRESS>/30',inet '<IP_ADDRESS>/30',inet '<IP_ADDRESS>/30',inet '<IP_ADDRESS>/30' AND ("addresses"."inet" < inet '<IP_ADDRESS>/30',inet '<IP_ADDRESS>/30',inet '<IP_ADDRESS>/30',inet '<IP_ADDRESS>/30' OR "addresses"."inet" = inet '<IP_ADDRESS>/30',inet '<IP_ADDRESS>/30',inet '<IP_ADDRESS>/30',inet '<IP_ADDRESS>/30' AND "addresses"."id" < 20)) ORDER BY "addresses"."inet" DESC, "addresses"."id" DESC LIMIT $1  [["LIMIT", 1]]
  ↳ app/models/address.rb:47:in `prev'
  Rendered addresses/show.html.erb (Duration: 2.6ms | Allocations: 2915)
Completed 500 Internal Server Error in 4ms (ActiveRecord: 1.1ms | Allocations: 3562)

ActionView::Template::Error (PG::DatatypeMismatch: ERROR:  argument of AND must be type boolean, not type inet
LINE 1: ....4/30',inet '<IP_ADDRESS>/30',inet '<IP_ADDRESS>/30',inet '<IP_ADDRESS>/...
                                                             ^

The expected SQL where clause should not contain the 4 enumerated IP addresses covered by '<IP_ADDRESS>/30'. It should contain only a single inet '<IP_ADDRESS>/30', something like:
... WHERE ("addresses"."inet" <= inet '<IP_ADDRESS>/30' ... )

I suspect this would happen for any model attributes that include Enumerable, probably as a result of splatting the attribute when building the where clause ?
FYI: The inet '<IP_ADDRESS>/30' is created by a custom PostgreSQL adapter as required to handle PG inet data types. It works as expected.
FYI: Address.by_inet works as expected:
  Rendering addresses/index.html.erb within layouts/application
  Address Load (0.4ms)  SELECT "addresses".* FROM "addresses" ORDER BY "addresses"."inet" ASC, "addresses"."id" ASC
  ↳ app/views/addresses/_table.html.erb:29

BTW: Thanks for creating this gem, it solves an unexpectedly difficult problem very well.

I finally tracked down the actual cause of the issue, sorry for the false alarm.
When quoting bound attribute values ActiveRecord enumerates objects that respond to map:
def quote_bound_value(value, c = connection)
  if value.respond_to?(:map) && !value.acts_like?(:string)
    values = value.map { |v| v.respond_to?(:id_for_database) ? v.id_for_database : v }
    if values.empty?
      c.quote(nil)
    else
      values.map! { |v| c.quote(v) }.join(",")
    end
  else
    value = value.id_for_database if value.respond_to?(:id_for_database)
    c.quote(value)
  end
end

Patching IPAddress with #acts_like? prevents the enumeration and builds a correct SQL where clause.
Thanks for listening (I really must buy a rubber duck :-).

Glad you've figured it out!