Borders around "Active", "Completed" and "Expired" (except the lower / bottom border) on Home Page should probably be removed (UI enhancement)

Testing Environment:
Windows 10, Google Chrome 73.0.3683.86 (64bit), 15.6", 1920х1080
Steps to reproduce:

Navigate to http://rfai.singularitynet.io.s3-website-us-east-1.amazonaws.com/
Press names of "Active", "Completed" and "Expired" tabs one by one
Observe the borders around these names

EXPECTED RESULT:
I think that it could look better if user only saw the bottom border (line under the name of the tab)
ACTUAL RESULT:
User sees some borders around the name of each tab. By the way when user clicks somewhere outside of these names then all borders (except the bottom one) disappear
Please see the video below

@raamb Can you please request Ananya to look into this UI issue. Thanks


Please allow https://chat.openai.com/

I am having issues accessing the above website due to security measures and controls on my computer. Please kindly add this website as a secured website as I need it for my role as a Capacity Building Officer. Many thanks, Buffy

Sorry Buffy, this is the wrong place to ask.


feat(viz picker): Remove some tags, refactor Recommended section

SUMMARY
Implements changes from discussion https://github.com/apache/superset/discussions/27557

Remove the Recommended tags section and move the charts in "Popular" to a "Featured" section underneath All Charts, "Featured" is selected by default
Remove the following tags:

Aesthetic
Coordinates
Deprecated
Description
Discrete
Experimental
Vertical

Bar Chart (legacy) : Remove Popular/Featured tag from Bar Chart (legacy)
Collapse Tags section by default

Also, refactored the svg icons used in viz picker (ballot, category, tags) as their viewboxes were weird, which resulted in additional css used to make them look good
CC @yousoph
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
Before:

After:

TESTING INSTRUCTIONS
Open the viz picker, verify that Recommended section is gone and Featured contains all charts with "Popular" tag, verify that everything looks good
ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

/testenv up

My only non-blocking comment would be to replace the Featured icon with a ⭐

@yousoph @kasiazjc wdyt?

My only non-blocking comment would be to replace the Featured icon with a ⭐

@yousoph @kasiazjc wdyt?

@michael-s-molina also thought about ⭐ at first, but we already use it for "favorites", so wanted to avoid duplicates. Sadly ant-d doesnt have the best choice for icons, but the other contenders (besides pin) would be:

or just

Thoughts @kgabryje @michael-s-molina ?

The first one looks good!

@kasiazjc The first one works. There's also:

I'll let you make the call 😉

@kasiazjc The first one works. There's also:

I'll let you make the call 😉

Actually I think fire works well 🔥
Cc: @kgabryje

@kasiazjc @michael-s-molina Looks pretty good 🙂

Thank you! 🔥


[Github] See the pull requests of others in the same repo

Extension
https://www.raycast.com/raycast/github
Description
When in a repository with others, see their pull requests as well
Who will benefit from this feature?
No response
Anything else?
No response

In which command would you like to see that? Note that it's already possible when searching for repositories (via the Show Pull Requests action, cf recording below)
https://user-images.githubusercontent.com/16003285/206703779-b84ae02d-d5df-4b48-9e98-f8ce49f86a52.mp4

If it's already available, than my issue can be closed. In my opinion it makes more sense to put it in the "Search Pull Requests" tab. For example you would be able to choose a repo in the Pull requests tab and then look at the pull requests that has been made for the repo.
In other words; I think the feature is way too hidden.

That's a fair point. I'm going to close this thread, since my question has been answered. Thank you!


BuildStep doesn't run commands when --skip-preview is used

The BuildStep pulumi module doesn't run the cmd when --skip-preview is passed to pulumi.
Unfortunately pulumi doesn't expose whether --skip-preview has been used. I've check the process.env and process.argv as well to see if we can determine if --skip-preview is used, and we can't :(
I have 2 solutions in mind:

Run the pulumi command with an env variable called PULUMI_SKIP_PREVIEW and that could checked in our BuildStep component.
Change the BuildStep into a DynamicResourceProvider and always run the command in the diff method

BuildStep needs to move the run into the apply stage.
Either we build outside of the pulumi process (which the NX pulumi plugin can do), or it only builds in plan and not preview.


Two pods on the same node cannot access a volume at the same time?

Hi!
I am trying to use Okteto with Linstor. Okteto is a tool for development in Kubernetes, and it creates a persistent volume to keep work files etc. There is a main deployment and optional additional services that can be activated, and these services are scheduled on the same node as the main one so they can share the volume. This works with Ceph, Longhorn and OpenEBS.
However with Linstor I get the following error when the second service/pod attempts to mount the volume:
MountVolume.SetUp failed for volume "pvc-d264ff30-cf3c-4494-a28e-38887156a454" : kubernetes.io/csi: mounter.SetupAt failed: rpc error: code = Internal desc = NodePublishVolume failed for pvc-d264ff30-cf3c-4494-a28e-38887156a454: unable to get an exclusive open on /dev/drbd1009, check device health

Is there a workaround for this? Thanks!

Closing because it's been discussed already.


Recipe for Tempo change and volume change?

