S3mpi versioning?

Another potential solution to the S3mpi accidental overwrite problem is to implement versioning.  It could work like the following:
x <- 'test message'
s3store(x, 'test')
s3read('test')
[1] "test message"

x2 <- 'overwriting test message'
s3store(x2, 'test')   # Overwriting creates a warning (can be turned off) and stores to a separate version.
Warning message:
There is already a file in 'test'.  Saving to 2.

s3read('test')         # Reading reads from the latest save by default.
[1] "overwriting test message"
s3read('test', version = 1)  # By passing a version, we can go back.
[1] "test message"

# We can also get all the version information
s3read('test', version = 'all')
[[1]]
[1] "test message"

[[2]]
[1] "overwriting test message"

# And maybe even offer a revert
s3store(1, 'test', warn = FALSE)
s3store(2, 'test', warn = FALSE)
s3store(3, 'test', warn = FALSE)
s3store(4, 'test', warn = FALSE)
s3read('test')
[1] 4
s3revert('test')  # Revert just goes back one version.
Reverted to 3.
s3read('test')
[1] 3
s3revert('test', to = 1)  # You can also revert to a particular version.
Reverted to 1.
s3read('test')
[1] 1

Thoughts?

Good idea. I think S3 buckets have versioning built in as well, I wonder if there is a way to incorporate that into this package?
http://docs.aws.amazon.com/AmazonS3/latest/dev/Versioning.html

Maybe versioning could be done through objectdiff?


[Feature Request]: Return doc_id along with source_nodes

Feature Description
Currently, when we make ComposableGraph from existing indices, and if we query, we can see the source_nodes, but we need to go through additional steps to see what doc_id that source_nodes belong to. It will be a good idea to return a source_doc_ids to see which document the answer is from.
Reason
No response
Value of Feature
Can clearly acknowledge which document the response has been created from.

for the additional steps, do you mean just calling the appropriate property on the source nodes? If so, I think the usability improvement is small and we probably won't prioritize for now.


assess searchable Json file

@heyarviind,
after checking out #2, you could also please let me know whether we can make the database searchable (the TITLE and AUTHOR field), e.g., using a field that looks like this

duplicated in #11


Some Readings of the Unknown Signs

Hey, first of all, very exciting project,
I had also read your thesis, not the ACM, but the one you had written for the university, i believe on the recognition of hieroglyphs.
I am an egyptologist by formation, and i thought i might help with some identification and if possible with some developpement. I am also working on a feature extractor for Manuel de Codage for applying text learning algorithms to ancient egyptian texts in the future. I think your project complements really well to mine.
Some Identifications:
I don't have time to take a look at all of them. But here are some of the immediate identifications that i could come up with.
090071 is the heka sign: Wb 3, 170.5-21, meaning Wörterbuch der Ägyptischen Sprache of Adolf Erman, volume 3, page 170, etc.
090101 is aHa, ship: Wb 1, 222.4-8
My first guess for 090103 would be: Hm.t: Wb 3, 76.16-77.19
090123 is either Y1 or Htp: Wb 3, 183.4-7
090128 looks like ab:  Wb 1, 173.12-174.1
090166 is a simple H
090168 is probably Ssr: Wb 4, 547
090169 is probably a part of tm: Wb 1, 144.5
090194 consist of two signs, the one on the above is n, below is i
090233 is Xn: Wb 3, 384.2-3
090239 consist of two signs, one royal cartouche and wn:
090299 is a determinative as seen in: Wb 1, 225.15-226.5
090308 is a part of m the bird,:
090311 is probably a part of Hr the head:
090384 is n:  Wb 2, 193.3-194.7

Hi D-K-E,
Thanks for your comments! Seems like we have similar interests, what are you using to extract features from the hieroglyph?
Regarding the incorrect labels, these are part of the automatically detected hieroglyphs, and the labels are computed base on most overlap with the actual labels inside the Manual folder. The images found the Automated folder are generated by using a text-detection method to segment the hieroglyphs in an image, and here is still plenty of room for improving the automatic hieroglyph detection. I would like to add this part to the repository as well in the future, and I'm in contact with the author to make the text-detection algorithm opensource as well.
But in the meantime, if you are interested in creating an improved hieroglyph detection system, I might be able to help you with that.

Hey,
I started writing something for jsesh type inputs, that is the user would give a text encoded in jsesh to the program, and the program would spit out, a dictionary with some features regarding the text. The goal is to be able to exploit those features with ML algorithms afterwards. Though i say jsesh encoded input, it doesn't have to involve java, because jsesh as you might know, works with a special flavour of Manuel De Codage, and a MDC file is just a bunch of special characters plus Gardiner Sign notations.
I am mainly interested in transforming an image to jsesh file, which evidently involves hieroglyph detection. If hieroglyph detection and textual feature extraction can be automated, we could even go for machine translation of texts from there.
Quick question, by text-detection, what do you mean exactly, what type of texts, and what do we detect in texts ? Does it involve hieratic/demotic as well, or is it just transliteration ?


corrected config for raw3-1

Add constratints_limit.yaml (set joint constraints)
Add stomp configuration
Correct arm group in SRDF of raw3-1

This PR is replaced by #758


admin logout then login problems

If you start the client app as admin:
npm run start:admin
This works now - thx!.  If you log out of the app,
two buttons on the right appear: Login  Register
two things:

you probably wouldn't have Register button on an admin site.
if you click on Login, you go to the USER login, which takes you away from the admin.  When you login as admin, you end up on the user profile for admin.

But the login link below the Logged Out in the middle of the screen works.

Thanks a lot for the feedback, it is now fixed in https://github.com/FredericHeem/starhackit/pull/554
Let me know if you find any other issue.

Yes, that works thanks!
Thinking about this more you do want the Register/Signup/Create Account button there.   Register would only work for type=user (admin is hard-coded).  For example, in genetic testing you register (create an account) using a Kit# printed on your sample collection kit.
And based on what "kind" of user was logging in - a general user or admin, you get routed to either /user/profile or /admin/user.  I think I saw in the code "groups".
But right now, if you npm run start:admin, logout, and then login as a general user, alice, the system really crashes.  If you routed the user as they logged in, based on type, then you wouldn't need multiple npm run starts - just one.
Just some thoughts...


Tech writer review for Azure Fairing docs.

Updated the Fairing on Azure docs for clarity and to correct Markdown.

This change is

/hold
To clarify how the secrets are used, I looked up the docs and added additional description. It would be great if someone who knows Azure could comment on the PR before it is merged.

Fixes kubeflow/website #1477 ](https://github.com/kubeflow/website/issues/1477)

/assign
Preview:

https://deploy-preview-1478--competent-brattain-de2d6d.netlify.com/docs/fairing/azure/
https://deploy-preview-1478--competent-brattain-de2d6d.netlify.com/docs/fairing/azure/az-kubeflow-notebook/

@sarahmaddox , Thanks for your review! The updated content is available at:
https://deploy-preview-1478--competent-brattain-de2d6d.netlify.com/docs/fairing/azure/

Thanks @joeliedtke
Since this PR is a tech writing improvement on an original PR from a Microsoft representative, I think we can go ahead and merge it.
@vjrantal if you see any updates that need doing, please do send a followup PR.
/lgtm
/approve

/hold cancel


Use class/function name as range for checks about names/classes/functions

To my surprise many ruff checks highlight the whole class / function, for example D101 (Missing docstring in public class) and D103 (Missing docstring in public function) are currently displayed by the Ruff VS Code extension as follows:

This makes the code quite unreadable and also doesn't let you visually recognize other errors within the class/function. So I'd prefer it to only highlight the class/function name. Looking into it we cannot currently easily fix this in ruff because RustPython's StmtKind doesn't appear to make the Range of function/class names accessible:
pub enum StmtKind<U = ()> {
    FunctionDef {
        name: Ident,
        args: Box<Arguments<U>>,
        body: Vec<Stmt<U>>,
        decorator_list: Vec<Expr<U>>,
        returns: Option<Box<Expr<U>>>,
        type_comment: Option<String>,
    },
    // ...
    ClassDef {
        name: Ident,
        bases: Vec<Expr<U>>,
        keywords: Vec<Keyword<U>>,
        body: Vec<Stmt<U>>,
        decorator_list: Vec<Expr<U>>,
    },

(and Ident is just String as per type Ident = String; defined earlier)
Note that there are several more such checks which should really be using the names as their ranges e.g:

E742 (AmbiguousClassName)
E743 (AmbiguousFunctionName)

We actually have a dedicated method for this (identifier_range), it's just not being used for those checks.

I fixed a bunch of these but please let me know if you run into others.

D417 (Missing argument descriptions in the docstring) also highlights the whole function ... ideally it would only highlight the docstring.

@charliermarsh  PLR0913 (too-many-arguments) also highlights the whole function, it would be nice to only have it highlight the function name. Not sure if this was worth opening a new issue, so I've put it here.

Thanks, will fix!

@charliermarsh PT004 also highlights the whole function, it would be nice if it would highlight only the function name.

Thanks, fixed :)

Thank you for the quick fixes! 🙇🏻


[ButtonGroup]: Displays extra space when size set to "small"

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Current behavior 😯
The ButtonGroup component displays an extra space attached to the button group when the size of the button group is set to small and orientation set to vertical. So, the following...
<ButtonGroup
         size={"small"}
         variant={"contained"}
         orientation={"vertical"}>
    <Button>Left</Button>
    <Button>Center</Button>
    <Button>Right</Button>
 </ButtonGroup>

yields the following with extra space

Expected behavior 🤔
Expect a vertical button group with no bottom space associated to group.
Steps to reproduce 🕹
After setting up a basic react project with typescript and latest version
Steps:

Setup basic project with React
Create basic component (functional or class) with the following code and render it.

<ButtonGroup
         size={"small"}
         variant={"contained"}
         orientation={"vertical"}>
    <Button>Left</Button>
    <Button>Center</Button>
    <Button>Right</Button>
 </ButtonGroup>

Context 🔦
Just trying to make use of a ButtonGroup vertically (with elevation) but no extra spaces attached.
Your environment 🌎

Ubuntu 20.04 LTS
Browsers -> Firefox & Chrome

Could you provide a codesandbox so we can reproduce this problem? I wasn't able to using the steps you wrote. https://codesandbox.io/s/happy-cerf-x1fg0k?file=/src/Demo.tsx renders just fine:

(I'm on Windows, though)

Interesting. Got the correct/expected view on codesandbox. Guess it's my setup that's got an issue (even though I have the same dependency versions as those in the sandbox environment). Will look into my setup more. Thanks for the quick response!


Add Starburst folder and files

Added folder and supporting files for Starburst Enterprise tiles in RHODS.
Description
Added a new folder in "apps-on-prem" to hold the YAML for the Starburst Enterprise tiles.  Added the following YAML files to create Starburst Enterprise tiles:

starburstenterprise-app.yaml --> Tile to display the Starburst Enterprise application
starburstenterprise-quickstart.yaml --> Tile that provides a quickstart for connecting a Jupyter notebook to a Starburst Enterprise cluster
starburstenterprise-docs.yaml --> This file contains YAML for 3 different documentation tiles (installation requirements, overview of Starburst Enterprise on OpenShift deployment guide)

How Has This Been Tested?
Yes, the tiles have been previewed on an OpenShift cluster to ensure they are displaying as expected, that all text is correct and that all links are valid.
Merge criteria:

[ ] The commits are squashed in a cohesive manner and have meaningful messages.
[ ] For commits that came from upstream, [UPSTREAM] has been prepended to the commit message.
[ ] JIRA link(s):
[ ] The Jira story is acked.
[ ] Live build image:
[ ] Testing instructions have been added in the PR body (for PRs involving changes that are not immediately obvious).
[x] The developer has manually tested the changes and verified that the changes work.
[ ] QE contact acknowledges that this has been tested and is approved for merge.

Hi Lucas,
I thought I had removed that file before submitting the PR, but I guess
not.  I am out of town for the next week, so I won’t be able to remove the
file until I get back on the 18th.
Regards,
Ben
On Wed, Apr 12, 2023 at 8:51 AM Lucas Fernandez @.***>
wrote:

@.**** requested changes on this pull request.
On odh-dashboard/apps-on-prem/.DS_Store
https://github.com/red-hat-data-services/odh-deployer/pull/323#discussion_r1164090673
:
@starburst-blumbert https://github.com/starburst-blumbert can you
remove this file? it's an internal macos file and it seems we don't have
that added to gitignore 😅
—
Reply to this email directly, view it on GitHub
https://github.com/red-hat-data-services/odh-deployer/pull/323#pullrequestreview-1381250041,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AQ62EEGGEMFZHF34YGRUZQLXA2QNDANCNFSM6AAAAAAW22KBLU
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
https://www.starburst.io
Ben Lumbert
Partner Solutions Architect
he/him
517-410-7814
@.***
http://bit.ly/3Yt1Dbr


[Bug] [Dao] The data source configuration file does not take effect.

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
The data source configuration file does not take effect
What you expected to happen
The data source configuration file takes effect.
How to reproduce
Start ApiServer, you can see that the data source configuration is not effective
Anything else
No response
Version
dev
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@zhuangchong is this issue already fixed？

@zhuangchong is this issue already fixed？

already fixed.


Add IsUnique.java using string length comparison

Although my naming wasn't as clear as it could have been, my isUniqueUsingHash is the exact same as your proposed function.


Add a page containing download links for folder contents (Use GitHub APIs)

I'm submitting a ...

[x] feature request

Current behaviour:
Script is being downloaded
Expected behaviour:
Page must be loaded which contains the downloading links for all the papers uploaded in the specific folder.
Steps to reproduce:

Get familiar with the way papers are uploaded in the repository, it's quite simple just take a view of the folder structure.
Checkout GitHub Rest APIs or GraphQL APIs, there are APIs for almost anything in GitHub.
Delete the Current PHP Script, as it is not supported by GitHub Pages.
Make an HTML page using the Specific GitHub API and make copies of it in every folder wherever it is required.
Send us a PR for it 😄

Yes, great issue @yashovardhanagrawal . I too thought it was very wrong that scripts are being downloaded. How about we just directly link the PDFs, that way they open in the Chrome Viewer, people can easily download/view it that way.

That's a great idea too
first let's wait for this issue to be solved, then we might open that one too 😄

I am not getting the point. How and where are we trying to execute the PHP  script ?

Got it! I am working on PR.

@MohitDeshwal great, if there's any problem you can ask us anytime!
Also, the PHP script was just there to track the files present in that particular folder. I hope you got that. The script is running perfectly well on the localhost but fails to run on Github Pages. I think they do not support PHP scripts there.
You can find GitHub APIs here: https://developer.github.com/v3/

@MohitDeshwal you PR was merged, please add that download page to other folders as well then we can close this issue


Split bill - Participant display name changes to email when split bill is created offline

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Go to staging.new.expensify,com
Go offline
Create split bill with a few users with display name
Go to + > Split bill > Enter amount > Next

Expected Result:
The display name for users involved in the split bill in Step 3 will remain
Actual Result:
The display name for users involved in the split bill in Step 3 changes to their contact method (email)
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: 1.3.57-0
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
https://github.com/Expensify/App/assets/78819774/effaea9c-1a10-49cb-9153-2814209a2e6d
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub

Will get to this tomorrow.

What's interesting is this seems to only happen to some accounts
See the video. Test User - display name shows. But for the other user (Kevin), the display name is replaced with email.
https://github.com/Expensify/App/assets/14225673/9cedfe24-bf5e-44b6-b77c-70f7736ebbe5

@anmurali @bernhardoj The issue is repeated with avatars as well.
Do we need to open another bug or is this related to this bug?
Thanks
https://github.com/Expensify/App/assets/78819774/face49d2-cc0e-446f-a3a9-79d1b3c0bde6

Issue not reproducible during KI retests. (First week)

https://expensify.slack.com/archives/C049HHMV9SM/p1695659246996439

Cannot reproduce


Add NULL checks where ContentInfo data can be NULL

PKCS12 structures contain PKCS7 ContentInfo fields. These fields are
optional and can be NULL even if the "type" is a valid value. OpenSSL
was not properly accounting for this and a NULL dereference can occur
causing a crash.
CVE-2024-0727

Pushed to master/3.2


sops/keyservice: properly fallback to default

(AMD64) image for testing available at: hiddeco/kustomize-controller:fix-azkv-fallback-dc5486f
Potentially addresses: https://github.com/fluxcd/kustomize-controller/issues/595

Although there is no positive confirmation yet that this solves the issue described in #595, I am going ahead and merge this, as I am confident it addresses at least one variant of the issue.


Adding code for a UWP C# application that uses the Windows SDK SQLite

This is related to the corresponding blog post that is set to be published on 1/27/2017 to walk users through how to use the SDK version of SQLite in their C# application.

Hi @vrk7bp, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;


Exactly same two .vue file, one with many tslint errors while the other not

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.72.0
OS Version: wsl2 ubuntu 20.04

Steps to Reproduce:

Just open App.vue and App copy.vue
enjoy

Thanks for creating this issue! It looks like you may be using an old version of VS Code, the latest stable release is 1.72.1. Please try upgrading to the latest version and checking whether this issue remains.
Happy Coding!

https://github.com/vitejs/vite/discussions/10426

after updating to 1.72.1, now there're tslint errors in both files.

This issue is caused by an extension, please file it with the repository (or contact) the extension has linked in its overview in VS Code or the marketplace for VS Code. See also our issue reporting guidelines. If you don't know which extension is causing the problem, you can run Help: Start extension bisect from the command palette (F1) to help identify the problem extension.
Happy Coding!


Hardware Configuration Manager

This allows for hardware various hardware settings to be chosen, such as where things are plugged in, or which sensors are enabled.

As a bit of an explanation: Several "devices" are defined, such as the serial connection, airspeed sensor, etc. For each device, there is a choice of several configurations. For example serial connection (device 0) can be over VCP (configuration 0), or UART (configurations 1-3). The airspeed sensor (device 1, I think) can be disabled(configuration 0), or connected to the Flexi port (configuration 1).
Configuration settings are saved at the same time as parameters. Currently this is the /param_write service, but I plan on changing the name to reflect that it saves configurations as well.
All devices default to configuration 0, which are/will be defined to "safe" settings.
Configurations are currently only applied at boot. Currently, there are no checks for invalid configurations (such as GNSS and Airspeed both on the Flexi Port), but those could be added at the board level. There are currently no checks when a configuration is chosen that such a configuration even exists. In rosflight_io, I have left some room for adding verbose error messages.

I've implemented requested changes, except for changes to the battery monitor, and changes that weren't practical, to which I responded.

What is the status of this PR?

This change depends on the corresponding mavlink and airbourne PR's, so those need to be merged first.

Hey @BillThePlatypus,
I think I'm struggling to follow the Github review process.
Could you please "Resolve Conversation" if you're responded to our feedback?  If all the conversations are resolved, then I think I'm happy to approve.

This looks like a lot of great work that we should revisit at some point.


publish binary data will close the channel

a quick description
data = ... # some binary data
conn.publish('channel', data) # will close all subscribed channels

# if we do a base64 on it, i works, though the data is still bytes
import base64
data = base64.64encode(data)
conn.publish('channel', data)

Everything within computer memory is a binary data, please provide at least one of the following:

a stack trace showing error (if any);
a simple runnable script to re-create the issue;

I think it was my mistake to mixed some connection and pub/sub, closing util I could reproduce it


Support for steam tokens

First of all, really nice app, well worth the price I paid! Thanks!
So the only issue I've faced till now is that the app does not work with exported Steam Tokens. Apparently Steam uses 5 digit keys or something. I haven't tried it but I am pretty sure it'll work the same way directly from QR codes as well.
So other 2FA apps have a special steam mode or some other mechanism.
I'll link some issues regarding this faced by other apps-
https://github.com/helloworld1/FreeOTPPlus/pull/113
https://github.com/andOTP/andOTP/issues/475
https://gitlab.gnome.org/World/Authenticator/-/issues/115
It'll be really helpful if you could add Steam support, bonus points if I don't have to re-enter the token!

Hi @inprealpha
Thank you for the information, I will try to implement this.

Hi @inprealpha
This project depends on SwiftOTP.
And the bad news is SwiftOTP does not support 5 digits.
https://github.com/lachlanbell/SwiftOTP/blob/0ace411e76ae290fecf6c7d15c6d7c3eee9ccd0c/SwiftOTP/TOTP.swift#L45-L48
Looks like I can't implement it in the short term.

Hi @ososoio
According to this
https://github.com/lachlanbell/SwiftOTP/issues/22
the digit limit is only a soft limit and can be circumvented.

That said, perhaps digits are not the only limitation? According to this
https://github.com/andOTP/andOTP/issues/38 - there are also letters.
It links some tiny implementations, perhaps you could look into it?

Hi @inprealpha
Thanks for the information.
Looks like this Steam case is more complicated than I thought.
There's too much work to do to implement it, I think maybe this app should just focus on the normal TOTP.


Add example Makefile and documentation for building without CMake

Codecov Report

Merging #52 (0c70355) into main (17c18cb) will decrease coverage by 1.79%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #52      +/-   ##
==========================================
- Coverage   96.46%   94.67%   -1.80%
==========================================
  Files          29       13      -16
  Lines        2010      882    -1128
==========================================
- Hits         1939      835    -1104
+ Misses         71       47      -24

Impacted Files
Coverage Δ

RTNeural/ModelT.h
88.42% <0.00%> (-0.90%)
:arrow_down:

RTNeural/activation/activation_xsimd.h

RTNeural/conv1d/conv1d_eigen.tpp

RTNeural/lstm/lstm_eigen.h

RTNeural/activation/activation_eigen.h

RTNeural/lstm/lstm_xsimd.tpp

RTNeural/conv1d/conv1d_eigen.h

RTNeural/gru/gru_xsimd.h

RTNeural/lstm/lstm_xsimd.h

RTNeural/gru/gru_eigen.h

... and 8 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 17c18cb...0c70355. Read the comment docs.


Revert "Revert "[HTTP2] Fix inconsistencies in keepalive configuration (#33428)""

Reverts grpc/grpc#33512

OOO now, but just noting yesterdays request: let's add an experiment for this so that if we inadvertently break anyone we've got a clean rollback path for them.

Closing in favor of https://github.com/grpc/grpc/pull/33586
That adds the experiment


Changing card-header from div to h3 adds margin-bottom

The card header adds .5em margin to the bottom when switched from div to h3, that shouldn't happen.

Fixed by #19048.


Add missing include

See: https://github.com/ros2/rmw_connext/pull/192#issuecomment-231113289

lgtm


Plexus - The message " 9 Locations is not a city in Romania" is displayed

ENVIRONMENT: production
URL: https://scrapers.peviitor.ro
Browser: Microsoft Edge
Device: Laptop
OS: Windows 10
STEPS TO REPRODUCE:

Open the URL in the browser
Search for "Plexus" Company
Check the city of the "Project Manager - Construction" job
Open https://plexus.wd5.myworkdayjobs.com/en-US/Plexus_Careers/details/Program-Manager---Construction_R022818?locationCountry=f2e609fe92974a55a05fc1cdc2852122
Check the job location for the "Project Manager - Construction" job

EXPECTED RESULTS:
The city/location of the job is: Oradea, Romania and  Remote for the following countries: United States, Austria, Hungary, Germany and Ireland, as on the company's website
ACTUAL RESULTS:
The message "9 Locations is not a city in Romania" is displayed on scrapers.peviitor.ro and on solr.peviitor.ro

Fixed!

Scraperul funcționează normal, încearcă sa dai un clear cache la browser.

The number of jobs is not the same in scrapers.peviitor.ro and website Company (72 jobs), as in solr.peviitor.ro and peviitor.ro (68 jobs)

After retesting, the number of jobs is the same on solr.peviitor as on UI scrapers and the company website


Cannot parse vlc.spec

What happened? What is the problem?
With this script:
#!/usr/bin/python3
from specfile import Specfile
import sys

# this is dump, but do the work
filename = sys.argv[1]
specfile = Specfile(filename, force_parse=True)

with specfile.sections() as sections:
    print(sections.changelog)

I cannot parse vlc.spec from Fedora.
It fails with this TB:
Created dummy sources for nonexistent files:
rpm-specs/macros.vlc
Traceback (most recent call last):
  File "/usr/lib/python3.12/site-packages/specfile/spec_parser.py", line 226, in get_rpm_spec
    return rpm.spec(tmp.name, flags)
           ^^^^^^^^^^^^^^^^^^^^^^^^^
ValueError: can't parse specfile

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/usr/lib/python3.12/site-packages/specfile/spec_parser.py", line 294, in _do_parse
    spec = get_rpm_spec(content, rpm.RPMSPEC_ANYARCH | rpm.RPMSPEC_FORCE)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/site-packages/specfile/spec_parser.py", line 228, in get_rpm_spec
    raise RPMException(stderr=stderr) from e
specfile.exceptions.RPMException: /tmp/tmp5e9nmd30: line 51: failed to load macro file rpm-specs/macros.vlc

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/usr/lib/python3.12/site-packages/specfile/spec_parser.py", line 226, in get_rpm_spec
    return rpm.spec(tmp.name, flags)
           ^^^^^^^^^^^^^^^^^^^^^^^^^
ValueError: can't parse specfile

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/home/msuchy/projects/license-validate/./print-spec-changelog.py", line 7, in <module>
    specfile = Specfile(filename, force_parse=True)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/site-packages/specfile/specfile.py", line 72, in __init__
    self._parser.parse(str(self))
  File "/usr/lib/python3.12/site-packages/specfile/spec_parser.py", line 380, in parse
    self.spec, self.tainted = self._do_parse(content, extra_macros)
                              ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib/python3.12/site-packages/specfile/spec_parser.py", line 319, in _do_parse
    spec = get_rpm_spec(
           ^^^^^^^^^^^^^
  File "/usr/lib/python3.12/site-packages/specfile/spec_parser.py", line 228, in get_rpm_spec
    raise RPMException(stderr=stderr) from e
specfile.exceptions.RPMException: file trigger conditions must begin with '/': %{vlc_plugindir}

What did you expect to happen?
Printing changelog of this specfile.
Example URL(s)
No response
Steps to reproduce
No response
Workaround

[ ] There is an existing workaround that can be used until this issue is fixed.

Participation

[ ] I am willing to submit a pull request for this issue. (Packit team is happy to help!)

I don't think there is much that can be done about this - parsing depends on a macro that has to start with a /. For example this will succeed:
spec = Specfile('vlc.spec', force_parse=True, macros=[("vlc_plugindir", "/")])

I think for cases like this it would make sense to make Specfile usable even if rpm parsing failed, as proposed in  https://github.com/packit/specfile/issues/233#issuecomment-1589408369.


Add platform to signup, device register

General comment for the future, I think we should add a header to all requests to the API server with the platform on it, something like X-Keybase-Platform: ios or whatever. Not sure if want to add that to this PR, but I think it would be useful to have.


Infoblox 2.0.10

Change(s):

Update 2.0.9 -> 2.0.10 package

Reason for Change(s):

2.0.9 mainTemplate.json caused grid tables in workbook to appear blank. Reverted workbook ARM code serializedData to 2.0.7

Update outdated text as per publishing pointers

Changed '2.0.9' to '2.0.10' where applicable
Version Updated:

Required only for Detections/Analytic Rule templates
No

Testing Completed:

Tested in closed enviro

Question - In the data connector page how do I show the workbook as related content? it only seems to find the analytic queries

Hi @sschuur, please include this Solutions\Infoblox NIOS\Data\Solution_InfobloxNIOS.json file in the PR. Thanks

Hi @v-rbajaj, updated the version in Solutions\Infoblox Cloud Data Connector\Data\Solution_Infoblox.json

Hi @sschuur, can you please update the API versions in maintemplate, that will fix the failing ARM ttk validation

Hi @v-rbajaj,
Looked through the failure details. Looks like I should update every apiVersion to either  2022-10-01, or 2021-12-01-preview. Is that right? Is one better than the other? Are there any I should not update?

Hi @v-rbajaj, I updated the versions on lines 1028 and 976 in maintemplate.json. I did not see any other /savedsearches with an apiVersion right above or below so hopefully that will work? I don't think I can verify that on my end.

Hi @v-rbajaj, I updated the versions on lines 1028 and 976 in maintemplate.json. I did not see any other /savedsearches with an apiVersion right above or below so hopefully that will work? I don't think I can verify that on my end.

Thanks @sschuur, that error got fixed.
To fix the ARM TTK validation failure we need to remove the "blanks" variable (under variables array) from the maintemplate.json file


Add account registration and activation

Ah jesus spotless broke on the last mile

I really need to align my intellij with spotless for this project


Implement SendEach, SendEachDryRun, SendEachForMulticast,

SendEachForMulticastDryRun

Add SendEach, SendEachDryRun, SendEachForMulticast, SendEachForMulticastDryRun
Deprecate SendAll, SendAllDryRun, SendMulticast, SendMulticastDryRun

SendEach vs SendAll

SendEach sends one HTTP request to V1 Send endpoint for each message in the array. SendAll sends only one HTTP request to V1 Batch Send endpoint to send all messages in the array.
SendEach calls fcmClient.Send to send each message and constructs a SendResponse with the returned message id or error. SendEach uses sync.WaitGroup to execute all fcmClient.Send calls asynchronously and wait for all of them to complete and construct a BatchResponse with all SendResponses. Therefore, unlike SendAll, SendEach does not always returns an error for a total failure. It can also return a BatchResponse with only errors in it.

SendEachForMulticast calls SendEach under the hood.
Will send the integration tests in another PR.

Please make sure that this branch is based off of dev and not mater. In the Go repo, we make all the new changes in the dev branch and only the releases are merged to the main (master) branch. Thank you!


shim support for channelid and mspid

This PR enables the chaincode enclave to retrieve (untrusted) values related to channel id and msp id. These will be needed at enclave creation time for populating the cc parameters structure -- related code will be pushed later.
It partially addresses #404 .

Or otherwise, at least start from the signedproposal which eventually will need to securely implement shim.h

Actually, thinking of it a bit more, i also don't think an ocall to signedproposal makes sense as it should eventually replace the parameters of ecall_cc_invoke.    Which raises the bigger question: shouldn't we in first step re-think the whole .edl in light of what we all will have to change?   Not that we really want to implement everything immediately, but at least have the design plus corresponding comments in the edl) would be imho better done in a more holistic way rather than incrementally in tiny steps?

A few additional thoughts which got triggered by a bike ride on Saturday (and before the heat-wave, so hopefully not based on heat-induced hallucinations :-)

adding GetMSPID/get_mspid to the shim would add a source of non-determinism to chaincodes which is a problem for us, in particular for state encryption once we go beyond designated peer (see #183).   Now we could punt on the programmer to make sure their code is deterministic (or face potentially confidentiality failures) but in that case GetMSPID it is not clear what an app-writer could actually do with GetMSP?  Given that the do not have any use-case which requires that function in a chaincode and there or other "standard" go fabric shim functions which we do not implement, i suggest we do not add it yet to the shim.  Adding when required is easier than removing and not having it might make our lives easier when going to single peer endorsement.
A  GetChannelID/get_channel_id (with ercc/tlcc-validated return value) should not have that problem. Additionally, that seems to have more use-cases, e.g., to bootstrap application-level cross-cc/cross-ledger transactions and so probably seems a good addition to shim.h.

regarding ocall vs ecall, i was wondering also how will we handle sealed state? The easiest approach for designated peer would just return it as response of create_enclave ecall.  That said, once we go to key distribution it gets a bit more complicated. E.g., it raises the question whether we have one or two sealed blobs (my feeling is that probably two separate blobs for enclave and chaincode secrets is cleaner as there is no update involved but just two write-once blobs ..).  So maybe from an abstraction perspective it might be cleaner to have a sealed state module which transparently via ocalls retrieves/stores state? Should also make it easier to add it?

adding GetMSPID/get_mspid to the shim would add a source of non-determinism to chaincodes

First, this comment is about Fabric, rather than FPC.
Second, I disagree. Both for chaincode and FPC-chaincode, as clients (should) know the code, they can determine the circumstances where that API is actually called. Some additional info are here.

Now we could punt on the programmer to make sure their code is deterministic (or face potentially confidentiality failures) but in that case GetMSPID it is not clear what an app-writer could actually do with GetMSP?

We can delegate to the programmer anyway.
Even SGX delegates to a programmer to make sure that the enclave does not write confidential data outside of an enclave (assuming that's a requirement). Hence, it does not prevent to implement an app that voluntarily leaks data.
Similarly for the chaincode, it is easy to implement one that uses getmspid in a such a way that no transaction can ever be committed. That chaincode is useless, but this does not prove that the API is useless too. Also, it's not a question for FPC, it's for Fabric.

Given that the do not have any use-case which requires that function in a chaincode and there or other "standard" go fabric shim functions which we do not implement, i suggest we do not add it yet to the shim.

This is not helpful.
Fabric enables a chaincode to access the local mspid, and we have identified no issue whatsoever to support it in FPC.
Hence, there is no reason to restrict the domain of possible applications by not implementing it.

Adding when required is easier than removing

I disagree. Both adding it (i.e., this PR) and removing it (i.e., essentially the negative of this PR) involve few lines of code.

and not having it might make our lives easier when going to single peer endorsement.

This is not clear.

A  GetChannelID/get_channel_id (with ercc/tlcc-validated return value) should not have that problem. Additionally, that seems to have more use-cases, e.g., to bootstrap application-level cross-cc/cross-ledger transactions and so probably seems a good addition to shim.h.

Again, whether it has more (or less) use cases, it is not up to us to determine.
It's an API of the Fabric shim. Even in this case we have identified no issues whatsoever. Hence, there is no reason to restrict the domain of possible applications by not implementing it.

regarding ocall vs ecall, i was wondering also how will we handle sealed state? The easiest approach for designated peer would just return it as response of create_enclave ecall.

I would have said yes months ago, but I changed my mind after looking at the external launcher.
A tentative approach I am thinking of is: ocall at create, to save the sealed blob; then ecall, immediately at chaincode startup by external launcher, to inject the sealed blob. So the seal is in the invoke path; while the unseal is not.
There are multiple reasons:

it is not useful for the admin to get the sealed blob; besides it should be sent back to the platform when necessary
after createenclave, the enclave is created, and we only need to give it continuity despite failures. For instance, assume createenclave completed at Peer1; if for any reason all chaincodes/Peer1 have to restart, the external launcher can inject the sealed blob on run; most importantly, the next invocation will work seamlessly.

That said, once we go to key distribution it gets a bit more complicated. E.g., it raises the question whether we have one or two sealed blobs (my feeling is that probably two separate blobs for enclave and chaincode secrets is cleaner as there is no update involved but just two write-once blobs ..).  So maybe from an abstraction perspective it might be cleaner to have a sealed state module which transparently via ocalls retrieves/stores state? Should also make it easier to add it?

This aspect can be clearly addressed together with #275 .

adding GetMSPID/get_mspid to the shim would add a source of non-determinism to chaincodes

First, this comment is about Fabric, rather than FPC.

Not really, see below.

Second, I disagree. Both for chaincode and FPC-chaincode, as clients (should) know the code, they can determine the circumstances where that API is actually called. Some additional info are here.

Hmm, the example above doesn't really apply to us either as we do not support (org-public) private data? Also remember that some of that was added together with the change in Fabric v2 which allows to have different code which does not have to be deterministic in the sense of execution, only deterministic in the sense of read/write-sets and response for a given invocation.  So in Fabric v2 you could implement things that an auditor  with private approval policies -- it is one of their samples, although they didn't make it explicit that this requires new v2 feature of different code at different peers nor did they implement it, at least when i looked at it a few months back.  -- could do additional checks.   However, that's something we can't.  And if we do the encryption as right now on the fly then even the sequence of writes has to be the same, not only the read/write-set at the end.  Lastly, note also that for Fabric any accidental non-determines is not necessarily a problem. As long as it is rare, nothing really breaks.  In our case, non-determinism could potentially lead to confidentiality failures.   That said, in an ideal world of course we develop a system which handles arbitrary non-determinism gracefully (and eventually you might have to do that also handle cases like fault-injections). However, minimizing non-determinism and laying potential traps for CC writers to fall into still seems desirable to me.

Now we could punt on the programmer to make sure their code is deterministic (or face potentially confidentiality failures) but in that case GetMSPID it is not clear what an app-writer could actually do with GetMSP?

We can delegate to the programmer anyway.
Even SGX delegates to a programmer to make sure that the enclave does not write confidential data outside of its range (assuming that's a requirement). Hence, it does not prevent to implement an app that voluntarily leaks data.
Similarly for the chaincode, it is easy to implement one that uses getmspid in a such a way that no transaction can ever be committed. That chaincode is useless, but this does not prove that the API is useless too. Also, it's not a question for FPC, it's for Fabric.

Given that the do not have any use-case which requires that function in a chaincode and there or other "standard" go fabric shim functions which we do not implement, i suggest we do not add it yet to the shim.

This is not helpful.
Fabric enables a chaincode to access the local mspid, and we have identified no issue whatsoever to support it in FPC.
Hence, there is no reason to restrict the domain of possible applications by not implementing it.

Hmm, there are quite a few other functions which we also do not implement?  Are you saying we all have to add them now?

Adding when required is easier than removing

I disagree. Both adding it (i.e., this PR) and removing it (i.e., essentially the negative of this PR) involve few lines of code.

It's not about removing the code, it's removing/obsoleting a published api which i see as the potential issue.  (the actual effort to remove (or add) the code to shim.h and exporting it in the library should be minimal?)  I guess you heard Matt ranting about some fabric legacy APIs which are hard to change/get rid off. (I will admit, though, that we do not have the same level of legacy commitments as Fabric and won't have it soon. Although given that we explictly link to shim.h  in the RFC as the interface specification, we would commit to it more bindly when we add it to master. Thinking of which: we probably also shouldn't reference master in the RFC but some specific version, e.g., right now the tech-preview!)

and not having it might make our lives easier when going to single peer endorsement.

This is not clear.

A  GetChannelID/get_channel_id (with ercc/tlcc-validated return value) should not have that problem. Additionally, that seems to have more use-cases, e.g., to bootstrap application-level cross-cc/cross-ledger transactions and so probably seems a good addition to shim.h.

Again, whether it has more (or less) use cases, it is not up to us to determine.
It's an API of the Fabric shim. Even in this case we have identified no issues whatsoever. Hence, there is no reason to restrict the domain of possible applications by not implementing it.

As mentioned above, if you follow your reasoning to the conclusion we have to implement all function?  We already do not implement features from Fabric, certainly not in MVP, so adding or not adding it doesn't really change anything fundamentally from the perspective of allowing or not allowing all apps fabric does?  And given that we can't implement everything right now, choosing exactly the functionality we know we need for concrete use-cases (which i think is true right now: all our shim functions are required by our auction demo) seems to match Occam's Razor?

regarding ocall vs ecall, i was wondering also how will we handle sealed state? The easiest approach for designated peer would just return it as response of create_enclave ecall.

I would have said yes months ago, but I changed my mind after looking at the external launcher.
A tentative approach I am thinking of is: ocall at create, to save the sealed blob; then ecall, immediately at chaincode startup by external launcher, to inject the sealed blob. So the seal is in the invoke path; while the unseal is not.
There are multiple reasons:

it is not useful for the admin to get the sealed blob; besides, it should be sent back to the platform when necessary

Definitely agree. But just to be clear, above i've talked about the create_enclave ecall not the protocol message which flows back to the client. So the ecall could return the protocol message and additionally the response message (similar to the input parameters also necessarily have to be (only) the initial protocol message?

after createenclave, the enclave is created, and we only need to give it continuity despite failures, reboots, etc. For instance, assume createenclave completed at Peer1; if for any reason all chaincodes/Peer1 have to restart, the external launcher can inject the sealed blob on run; most importantly, the next chaincode invocation will work seamlessly.

yep yep, definitely agree, sealed storage has to be handled local to the peer's external build/ecc itself.  I think we are on the same page here.

That said, once we go to key distribution it gets a bit more complicated. E.g., it raises the question whether we have one or two sealed blobs (my feeling is that probably two separate blobs for enclave and chaincode secrets is cleaner as there is no update involved but just two write-once blobs ..).  So maybe from an abstraction perspective it might be cleaner to have a sealed state module which transparently via ocalls retrieves/stores state? Should also make it easier to add it?

This aspect can be clearly addressed together with #275 .

You mean #274 (assuming we do the easier endorsement policy beyond designated peer first?)    But that said, i think there is value already now to have a sealed-state library which handles the sealed state and any related separate edl and ecalls/ocalls.  it automatically provides looser coupling via a clean abstraction, smaller components and  implementation parts which can be concurrently worked on. i think will also make it easier to add after an initial version without sealed state.  Whether we store it as one or two blobs is of course immaterial (and should also be easy to change if we have this sealed storage library/module)

Closing as outdated.
Code might be useful as FPC expands support for the Fabric shim.


Fix rendering of page ranges for chicago/chicago16

Fixes two CSL tests together with typst/hayagriva#125

Thank you!


Error In SINDyLib when Running for loop for Double Pendulum

Hello,
I am currently trying to run the downloadable code for the Double Pendulum. I downloaded and am running the file and get the error
Index in position 2 exceeds array bounds (must not exceed 1).

Error in SINDyLib (line 53)
Data(:,Index)=dX(:,2).^2;

Error in DoublePendulum_Main (line 125)
    [SINDy_Data,SINDy_Struct]=SINDyLib(Data,dData(:,iter),iter,u,Highest_Poly_Order,Highest_Trig_Order,Highest_U_Order,Highest_dPoly_Order);

Which I am finding difficult to work around since the data provided seems to have more than 1 column. Please let me know if there is a simple fix for this issue.
Thanks

Hi,
Thanks for finding this bug! I somehow uploaded a wrong function file and that's why it is not working, the problem is fixed after I uploaded the function file, please download the updated file and have a try.
Thanks!


Pin the kind CLI version

This prevents GitHub rate limiting causing failures in the tests. It
additionally provides a consistent version on each test run to improve the
stability of the tests.
This also makes the run-e2e-tests.sh script fail on HTTP errors.

@dhaiducek could you please review again? It seems that the OS TravisCI is using has an older version of curl so I had to use a different flag.

/cherry-pick release-2.6

/cherry-pick release-2.5

/cherry-pick release-2.5


This description is confusing

This description is confusing
Author: tessaSAC
Slide: Local,Public

Would be nice to make the whole template section interactive


Lost Entities

Hello, First i want to thank you this great job, to be able to control my pool with home assistant for me is great. Since last update (AUGUST 2023) I lost a lot of entities like PH, ORP, salt concentration, temperature. I can only see 4 Relays and 4 Digital inputs. You know why?

That's weird. There was a bug in an earlier version of this addon that messed the entity names, but it was fixed within a day. Are you both sure you are on the latest code? I do have all the sensors available.

This is what it says, since last update:
Logger: homeassistant.loader
Source: custom_components/poolstation/sensor.py:45
Integration: Poolstation (documentation, issues)
First occurred: 11:38:22 (1 occurrences)
Last logged: 11:38:22
Unexpected exception importing platform custom_components.poolstation.sensor
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/loader.py", line 779, in get_platform
cache[full_name] = self._import_platform(platform_name)
File "/usr/src/homeassistant/homeassistant/loader.py", line 796, in _import_platform
return importlib.import_module(f"{self.pkg_path}.{platform_name}")
File "/usr/local/lib/python3.10/importlib/init.py", line 126, in import_module
return _bootstrap._gcd_import(name[level:], package, level)
File "", line 1050, in _gcd_import
File "", line 1027, in _find_and_load
File "", line 1006, in _find_and_load_unlocked
File "", line 688, in _load_unlocked
File "", line 883, in exec_module
File "", line 241, in _call_with_frames_removed
File "/config/custom_components/poolstation/sensor.py", line 45, in
device_class=SensorDeviceClass.PH,
File "/usr/local/lib/python3.10/enum.py", line 437, in getattr
raise AttributeError(name) from None
AttributeError: PH

Logger: homeassistant.setup
Source: setup.py:344
First occurred: 11:38:22 (2 occurrences)
Last logged: 11:38:22
Unable to prepare setup for platform poolstation.sensor: Platform not found (Exception importing custom_components.poolstation.sensor).
Unable to prepare setup for platform poolstation.number: Platform not found (Exception importing custom_components.poolstation.number).

In that case I believe that the reason is that you have an outdated version of home assistant. A new kind of device class for PH sensors has introduced in 2023.08 (I added it to home assistant), and this integration started using a couple weeks after.

I am running 2023.5.3 but don't want to upgrade because other integrations stop working.
Is this possible to install prior versions of your integration ?
It always offers me to download 1ca2ce3

Probably, but first met me check if it’s possible to conditionally use this new feature only if present.

It will be great to preserve backward compatibility.

In that case I believe that the reason is that you have an outdated version of home assistant. A new kind of device class for PH sensors has introduced in 2023.08 (I added it to home assistant), and this integration started using a couple weeks after.

That was right, I updated my home assistant and it works fine.!!! Thanks

@luigixx73 I released a new version that I think will allow the integration to work in versions older than 2023.08, but since I'm in that version I couldn't verify that it works myself. Give it a try.

Hi @cibernox, it is beautifully working now with 2023.5.3
All entities added successfully.
Thanks for the prompt fix!

Great, i'll close this then.


BUG: weird conda/server behaviour?

I'm not sure why, it could be a server related issue. But after switching to the develop version and updating the snakemake-workflows env, i got  the next error multiple times when trying to making the gimme.yaml environment for the chipseq workflow:
_> CondaVerificationError: The package for scipy located at /home/jsmits/anaconda3/envs/snakemake-workflows/pkgs/scipy-1.4.1-py36h921218d_0

appears to be corrupted. The path 'lib/python3.6/site-packages/scipy/pycache/init.cpython-36.pyc'
specified in the package manifest cannot be found.
... etc
_

Since something looked corrupted I removed the snakemake-workflow env and regenerated it fresh and new from the develop .yml file. However when I then tried to run it I ran into a differend error at exactly the same step:
_> CreateCondaEnvironmentException:

Could not create conda environment from /home/jsmits/snakemake-workflows/rules/../envs/gimme.yaml:
Collecting package metadata (repodata.json): ...working... failed

>>>>>>>>>>>>>>>>>>>>>> ERROR REPORT <<<<<<<<<<<<<<<<<<<<<<
Traceback (most recent call last):
  File "/home/jsmits/anaconda3/envs/snakemake-workflows/lib/python3.6/site-packages/conda/gateways/disk/update.py", line 107, in touch
    mkdir_p_sudo_safe(dirpath)
  File "/home/jsmits/anaconda3/envs/snakemake-workflows/lib/python3.6/site-packages/conda/gateways/disk/__init__.py", line 84, in mkdir_p_sudo_safe
    os.mkdir(path)
FileExistsError: [Errno 17] File exists: '/home/jsmits/anaconda3/envs/snakemake-workflows/pkgs'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/jsmits/anaconda3/envs/snakemake-workflows/lib/python3.6/site-packages/conda/core/subdir_data.py", line 189, in _load
    mtime = getmtime(self.cache_path_json)
  File "/home/jsmits/anaconda3/envs/snakemake-workflows/lib/python3.6/site-packages/conda/core/subdir_data.py", line 157, in cache_path_json
    return self.cache_path_base + '.json'
  File "/home/jsmits/anaconda3/envs/snakemake-workflows/lib/python3.6/site-packages/conda/core/subdir_data.py", line 148, in cache_path_base
    create_cache_dir(),
  File "/home/jsmits/anaconda3/envs/snakemake-workflows/lib/python3.6/site-packages/conda/core/subdir_data.py", line 649, in create_cache_dir
    cache_dir = join(PackageCacheData.first_writable(context.pkgs_dirs).pkgs_dir, 'cache')
  File "/home/jsmits/anaconda3/envs/snakemake-workflows/lib/python3.6/site-packages/conda/core/package_cache_data.py", line 160, in first_writable
    created = create_package_cache_directory(package_cache.pkgs_dir)
  File "/home/jsmits/anaconda3/envs/snakemake-workflows/lib/python3.6/site-packages/conda/gateways/disk/create.py", line 435, in create_package_cache_directory
    touch(join(pkgs_dir, PACKAGE_CACHE_MAGIC_FILE), mkdir=True, sudo_safe=sudo_safe)
  File "/home/jsmits/anaconda3/envs/snakemake-workflows/lib/python3.6/site-packages/conda/gateways/disk/update.py", line 125, in touch
    raise NotWritableError(path, e.errno, caused_by=e)
conda.exceptions.NotWritableError: The current user does not have write permissions to a required path.
  path: /home/jsmits/anaconda3/envs/snakemake-workflows/pkgs/urls.txt
  uid: 14752
  gid: 14752

If you feel that permissions on this path are set incorrectly, you can manually
change them by executing

  $ sudo chown 14752:14752 /home/jsmits/anaconda3/envs/snakemake-workflows/pkgs/urls.txt

In general, it's not advisable to use 'sudo conda'.

During handling of the above exception, another exception occurred:
etc..._

Which was very weird, i tried multiple things, and eventually updating the version of python in the environment seemed to have worked
_

The following packages will be UPDATED:
python                pkgs/main::python-3.6.10-h191fe78_1 --> conda-forge::python-3.6.10-h9d8adfe_1009_cpython
readline               pkgs/main::readline-7.0-h7b6447c_5 --> conda-forge::readline-8.0-hf8c457e_0
The following packages will be SUPERSEDED by a higher-priority channel:
ncurses                 pkgs/main::ncurses-6.2-he6710b0_0 --> conda-forge::ncurses-6.1-hf484d3e_1002_

_
Now it succesfully generated all conda environments and is running the pipeline. However I'm not sure if it is a weird server flockup or if the updating of python really did something and that is the thing that fixed it. Instead of forgetting this while somebody else might run into it, I thought I'd make it an isue so you guys also know if something similar pops up with somebody else.
If an issue is not the right place for this kind of thing or not needed, just let me know!
Gr Jos

Unfortunately this is a conda-related thing. I am not sure how this happens, but i had it before myself as well. I think (hope) you were just really unlucky to get these rare errors twice in a row.

Alright then I'll close it :)

Just for future reference:

removing the .snakemake directory should solve this issue.
if that does not resolve it try removing both the snakemake-workflows environment and again the .snakemake directory.


Added CRONTAB_MODULE setting.

Allows to keep cronjobs apart from other dramatiq tasks.

Hi @m000m,
Thanks for reaching out. Looks like you want to modify the task loading process.
The autoloading is a convenience feature, which will work for many but not all users.
I can imagine a couple of ways to extend this functionality:

Provide an exploit list of tasks or modules as a command line argument.
Provide a Python path to a task loader function (setting or CLI).
Provide a module name in which we expect cron tasks per Django app.

Honestly, I would probably prefer the second solution. As it keeps complexity inside the package low, while providing the flexibility to developers to implement whatever suits their specific case.
You are always welcome to open another PR. We love to see other people contribute to our projects!
Thanks,
Joe


scripts: Don’t remove node_modules from subdirectories of presets in e2e tests

Summary
After yarn and yarn jest the result of yarn clean-all is that git status displays:
deleted:    e2e/presets/js/node_modules/jest-preset-js/jest-preset.js
deleted:    e2e/presets/js/node_modules/jest-preset-js/mapper.js
deleted:    e2e/presets/json/node_modules/jest-preset-json/jest-preset.json
deleted:    e2e/presets/json/node_modules/jest-preset-json/mapper.js

#6185 added these files and made an exception in .gitignore but not in package.json file.
Y’all are welcome to improve the find command in new clean-e2e script. I wrote it as clear and cross platform as I know how.
Test plan
I used find . -type d \( -name node_modules -prune \) -print to explore node_modules directories:

After git clone there are 2 under ./e2e and 4 under ./packages
./e2e/presets/js/node_modules
./e2e/presets/json/node_modules
./packages/jest-resolve/src/__mocks__/bar/node_modules
./packages/jest-resolve/src/__mocks__/foo/node_modules
./packages/jest-resolve-dependencies/src/__tests__/__fixtures__/node_modules
./packages/jest-runtime/src/__tests__/test_root/node_modules

After yarn there are more, of course

./node_modules

./website/node_modules

12 under ./examples

21 under .packages including the following 4 related to the preceding 4
./packages/jest-resolve/build/__mocks__/bar/node_modules
./packages/jest-resolve/build/__mocks__/foo/node_modules
./packages/jest-resolve-dependencies/build/__tests__/__fixtures__/node_modules
./packages/jest-runtime/build/__tests__/test_root/node_modules

After yarn jest there are 8 more under ./e2e
./e2e/babel-plugin-jest-hoist/node_modules
./e2e/coverage-remapping/node_modules
./e2e/coverage-transform-instrumented/node_modules
./e2e/native-async-mock/node_modules
./e2e/stack-trace-source-maps/node_modules
./e2e/transform/babel-jest/node_modules
./e2e/transform/multiple-transformers/node_modules
./e2e/typescript-coverage/node_modules

After original version of yarn clean-all

removed ./node_modules

removed 17 under ./packages and kept the 4 from git clone

removed 4 under ./e2e including the 2 from git clone that it should have kept but kept the following 6 the it should have removed (but were not because of ./e2e/*/*/node_modules pattern)
./e2e/babel-plugin-jest-hoist/node_modules
./e2e/coverage-remapping/node_modules
./e2e/coverage-transform-instrumented/node_modules
./e2e/native-async-mock/node_modules
./e2e/stack-trace-source-maps/node_modules
./e2e/typescript-coverage/node_modules

Repeat steps 1, 2, 3, and then after improved version of yarn clean-all

removed ./node_modules
removed 17 under ./packages and kept the 4 from git clone
removed 8 under ./e2e and kept the 2 from git clone therefore git status displays nothing to commit, working tree clean

Hey @pedrottimark! 👋
I wonder if it might be easier to just write a node script, and execute that before running rm -rf ./node_modules/? Although if it works it's probably good enough 🙂 Thoughts?


Add/fix some holidays

Try to initially add/fix holidays like "halloween" & "black friday".

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: wlifuture sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Lebovits/issu946 add backup vacant land data

This PR adds a check to the pipeline to confirm that the vacant lots dataset contains at least 20,000 observations. If it does not, it defaults to the June 2024 vacant lots dataset provided by CityGeo (the last known accurate version of the dataset) and loads it from our GCS bucket.
Also a small fix: I added the Google Cloud Project explicitly to the Docker env.

Going to test the full pipeline locally to confirm everything looks okay and then will merge.


QR code recovery or more than once use.

The issue was already discussed, but was closed because of: "feature decision".
There must be a way to scan your QR code a second time.
Either through some special PIN associated with the QR or through the users phone number: two steps authentication.
Why again?

The user can easily lose access to his app for various reasons..
QR code are usually not a one time use tool.
Users who deleted their test for various reasons.

It should be decided how to proceed here, either answer the question, close the issue as answered (in the comments above) or transform it to a feature request and move to the wishlist.


[Improve][Catalog] Make Catalog support table change operations

Search before asking

[X] I had searched in the feature and found no similar feature requirement.

Description
This is a subtask of https://github.com/apache/incubator-seatunnel/issues/4050 to track completion.
Make org.apache.seatunnel.api.table.catalog.Catalog support various change operations of the CatalogTable.
And improve the MySQL Catalog.
Related pseudo-code
// create, get, update catalog
public interface Catalog {
    // query
    List<String> listDatabases() throws CatalogException;
    List<String> listTables(String databaseName) throws CatalogException, DatabaseNotExistException;
    // get catalog table
    CatalogTable getTable(TablePath tablePath) throws CatalogException, TableNotExistException;
    // update catalog table
    void createTable(TablePath tablePath, CatalogTable table, boolean ignoreIfExists) throws TableAlreadyExistException, DatabaseNotExistException, CatalogException;
    void dropTable(TablePath tablePath, boolean ignoreIfNotExists)
            throws TableNotExistException, CatalogException;
}

Usage Scenario

Validate the table structure
Automatic table creation
Drop table
Clear table data

Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@ashulin  I'm not sure how to implement this issue for now, but I'm interested in cdc and would like to find an opportunity to participate in it and take part of the work.Do you have any good suggestions?

please assign to me; thx。

please assign to me; thx。

I am working on this issue :).


Notify/Listen for Documents and Events

For document insert/update/delete as well as for event source append events, use the postgresql notify/listen feature to allow for notifications. This would allow some interesting scenarios such as listening for event notifications and then publishing to a bus. This could then be expanded to fulfill more reactive style scenarios for pub/sub notifications when documents are created/modified/deleted and/or events are appended.
This might be as easy as modifying the functions for document upsert/delete to add a call to the notify function. I see that @jeremydmiller may already be thinking along similar lines for events as well:
https://github.com/JasperFx/marten/blob/master/src/Marten/Schema/SQL/mt_rolling_buffer.sql#L113
Open for dicussion would be how to configure this. Always on by default, turned all for all document types, configurable on a per document type basis, etc.? Similar questions related to appending events as well.

This would be great for updating user interfaces real-time based on read models! And it also can be used to be used implement a catch-up subscription..
I tend to lean to an opt-in approach for this to avoid overhead, but I'm not really aware of the cost of using pg_notify. It might be not worth it.
Possible config:
DocumentStore.For(options => {
  option.Schema.For<User>().WithNotifications()
  option.Schema.For<User>().WithNotifications(for => for.Inserts())
  option.Events.WithNotifications();
});

For the client side I was thinking of something like this:
using (var session = store.NotificationSession()) {

  session.ListenToAll<User>(handlerMethod);
  session.ListenToUpdates<User>(handlerMethod);
  session.ListenToInserts<User>(handlerMethod);
  session.ListenToDeletes<User>(handlerMethod);

  session.ListenToAllEvents(handlerMethod);
  session.ListenToEvent<UserCreated>(handlerMethod);

}

But other approaches might be worth looking into. Like listener classes that implement specific interfaces: IListenToDocumentUpdates<User>, IListenToAllEvents, ...
One other thing we need to take into account is connection recovery.

My consideration of pg_notify is related to trying to avoid publishing phantom messages/events or experiencing dropped notifications. If you put the notifications into Marten itself, you're going to have the possibility of either one or the other of these two nefarious scenarios:

you publish before you persist, and the persist fails, leading to a phantom notification
you persist before you publish, and the publish fails, leading to a dropped notification

Of course pg_notify doesn't necessarily perfect deal with item 3 either, but I was thinking it could at least help address issue 1.
I'm not an expert on pg_notify by any means, but I think Marten would benefit from some notion of adopting a reactive approach.

@jeffdoolittle @tim-cools My thought with the event store is to only really use pg_notify as a "Claim Check" that some kind of data is available, but really depend on the rolling buffer table as a de facto queueing mechanism.
In this case, the rolling buffer table is the real thing, and pg_notify would just be used to tell a listener that there's some new stuff available in the queue. Mostly just to try to optimize the polling.
I don't know that I completely trust pg_notify for any kind of high volume activity or anything that requires guaranteed delivery. I thought it was flakey when I was using it against older versions of Postgres a couple years ago.

so basically, subscribe and tell me if if something has changed, but in case I miss anything, I can still ask for anything after a specific version/sequence number?

You'd subscribe through a queueing mechanism maybe, but yeah.
What kinds of things are you wanting to do? This is specifically to subscribe for notifications outside of the original TX, right?

I was thinking to push updates of my view models by listening to marten and then publishing them through web-sockets to the web client . If you could subscribe to this you don't need an external bus to send these messages. This can be useful in case you have multiple web-servers, with only one server you can send these updates from the software itself
I had only visualization in mind. And not intended to use this for logic that need consistency. If I send view model updates on a bus I don't use persistent messaging anyway...

I'm thinking about client updates in near-real-time like firebase and meteor

Should this really be implemented in Marten? You can already opt in to it like this right?
public class MartenEventPublisher : DocumentSessionListenerBase
{
    private readonly IEventBus _eventBus;

    public MartenEventPublisher(IEventBus eventBus)
    {
        _eventBus = eventBus;
    }

    public override async Task AfterCommitAsync(IDocumentSession session, CancellationToken token)
    {
        var last = session.LastCommit;

        foreach (var @event in last.GetEvents())
        {
            await _eventBus.PublishAsync((dynamic)@event.Data).ConfigureAwait(false);
        }
    }
}

The IEventBus can be anything you want right?
The only thing missing right now, is that the IEvent version does not get properly updated after the insert and a global version/sequence number is missing.
In your services you could query of the Event store yourself, based on version numbers and catch up if needed.
Or I am missing something important?

The feedback from 2nd Quadrant was that notify/listen is really only useful to prompt other processes to "wake up." There's absolutely no guaranteed delivery and it doesn't guarantee the order.
For the event store, the current theory is that we'll use sequential Guid's for the event Id, then just use that to "know" where to start catching up in the async daemon. If that works out, it becomes pretty easy to poll for new events, but we could still use pg_notify() in the event capture as just a "wake up, there are events ready for you" coat check kind of message.

sounds very reasonable to me. basically, notify/listen may be nice-to-have but certainly not required nor particularly reliable for anything except "hey, you may want to check for more stuff."

Good to know! Sounds valuable feedback!

Hey everyone, just for the sake of pruning the GH list, I'm going to close this one for now. You could do this on your own in a document listener and I probably will add this to the event store as part of the async daemon.


Use dataLayer.events to store adblock information

Adblock checker loads before Google Analytics so in order to send the event, store it for later in lp.analytics.dataLayer.events.

👍


chore: upgrade storybook to 6.1

Summary:
Pulls out @dcwither's commit from #224: https://github.com/chanzuckerberg/lp-design-system/pull/224/commits/3dae9d81a9813e48e0c38d201e499995d82c66c9
We want these upgrades regardless of what CSS-in-JS framework used
Test Plan:

build passes
npm start works
note: looks like percy changes are from storybook default padding changing

thank ya!


process.getuid is not a function

I have a problem when i run kotatsu command
return process.getuid is not a function
i'm using windows ;/ with node v4.3.1

Hello @armandomendivil. It's because process.getuid doesn't exist on windows :). Let me fix this for you.

@armandomendivil I published 0.12.1 on npm with a fix. Can you tell me whether this fixes your issue please? I have currently no way of testing on windows to assert the fix worked.

@Yomguithereal  Thank you so much! but when i try to run kotatsu command return error:
[kotatsu] (v0.12.1)
[kotatsu] Compiling...
[kotatsu] Done!
[kotatsu] Starting your script...
module.js:327
throw err;
^
Error: Cannot find module 'C:UsersarmandoAppDataRoaming
pm
ode_moduleskotatsu
ode_modulessource-map-supportsource-map-support.js'
at Function.Module._resolveFilename (module.js:325:15)
at Function.Module._load (module.js:276:25)
at Module.require (module.js:353:17)
at require (internal/module.js:12:17)
at Object. (C:\dev\webpack\kotatsu.kotatsu\bundle.js:1:63)
at Module._compile (module.js:409:26)
at Object.Module._extensions..js (module.js:416:10)
at Module.load (module.js:343:32)
at Function.Module._load (module.js:300:12)
at Function.Module.runMain (module.js:441:10)
[kotatsu] The script crashed. Waiting for changes to reload...
I have tried many solutions I've looked on internet, but none has helped me :(

@armandomendivil I know where the problem comes from. It's also a windows specific quirk. Until I fix this, you can just disable source maps through --no-source-maps and it should work.

Thanks a lot! it works now! one thing, in your express example you need to add module.exports = app; =) Thanks again! this is very helpful!

I opened #123 to tackle the other bug you are having relating to source maps.

Hello @armandomendivil. Just releases 0.12.2. This should let you use source maps on windows normally.


Materials lost when transforming a SnapshotDb

Copied from https://github.com/iTwin/itwinjs-core/issues/5307, original issue posted by @fatalstar
Describe the bug
I try to transform a SnapshotDb to another, but materials lost.
And then, I try to transform it to an empty SnapshotDb，materials get lost and error.
To Reproduce
Steps to reproduce the behavior:

Convert House_Model.dgn to a local house.bim file. And do same thing with testY1.dgn.
Build an iModelTransformer with<EMAIL_ADDRESS>Excute iModelTransformer.processAll( house.bim, testY1.bim, no options),
When transform done, excute targetDb.saveChanges( )
Open the targetDb in view, and could see the materials lost.

Expected behavior
Materials exist and be correct.
Screenshots
The house.bim view :

The result.bim view: (materials get lost)

If transform to an empty db, and view: (roof material lost, wall and steps meterial error)

Desktop (please complete the applicable information):

iTwin.js Version [3.4.4]
iTwin.js Version [3.6.3] also tested

I have confirmed that this is because several json properties are not tracked by the transformer. I have added a tool to the test-app to detect them, ran it on the house_model, and am now able to find several. Can use it as a starting point to add better support for tracking known json properties in the transformer.


nix flake check does not work

I have the following error if I try to run the nix flake check command:
nix flake check .

warning: creating lock file '/Users/chrishrb/dev/home/bla/flake.lock'
warning: unknown flake output 'utils'
warning: unknown flake output 'dependencyOverlays'
warning: unknown flake output 'categoryDefinitions'
warning: unknown flake output 'packageDefinitions'
warning: unknown flake output 'baseBuilder'
warning: unknown flake output 'customPackager'
error:
       … while checking flake output 'overlays'

         at /nix/store/na7sykizsgkzh9i3wc8m8pz5xfqib2rv-source/lib.nix:39:17:

           38|               {
           39|                 ${key} = (attrs.${key} or { })
             |                 ^
           40|                   // { ${system} = ret.${key}; };

       … while checking the overlay 'overlays.x86_64-linux'

         at /nix/store/na7sykizsgkzh9i3wc8m8pz5xfqib2rv-source/lib.nix:40:24:

           39|                 ${key} = (attrs.${key} or { })
           40|                   // { ${system} = ret.${key}; };
             |                        ^
           41|               }

       error: overlay is not a function, but a set instead

To reproduce:
mkdir nix-cats && cd nix-cats
nix flake init -t github:BirdeeHub/nixCats-nvim
nix flake check .

Hmmmm....
So, it appears this is because the overlays from nixCats are at overlays.${system}.packageName
I will need to look into this, but it's possible that this is simply due to the need to use flake-utils.eachSystem?
I actually did not know about this command

hmmmm Yep its because the overlays are output from within flake-utils.
The overlays just contain the final package. they are at overlays.${system}.packagename instead of at overlays.packagename
I can rework that output. I am not sure many people are using it, as its usually easier to just pass the package where you want yourself. Seems worthwile, hopefully I can do it in a way that isnt messy.

this is currently how the overlays output works. Very simple.
and where it gets exported
the issue is, these functions assume you already told the builder what system it was building for....
So it gets output behind flake-utils so that its outputs.overlays.${pkgs.system}.packagename
So to change it, I would need to do a little bit of messing around.
I dont want to break anything other than the overlays output when I do so, so that people can just put the 1 new line into their flake.nix for the overlays output and be done with it when i make the change

``nix
# choose your package for devShell
# and add whatever else you want in it.
devShells = {
default = pkgs.mkShell {
name = defaultPackageName;
packages = [ (nixCatsBuilder defaultPackageName) ];
inputsFrom = [ ];
shellHook = ''
'';
};
};

I also need to change the devShells output to the above. because the current one is deprecated, fyi

https://github.com/BirdeeHub/nixCats-nvim/tree/overlaysCheckFix
I have a working makeOverlays function and a (maybe working?) depreciation warning for the old one in this branch.
Still need to update the help and stuff, and test that I actually did the depreciation warning correctly before I merge it, but the main flake and the templates are all updated and work with nix flake check . in that branch.

Lmao I think I made someone mad cause someone unstarred. That's ok. It's a better project now that that issue is fixed.


Target a total in rf_impute

rf_impute should take an argument for a target total (e.g. an administrative total) to hit. It can achieve this target by adjusting the quantiles to select. Currently, it samples randomly from the conditional distribution, effectively using the uniform distribution to select quantiles. To achieve the target while still selecting realistic values (those in the conditional distribution), it can instead adjust the distribution of quantiles to select, by using something other than the uniform distribution.

Fixed in #45


Fix docs format

A companion PR to https://github.com/ros-controls/control.ros.org/pull/82

@mergifyio backport humble


Binance signature data api

Binance signature data
chainId  accountNumber  sequence
https://dex.binance.org/api/v1/node-info get network
https://dex.binance.org/api/v1/account/ get accountNumber   sequence

covered here: https://developer.trustwallet.com/wallet-core/integration-guide/wallet-core-usage#binance-chain-bnb-transaction-signing


Re-think handlers.select option.

handlers.select(e, {selectOnClick: true})
That's weird.

Should it be just (e) like other handlers?
I suppose by "weird" you mean these are possible:
onClick={(e) => select(e, { selectOnClick: false })
onHover={(e) => select(e, { selectOnClick: true })


Attempt coverage.

Codecov Report

:exclamation: No coverage uploaded for pull request base (stable/v8.x@3165101). Click here to learn what that means.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@              Coverage Diff               @@
##             stable/v8.x       #2   +/-   ##
==============================================
  Coverage               ?   56.99%
==============================================
  Files                  ?      121
  Lines                  ?    10589
  Branches               ?     1038
==============================================
  Hits                   ?     6035
  Misses                 ?     4216
  Partials               ?      338

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Add generated API docs for grpc-dotnet

They're generated using the scripts in https://github.com/grpc/grpc-dotnet/pull/531,
docs are added to a new "csharp-dotnet" directory (same directory name used in grpc.io site).
A new link will be added to https://grpc.io/docs/reference/ afterwards.

CC @shirhatti @JamesNK @JunTaoLuo

The merged API docs is here: https://grpc.github.io/grpc/csharp-dotnet/api/Grpc.Core.html,
I still need to create a PR to link the API docs from the grpc.io menu: https://grpc.io/docs/reference/


Android NDK C++Analysis

How should I analyze Android ndk cmake based projects，thank u.pls

I would like to contribute on this issue please assign me this issue @chongbo2013

you may follow the steps to analyze ndk cmake based projects
#Project Design:
Begin by understanding the task's registry structure. Normal parts incorporate source code, CMakeLists.txt documents, header records, and assemble scripts.
Search for the high level CMakeLists.txt document, which fills in as the passage point for the CMake construct process.
#Conditions:
Distinguish the outer libraries and conditions utilized in the venture. These may incorporate outsider libraries, framework libraries, and other NDK parts.
Check in the event that the venture utilizes CMake's find_package or target_link_libraries orders to oversee conditions.
Construct Framework:
Figure out which Android construct framework is being utilized. Android tasks can utilize either Gradle or CMake as their construct framework. Assuming that it's CMake-based, you'll see CMakeLists.txt records all through the task.
#CMake Arrangement:
Assess the CMakeLists.txt documents, beginning with the high level one. This document characterizes the undertaking settings, including compiler banners, source records, and target properties.
Audit any custom CMake macros or capabilities utilized inside the task.
Android.mk Documents (if present):
Some NDK undertakings might in any case incorporate Android.mk records for heritage support. These records portray how to assemble local code. On the off chance that they exist, concentrate on them to comprehend how they add to the form cycle.
Source Code:
Look at the source code documents, particularly the local C/C++ code. Comprehend the reason and usefulness of each source document.
Focus on any JNI (Java Local Point of interaction) code assuming that the venture incorporates local code with Java code.
Construct Cycle:
Figure out how the undertaking is assembled. Regularly, CMake creates fabricate scripts for the chose assemble framework (e.g., Ninja, Makefiles).
Find out more about the form targets and how they are arranged in the CMakeLists.txt documents.
Aggregation Banners and Choices:
Check for compiler banners and choices determined in the CMakeLists.txt documents. These banners can influence code streamlining, troubleshooting, and similarity.
Troubleshooting and Logging:
Comprehend how the task handles troubleshooting and signing in local code. Search for troubleshoot images and logging explanations.
Testing:
On the off chance that the task incorporates unit tests or joining tests, investigate how they are designed and run. CMake frequently upholds testing systems like Google Test.
IDE Combination:
Assuming you're utilizing a coordinated improvement climate (IDE) like Android Studio or Visual Studio with the Android NDK module, ensure the task is set up for consistent turn of events and troubleshooting.
Adaptation Control:
Check assuming the undertaking utilizes variant control frameworks like Git. Understanding the commit history can give experiences into project improvement.
Documentation:
Search for project documentation, README records, or remarks inside the codebase. These assets can give significant setting and utilization guidelines.
Assemble Variations:
In the event that the task has numerous form variations (e.g., discharge, troubleshoot, various designs), comprehend how to switch among them and their separate arrangements.
Construct and Run:
At last, form and run the undertaking to guarantee it capabilities true to form. Address any form or runtime issues as they emerge.


Enable the triage owner rotations tool

Checklist

[ ] Type annotations added to new functions
[ ] Docs added to functions touched in main classes
[ ] Dry-run produced the expected results
[ ] The to-be-announced tag added if this is worth announcing

LGTM, let's merge this after we fix #1174 and after we update docs

The firefox-source-doc are updated now. We can merge this once we fix #1562.


partial dependence should have X optional

if X is None, then generate a grid of 200 X-values via edge knots

ok, NO!
i think it'll be cleaner if i just create a function to generate a X grid.
because you will still want the X grid for the prediction method, confidence intervals....


Windows: Service intermittently stops

We've had 3 occurrences over the last 6 months where the service stops by itself, there are currently no logs to indicate what the cause is other than in the system event log:
The monitoring-agent service terminated unexpectedly. It has done this 3 time(s).

Release https://github.com/infraweavers/monitoring-agent/releases/tag/0.1.792 adds extra stderr logging to investigate this

Just occurred, output.log.stdout is empty, normal output.log contains as the last thing:
ma 2022/08/05 09:45:18 WARNING : Request Timed Out. Timeout: '10s'; Command: 'C:\Windows\System32\WindowsPowerShell\v1.0\powershell.exe'; Arguments: []string{"-command", "-"}

The https://github.com/infraweavers/monitoring-agent/blob/master/main.go#L107 log isn't logged; so whatever process termination is occurring, isn't "just main ending".

Just occurred on Debian 9 as well when the box was under significant IO pressure. I'm pretty sure this is the same problem as a few of the instances where we have seen this occur are when disks have failed in RAID arrays etc causing IO stalls.
$:/var/log/monitoring-agent$ cat monitoring-agent.log.stdout
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x0 pc=0x4d2db9]

goroutine 2691924 [running]:
os.(*Process).signal(0x0, {0x8d18a0, 0xb16600})
        /opt/hostedtoolcache/go/1.17.12/x64/src/os/exec_unix.go:64 +0x39
os.(*Process).Signal(...)
        /opt/hostedtoolcache/go/1.17.12/x64/src/os/exec.go:138
os.(*Process).kill(...)
        /opt/hostedtoolcache/go/1.17.12/x64/src/os/exec_posix.go:68
os.(*Process).Kill(...)
        /opt/hostedtoolcache/go/1.17.12/x64/src/os/exec.go:123
monitoringagent/internal/web.runScriptWithStdIn.func1()
        /home/runner/work/monitoring-agent/monitoring-agent/go/src/monitoring-agent/internal/web/helpers.go:163 +0x5f
created by time.goFunc
        /opt/hostedtoolcache/go/1.17.12/x64/src/time/sleep.go:180 +0x31

Fixed in: https://github.com/infraweavers/monitoring-agent/pull/22

Fixed in release 0.1.800


ltemplify.py typo in line 4823

        if len(self.l_data_improper_coeffs) > 0:
            l_data_improper_coeffs.insert(
                0, (' ' * self.cindent) + 'write_once(\"' + data_improper_coeffs + '\") {')

should be
        if len(self.l_data_improper_coeffs) > 0:
            self.l_data_improper_coeffs.insert(
                0, (' ' * self.cindent) + 'write_once(\"' + data_improper_coeffs + '\") {')

Great catch!
I'm happy to implement the change today.
But if you want to make your contribution official, I'm happy to accept a pull request from you with this change.  I actually prefer to do this.  If I don't hear back from you by tomorrow, I'll make the change myself.  (We can retroactively do a pull request later if you don't get this message in time.)
Cheers and thanks.

...Actually, I changed my mind.
Your bug report caused me to notice something else that was also wrong.  (Thanks!)  I'm worried I might forget to fix this if I don't do it today.  So I'm patching the issue now.
Again, we can do a pull-request retroactively.  I would like to do this if you have time.  (Just add a blank line to the end of the most recent version of "ltemplify.py", issue a pull-request, and I will approve the pull request and add a comment to the commit explaining the real contribution that you made and a link to the relevant commit.)

Again, if you issue a retroactive pull request, I will approve it.  (See instructions above.) This helps me keep track of who the contributors are.
Thanks again!
closing this issue.

Hi,
Thanks for the offer! I'll take you up on that and add an empty line to the end of the file. Glad to help.


added streaming remote read client side functionality

For #5926
Thank you to @YaoZengzeng for his initial design. I've implemented his comments from here.
FYI @bwplotka

Sorry I didn't try @wwformat suggestion before reaching you on slack because I thought using &prompb.ChunkedSeries is right as no objections about it were raised on my previous PR. But now I think I should have done it. Sorry!

Actively working on the fix and some issues with TestEndpoints. Will try to get the revision within a week. :-)

Can you also resolve comments which are addressed to keep this PR discussion clear?

Tests are definitely failing because of this PR. I can see the endpoint tests are failing with context.Cancelled. Maybe we close connection sooner than we should?

This has been idle for a couple of months now, and @wwformat wants to continue work on it in #8351 - I suggest we close this and continue there?

We have looked at this pull request during our bug scrub.
This is now a duplicate of #11379.
Thank you for your contribution.


Rename SUIT-specific test fixtures

If SUIT is no longer the only pattern, the filenames for these fixtures are too generic.

Closed by 96ae59ba1caef093792bba4d02d02e5ffe0f0373


net/http/cgi: Serve function passes a *http.Request value with a nil Body

The cgi.Serve function passes to the http.Handler a *http.Request value with a nil Body field when the request "Content-Length" header is not greater than zero.
But the documentation of Request.Body states:
// For server requests, the Request Body is always non-nil
As a conseguence, if the  http.Handler calls the Request.ParseForm method a "missing form body" panic occurs.

/cc @bradfitz @bcmills

Want to send a CL for when the tree opens for Go 1.16? (A regression test using httptest.Server seems straightforward.)

@bcmills Yes, I will send a CL with a regression test.


crashed on windows 10

https://gist.github.com/satdav/dce23cd5c048c681a8443fb5976aafbc
This is the logs for the crash I have windows defender

Please try disabling Defender and try again.

I have disabed Defender, but this bug is also exist ...

Can you show a screenshot of the crash or the error message? The log looks okay.

only atom 1.0.11 is ok
another Atom version will be crashed when opening a folder  which contains a git repository
such as 1.7.0  ,1.7.4 beta , 1.8.0 beta  &etc....

help me.. windows 10 32bits

Your path is too long.  I would recommend downloading the portable version and extracting it to a short path such as c:\atom


Adding stripe checkout for automating transactions

Adding stripe checkout for automating transactions

I want to work on this issue kindly assign me this issue @rohansx @rajlaxmi03

since it will be an open source boiler plate.. there is no need of payment gateway.


[KARAF-6488] Upgrade to kafka 2.4.1

FYI: I tested with kafka broker 2.2.0 and 2.4.1, meaning that Decanter can work with several kafka version.


Lost local library folder (/usr/local/go/src/MyGoLibrary)

After upgrade Golang from 1.19 to 1.19.1 my local library lost ( /usr/local/go/src/MyGoLibrary).
Please help to fix. Thank you.
What version of Go are you using (go version)?

$ go version
1.19.1

Does this issue reproduce with the latest release?
Error loading workspace: You are outside of a module and outside of $GOPATH/src. If you are using modules, please open your editor to a directory in your module. If you believe this warning is incorrect, please file an issue: https://github.com/golang/go/issues/new.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
MacOS/amd64

What did you do?
upgrade from 1.19 to 1.19.1

What did you expect to see?
What did you see instead?

Unlike many projects, the Go project does not use GitHub Issues for general discussion or asking questions. GitHub Issues are used for tracking bugs and proposals only.
For questions please refer to https://github.com/golang/go/wiki/Questions


Add ability to join user lobby from their profile page

This is for https://github.com/beyond-all-reason/teiserver/issues/178.
I only tested this through tests and have been unable to test locally. When running locally, I get region errors trying to create a lobby. I've come to understand that it's because I don't have SPADS running, but I'm unable to get SPADS working locally.
Details

Join button is only visible if the user is in a lobby

Screenshots

Hi @darcymiranda I've left this project (now implementing it as a library over at https://github.com/Teifion/teiserver) and instead I think @Beherith would be the better person to respond to this PR :)

Hit me up in DM on discord :)

