Timestamp_16 support

Currently in FIT we have timestamps which are 32bit and timestamp_16 which are 16bit difference from previous timestamp and are currently not read into nice values.
I wrote generator function based on ParseVivosmartHR. Here seems to be offical function how to do it (in Java):
mesgTimestamp += ( timestamp_16 - ( mesgTimestamp & 0xFFFF ) ) & 0xFFFF;

It works but it definitely isn't written as it should, but I don't know fitparse enough to improve it.
#Input is what we get in FitFile.get_messages
def fix_times(messages):
    timestamp=None
    timestamp_16=None
    real_time=0
    UTC_REFERENCE = 631065600  # timestamp for UTC 00:00 Dec 31 1989
    for message in messages:
        #print (message)
        #message["full_timestamp"]="TIMESTAMP"
        field = message.get("timestamp")
        if field is not None:
            timestamp = field.raw_value
            #I'm not sure if this is correct to set this to None. Without it records with just timestamp doesn't have same new timestamp anymore which could be a bug or it should be that way
            timestamp_16 = None
        tm_16 = message.get("timestamp_16")
        if tm_16 is not None:
            timestamp_16 = tm_16.raw_value
        #print(timestamp, timestamp_16)
        if timestamp_16 is None:
            ts_value = timestamp
        else:
            new_time = int( timestamp/2**16) * 2**16 + timestamp_16
            if new_time >= real_time:
                real_time = new_time
            else:
                real_time = new_time + 2**16
            ts_value = real_time
        if ts_value is not None and ts_value >= 0x10000000:
            value = datetime.datetime.utcfromtimestamp(UTC_REFERENCE
                    + ts_value)
        else:
            value = None
        message.fields.append(
                FieldData(
                    field_def=None,
                    field=FIELD_TYPE_TIMESTAMP_1,
                    parent_field=None,
                    value=value,
                    raw_value=ts_value
                    ))
        yield message

As an FYI for testing this there is already a file that has these timestamp_16 fields in the repo: tests/files/MonitoringFile.fit

Any progress on this issue?  I am struggling to make sense of heartrate records which utilize timestamp_16.

Hi @buma, I seem to be doing something wrong.  How should fix_times be called?
for message in fix_times(fitfile.get_messages()):
    ts =  message.get_value('timestamp_16')
    if ts:
        print ts

Traceback (most recent call last):
  File "/home/bs/.local/lib/python2.7/site-packages/IPython/core/interactiveshell.py", line 2878, in run_code
    exec(code_obj, self.user_global_ns, self.user_ns)
  File "<ipython-input-64-a1a3e382974a>", line 1, in <module>
    for message in fix_times(fitfile.get_messages()):
  File "<ipython-input-25-b548c6307590>", line 33, in fix_times
    FieldData(
NameError: global name 'FieldData' is not defined

Happy to rest any updates code.

Hello! Happy to see this discussed here :)
I am struggling with the same problem, and for me the formula in the top post doesn't work! Following the instructions from here, I tried to use that formula by hand in order to see if things made sense.
I downloaded my heart rate data recorded on May 1st 2019 from the Garmin Connect web page, i.e. the first data point is at 12AM.
Looping through the FitFile.get_messages(), the first heart-rate-related entry I find is this, including the message right before it:
monitoring
 * activity_type: sedentary
 * current_activity_type_intensity: (8,)
 * intensity: 0
 * timestamp: 2019-04-30 22:00:00
monitoring
 * heart_rate: 72
 * timestamp_16: 31132

Following the instructions linked above, I set
dt = datetime.datetime(2019,4,30,22)
mesgTimestamp = datetime.datetime.timestamp(dt)
timestamp_16 = 31132

but as a result of the above formula I get
print('result:', datetime.datetime.fromtimestamp(mesgTimestamp))
>> result: 2019-05-01 12:49:00

which is wrong!
Has anyone found a fix to this?

I tried to look at things in detail going step by step in the above formula:
data:
mesgTimestamp: 2019-04-30 22:00:00 -><PHONE_NUMBER> :   0b1011100110010001011111001011000
timestamp_16: 31132                              :                   0b111100110011100
start:
step1: mesgTimestamp & 0xFFFF                    :                  0b1011111001011000
step2: timestamp_16 - step1                      :                  -0b100010010111100
step3: step2 & 0xFFFF                            :                  0b1011101101000100
step4: mesgTimestamp + step3                     :                  0b1011101101000100 -><PHONE_NUMBER> -> 2019-05-01 12:49:00

and, as before the result is wrong by more than 12 hours.
(note that step2 is negative!)
Also, the instructions say

essentially swaps the lower 16 bits of the timestamp field with the timestamp_16 field

but the final result is not a 32-bit variable in which the first 16 are from the original mesgTimestamp and the last 16 are from timestamp_16.
On the other hand I tried to build it by hand, and I get the final binary variable as:
0b1011100110010001111100110011100

which then translates in this datetime:
datetime.datetime(2019, 5, 1, 3, 42, 52)

which, again, is wrong :\ but different from before!

Does this maybe help?
adjust_time.R

thanks @tuxinaut ! I tried it but I still get a sizeable difference, i.e.: datetime.datetime(2019, 4, 30, 18, 36, 44)
On the other hand I browsed the details of the code and I see that the UTC offset shift due to Garmin's different starting time is applied to timestamp, but I cannot find anything similar for timestamp_16.
Following this hypothesis I tried to work with raw timestamp* values (I guess they are the values before this UTC offset correction) and here is what I get:
# values taken from full printouts
timestamp_raw = 925596000
timestamp_16_raw = 31132

out = timestamp_raw
out += ( timestamp_16 - ( out & 0xFFFF ) ) & 0xFFFF

datetime.datetime.fromtimestamp(out + utc_offset)

The result is datetime.datetime(2019, 4, 30, 23, 1)
[here is the full notebook, just search for "925596000"]
This is the closest I got so far to the right result, and it is shifted only by one hour (the minute might be due to the granularity of the data) which might now be a timezone related issue.
What do you think @dtcooper ?

@francescolosterzo really cool work :+1: and yes I'm pretty sure that's timezone related (take a look on the code below)
>>> timestamp_16 = 31132
>>> tz = pytz.timezone('UTC')
>>> utc_offset = int(datetime.datetime.timestamp(datetime.datetime(1989,12,31,tzinfo=tz)))
>>> timestamp = int(datetime.datetime.timestamp(datetime.datetime(2019,4,30,22,0,tzinfo=tz)))
>>> timestamp_raw = timestamp - utc_offset
>>> out = timestamp_raw
>>> out += ( timestamp_16 - ( out & 0xFFFF ) ) & 0xFFFF
>>> datetime.datetime.fromtimestamp(out + utc_offset,tz=tz)
datetime.datetime(2019, 4, 30, 22, 1, tzinfo=<UTC>)

This is what I came up with after reading this thread. The generated timestamps and bpms are in line with the data in Garming Connect.
import fitparse

from datetime import datetime, timezone
from glob import glob

garmin_epoch = int(datetime.timestamp(datetime(1989, 12, 31, tzinfo=timezone.utc)))

files = sorted(glob("2023-10-25/*WELLNESS*.fit"))
for f in files:
    fitfile = fitparse.FitFile(f)
    last_timestamp = None
    for m in fitfile.get_messages("monitoring"):
        last_timestamp = m.get_raw_value("timestamp") or last_timestamp
        heart_rate = m.get_raw_value("heart_rate")
        timestamp16 = m.get_raw_value("timestamp_16")
        if last_timestamp and heart_rate and timestamp16:
            timestamp = last_timestamp
            timestamp += (timestamp16 - (last_timestamp & 0xFFFF)) & 0xFFFF
            date = datetime.fromtimestamp(garmin_epoch + timestamp)
            print(date.astimezone().strftime("%Y-%m-%dT%H:%M:%S %Z"), heart_rate)

2023-10-25T00:01:00 CEST 75
2023-10-25T00:02:00 CEST 72
2023-10-25T00:03:00 CEST 73
...


[Event Request] Codeunit 90 Purch.-Post - OnInsertReceiptLineOnAfterInitPurchRcptLine

Hi, would it be possible to modify the event OnInsertReceiptLineOnAfterInitPurchRcptLine to add xPurchLine as a parameter ?
OnInsertReceiptLineOnAfterInitPurchRcptLine(PurchRcptLine, PurchLine, ItemLedgShptEntryNo,xPurchLine);
Thanks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Update Pelias documentation

Some of the links in the Pelias documentation are not up to date.
This updates them to refer to the up to date documentation available at:
https://github.com/pelias/documentation
This also adds information about a couple of service providers for managed instances I found. There doesn't seem to be any references available to any other service providers than the "official" one (Geocode Earth). I just accidentally noticed Geoapify is also providing such service when implementing #1462 (their cloud service is not Pelias compatible but they provide managed Pelias as a separate service). I think these deserve to be listed somewhere for easier discovery.

Hi! Pelias core maintainer here (and co-founder of Geocode Earth). Figured I'd chime in here even though this PR is almost a year old.
All the information mentioned in this PR is good. The links to Pelias documentation are correct and much more relevant than the link to the general Pelias website.
Personally I'd recommend removing all mention of any Pelias-based service providers (even Geocode Earth). It makes more sense to add service providers as their own entry as you've done in some of your other PRs. Especially if, as you've mentioned, they don't actually provide a Pelias-compatible public API.
I'm happy to open a new PR with those changes if it would be merged and the original contributor of this PR isn't around any more :)

@orangejulius thanks for this. I'm going to merge this PR and then why don't you submit another one with whatever changes you'd recommend? I don't know how this PR got overlooked. (@ahukkanen sorry!)

Thanks for merging @alexreisner !
Regarding your point @orangejulius

Especially if, as you've mentioned, they don't actually provide a Pelias-compatible public API.

They do provide a default Pelias installation which is naturally Pelias compatible. Their own geocoding API is just not Pelias compatible, although I believe it has drawn some inspiration from it being quite similar. So, they are two separate things.


AAD least privilege overview - What about Microsoft Defender for Endpoint?

Can't see anything about Microsoft Defender for Endpoint in the overview.

https://docs.microsoft.com/en-us/azure/active-directory/roles/delegate-by-task

Just for read only permissions for "Machine Inventory" in the "Microsoft Defender for Endpont" (old portal, new portal), one would think that "Security Reader" would be sufficient:

https://docs.microsoft.com/en-us/azure/active-directory/roles/permissions-reference#security-reader

But no. "Security Operator", which has microsoft.windows.defenderAdvancedThreatProtection/allEntities/allTasks, is not sufficient either:

https://docs.microsoft.com/en-us/azure/active-directory/roles/permissions-reference#security-operator

Seems "Security Administrator" is least privilege for this?

https://docs.microsoft.com/en-us/azure/active-directory/roles/permissions-reference#security-administrator

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

ID: 0f9558c0-abb6-24f8-81c8-9000b80539e8
Version Independent ID: a4739135-863b-27bb-1f2e-cf297f21b4cc
Content: Least privileged roles by task - Azure Active Directory
Content Source: articles/active-directory/roles/delegate-by-task.md
Service: active-directory
Sub-service: roles
GitHub Login: @rolyon
Microsoft Alias: rolyon

"Security Operator" is not sufficient either. Is "Security Administrator" least privelege to be able to see machines?
Old portal:
https://securitycenter.windows.com/machines
New portal:
https://security.microsoft.com/machines

Oops, I hit the "close" button by mistake.

@o-l-a-v Thank you for your feedback and it is alright to reopen it :) . Thank you for sharing your findings. We will investigate and update the thread.

@o-l-a-v  Thanks for reaching out, for MDE, you can go to the MDE portal directly and create a custom role which gives you capability to choose the permissions you want :

Let me know if this helps.

@o-l-a-v  Thanks for reaching out, for MDE, you can go to the MDE portal directly and create a custom role which gives you capability to choose the permissions you want :

Let me know if this helps.

Thanks, figured that out after creating this issue.
Thats not a AAD built in role though. So I still think this issue is valid.

@o-l-a-v  For AAD roles, you need to have the security admin role.

@o-l-a-v  For AAD roles, you need to have the security admin role.

Yep. Would be great with a less privileged built in AAD role. Like "Security Reader". Until that happens, the docs should still mention what is currently the least privelege built in AAD role for MDE device inventory IMO.

@o-l-a-v Thanks for the feedback. We will engage the content development team for further investigation, they will verify and perform necessary changes in the document as needed.

Hi @o-l-a-v
Thanks for your feedback. We made an update to the Security Administrator role description.
https://docs.microsoft.com/en-us/azure/active-directory/roles/permissions-reference#security-administrator
Let us know if you have any additional feedback.
thanks

#please-close


@conda-forge-admin,please add bot automerge

[ ] I read the conda-forge documentation and could not find the solution for my problem there.

Issue:

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/setupmeta-feedstock#15.

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/setupmeta-feedstock#15.


Support CORS middleware

Support all options from https://github.com/expressjs/cors

Great addition! I will update the readme slightly and hopefully make a release sometime today.


Namespace Updates

Feedback from UX Review for Namespace Create/Edit/Detail
Add detail top to namespace pages.
Remove resource quota, Add Container Resource Limits component. I did not reuse the resource quota component because they do not deal with the same data. It is similar but the Container Resource Limits exist as a annotation on the namespace itself. Container Limits is also a set json map of 4 targets while Quotas can target many different items. Our resource quota component should be updated to allow that as we do on the ember side.
Various small updates and tweaks to styles and minor bug fixes.
rancher/dashboard#568
rancher/dashboard#569
rancher/dashboard#594
Screenshots
Create

Detail

Edit

@westlywright Can we hide the header/title in the Container Default Resource Limit tab in the detail view?
Specifically the Configure how much of the resources the container can consume by default text.

@deniseschannon Updated and I've attached the new detail screenshot below.

@vincent99 I can remove the component if you want but I added a comment as to why I left it in. Reverted my spacing fix in favor of Cody's.


Fixed "Line goes against our challenge" issue for Hacktoberfest.

Checklist:

[ ] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

This PR will address #48011

As a new contributor, we encourage you to read our contributing guidelines.
We expect our contributors to be aware of the process specific to this project. Following the guidelines religiously earns you the respect of fellow maintainers and saves everyone time.
Some examples of this are:

Do not edit off of GitHub–while you can, it's not a good idea.
Make sure you follow the PR checklist and not just tick things off; otherwise, we won't take you seriously.
Use the correct way to link PRs in the description of the PR by updating the XXXXXX. Do not just add issue numbers everywhere and anywhere you feel like.
Use mentions in comments or review requests to a minimum or not. We understand you are excited about contributing, and our maintainers will get back to you as soon as they get a chance.

We appreciate you taking the time to help us, and we hope to see more contributions from you.
Happy Contributing.

Closed this PR because source branch is main branch. I have created another PR(#48017) for this.


feat: add create page for notion

Description
add notion create page for raycast extension, now we only support create database page and append to a existing page.

Screencast

Checklist

[x] I read the extension guidelines
[x] I read the documentation about publishing
[x] I ran npm run build and tested this distribution build in Raycast
[x] I checked that files in the assets folder are used by the extension itself
[x] I checked that assets used by the README are placed outside of the metadata folder

Could this be implemented into the Create database page command?
For example, you could add My Workspace as an option at the bottom of the database selector.

Could this be implemented into the Create database page command?
For example, you could add My Workspace as an option at the bottom of the database selector.

Get，I have considered this logic; it's possible that the code is deemed to have too much coupling.

Could this be implemented into the Create database page command?
For example, you could add My Workspace as an option at the bottom of the database selector.

I thourouly review the two parts code, 80% of  CreatePageForm.tsx logic is relative with database, I hava a idea of the following code, move action part to higher level. @bkeys818 please have a look.
// PageForm.tsx
<Form
      isLoading={isLoadingDatabases || isLoadingRelationPages}
      navigationTitle={initialDatabaseId ? "Create New Page" : "Create Database Page"}
      actions={
        <ActionPanel>
          <Action
                  title="Switch database page and page mode"
                  onAction={() => {
                    setMode((databaseMode) => !databaseMode);
                  }}
         />
        </ActionPanel>
      }
    >
   databaseMode && (<DatabasePage></DatabasePage>)
   !databaseMode && (<NormalPage></NormalPage>)
</Form>

Ah, I may have misunderstood the feature you are trying to add.
Are you trying to add the ability to create a new subpage inside of the content of an existing page?

Ah, I may have misunderstood the feature you are trying to add.
Are you trying to add the ability to create a new subpage inside of the content of an existing page?

yes, that is what I want to do

Hmmmm... Ok. That's going to be difficult to do without adding features that will be unnecessary for 99% of users. If we can't think of implementing it without changing functionality or adding commands, then we'll use what you have.
Can I ask what your use case is?

Hmmmm... Ok. That's going to be difficult to do without adding features that will be unnecessary for 99% of users. If we can't think of implementing it without changing functionality or adding commands, then we'll use what you have.
Can I ask what your use case is?

Ok, I usually use it for taking a note between work

When are you using the Raycast Notion extension and need to create a new subpage inside a page?
I create subpages all the time, but only when I'm inside the app.

@thomaslombart, can I get your input here?

I see three solutions here:

Merge Create Page in Create Database page meaning that the database field should return all pages, which would add complexity to this command both on a technical and user's point of view (Add pages to the Database field and renaming it, change the UI if we're not adding a database page, hide some actions). It's not that complex but still.
Create a different command as it's done here.
Merge that in the Add Text to Page with a checkbox called Add as Page. This could simplify things but the command should use a better name to make it clear you can both add text to a page and also create a subpage.

Solutions 2 and 3 are fine to me. What do you think?

@thomaslombart, What about an action inside of Add Text to Page to create a subpage?

That's what I meant with the 3rd solution 🙂

thanks for @bkeys818 @thomaslombart , I think third solution is making sense , ie, merge the create page into "Add Text to Page". I will have a try.


Suggestion: Better error message when unable to resolve modules

Related to our woes with #8189, if the compilation errors gave some better hinting when it was unable to resolve a module, we might not have taken so long to find the solution (and we wouldn't have filed an issue!).
The idea would be that if there is a compilation error of cannot file module "whatever", to make some intelligent checks against the parameters the compilation is running under and suggest using --moduleResolution node if the user is not already doing so.

There's already --traceResolution to show which files are being searched for.
I'm not sure how we'd determine that --module node is the right thing to suggest when a file isn't found?

Happy to PR "Cannot find module '{0}'. If you're trying to import from node_modules you probably want '--moduleResolution node'" if that's deemed acceptable.

It's really not great for us to just guess at what the problem might be. There are at least a dozen reasons why module resolution might fail.

It's really not great for us to just guess at what the problem might be.

You'll do a LOT better at guessing what the problem might be than I will. I had to ask a dozen people and then file an issue before I had someone tell me what I was doing wrong. The compiler error expected me to just "know".
I think it's important to recognize when it comes to the user experience here that all of your error messages are a cue to being troubleshooting. This is one of the primary purposes of a compiler, after all, to identify problems and help the developer solve them. It would be ideal if you provided the users with some help to that effect.
Microsoft has always had really solid sets of guidelines when it comes to error messages. I don't think they don't apply here just because it's a CLI tool. There's user experience to think about.

There are at least a dozen reasons why module resolution might fail

These guidelines from Microsoft suggest listing out possible causes in order of likelihood if a cause can't be determined. Barring that, why not at least add a link to a URL with possible steps to take.

I believe what @blesh is suggesting is that if the compiler can't give you recommendations directly, TS should consider having web docs that lists all known errors w/ their error codes and then explains the error in depth and offers common solutions, e.g. using --moduleResolution node; when someone googles "Cannot find module 2307" ideally the first result would be a page on typescriptlang.org that has this info. Microsoft has been really good about this, so we just would like to see something similar here cause it's very helpful. 😄

The main issue is that when the resolution is done, it does not know if it is an error to not find the module, as the module could be an ambient module declaration (i.e. declare module "foo" { .. }). but when we are sure it is an error, later on in the checker, resolution has long been over and we do not have the context. so it is mainly an implementation/language semantics issue.

@mhegazy totes, we realize that now; there are quite a few reasons it could be failing. We're now suggesting better documentation :trollface:

We can definitely update the documentation. the current documentation is at http://www.typescriptlang.org/docs/handbook/module-resolution.html, which has the error code in it, but we can also add a SO question with the link to help users find the docs.

@mhegazy wow totally missed that doc page. It doesn't come up on the first page of google, so not sure what (if anything) you could do to help the docs have a better SEO rank. The SO question with a link would hopefully help, as you described. Thank you.


Added WP editor as new form element

Feature #126
What do you think? As wp_editor() unfortunately echos the HTML directly, I used an output buffer.
Also I used array short syntax (supported since PHP v5.4). I think it's pretty safe to use nowadays, but if you'd like to support PHP 5.3 and below, I can also use the old syntax. I simply like it more personally.

Hi Jonathan,
Thanks for doing this. Unfortunately the recommendation from the WordPress team is to maintain 5.3- compatibility as up to 15% of installations still run this version or below.
Best,
Robert
Sent from my iPhone

