[OAG] Error: undefined method '[]' for Financialmodelingprep::IncomeStatementListResponse

When calling get_income_statements_with_http_info, the following error occurs.
% crystal main.cr
Showing last frame. Use --error-trace for full trace.

In main.cr:9:26

 9 | infos, status, headers = api.get_income_statements(symbol: "AAPL", apikey: apikey, datatype: "json", limit: 1, period: "annual")
                              ^-
Error: undefined method '[]' for Financialmodelingprep::IncomeStatementListResponse

This is due to the following line:
return IncomeStatementListResponse.from_json(data), status_code, headers
It can be resolved by changing this to:
return Array(IncomeStatement).from_json(data), status_code, headers
This is an issue in the upstream OpenAPI Spec.

Fixed in: d774bc9301d929b0a4f8bb39fe2bbbab4323c0d2


[WIP] feat(users): Initial server-side Token Authorization

Initial implementation of Token based authentication on the server-side.
Adds tokenAuth settings to default env config
Adds jsonwebtoken library to project
Adds authorization configuration to config/lib
Authorization middleware
Adds the authorization.authorize to the Express middleware
Verify User data Token
Updated Token payload to just store the user._id rather than the whole
User object.
Adds an extra verification for the Token stored with the User data.
User model Auth field
Adds the auth field to the User model
auth.token
auth.expires
Login & Signup Token creation
Modified signup & signin server-side API methods to generate a signed
JWT, and store it with the User data.

This is still a work in progress, but I wanted to get it out there. Figured it would be easier to track & get feedback this way.
With these changes, the application provides Token Auth capabilities to the authorization & authentication processes. If the the incoming request doesn't provide the Authorization header, then the application will fallback to Session based authorization.
I'm not sure if we'd even want Session based auth & Token-based Auth to co-exist, but is an interesting concept.
Any thoughts, critics, or concerns?

@mleanos @ilanbiala @codydaig
Any thoughts on using this library for the frontend implementation of JWT?
https://github.com/auth0/angular-jwt
We would have to account for:
https://github.com/auth0/angular-jwt#not-sending-the-jwt-for-template-requests
On the server side since we are using passport would it make sense to use the JWT module?
https://www.npmjs.com/package/passport-jwt
Then you only secure the routes that you want instead of securing every route with app.use in the express middleware?
I was thinking of having a user/auth route that gets the JWT token and store it locally, possibly with the module above, then using the passport module on the routes?
Also I am not sure why we are storing the JWT Token with the user model.  Isn't the point of JWT to require the token or re request one?

https://github.com/meanjs/mean/pull/1163

Closing this in favor of #1300


Line number of failed assert would be very helpful

Component
Forge
Describe the feature you would like
It would be very helpful if the line number of the failed assert were printed in the console.
Additional context
No response

Agreed! Marking as duplicate of https://github.com/foundry-rs/foundry/issues/7930, added your request for printing this on failed asserts to the ticket description


[📊 DATA] Nadabot: Most Used Applications for Verfied Humans 🟠

Description
Objective / Business Insight

better understand applications that humans are using
promote comarketed quests

Acceptance Criterion