Hit me up in DM on discord, DarcyMiranda :)

I was able to get spads setup (Thanks @jauggy) and successfully tested this manually


Utilize URL in FileManager

Now that we have URL in FileManager, we can adopt it throughout the APIs. Many of the APIs already existed using the URL stub previously available, but now the delegate methods and other APIs have been updated to use the full URL implementation.
Some APIs are currently still not available/ported because they rely on the URLResourceValues implementation which hasn't been ported yet. Those will be added in a followup PR. I also made 2 minor warning fixes while I was here.

@swift-ci please test

@swift-ci please test

@swift-ci please test


Add some snippet

Description
#1880
Related Issues

Checklist

[ ] Is this code covered by new or existing unit tests or integration tests?
[ ] Did you run make unit-testing and make integration-testing successfully?
[ ] Do new classes, functions, methods and parameters all have docstrings?
[ ] Were existing docstrings updated, if necessary?
[ ] Was external documentation updated, if necessary?

Additional Notes or Comments

Nice! Please remove db parameter to to_listener.

Done


[Config] Port 'RCNConfigFetch'

#no-changelog

Would you look at these test failures in Value.swift - https://github.com/firebase/firebase-ios-sdk/actions/runs/12520611431/job/34926546021?pr=14286#step:6:77

Failures look similar to that in #14284: https://github.com/firebase/firebase-ios-sdk/actions/runs/12506161513/job/34890668118?pr=14284
The catalyst test failure seems new but the unit test is the same flaky one as in ^.


#309 Fixed Update Job form spacing

Closes #309
Changed Update Job form spacing to match the create job page.

The file changing is being removed so issue is OBE in PR #324


chore: Fix error checking for the node class controller

Fixes #N/A
Description

fix Error handling for the node class controller

How was this change tested?

make presubmit

Does this change impact docs?

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

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

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

6 of 9 (66.67%)  changed or added relevant lines in 1 file are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.06%) to 82.254%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/controllers/nodeclass/controller.go
6
9
66.67%

Files with Coverage Reduction
New Missed Lines
%

pkg/controllers/nodeclass/controller.go
3
74.87%

Totals

Change from base Build<PHONE_NUMBER>:
-0.06%

Covered Lines:
5349

Relevant Lines:
6503

💛  - Coveralls