On 12 Jun 2016, at 19:51, Jonathan Gruber<EMAIL_ADDRESS>wrote:
Feature #126
What do you think?
Also I used array short syntax (supported since PHP v5.4). I think it's pretty safe to use nowadays, but if you'd like to support PHP 5.3 and below, I can also use the old syntax. I simply like it more personally.
You can view, comment on, or merge this pull request online at:
https://github.com/tombenner/wp-mvc/pull/136
Commit Summary
Added WP editor as new form element
File Changes
M core/helpers/mvc_form_helper.php (39)
Patch Links:
https://github.com/tombenner/wp-mvc/pull/136.patch
https://github.com/tombenner/wp-mvc/pull/136.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

No problem, I switched to the traditional syntax :).
Best,
Jonathan

Thanks!


Validate params for id field

Currently user can configure monitor certain monitor configurations with params that cause problems.
For instance, it's allow params in id field. We should validate this and don't allow params in certain fields. Since this will have an issue with how results are displayed against a monitor in UI
For example, for a lightweight monitor:
heartbeat.monitors:
- type: http
  name: Manager Login ${environment}
  id: "manager-login-${environment}" <--- this should not be allowed
  enabled: true
  urls: "https://${baseUrl}/manager/"
  schedule: '@every 5m'
  timeout: 16s
  alert.status.enabled: false

ACs:

Stop expanding variables in:

name
id
schedule
tags

Escape all escape chars in those fields so heartbeat does not interpolate them
Document that users should use local environment variables to parameterize things at push time, not runtime

Provide example of pushing to multiple environments