[ ] table of each human with applications used and frequency
[ ] most popular application across usage for verified humans
[ ] number of verified humans. Percetnage of each major application
[ ] which ap[plication has dau, wau with verified huamns

Data Sources

v1.nadabot.near isHUman()
need to query an account for transaction history

Similar Queries
Challenges

querying all applcaitons, can check access keys. May need flipside crypto

Resources
How to Present
Future

whether veriying as human resulted in icnrease activity

https://flipsidecrypto.xyz/susanahd56/2024-03-14-11-40-pm-hQ5P1t

https://flipsidecrypto.xyz/edit/dashboards/b89875f4-906e-474b-8e10-189299ebba4c
Validated only is showing verified users with successful transactions.


Scripts to bootstrap devstack setup for bigchaindb with tendermint

Description
Adds a single command workflow to bring up development setup with bigchaindb and tendermint
Issues This PR Fixes
Fixes #1850
Todos

[ ] Tested and working on development environment

How to QA
System requirements: vagrant and virtualbox (install latest versions and you'll be fine)
Set git branch to use: export GIT_BRANCH=shahbaz-dev
Set stack to deploy: export STACK=devstack
down load script: curl -fOL https://raw.githubusercontent.com/bigchaindb/bigchaindb/${GIT_BRANCH}/pkg/scripts/install_stack.sh
run script: bash install_stack.sh

Codecov Report

Merging #1887 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1887      +/-   ##
==========================================
+ Coverage   98.45%   98.46%   +0.01%
==========================================
  Files          60       60
  Lines        3037     3319     +282
==========================================
+ Hits         2990     3268     +278
- Misses         47       51       +4

@muawiakh can you please review again

I needed to install the vagrant-vbguest and vagrant-cachier plugins additionally. So maybe we could add them to a required or optional list.
Also, localhost was not working for me.

Some suggestions for documentation around this script:

A description of what the script will do
How to operate the environment after installed (vagrant up, vagrant halt)
How to export environment variables (in case you forgot like me :))
Install vagrant plugins as @codegeschrei mentioned
If you mess up, delete .vagrant and try again

You can ignore the license/cla thing. More details than you ever wanted to know are available at https://github.com/bigchaindb/org/issues/34

@shahbazn TravisCI is failing on the same test where it failed for me earlier today in PR #1947 --- I fixed it with PR #1948 (not merged yet but you can probably git cherry-pick the commit into this PR).

On my Lenovo X-1 Carbon laptop, I got the error message:
There was an error while executing `VBoxManage`, a CLI used by Vagrant
for controlling VirtualBox. The command and stderr is shown below.

Command: ["startvm", "6b9c4902-7b1c-44bb-b891-d4ed4e5699ae", "--type", "headless"]

Stderr: VBoxManage: error: VT-x is disabled in the BIOS for all CPU modes (VERR_VMX_MSR_ALL_VMX_DISABLED)
VBoxManage: error: Details: code NS_ERROR_FAILURE (0x80004005), component ConsoleWrap, interface IConsole

I'm guessing that I have to turn on some virtualization settings in my laptop's BIOS. I will try some of the instructions given at:  https://stackoverflow.com/questions/34746462/error-vt-x-is-disabled-in-the-bios-for-all-cpu-modes-verr-vmx-msr-all-vmx-disa
More generally, the install_stack.sh script needs to be documented somewhere (i.e. in the docs), as Morgan noted. Who is it for? What is it for? What does it do? And troubleshooting tips like the one described above in this comment.


switch all worker to private dns

This PR makes the following changes:
switch all worker to private dns
I tested this PR by...
launching cluster where i am seeing slow startup(flintrock needed bend-aid to start cluster).
After this change cluster start in first try.
Fixes #157.
Fixes #129.
When instance start on aws, aws update public dns host from public ip -> private ip,
This create issue with cluster communication. If you switch all worker to private dns name,
Cluster does not have any issue in starting which fix band-aid required for starting hdfs cluster and spark cluster.

Tested this out and it seems to work well. End-to-end tests all pass, too. Nice work @pragnesh.

thanks @nchammas

@nchammas does flintrock project intrested in Alluxio service ? I am using it sometime with flintrock and if there is interest, I can work on it, here is my Alluxio change

Thank you for sharing this change. I'd like to keep the services bundled with Flintrock to a bare minimum to keep the maintenance burden low, so I wouldn't want to add Alluxio at this time. I wish I could offer a neat plugin system that lets users contribute new services like Alluxio without having to bake them directly into Flintrock.
Perhaps the one new service I'd consider adding directly into Flintrock is Jupyter (#23) because of how widely used it is, but otherwise I think Flintrock will support only Spark and Hadoop for the foreseeable future.

I understand @nchammas, no issue.


 But I can only see one branch and that one is locked to square size, both when taking a picture and selecting a picture. Is there a branch that can be used as a normal camera?
Thanks in advance!

@woobione,
The project was designed to capture square images, but you can change these settings:
TGCameraViewController - 254
When calling this method the crop size is set. You can change it to the size you want.
TGCameraViewController.xib (Optional)
If you wish, change the size of Capture View.

@Bruno-Furtado Thank's for the help! I will make a fork of this project (probably tomorrow) here that will instead take regular sized pictures if anyone else asks for this ability. Would also like to thank you for a great project and let you know that we will use it in our application :)

I have a fork of this project here that can be used if anyone wants to use normal sized images: https://github.com/DicomExpense/TGCameraViewController
To get through pod use:
pod 'TGCameraViewController', :git => 'https://github.com/DicomExpense/TGCameraViewController.git'

@woobione, your solution is not working. when i launching cameraviewcontroller it showing nothing but a blank window with black color background.

@swabzz We actually use the exact version above in our project with 20k+ users every day and it works just fine for us. However, I haven't updated the code to match the latest version and haven't touched anything of the branch in over a year so there is a big chance it's no longer compliant with the new SDK. Since we are making a new major version of our app which will not use this controller I will most likely not fix it either. Feel free to make your own branch and do the adjustments mentioned by @Bruno-Furtado above. Good luck!

@woobione thanks for quick reply.
Actually i am using this library to capture two different size images with different aspect ratio in my application. can you suggest me what change i should make here please?

@swabzz As I haven't looked at the code since the new changes I don't know if it still works the same. But see @Bruno-Furtado 's answer above. You have to change the capture view to support full screen live view instead of square live view if that's something you want. Then you also have to change the cropping-function to not crop. Then you can do the final cropping in your app code instead of in the camera controller. Best of luck.


Layout dagre switch side of nodes

When I add a node and render again the graph, the nodes change their side.

I did not reproduce the problem: https://codepen.io/Yanyan-Wang/pen/LYGxjGw

应该是和 #1314 同样的问题

duplicated to  #1314


Updated validations.py python script

Updated validations.py python script.
Fixed the behaviour of validate_user function in validations.py.
Closes #1
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!

@googlebot I fixed it.

@googlebot I fixed it.


Multiplication for decimals is incorrect

Describe the bug
Multiplication for decimals is wrong for decimals
To Reproduce

    #[test]
    fn test_decimal_multiplication() {
        let a = Decimal128Array::from_iter_values([1]).with_precision_and_scale(1, 1).unwrap();

        let b = Decimal128Array::from_iter_values([1]).with_precision_and_scale(1, 1).unwrap();

        let e = Decimal128Array::from_iter_values([1]).with_precision_and_scale(2, 2).unwrap();
        let r = multiply(&a, &b).unwrap();
        assert_eq!(e, r);
    }

0.1 * 0.1 should return 0.01.
The current kernel just multiplies the raw integers but ignores the scale/precision values of the.
  left: `PrimitiveArray<Decimal128(2, 2)>
[
  1,
]`,
 right: `PrimitiveArray<Decimal128(38, 10)>
[
  1,
]`', arrow-arith/src/arithmetic.rs:2958:9

Expected behavior
The kernel should add scale as well, values 0.1 * 0.1 should have primitive value 1 with scale 2.
Additional context

Probably we don't expect the kernels in arrow-rs to handle this (but should do it in datafusion).


dbfile

To preprocess a database, what is the data format of dbfile.db? Thank you!

We have a new way to load data into the server. Read more in the docs here: https://docs.blyss.dev/api/class/Bucket#write


/en-US/docs/Web/Tutorials

Request type

[ ] Please close this issue, I accidentally submitted it without adding any details
[ ] New documentation
[x] Correction or update

Details

The information under Introductory Level CSS Tutorials on https://developer.mozilla.org/en-US/docs/Web/Tutorials is outdated and not current with https://developer.mozilla.org/en-US/docs/Web/CSS and https://developer.mozilla.org/en-US/docs/Learn/CSS.
Namely, the Introduction to CSS link should be updated to CSS first steps, and Styling boxes should be updated to CSS building blocks. Both links currently redirect to the new modules but it would help avoid confusion to be consistent across the site. I considered making the edits myself but wasn't sure if the descriptions on the Tutorials page are intentionally different than the more current pages mentioned above, or if it should be copied over from them.

Yup, this needed updating. Thanks for reporting this; I've sorted it out.


num256 removal

still WIP

I was unfortunately unable to remove num because of a rational square root, I won't have time to replace it in the near future.

I should detail my unwrapping strategy:

If in a #[test] or function that will always be locally controlled, unwraps can be used
If there is something like a nonce or block number and we are adding or multiplying small constants to it, 2^256 is a very large number and unwraps are warranted
If the context means that values cannot overflow or indexes cannot be violated (e.g. if from_bytes is called on a slice with a constant range of 32..64, it means from_bytes cannot fail)
Something like ludicrous gas prices that shouldn't go beyond a certain level because of the finite amount of tokens in the system

If you see an unwrap that doesn't look write point it out

There are also places where I replaced f32 with f64 because there's no reason not to use the higher precision


Session templates

Closes #495.
Will be writing extensive tests, just wanted to get the review of the code going now while they are being written.
Review Checklist

Tests were added to cover all code changes
Documentation was added / updated
Code and tests follow standards in CONTRIBUTING.md

@dpuccetti included are the RAML docs for the two new endpoints. Let me know if anything there isn't clear. We had originally talked about blocking the update to a project template if there were sessions without something like a force=true flag (to prevent unexpected long waits when session count is in the thousands). I'm going to hold off on that until we see any actual problems, it's been calculating extremely fast in all my tests.

This looks great :+1:


host: fix isImportFromLibvirt

should check __origin_id not getOriginId
Signed-off-by: wanyaoqi<EMAIL_ADDRESS>What this PR does / why we need it:

Does this PR need to be backport to the previous release branch?:

/cc @swordqiu


Intention to add Checksum to detect crashes and recover?

Is there any intention to add support for checksums to the WAL so that when a crash occurs, log messages can be rolled back to a known good state (I'm thinking about something similar to how lmdb or bbold handles their crash recovery)?

Good point. I think this work can be planned for the near future improvements.

added checksums: https://github.com/vadiminshakov/gowal/releases/tag/v0.0.1


Automatically update packages

This will resolve some issues with using EPEL on older AWS AMIs where ca-certs aren't updated.

Should this be skippable?

I think this should be skippable, as otherwise it may be a hindrance to repeatability.  Just guard it with an attribute then it should be good

Updated. Please review.

:+1: lgtm


Potential New Feature?

Not sure if you want to include this in this project, but I made a  set of config file for selecting and loading binds for individual heroes without having to open the console. Not sure how useful this will be in practice, but it's there.
https://github.com/Sembrani/D2HeroKeys

For selecting too?
 Like Q being meepo, W being ember, and so on?
If that's the case, I think that's too particular. Ideally the perfect implementation we can't make is have a guy with a search box to appear to type hero name, then you press enter.
Closest one is the hero custom mode I've setup, that requires console run.

The keys you press for selecting the hero are you typing the name of the hero, some heroes only need 1 keypress, others require a few...

like:
Key_Shift -> A -> L -> C
turns into alchemists binds and then it shows a say_student message that it loaded?
If that's the case, it seems fast and useful for the main cfg engine too.
Is this switching functionality implemented for all heroes?

it's not a key-shift, it's a toggle, but yes, that's how it works. So the toggle remains selected until a hero is selected, or F1 is pressed, to cancel the hero selection.

I love it. Fairly ingenious and more user friendly, especially if it prints in student chat.
I would love to have it on the super compact layout, integrated or replacing the current hero modes in place (with split functionality of ALT, SPACE, DEFAULT).
I would love it bound to an F key and I think double pressing it should remove binds (nohero mode)

I already have a branch which can be copied over the super compact binds, and is bound to "p", so it would only need a few minor changes after that. Looking at the keybinds, all the F-keys are already bound, unless you use a modifier layer, and I'm not too sure how well that would work.

Just let me know what key you want it bound to, and any other keys that need to be rebound around it and I should be able to have it done fairly quickly.

I have a few ideas for the F keys to make them more usable:

Toggle combat log on F10
Toggle auto select summons on Space-F5
Toggle select hero mode (new) on F6
I would also like SPACE+F2 to jump to courier (new), to make it consistent with other things
Take from stash on SPACE+Y
And the visual layouts should be modified with the changes.

This toggle should be on a far to reach key, because of the trigger frequency.
On the side, I haven't tested new version. Is SPACE+H the scoreboard and SPACE+Tab cycle jump correctly?
Note is that Space+Y was unused.

These changes should be traced to notify changes for then ext changelog

Another idea I had is if there is a way to make F5 enable/disable switch the in-game menu.
It only enables menu now.

Yes, SPACE+H for scoreboard works well, but I can't say anything for SPACE+Tab

Awesome. Testing it right now.
I'll handle the visual layout keybinds later on.

Alright, I cleaned stuff up, done some testing and added everything to https://github.com/loopuleasa/Dota2-TheCore-Config-Engine/issues/26

Good job.

Thanks for the reminder. will get onto that pretty soon.


[stable/openebs] Update charts to 1.5.0 release

Updated image tags as per 1.5.0 release
Updated values.yaml
Updated README
Updated Chart.yaml
Updated deployment-local-provisioner.yaml
Updated deployment-maya-apiserver.yaml
Updated deployment-ndm-operator.yaml

Signed-off-by: ChandanSagar<EMAIL_ADDRESS>Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/mychartname])