I'm loving mido.  Wow, it works so well.
I'm using the loop for my playback
        #------- Loop through MIDI file sending commands ------- #
        for i, msg in enumerate(mid.play()):
            if i%5:
                rc = pback.PlayerPlaybackGUIUpdate('Now Playing \n\n{}\n\n              {:0d}:{:02d}'.format(movie_filename,i//60, i%60))
            else:
                rc = PLAYER_COMMAND_NONE

            time.sleep(.25) if  paused else port.send(msg)

I want to add tempo and volume adjustments.  What are the calls I need to make.  It wasn't clear to me.
Thanks for your time.

I see now how Tempo could be changed.
Exposing tempo in the MidiFile object is one potential way I see of implementing tempo in the module as a whole.
In play the time to sleep between messages is computed.  Tempo could be taken into account there.
Does this sound like a reasonable approach?
Something like this:
    def play(self, meta_messages=False, tempo_scaler=1):
And then there's a change here:
            sleep(msg.time/tempo_scaler)
Seems like this would give new users the capability, keep the old code working as is, and you've added the ability to directly change the tempo to your mido feature list.

I ended up making a superclass from the MidiFile class
class MyMidiFile(mido.MidiFile):
    def __init__(self, **kwargs):
        self.tempo_scalar = 1.0
        super().__init__(**kwargs)

This allowed me to create and expose a tempo_scalar attribute that is then used in the play method which was included in the superclass as shown on this line of code:
            sleep(msg.time*self.tempo_scalar)
Tempo control would be a good feature for the mido team to include in this package, especially since it has no backward compatibility issues and has a low risk as it's only used with the play method.

Closing this one since volume can be implemented in the same location.


Using lazy imports to avoid having to append strict requirements

@bendichter Using a similar strategy to spikeextractors, this allows the format specific interfaces that use specific outside packages to import them without having to add the packages to the main requirements.
Currently the only two interfaces that do this are intan and neuroscope, but it sets the precedent early on do always use this strategy to avoid bloating the nwb-conversion-tools setup.

@bendichter Actually, let's hold off on reviewing this until we discuss the approach with ecephys_metadata and static methods.
If we have any static methods for use across interfaces or what not we would want the install message and installation status to be set as a class attribute for each interface.

@bendichter Should be good for review now.


Add Frame-Options header

Problem:
Frame-Options header is not being set, this is a potential secutiry
issue
Solution:
Set the header "X-Frame-Options:deny"

LGTM

LGTM


Add on progress

Hi guy.
I added a function progress in android and ios from your code. I hope you grant me a permission request to your project.

Hi, thanks for your pull request. Will try to review it during this week.

Any progress on this request?

Oops, thanks for the reminder. New ETA: end of this week!

Thanks for your response. I also have some plan to contribute to this project later. Cheers!

Any news on when this can be merged? It would be extremely useful

Huge apologies for taking so long on this, it's on me as I've been very busy with deadlines in several other projects. We have to find someone else to maintain this if my schedule continues like this. Sorry again!

Wow! Thanks for coming back, I am truly well using this repository (almost ready for production.)


v0.7.0 Release

Last call for 0.7.0 release. Please say something now if you have an issue with this release.
@fraga @bitbeans @deckar01
Change Log:

libsodium v1.0.2
New documentation (h/t @bitbeans)
Added support for streaming BLAKE2b hashing (h/t @deckar01)
Updated HashSaltPersonal to support variable length output (h/t @bitbeans)
Fixes and test for thread safety (h/t @deckar01)
Better handling for unit test with unusual memory requirements (h/t @deckar01)

Seems fine to me :)

:+1:
the documentation should be up-to-date.

Released:
https://github.com/adamcaudill/libsodium-net/releases/tag/v0.7.0
https://www.nuget.org/packages/libsodium-net/


Envoy

Description
Reference issue
Fixes #...

I think the new ion does not need Envoy, and the new ion supports grpc and grpc-web at the same time on a single port.
https://github.com/pion/ion/blob/master/cmd/biz/server/wrapped.go#L177


Wrong parsing of output fields

The OutputField in a custom dspy.Signature are not getting parsed
class EquivalenceC(dspy.Signature):
    """Compare source and target:
    [...]
    """

    source = dspy.InputField(desc="extensive background information")
    target = dspy.InputField(desc="information that needs to be verified")
    match = dspy.OutputField(desc="one word of: yes, no, approximately; note that 'approximately' implies within 10% error.")
    correction = dspy.OutputField(desc="corrected replacement for 'target' if needed")

rationale_type = dspy.OutputField(prefix="Reasoning: Let's think step by step in order to",
                             desc="${understand whether the two inputs match and provide correction if needed}. The values provided are ...")

dspy.ChainOfThought(EquivalenceC, rationale_type=rationale_type)(source=source, target=target)

The result is:
Prediction(
    rationale='understand whether the two inputs match and provide correction if needed. The values provided are both units of time, but they are not exactly equivalent. There are 7 days in a week, so 4 weeks would be equal to 28 days. Therefore, the values mentioned in the target are not exactly equivalent to the information within the source.\n\nCorrection: 4 weeks is approximately equivalent to 28 days.',
    match='',
    correction='no\n\nCorrection: 4 weeks is approximately equivalent to 28 days.'
)

inspecting the LLM log:
Reasoning: Let's think step by step in order to understand whether the two inputs match and provide correction if needed. The values provided are both units of time, but they are not exactly equivalent. There are 7 days in a week, so 4 weeks would be equal to 28 days. Therefore, the values mentioned in the target are not exactly equivalent to the information within the source. Correction: 4 weeks is approximately equivalent to 28 days.

Match: no

Correction: 4 weeks is approximately equivalent to 28 days.

Hi @DSLituiev , which LM is this for? Supporting backends for expected generations/parsing for chat models is WIP. You can overcome some of this through proper stopping conditions or external parsing of the generations where needed.

This is gpt3.5
it works better with gpt4
anyhow, the output of gpt3.5 seems reasonable-ish to be parseable
On Thu, Apr 18, 2024 at 2:00 PM arnavsinghvi11 @.***>
wrote:

Hi @DSLituiev https://github.com/DSLituiev , which LM is this for?
Supporting backends for expected generations/parsing for chat models is
WIP. You can overcome some of this through proper stopping conditions or
external parsing of the generations where needed.
—
Reply to this email directly, view it on GitHub
https://github.com/stanfordnlp/dspy/issues/847#issuecomment-2064767740,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAJGMUO2CILW6Z476YWXXTY6AC4FAVCNFSM6AAAAABGKM4FEOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRUG43DONZUGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Dima Lituiev, PhD

@DSLituiev Were you able to get past this issue?

No. Using Gpt4 for now
On Mon, Apr 22, 2024 at 10:03 PM canada4663 @.***>
wrote:

@DSLituiev https://github.com/DSLituiev Were you able to get past this
issue?
—
Reply to this email directly, view it on GitHub
https://github.com/stanfordnlp/dspy/issues/847#issuecomment-2071413760,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACAJGMT2XXKVEELBX2U6ZALY6XTS5AVCNFSM6AAAAABGKM4FEOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZRGQYTGNZWGA
.
You are receiving this because you were mentioned.Message ID:
@.***>

--
Dima Lituiev, PhD

This issue is being observed with Bedrock Anthropic Sonnet LM as well. Using similar signature with ChainOfThought module. I also observed that on each invocation, predictor is calling invoke API twice and updating request args.

Have you figured out better parsing approach? I came across the same issue with Bedrock Anthropic Sonnet


Dev/auth

add authentication to app including login and logout views, staff_member_required and reverse routing

merge dev/auth brach into main delete dev/auth branch.


V2.0.3

All Submissions Basics:

[ ] Have you followed the guidelines in our Contributing document?
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?
[ ] Have you checked all Issues to tie the PR to a specific one?

All Submissions Cores:

[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your core changes, as applicable?
[ ] Have you successfully ran tests with your changes locally?
[ ] Does your submission pass tests, including CircleCI, Travis CI, and AppVeyor?
[ ] Does your submission have appropriate code coverage? The cutoff threshold is 95% by Coversall.

New Model Submissions:

[ ] Have you created a .py in ~/pyod/models/?
[ ] Have you created a _example.py in ~/examples/?
[ ] Have you created a test_.py in ~/pyod/test/?
[ ] Have you lint your code locally prior to submission?

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

478 of 485 (98.56%)  changed or added relevant lines in 45 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 95.361%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pyod/models/base.py
42
49
85.71%

Totals

Change from base Build<PHONE_NUMBER>6:
0.1%

Covered Lines:
10361

Relevant Lines:
10865

💛  - Coveralls


Since column.type was added, complex data is saved as string

Hello. Since somewhat recent update which implemented column.type broke saving of more complex data structures (like arrays or objects). As written in docs, column.type supports only string, number, bool and date.
If you do not set type, it is undefined which passes the first if (!type) in typeConvert and is automatically converted to string.
As a quick-fix you can define type as any random string so it saves it as it is, but it will show a error in console, warning that wrong type is supplied to column.
#1004 is related to this.

@RudPer please follow the instructions when you open an issue so that I can help you.
I think for me the most important thing is put your code in codesandbox and keep it simple and minimal.
According your descriptions, Im still unclear about below parts:

why you add column.type for a complex data type?  any reason?
Even you don't use column.type, if your column type is array or object, how you made it editable?

I will reopen this issue when I got the clear code or explanation
AGAIN, PLEASE FOLLOW THE INSTRUCTION OF OPENING ISSUE, THANKS

Hello, sorry for not being clear.
Here is codesandbox link - https://codesandbox.io/s/react-bootstrap-table-next-basic-example-tsrqc?fontsize=14
For example, this is customer table. Customers can have multiple phones, so having them in array is the best solution. But now when trying to save array, it will be transformed into string and will break everything, as cell formatter expects there to be array which can be joined together in a string.
To break it, try changing customer phone field.
And to answer your questions:

No, I would not add column.type for complex data type, but currently it automatically is transformed into string by typeConvert function.
By using editorRenderer and using something like that i have in codesandbox or multi select

I hope I'm more specific now.

@RudPer thanks your explanation. For the unsupport data type, I will not do any type change to avoid this issue, thanks!

@RudPer I fixed in newest version, please upgrade to newest version, thanks
here is release note


does not recognize the uri

import (
	"encoding/json"
	"fmt"

	"github.com/BishopFox/jsluice"
)

func main() {
	analyzer := jsluice.NewAnalyzer([]byte(`
	function C() {
        return C = (0, o.default)(c().mark(function e(t) {
                return c().wrap(function (e) {
                    while (1)
                        switch (e.prev = e.next) {
                        case 0:
                            return e.abrupt("return", (0, i.default)("/admin/article/paging", {
                                    method: "POST",
                                    body: t
                                }));
                        case 1:
                        case "end":
                            return e.stop()
                        }
                }, e)
            })),
        C.apply(this, arguments)
    }
	`))

	for _, url := range analyzer.GetURLs() {
		j, err := json.MarshalIndent(url, "", "  ")
		if err != nil {
			continue
		}

		fmt.Printf("%s\n", j)
	}
}

does not recognize the uri:
"/admin/article/paging"

Hey @michaellcader, thanks for raising an issue!
This is related to the logic that jsluice uses to identify paths. At the moment the MaybeURL function is quite dependent on there being a file extension, query string, or something else like that to increase confidence.
I've been thinking about this for a while and think there are some tweaks we can do to improve the situation without catching too many false positives.
I'll leave this issue open and use it as a place to track that work.
Thank you again!

Hey @michaellcader! Thank you again for raising a detailed issue and for your patience.
This should be fixed as of #15 being merged. Please let us know if you encounter more issues!


Provide Hadoop ecosystem ulimits via LWRP

This pull request replicates what used to be delivered into /etc/security/limits.d in HDP2.0 to be provided by Chef. (For reference those files have now moved to /usr/hdp/<version>/etc/security/limits.d but I think we can manage better not playing musical files.)

Thanks for catching my metadata.rb miss; it's committed now.
The ulimits have largely not changed in my 5 years of Hadoop-ing that I can recall. Admittedly, I grabbed these ulimits by running cat /usr/hdp/<version>/etc/security/limits.d/* on a cluster, however. And the fact that I know ZK could run out of sockets (file descriptors) so I added that which is new from the HDP2.0 process.
The number of processes really should only matter for yarn and mapred from my view (they scale with the number of containers on a worker node). I can not contrive why one would end up with thousands of HDFS processes; but Hortonworks seems to think that is (or once was) wise.
The user_ulimit resource is provided by the ulimit cookbook. We have been using it in the kafka-bcpc::kafka recipe

Thanks for the explanation. LGTM.

Looks good


Update for latest time-warp-nt: queue EKG counters

Waiting on https://github.com/serokell/time-warp-nt/pull/87

@dcoutts @avieth this updates cardano-sl for https://github.com/serokell/time-warp-nt/pull/87 and makes some generalizations that we discussed; there are still a few TODOs in the code though; for instance, we don't set a maximum bucket size yet (though we can), and we don't us the static topology yet for unknown nodes (though the queue is ready for it).

One hlint complaint, and the OSX build is timing out.

@dcoutts and @avieth , removed the fields that @avieth 's later PRs will obsolete, and pushed a commit that should placate hlint. I think the order now is

merge this PR, so we have at least the counter stuff available
rebase and merge https://github.com/serokell/time-warp-nt/pull/90
merge https://github.com/input-output-hk/cardano-sl/pull/1350 (adjust for https://github.com/serokell/time-warp-nt/pull/90: move policies to cardano-sl, introduce the mapping for node IDs)
rebase and merge https://github.com/input-output-hk/cardano-sl/pull/1347 (make policy .yaml configurable)

Rebased on latest master.

CI fails because 0curl: (7) Failed to connect to hydra.iohk.io port 443: Operation timed out

Sorry, we were upgrading Hydra with bigger HDD as it ran out of disk space. I'll restart the CI.


0.60.1 Release notes

@sgny @cknitt & others, does this looks good?

@sgny any feedback before we cut this release?


[Authoring] Input.Date Card does not work for Global Applications

Copied bug from: https://github.com/microsoft/AdaptiveCards/issues/6441
Details from original post:
Target Application
Microsoft Teams
Application Operating System
Windows
Schema Version
1.3
Problem Description
We have a Teams application using the Input.Date Adaptive card. This application is used by Global Users all in different Local Timezones... When the raw input from the card is sent to our application, it sets the date by sending Local TZ -> GMT 00:00:00. For example,
Ireland Standard Time (GMT+1):
Inputs 09/30/2021 @ 5pm
Output 09/29/2021 @ 5pm
Pacific Standard Time (GMT-7):
Inputs 09/30/2021 @ 5pm
Output 09/30/2021 @ 5pm
Expected Outcome
The backend should be using the date/time to determine the output GMT date.
Actual Outcome
Since the adaptive card backend seems to be only using input the date to determine the output GMT date, this causes issues for a Global use case..
Card JSON
"$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
  "type": "AdaptiveCard",
  "version": "1.0",
  "body": [
    {
      "type": "TextBlock",
      "text": "Date Input"
    },
    {
      "type": "Input.Date",
      "id": "date",
      "placeholder": "Enter a date",
      "value": "2017-10-12"
    }
  ],
  "actions": [
    {
      "type": "Action.Submit",
      "title": "OK"
    }
  ]
}

Repro Steps

Create an adaptive card receiving date/time input field
Have a user in IST test this versus a user in PST or EST
Observe that for users in TZ where it is GMT+# the previous date will be returned, but for users in TZ where it is GMT-# the correct date will be returned.---

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

ID: 3fb4394d-25b1-8752-3996-fe3a2934732d
Version Independent ID: 6d391381-89d0-117c-27cc-00043bd26a36
Content: Cards reference - Teams
Content Source: msteams-platform/task-modules-and-cards/cards/cards-reference.md
Product: msteams
GitHub Login: @surbhigupta12
Microsoft Alias: surbhigupta12

We are checking this internally. Will get back to you once we get any update

Thank you.
We are one of the users experiencing this problem in production.
It's a really frustrating issue since our product is a leave tracker which integrates with Microsoft Teams, and as such it really depends on correct behavior of a date picker.
As far as I can tell, who experiences the issue is related to the specific instance of the Teams app. So one person can have it on Mac app, but not on web app.
Some kind of canary deployment or A/B testing in progress?

We also have this problem, has any solution been found?

This problem has now appeared in the browser!
When will the update?

@koaskas , We tried to repro and can confirm that its working fine at our end both in Desktop client and web client. Please try to update your Teams version and check if it helps. Attaching screenshot below for reference:
Desktop:

Web:

@Prasad-MSFT Thanks for fixing this issue.

@Prasad-MSFT yes, updates have started to come in, thanks

@koaskas ,Thanks for confirming. Closing the issue for now. Please feel free to re-open if needed.

@Prasad-MSFT
The problem has resumed
Please look

@brokkolifever - which teams version are you using? Could you please try to update it once and check?

@Prasad-MSFT version is <IP_ADDRESS>76

Could you please try to update it to Microsoft Teams Version <IP_ADDRESS>167 and check once ?

@Prasad-MSFT i can only update to <IP_ADDRESS>469.

and i see the same problem in web version

@brokkolifever - Its quite strange. Now we checked this again in both Teams Desktop(Microsoft Teams Version <IP_ADDRESS>167) and Teams web ,and at both the places its working fine.
We suggest you to update your Teams version and check.
https://microsoft-teams.en.uptodown.com/windows/versions

Hello. Automatically I can only update to <IP_ADDRESS>469
I can't find link to version <IP_ADDRESS>167 here:
https://microsoft-teams.en.uptodown.com/

Can you send a link?

@Prasad-MSFT The issue is also back on Mac desktop. I tried to update to the latest version but all I get is this:

I think this issue should be re-opened.

@Prasad-MSFT I have now been able to update Teams on Mac to a version that fixes the issue. Thanks!

@Prasad-MSFT My version is <IP_ADDRESS>771 now but i still but I still get the same problem (in the web version too)

@brokkolifever - We have informed the concerned team about the issue. We will inform you once we get any update. Thanks!


ionic-timepicker.templates module not found

ionic-timepicker module depends on ionic-timepicker.templates, but the ionic-timepicker.templates.js is not found!!

Please check your imports in index file and the injection of the dependency in the main module. There is no issue with the template.js file. Also please check the demo and the instructions for more info.


Bug: Extracting tar failed

When extracting cbindgen-0.24.3-x86_64-apple-darwin.tar.gz using the following equivalent code:
use flate2::read::GzDecoder;
use tar::Archive;

let tar = GzDecoder::new(dat);
Archive::new(tar).unpack(path)?;

where dat is some readable that is pipelined from the downloading stage, I got:
/var/folders/4h/3pck4_r16tn6960znvv4w0nw0000gn/T/.tmp7GLCIo/bin-cbindgen
└── GNUSparseFile.0
    └── cbindgen

1 directory, 1 file

While it should be:
/var/folders/4h/3pck4_r16tn6960znvv4w0nw0000gn/T/.tmp7GLCIo/bin-cbindgen
└── cbindgen

I used bsdtar 3.5.1 - libarchive 3.5.1 zlib/1.2.11 liblzma/5.0.5 bz2lib/1.0.8 and it handles it just fine.
Here's the relevant part of the Cargo.toml:
flate2 = { version = "1.0.24", features = ["zlib-ng"], default-features = false }
tar = "0.4.38"

I discovered this bug in https://github.com/ryankurte/cargo-binstall/pull/174

Here's the reproduction.
Just run cargo r and you will notice the GNUSparseFile.0 directory.

@alexcrichton I also thought that until I saw this:

Also it seems that quick-install is only using normal tar -czf, not the --sparse option? So I'm unsure what caused this package, unless the tar they're using has it on by default or turns it on based on some heuristic.

The cbindgen tarball seems to be built without --sparse option.

@alexcrichton Pinging

I'm having the same issue when using flate2 and tar in my Cargo.toml like this:
flate2 = "1.0"
tar = "0.4.38"

When I'm trying to unpack tokei-x86_64-apple-darwin.tar.gz using the following code:
fn unpack_tar(tar_path: &PathBuf, tmp_dir: &Path) -> Result<(), std::io::Error> {
    let tar_file = File::open(tar_path)?;
    let tar_decoder = GzDecoder::new(tar_file);
    let mut archive = tar::Archive::new(tar_decoder);
    archive.unpack(tmp_dir)
}

I see that the unpacking result produces the GNUSparseFile.0 directory:
$ exa --tree
.
├── GNUSparseFile.0
│  └── tokei
└── tokei-x86_64-apple-darwin.tar.gz

I'm not sure if the file actually valid (since I'm on Ubuntu 20.04 and the archive contents a macOS executable so I can't run it). But is this expected and can/should it be fixed?

@chshersh #298 should fix this, but I haven't tested the solution and @alexcrichton hasn't reviewed it yet.

Same issue when unpacking cargo-generate tar.gz using:
flate2 = "1.0.25"
tar = "0.4.38"


Bug: simp? does not take reverse directions into account

MWE:
import Std
def foo : Nat → Nat := sorry
theorem foo_eq_iff (z : Nat) : z = 17 ↔ foo z = 3 := sorry
theorem fail (z : Nat) (h : z = 17) : foo z = 3 := by
  simp? [← foo_eq_iff]
  exact h

The simp? works fine, but the suggestion is to use simp only [foo_eq_iff].

Caused by core: leanprover/lean4#2131


Change 'signature' -> 'sig' in generated URL

The third and final piece of the 403 error that I can find, submitted as a separate PR because I'm not sure that downloading still works in all cases with this change. Combine with #425 and #426 for the complete fix.

Coverage decreased (-0.3%) to 77.261% when pulling acf75d7e9031bfc3efa18e555b177addf8a795c0 on CraftSpider:patch-2 into 7e0cd4e8639ab5754227eafa8a98613939c8b61a on nficano:master.

@CraftSpider  is functionality in this PR still needed? Would be happy to assist in getting it merged

This line was fixed some time ago https://github.com/nficano/pytube/blob/ee0deb6dc1c465927472f3c01f8644ddd909ce50/pytube/extract.py#L268 this PR is unneeded

Thank @CraftSpider ! :)


Make e2e test (cpu comparison) python based

Python is easier to write and maintain than bash. What finally pushed me into doing this is that I am working on a way to use python (pure numpy for example) to generate golden values, instead of the llvm-cpu backend of IREE. This resulted in a bash script with 4+ calls into python. Rather than do that, I think it's better to start in python, and call bash when necessary.

aww man i was just getting really good at bash :( jk god bless you for doing this :pray:
but also btw i recently updated run_matmul https://github.com/nod-ai/iree-amd-aie/pull/583 can we not lose that (sorry i didn't look at the PR, maybe you already ported)

Overall LGTM. A feature request: can you add some print info about time spent for compilation and execution? Similar to this https://github.com/nod-ai/iree-amd-aie/blob/main/build_tools/ci/run_matmul_test.sh#L472

Overall LGTM. A feature request: can you add some print info about time spent for compilation and execution? Similar to this https://github.com/nod-ai/iree-amd-aie/blob/main/build_tools/ci/run_matmul_test.sh#L472

Yes, good idea


Can't add the helper, it doesn't appear...

Before you submit a new bug report, please check that

[X] I have read frequently asked questions and limitations
[X] I'm running the newest version of Holidays, or the next pre-release.
[X] I have enabled debug logging for my installation.

Describe the bug
I had a message after a restart that the integration "Holidays" couldn't be loaded. I uninstalled the helpers, removed the integration, restarted, redownloaded the integration via hacs, restarted home assistant...
and now I can't add the helpers... they don't appear... :-(
Configuration
N/A

Debug logs
N/A

Yes same for me. I tried downgrading to 1.9.4 and that has the same issue as well.

yep, same here. HDongrading doesn't solve the problem.

same issue here

Indeed.

Same here, running the beta and it says the integration doesn’t exist

I've got even further: after installing 1.9.5 my HA began to crash regularly every 2-3 minutes. HA console (ssh login) said something about memory overflow error and some funny numbers  -  sadly i didn't wrote them down...

ls let me know if 1.9.6 release fixes that. Thanks.

That seems to have fixed it for me. Thanks for the quick turnaround. Much appreciated.

It works here, too. Many thanks for this really quick fix!

yep, same here. Downgrading doesn't solve the problem.

It works again like expected. Many thanks for the quick response.

Fixed! Thanks!


[Tracking issue]: Add nerdctl 2.0 compatibility

Description
This is a tracking issue for a list of tasks which need to be resolved to unblock nerdctl 2.0 integration.
finch-core https://github.com/runfinch/finch-core/pull/472 integrates nerdctl 2.0.2 as Finch's core container runtime on Finch on macOS and Windows platforms. (Note: Finch on Linux utilizes the platform's container runtime components)
Integration test failures

[ ] #196
[ ] [FAIL] Finch Core E2E Tests Compose down command [It] should stop services defined in compose file without deleting volumes
[ ] [FAIL] Finch Core E2E Tests Compose down command [It] should stop compose services and delete volumes by specifying --volumes flag
[ ] [FAIL] Finch Core E2E Tests Compose down command [It] should stop compose services and delete volumes by specifying -v flag
[ ] #202
[x] [FAIL] Finch Core E2E Tests Compose kill command [AfterEach] should not kill running containers with --signal TERM
[x] [FAIL] Finch Core E2E Tests Compose kill command [AfterEach] should not kill running containers with -s SIGTERM
[x] [FAIL] Finch Core E2E Tests Compose kill command [AfterEach] should not kill running containers with -s TERM
[x] [FAIL] Finch Core E2E Tests Compose logs command [AfterEach] should show the logs with prefixes
[x] [FAIL] Finch Core E2E Tests Compose logs command [AfterEach] should show the logs without prefixes
[x] [FAIL] Finch Core E2E Tests Compose logs command [AfterEach] should show the logs with no color
[x] [FAIL] Finch Core E2E Tests Compose logs command [AfterEach] should show the last line of the logs
[x] [FAIL] Finch Core E2E Tests Compose logs command [AfterEach] should show the logs with timestamp with no prefixes and no color [flag --timestamps]
[x] [FAIL] Finch Core E2E Tests Compose logs command [AfterEach] should show the logs with timestamp with no prefixes and no color [flag -t]
[ ] #200
[ ] [FAIL] Finch Core E2E Tests display detailed information on one or more images [It] should not display information if image doesn't exist
[ ] #198

Deduped compose kill command and compose logs command integration test failures to #202.

Deduped compose down command integration test failures to #206

Release PR https://github.com/runfinch/common-tests/pull/189

v0.9.0 tag has been created! 🥳


Fixing issue with doctestAddTests.cmake

Newer versions of CMake won't work with line 60: if(${add_labels} EQUAL 1), changed it to simply if (${add_labels})

Description

Builds using the CMake doctest_discover_tests function are failing due to a syntax error. This is likely due to newer versions of CMake syntax. This fixes those issues by updating the syntax.
GitHub Issues

Addresses #524

Not sure if the cmake scripts are tested in the project's CI/CD, but it could be done by creating a basic CMake project that uses these scripts to detect tests. This would allow for future regressions to be spotted

Hi Can you also revert the changes in #490 in this PR. Those are not necessary with this fix. Thanks

Is there a way to speed up the inclusion of this PR?

Not sure if the cmake scripts are tested in the project's CI/CD, but it could be done by creating a basic CMake project that uses these scripts to detect tests. This would allow for future regressions to be spotted

PRs welcome :)


[AMBARI-24923] Create tproxy-configuration category in Ambari Configurations data

What changes were proposed in this pull request?
Create tproxy-configuration category in Ambari Configurations data with the following properties:

ambari.tproxy.authentication.enabled

Determines whether to allow trusted proxy authentication when logging into Ambari
true | false

ambari.tproxy.proxyuser.$username.hosts

List of hosts from which trusted-proxy user ‘$username’ can connect from
* | c7401.ambari.apache.org | <IP_ADDRESS>,<IP_ADDRESS> | <IP_ADDRESS>/16,<IP_ADDRESS>

ambari.tproxy.proxyuser.$username.users

List of users which the trusted-proxy user ‘$username’ can proxy for
* | user1,user2

ambari.tproxy.proxyuser.$username.groups

List of user-groups which trusted-proxy user ‘$username’ can proxy for
* | group1,group2

Note: $username is a variable, declaring the values for a particular proxy user. For example "knox".
How was this patch tested?
Manually tested using the following REST API Calls

POST /api/v1/services/AMBARI/components/AMBARI_SERVER/configurations

{
  "Configuration": {
    "category" : "tproxy-configuration",
    "properties": {
      "ambari.tproxy.authentication.enabled" : "true",
      "ambari.tproxy.proxyuser.knox.hosts": "c7401.ambari.apache.org",
      "ambari.tproxy.proxyuser.knox.users": "*",
      "ambari.tproxy.proxyuser.knox.groups": "users"
    }
  }
}

PUT /api/v1/services/AMBARI/components/AMBARI_SERVER/configurations/tproxy-configurations

{
  "Configuration": {
    "properties": {
      "ambari.tproxy.authentication.enabled" : "false"
    }
  }
}

GET /api/v1/services/AMBARI/components/AMBARI_SERVER/configurations/tproxy-configurations

Updated unit tests:
mvn clean package install test -pl ambari-server,ambari-project
...
[INFO] ------------------------------------------------------------------------
[INFO] Reactor Summary:
[INFO]
[INFO] Apache Ambari Project POM .......................... SUCCESS [  3.020 s]
[INFO] Ambari Server ...................................... SUCCESS [42:19 min]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 42:22 min
[INFO] Finished at: 2018-11-21T08:54:29-05:00
[INFO] Final Memory: 209M/964M
[INFO] ------------------------------------------------------------------------

Please review Ambari Contributing Guide before opening a pull request.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/4611/
Test PASSed.


Santoshkota17 patch 1

 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: Santoshkota17 sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


fix: broken compilation on ubuntu 17.10

Description:
commit 45a126da430232dc0ebc65d56cbfe0b525196452 broke complication on my machine (ubuntu 17.10, gcc (Ubuntu 7.2.0-8ubuntu3.2) 7.2.0. This adds a missing include to make envoy compile.
Risk Level: Low

closing this PR as https://github.com/envoyproxy/envoy/pull/2972 does the same


Some issues I noticed with latest commits

So glad to see @Prefinem working on this again =]

I pulled the latest commit hoping it would fix torrent updates in loader.js but it seems the same behaviour to me. I don't see seed/leech being updated. Nothing shows up in loader-out.log, or loader.error.log with debug = true in index.js. When I run scraper.js I see in log:

Total Torrents: 785004
Torrents without Tracker: 781921
Torrents not in Search: 361

Another issue I found was that I kept noticing 0 byte files being scraped, or at least I thought they were 0byte! When I checked mysql I found they had a proper length field BUT they had a huge files field which was always truncated abruptly due to max length on TEXT type field in mysql. You can maybe see what I mean here: https://pastebin.com/STBrvykL

I did this to test (ALTER TABLE torrents MODIFY `files` MEDIUMTEXT;) and it seems no more broken file lists but this opens us up to 16mb max record sizes on this field. Might be worth it to truncate these instead in parser.js but I'm not sure the best approach :/

The last issue I noticed is I still cannot seem to filter out the invalid character � from scrapes, I've tried 0xFFFD, � - and a few others...but I'll keep trying. I haven't tested but I think this should work on all unexpected characters:  dataStr.replace(/[\u{0080}-\u{FFFF}]/gu,"");

Did you do a pm2 restart?
I added a ticket to truncate file lists since I am working on updating the system as a whole.
Where does that invalid character come from?

Did you do a pm2 restart?

Yes. pm2 restart all

Where does that invalid character come from?

Unsure! It appears exactly like that in the database.  �

Could it be that his server blocks access to UDP or the tracker? would you get some error if so? it seems 90% of his torrents have a status of NULL in tacker updated in mysql

Maybe.
Can you set config.debug = true and see if there is any output?
Add a line to src/tracker.js on line 58 of console.log(records.length); and see if you are getting records

Just tested with portquiz to make sure port 6881 isn't blocked:
$ curl portquiz.net:6881
Port 6881 test successful!
Your IP: xx.xx.xx.xx
$ wget -qO- portquiz.net:6881
Port 6881 test successful!
Your IP: xx.xx.xx.xx

Can you set config.debug = true and see if there is any output?
Yes it is.

Add a line to src/tracker.js on line 58 of console.log(records.length); and see if you are getting records

I see in the log for scraper now:
scraper > 75

The logs show scraper > 75?

Ahhaaaa. So made some changes to see if maybe I am being rate limited or something:
		host: 'udp://tracker.coppersurfer.tk:6969/announce',
		limit: 55,```

And now it works! I am seeing records like this now:

```scraper > 004787ab3e6071b96b71fc18667156441ffd1b76 - 136:8```

What a relief!

Q: Is there a way to just run the SE:LE updater without scraping more torrents? I tried only running loader.js but that doesn't seem to work
I just changed the bind ip address in config/index.js from <IP_ADDRESS> to <IP_ADDRESS> to disable getting new torrents

I just changed the bind ip address in config/index.js from <IP_ADDRESS> to <IP_ADDRESS> to disable getting new torrents

tyvm =]

I am splitting up the tracker and scraper in the current working branch which will allow this

im also now struggling with the tracker.
Total Torrents: 1983006
Torrents without Tracker: 208771

After checking it looks like the scraper seems to be scraping 75 every 1 minute give or take. Could it be that the tracker.js is querying MySQL for torrents with seeders info less than X age and this query is taking time and the tracker has to wait for this before it can update? @Prefinem

i can confirm this by checking the process list of mysql. and also running the command myself.
Showing rows 0 - 74 (75 total, Query took 96.7467 seconds.)
select * from torrents where trackerUpdated is null limit 75

maybe i need to tweak mysql lol

This is something I just ran into with reworking things. I think I am going
to have to drop MySQL completely since it’s too slow.
On Thu, Feb 7, 2019 at 18:02 ash121121<EMAIL_ADDRESS>wrote:

i can confirm this by checking the process list of mysql. and also running
the command myself.
Showing rows 0 - 74 (75 total, Query took 96.7467 seconds.)
select * from torrents where trackerUpdated is null limit 75
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AlphaReign/scraper/issues/52#issuecomment-461642012,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACnvZhjB5Y-nsVP0S8PC4h6Wj-b0a1knks5vLL6NgaJpZM4amw9r
.

i found tweaking the InnoDB pool size took the query time down to 1 second

What was the tweak?
On Thu, Feb 7, 2019 at 18:26 ash121121<EMAIL_ADDRESS>wrote:

i found tweaking the InnoDB pool size took the query time down to 1 second
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AlphaReign/scraper/issues/52#issuecomment-461646743,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACnvZsgouFigBYECT-wZ-xLPkkkgzuNxks5vLMQ1gaJpZM4amw9r
.

i dont know what all the settings do but i tweaked inside # InnoDB Settings.
[mysqld]

bind-address=<IP_ADDRESS>
# InnoDB Settings
default_storage_engine          = InnoDB
innodb_buffer_pool_instances    = 20     # Use 1 instance per 1GB of InnoDB pool size
innodb_buffer_pool_size         = 20G    # Use up to 70-80% of RAM & optionally check if /proc/sys/vm/swappiness is set to 0
innodb_file_per_table           = 1
innodb_flush_log_at_trx_commit  = 0
innodb_flush_method             = O_DIRECT
innodb_log_buffer_size          = 31M
innodb_log_file_size            = 512M
innodb_stats_on_metadata        = 0

#innodb_temp_data_file_path     = ibtmp1:64M:autoextend:max:20G # Control the maximum size for the ibtmp1 file
#innodb_thread_concurrency      = 10     # Optional: Set to the number of CPUs on your system (minus 1 or 2) to better
                                        # contain CPU usage. E.g. if your system has 8 CPUs, try 6 or 7 and check
                                        # the overall load produced by MySQL/MariaDB.
innodb_read_io_threads          = 64
innodb_write_io_threads         = 64

# MyISAM Settings
query_cache_limit               = 4M    # UPD
query_cache_size                = 48M   # UPD
query_cache_type                = 1

key_buffer_size                 = 48M   # UPD

low_priority_updates            = 1
concurrent_insert               = 2

# Connection Settings
max_connections                 = 100   # UPD

back_log                        = 512
thread_cache_size               = 100
thread_stack                    = 192K

interactive_timeout             = 180
wait_timeout                    = 180

# Buffer Settings
join_buffer_size                = 4M    # UPD
read_buffer_size                = 3M    # UPD
read_rnd_buffer_size            = 4M    # UPD
sort_buffer_size                = 4M    # UPD

# Table Settings
# In systemd managed systems like Ubuntu 16.04 or CentOS 7, you need to perform an extra action for table_open_cache & open_files_limit
# to be overriden (also see comment next to open_files_limit).
# E.g. for MySQL 5.7, please check: https://dev.mysql.com/doc/refman/5.7/en/using-systemd.html
# and for MariaDB check: https://mariadb.com/kb/en/library/systemd/
table_definition_cache          = 10000 # UPD
table_open_cache                = 10000 # UPD
open_files_limit                = 60000 # UPD - This can be 2x to 3x the table_open_cache value or match the system's
                                        # open files limit usually set in /etc/sysctl.conf or /etc/security/limits.conf
                                        # In systemd managed systems this limit must also be set in:
                                        # /etc/systemd/system/mysqld.service.d/override.conf (for MySQL 5.7+) and
                                        # /etc/systemd/system/mariadb.service.d/override.conf (for MariaDB)

max_heap_table_size             = 128M
tmp_table_size                  = 128M

# Search Settings
ft_min_word_len                 = 3     # Minimum length of words to be indexed for search results

# Logging
log_error                       = /var/lib/mysql/mysql_error.log
log_queries_not_using_indexes   = 1
long_query_time                 = 5
slow_query_log                  = 0     # Disabled for production
slow_query_log_file             = /var/lib/mysql/mysql_slow.log

[mysqldump]
# Variable reference
# For MySQL 5.7: https://dev.mysql.com/doc/refman/5.7/en/mysqldump.html
# For MariaDB:   https://mariadb.com/kb/en/library/mysqldump/
quick
quote_names
max_allowed_packet              = 64M

ofcourse you will need to tweak depending on your ram

@Prefinem using this query cuts time down to 0.5 seconds. instead of selecting all colloms we select the table we only need. can you change this in tracker?
Showing rows 0 - 74 (75 total, Query took 0.5823 seconds.)
select trackerUpdated from `torrents` where `trackerUpdated` is null limit 75

You need the infohash from the table, but I can make the changes.

oh i see my bad

No worries.  I added the correct select in the query. Hopefully it will
speed it up
On Thu, Feb 7, 2019 at 7:29 PM ash121121<EMAIL_ADDRESS>wrote:

oh i see my bad
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AlphaReign/scraper/issues/52#issuecomment-461659033,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACnvZrWw4Hl5gGXUXU60eEvE768eLoqaks5vLNLqgaJpZM4amw9r
.

Now its super speed :)

shall we do this also for getting records in loader?

Showing rows 0 - 751 (752 total, Query took 3.2976 seconds.)
select * from `torrents` where `searchUpdate` = false limit 1000

vs
Showing rows 0 - 419 (420 total, Query took 0.8087 seconds.)
select infohash from `torrents` where `searchUpdate` = false limit 1000

or do we need to query all collunms in loader?

In the loader, yes
On Thu, Feb 7, 2019 at 19:42 ash121121<EMAIL_ADDRESS>wrote:

or do we need to query all collunms in loader?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AlphaReign/scraper/issues/52#issuecomment-461661468,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACnvZrqYxbXsvcDdOuOciuieFPcbGWVqks5vLNYBgaJpZM4amw9r
.

if you choose to drop mysql. do you have an alternative idea?

Redid for fast storage and just elasticsearch for the data
On Thu, Feb 7, 2019 at 19:53 ash121121<EMAIL_ADDRESS>wrote:

if you choose to drop mysql. do you have an alternative idea?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AlphaReign/scraper/issues/52#issuecomment-461663407,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACnvZobDblm4uaKkIrToJGupNbQlMFgvks5vLNiIgaJpZM4amw9r
.

:) nice!
BTW i was checking how many outdated records i have and looks like the updater cannot keep up. or maybe because the tracker wasnt working before?
... 1808460
... 1809476
... 1829075

So lastnight i stopped crawling new torrents and let it update torrents with trackerupdate = null .
this morning im back were i started. i have 2 trackers running and still increasing.
x scraper > Torrents without Tracker: 355629           x
x[ 0] scraper          xx scraper > Torrents not in Search: 301                x
x                      xx scraper > Total Torrents: 2713681                    x
x                      xx scraper > Torrents without Tracker: 355704           x
x                      xx scraper > Torrents not in Search: 129                x
x                      xx scraper > Total Torrents: 2713821                    x
x                      xx scraper > Torrents without Tracker: 355833           x
x                      xx scraper > Torrents not in Search: 139                x
x                      xx scraper > Total Torrents: 2713966                    x
x                      xx scraper > Torrents without Tracker: 355994           x
x                      xx scraper > Torrents not in Search: 298                x
x                      xx scraper > Total Torrents: 2714100                    x
x                      xx scraper > Torrents without Tracker: 356121           x
x                      xx scraper > Torrents not in Search: 216

it appears overnight its not even querying the database to update trackers... and i havent changed anything.so strange on hetzner server and digital ocean


How to remove /canvas-ui from url?

If we want to expose the ui to root path then how we can do that properly what file I need to modify because change web.php causes issue so that its accessible from base domain instead of /canvas-ui?
Thanks

Hello @AsifMian! After you install Canvas UI with the canvas:ui Artisan command, you should have new routes populated in the routes/web.php file. From here, you can either update or remove the prefix.


Demo Files

https://github.com/SforAiDl/Neural-Voice-Cloning-With-Few-Samples/tree/whole_model#some-cloned-voices
Broken link for cloned voice demos.

Fixed


Expose ignore_eos_token on /generate and/or /v1/completions endpoints

Feature request
Hi! Seems like internally, the lorax gRPC server allows setting ignore_eos_token, but there's no way to set that param in the /generate endpoint of the router.
Motivation
This parameter is useful for benchmarking and being able to consistently and correctly compare runs.
Your contribution
Would need some guidance but could contribute it

Hey @martinbomio, thanks for the feature request! This should be a pretty simple one to add. I can definitely take a look soon if no one else picks it up first.

@tgaddair @jeffreyftang thanks for the quick addition! When could we expect to have this released? (we are using the docker image right now)

Hey @martinbomio, this should be available in the latest docker image to try out now :)
Let me know if you need a new version of the Python client to be pushed to PyPI.

@tgaddair that's great!
I tested passing ignore_eos_token to the openai compatible api /v1/completions and I still do not get the expected output tokens on some prompts. Is this expected? I can always switch to the /generate one and try

Hey @martinbomio , I think the current implementation didn't update the OpenAI endpoints as this param isn't native to their spec.
@jeffreyftang   what do you think about adding this param to the OpenAI endpoints as well?

I think the current implementation didn't update the OpenAI endpoints as this param isn't native to their spec.

Yep, that was my reasoning. However, happy to add to the OpenAI endpoints as well so long as we're not concerned with going beyond the OpenAI spec.

@jeffreyftang thanks for the quick PR! Would you mind sharing which base model you used for the example you posted in this PR https://github.com/predibase/lorax/pull/340? I am trying the latest version and I can't seem get the max tokens on certain prompts even if the ignore_eos_token is set

nvm, I can confirm it is working as expected!

Glad it's working! For posterity, I was using mistralai/Mistral-7B-Instruct-v0.1.


bulk-promotion-rules :   SQL syntax Error

curl: http://<IP_ADDRESS>:3000/api/bulk-promotion-rules
{ Code: "ERROR", Message: "near "?": syntax error", Details: null }
orchestrator  version : v3.1.2

cc @sjmudd , would you like to investigate?

@JoseFeng  I didn't reproduce the issue, could you provides more logs from journalctl -xu orchestrator

I can't reproduce the problem too.

@yangeagle  @jianhaiqing
step：
go get github.com/github/orchestrator

cd $GOPATH/src/github.com/github/orchestrator

git checkout  v3.1.2

go build -o orchestrator go/cmd/orchestrator/main.go

cp conf/orchestrator-sample-sqlite.conf.json orchestrator.conf.json

./orchestrator http

curl   http://ip:3000/api/bulk-promotion-rules
{ Code: "ERROR", Message: "near "?": syntax error", Details: null }


test::Context

Introduce test::Context - a helper resource manager for test set-up. It reduces boilerplate in tests.

Though I understand we're in test scope I'd advise to

not use one letter vars
maybe change test::Setup to test::Context (or DataContext) ?

Sure, done.


Dependent on tensorflow==2.0:  fix use_multiprocessing (see discussion)

Alright, so use_multiprocessing is causing an issue:
https://github.com/keras-team/keras/issues/11101
This might be fixed by switching from keras to tf.keras in tensorflow 2.0, so it's worth running a model and seeing if an error appears with use_multiprocessing set to True.
The commit in which use_multiprocessing was removed is:
e3ee4280dbfbc93493181a258479dc160787013b
and it should be trivial to add it back in with those changes highlighted.

@pgbrodrick :  You had mentioned use_multiprocessing today and I just ran into this. It might be worthwhile to wait for implementing multiprocessing until this is worked out.

Cool - I noticed an issue in tests today, that must be why.  Thanks for flagging.  It seems to run fine as long as it gets set to False, so I'll leave the option in there and just force it as False so we can change it later.


Add Cronet remote interop test to Jenkins

Include InteropTestsRemoteWithCronet into Jenkins MacOS Tests.
Masked testResponsesOverMaxSizeFailWithActionableMessage for it failing for unknown reason and it not being a key test. Will debug later.

Test failures:
gRPC_performance_pull_requests - #9499


refactor: using compiler base session in the kclvm parser to prepare for the parser error recovery.

1. Does this PR affect any open issues?(Y/N) and add issue references (e.g. "fix #123", "re #123".):

[ ] N
[x] Y

re: #420. Using compiler base session in the kclvm parser to prepare for the parser error recovery.

2. What is the scope of this PR (e.g. component or file name):

3. Provide a description of the PR(e.g. more details, effects, motivations or doc link):

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[x] Other

4. Are there any breaking changes?(Y/N) and describe the breaking changes(e.g. more details, motivations or doc link):

[x] N
[ ] Y

5. Are there test cases for these changes?(Y/N) select and add more details, references or doc links:

[x] Unit test
[x] Integration test
[ ] Benchmark (add benchmark stats below)
[ ] Manual test (add detailed scripts or steps below)
[ ] Other

All kclvm parser unit tests and grammar tests.

6. Release note

Please refer to Release Notes Language Style Guide to write a quality release note.
None

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

1 of 7 (14.29%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 90.347%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

compiler_base/session/src/lib.rs
1
7
14.29%

Totals

Change from base Build<PHONE_NUMBER>:
-0.2%

Covered Lines:
2106

Relevant Lines:
2331

💛  - Coveralls


Automatically select organization if there's only one

I think it's likely common for most people to only have one DevOps organization. Could this extension automatically select that single organization if there is only one?

Hi @carlin-q-scott, thanks for bringing this up! We are working on more prioritized issues at the moment but will get back to this one as soon as possible.

I forgot to mention that my IT admin has the sessions set to expire every 5 days, so I have to do this same process every week:

Click Log In
Click through OAuth questions
Close browser
Click "Select Organization"
Select my only organization


discard messages if there are pending mutations

This helps us to always respond promptly.
Also, upgrade to Salsa 0.8.0, which has a cleaned up API that features fewer footguns. =)

Works for me with 2018-10-31 nightly, at least. Mac OS X doesn't have the latest nightly so can't test that (!).

Definitely seems like a rustc bug. And it's using a pretty recent one:
$ rustc --version
rustc 1.31.0-nightly (de9666f12 2018-10-31)
$ rustup --version
rustup 1.14.0 (1e51b07cc 2018-10-04)
$ cargo --version
cargo 1.31.0-nightly (2d0863f65 2018-10-20)

OK, I updated this to not be wrong. Going to merge.


User Cancel out of FileOpenDialog fails

Thanks for this tool Jon. In Command2_Click event handler, if user hits Cancel in FileOpenDialog, then throws an error. I worked-around by swapping these two lines from:
.Show Me.hWnd
On Error Resume Next

to:
On Error Resume Next
.Show Me.hWnd

Thanks!

Oops yeah meant to have it on that line. Thanks.
Fixed in v1.0.2, all files and binaries updated.


AGFEO.AGFEODashboard version 1.8.21

[ ] Have you signed the Contributor License Agreement?
[ ] Have you checked that there aren't other open pull requests for the same manifest update/change?
[ ] Have you validated your manifest locally with winget validate --manifest <path>?
[ ] Have you tested your manifest locally with winget install --manifest <path>?
[ ] Does your manifest conform to the 1.0 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Missing: #27280 (comment)

I was not home earlier to change it sorry


Only Drupal 8 is loading upon "vagrant up" even when I edit my config.yml and drupal.make.yml

I have attached the yml files as text files
drupal.make.txt
config.txt

Inside ~/Sites/drupalvm, make sure you delete any existing drupal folder, then run vagrant up --provision again and it should reinstall a D7 site.

Thanks.


add coverage analysis for tests of the python package

I have created a .ini for running the coverage analysis and have gotten the report as a html file and its generated locally. Could you please suggest me what to do next?

Hello, can you create a new PR (referencing this issue) and we take it from there? The overall idea is to start monitoring the code coverage of our tests, add it to one of the existing workflows, and, later on, display some coverage badge in our README. I'm not sure about the details, but there seems to be many posts describing the process (e.g., https://nedbatchelder.com/blog/202209/making_a_coverage_badge.html). Thanks!

Hello,  I have created a pull request addressing the code coverage monitoring for our tests as referenced in Issue #221.
In the PR, I have implemented the necessary changes to start monitoring code coverage . Please let me know if this aligns with your expectations or if there are any additional changes or details needed.
below are the screenshots of the code coverages

using pytest

using tox


adding support for alert aggregation

Summary of the Pull Request
...

References
...

PR Checklist
By submitting this pull request, I confirm the following:
please fill any appropriate checkboxes, e.g: [X]

[ ] Closes #xxx
[ ] Requires documentation to be updated
[x] I've discussed this with core contributors already. If not checked, I'm ready to accept this work might be rejected in favor of a different grand plan. Issue number where discussion took place: #xxx
[ ] I have made only one major change in my proposed changes.
[ ] I have commented my proposed changes within the code.
[ ] I have tested my proposed changes, and have included unit tests where possible.
[x] It is compatible with the MIT License
[ ] I have squashed any insignificant commits. (git rebase)

Detailed Description of the Pull Request / Additional comments
A detailed description, screenshots (if necessary), as well as links to any relevant issues

Validation Steps Performed
A detailed description of thew validation steps
How does this PR accomplish the above
A detailed description (such as a changelog) and screenshots (if necessary) of the implemented fix
What documentation changes (if any) are needed to support this PR
A detailed list of any necessary changes

You must follow the template instructions. Failure to do so will result in your pull request being closed.

Happy to have a crack at this, just having issues at the moment trying to instantiate the ScheduledAlertProp class with the extra variables. As soon as I add another variable when instantiating I get OperationStopped: A non-collectible assembly may not reference a collectible assembly ☹
This will require some new enums and ScheduledAlertProp class properties:
Entities:
enum Entities {
    Account
    Host
    Ip
    Url
}

MatchingMethod:
enum MatchingMethod {
    All
    None
    Custom
}

ScheduledAlertProp properties:
    [object] $IncidentConfiguration
    [bool] $CreateIncident
    [bool] $GroupAlerts
    [bool] $ReopenClosedIncident
    [string] $LookbackDuration
    [MatchingMethod] $EntitiesMatchingMethod # All, None, Custom
    [Entities[]] $GroupByEntities

And something like this in the constructor:
    $this.IncidentConfiguration = @{
            'createIncident' =  if ($CreateIncident -eq $null ) { $true } else { $CreateIncident }
            'groupingConfiguration' = @{
                'enabled' = if ($GroupAlerts -eq $null) { $false } else { $GroupAlerts }
                'reopenClosedIncident' = if ($ReopenClosedIncident -eq $null) { $false } else { $ReopenClosedIncident }
                'lookbackDuration' = [AlertProp]::TimeString($LookbackDuration)
                'entitiesMatchingMethod' = $EntitiesMatchingMethod
                'groupByEntities' = if ($GroupByEntities -eq $null) { "" } else { $GroupByEntities }
            }
        }


Add GeoTIFF validator to list

Add GeoTIFF validator to list of docker images that shall be built.

Thanks!


Issue #2584: unacked message is not redelivered on time

Motivation
unacked message is not redelivered after setting ackTimeout, but it is actually redelivered after 2*acktimeout.
The main reason is in UnAckedMessageTracker.
    public void start(PulsarClientImpl client, ConsumerBase<?> consumerBase, long ackTimeoutMillis) {
        this.stop();
        timeout = client.timer().newTimeout(new TimerTask() {
            @Override
            public void run(Timeout t) throws Exception {
                if (isAckTimeout()) {   < === first timeout, it is false, because oldOpenSet is empty.
                    log.warn("[{}] {} messages have timed-out", consumerBase, oldOpenSet.size());
                    Set<MessageId> messageIds = new HashSet<>();
                    oldOpenSet.forEach(messageIds::add);
                    oldOpenSet.clear();
                    consumerBase.redeliverUnacknowledgedMessages(messageIds);
                }
                toggle();    < === toggle after timeout
                timeout = client.timer().newTimeout(this, ackTimeoutMillis, TimeUnit.MILLISECONDS);
            }
        }, ackTimeoutMillis, TimeUnit.MILLISECONDS);
    }

before first timeout, all messageId was added in CurrentSet, not in OldOpenSet, so isAckTimeout() is false, and redeliverUnacknowledgedMessages was not called at first timeout.
Modifications

move toggle() from behind if clause to before if clause.
add ut

Result
ut passed

retest this please

retest this please

run java8 tests

seems some ut need change, will look into it later.

retest this please


Error involving Bootsnap gem

Bootsnap version should be updated in gemfile as this version breaks with ruby 2.7

Hey @SVRourke! I was not able to recreate this issue but we've done some updating to the gems on this lab so hopefully that will fix this problem as well. I'm going to go ahead and close this issue 💙


Fix R2R seg fault by checking if GameState is null first

Noticed when GameController didn't exist but RobotComms did, it seg faults. Optional GameState is used without checking for nullptr first. Fixed up so there's a check.

Did you want to make it explicit what you do when you don't have one?

Did you want to make it explicit what you do when you don't have one?

Default for state is unknown_state, also if we want to do this then it should probably be done for all of the if statements rather than just this one, but then it seems like it'd be a bit cluttered when protobuf already does defaults? And out of scope of this PR.

Fair enough


Where can I find the default config file?

I've installed dynamic trees through a custom modpack, testing it both locally and on a server, and the config file for it generated empty in both places. (I didn't actually know there were configuration options until I saw #382, but the formatting there means I can't really copy/paste from it.) Is there somewhere I can find a clean copy of the default config so I can edit it? (I just need to turn off the slow fall, makes me queasy!) Thanks.

Aha, disregard -- I was looking in the config/dynamictrees/ directory but I see now there are also loose config files in the parent directory.

PLEASE! Where are you finf a congfig file?? Where is parent directory!!?? I dont understand sorry.. D: Please help! Im using 1.16.5 dynamic trees.


Update 06a-data-warehouse-load.md

CustomerID
Module: 00
Lab/Demo: 00
Fixes # .
Changes proposed in this pull request:

Hello @sophiexu
Changes were made in another commit, closing this PR.
Thank you,
Mihai


how to use this in Angular Ionic application

How to integrate this into Angular or Ionic application?

I also want to know. I tried doing it like this
angular.module('key', ['ionic'])
//.factory('udpfact')
.controller('keyctrl', ['$scope', 'chrome', function($scope, chrome) {
  //nothin
}])
  .run(function($ionicPlatform, $timeout) {
    alert("runnin");
    $ionicPlatform.ready(function() {
      alert('window.cordova: ' + Object.keys(window.cordova).join(', '));
      alert('window.cordova: ' + Object.keys(window.cordova.plugins).join(', '));
      alert(JSON.stringify(window.cordova));
      alert(JSON.stringify(window.cordova.plugins));

      alert("printed windows.cordova");
      $scope.sendcmd = function sendcmd(cmd) {
        alert("cmd: " + cmd);
        chrome.sockets.udp.create({}, function(socketInfo) {
          var socketId = socketInfo.socketId;
          chrome.sockets.udp.send(socketId, cmd,
            '<IP_ADDRESS>', 2390, function(sendInfo) {
              alert("sent " + sendInfo.bytesSent);
          });
        });
      };
  });
});

But it only shows one plugin (keyboard).

Hi,
I was able to use this in my Ionic application by referencing chrome.sockets.udp..  Don't use 'window.cordova.plugins' .

I've added this plugin with ionic plugin add, then I need to load any js or is automatic? Because if I try to run a sample app, I always get that chrome is not defined... What's wrong?
Thanks...

Are you checking that chrome is defined in the $ionicPlatform.ready callback?  It should be in your module.run function.

Hi digitialcatnip,
Thank you for your info. I am using chrome.sockets.udp. The plugin lines were to print a list of plugins loaded. It did not show chrome. Oh, so are you saying chrome should be passed to run and not controller?

Nice... Waiting for deviceready before using the plugin works for me!
Thanks.

I used the following command to add the plugin to my project:
https://github.com/MobileChromeApps/cordova-plugin-chrome-apps-sockets-udp
Then I created a controller with a function below:
app.controller('ctrl3', function ($scope) {
  $scope.udpClient = function () {
    console.log("success");
    var data, addr, port;
    port = 5000;
    addr = "<IP_ADDRESS>";
    data = "hello";
    chrome.sockets.udp.create({}, function(socketInfo) {
      var socketId = socketInfo.socketId;
      chrome.sockets.udp.send(socketId, data,
        addr, port, function(sendInfo) {
          alert("sent " + sendInfo.bytesSent);
        });
    });
    $scope.Message = "Button Clicked";
  };
});

And I get an error:
Cannot read property 'udp' of undefined
    at Scope.$scope.udpClient

Am I doing something wrong?
Cordova CLI: 6.3.1
Ionic Framework: 1.3.1
Ionic CLI: 1.7.16
OS: Windows 7 Pro
Node: 4.4.7
Below is the whole code for app.js

var app = angular.module('ctrl1', ['ionic']);
app.run(function ($ionicPlatform) {
$ionicPlatform.ready(function () {
if (window.cordova && window.cordova.plugins.Keyboard) {
// Hide the accessory bar by default (remove this to show the accessory bar above the keyboard
// for form inputs)
cordova.plugins.Keyboard.hideKeyboardAccessoryBar(true);
//cordova.plugins.
//window.plugins.cordova-plugin-chrome-apps-sockets-udp.create()
  // Don't remove this line unless you know what you are doing. It stops the viewport
  // from snapping when text inputs are focused. Ionic handles this internally for
  // a much nicer keyboard experience.
  cordova.plugins.Keyboard.disableScroll(true);
}
if (window.StatusBar) {
  StatusBar.styleDefault();
}

console.log("Hello");

});
});
app.controller('ctrl2', function ($scope) {
$scope.dateToday = new Date();
$scope.x = 5;
$scope.y = 6;
});
app.controller('ctrl3', function ($scope) {
$scope.udpClient = function () {
console.log("success");
var data, addr, port;
port = 5000;
addr = "<IP_ADDRESS>";
data = "hello";

chrome.sockets.udp.create({}, function(socketInfo) {
  var socketId = socketInfo.socketId;
  chrome.sockets.udp.send(socketId, data,
    addr, port, function(sendInfo) {
      alert("sent " + sendInfo.bytesSent);
    });
});

$scope.Message = "Button Clicked";

};
});

@nutella Can you please paste your code here for reference?

You can look at:
https://github.com/nutella/ionic-cordova-chrome-udp-example

NICE! :)
Thanks!!!


[Doc]  Support chunking with Shared subscription

Search before asking

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

What issue do you find in Pulsar docs?
https://pulsar.apache.org/docs/2.11.x/concepts-messaging/#chunking
limt ：
second item：Chunking is only available for the exclusive and failover subscription types.
but now support "Shared subscription"
@momo-jun
What is your suggestion?
?
Any reference?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

@startjava is this a feature request? Do you want to enable the chunking feature when using the shared subscription?

2.11 version Chunking support key_shared，
but doc is old ：Chunking is only available for the exclusive and failover subscription types.

@startjava Thanks for pointing it out, I'll update the doc.


Access control

This is a major update to some rest internals. It add jsx support, access control, and restful validations out of the box.


Boneyard support

Boneyard is now recognized if /* and */ stand alone in their lines as we can see in the examples. Placed comment about this in the code and also the docs.

I created the .nuspec file the nuget package also.


Add ability to parameterize S3 Saves

Describe your changes and why you are making these changes
This PR allows a user to parameterize the filepath of an S3 save like this (as shown in the demo):
demo = client.resource("Demo")
s3 = client.resource("test_s3")

table = demo.sql("Select * from hotel_reviews")
dir_name_param = client.create_param("dir_name", default="test")
file_name_param = client.create_param("file_name", default="hotel_reviews")
s3.save(table, "{dir_name}/{file_name}", format="parquet")

client.publish_flow("Test S3 Save", artifacts=[table])

Considerations:

Using {} interpolation by name is a bit tricky, because it is possible for there to be multiple explicit parameters with the same name. I decided to resolve this by picking the one that was most recently created. Implicit parameters are excluded from this search for obvious reasons.

Limitations and bugs:

We're currently filtering out these parameterized filepaths from S3 due to time constraints and complexity of the problem. See the slack link in this task: https://linear.app/aqueducthq/issue/ENG-3015/get-list-saved-objects-to-work-with-parameterized-s3-saves
We should show the resolved filepath in the UI sidesheet: https://linear.app/aqueducthq/issue/ENG-3016/show-the-resolved-names-for-the-spec-fields-on-the-workflow-sidesheet

Related issue number (if any)
ENG-2998
Loom demo (if any)
https://www.loom.com/share/1e616042edd843e9a7845bd619f696c7
Checklist before requesting a review

[ ] I have created a descriptive PR title. The PR title should complete the sentence "This PR...".
[ ] I have performed a self-review of my code.
[ ] I have included a small demo of the changes. For the UI, this would be a screenshot or a Loom video.
[ ] If this is a new feature, I have added unit tests and integration tests.
[ ] I have run the integration tests locally and they are passing.
[ ] I have run the linter script locally (See python3 scripts/run_linters.py -h for usage).
[ ] All features on the UI continue to work correctly.
[ ] Added one of the following CI labels:

run_integration_test: Runs integration tests
skip_integration_test: Skips integration tests (Should be used when changes are ONLY documentation/UI)

@likawind added a data resource test for it.


Fix formatting in docs (intro.md)

PR type: doc improvement
Summary
Minor clean-up of documentation: notebook intro's formatting was broken.
PR checklist

[X] I ran the black formatter (make format)
[X] I locally tested that the tests pass (make check-all)

Release notes
Fully backwards compatible: yes
Commit message (for release notes):

Fix formatting in documentation.

Test failures are unrelated: cloudpickle just released 1.5.0, which breaks tensorflow_probability. :facepalm:

Fixed in tensorflow_probability: https://github.com/tensorflow/probability/pull/993/files


StoreCopier now warms up transformer i.e. allows it to invoke batched…

… storeKeyConverter calls

Codecov Report

Merging #1035 into master will decrease coverage by 0.05%.
The diff coverage is 54.54%.

@@             Coverage Diff              @@
##             master    #1035      +/-   ##
============================================
- Coverage     68.37%   68.32%   -0.06%
+ Complexity     4749     4748       -1
============================================
  Files           378      378
  Lines         30047    30058      +11
  Branches       3831     3834       +3
============================================
- Hits          20546    20537       -9
- Misses         8469     8488      +19
- Partials       1032     1033       +1

Impacted Files
Coverage Δ
Complexity Δ

.../main/java/com.github.ambry/store/StoreCopier.java
54.12% <0%> (-1.54%)
10 <0> (ø)

...hub.ambry.messageformat/ValidatingTransformer.java
90.24% <0%> (-2.26%)
5 <0> (ø)

...om.github.ambry.replication/BlobIdTransformer.java
98.5% <85.71%> (-1.5%)
11 <2> (+2)

...rc/main/java/com.github.ambry.store/BlobStore.java
81.45% <0%> (-3.23%)
67% <0%> (-4%)

...b.ambry.network/BlockingChannelConnectionPool.java
63.47% <0%> (-2.18%)
8% <0%> (ø)

...in/java/com.github.ambry.store/BlobStoreStats.java
71.05% <0%> (-0.22%)
101% <0%> (-1%)

...src/main/java/com.github.ambry.commons/BlobId.java
87.39% <0%> (+0.42%)
60% <0%> (+1%)
:arrow_up:

.../java/com.github.ambry.router/DeleteOperation.java
93.61% <0%> (+1.41%)
48% <0%> (+1%)
:arrow_up:

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 f3fe628...62a3efc. Read the comment docs.


Enhance E2E tests to use mesheryctl to deploy Meshery

Current Behavior
The tests in https://github.com/meshery/meshery-kuma/.github/workflows/e2etests.yaml use Kubernetes manifests to deploy Meshery .
Desired Behavior
Edit the workflow to use mesheryctl in deploying meshery.
Implementation

Acceptance Tests

Mockups

Contributor Guides and Resources

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma
🙋🏾🙋🏼 Questions: Layer5 Discussion Forum and Layer5 Community Slack

Thank you, @Zanda256

@tapanhalani, interested?

I was thinking this should be closed aswell. If the core workflow uses mctl to deploy meshery. @Revolyssup , what do you think?


Code Engine backend is broken with one of the latest PR

Code Engine Backend is broken due to call to from lithops.storage.utils import create_runtime_meta_key which was removed in https://github.com/lithops-cloud/lithops/commit/998710a3a19277e32eb9aea785ba35e6504b9242
@JosepSampe can you fix it please?

Fix in #442


Hi guys,

Hi guys,
The docs seems still is not updated.
Originally posted by @BrightRan in https://github.com/github/docs/pull/531#issuecomment-721493071

Everything is being addressed in the referenced PR, so I'm going to close this issue out.


Adds documentation

A rendered version of this markdown file can be seen here:
https://github.com/ebryn/jsonapi-ts/tree/joel/documentation
Sections:

[x] Repo description
[x] Feature (brief description)

[x] Operation-driven API
[x] Declarative style for resource definition
[x] CRUD database operations
[x] Transport layer integrations
[x] Relationships and sideloading
[x] Error handling
[x] User/Role presence authorization
[x] Extensibility

[x] Getting started
[x] Resources

[x] What is a resource?
[x] Declaration syntax
[x] Accepted types

[x] Operations

[x] What is an operation?
[x] Operation types

[x] get
[x] add
[x] update
[x] remove

[x] Transport layers

[x] jsonApiKoa

[x] Usage
[x] Converting operations into HTTP endpoints
[x] URL mapping

[x] Processors

[x] What is a processor?
[x] The OperationProcessor class
[x] How does an operation gets executed?
[x] Controlling errors while executing an operation
[x] Extending the OperationProcessor
[x] The KnexProcessor class
[x] Extending the KnexProcessor

[x] Authorization

[x] Defining a User resource
[x] Implementing a User processor
[x] Using the @Authorize decorator
[x] Using the IfUser() conditional helper
[x] Front-end requirements

[x] The JSONAPI Application

[x] What is a JSONAPI Application?
[x] Referencing types and processors
[x] Using a default processor

Resolves #66.

In both examples of the custom OperationProcessor there's a type error:
https://github.com/ebryn/jsonapi-ts/tree/joel/documentation#how-does-an-operation-gets-executed

and in the Controlling Errors example:
https://github.com/ebryn/jsonapi-ts/tree/joel/documentation#controlling-errors-while-executing-an-operation

https://github.com/ebryn/jsonapi-ts/tree/joel/documentation#extending-the-operationprocessor-class

Also:
https://github.com/ebryn/jsonapi-ts/tree/joel/documentation#extending-the-operationprocessor-class

and

Finnally in the last example:
https://github.com/ebryn/jsonapi-ts/tree/joel/documentation#extending-the-knexprocessor-class

The property relationships is missing.

https://github.com/ebryn/jsonapi-ts/tree/joel/documentation#defining-an-user-resource

In the user creation example, the user class doesn't seems to have the needed attributes

https://github.com/ebryn/jsonapi-ts/tree/joel/documentation#controlling-errors-while-executing-an-operation
There's a fs.readFileSync(file)
fs is undefined, and you are already importing it above.

https://github.com/ebryn/jsonapi-ts/tree/joel/documentation#implementing-a-session-processor

Neither hash nor Log are being used in the example, we could remove those.

Missing prop relationships

https://github.com/ebryn/jsonapi-ts/tree/joel/documentation#implementing-a-session-processor
We could add anoter notice letting the user know that under no circumstances storing the password in the db as plain text is a good idea, and that this is just an example.

https://github.com/ebryn/jsonapi-ts/tree/joel/documentation#using-the-authorize-decorator
As far as I can see in the jsonapi-ts code, the current implementation of the Authorize decorator is applicable only to "get", "add", "update", "remove" operations.
We should let the user know that, I think.

Comments will be taken care of in future PRs.


WIP: Using repr to make it easy to understand externally

Issues: #8

[x] channel
[x] guild
[x] user
[x] emoji
[x] invite
[x] role
[x] stage
[x] sticker
[x] voice
[x] webhook

Please resolve conflicts.

Thanks!


GlideRecord.query has potential memory leak or unnecessary list accumulation

I'd like to use pysnc to iteratively (with fixed space/memory) consume a very large ServiceNow table. Based on what I saw in the docs and looking at the source code of PySNC, it appears GlideRecord.query performs pagination and uses an iterator to yield results.
However, I'm using pysnc==1.1.7 and experienced about 9% increase in memory use for every 5000 records queried. It's as if pysnc is storing all records in memory even after I've consumed or iterated past those records, which isn't very standard Python iterator/generator behaviour.
I'd like the issue resolved in way where the script below runs without crashing and consumes only static/fixed memory, perhaps only storing/yielding rows of a single "batch".
Here's a minimum repro script
#!/usr/bin/env python3

import pysnc
import dataclasses
import logging
import os
import typing as t

@dataclasses.dataclass(frozen=True)
class Filter:
    key: str
    val: str

    def __str__(self) -> str:
        return f"{self.key}={self.val}"

@dataclasses.dataclass(frozen=True)
class TableQuery:
    table: str
    filters: t.List[Filter]
    limit: int

BATCH_SIZE = 5000

def query(client: pysnc.ServiceNowClient, params: TableQuery) -> t.Iterator[pysnc.GlideRecord]:
    gr = client.GlideRecord(params.table, BATCH_SIZE)
    gr.display_value = False
    gr.limit = params.limit
    for filter_ in params.filters:
        gr.add_query(filter_.key, filter_.val)
    gr.query()
    for r in gr:
        yield r

def consume_stream(records: t.Iterator[pysnc.GlideRecord]) -> None:
    """ dummy consumer function kept short for brevity
        in reality there is extra business logic before using csv.DictWriter """
    for r in records:
        r.serialize()

def main() -> None:
    # Setup
    logging.basicConfig(level=logging.DEBUG)
    env = os.environ.copy()
    username, password = env["SERVICE_NOW_TOKEN"].split(':', 1)
    client = pysnc.ServiceNowClient(env["SERVICE_NOW_URL"], (username, password))

    # Go!
    params = TableQuery("sys_user", [Filter("active", "true")], 10**6)
    stream = query(client, params)
    consume_stream(stream)

if __name__ == "__main__":
    main()

Script output
$ ./pysync_query-table-oom.py
DEBUG:urllib3.connectionpool:Starting new HTTPS connection (1): ${REDACTED}.service-now.com:443
DEBUG:urllib3.connectionpool:https://${REDACTED}.service-now.com:443 "GET /api/now/table/sys_user?sysparm_query=active%3Dtrue%5EORDERBYsys_id&sysparm_display_value=false&sysparm_limit=5000&sysparm_offset=0&sysparm_exclude_reference_link=true&sysparm_suppress_pagination_header=true HTTP/1.1" 200 None
DEBUG:urllib3.connectionpool:https://${REDACTED}.service-now.com:443 "GET /api/now/table/sys_user?sysparm_query=active%3Dtrue%5EORDERBYsys_id&sysparm_display_value=false&sysparm_limit=5000&sysparm_offset=5000&sysparm_exclude_reference_link=true&sysparm_suppress_pagination_header=true HTTP/1.1" 200 None
DEBUG:urllib3.connectionpool:https://${REDACTED}.service-now.com:443 "GET /api/now/table/sys_user?sysparm_query=active%3Dtrue%5EORDERBYsys_id&sysparm_display_value=false&sysparm_limit=5000&sysparm_offset=10000&sysparm_exclude_reference_link=true&sysparm_suppress_pagination_header=true HTTP/1.1" 200 None
DEBUG:urllib3.connectionpool:https://${REDACTED}.service-now.com:443 "GET /api/now/table/sys_user?sysparm_query=active%3Dtrue%5EORDERBYsys_id&sysparm_display_value=false&sysparm_limit=5000&sysparm_offset=15000&sysparm_exclude_reference_link=true&sysparm_suppress_pagination_header=true HTTP/1.1" 200 None
DEBUG:urllib3.connectionpool:https://${REDACTED}.service-now.com:443 "GET /api/now/table/sys_user?sysparm_query=active%3Dtrue%5EORDERBYsys_id&sysparm_display_value=false&sysparm_limit=5000&sysparm_offset=20000&sysparm_exclude_reference_link=true&sysparm_suppress_pagination_header=true HTTP/1.1" 200 None
DEBUG:urllib3.connectionpool:https://${REDACTED}.service-now.com:443 "GET /api/now/table/sys_user?sysparm_query=active%3Dtrue%5EORDERBYsys_id&sysparm_display_value=false&sysparm_limit=5000&sysparm_offset=25000&sysparm_exclude_reference_link=true&sysparm_suppress_pagination_header=true HTTP/1.1" 200 None
DEBUG:urllib3.connectionpool:https://${REDACTED}.service-now.com:443 "GET /api/now/table/sys_user?sysparm_query=active%3Dtrue%5EORDERBYsys_id&sysparm_display_value=false&sysparm_limit=5000&sysparm_offset=30000&sysparm_exclude_reference_link=true&sysparm_suppress_pagination_header=true HTTP/1.1" 200 None
DEBUG:urllib3.connectionpool:https://${REDACTED}.service-now.com:443 "GET /api/now/table/sys_user?sysparm_query=active%3Dtrue%5EORDERBYsys_id&sysparm_display_value=false&sysparm_limit=5000&sysparm_offset=35000&sysparm_exclude_reference_link=true&sysparm_suppress_pagination_header=true HTTP/1.1" 200 None
Killed

Condensed monitoring output: CSV size vs System Memory use
$ while true; do ps auxw && wc -l /root/Downloads/2024-06-09_sn-sys-users.csv; sleep 1; done

USER       PID %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
root         1  0.0  0.1 230208  6992 pts/0    Ss   08:48   0:01 /mnt/lima-rosetta/rosetta /usr/bin/bash
root      5300  0.0  0.1 229888  6224 pts/1    Ss   21:17   0:01 /mnt/lima-rosetta/rosetta /usr/bin/bash
root      6698  4.0  9.9<PHONE_NUMBER>44 pts/0   S+   21:44   0:02 /mnt/lima-rosetta/rosetta /root/.local/share/virtualenvs/app-4PlAip
0    /root/Downloads/2024-06-09_sn-sys-users.csv
...
USER       PID %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
root         1  0.0  0.1 230208  6992 pts/0    Ss   08:48   0:01 /mnt/lima-rosetta/rosetta /usr/bin/bash
root      5300  0.0  0.1 229888  6224 pts/1    Ss   21:17   0:01 /mnt/lima-rosetta/rosetta /usr/bin/bash
root      6698  2.8 10.2<PHONE_NUMBER>56 pts/0   S+   21:44   0:02 /mnt/lima-rosetta/rosetta /root/.local/share/virtualenvs/app-4PlAip
5000 /root/Downloads/2024-06-09_sn-sys-users.csv
...
USER       PID %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
root         1  0.0  0.1 230208  6992 pts/0    Ss   08:48   0:01 /mnt/lima-rosetta/rosetta /usr/bin/bash
root      5300  0.0  0.1 229888  6272 pts/1    Ss   21:17   0:01 /mnt/lima-rosetta/rosetta /usr/bin/bash
root      6698  3.2 19.0 1054144 764044 pts/0  S+   21:44   0:04 /mnt/lima-rosetta/rosetta /root/.local/share/virtualenvs/
9999 /root/Downloads/2024-06-09_sn-sys-users.csv
...

USER       PID %CPU %MEM    VSZ   RSS TTY      STAT START   TIME COMMAND
root         1  0.0  0.1 230208  6992 pts/0    Ss   08:48   0:01 /mnt/lima-rosetta/rosetta /usr/bin/bash
root      5300  0.0  0.1 229888  6232 pts/1    Ss   21:17   0:01 /mnt/lima-rosetta/rosetta /usr/bin/bash
root      6698  3.9 63.2 2845000 2539212 pts/0 S+   21:44   0:15 /mnt/lima-rosetta/rosetta /root/.local/share/virtualenvs/app-4PlAip
root      7303  0.0  0.1 262124  6272 pts/1    R+   21:50   0:00 /usr/bin/ps auxw
34999 /root/Downloads/2024-06-09_sn-sys-users.csv

Good point, I could drop records on yield, as that would be the expected python behavior. I would likely keep .next() from dropping, and require a new .query() if you needed to start over.

Thank you for the quick response!
Does a major version upgrade imply a longer turnout time (bundling other breaking changes together) compared to regular fixes?

No, just indicates the API changed enough that it could break how people use it.

@naushadh what I've done is wrong, here, but I'm trying to see if I can avoid breaking changes. What do you think about this syntax instead:
def query(client: pysnc.ServiceNowClient, params: TableQuery) -> t.Iterator[pysnc.GlideRecord]:
    with client.GlideRecord(params.table, BATCH_SIZE) as gr:
        gr.display_value = False
        gr.limit = params.limit
        for filter_ in params.filters:
            gr.add_query(filter_.key, filter_.val)
        gr.query()
        for r in gr:
            yield r

Hey @vetsin, I don’t mind using an alternate syntax to achieve the right behavior.
However, it might be challenging to reconcile the difference between streaming (new) and rewind (current) behavior in the same GlideRecord class.
Perhaps the two consumptions styles require 2 separate GlideRecord implementations?
Another approach is to update the client to support a “query” function which yields an iterator of GlideRecords. Current users get current behavior with GlideRecord query (and a deprecation warning), and new users get fixed behavior from the client.

Writing the following down for notes/myself:
I believe, in pythonic terms, this is what is what would truly be expected:
gr = client.GlideRecord(table)
gr.query()

for r in iter(gr):
   assert r.sys_id
for r in iter(gr):
   assert r.sys_id

while gr.next():
  assert gr.sys_id

A possibility would be query() to return an iterable that's different from self (perhaps even just iter(self)?), however the PEP does require that __iter__ return self

Been swamped so haven't got this working -- some work to be done on how .query() behaves with non-rewinding

python's built-in range keeps producing values as is, but wrapping it in iter makes it exhaustible.
>>> x = range(1,5)
>>> list(x)
[1, 2, 3, 4]
>>> list(x)
[1, 2, 3, 4]
>>> y = iter(x)
>>> list(y)
[1, 2, 3, 4]
>>> list(y)
[]
>>> list(x)
[1, 2, 3, 4]

Another possible API is also specifying to the GlideRecord rewind=False in the constructor. Existing users will get rewind=True as a default, and new users can opt-in to the single use behaviour.

released into 1.1.8 but didn't document the change -- mostly to get dependency updates out

@naushadh i presume by the lack of complaints it works-as-expected from you? If so i'll document it into the next release.


specific motion time

I want the animation was done in a moment.
like $(#***).animated({}, 0);
How am i gonna set the variables.

First point on FAQ: https://github.com/chenglou/react-motion#faq


Triples not passed into subsequent CONSTRUCT queries

The homepage says:

In Tarql mappings with multiple CONSTRUCT queries, the triples generated by previous CONSTRUCT clauses can be queries in subsequent WHERE clauses to retrieve additional data. When using this capability, note that the order of OPTIONAL and BIND is significant in a SPARQL query.

But since v1.1, this no longer works. In v1.1 we introduced streaming, and the constructed triples are now written directly to disk, and are no longer added to an in-memory graph where they could be queried by subsequent CONSTRUCTs.

The code where it works (now only called from tests):  TarqlQueryExecution.exec(Model)`
The code that is actually executed: TarqlQueryExecution.execTriples()

At a minimum, the documentation (index.md) needs to be updated to reflect this. But it's also a useful feature, so maybe it should be made possible again?

Use case from Johan W. Klüwer: “What I had in mind was to use aggregates in construct queries […], triples from the first construct could be subject to aggregate querying in later constructs.”

@cygri It's very useful to be able to use triples from the SAME row, which won't break streaming.


Included server and database name in tab

SQL Operations Studio Version:0.23.6 (Windows)

When working across a number of different servers and databases it would be helpful to have the server and database name in the tab and it's tooltip e.g. SQLQuery2 - server\database

@stevieg402 why did you close this? it's a valid suggestion 😄

I noticed, after I'd raised the issue, that the server and database name are visible towards the right hand side of the status bar and thought that this may suffice. But, whilst this helps somewhat, it would be useful to also have the server and database name in the tab and it's tooltip in a number of scenarios (e.g. ALT+TAB through open tabs, hovering over inactive tabs) so happy to re-open.

Yes, this is definitely needed, so when working with multiple servers we don't need to activate the tab to see which server it's pointing at.

also see this issue https://github.com/Microsoft/sqlopsstudio/issues/327

This feature is available in the current insiders build by enabling the sql.showConnectionInfoInTitle option.


Query is not taken into account

?param=23, for instance, will resolve to http://localhost:2000/?param=23#/login.

Brainfart. This is actually normal...


How can i add?!

How can i add setTimeout to the minions
like Delay Splitting delay eject food delay mouse follow movements etc
how ;-;

duplicate.


unsupported file format

Looks like they changed something with the upgrade to Chauvet 2.12.28_beta
When I run the code below (or try to convert to PDF) I get this stack trace
Any help would be awesome!
C:\Users\jumper\AppData\Roaming\Python\Python312\Scripts\supernote-tool.exe analyze "D:\20230525_165407.note"                                                                                Traceback (most recent call last):   File "<frozen runpy>", line 198, in _run_module_as_main   File "<frozen runpy>", line 88, in _run_code   File "C:\Users\jumper\AppData\Roaming\Python\Python312\Scripts\supernote-tool.exe\__main__.py", line 7, in <module>   File "C:\Users\jumper\AppData\Roaming\Python\Python312\site-packages\supernotelib\cmds\supernote_tool.py", line 203, in main     args.handler(args)   File "C:\Users\jumper\AppData\Roaming\Python\Python312\site-packages\supernotelib\cmds\supernote_tool.py", line 49, in subcommand_analyze     metadata = sn.parse_metadata(f, policy=args.policy)                ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^   File "C:\Users\jumper\AppData\Roaming\Python\Python312\site-packages\supernotelib\parser.py", line 62, in parse_metadata     raise exceptions.UnsupportedFileFormat('unsupported file format') supernotelib.exceptions.UnsupportedFileFormat: unsupported file format

I'm not a beta user so I can't confirm, but adding an option --policy=loose may work around that issue.  The option allows parser try to parse for files with unknown signature.
Example:
supernote-tool convert --policy=loose -a -type pdf file.note out.pdf

I have gotten a A6X2 Nomad and started to analyze the latest file format Chauvet 3.14.27.  There are some changes so I need to modify my code to support it.
I believe that A6X (& A5X) and A6X2 will receive common updates, so once support for A6X2 is complete, I think it will naturally be possible to support the next update for A6X.

I just added this and it seems to work, will let you know for sure.  Thanks!

It worked, thanks!


Splitting Terminals

Description of the new feature/enhancement
There could be a feature allowing to split terminals, rather than the current tabs functionality like your other products like Visual Studio Code or as is Linux Terminator so that user can see other terminals at the same time without having to change the tab.

Proposed technical implementation details (optional)

Wha? You can totally do this today. Use alt+shit+plus or alt+shift+minus and read our docs.


Treeview disposed on tabhide

We have a MultiSelectTreeView inside a TabItem, and we've found that switching to another tab triggers the Unload event on the control (MultiSelectTreeView.cs:688). This in turn calls a series of dispose events which ultimately result in unsubscribing the mouse event handlers (BorderSelectionLogic.cs:67), which kills the drag-selecting.
It looks like this code is inherited from the original TreeViewEx, but it also seems to have been removed from that code base at some point. I recommend the same here.

Changed in 1e8a18e6, can you please verify the fix?

Sorry, there was a problem with it. Please use 6ecee8f4.

Yep, this does it, thanks!

Alright. Thank you for the report and investigation! TreeViewEx has indeed changed a lot. And also due to my improvements, large parts of the code have been altered and don't look similar anymore.