@shahzad31 is this just for lightweight (doesn't the risk exist for browser monitors too?)

Updated the description/ACs based on a recent discussion

this isn't needed anymore, since we skip parsing id key in kibana for params values

this has alraedy been completed


Plot 2D improvement for models with complex topographies

Hi Bryant,
This patch adds plot_2d_image function and uses it in seisflows.tools.specfem.Model
It adds NaN values to the top edge of the model before interpolation to the regular grid. In order to fix the representation issue with complex topographies. I have also tested with flat topography, it didn't cause problems.
I couldn't it make it work tricontourf. Feel free to explore similar fix for it. plot_2d_image might be slower than plot_2d_contour since it uses imshow which requires a interpolated regular grid.
Comparison image for the simple_topography_and_also_a_simple_fluid_layer specfem2D model.
from seisflows.tools.specfem import Model
m = Model("DATA")
m.plot2d(parameter="vs")

Thanks Ridvan! The new figure looks great


[PE-D][Tester C] Undo command not working for remove assigment command

After removing an assignment: removea a/ass1, then using undo command to undo the changes, an error is thrown.

Labels: severity.Medium type.FunctionalityBug
original: Lin-Shuang-Shuang/ped#19

Duplicate of #134


Archive this repo please

Please archive this repo please now 🥹🥹🙏

If nodegit gets archived, what is the alternative?

If nodegit gets archived, what is the alternative?

I think that https://github.com/octokit/core.js/

Thats not even close to a dropin replacement (only for very specific use cases might be an alternative)

If nodegit gets archived, what is the alternative?

check isomorphic-git

Albeit isomorphic-git is somewhat maintained by a person that ones to keep it alive, they are not too familar with the code base. And it has some bugs like not being to able read commits in repository that have been archived. It also misses some features like reading/writing filter-lists (which nodegit has), and it's also quite slower. I tested it a bit, on serverside I don't consider it a suitable replacement, if you do something funky in browser, of course its without alternative.

This is the most bold callout post I've ever seen on github what please be nice


feat: Auto generate README.md file

Add script to automatically generate README.md file

Can you please add a screenshot of how the README looks after it is generated...

Test Comment


IGNITE-23040 Fix GridNioSslSelfTest

Thank you for submitting the pull request to the Apache Ignite.
In order to streamline the review of the contribution
we ask you to ensure the following steps have been taken:
The Contribution Checklist

[ ] There is a single JIRA ticket related to the pull request.
[ ] The web-link to the pull request is attached to the JIRA ticket.
[ ] The JIRA ticket has the Patch Available state.
[ ] The pull request body describes changes that have been made.
The description explains WHAT and WHY was made instead of HOW.
[ ] The pull request title is treated as the final commit message.
The following pattern must be used: IGNITE-XXXX Change summary where XXXX - number of JIRA issue.
[ ] A reviewer has been mentioned through the JIRA comments
(see the Maintainers list)
[ ] The pull request has been checked by the Teamcity Bot and
the green visa attached to the JIRA ticket (see TC.Bot: Check PR)

Notes

How to Contribute
Coding abbreviation rules
Coding Guidelines
Apache Ignite Teamcity Bot

If you need any help, please email<EMAIL_ADDRESS>or ask anу advice on http://asf.slack.com #ignite channel.

https://ci2.ignite.apache.org/buildConfiguration/IgniteTests24Java8_Basic1/8054171?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&expandBuildTestsSection=true&hideProblemsFromDependencies=false&expandBuildChangesSection=true

https://ci2.ignite.apache.org/buildConfiguration/IgniteTests24Java8_Basic1/8052500?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandBuildChangesSection=true&expandBuildTestsSection=true

https://ci2.ignite.apache.org/buildConfiguration/IgniteTests24Java8_Basic1/8054298?expandBuildDeploymentsSection=false&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandBuildTestsSection=true

https://ci2.ignite.apache.org/buildConfiguration/IgniteTests24Java8_Basic1/8055252?buildTab=overview&hideTestsFromDependencies=false&hideProblemsFromDependencies=false&expandBuildChangesSection=true&expandBuildTestsSection=true&expandBuildDeploymentsSection=false

https://ci2.ignite.apache.org/buildConfiguration/IgniteTests24Java8_Basic1/8080695
https://ci2.ignite.apache.org/buildConfiguration/IgniteTests24Java8_Basic1/8080697
https://ci2.ignite.apache.org/buildConfiguration/IgniteTests24Java8_Basic1/8080703


Routing quota soft_limit toggle description is wrong

Description
While playing with user quota soft_limits, i've realized that routing soft limit toggle description is wrong.

Last toggle in the picture above ("Extra Here Isolines"), should be something like "Routing"
If there is any chance to prioritize it, it would be great to have it before the Onpremises LDS release (around two weeks from now).
Thanks!

Absolutely,
I propose "Extra Routing API calls"

->  https://github.com/CartoDB/cartodb/pull/11984


I think there are tcp, udp, and google protobuf in this. How do they work?

TCP is used to transport command. App uses command to request some jobs, and server will also reply info by TCP. Command types define in below.
https://github.com/mkckr0/audio-share/blob/0e7afa74a3a8c6579a7a6696840c7916cf3e6ee5/audio-share-app/app/src/main/java/io/github/mkckr0/audio_share_app/NetClient.kt#L230-L234
As you can see, the CMD_GET_FORMAT command request audio format info. The audio format info has multiple fields, so I use protobuf to pack them as a Message. It's convenient to access these fields.
https://github.com/mkckr0/audio-share/blob/0e7afa74a3a8c6579a7a6696840c7916cf3e6ee5/protos/client.proto#L5-L11
The client.proto file is referenced by both app side and server side.
UDP is used to transport audio PCM data with a lower latency compared to TCP.

Can you give me a detailed explanation? I now want to use C # to make a program to send audio from a PC and dock with your Java app, but I can't understand your C++code. I see that the Google prototype you wrote only sets audio parameters. The data field of the audio. Which one is used? Or is the audio directly sent using udp without requiring protobuf processing?

It seems that pData has not undergone protobuf processing, but has been sent directly through UDP. But you seem to have handled the number of bytes sent by UDP each time.

Yes, pData is directly sent by UDP
https://github.com/mkckr0/audio-share/blob/0e7afa74a3a8c6579a7a6696840c7916cf3e6ee5/audio-share-server/audio_manager.cpp#L330-L355

The protobuf only handles the audio format info at present.
https://github.com/mkckr0/audio-share/blob/0e7afa74a3a8c6579a7a6696840c7916cf3e6ee5/audio-share-server/audio_manager.cpp#L306-L328
This function writes audio format info to AudioFormat protobuf message.

Yes, pData is directly sent by UDP
https://github.com/mkckr0/audio-share/blob/0e7afa74a3a8c6579a7a6696840c7916cf3e6ee5/audio-share-server/audio_manager.cpp#L330-L355

pData is just copy then send, but need to care about MTU.

The server start by calling network_manager::start_server. It starts the loopback recording, listens a port to accept app to connect and starts to receive UDP data. Socket accepting and loopback recording are running in different coroutine, so they don't affect each other directly.
asio::co_spawn is used to start a new coroutine. The function that returns a asio::awaitable<void> type is a coroutine. In this kind of function, you can read code line by line, just as the normal function. co_await asio::async_read,  co_await asio::async_write or any other coroutine invoke begin with co_await makes the async operation to sync style, so you can imagine it's just a blocking operation although it's nonblocking in fact.
network_manager::accept_tcp_loop coroutine accepts app to connect and then starts a network_manager::read_loop coroutine to handle cmd from app. It reads the request from app and then writes a relavent reply. As you can see, every request and reply both begin with a relavent cmd_t.
network_manager::read_loop is the most important coroutine in a TCP connection. While app sends cmd_t::cmd_get_format, the server will reply a AudioFormat protobuf message that is already prepared in audio_manager::do_loopback_recording.
While app wants to play audio, the server will read  cmd_t::cmd_start_play in network_manager::read_loop. Then server add this app to _playing_peer_list and generate an id used to identify peer in UDP communication. Server sends this id to app. The app then sends this id by UDP. So the server can associate UDP peer with the TCP peer of a same app.
The app UDP peer has been added to _playing_peer_list, so the audio_manager::do_loopback_recording coroutine can send pData to app by UDP.

Google Protobuf just formats the audio format, right? I thought the packet was sent using protobuf. But the question arises. I now read a file in a.wav format. How can I format it using this protobuf?

I use NAudio as a library. How do I format audio data in your protobuf format?

Write a AudioFormat message by calling audio_manager::set_format
https://github.com/mkckr0/audio-share/blob/0e7afa74a3a8c6579a7a6696840c7916cf3e6ee5/audio-share-server/audio_manager.cpp#L306-L328
In this function, fill audio format info to a AudioFormat message object, then serialize to a byte array that is for sending to app.
I'am not familiar with NAudio, but I have seen a example about reading WAV file by NAudio.
https://markheath.net/post/trimming-wav-file-using-naudio
As the example shown, you may get all fields that is needed for filling AudioFormat message by reader.WaveFormat.
And use reader.Read to read PCM data then send it to app by UDP.

I'm a Chinese. Are you a Chinese? You can use Chinese if you can.

是的，我也是 Chinese，感谢你耐心的解答。

我们现在在做一个功能，把网页端麦克风收到的声音 ，送到手机里，让手机里的游戏，可以听到声音。类似于把手机的麦克几+喇叭，全部接管到网页上了。 这个功能太小众了，搜索都不好找，不知道您有什么好的见意吗?

好的。感谢。


Wrong number in comments

Testing #5862
In the last comment, as you can see, it shows the first comment with its text, and then it says 3 comments. That makes me think that in addition to the one shown first there are 3 more, when that is not the case.
It makes more sense for me to say 2 comments.

That's intentional, but I can see how it would be confusing.

@paulacamargo25 - can you verify this issue since you have already experienced it?


Adding name to parameters via earthsim.parameters causes silent failure

From Issue #254 example code from @philippjfr , I modified to add mesh_opts as an example:
from earthsim import parameters
from holoviews.streams import Params

points = gv.operation.project_points(gv.Points((verts.x, verts.y), crs=ccrs.UTM(11)))

# Create a parameterized class to select between ERROR and Depth
results = parameters(display_result=param.ObjectSelector(default=internal_names[0], objects=internal_names))

# Define callback which returns TriMesh for specific time and ERROR/Depth
def time_mesh(time, display_result):
    depth_points = points.add_dimension(display_result, 0, dfs[time][display_result].values, vdim=True)
    return gv.TriMesh((tris, depth_points), label=label, crs=ccrs.GOOGLE_MERCATOR)

# Declare DynamicMap which varies by time and is linked to results class
meshes = hv.DynamicMap(time_mesh, kdims='Time', streams=[Params(results)]).redim.values(Time=sorted(dfs.keys()))

# Declare classes to control colormap and display range
cmap_opts = parameters(colormap=param.ObjectSelector(default=cc.rainbow, objects={'Rainbow': cc.rainbow,
                     'Fire': cc.fire,
                     'Gray': cc.gray,
                     'CoolWarm': cc.coolwarm}))
display_range = parameters(color_range=param.Range(default=(-0.3, 0.3), bounds=(-0.3, 0.3)))

class mesh_opts(param.Parameterized):
    mesh_type = param.ObjectSelector(default='Color Contour', objects=['Color Contour','Wireframe'])

# Define function which applies colormap and color_range
def apply_opts(obj, colormap, color_range):
    return obj.options(cmap=colormap).redim.range(**{obj.vdims[0].name: color_range})

# Apply the colormap and color range dynamically
dynamic = hv.util.Dynamic(rasterize(meshes), operation=apply_opts, streams=[Params(cmap_opts), Params(display_range)])

# Display everything as a Panel
hv_panel = pn.panel(dynamic)
right_column = pn.Column(hv_panel[1], cmap_opts(name=''), display_range, results, mesh_opts(name=''))
pn.Row(hv_panel[0], right_column)

Prior to this example, I was defining parameters and widgets similar to how I've specified the mesh_opts class. When this class gets visualized (with right_column) I will get a UUID type name unless I specify it as mesh_opts(name=''). However, with the new method in earthsim.parameters (as seen in cmap_opts above), the name is automatically generated without the UUID number.
The catch is that IF I specify cmap_opts(name='') like I've done above, it silently breaks the colormapping link to the widget. The map is displayed with the default color, but changing the widget value has no effect on the map. I get no errors or any indication that specifying name='' is not valid.
So in the above code, if you remove name='' it will work perfectly.

Thanks for the detailed report, I'll try to track that down asap.

As you can see in the image from the referenced issue, I now get the word 'Parameters' in bold above each widget. That's what I was removing with name=''. How would I remove it with the new method?

Once I tag a new panel release you'll be able to do: pn.panel(cmap_opts(), show_name=False).

The new approach should be preferred over setting name='' so I'll close.


Improve regex backref replace casts that need to add a postfix digit

https://github.com/rapidsai/cudf/pull/8841 allows us to specify a boundary around replacement groups so we can follow a group by a digit (e.g.: 0).  This should allow us to simplify some of the regex replaces used for timestamp casting.

@andygrove FYI, not high priority, but it may be useful if you run across this use case again in some new code.


Bootstrap 5 not working with Vite and jQuery

Prerequisites

[X] I have searched for duplicate or closed issues
[X] I have validated any HTML to avoid common problems
[X] I have read the contributing guidelines

Describe the issue
Bootstrap 5 provides instructions for getting started with Vite, and separate instructions for optionally using jQuery.
The documentation states

If Bootstrap detects jQuery in the window object it’ll add all of our components in jQuery’s plugin system;

Based on that I am setting window.jQuery in my main.js file, before including a bootstrap plugin.
// Import jQuery
import jQuery from 'jquery';

// Add jQuery to the Window (see https://getbootstrap.com/docs/5.2/getting-started/javascript/#optionally-using-jquery)
window.jQuery = jQuery;

// Import only the Bootstrap components we need
import { Popover } from 'bootstrap';

// Initialise the popover onload
jQuery(function() {
    jQuery('[data-bs-toggle="popover"]').popover();
});

However it does not appear to be correctly registering the plugin.

Reduced test cases
This is a fork of the bootstrap example repo, a copy has been made of the vite example and the following commits attempts to make it work with jQuery instead of the Vanilla JS.
https://github.com/liamkeily/bootstrap-examples/pull/1/commits/09fe7f2bbb9d11e68d43ab3c273b7fbc039e3043
What operating system(s) are you seeing the problem on?
macOS
What browser(s) are you seeing the problem on?
Chrome
What version of Bootstrap are you using?
5.3.0

This also worked fine for Bootstrap 4, which we are upgrading from. Made a separate branch on that repo to prove that Bootstrap 4 works https://github.com/twbs/examples/pull/268/commits/afb82d2d1a03ad5fa1844dca0d5676a17ce9bcad

Any PRs that fix the issue are welcome. But please try to add a test case.

Any news?

I encountered this as well while listening for a Bootstrap modal event - jQuery event wasn't triggered but listening with addEventListener was.
Could Bootstrap expose a function (use() for example, or setJquery()) which would let us set the jQuery library for Bootstrap to use, falling back to the window.jQuery if not present?
Then the usage would be:
import $ from 'jquery';
import bootstrap from 'bootstrap';

bootstrap.use($);

instead of doing window.jQuery = $;?
I can create a PR if this might be of interest?

This is actually really easy to fix, it's just not documented anywhere.
Create a "wrapper" for jquery like this:
jquery_import.js
import $ from '../../node_modules/jquery/dist/jquery.js';
window.$ = window.jQuery = $;

In your main js file, import this wrapper before importing the JS for bootstrap 5.
If you don't use a wrapper file, Vite moves the global declaration to the end of your file, and it won't work.

Thanks @skeets23 that did actually work

Let's reopen the issue and wait for someone to submit a PR. I'm pretty sure it's a common issue worth documenting.


update cmake NuttX helper save behaviour (oldconfig, menuconfig, etc)

This PR updates the behavior of the PX4 NuttX cmake helper targets oldconfig, olddefconfig, menuconfig, qconfig to now compress and save back to PX4 by default.
There are also new helper targets for saving the uncompressed version back.

oldconfig_uncompressed
olddefconfig_uncompressed
menuconfig_uncompressed
qconfig_uncompressed

@dagar
make clean
make px4_fmu-v5 oldconfig
git diff
git commit -m"1" boards/px4/fmu-v5/nuttx-config/nsh/defconfig
make px4_fmu-v5 oldconfig <- Dies

[1/3] Running NuttX make oldconfig for nsh
LN: arch/arm/src/board to /home/david_s5/src/PX4/repos/mainline/Firmware/build/px4_fmu-v5_default/NuttX/nuttx/configs/px4_fmu-v5/src
No directory at /home/david_s5/src/PX4/repos/mainline/Firmware/build/px4_fmu-v5_default/NuttX/nuttx/configs/px4_fmu-v5/src
tools/Makefile.unix:314: recipe for target 'arch/arm/src/board' failed
make[1]: *** [arch/arm/src/board] Error 1
FAILED: NuttX/CMakeFiles/nuttx_oldconfig
cd /home/david_s5/src/PX4/repos/mainline/Firmware/build/px4_fmu-v5_default/NuttX/nuttx && make --no-print-directory --silent -C /home/david_s5/src/PX4/repos/mainline/Firmware/build/px4_fmu-v5_default/NuttX/nuttx CONFIG_ARCH_BOARD=px4_fmu-v5 oldconfig
ninja: build stopped: subcommand failed.
Makefile:197: recipe for target 'px4_fmu-v5' failed
make: *** [px4_fmu-v5] Error 1

@dagar -
1:20:56 {pr-nuttx_helpers *$>} ~/src/PX4/repos/mainline/Firmware$ make px4_fmu-v5 menuconfig_uncompressed
ninja: error: unknown target 'menuconfig_uncompressed'
Makefile:197: recipe for target 'px4_fmu-v5' failed

please have help list all the target modifies
oldconfig
menuconfig
oldconfig_uncompressed
olddefconfig_uncompressed
menuconfig_uncompressed
qconfig_uncompressed

Please give it another try. There were issues with the detection of compressed or uncompressed defconfig only happening at cmake configure time. The other issue was CONFIG_ARCH_BOARD_CUSTOM=y.

@dagar -
1:20:56 {pr-nuttx_helpers *$>} ~/src/PX4/repos/mainline/Firmware$ make px4_fmu-v5 menuconfig_uncompressed
ninja: error: unknown target 'menuconfig_uncompressed'
Makefile:197: recipe for target 'px4_fmu-v5' failed

please have help list all the target modifies

This is available in the per target help (eg make px4_fmu-v4_default help).

@dagar i will get to this in the next AM.


Example for link prediction using unsup GraphSAGE

Restarting the kernel (google colab) seems to fix this issue. Code works for both PubMed and CiteSeer.


oapi-codegen: 1.13.0 -> 1.13.3

Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 246450 run on x86_64-linux 1

1 package built:

oapi-codegen

https://github.com/deepmap/oapi-codegen/releases/tag/v1.13.2

https://github.com/deepmap/oapi-codegen/releases/tag/v1.13.3

@sagikazarmark I think what @dotlambda meant is to include the content of the changelog into the commit body. (or the link to it maybe?).
Otherwise LGTM, thanks for updating it!

@sagikazarmark Seems another version was released, maybe you should update this PR

Looks like another PR has been merged in the meantime.


having issue with Chrome displaying emojis, works well in fr

I tried to display selected emoji in one of my preview part with angularjs scope binding, it works fine in Firefox but not in Chrome.

ref https://github.com/mervick/emojionearea/issues/18
ref https://github.com/mervick/emojionearea/issues/122


Accept merge should open SCM view

Testing #150389
So the user is made aware of what got added.

open merge editor (see below)
reveal another view, e.g explorer
select accept merge
ensure the SCM viewlet is revealed

To get a merge conflict and editor

use this setting "git.mergeEditor": true
run the commands below
open the conflict from the SCM view

git clone https://github.com/hediet/vscode-3wm-test-repo.git
cd vscode-3wm-test-repo
git fetch --all
git checkout input1
git merge input2

This does not work for me.
Here's a gif. Reopening so you can double check if it works for you or if I am doing something wrong

Please try with a non-error state, e.g truly resolve all conflicts and then press Accept. E.g there should be no modal dialog involved

Not working even when there is no modal dialog
Version: 1.69.0-insider
Commit: 0e3304bca9e862eddc7e437e2bab8a4a4a4f33c9
Date: 2022-06-29T05:18:24.550Z
Electron: 18.3.5
Chromium: 100.0.4896.160
Node.js: 16.13.2
V8: <IP_ADDRESS>-electron.0
OS: Darwin arm64 21.5.0
https://user-images.githubusercontent.com/10746682/176412510-9c4a9f90-99d4-490f-8d88-712843f68632.mp4

d'oh - I never created a PR for the fix here... No wonder it doesn't work. Sorry for that


How to enter an otpauth:// URL?

I may be a bit dense, but I can't figure out how to enter the data from an otpauth:// URL for my Versa 2. The other settings such as color, counter etc. sync more or less immediately, but the auth entries just vanish.
The URLs I have look like this: otpauth://totp/PrusaAccount%3Afloe%40butterbrot.org?secret=XXXXXXXXXXXXXXXXXX&issuer=PrusaAccount
So I entered "Prusa:XXXXXXXXXXXXXXXXXX" as a new token, but it simply disappears and nothing shows up on the watch.
What am I missing?

Yeah, well, as usual I solved it myself within 5 minutes of opening the issue 🙄😁 It seems to work as long as I have the watch app and the settings open at the same time.


Bug for read -> write -> write sequence in transaction

Write operations performed inside an interactive transaction should order the updates according to the session order. This works for update-only transactions. But performing a read before-hand leads to a confusion of the update-order.
Here an example of the problem in Java:
MVRegisterKey<String> regKey = Key.multiValueRegister("testReg");
InteractiveTransaction txn = antidoteClient.startTransaction();

// this read is a show killer!
List<String> initialRead = bucket.read(txn, regKey);

bucket.update(txn, regKey.assign("test1"));

bucket.update(txn, regKey.assign("test2"));

List<String> valInsideTx2 = bucket.read(txn, regKey);

txn.commitTransaction();
List<String> valOutsideTx = bucket.read(antidoteClient.noTransaction(), regKey);

assertEquals(singletonList("test2"), valInsideTx2);
assertEquals(singletonList("test2"), valOutsideTx);

When commenting out the marked line, the test runs through. But reading the value before the update makes the test fail because both "test1" and "test2" are returned as result of the multi-value register.

Quick and dirty fix:
Change clocksi_interactive_coord line 488 and don't update the ReadSet.
@deepthidevaki Can you have a look at it? Probably it is better to also update the internal_read_set after updates.

fixed in pull request #353


Add GitHub actions and test in CI

Run unit tests yarn test from GitHub actions on each push.

Done in 97559a7.


UserErrorSQLNoSysadminMembership additional details/clarification when dealing with multiple instances of sql server

I have a SQL server in azure with 4 instances of sql server installed.  The default instance and 3 additional instances.  I am not using the default instance currently, so I followed the directions in this article to give NT SERVICE\AzureWLBackupPluginSvc the permissions specified on the 3 additional instances.  When I went to discover the instances I was still getting the UserErrorSQLNoSysadminMembership error.
Turns out I needed to grant permissions on the default instance (MSSQLSERVER) as well even though I was not using it and had nothing there to back up.  Once I did this the agent was able to scan my instances and everything worked smoothly from there.
It might be worth clarifying that the backup agent account needs access to ALL instances even if you do not intend to back them up.  This might help anyone who runs into this issue in the future.
Additionally, in the future it might be a good idea to have the agent discover what instances it can and then in the portal give a list of successfully discovered instances and failed instances rather than how it currently works where it fails all instances if the agent does not have permissions on one.  Then in the portal the user can get a list of discovered instances and instances with permissions errors.  If the user intended to backup the failed instances, this gives them more specific details on what is missing so they know which instance to fix.  If they don't intend to backup the failed instance, they could then ignore the failed instance(s) and continue on to backup only the discovered ones.

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

ID: ef7b40d1-bbe9-c938-b06c-8380ec17f262
Version Independent ID: 3d3f8780-cf1e-a75a-ac24-0e9162cb4e56
Content: Back up SQL Server databases to Azure
Content Source: articles/backup/backup-azure-sql-database.md
Service: backup
GitHub Login: @rayne-wiselman
Microsoft Alias: raynew

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

@SHAGGAR Appreciate your time and patience! Based on your comments we have added a Note in Fix SQL sysadmin permission to resolve user concern - Link We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.

Looks good! 👍

I didn't find it in any documentation, gladly this post exists so I resolved the issue by giving rights to the account on an instance I didn't even know it existed on the sql server


403 Forbidden Error

Started seeing this error in the logs after a months of working fine.   v2.2.3
Just me?
Logger: custom_components.jlrincontrol
Source: custom_components/jlrincontrol/__init__.py:205
Integration: Jaguar Landrover InControl (documentation)
First occurred: April 23, 2023 at 3:45:15 PM (1 occurrences)
Last logged: April 23, 2023 at 3:45:15 PM

Error connecting to JLRInControl. Error is HTTP Error 403: Forbidden

How frequently do you see these errors?     Sorry about that!


Create realistic, real-world scale model that works for AR

We don't currently have a good demo model that looks realistic, is centered at the origin (with the base of the object at the origin). This would be great for AR demos. Ideally a .glb and a .usdz would be useful.

This has been completed in this demo: https://zircon-jeep.glitch.me

@cdata hi! I was wondering... what is the secret to knowing what scale your glb is going to be in the real world?
Is there an equivalency like 10 3D "units" = "10cm in real world" AR, or something like that?

@darkvertex great question. The glTF spec is very clear on this point:

The units for all linear distances are meters.

All units in your .gltf or .glb file are specified in real-world meters. That's how we treat them when presenting a model in WebXR-based AR. That's also how Scene Viewer treats them. And any viewer that doesn't treat them that way is wrong (according to the spec) 👍


Add site classes & AfroLA newsletter-signup event validator

Description
This PR adds some logic to determine whether a form-submission event on the AfroLA site is of a newsletter-signup form. It's encapsulated in an AfroLaNewsletterSubmissionValidator, which inherits from an abstract base class SiteNewsletterSubmissionValidator. Since this marks the introduction of partner-specific logic in our code, a Site abstract base class and its children, which represent individual partners' sites, are also created. In the style of composition, each such child class, e.g., AfroLa, stores an instance of its newsletter-signup validator class (here, AfroLaNewsletterSubmissionValidator) as an attribute.
All of these additions happen in a newly created directory ata_pipeline0/site. Changes elsewhere in the source code only accommodate them.
Testing
Three unit tests for AfroLA's newsletter-signup validation are created. All tests passed.
Notes
📌 If we're happy with this addition (i.e., this PR this merged), the next PR should add newsletter-signup validation logic to DFP, OpenVallejo and The 19th*. The one after that should add a new preprocessor that uses these site validators to identify (and possibly remove) non-newsletter-form-submission events.

Yep, can do!


create necessary files to run jitsi-meet on the grid

Description
Created files for jitsi application in the playground
Changes

Frontend files/views needed to for jitsi application
Edit on existing files to include jitsi

Related Issues

https://github.com/codescalersinternships/home/issues/253

Documentation PR

https://github.com/threefoldtech/info_grid/pull/631

Checklist

[ ] Tests included
[x] Build pass
[x] Documentation
[x] Code format and docstrings
[x] Screenshots/Video attached (needed for UI changes)

While resolving conflicts, you've removed some code. Please, check this.

@maxux Can you promote this? https://hub.grid.tf/eyadhussein.3bot/eyadhussein-jitsi-latest.flist
Should be named https://hub.grid.tf/tf-official-apps/jitsi-latest.flist

@maxux Can you promote this? https://hub.grid.tf/eyadhussein.3bot/eyadhussein-jitsi-latest.flist Should be named https://hub.grid.tf/tf-official-apps/jitsi-latest.flist

Promoted :)
https://hub.grid.tf/tf-official-apps/jitsi-latest.flist.md