/hold

/ok-to-test

/unhold

The release notes for 1.5.0 are available at : https://github.com/openebs/openebs/wiki/Change-Summary-v1.5
Waiting for dog fooding of new version to be completed on OpenEBS Director.


OTFCompiler: allow specifying a custom feature file

Useful when you want to use a FEA that's separate from the UFO itself.
r? @typesupply

Just pushed a fix to this.

This seems fine. I'll merge it and open an issue for it to be fixed in the ufo3 branch (unless you want to make the change there now).

This seems fine. I'll merge it and open an issue for it to be fixed in the ufo3 branch (unless you want to make the change there now).

Maybe I'll do another PR later to merge all patches that aren't in ufo3 yet.

That would be incredibly helpful. I'm overwhelmed with stuff to do right now.


[Blazor] Ports all the missing commits from 3.1 into master

I'm starting this PR, but I'm expecting everyone to collaborate on this to get it to a good place.

What happened to the merge bot?

@Tratcher Not sure, but there are a huge amounts of missing commits. I'm porting the ones from components manually

This was done separately by the merge bot.


 So I created a separate branch for this.

Not needed.


Please correct Windows-driver-samples/hid/vhidmini2/ project

Project documentation states "The HID minidriver sample demonstrates how to write a HID minidriver using User-Mode Driver Framework (UMDF)."
However, the code never calls HidRegisterMinidriver which documentation says "The HidRegisterMinidriver routine is called by HID minidrivers, during their initialization, to register with the HID class driver."
At the very least, documentation for the project should remove all mention of "HID minidriver".

@LarryK348 I have been trying to get the same sample working today without much success. Though my problem has been that devcon install fails.
Digging into setupapi.dev.log, I notice that the issue is root\vhidminium is not found. Here's the relevant snippet:
{Build Driver List} 17:27:23.681
     dvi:           Searching for hardware ID(s):
     dvi:                root\vhidminium
     dvi:      {Build Driver List - exit(0x00000000)} 17:27:23.730
!    ndv:      No matching drivers found in single INF

Did you manage to build the sample and install it?
BTW, the doc now says thus: "Both MsHidUmdf.sys and MsHidKmdf.sys call the HID class driver's HidRegisterMinidriver routine to register as the actual HID minidriver. Although these drivers act as the device's function driver, they just pass I/O requests from the class driver to your driver (and are thus sometimes called pass-through drivers)." Does that mean that vhidmini2 minidrivers do not need to call HidRegisterMinidriver? Or the sample still needs to be updated.
Would appreciate any feedback. Tks.

Ignore my previous comment - I was trying to deploy x86 driver on x64 OS, which obviously would fail!


Allow offline execution of validation tests

Allow offline execution of validation tests.
This is particularly relevant to running tests on HPC resources where the compute nodes might not have access to external URLs.

Mechanism to avoid duplication can be incorporated after implementation of HumanBrainProject/hbp-validation-framework#230

Closed via #37


ci: add disruptive pods to I/O soak test MQ-252

Create a fio docker image specifically designed for mayastor
e2e testing.
Add CIRegistry entry to e2econfig so that this image and future
test docker images are retrieved from the CI registry, independent
of Mayastor images.
Re-jig io soak tests to use the mayastor/e2e-fio docker image.
This removes the need to run multiple threads to run fio, as
the e2e-fio pod can be deployed to run fio on creation.
Add disruptor fio jobs which deploy pods running the e2e-fio image,
restart policy set to restart always, and configured to raise SIGSEGV
after a configurable delay, whilst running fio.
Update the CI configuration to scale up the number of test pods
and volumes to run as part of the IO soak test.

bors try

bors merge


Govway API gateway for PDND

Given https://govway.org is the suggested Open Source API Gateway that implements the authentication protocol for PDND https://developers.italia.it/it/software/link-it-govway-7218c1.html
would it be possible to documentation on how it should be configured for PDND purposes?
thanks a lot
nicola

Hi @nicolabeghin,
We don't have a specific example to provide you for integrating the Govway API Gateway with the PDND. However, the documentation for this framework includes information regarding that.
The choice of API Gateway depends on your software architecture, and there are many other API Gateway solutions you can use. My recommendation is to refer to the PDND documentation to select the API Gateway that best fits your architecture for integrating with the PDND.
Finally, please note that integration with the PDND does not necessarily require the use of an API Gateway for service delivery.
Thanks

I finally got to have a working govway instance both for UAT and PROD PDND - for anyone interested, feel free to contact me privately, the configuration is pretty complex and convoluted


HadoopJobUtils shouldn't obtain application ID from execution dir's log file.

findApplicationIdFromLog(String logFilePath, Logger log) is trying to grep application_id from job log in the execution dir. However execution dir has retention period which is 1 day. We might shorten the retention period in the future (see https://github.com/azkaban/azkaban/pull/1803), so HadoopJobUtils shouldn't rely on log file to get the application ID.

Who uses this method and for what purposes?

@HappyRay HadoopJobUtils relies on this method to kill a hadoop job with application id.

actually I took it back, execution dir cleaning up will skip currently running executions. So until the execution is killed, the execution dir won't be deleted.
azkaban.execapp.FlowRunnerManager
      for (final File exDir : executionDirs) {
        try {
          final int execId = Integer.valueOf(exDir.getName());
          **if (FlowRunnerManager.this.runningFlows.containsKey(execId)
              || FlowRunnerManager.this.recentlyFinishedFlows.containsKey(execId)) {**
            continue;
          }
        } catch (final NumberFormatException e) {
          logger.error("Can't delete exec dir " + exDir.getName()
              + " it is not a number");
          continue;
        }

        synchronized (FlowRunnerManager.this.executionDirDeletionSync) {
          try {
            FileUtils.deleteDirectory(exDir);
          } catch (final IOException e) {
            logger.error("Error cleaning execution dir " + exDir.getPath(), e);
          }
        }
      }


Prevent link alteration in translations

[ ] Add a validation to our translation spec that ensures the value of an href attribute in HTML matches the source value exactly
[ ] Alter any translation that accepts a URL variable but allows the translator to edit the  tag so that the variable is the entire tag (there are several existing examples of this style, usually in the form of start_tag and end_tag variables)
[ ] Consider altering the t ruby and js methods to insert rel="noopener" into all  tags. This seems like it might be hard and not worth it, so maybe time box it

JS Regex expression should handle item#1 to break into fragments with link exclusion.
For item#2 can you clarify demo input and desired output?

For item#2 can you clarify demo input and desired output?

Sure. If the translation looks like
some_key: This is a link

and gets used like t( "some_key", { url: "https://www.inaturalist.org" } ), we'd want to replace it with
some_key: This is %{start_tag}a link%{end_tag}

which would get used more like t( "some_key", { start_tag: "", end_tag: "" } )

Shall we append the solution to the existing spec audit for locales in TranslationsController?

No. That controller is just for listing available locales. This would be a part of https://github.com/inaturalist/inaturalist/blob/1b8b27493b2fa9938dda5bfe72c9959638528907/spec/lib/translations_spec.rb

Thanks, will tackle this in a bit after reading up on Crowdin and architecture.
For item#1:

Are there any special markup tags in the pipeline that I may need to account for if I were to apply a universal tag and attribute processor?

e.g., preprocessor with component/variable expansion

Otherwise I propose tokenizing standardized DOM tags, calling t(), and handling attribute translation separately. (for example, translating attribute-embedded text such as alt=, etc...)
Per your example above, this should facilitate moving away from the rigid key/value parameter call of t()?

Are there any special markup tags in the pipeline that I may need to account for if I were to apply a universal tag and attribute processor?

Hm, I don't think we have special XML-ish tags, but there are inflections, e.g. @gender{m:un|f:une|n:un(e)} %{resource}. FWIW, I'm not sure you need to write some kind of custom HTML parser in the spec. Nokogiri will probably work fine.


Enrich the conversations data with sentiment/question/emotion

For each message in the conversation, we would like to add additional metadata if the message is:

Question
Positive/Negative/Neutral as sentiment
What is the emotion

Here is an example:

TeacherFeedback [question:positive:happy]

StudentReply01 [statement:neutral:happy]
StudentTeacherReply02 [question:negative:sad]

Initial Estimates

Planning (issue writeup, initial rough time estimations): 30 min
Research (S3 access/updating, Python json file updates, Python syntax/methods, inclusion of additional file access and compiling information for json inclusion): 5 hours
Implementation and Testing (code writing/optimizing, pulling information from multiple sources, testing adding of data to existing s3 object, connection/uploading to s3, error handling): 5 hours
Documentation (writeup code explanation and how-to for unfamiliar user): 1 hour
Deployment (PR Write-up): 1 hour

Enrich Conversation data with sentiment/question/emotion metadata
Pre-Requisites:

Imports for boto3 and nltk
AWS credentials saved in ~/.aws/credentials file

Implementation:

Script accesses the S3 bucket and the auto-floop-s3-export3.json file. Data for all json entries are pulled. To speed up verification, a list of 50 conversations are taken for analysis and put into a List.
For each conversation in the list, for each message in the conversation, the Text value is retrieved and run through three functions:

sentAnalyze (determines "question/sentence" state)
sentence_classification (determines "positive/negative/neutral" state)
emoAnalyze (determines "emotion" state)

Each message dictionary is appended with a new K:V pair: "metadata": ["question", "posNeg", "emotion"]
The updated conversations are then put into a new json file (enriched-conv-sample.json), and the json file object is added to the designated s3 bucket .

Input:

script execution, which pulls in json file from s3.

Output:

new json object, enriched-conv-sample.json, in the designated s3 bucket.

Activities Documentation

Date
Activity
Time Spent

2/23/22
Planning
15 minutes

Research
1 hour

Implementation/ Testing
1.25 hours

Task Management
15 minutes

2/24/22
Research
1.75 hours

Implementation/ Testing
30 minutes

2/25/22
Implementation/ Testing
2.75 hours

Documentation
15 minutes

Deployment
1 hour

Task Management
30 minutes

Time Totals

Activity
Time Spent

Planning
.25 hour

Research
2.75 hours

Implementation/Testing
4.25 hours

Task Management
.75 hour

Deployment
1 hour

Documentation
.25 hours

TOTAL TIME:
9.25 HOURS


Update background docs for Box

Update Background docs for Box to include options of size & repeat that are supported but not currently displayed.

@taysea update yarn lock?

Was not allowing me to build and preview the site -- tried re-forking the repo and started a new PR.


⚠ Bofrost.at has degraded performance

In 25d5378, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1415 ms

Resolved: Bofrost.at performance has improved in 5ae7c6b after 1 hour, 7 minutes.


Comments on deleted files can not be revealed

When we checkout to pull request https://github.com/auchenberg/pullrequest-demo/pull/7, we have comments on app.js but the file is deleted already, so clicking on comments on comments panel will fail.

I can't see the comments in the editor https://github.com/Microsoft/vscode-pull-request-github/issues/343 - but clicking them in the panel does open the editor so I think it's a new issue.


Remove releases section from the site

The website for querydsl has a releases page that is seriously out of date. Only because I was fortunate enough to check the github repository I found that QueryDSL is alive and not dead since 2016.
Just link the releases to the github.com releases page and you're set to go.

Oh that is also possible.
I was (a long long while ago) busy with updating the DOM to be more semantically meaningful, but is it really worthy to throw the page away? 🤔

The site has since been updated.


Change design of "PR" tag to be more easily distinguishable from NR/CR

Again referencing this forum thread, the change from "PB" to "PR" has been implemented, but as multiple users note, it looks a bit confusing as the design is very similar to the NR/CR tag.
@nsilvestri would you be interested in an update to #122 where you make a minor design change to the tag? My first thought would be to make it more opaque and thus less attention-grabbing than the NR/CR tags (I don't think a pure colour change would be enough, as I think NR/CR are already different colours?). I'm not a UI designer, though, so up to you :D

Thanks for the feedback Duncan, I've been monitoring the response to this as well. I've thrown together a quick alternative, and I'd appreciate feedback on it. Note that these styles apply globally to the PR tag, not just on the round results table.
Dark mode:

Light mode:

@nsilvestri I think there isn't enough contrast between the tag background and page background. I would try reversing the background and font colour, so on light mode it can be a dark grey background and white font.

@jonatanklosko here's the inverse of the previous iteration:

In my own opinion, this is about equally as attention-grabbing as the original iteration, especially on light mode. Maybe the solution to this problem is keeping the blue PR tags, but hiding them behind a user-controlled switch that toggles showing/hiding PRs? The switch would pass a hidePr prop to the RecordTagBadges on the RoundResultsTable.

Yeah, I agree that feel to much, but if there are are a lot of PRs I think any colour will have this issue. So another alternative I can think of is removing the background entirely, with a little less padding; this way it wouldn't stand out, but the information would be there.
(Theoretically we could have a user configuration flag, but I'd rather figure out a design that is not concerning and then let people get used to it)

Here's a version with 0.1em left and right padding and no background color.

It's not terrible, but in my opinion it now feels like the PR badge has some styling error because it's not consistent with the other badges. I'll admit my personal favorite so far is the first draft, with light gray on light theme/dark gray on dark theme. Maybe these designs could be put to a poll to get some more community feedback?

My personal bias is also towards the first iteration - I think it is clearly different from NR/CR and doesn't grab attention unnecesarily, but is still easily visible and distinguishable - especially to someone looking for it.
If we want to get community feedback, I'd suggest the forum thread as a place to post these screenshots.

Alright, I'm ok with the first one then :)

@jonatanklosko what if PR badge had this lower contrast style only on the round results table, while it stayed blue in all other contexts? I think there's value in maintaining the higher visibility color in the dialog boxes and competitor results table, and would allow the best of both worlds.
As a preliminary complexity analysis, I'd probably implement it with a RecordTagProps that gets passed as a prop to the RecordTagBadge in RoundResultsTable, which then gets spread in the RecordTag props, allowing a sort of one-off override of the styles specified in the RecordTag.

@nsilvestri I'm not sure if having two distinct variants for PRs is going to be confusing, but we can start with that and see.
As for the implementation I'd rather add litePr flag, rather than style overrides, just to encapsulate that variant more :)

Closed in #127.


Implement set operations: Union, Intersection, Difference

Description
This PR extends the functionality of Set by adding the following methods.
func (s *Set[T]) Union(s2 *Set[T]) *Set[T] -> returns a new set as the union of both sets
func (s *Set[T]) Intersection(s2 *Set[T]) *Set[T] -> returns a new set with elements that exist in both sets
func (s *Set[T]) Difference(s2 *Set[T]) *Set[T] -> returns a new set with the elements that exists in s1 but not in s2
Related Issues
Closes #30
Checklist

[x] I have tested this code.
[x] I have reviewed the code changes.
[x] I have updated the documentation.
[x] This PR follows the coding standards.
[x] I have added tests for the changes.

I've just noticed something. In the issue, you described to implement the methods in one way:
func Union(s1 * Set[T], s2 *Set[T]) *Set[T] // like one of the factory methods.
but I actually implemented it in another way:
func (s *Set[T]) Union(s2 *Set[T]) *Set[T] // as I was going to update the s set.
Should I fix it to be like the factory methods instead (I guess it makes sense)?

I've just noticed something. In the issue, you described to implement the methods in one way: func Union(s1 * Set[T], s2 *Set[T]) *Set[T] // like one of the factory methods.
but I actually implemented it in another way: func (s *Set[T]) Union(s2 *Set[T]) *Set[T] // as I was going to update the s set.
Should I fix it to be like the factory methods instead (I guess it makes sense)?

I am okay with both implementations.
@aswath-s-tw @gopinathr143 What do you think?

Yeah, I'm totally fine with the current implementation. I don't see a reason to change to the factory method signature. We can create another issue later on if we see someone else's usage requires the factory signature !

@zegabr Thanks for your contribution!


Grok Exporter does not support web.telemetry-path

Grok Exporter does not support the web.telemetry-path option to set Path under which to expose metrics.
This is available in node_exporter as well as consul exporter, can this option be provided?
eg. Line 74 here: https://github.com/prometheus/node_exporter/blob/master/node_exporter.go

This should be easy to implement, and you are not the first person to ask :)
Will include it in the next release.

Thanks Fabian, looking forward to the next release.

Made the path configurable in the config file:
server:
    host: localhost
    port: 9144
    path: /bla

The code will be included in the next release.

Released v0.2.5 including the change.

@fstab 👍 Thanks will check this next week


Backport: Helm Chart needs to support L7 TLS offloading for external termination

https://github.com/rancher/rancher/issues/16155

No need to backport server-chart issues. Closing in favor of original issue.


Changing the SteelCore semantics and the Steel libraries to use tokens for invariants

In response to issue #2814 , this PR revises the monotonic state effect in FStar.MST and MSTTotal to use an explicit token for the witnessed predicate.
This change bubbles up through to Steel as an explicit token for invariants too.
Recapping the changes here, as discussed also on #2814

The main change is that inv p is now also an explicit token of type Type0 and the i >--> p prop is no longer visible in the Steel.Memory interface.

Additionally, the libraries that provide monotonic references are changed to provide their own witnessed tokens, rather than just providing prop-based witnesses.

The new_invariant, with_invariant and with_invariant_g operations are no longer ghost, since this would allow them to reveal an erased (witnessed p) as a witnessed p and break the abstraction. Instead, with_invariant_g is now classified as SteelAtomic unobservable and with_invariariant is parametric in the observability flag of computation it takes as argument.

All the Steel files (except one, noted below) in the F* repo work again without significant change. Also, the Zeta project is also not significantly impacted.

However, the this change results in a loss of expressiveness since it is no longer possible to turn an invariant into an slprop/vprop. That is, pure (i >--> p) is no longer expressible in the Steel---you need an explicit invariant token inv p. This results in a change to the style used in specifying the invariant of a lock-coupling list in Steel.LockCoupling.fsti. That invariants is now phrased using a combination of a inductive type that simultaneously specifies the type of a list cell and the invariant that the lock protects, combined with a recursive predicate on that structure, where the two interact via the use of fractional permissions.

We also have extraction support for invariant-related constructs in F*. It is important from the F* type checker's perspective that witnessed and hence inv is not an erasable type. However, for concrete executions, these tokens have no information content (since they are only eliminated by recall, an axiom). So, in FStar.Extraction.ML.Term, applying to extracting Steel to both OCaml and C, we now

extract inv p to unit
extract (with_invariant_g ... (fun _ -> some ghost function)) to ()
extract (with_invariant ... f) to f().

With these changes, the extraction behavior is mostly unchanged:
For example,
let test (_:unit)
  : STT (inv emp & nat) emp (fun _ -> emp)
  = let i = new_invariant emp in
    i, 17

is extracted to ML as
let (test : unit -> (unit * Prims.nat)) =
  fun uu___ -> ((), (Prims.of_int (17)))

and via krml (which does unit elimination) to C as
Prims_int TestDTup_test(void)
{
  return (krml_checked_int_t)17;
}

Thanks Nik for merging.
I took care of pushing a release recording the last state of F* before merging this PR: https://github.com/FStarLang/FStar/releases/tag/v2023.02.01


Ask user for old password when changing password

This is accomplished by using SetPasswordForm instead of PasswordChangeForm.
This adds extra security, as without this commit, an attacker that has access to
a user's session at one point in time will be able to change the user's password
and gain permanent access.

I see the drone test fails, is there something I need to do on my side to rectify this? I'm not quite sure about the cause of the failure.

@math1985 That was a code style error that existed in master (fixed now) at the time you submitted the PR - if you rebase this PR on top of current master (or, if you're not familiar with rebasing, merge current master into it) then the drone build should hopefully pass.

Thanks, rebased, the error is gone now.

This looks good to me :+1:

This has been merged in 03cba5b, and you have been added to the CONTRIBUTORS.rst list. Thanks!

Thanks for reviewing! The PR didn't close automatically, so I'll close it manually.


feat: fee foresight support

Please read contributing guidelines and remove this line.

#10262  👈 (View in Graphite)
master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Updated the artifact hash since it had changed after the noir sync


How to change *_lbb.npy to the .mha file?

Hi,
When I run the main.py with "--test 1",
I got the the result folder "./bbox" which contain some *_pbb.npy and *_lbb.npy files.
I want to change them to the .mha file.
But I do not know how to do.
Hope you can help me :)
Best,
Yaliang

You can write python script to transform numpy format to your format.

Hi,
Thanks a lot for your reply.
Do you have some examples of the script to show how to do that?
Best,
Yaliang


Conflated "Install" and "Use" steps in README

The current "Use" step in the README should really be part of the "Install" step:

Install
Install steal-less as an npm dependency:
npm install steal-less --save-dev

Next, add steal-less as a plugin to the steal section of package.json:
...

"steal": {
  "plugins": [
    "steal-less"
  ]
}

Then, the "Use" step should probably be updated to something like:

Use
Once installed, steal-less allows you to "import, require, or steal" Less files within your project:
// ES6
import "style.less!";

// AMD
define(["style.less!"],function(){ ... });

// CommonJS
require("style.less!");

// steal
steal("style.less!")

I can submit a PR for this if it's agreeable.

Sure

Closed by #51


Preferences for editing dataTable attributes

For fun, I tried to lay out a "perfect" dataTable attribute editor. A whiteboard or sketch would work best, but I used a google doc.
https://docs.google.com/document/d/1O4xziJ1rpYb06AgI9NV2ziHIXOd01bv1_agv803OO48/edit#
Summary goals for a pie-in-the-sky dataTable attribute editor:

User is told what fields will need to be filled in (specific to that measType). Maybe this is a help-popup.

Nominal/ordinal: enumerations  (optional)
dateTime: a format string
interval/ratio: unit

User can know when they’re done from a summary page. (maybe rows turn green when minimum info is added?)
User can view and edit common info together (e.g, fields used by all are on the summary page)

Margaret's link doesn't work now, but this sounds like something I just came here to add. One thing that keeps me from using ezEML on some datasets is a wide table. With 50 columns, it just isn't workable to click through each column to set definitions and units after a table import. With a gridded attribute editor as suggested here this barrier to using ezEML would be solved!

@hubbardbrook I updated the sharing on that link.


Fix #6051 Deprecate keycode to code

Description
Replace .keyCode with .code to follow new MDN API
https://developer.mozilla.org/en-US/docs/Web/API/KeyboardEvent/keyCode#browser_compatibility
Closes: #6051

This breaks non-QWERTY layouts (Workplace: https://fb.workplace.com/groups/lexicaldiscussion/posts/1950653992035669/) and the numpad enter key (https://fb.workplace.com/groups/lexicaldiscussion/posts/1950753535359048/), because the web platform assumes a QWERTY layout:

For example, the code returned is "KeyQ" for the Q key on a QWERTY layout keyboard, but the same code value also represents the ' key on Dvorak keyboards and the A key on AZERTY keyboards. That makes it impossible to use the value of code to determine what the name of the key is to users if they're not using an anticipated keyboard layout.

Probably safer to use key instead of code (e.g. https://github.com/Daniel15/Website/blob/master/Daniel15.Web/wwwroot/Content/not_an_easter_egg/secret.ts#L20)


English subtitles youtube

Can you upload English subtitles for youtube videos? I need to use a chrome plug-in to translate to understand. The automatically generated subtitles experience is very poor and the speed is very slow, which makes me unable to understand the content. I sincerely hope to upload English subtitles, thank you very much

Thanks for reaching out. This issue tracker is for the issues related to the codebases on GitHub. Please reach out to the community forum: https://forum.freecodecamp.org for your query.


Parameter to specify the frame_id used for all published images and point clouds.

Suggestion
Hello,
When using the camera with a robot it is useful to calibrate it in order to obtain a calibrated frame that removes any error created by the differences between the real hardware and the TF being published.
Proposal
In other drivers such as the ROS driver for the ZIVID cameras or the Intel cameras, you have the ability to change the frame_id to which the pointcloud is referred.

It simply changes the field of header/frame_id of the topic that publishes the topic and images to a proper frame that is calibrated for the given robot or to any frame really that the user wants to refer everything to.
Warm regards.

Hello, is there any bandwith available for this? If time is an issue I could go into the code and recreate the changes made by the ZIVID driver to equally implement it.
Thanks in advance.

@Danilrivero
Apologies, I’ve been quite busy lately. I’ll add one later today and check if it meets your expectations.

Hi @Danilrivero
I checked the camera param by ros2 param list, and I see all frames are configurable.
ros2 param list
/camera/camera:
  .color.image_raw.format
  .color.image_raw.jpeg_quality
  .color.image_raw.png_level
  .color.image_raw.tiff.res_unit
  .color.image_raw.tiff.xdpi
  .color.image_raw.tiff.ydpi
  .depth.image_raw.format
  .depth.image_raw.jpeg_quality
  .depth.image_raw.png_level
  .depth.image_raw.tiff.res_unit
  .depth.image_raw.tiff.xdpi
  .depth.image_raw.tiff.ydpi
  accel_optical_frame_id
  accel_qos
  accel_range
  accel_rate
  align_mode
  align_target_stream
  angular_vel_cov
  camera_accel_frame_id
  camera_color_frame_id
  camera_depth_frame_id
  camera_gyro_frame_id
  camera_ir_frame_id
  camera_left_ir_frame_id
  camera_name
  camera_right_ir_frame_id
  color.image_raw.enable_pub_plugins
  color_camera_info_qos
  color_delay_us
  color_exposure
  color_format
  color_fps
  color_gain
  color_height
  color_info_url
  color_optical_frame_id
  color_qos
  color_white_balance
  color_width
  connection_delay
  decimation_filter_scale_
  depth.image_raw.enable_pub_plugins
  depth_camera_info_qos
  depth_delay_us
  depth_filter_config
  depth_format
  depth_fps
  depth_height
  depth_optical_frame_id
  depth_precision
  depth_qos
  depth_registration
  depth_width
  depth_work_mode
  device_num
  device_preset
  diagnostic_period
  diagnostic_updater.period
  enable_3d_reconstruction_mode
  enable_accel
  enable_color
  enable_color_auto_exposure
  enable_color_auto_white_balance
  enable_color_undistortion
  enable_colored_point_cloud
  enable_d2c_viewer
  enable_decimation_filter
  enable_depth
  enable_depth_scale
  enable_frame_sync
  enable_gyro
  enable_hardware_d2d
  enable_hdr_merge
  enable_heartbeat
  enable_hole_filling_filter
  enable_ir
  enable_ir_auto_exposure
  enable_ir_long_exposure
  enable_laser
  enable_ldp
  enable_left_ir
  enable_noise_removal_filter
  enable_point_cloud
  enable_right_ir
  enable_sequence_id_filter
  enable_soft_filter
  enable_spatial_filter
  enable_sync_host_time
  enable_sync_output_accel_gyro
  enable_temporal_filter
  enable_threshold_filter
  enumerate_net_device
  flip_color
  flip_depth
  flip_ir
  flip_left_ir
  flip_right_ir
  frames_per_trigger
  gyro_optical_frame_id
  gyro_qos
  gyro_range
  gyro_rate
  hdr_merge_exposure_1
  hdr_merge_exposure_2
  hdr_merge_gain_1
  hdr_merge_gain_2
  hole_filling_filter_mode
  ir_camera_info_qos
  ir_exposure
  ir_format
  ir_fps
  ir_gain
  ir_height
  ir_info_url
  ir_optical_frame_id
  ir_qos
  ir_width
  laser_energy_level
  laser_on_off_mode
  left_ir_camera_info_qos
  left_ir_format
  left_ir_fps
  left_ir_height
  left_ir_optical_frame_id
  left_ir_qos
  left_ir_width
  linear_accel_cov
  log_level
  max_depth_limit
  max_save_images_count
  min_depth_limit
  net_device_ip
  net_device_port
  noise_removal_filter_max_size
  noise_removal_filter_min_diff
  ordered_pc
  point_cloud_qos
  publish_tf
  qos_overrides./parameter_events.publisher.depth
  qos_overrides./parameter_events.publisher.durability
  qos_overrides./parameter_events.publisher.history
  qos_overrides./parameter_events.publisher.reliability
  qos_overrides./tf.publisher.depth
  qos_overrides./tf.publisher.durability
  qos_overrides./tf.publisher.history
  qos_overrides./tf.publisher.reliability
  qos_overrides./tf_static.publisher.depth
  qos_overrides./tf_static.publisher.history
  qos_overrides./tf_static.publisher.reliability
  retry_on_usb3_detection_failure
  right_ir_camera_info_qos
  right_ir_format
  right_ir_fps
  right_ir_height
  right_ir_optical_frame_id
  right_ir_qos
  right_ir_width
  sequence_id_filter_id
  serial_number
  soft_filter_max_diff
  soft_filter_speckle_size
  software_trigger_period
  spatial_filter_alpha
  spatial_filter_diff_threshold
  spatial_filter_magnitude
  spatial_filter_radius
  sync_mode
  temporal_filter_diff_threshold
  temporal_filter_weight
  tf_publish_rate
  threshold_filter_max
  threshold_filter_min
  time_domain
  trigger2image_delay_us
  trigger_out_delay_us
  trigger_out_enabled
  usb_port
  use_sim_time

If you want to change the color_optical_frame_id, use the following line:
DeclareLaunchArgument('color_optical_frame_id', 'my_frame_id')

In the launch file. This will set the color_optical_frame_id to 'my_frame_id'.

Hello, would it be possible to apply the same structure to the pointcloud?
The generated pointcloud from depth and color images is correct. I simply need it's frame_id field to be changed like you have proposed from the original 'camera_color_optical_frame' to my new frame_id 'calibrated_camera_color_optical_frame'. It seems like there is no such parameter available for the pointcloud, only for the color and depth images.
Thanks for your swift response and help with the issue as well

@Danilrivero
I got your meaning. Can I add point cloud frame_id but not add it to the tf tree?

@Danilrivero I got your meaning. Can I add point cloud frame_id but not add it to the tf tree?

Exactly, when it gets published I can change the 'frame_id' value only, this way, when I do a tf transformation (because the new  'frame_id' I am using is being published and exists in the tf tree) I can properly do it without having to do the hassle of changing it myself.
In short, changing the value of std_msgs/msg/Header.msg/string frame_id for each pointcloud being published at runtime would be sufficient. Checking your commit it seems to be exactly what I was looking for, this will be helpful for many. I will test it as soon as possible and close the issue if solved.
I would like to thank you again for your swift response and help, cheers.


RoleMembers collection in advanced scripting

Describe the bug
I usually use the 'Members' collection, but for some reason tried the 'RoleMembers' collection. When the following code is executed, it returns a new output for every letter of every role member. As an example, if I have a role member of<EMAIL_ADDRESS>in the Reader role, it would output 'H'...'e'...'l'...'l'...'o'... etc. It's pretty strange. Think the solution is to just use the Members collection but figured I would raise this as an issue.
foreach (var m in Model.Roles["Reader"].RoleMembers.ToList())
{
m.Output();
}
Application specifics

Tabular Editor version: 2.14.2

RoleMembers is a string (which is just the combined list of members separated by newline characters). When you call .ToList() on a string, you get a list of every single character in the string. So in other words, it's a feature, not a bug ;-)

I see ahaha, thanks :)


Add magefiles

Adds magefiles to replace makefiles
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO)
[ ] A PR is opened to update the documentation on our docs repo

Fixes #42

@khaosdoctor a few comments. can you also keep the Makefile for now? we can phase it out in a follow-up after things seem to be working ok

Since I already deleted it from here, can we remove those Makefiles and I create a mage binary so we don't need to have mage installed?

The docs are the only thing missing now

Added docs, these are good to go

Since I already deleted it from here, can we remove those Makefiles and I create a mage binary so we don't need to have mage installed?

@khaosdoctor I'd like to leave the Makefile in completely until we all get comfortable using mage (or the binary). at that point, I'm perfectly fine removing

@arschles done with the previous review

@khaosdoctor this looks great except for one more piece - can we not have a ./build binary? Instead, can we ask people to download mage and run mage <command>?

Done on 4a93697

Closing this to try to fix DCO


Set current version defined by version number and build number in main build file

This should allow the downstream F-Droid build to pick up the current version numbers.
Code Review Checklist

[ ] Description accurately reflects what changes are being made.
[ ] Description explains why the changes are being made (or references an issue containing one).
[ ] The PR appropriately sized.
[ ] New code has enough tests.
[ ] New code has enough documentation to answer "how do I use it?" and "what does it do?".
[ ] Existing documentation is up-to-date, if impacted.

There are some IOS related commits. Should they be here?

No, they are gone now.

Are these versions public (like, do users see them)? It would be odd to "upgrade" from 16.0.0 to 1.0.1+30 (whatever that means). Elsewhere we talk about CalVer; does this have a "versioning scheme"?

This is more cleanup activity as pubspec has demo entry. Current public version is 1.0.1 (previous 1.0.0), if we are not using parameters during build, then value from pubspec will be taked. 1.0.1 is version name, 30 is version code (which in android case should be incremental number): https://developer.android.com/studio/publish/versioning

So far we did not reflect the version number we were actually using somewhere in the configuration/code. This is what we should do and what is a requirement when deploying to F-Droid. If you look at the releases on GitHub or in our Play Store entry you will see the versions 1.0.0 and the recent update to 1.0.1.


Consider separating SDL from package management

I'm starting to dislike SDL integration through vcpkg, (or any package manager) as it hijacks the declaration of main globally in all VS solutions and projects even if they are unrelated to the project that actually needs it due to how vcpkg works. It fundamentally breaks WinMain projects as well.
Is there a way to add the required files to a Thirdparty folder and manually compile SDL in? It'll add a maintenance step whenever SDL adds new features that you may want to take advantage of, but It's not that hard to copy-paste the source code.

The hijacking of main has nothing to do with vcpkg. That's part of SDL. In the header, it redefines main so it can hijack it:
#define main SDL_main

Regardless of how we install and use SDL, that will still be an issue. In particular, Linux uses the apt package manager to install SDL, and it has the same issue (#889). You can partially avoid it by not including the SDL header from your main.cpp, however you also need to avoid adding compiler flags such as -Dmain=SDL_main, which defines that same macro from the command line. That means being careful about how sdl2-config is used, since it may define that macro by default.
I recommend we close the case.

Some relevant material, in case you're interested:
https://stackoverflow.com/questions/11976084/why-sdl-defines-main-macro

The global installation via vcpkg does affect unrelated projects though because it's system-wide. I think that's my main issue with it. If you can reduce the scope of its changes to just the relevant projects I wouldn't be as concerned.

In fact, the "feature" can be turned off with a small bit of code:
https://wiki.libsdl.org/SDL_SetMainReady
#define SDL_MAIN_HANDLED
#include "SDL.h"

int main(int argc, char *argv[])
{
    SDL_SetMainReady();
    SDL_Init(SDL_INIT_VIDEO);

    ...

    SDL_Quit();

    return 0;
}

Sounds like there are other reasons for wanting to move away from vcpkg.
I have been meaning to look into Chocolatey, though I don't know if it would be any different.
In particular, I've been pondering switching the Windows CI builds from AppVeyor to CircleCI, so that all platforms can be handled with a single CI provider. The current Windows image on CircleCI doesn't have vcpkg installed, but I believe it does have Chocolatey.

The SDL_main discussion should probably be part of #889.

I looked a bit into Chocolatey. It seems more geared towards installing end user programs, not developer libraries. I didn't see any packages available for SDL/SDL2, nor for TinyXML. Looks like that idea was a bit of a dead end.

I think we may need to update the title of this issue, and perhaps redefine the scope. I believe the major point was:

The global installation via vcpkg does affect unrelated projects though because it's system-wide. I think that's my main issue with it. If you can reduce the scope of its changes to just the relevant projects I wouldn't be as concerned.

I suppose end users could potentially choose to use vcpkg without system wide integration, which might solve that immediate need, though could add extra environment setup each time a developer works on the project on such systems.
Outside of that, I'm not entirely sure what can be done. We're not exactly willing to walk away from package management, there are just too many benefits. So unless I'm missing something, there doesn't seem to be any clear actionable steps we can take on our end for this. Without something clear to do on our end, I'm thinking we should close this issue.

I think we may need to update the title of this issue, and perhaps redefine the scope. I believe the major point was:

The global installation via vcpkg does affect unrelated projects though because it's system-wide. I think that's my main issue with it. If you can reduce the scope of its changes to just the relevant projects I wouldn't be as concerned.

I suppose end users could potentially choose to use vcpkg without system wide integration, which might solve that immediate need, though could add extra environment setup each time a developer works on the project on such systems.

Correct. Requiring system-wide integration of vcpkg is the issue. If there's a way to make it project-only such that it doesn't interfere with other visual studio projects and only runs when compiling nas2d or ophd that would be great.

Do you mean to say that other projects which don't depend on SDL are being compiled with SDL, and having their main method hijacked?

Do you mean to say that other projects which don't depend on SDL are being compiled with SDL, and having their main method hijacked?

Yes. :)

This might be relevant:
I can't use user-wide integration. Can I use a per-project integration?
Tagging #373 as related.

See #952 update to ReadMe  for workaraounds.


missing company when download PDF

When i want to download PDF or send to customer,some errors happen.
It seems like require company,company.phonenumber,company.email and other.
But when i create the invoice,there is no place to input company.
Am i wrong or you miss the company?

@Catw1thu Can you assign me this ?

hey @Catw1thu can you assign me this issue


Upgrade to make sure crawler still works

Hi there, I think this script not works anymore and we need to update it.
If anyone is ready to contribute and fix it. I welcome.

Maybe you are interested. @nimafire

fixed


radar: ngld support

Based on https://github.com/quisquous/cactbot/pull/739
Updates:

Remove SplitSameMonster config to make it simple to retrieval the mob.
Add a Puller to the config file to record the earliest puller.
Use 19: logs to remove mobs.

TODO:

Resource generator of xivapi

Sorry I forget the lint again.

Thanks for all the changes and patience with my comments. This is looking good.  I'm going to play around with it some this weekend, see if I have any last comments, and will hope to merge it soon.

Thanks for all the changes and patience with my comments. This is looking good. I'm going to play around with it some this weekend, see if I have any last comments, and will hope to merge it soon.

Sure, you can follow a hunt train to test it~

Seemed to work great for me.  I've got a couple more requests, then this is good to go:

Did you forget to include the python script to generate hunt.js?

The Forgiven Pedantry mob did not pop in the radar for me, although everything else did.  I think it's because it's being case sensitive when matching mob names.

Seemed to work great for me. I've got a couple more requests, then this is good to go:

Did you forget to include the python script to generate hunt.js?
The Forgiven Pedantry mob did not pop in the radar for me, although everything else did.  I think it's because it's being case sensitive when matching mob names.

Yeah, I forget to put the script, but it's still using coinach, xivapi version may be implemented later this weekend.

I lowercased the mobnames in the dict though I cannot test it in other servers, hope it works.

I think there's some issues with lowercasing, but I'll follow up myself.


deno: 1.23.4 -> 1.24.0

Description of changes

Bump deno to 1.24.0
In draft until tinycc issues are resolved
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] 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