@eyad-hussein
Please resolve the conflicts

@eyad-hussein is the PR ready?

I believe it lacks the right jitsi logo https://github.com/threefoldtech/tfgrid-sdk-ts/pull/3346#discussion_r1735166609. Everything else is fine

use this icon ll be better

LGTM, Good job ya Eyad


[RADJABOV] Update lockfile after backport of ManageIQ/manageiq-gems-pending#589

@kbrock Please review.

kicking


The field "Operating Systems" contains the value "Windows; Linux" that is not in the list of supported values

I followed the instructions as described in the page of the control.
If I try to set one OS  it works fine. If select more than one, I get the error as presented in the title.
Please fix.

Ok, it worked. I understood wrongly how to configure the control from the instructions in vsts marketplace... or are the instructions confusing?
So ok it is working.


Is it possible to add multiple buckets to the same ServerlessClamscan?

I am trying to add multiple buckets to the same scanner, but scanning only works in a single bucket.
Should I create two scanners in case I want to scan two buckets?
	const bucketArnList = this.getBucketArnList();
	let buckets: cdk.aws_s3.IBucket[] =[];
	bucketArnList.forEach((bucket_arn: string) => {
		const bucket_name = bucket_arn.replace('arn:aws:s3:::','');
		const source_bucket = Bucket.fromBucketArn(this, bucket_name, bucket_arn);
		buckets.push(source_bucket);
	});

    const scanner = new ServerlessClamscan(this, 'antimalware-scanner',
	  {
		  acceptResponsibilityForUsingImportedBucket:true,
		  buckets: buckets
	  }
  );

You should be able to do this with 1  instance of  ServerlessClamscan.

I destroyed the whole stack and redeployed it.. and it started working on both buckets :) Thanks a lot!


Fix AzMerit Scatter Chart