[ ] 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/)
22.11 Release Notes (or backporting 22.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
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

1.25.0 is out: https://github.com/denoland/deno/releases/tag/v1.25.0

Working on it :) ty

updated and undrafted. no point waiting for tinycc fix since I've put quite a bit of time into it and still not fixed it. Since it's already broken for darwin on unstable we may as well update it

The whole of tinycc didn't actually need fixing for darwin. When building just libtcc.a fixing the VERSION conflict was enough but main tcc is still broken.
I had it working in a VM but It'd be good to also wait for ofborg
cc: @joachifm @AndersonTorres
LMK if you're happy with the tinycc change or if it should be done differently

Hi,
New to Nix here.
I get this is temporary, but FYI this is what I'm getting using the fix in the first comment in my default.nix.
I think this is the tinycc issue you are talking about?
let
    getpkgs = import <nixpkgs>;
    pkgs = getpkgs {};
    # temporary deno fix
    # deno 1.23.3 before the introduction of tcc
    denoImport = import (builtins.fetchTarball {
        url = "https://github.com/NixOS/nixpkgs/archive/e703a434b7210164d8042585509b6e87eb3fadb6.tar.gz";
    }) {};
    denoPkg = pkgs.deno;
in
    pkgs.mkShell {
        buildInputs = [
            denoPkg
        ];
    }