Fix grouping.  Instead of by grade, group by ela score range by 20% intervals.

Also add state average lines


custom comments template breaks after updating to 1.5

hi,
we been using the upvote plugin in the <IP_ADDRESS> version with a custom comments template as described in your faq. after updating to the newest version 1.5 the plugin will not use our custom comments template anylonger. to debug i stripped out any code from the functions.php and comments.php associated with the plugin, but still no success
any ideas?

any update to this? i just found out this plugin recently and was attempting to do the same

I can take a look at this next week probably. The plugin hasnt been actively maintained, as you can tell.

thanks, i couldn't figure it out in the end, so i just modified the source a tiny bit to not add the custom comments template filter in the first place, should be enough for personal use imo


Incremental merge or insert+update fail if configuring composite unique key

Example:
  materialized='incremental',
  unique_key=['pull_request_number', 'repo_id']

Using dbt-vertica version 1.5.0.

It generates query such as:
delete from "XXX"."github_input_stage"."pull_requests_extract_json"
where (['pull_request_number', 'repo_id']) in (
  select (['pull_request_number', 'repo_id'])
  from "pull_requests_extract_json__dbt_tmp"
  ....

Hello.
Use it via concat or hash unique_key='hash(load_dt,table_schema,table_name)',. Adapter doesn't support list for this option, only string value.
From documentation: The unique_key config parameter is required for using the merge strategy, the value accepted by this parameter is a single table column.

I see.
On the other hand, the list is supported by Snowflake ;-)

Working on to support unique_key parameter as a list. So it will always accept list not string value even for single value.
Examples are below:
for single column, configuration should be : unique_key=['columname']
for multiple columns, configuration should be: unique_key=['col1','col2','col3']


[KYUUBI #6537] Upgrade Flink from 1.19.0 to 1.19.1

:mag: Description
Issue References 🔗

This pull request fixes #6537
Describe Your Solution 🔧
Upgrade Flink from 1.19.0 to 1.19.1.
Types of changes :bookmark:

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Test Plan 🧪
Behavior Without This Pull Request :coffin:
Behavior With This Pull Request :tada:
Related Unit Tests

Checklist 📝

[x] This patch was not authored or co-authored using Generative Tooling

Be nice. Be informative.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 0.00%. Comparing base (1e3afdb) to head (b39d308).

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #6538   +/-   ##
======================================
  Coverage    0.00%   0.00%
======================================
  Files         677     677
  Lines       41789   41789
  Branches     5707    5707
======================================
  Misses      41789   41789

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Thanks, merged to master


Mark converted_to_discussion issue event type as available in timeline API

Why:
Closes: #35045
What's being changed (if available, include any code snippets, screenshots, or gifs):
The converted_to_discussion issue event type is now correctly reported as available in the timeline REST API.
Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

@Fs00 Thank for opening a PR with this fix! I'll get this merged 🍏


Snippets no longer filetype specific

Hey,
Since https://github.com/garymjr/nvim-snippets/commit/8f85fc2a27d54434a22e9d9bc71655ce632fbef0 snippets are no longer filetype specific. All snippets loaded, for multiple different filetypes, are available for all buffers.
I'm assuming that this is an unwanted behaviour?
Cheers 🍻

This is fixed in #39. Thanks for pointing this out!


Add New Jersey Gov. Chris Christie

Declared candidacy, June 30, 2015.

Closed in commit #0b077bb04287b3de92c9fd067b25f25513c7e07d


CONSTRAINT not supported in CREATE TABLE (solves issue #137).

Coverage decreased (-0.3%) to 99.134% when pulling 6c55566a09025913a834beeb108ac84fb8538ab7 on wwe_issue_137 into c6258ee954b27690d05ea908db31b1549bf1b73e on master.


Store email info in a separate config file

We need to store the info for sending the email in a separate config file which can be modified as per our convenience.

#545


simplenote: init at 1.0.6

Motivation for this change
Simplenote Package
Things done

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

[x ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

Please also clean up your history.

@dyrnade why close?

My mistake, i will redo pr. Sorry


Question in coordinate transform in data converter

Hi there,
I tried the pertained model and it works well. I am currently reading the code and got confused by the following code:
https://github.com/tjiiv-cprg/EPro-PnP/blob/2bd75c410b7c75fe7fb51b9efaa0746f85b6b095/EPro-PnP-Det/tools/data_converter/nuscenes_converter.py#L472
where it says
    # get image projection
    cam_points = (lidar_points - cam_info['sensor2lidar_translation']
                  ) @ cam_info['sensor2lidar_rotation']

Why do you use cam_info['sensor2lidar_rotation'] here?  It seems that cam_info['sensor2lidar_rotation'] means rotation from sensor to lidar and so if you would like to transform from lidar, you may need to use inverse like np.linalg.inv(cam_info['sensor2lidar_rotation']). why not use the inverse? Thank you.

Since the coordinates in lidar_points are arranged as row vectors instead of the usual column vectors used in equations, we can accordingly put the transposed rotation matrix to the right of the multiplication. Therefore the rotation matrix needs to be inverse and transposed, returning to its original form.

Since the coordinates in lidar_points are arranged as row vectors instead of the usual column vectors used in equations, we can accordingly put the transposed rotation matrix to the right of the multiplication. Therefore the rotation matrix needs to be inverse and transposed, returning to its original form.

Thank you for your response. My question is the code I quoted above does NOT use inverse rotation matrix...
 @ cam_info['sensor2lidar_rotation']

Another question: In nuscenes_converter.py we need to extract the lidar points within each 3d bounding box to construct the 2D-3D-2DW correspondence. So what are the definition of the axes in the detected object (i.e. for each detected object, which direction means its x, y, z axis)? or you use axes definition same to each camera? Thank you in advance.

The inverse of an SO3 matrix equals its transpose, thus the transpose of inverse is itself.
This converter uses the object coordinate system defined in the nuScenes dataset. But the dataloader will convert it to the KITTI format directions (x forward, y downward, z leftward):
https://github.com/tjiiv-cprg/EPro-PnP/blob/2bd75c410b7c75fe7fb51b9efaa0746f85b6b095/EPro-PnP-Det/epropnp_det/datasets/nuscenes3d_dataset.py#L186

The inverse of an SO3 matrix equals its transpose, thus the transpose of inverse is itself. This converter uses the object coordinate system defined in the nuScenes dataset. But the dataloader will convert it to the KITTI format directions (x forward, y downward, z leftward):
https://github.com/tjiiv-cprg/EPro-PnP/blob/2bd75c410b7c75fe7fb51b9efaa0746f85b6b095/EPro-PnP-Det/epropnp_det/datasets/nuscenes3d_dataset.py#L186

Got. Thank you!


Add a port file to vcpkg?

FYI, you can create a port file in vcpkg so that we can add a feature to SEAL:
./vcpkg install seal       # installs SEAL without HEXL
./vcpkg install seal[hexl] # installs SEAL with HEXL

Thanks for the tip! This would be a nice feature; we'll take a look.

Let me know if I can help.

I took the liberty of adding "hexl" to vcpkg, See pull request.
We want to add "building APSI with HEXL enabled" to Microsoft/APSI's readme. Since we recommend building dependencies of APSI via vcpkg, we need HEXL to be there. :)
There are requirements in vcpkg that makes this PR really difficult. I'm working on it. Basically, HEXL's downloading cpu-features is not allowed in vcpkg, since the better way is to install cpu-features from vcpkg and to call find_package(CpuFeatures ...) in HEXL. But cpu-features installed by vcpkg has an issue too, which make find_package(CpuFeatures ...) fail all the time. I might have to add patches to both packages.
Anyway, nothing requires any change in HEXL so far. This is likely to be approved next week.

I took the liberty of adding "hexl" to vcpkg, See pull request.
We want to add "building APSI with HEXL enabled" to Microsoft/APSI's readme. Since we recommend building dependencies of APSI via vcpkg, we need HEXL to be there. :)
There are requirements in vcpkg that makes this PR really difficult. I'm working on it. Basically, HEXL's downloading cpu-features is not allowed in vcpkg, since the better way is to install cpu-features from vcpkg and to call find_package(CpuFeatures ...) in HEXL. But cpu-features installed by vcpkg has an issue too, which make find_package(CpuFeatures ...) fail all the time. I might have to add patches to both packages.
Anyway, nothing requires any change in HEXL so far. This is likely to be approved next week.

Thanks, @WeiDaiWD! Let us know if you need any support from the hexl side.

FYI, two minor things that I've noticed:

HEXLConfig.cmakeis not ignored.
HEXLConfig.cmake, HEXLConfigVersion.cmake, HEXLTargets.cmake, and HEXLTargets-release.cmake are installed to /usr/local/lib/cmake rather than /usr/local/lib/cmake/HEXL (or better /usr/local/lib/cmake/HEXL-1.1).

These are unrelated to this issue. :)

Finally we have hexl added to vcpkg, this PR is merged. So now you can install hexl with ./vcpkg install hexl. On Windows, you have to use ./vcpkg install hexl:x64-windows-static or ./vcpkg install hexl:x64-windows-static-md. The reason why dynamic linking won't work on Windows is that HEXL does not export symbols.
I'm gonna add an option to install SEAL with HEXL enabled to vcpkg, e.g., ./vcpkg install seal[hexl] soon.

Thanks, @WeiDaiWD! We'll see if we can fix the Windows dynamic build.

No pressure. I was just describing what is supported when installing via vcpkg. SEAL does not support windows dynamic either. :)


Fixed diagnostics for already existing symbol

Fixes #1654
SymbolTable during build has a pointer to currently processed source and string view to symbol.
When symbol is redefined in another file, the source pointer is no longer valid and resolving the location of the previous symbol in the current file was resulting in assertion error.
This commit uses the file_origin from SymbolTableNode for previous symbol to provide symbol location.
The current behaviour is that we keep the first observed definition of the symbol, while returning error in diagnostics for other redefinitions.

Filed #1663 for the CI hickup investigation.

The CI issue is resolved, so if you rebase, that should work again.
In addition to the e2e verilog-language-server_test, can you also add a test in symbol-table_test.cc that fails before and is fixed after ? That way isolated work on the symbol-table.cc will be possible just with running the corresponding test.


Bump alpaca-eval dependency.

Bump alpaca-eval version. Fixes a bug where alpaca-eval sometimes hung during evaluation.
I upgrade openai as well because the current alpaca-eval requires openai >= 1.5.0. From what I could tell, the only dependence on openai was TruthfulQA, which is broken at present anyway.

Did you test that the version bump doesn't significantly change scores?

Good point, I'll do a check this evening.

Also, I think it would be good to edit https://github.com/allenai/open-instruct/blob/main/scripts/eval/alpaca_farm.sh to include the alpacaeval 2 environment variable!

OK, I ran a few tests and the results are a bit different.
Setup: I ran alpaca-eval on tulu-2-dpo-7b (beaker 01HE8H1MBSVN09ZZ82X6K90NTF) and tulu-2-dpo-70b (beaker 01HEXKXP0MFM60PT7SY71XXSWD), using beaker images with alpaca-eval==0.3.1 (old) and alpaca-eval==0.5.3 (new).
Results below. Seems like there's a fairly large discrepancy for the 7b model; substantially larger than the standard errors. Should we check with the alpaca folks about what's going on?

setting
win_rate
standard_error
n_wins
n_wins_base
n_draws
n_total
discrete_win_rate
mode
avg_length

tulu 70b / new
94.53
0.80
760
43
2
805
94.53
community
1433

tulu 70b / old
92.05
0.95
740
63
2
805

community
1433

tulu 7b / new
84.97
1.26
684
121
0
805
84.97
community
1446

tulu 7b / old
76.27
1.50
614
191
0
805

community
1446

This seems fine to me, actually for the Tulu 2 paper we used the newer GPT-4 judge version, and the older gpt-4 is deprecated. So it's good to update, and the "new" results dave shared match those in the Tulu 2 paper better (~85 for 7b, ~95 for 70b).

Looks good to me! Just to clarify, I think the results we ran for tulu 2 are still valid? The old results that Dave showed are actually lower than we have in our result sheet. The newer version actually matches better?

To clarify: the new version matches better because I manually made sure our Tulu 2 results used the newer GPT-4 version (0613), which the pinned older version of AlpacaEval doesn't use it (it explicitly uses GPT-4-0314).

Glad this worked out!

One other thing we talked about was making sure we pin the version of GPT that's being used as an eval. Are we confident this is fixed or should I look into this?


op-exec error propagation, tests

Codecov Report

Merging #16 into master will increase coverage by 2.63%.
The diff coverage is 32.69%.

@@            Coverage Diff            @@
##           master     #16      +/-   ##
=========================================
+ Coverage    5.94%   8.57%   +2.63%
=========================================
  Files          63      63
  Lines        6157    6179      +22
  Branches     1126    1127       +1
=========================================
+ Hits          366     530     +164
+ Misses       5673    5485     -188
- Partials      118     164      +46

Impacted Files
Coverage Δ

op-model/src/defs/mod.rs
3.9% <ø> (ø)
:arrow_up:

op-cli/src/main.rs
0% <0%> (ø)
:arrow_up:

op-exec/src/core/op/context.rs
0% <0%> (ø)
:arrow_up:

op-cli/src/options.rs
0% <0%> (ø)
:arrow_up:

op-exec/src/core/op/impls/model.rs
0% <0%> (ø)
:arrow_up:

op-model/src/model.rs
0% <0%> (ø)
:arrow_up:

op-exec/src/core/op/impls/mod.rs
0% <0%> (ø)
:arrow_up:

op-exec/src/core/model.rs
20.74% <25%> (+20.74%)
:arrow_up:

op-exec/src/config.rs
61.78% <55.17%> (+49.84%)
:arrow_up:

... and 10 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 b1fb45f...2fca545. Read the comment docs.


Retry interval much longer than what is set

I have a dag with retry_interval set in the default_args to timedelta(seconds=10). However, when a task failed, it did not retry until 5 minutes and 6 seconds later (according to the logs). Another retried after 5 minutes and 3 seconds.
Is there a workaround for this? Has anyone encountered this before?

At which interval is your scheduler running?
Look for SCHEDULER_HEARTBEAT_SEC in your airflow.cfg
Also, what's your JOB_HEARTBEAT_SEC set to?

Both are set to 5 seconds.

The docs/tutorial wasn't using the right param name:
https://github.com/airbnb/airflow/pull/189
Thank you for reporting this!

Thanks a bunch!


user service cache and hystrix fallbacks for endpoints

maybe caching on user information by id/login would be a good idea.
services using the user Feign client should implement hystrix fallbacks.

I think this is covered pretty well by Optionals.


Use Security-Scoped Bookmarks

As the app is sandboxed, after a restart of the app the access to the selected folder by the user is no longer valid. This means the project opened last time will be forgotten after restart, but it can be fixed by using Security-Scoped Bookmarks.

Fixed, but please review the code as it relies too much on userDefaults in many places


Add Attachment

I'm unable to add attachment to existing issue using jira-connector jira.issue.addAttachment() method , where to provide the issue key and file ? can any provide me the sample code?

Here is an example of what you code should look like:
jira.issue.addAttachment({       issueKey: "PRO-1111",       filename: '/path/to/your/file'  }, function(error, response) {       console.log(response); })


Define use case scenario for docker_deploy and provide document/configuration for them

docker deploy is firstly designed for PoC use-case, that is, launching jenkins-master/ELK stack in container of laptop PC and put jenkins slave to baremetal. At that time all jenkins job will be run at jenkins slave because jenkins_master of laptop PC is not powerful.
Bu on the other side, we would like to launch container with powerful baremetal. In this case, we'd like to run jenkins jobs at jenkins_mater. So there are several use case scenarios in docker deploy.
We need to clarify various use cases for docker deploy and provide configuration for them with document.

One scenario is run PoC:

run container in laptop or low spec machine
need to run various task in baremetal (i.e. jenkins slave)

it's convenient to run image downloader jobs on jenkins master and distribute images to slaves; it's particularly useful in below scenario:  1) you have multiple slaves and each need to download the same images from internet, 2) downloading large images like undercloud and overcloud image is time-consuming.

@zshi-redhat thanks, I will add above description. Let's listed up in this week (Mar 3, 2017), then implement (ansible/documnet).

in order to install docker behind proxy, need to add http_proxy & https_proxy support in docker-compose.yml or dockerfiles

@zshi-redhat  , thank you for your comment!

build and tag stable images and push to docker hub for easy consumption.


double quote when populating consul KV store

We are using this consul library for pulling our ansible/bitbucket-backed configuration file to the consul KV store. One issue that we have been experiencing is, certain type(e.g. String, Obj) of values that populated into the KV store will have the double quotes. For example.
Here is my configuration file:
test:
  testHost: localhost:8080

After populating consul KV store:

Which will cause issue when we are trying to bind the consul KV pair to our configuration instance on the service level since the string will includes unexpected double quote.
Any ideas or suggestion?

We're experiencing the same issue. Looking through the python-consul source, I don't see where any quoting is being done explicitly, so it looks like this might be coming from the http library?

There's a check here to ensure the value being supplied to put is a string or binary type:
https://github.com/cablehead/python-consul/blob/master/consul/base.py#L513
It sounds like coming from ansible you have rich types of some sort? String with a capital S. Is there something that coerces these into raw strings? If so, could that coercion be adding the quotes?
My question may not even make sense, I'm not familiar with the ansible library you're using.


embree: Allow building against system library on Linux

Tested on my distro Mageia, I could successfully link against system-packaged lib64embree3-devel-3.12.1-1.mga8 which seems to be the same version we vendor (and our patches are only for Windows/MinGW support).

Actually doesn't work with a build from scratch as the common/math stuff from Embree is not part of the public headers it seems, yet we rely on them:
modules/raycast/lightmap_raycaster.cpp:34:10: fatal error: math/vec2.h: No such file or directory
   34 | #include <math/vec2.h>
      |          ^~~~~~~~~~~~~

I think I can get rid of those imports, I will take a look.

Rebased after #48089 and removed thirdparty/embree/common from the include path. Seems to work fine locally with both vendored and system Embree.

Cherry-picked for 3.3.1.


PythonAPI - List of undocumented (or outdated ) documentation

Certain functions/classes are missing from https://carla.readthedocs.io/en/latest/python_api
I am currently aware of these:

[ ] Client.load_world, Client.reload_world Missing Return Type: World
[ ] CityObjectLabel; see #6216
[ ] command.FutureActor Undocumented (referenced by some examples)
[ ] Client.load_world_if_different Undocumented; see #6651

Thank you for raising this issue with the docs, these have now been fixed.


Memory allocation of<PHONE_NUMBER>45856 bytes failed

I have followed the basic setup. This is my Cargo.toml file:
[package]
name = "love-story"
version = "0.0.0"
edition = "2018"

# See more keys and their definitions at https://doc.rust-lang.org/cargo/reference/manifest.html

[dependencies]
proconio = { version = "0.4.3", features = ["derive"] }

I'm on Ubuntu.
The full error message:
memory allocation of<PHONE_NUMBER>45856 bytes failed
error: could not bundle the code

- `proconio 0.4.3 (registry+https://github.com/rust-lang/crates.io-index)` as `crate::__cargo_equip::crates::proconio`
- `proconio-derive 0.2.1 (registry+https://github.com/rust-lang/crates.io-index)` as `crate::__cargo_equip::crates::__proconio_derive_0_2_1`

Caused by:
  rust-analyzer error

Caused by:
  server exited

[dependencies]
proconio = { version = "0.4.3" }

It seems that memory allocation errors are less likely to occur when derive features are not used, but I have no idea of the exact cause at this time.

Same issue on Windows. Removing "features = ["derive"]" could solve this problem, but fastout will be unavailable.

I updated the embedded Rust Analyzer to 2023-07-31, and released v0.20.0. Please reopen this issue if the problem remains.

I updated the embedded Rust Analyzer to 2023-07-31, and released v0.20.0. Please reopen this issue if the problem remains.

For me, the problem is gone. :)


Can't read/write from CanvasPixelArray

It seems the underlying type of ImageData is a Uint8ClampedArray. Giving it a proper type instead of * would make it possible to read and write image data. As it is right now, getImageDataPixelArray is just about useless because it produces a value that no other function knows how to consume (without using the FFI).
These libraries could be useful

https://github.com/purescript-contrib/purescript-arraybuffer-types
https://github.com/purescript-contrib/purescript-arraybuffer

Would be willing to PR if what I'm suggesting makes any sense.

I agree it's pretty useless (without the FFI) right now. I think we have to be a bit careful about what we allow, since I'm not sure if it's always possible to treat the image data like an array, but I might be wrong. Maybe let's come up with something though, and we can iterate on it.

I'm also interested in getting access to the pixel data. @paf31 would you be interested in accepting an updated version of #19? Uint8ClampedArray is definitely the right type (see https://developer.mozilla.org/en-US/docs/Web/API/Canvas_API/Tutorial/Pixel_manipulation_with_canvas).

Yes, sounds good, thanks. The data property is certainly an Uint8ClampedArray, and that type exists in purescript-arraybuffer-types.