Gives me the:
error: Package ‘tcc-unstable-2022-07-15’ in /nix/store/hjxd6dd89ky8whv01ld0s5s85hgmpkk3-nixpkgs/nixpkgs/pkgs/development/compilers/tinycc/default.nix:76 is marked as broken, refusing to evaluate.

Full output:
https://app.warp.dev/block/qZENfW2QLHPLcFvMVsNEZI

with that nix it's still using the original. Try this
let
    pkgs = import <nixpkgs> {};
    # temporary deno fix
    # deno 1.23.3 before the introduction of tcc
    denoImport = import (builtins.fetchTarball {
        url = "https://github.com/06kellyjac/nixpkgs/archive/817a35be0bfcb9d29146ef89a2d927a315c67f3e.tar.gz";
    }) {};
    denoPkg = denoImport.deno;
in
pkgs.mkShell {
    buildInputs = [
        denoPkg
    ];
}

or
{
    pkgs ? import <nixpkgs> {},
    # temporary deno fix
    # deno 1.23.3 before the introduction of tcc
    denoImport ? import (builtins.fetchTarball {
        url = "https://github.com/06kellyjac/nixpkgs/archive/817a35be0bfcb9d29146ef89a2d927a315c67f3e.tar.gz";
    }) {}
}:
pkgs.mkShell {
    buildInputs = [
        denoImport.deno
    ];
}