Since ImageData is now defined as
type ImageData = { width :: Int, height :: Int, data :: Uint8ClampedArray }

this should no longer be an issue (although there is also #36 to sort out...)


3.0.4: fix Link Object example

Same fix has already been applied to 3.1.1-dev

@MikeRalphson @handrews @darrelmiller Please review and merge


Redesign the Writer

The Reader and Writer have been designed with various JSON libraries in mind. Whereas the design for Reader seems fine, the Writer needs an update.
The problem is that many libraries for binary formats require you to directly assign children to their parents as opposed to first creating the children and then assigning them to their parents.
For instance, pugixml requires you to do this:
auto node_child = node_parent.append_child(_key.c_str());

Likewise, libbson (https://mongoc.org/libbson/current/creating.html):
bson_append_array_builder_begin (&parent, "foo", 3, &bab);
bson_array_builder_append_int32 (bab, 9);
bson_array_builder_append_int32 (bab, 8);
bson_array_builder_append_int32 (bab, 7);
bson_append_array_builder_end (&parent, bab);

The current workaround it to have structures like OutputArray and OutputObject (https://github.com/getml/reflect-cpp/blob/f/xml/include/rfl/flexbuf/OutputArray.hpp). This works well, but it adds an unnecessary extra layer and runtime overhead.
This could be avoided by a redesign of the writer that takes these limitations into account.

I personally don't think the Writer and Reader interfaces (with associated Parser) don't provide the clearest and flexible approach - especially in comparison to Rust Serdes design.  Even though Rust native traits don't have a natural match in cpp we could strive a similar approach.

create a well defined data model -> https://serde.rs/data-model.html
design Serialize and Deserialize "Traits" that map from and into the data model. -> https://serde.rs/impl-serialize.html

@jimixxperez, would that mean that the users of our library would have to inherit some base class from our library?


Typing information for pyosmium

This provides *,pyi files with type annotations. As pyosmium is still supporting Python 2.7 I decided to implement type annotations as separate files.
I found this quite useful and informative when using with PyCharm.
For now, I decided to ship *.pyi files together with *.py files. Python community is working on target solution

I'm not entirely happy with all the code duplication. Keeping the py and pyi files in sync will be a pain. So I'm not sure how to go forward with this. I'm not against adding typing information but it needs to be maintainable.

I'm not happy too. Though:

it's easiest way to maintain compatibility with wide range of python versions
it makes easy to annotate pure C(++) functions that doesn't have python representation

We can also try with this approach:
https://mypy.readthedocs.io/en/latest/python2.html#type-checking-python-2-code
But then we need to add typing module backport as runtime requirement.
Then we would need to have stub methods for pure C++ functions.

I've managed to inline all annotations to *py files, where they do exists:
https://github.com/osmcode/pyosmium/compare/master...wiktorn:typing_annotations_inline?expand=1
For all pure C++ modules I kept the *pyi files.
Does it improve situation to manageable level?

To follow up on this. I was able to use mypy to verify the completeness of the annotations but only for some definitions of completeness, i.e. only if all arguments and fields in *.pyi files are annotated. There was no signature check and missing or superfluous annotations.
I also tried pytype but it fails loading C modules. I've also tried mypy's stubgen but this generates only empty stubs. I'll try to debug either of it to get it working and automate generation of *.pyi information first.

I've just found python/mypy#5814. Looks promising and was merged just 2 days ago. I gave it a try on current (boost_python) version and results are better than before, but still far from perfect (and even acceptable), as all C++ function signatures are guessed as (*args, **kwargs). I hope that it will be better with pybind as this was primary goal of mentioned issue.

Thanks for looking into this. If you want to play with the pybind version, the current state is on branch https://github.com/osmcode/pyosmium/tree/pybind-rewrite. It only works with python3 at the moment and some parts are still missing but it should be enough to play with mypy. You need to put a copy of pybind11 in contrib/pybind11 yourself. I still need to figure out how to best handle this external dependency (submodule, copy, external install, ...).

Just to let you know. My current plan to move this forward is:

extend mypy stubgen so it will generate flawless pyi files for C/C++ classes generated by pybind11 (python/mypy#5850 and one more planned with support for overloaded functions)
add inline declarations in Python classes in comments, so this will be Python27 compatibile
make invocation of stubgen task of packaging pyosmium

So this will achieve the goal of not having to think about pyi files when implementing changes.
Another approach to consider is to extend pybind11 with the ability to generate pyi files for classes it generates, but this way above my current C++ skills

Sounds all good to me. And I see that your mypy change got merged already. I couldn't help with pybind11 either, as my C++ skills do not include that level of template magic.

Using master branch of mypy/stubgenc it is possible now to get almost perfect stubs.
The only outstanding broken annotation are in lib/osmium.cc:
def apply(reader: osmium.io.Reader, handler: osmium::handler::NodeLocationsForWays<osmium::index::map::Map<unsignedlong, osmium: :Location>, osmium: :index::map::Dummy<unsignedlong, osmium: :Location>>) -> None: ...
def apply(reader: osmium.io.Reader, node_handler: osmium::handler::NodeLocationsForWays<osmium::index::map::Map<unsignedlong, osmium: :Location>, osmium: :index::map::Dummy<unsignedlong, osmium: :Location>>, handler: BaseHandler) -> None: ...

But I didn't find a way to provide proper types here (like it was possible elsewhere by adding py::import statements).
The other outstanding problem is that pybind generates docstring for stl iterators that doesn't give any type information, but that something I hope can by addressed on pybind side.
I've added step in setup.py / build_ext that generates stubs for C++ modules. For Python modules - typing information is included as comments to maintain compatibility with Python2.
I've added build-requirements.txt and test-requirements.txt so it is easy to setup build/test environment with pip install -r build-requirements.txt etc.

Closing in favour of #211.


Unwinding

Code for stack unwinding is using sspinc to  update ssp, does it possible to use csrw for updating ssp directly?

Thanks for explaining :)


Main Menu Got Crashed

Describe your bug here.
So, when I press title screen the game Got crashed for no reason
Command Prompt/Terminal logs (if existing)
there is no zero Error

Are you modding a build from source or with Lua?
Source
What is your build target?
Windows
Did you edit anything in this build? If so, mention or summarize your changes.
It Nothing but fixed Main Menu State

Do a git reset --hard and recompile again?


Fixed issue #3049:that is address bar clears when rotating to landscape mode

This is my first pull request , I have fixed the bug related to issue #3049 , that is address bar clears when rotating to landscape mode , There were small changes made in MessageCompose.java and AndroidManifest.xml( you can find the images of the code added)

below are the images of the error found on the app

Thanks for taking the time trying to fix a bug in K-9 Mail. However, this pull requests adds two new files and doesn't modify the existing ones. But from the attached images I can see that you tried to fix the issue by ignoring configuration changes. That's not the proper way to handle this (see comments on PR #3108 which used the same approach).
There's already PR #3116 dealing with this issue.


[Mobile] Unify the modal windows paddings

Description
Side paddings are 12 px, top and bottom are 16 px.
Need to correct:

Attachments

Confirm clearing

Share

Some items failed to import due to duplicate names ( also add external paddings)

See also #1194

Replace Confirmation Modal will be fixed as part of https://github.com/epam/ai-dial-chat/issues/1230

Use case/motivation
No response
Related issues
No response
Confidential information

[X] I confirm that do not share any confidential information

Verified on staging env. Reported #1445 , #1468 , #1473


Converted groovy scripts to full-on groovy classes

By making this into a proper project, we no longer need to rely on groovy to be installed on the server on which this runs.  We now only rely on the jdk.

The jira and email portions of the refactor are untested still, so don't merge yet.


:let added in v0.18 of LiveView

Get Started with Ash and Phoenix --> Connecting your Resource to a Phoenix LiveView.
<.form let={f} for={@create_form} phx-submit="create_post">
and
<.form let={f} for={@update_form} phx-submit="update_post">
:let was introduced in v0.18, so I needed to add the colon to get this example working.

Fixed in latest docs, will deploy :)


Resend verification email can not  verification

Click activate the email to jump to /type/error/dashboard, account information is still you must first verify your email address

I'm not fully understanding the request, are you saying the email you click on is taking you to the dashboard? Can you paste in the raw url that is sent to you in the email minus the domain name? I'm wondering if the url is incorrect?

I'm going to close this until more information is provided. Can you please see if my questions above helped resolve this issue.


Error in issue.load_history_into_changes

I've just update to v2.0.0. I received the following error:
NoMethodError: undefined method each' for nil:NilClass
←[0;31m          load_history_into_changes at [...]/jirametrics-2.0/lib/jirametrics/issue.rb:489`
It seems that history['items'] is nil. Could it be possible? Is there a way to find out what issue id is causing the error?
If I use another board, it works perfectly.

It's another "theoretically impossible but we've seen it in production" moment, I think. I'll add a check for that.
Unfortunately, there is no easy way to see which issue caused the problem. That's a good suggestion though - let me see what I can do.

Thanks for your work, Mike.
No exceptions are thrown now, but, unfortunately, with v2.0.1 no issues are plotted in the charts except for one that is in progress. Issues are downloaded, but not plotted.
The strange thing is that with version 1.2.1 everything was running smoothly. What more info can I give to help solve this?

If you take a look at the first JSON sample in this article, you'll see a well-formed Issue.
https://agiletechnicalexcellence.com/2024/04/09/jira-issue-history.html
The fix that I put into 2.0.1 was because you didn't have an items section under one of the histories, which is theoretically impossible. Can you open one of the downloaded issues on your machine and see what really is in that history block?
The issues will all be under your target directory.

Or you had an items element but it was empty. Would have been the same error either way.

False alarm. The fix v2.0.1 works!
Ignore my last message. In the meanwhile, someone had changed the filter of the Jira's board and so I had unexpected results.
All OK now.
Thanks again, Mike!

Fixed

However, is there a way to skip the download process?
When I have to do some tuning for html_export or csv files I would like to skip the download of the issues. The download process could be time consuming sometimes.

Yes, instead of jirametrics go, just use jirametrics export. Or download if that's the only part you want


fix(3156128399): align faq to Figma

Description of the Changes

Changed CSS properties to be like Figma
Changed icon to be 20px

Solves #3156128399

Checklist

[x] Manually tests of the main Application flows are done and passed.
[ ] New unit / functional tests have been added (whenever applicable).
[ ] Docs have been updated (whenever relevant).
[x] PR title is follow the Conventional Commits convention: <type>[optional scope]: <description>, e.g: fix: prevent racing of requests

This change is

:tada: This PR is included in version 1.9.0-dev.14 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

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

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