the main differences are:

pull deno from the "denoImport" copy of nixpkgs
moved it to point to the new copy that works (this PR) rather than the old copy that worked

@06kellyjac Oh yes of course, thanks for pointing that out!


fix the shellb module, adding support for path

Reason:
Currently the shellb module does not perform as expected.  This change fixes bugs and adds in path support (the module seems to look for the path, so that is added in).
Current behavior:
Name    Required    Value                     Description

Command True                                  Command to execute.
Agent   True        0P0JQPQU                  Agent to execute module on.
(Empire: python/management/osx/shellb) > set Command ls
(Empire: python/management/osx/shellb) > execute
[>] Module is not opsec safe, run? [y/N] y
[!] Exception: generate() takes exactly 1 argument (3 given)
New behavior:
Options:
Name    Required    Value                     Description

Path    True        /                         Path command should execute in
Command True                                  Command to execute.
Agent   True        AFNUWHAD                  Agent to execute module on.
(Empire: python/management/osx/shellb) > set Command ls
(Empire: python/management/osx/shellb) > set Path /root/
(Empire: python/management/osx/shellb) > execute
(Empire: python/management/osx/shellb) >
job 2 started
(Empire: python/management/osx/shellb) > back
(Empire: AFNUWHAD) > jobs
(Empire: AFNUWHAD) >
Desktop
Documents
Downloads
Music
Pictures
Public
src
Templates
Videos

Cleans up commit, splicing array rather than removing elements and accommodating error case where -p is specified but no path or command is entered afterwards.  Additional testing conducted using shellb from agent command line.
test cases:
shellb -p ls
[ ! ] Command or Path parameter value not found
shellb -p /root/ ls
[*] Tasked agent to execute ls in the background on path /root/
[>] Module is not opsec safe, run? [y/N] y
job 0 started
... (time) ..
Desktop
Documents
Downloads
Music
Pictures
Public
src
Templates
Videos
shellb ls
[*] Tasked agent to execute ls in the background on path /
[>] Module is not opsec safe, run? [y/N] y
job 0 started
... (time) ..
bin
boot
cmcgary
dev
etc
home
...

@caleb-sfdcsec The path argument should be thrown out. I'm not sure why that is in there. I don't even think I meant to include this module at all. I'm going to remove it. Thanks anyway.


Generator does not add the required query parameters to methods if the query parameters are arrays of objects

Python version installed
3.12.1
Meraki library version installed
1.45.0
OS Platform
Windows 10
Describe the bug
Ranges query parameter is missing in getOrganizationCameraBoundariesLinesByDevice it must have ranges query parameter which required in this endpoint /organizations/{organizationId}/camera/detections/history/byBoundary/byInterval
Code snippets

But it's working on adding ranges query parameter through postman

Hi @HamedOsama and thank you for this contribution.
I checked the operation docs and it seems the query params are correctly marked as required, however, the generator for the library does not seem to generate the necessary array of objects query parameters. Therefore, there is a bug in the generator, and not with this specific method.
The library is dynamically generated and thus all changes need to start with either the OAS (which is not the issue here) or the generator. If we were to merge this PR, it'd be lost in the next release of the library. So I will close the PR, but keep this issue open.
So, thanks again for this contribution, but if you'd like to see this fixed permanently, please consider submitting a PR to update the generator so that the generator correctly adds the required query params. Aside from generate_library.py, the jinja templates may also need updates.
In the meantime, we'll track this issue here.

Hi @HamedOsama I'm curious if you had a moment to look at a change to the generator that could fix this. No obligation, of course, but I'd be happy to have the help.


bPermission

i try to give fly permission to some jobs and the permission is not take
Architecte:
fullname: Architecte
shortname: Ar
description: Faite de largent en Construisent.