Welp Here we are again... | Python code being flagged by Antivirus as Immedient Virus

Text is selfexplaining
Python code being flagged by Antivirus as Immedient Virus by using curses. pls find another option.

Text is selfexplaining Python code being flagged by Antivirus as Immedient Virus by using curses. pls find another option.
Sorry for late response, have you tried disabling antivirus? Also I don't have any bad code in my program. And I don't even know to what option I should switch to.

Text is selfexplaining Python code being flagged by Antivirus as Immedient Virus by using curses. pls find another option.

Sorry for late response, have you tried disabling antivirus? Also I don't have any bad code in my program. And I don't even know to what option I should switch to.

Its okay that it is late its 2 am for me BTW.
So curses library shows up as an virus. This is not a me problem I could replicate this tomorrow in a live virtual machine.   The problem insists on curses source code and yea you may have old python so it might work for ya

Text is selfexplaining Python code being flagged by Antivirus as Immedient Virus by using curses. pls find another option.

Sorry for late response, have you tried disabling antivirus? Also I don't have any bad code in my program. And I don't even know to what option I should switch to.

Its okay that it is late its 2 am for me BTW.
So curses library shows up as an virus. This is not a me problem I could replicate this tomorrow in a live virtual machine. The problem insists on curses source code and yea you may have old python so it might work for ya

Wait what do you mean I have old version of python? I have python 3.8.6 and that's fully your antivirus. And what do antivirus you have?


How are Value Types exposed in JS?

Both overview from this repo and WebAssembly's GC v1 JS API don't mention how Value Types are exposed in JS. In examples they look like Well-known Intrinsic Objects, but they can't be Well-known Intrinsic Objects because the are surely not web compatible. So it's not clear where do uint8, int8, any come from.

True, that part isn't in the explainer yet. That's because we haven't settled on a solution—Typed Objects might end up being exposed as a builtin module, or in a way that's similar to Math.


fix(udf): use object.setattr for 3.0.0 compatibility

cc @tswast This brings ibis-bigquery up to date with changes needed to work with 3.0.0, which I'm about to kick off a release of.

Thanks! I can get a release prepared.


Remove setters from collection properties

We have removed most of the setters from the collection properties in this PR. This PR removes the remaining setters from the public collection properties.
Note: A setter is only generated for nullable collections. For more details, please see this conversation.

Thank you, Shivangi! After this change, the only collection properties left that have a setter are the following:

MessageCreationOptions.Attachments
AssistantCreationOptions.Metadata
AssistantModificationOptions.Metadata
MessageCreationOptions.Metadata
MessageModificationOptions.Metadata
RunModificationOptions.Metadata
ThreadCreationOptions.Metadata
ThreadModificationOptions.Metadata
VectoreStoreCreationOptions.Metadata
VectorStoreModificationOptions.Metadata

Paging in @trrwilson because we talked about something related a while ago. The properties above are getting a setter from the generator because they are explicitly specified as nullable in the TypeSpec (which itself comes from the swagger spec). Now, in basically every part of the REST API that I have encountered so far, setting a nullable property to null does not imply anything special and it's equivalent to not setting it at all. Assuming null doesn't mean anything special here either, I wonder if we should add a simple change to customize these properties to remove the setter in favor of consistency and a simpler API.


Changing reference from Netbox custom MultipleChoiceField subclass and adding Polish currency.

Hello,
I changing reference from Netbox custom MultipleChoiceField subclass to django_filter MultipleChoiceFilter. Netbox custom MultipleChoiceField subclass from utilities.forms.fields was removed in version 3.6 Netbox https://github.com/netbox-community/netbox/pull/13614. In addition, I added the Polish currency Polish Zloty (PLN) to the currency array.
Regards

I created the issue #91 to handle this.


Add Padding method make distinct from Margin

We use margin to define the space between the edge of the chart and the next closest item: the credit line the legend, the axis etc.
We use padding to define the space between the margin and the chart area.
We also have a method of extraPadding so that we can add or subtract from the default padding without modifying it.

Is this some code you can share? I ran into a race condition when trying to get this to work originally. Essentially if the user specifies an outer width or height for the chart then the inner chart dimensions are determined by subtracting the margin from the outer values. If the padding is configured later then the margin values are now invalid and I don't have a mechanism inside d4 to invalidate those values based on a later change. If you've cracked that nut i'd be happy to integrate that change. Additionally, the larger issue of making certain variable changes dependent on other variables is something I need to solve in general especially as we get into more complex charting types.

The way we implemented it leaves the chart size calculations untouched (for better or for worse)
We added the methods to the chart
https://github.com/yanofsky/d4/blob/master/src/base.js#L753-L771
then we changed the way scales are calculated https://github.com/yanofsky/d4/blob/master/src/builders/scales.js#L14-L20
(we also changed the way axes position
https://github.com/yanofsky/d4/blob/master/src/features/y-axis.js#L76-L85
but we've customized d3 to draw axes differently)

@yanofsky I've been giving this some thought and I think the way to handle this is to add a reactive programming component to base which will allow you to assign data dependencies to variables. For example if the inner width of the chart area is effected by the padding and margin then any change to those should cause a recalculation. Something like this:
d4.when(['padding', 'margin'], function (padding, margin) {
  // recalculate the available visual space for the chart.
});

This proposed approach is based off of @curran's model library https://github.com/curran/model

That looks like a good solution, but shouldn't it be applied to chart not d4?.
It would also be convenient if you could update a single property of margin/padding using this method e.g.
chart.when(['padding', 'margin'], function (padding, margin) {
  // recalculate the available visual space for the chart.
});

chart.when("paddingLeft", function (paddingLeft) {
  // recalculate the available visual space for the chart.
});

chart.set({
  margin: {top:10, right: 10, bottom: 10, left: 10},
  padding: {top:0, right: 0, bottom: 0, left: 0}
});

chart.set({
  paddingLeft: 10
})

console.log(chart.padding())
// {top:0, right: 0, bottom: 0, left: 10}

@yanofsky agreed, my thought was that it would happen transparently inside base when a user used one of the accessors like chart.width(100) or chart.marginLeft(20). This would trigger an event listener inside the base itself to recalculate all the dependent properties. This keeps the API consistent with what it is today.
With that said there is nothing to say we could not expose a when() function at the chart level to allow users to hook in their own dependencies where they so choose e.g.
chart.when(['data'], function(data){});

awesome!

yeah now i just need to write it ;-)

Here's an example that might help
http://bl.ocks.org/curran/015d34d6d3d562877e51 .
Also, here's "when" formulated as an extension to Backbone (called "wire"
in this example)
https://github.com/curran/phd/blob/dac07e2e8c38da7343645d7a07ec17a762120ea0/prototype/src/wire.js
.
Good luck with this work, looking forward to seeing how it goes.
On Thu, Mar 12, 2015 at 8:00 AM, Mark Daggett<EMAIL_ADDRESS>wrote:

yeah now i just need to write it ;-)
—
Reply to this email directly or view it on GitHub
https://github.com/heavysixer/d4/issues/35#issuecomment-78496805.


include typescript typings

@souporserious would you be open to including a .d.ts file in the NPM package for typescript consumers? i've been writing it locally and would be happy to contribute.
there'd be some additional maintenance cost to keep the definitions updated as the API evolves, but I'll be happy to help out if you mention me in the future!

Yeah for sure :) I don't know much about typescript, but I'd be happy to accept a PR for it.

cool i'll send it up once i'm happy with what i've got!
thanks for speedy replies 😃


Title: Improve the UI of the theme toggle button. I want to contribute under GSSOC' 24.

Which pre-existing JavaScript project will you be working on?
I have worked on many JS projects and would like to contribute in this one as well.
Describe your enhancement/improvement
the button looks quite big and a bit unprofessional. So I want to improve the UI of the button rendering it a professional and appealing look.
Anything you need to say to the Reviewers
No response
Add any other context or relevant screenshots about the enhancement here.
No response

@ParnaRoyChowdhury777 Thank you for providing your insight but it is not required as of now.


Question: how to change link preview with custom text?

Hi I want to preview the link with my own text in a way that if the user clicks on the preview text to open the browser?

The Autolink component exposes many different ways of customizing the behavior. I'd recommend looking at the renderLink prop if you want to customize how the link itself renders but I'm not really sure what you're trying to do.
Closing as this isn't really an issue with the library. If there is missing documentation in the README, please let me know.

@joshswan thanks for your answer, I meant something like this:

so you are saying that this isn't supported by default?

That functionality should be implementable using renderLink and/or custom matchers depending on where this text is coming from and what logic you want to apply to transform the links.
Given the content of that text though, I'm assuming this is text you control, in which case using this library is overkill since you can just wrap the links yourself. Generally, the main purpose of the Autolink component is to discover links in user-generated content.

Okay, I understand, thanks for your answer, have a great day


Bug: reset suggestion UI

Reproduce this:

Mark some text,
click "Add Comment"
interact with the comment Dialog, check "Include suggested change"
click cancel
repeat 1. and .2.

What should happen:

The from/to fields should not be visible (works not)
If "Include suggested change" is checked, "From" should contain the currently selected text.

What happens instead:

Though the checkbox for "Include suggested change" is unchecked, the from/to fields are visible if they have been activated when calling the dialog previously.
If "Include suggested change" is checked, "From" does not  contain the currently selected text, but is blank

@jdittrich you taking this on?

hmm, dunno – I assume a reset structure is already in place. You are probably way quicker. But: it would accelerate/ease it if you could point out the current place where you think the resets should/may happen (I don’t get the structure yet, partly because extensions are a bit tricky to debug (or I am doing it wrong))

Fixed by a82f171


stop Adam after certificate receive

Stop Adam after receive of root-certificate.pem
#266
Signed-off-by: Petr Fedchenkov<EMAIL_ADDRESS>

I agree -- at this point we may as well put a tad more work to fix it for real @giggsoff


Add example for drawing samples from a gcm

Did you decide against using this also as an example for a "ad-hoc" data generator?

Still wondering about this? As discussed offline, this could even just be a sentence saying that this can be used as a synthetic data generator.

I modified the text further already to make this clear or what else do you have in mind?

I modified the text further already to make this clear or what else do you have in mind?

I thought a simple sentence for inspiration: "This can be used as a data generator for synthetic data with known ground truth."


License violation issue as indirect dependency node-forge is now under GPL

Hi,
I am using angular & http-proxy-middleware in my project & whitesource gave us a license violation alert for an indirect dependency node-forge which is under GPL now. Is there an alternate for the same?
Thanks
Akanksha Gaur

Probably a false alert: https://github.com/jfromaniello/selfsigned/issues/34#issuecomment-553094099

node-forge is not used in this project


Theme file

Here's my theme file for your model in case useful. Had to add .txt to upload in GH. --Daniel
permissions.thm.txt

Thank you! I'll drop it in my repo!


Conflation in boston doesn't look like it is working

Description
Rapid buildings are being suggested ontop of already created buildings in Boston.
Screenshots

https://rapideditor.org/rapid#background=MassGIS_2021_Aerial&datasets=fbRoads,msBuildings&disable_features=boundaries&id=w845711633&map=20.17/42.37451/-71.10560
Version
2.1.1
What browser are you seeing the problem on? What version are you running?
Chrome v119.0
The OS you're using
mac
Steps to reproduce
Go to the link I pasted above and see that rapid buildings are being displayed even where buildings already exist
The browser URL at the time you encountered the bug
https://rapideditor.org/rapid#background=MassGIS_2021_Aerial&datasets=fbRoads,msBuildings&disable_features=boundaries&id=w845711633&map=20.17/42.37451/-71.10560
The auto-detected useragent string for your browser (leave blank if you're manually filling this form out)
Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36

It does seem to be working now, so I'm going to close this issue..
We also have #602 , which would provide a way to let the user when conflation isn't working.
(I don't know what could cause it - maybe our service being unable to talk to OSM?)


Calendar has weekdays, and is aligned to weekday columns

Closes #12
Pic worth 1,000 words...   Note the orange color is my app's theme, not this lib
Before

After

Looks absolutely great.
Made me fix a padding problem I had too.
Thanks alot


Where is the documentation source located?

I wanted to contribute a fix to the documentation but was not able to find the sources of the documentation located at https://quobix.com/vacuum
I'm sorry if I missed or overlooked it, please let me know whether it is possible to contribute to the docs directly.

Hi, unfortunately the docs source are not public as they are a part of my personal website. They were originally, but I've been building non-free / OSS content that is mixed in with it.
I can update them for you however?

Thanks for explaining. I had just seen a typo somewhere while reading up on some of the docs.
And then I wondered if I had really overlooked the doc sources. But now tbh I don't even remember the typo so all good 😅
Thanks for creating great tools like vacuum!

@daveshanley  you should leverage https://cspell.org/ as CLI as part of your publishing pipeline , it works as a charm


Fix :: Prevent Entity Providers from eliminating Users and Groups if synchronisation fails

Hey, I just made a Pull Request!
Fixes #22222
If the API used for user and group synchronisation is down or the requests to it fail for whatever reason when the catalog sync happens Backstage will clear the database of the previously synchronised entities. This PR attempts to fix this by only applying the mutation if there are users or groups present.
:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[ ] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[ ] All your commits have a Signed-off-by line in the message. (more info)

Agreed with the above; better to throw eagerly and give up, rather than trying to check anything after the fact.
Also remember to DCO sign all of your commits - there are instructions in the build check and the bot comment above.

Agreed with the above; better to throw eagerly and give up, rather than trying to check anything after the fact.
Also remember to DCO sign all of your commits - there are instructions in the build check and the bot comment above.

Looking at the code I would think if there was an error in the API it would stop the execution as I cannot see anything catching any errors from this API call. queryWithPaging is what actually triggers the request, as far as I can tell, and there is no catch there. Wouldn't the error propagate upward and stop applyMutation from happening with the code being as it is?

Am I mistaken thinking this would swallow any errors and cause the issue you experienced during the GitHub instability?
https://github.com/backstage/backstage/blob/ba1ae038f06e3a3ff2a21a0a6497f15af713fe58/plugins/catalog-backend-module-github/src/providers/GithubMultiOrgEntityProvider.ts#L295-L297
All this does is log an error, the mutation happens after this and would be working with an empty array/object

backstage/plugins/catalog-backend-module-github/src/providers/GithubMultiOrgEntityProvider.ts

Indeed it would! I was looking at the GithubOrgEntityProvider where there is no need for such a loop! So this issue must be specific to the multi-org setups, like our own. Could the fix be as simple as getting rid of that try catch?

Yes, I think removing the try/catch would be a good idea 👍

Oh sorry, I was too fast with the approve - typescript is unhappy due to the usages of any.

It doesn't look like I introduced any anys 😆

Oh sorry, I was too fast with the approve - typescript is unhappy due to the usages of any.

Fixed! There were some TS definitions missing for the mock data of the tests.


Roleplaying dimension support

I do want to keep having related-tables work in excel, which means I need FKs, which means tables. Can I somehow get them to work without duplicating the table? Or is that the most straightforward way?

I spawned a separate project for this, though I'd love to combine them someday: https://github.com/sdoubleday/scdRpGenerator


Empty properties are not allowed, or are they? ¯_(ツ)_/

test/arm-ttk/testcases/
Issue Details
My offer passed all tests, as shown here:
                                                                                Validating arm\createUiDefinition.json
  CreateUIDefinition
    [+] Allowed Values Should Actually Be Allowed (154 ms)

    [+] CreateUIDefinition Must Have Parameters (9 ms)
    [+] CreateUIDefinition Should Have Schema (26 ms)
    [+] Credential Confirmation Should Not Be Hidden (78 ms)
    [+] Handler Must Be Correct (5 ms)
    [+] Location Should Be In Outputs (7 ms)
    [+] Outputs Must Be Present In Template Parameters (7 ms)
    [+] Parameters Without Default Must Exist In CreateUIDefinition (20 ms)
    [+] Password Textboxes Must Be Used For Password Parameters (39 ms)
    [+] Textboxes Are Well Formed (35 ms)
    [+] Tooltips Should Be Present (35 ms)
    [+] Usernames Should Not Have A Default (34 ms)
    [+] VMSizes Must Match Template (39 ms)
Validating arm\mainTemplate.json
  deploymentTemplate
    [+] DeploymentTemplate Schema Is Correct (6 ms)
    [+] IDs Should Be Derived From ResourceIDs (114 ms)
    [+] Location Should Not Be Hardcoded (43 ms)
    [+] ManagedIdentityExtension must not be used (6 ms)
    [+] Min And Max Value Are Numbers (6 ms)
    [+] Outputs Must Not Contain Secrets (21 ms)
    [+] Parameters Must Be Referenced (15 ms)
    [+] Parameters Property Must Exist (6 ms)
    [+] PublicIP DNS Should Not Contain Concat (6 ms)
    [+] ResourceIds should not contain (26 ms)
    [+] Resources Should Have Location (8 ms)
    [+] Secure String Parameters Cannot Have Default (3 ms)
    [+] Template Should Not Contain Blanks (13 ms)
    [+] VM Images Should Use Latest Version (3 ms)
    [+] VM Size Should Be A Parameter (57 ms)
    [+] Variables Must Be Referenced (12 ms)
    [+] Virtual Machines Should Not Be Preview (7 ms)
    [+] adminUsername Should Not Be A Literal (58 ms)
    [+] apiVersions Should Be Recent (96 ms)
    [?] artifacts parameter (25 ms)
        ENV:SAMPLE_NAME is empty - using placeholder for manual verification: 100-blank-template

    [+] providers apiVersions Is Not Permitted (5 ms)

However, when it came time to go through the marketplace to live offer, the human reviewer @Olgaian  stated:
please remove empty properties

I assert that if a human reviewer is going to flag something, then the tests should flag it also.
Repro steps (if necessary, delete otherwise)

Create mainTemplate.json that has an empty property:

      {
         "type": "Microsoft.Storage/storageAccounts",
         "apiVersion": "2019-06-01",
         "name": "[variables('storageAccountName')]",
         "location": "[parameters('location')]",
         "sku": {
            "name": "[variables('storageAccountType')]"
         },
         "kind": "Storage",
         "properties": {}
      },

run the tests
observe they do not fail.

Note that this test seems to address this issue:
https://github.com/Azure/azure-quickstart-templates/blob/master/test/arm-ttk/testcases/deploymentTemplate/Template-Should-Not-Contain-Blanks.test.ps1
Line 18 - 32 on this test seem to address this.

I think the human is incorrect in this case - the test has exceptions for places where we know we need them.


Using getContext as a decorator breaks static variables

It seems as though using the @getContext() form doesn't propagate static variables to the resulting component type.
const { Component, PropTypes } = React;
const { render } = ReactDOM;
const { getContext } = Recompose;

// Uncomment this to show breaking behavior
// @getContext()
class App extends Component {
  static foo = "Hello";
  render() {
    return (

{App.foo}

    );
  }
}

// This still works though
// App = getContext()(App);

render(
  <App />,
  document.getElementById('app')
);

Expected output is:

Hello
, but instead output is

JSBIN showing the behavior:
https://jsbin.com/buqeqac/edit?html,js,console,output
I'm not sure if this affects the decorator form of other helpers, I only use @getContext() so far. I realize decorators aren't a standard thing yet, but your docs say they can be used as such.

The getContext() returns a new function. In fact, it shouldn't work in both snippets.
The snippet with App = getContext()(App); just works because of the way that Babel handles class transpilation.
I think that a function should only by used as decorator when it returns the given target. Maybe is better to remove the recommendation to use recompose functions as decorators. What do you think @acdlite?


Anchor points

Demo code for how multiple points can be provided to specify an anchor dynamically

https://github.com/user-attachments/assets/63625341-431a-40a1-b618-bbe13daaf822


this error occured when training finished

when I finish learning 100%, I got this error, and even if I extract mesh and open the mesh with the mesh lab, the file is weird
i trained with ns-train neus-facto --pipeline.model.sdf-field.inside-outside False --vis viewer --experiment-name neus-facto-fox nerfstudio-data

Hi, I didn't test the code in windows. Could you use tensorboard or wandb to visualize the training process (use --vis wandb)? If the log image looks good in wandb then the extracted mesh should be also good. If the error only happens at the end of training, then you could simply set a slightly large number of training iterations such as default+1 then the trained model will be saved to the disk and you can use it to extract the mesh.

i trained with ns-train neus-facto --pipeline.datamanager.train-num-rays-per-batch 1000 --vis wandb nerfstudio-data --data data/custom/fox2

https://wandb.ai/ju805604/sdfstudio/runs/zkes0blx?workspace=user-ju805604
eroor occured

Hi, I didn't test the code in windows. Could you use tensorboard or wandb to visualize the training process (use --vis wandb)? If the log image looks good in wandb then the extracted mesh should be also good. If the error only happens at the end of training, then you could simply set a slightly large number of training iterations such as default+1 then the trained model will be saved to the disk and you can use it to extract the mesh.

Sorry
https://wandb.ai/ju805604/sdfstudio/workspace?workspace=user-ju805604
I changed it public


Update forge version

Can this get an update to a more recent version of forge for 1.12.2? The author of heavy inventories is attempting to add compatibility with your mod but is unable to due to how NBT data is being handled.

particularly how nbt compounds are checked. example: https://github.com/copygirl/WearableBackpacks/blob/9b542812b766160613386ab676bd1a43d4351468/src/main/java/net/mcft/copy/backpacks/misc/util/NbtUtils.java#L141
newer forge versions use NBTTagCompound#hasNoTags()

Could a developer of the mod please give some feedback?

This is not a Forge update issue, these are MCP mappings. If you want to use Wearable Backpacks in your workspace you need to use deobfCompile on an obfuscated jar to ensure it is mapped correctly for your environment.

Fwiw, it is whatever environment you are in that is outdated, not ours. Wearable Backpacks was updated to the stable (and final) channel for 1.12 MCP back in September, which made the migration from hasNoTags to isEmpty: https://github.com/copygirl/WearableBackpacks/commit/141bd8ca14df3cbd01ea9ca173cdb60776095b1c


More nullable changes, Fix IITemHandler capability

Fixed some nullable issues with ItemStacks.
Also fixed the IItemHandler capability wrapper, which was using the wrong class (resulting in a constructor not found exception which led to a null capability being returned)
Fixes #4303

weird, I don't remember closing this.  Must have been a misclick. :)  Thanks a ton.

Self-merged, using isEmpty() instead of ItemStack.EMPTY equivalencies. Really appreciate your help!

:D No problem
Derp.. That method also exists


Validate evidence strings from internal parsers

We should run JSON validation after the evidence strings were generated in evidence_datasource_parsers

This has been solved with PR #103.


SSHClient disconnect never completes

On both iOS and Android, disconnect never completes.
This happens even for a simple use case of:
SSHClient _client;
await _client.connect();
await _client.disconnect();
I'm on Flutter Channel dev, v1.1.8, on Mac OS X 10.14.3.

Hi,
Sorry for the late reply. Is this resolved by your PR? I merged it in v0.0.3.
Thanks,
Qian

No.

+1 The disconnect never happens.

Thank you for the feedback.
I forgot that disconnect method doesn't return result from native code.
https://github.com/shaqian/flutter_ssh/blob/master/ios/Classes/SshPlugin.m#L455
The dart code should not be async. I will fix this in next version.

Any updates on this? @shaqian I don't like to just leave it there without disconnection.
I don't understand that .m file, I would like to help

I never connect server by SFTP , and the ip , host ,username,pass are all correct.Some tiem later, the app  creshed and exit.

Sorry for the delay. The fix is in v0.0.4.
Thanks,
Qian


Drag-n-Drop

Find better alternative to the drag and drop directive, or maybe a better implementation of what is currently there. These changes need to update the Trello board, as well.

Can I start this?

@sandaruny certainly, thanks! i haven't worked on this project in about a year, but i do remember improving upon the drag-n-drop functionality slightly


test_stripe: Add legacy remote server schedule upgrade E2E test.

Fixes:

Screenshots and screen captures:

Self-review checklist

[ ] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[ ] Explains differences from previous plans (e.g., issue description).
[ ] Highlights technical choices and bugs encountered.
[ ] Calls out remaining decisions and concerns.
[ ] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[ ] Each commit is a coherent idea.
[ ] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[ ] Visual appearance of the changes.
[ ] Responsiveness and internationalization.
[ ] Strings and tooltips.
[ ] End-to-end functionality of buttons, interactions and flows.
[ ] Corner cases, error conditions, and easily imagined bugs.

I'll be including a commit removing # nocoverage comments in this PR.

Merged, thanks @prakhar1144!


Fixes #4364 Removed dependency to fix build for Instrumentation Tests

Description
Removed powermock-mockito-release-full dependency from androidTestImplementation to fix build for Instrumentation Tests
Fixes #4364
Tests performed
Tested ProdDebug on Google Pixel 2 with API level 29

Codecov Report

Merging #4369 (448b9d7) into master (f8a8f92) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master    #4369   +/-   ##
=========================================
  Coverage     10.21%   10.21%
  Complexity      469      469
=========================================
  Files           342      342
  Lines         13104    13104
  Branches       1072     1072
=========================================
  Hits           1338     1338
  Misses        11698    11698
  Partials         68       68

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 f8a8f92...448b9d7. Read the comment docs.


When a table name is a sql keyword, the model is not generated

When the table name is groups, the command simply throws out a SQL error.
Here is the error:
Exception trace:
1   PDOException::("SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'groups' at line 1")

This has been fixed in the latest release


chore: pre-build server to speed up docker buildx

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 24.478%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3732

Relevant Lines:
16097

💛  - Coveralls

cc @chrisns this should speed up builds and prevent some strange issues with armv7 builds causing actions to freeze


Add versioning to kserver

When kserver is called from kompile or krun, version info should be exchanged as part of the protocol to ensure no inconsistencies between the two.  This is useful for devs with multiple versions of K, users who are upgrading between versions of RV-Match, etc.  The current latest commit hash can be used as in the version message.

Added to K Wish List.

This should be relatively easy to do. We already serialize the commit hash to the jar so it can be accessed by java, we just need to also serialize it somewhere it can be read by the k shell script, and then pass it in as an environment variable.
The one thing I'm not 100% sure on, though, is, what kind of behavior do you have in mind if it's not a match? Just because it's not a match doesn't mean the boot sequence knows how to find a kserver that does match, so unless you have a specific proposal, I don't know how to do much more than "fall back to not running kserver"

I will get to this when I get back from vacation after Thanksgiving.

Intended behavior should be something like a warning about running multiple incompatbile copies of the same kserver which will slow down the operation, instructions on how to kill the kserver, and proceeding without kserver.


Started adding code-owners.

I want to start to organise our code-base a bit more.
Code-owners allows us to have people assigned to certain parts of gluon-ts automatically.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Job PR-351/1 is complete.
Docs are uploaded to http://gluon-ts-staging.s3-accelerate.dualstack.amazonaws.com/PR-351/1/index.html


Storyshot throws invalid or missing MediaQuery

I'm using a storybook addon called storyshots to automate and link my snapshots with stories. For a component that uses Media component the following error is thrown: Invalid or missing MediaQuery! which in turn doesn't run a snapshot on this component.

Can you post a code snippet that shows the usage of Media in the component?

Sure, here's an overview of how it's used in of components
      <S.Wrapper>
          // ...
        <S.Center>
          <Media maxWidth={smallScreen}>
            <S.LogoWrapper path={LogoSmall} />
          </Media>
          <Media minWidth={smallScreen}>
            <S.LogoWrapper path={Logo} />
          </Media>
        </S.Center>
        <S.Actions>
          <Media minWidth={mediumScreen}>
            // ...
          </Media>
          // ...
          <S.SearchButton>
            <Media minWidth={smallScreen}>
              <S.Text>SEARCH</S.Text>
            </Media>
            <Icon name="search" size={24} />
          </S.SearchButton>
        </S.Actions>
      </S.Wrapper>

@jwm0 Just as an experiment before I dig into it, can you try the new beta release and see if that fixes it?

@contra just did, still getting the same error unfortunately. Also, I found that mount from enzyme also throws the exact same error, whereas shallow seems to work. It looks like a general issue with react-test-renderer. Can you confirm you're getting the same results?

@jwm0 Can you make a test repo with the issue? I haven't used react-test-renderer before so it would be quicker to debug with one.

@contra okay, the issue was on my end - not providing aliases for imports considering device width constants and looks like Media kept throwing this error when no/invalid width prop was provided.


fix: Remove print call that logs entire chat content

Pull Request Checklist
Note to first-time contributors: Please open a discussion post in Discussions and describe your changes before submitting a pull request.
Before submitting, make sure you've checked the following:

[x] Target branch: Please verify that the pull request targets the dev branch.
[x] Description: Provide a concise description of the changes made in this pull request.
[ ] Changelog: Ensure a changelog entry following the format of Keep a Changelog is added at the bottom of the PR description.
[x] Documentation: Have you updated relevant documentation Open WebUI Docs, or other documentation sources?
[x] Dependencies: Are there any new dependencies? Have you updated the dependency versions in the documentation?
[x] Testing: Have you written and run sufficient tests for validating the changes?
[x] Code review: Have you performed a self-review of your code, addressing any coding standard issues and ensuring adherence to the project's coding standards?
[x] Prefix: To cleary categorize this pull request, prefix the pull request title, using one of the following:

BREAKING CHANGE: Significant changes that may affect compatibility
build: Changes that affect the build system or external dependencies
ci: Changes to our continuous integration processes or workflows
chore: Refactor, cleanup, or other non-functional code changes
docs: Documentation update or addition
feat: Introduces a new feature or enhancement to the codebase
fix: Bug fix or error correction
i18n: Internationalization or localization changes
perf: Performance improvement
refactor: Code restructuring for better maintainability, readability, or scalability
style: Changes that do not affect the meaning of the code (white-space, formatting, missing semi-colons, etc.)
test: Adding missing tests or correcting existing tests
WIP: Work in progress, a temporary label for incomplete or ongoing work

Changelog Entry
Description
The Open WebUI logs print out the first chat exchanged between a user and the chatbot.
This is because there is a vanilla print() call in the method that generates tags for chats.
If users are using Open WebUI with proprietary information, this print statement means the chat content will end up in whatever log system they are using alongside Open WebUI.
Removed

Removed an unnecessary print call

Security

No longer unexpectedly logging chat exchanges

Additional Information
I'm happy to go through and remove/replace other print() methods  with log.debug
Screenshots or Videos

[Attach any relevant screenshots or videos demonstrating the changes]

Closing in favour of 7da29b6798186f9a7aa21d718086aff9772d6229, thanks!

hi @tjbck - could you also remove the print() call in def generate_chat_tags()? The linked commit only removes the print() call inside def generate_search_query()

@krishanbhasin-px it's been removed long time ago, check the dev branch!


Flaky tests: CanCloseStreamMethodEarly, CanInvokeClientMethodFromServer, etc

I set:

CanCloseStreamMethodEarly
CanInvokeClientMethodFromServer
CanSendAndReceiveMessage
CanStopAndStartConnection
ExceptionFromStreamingSentToClient
ServerThrowsHubExceptionIfHubMethodCannotBeResolved
ServerThrowsHubExceptionIfSteamingHubMethodCannotBeResolved
ServerThrowsHubExceptionOnHubMethodARgumentTypeMismatch
ServerThrowsHubExceptionOnStreamingHubMethodArgumentTypeMismatch_messagepack_LongPolling_dynamic
StreamDoesNotStartIfTokenAlreadyCanceled_json_WebSockets_dynamic

to ignore for now because they fail on OSX sometimes. It should be investigated and re-enabled once fixed.

Ignore on the CI?

Sorry, was delayed in pushing the change. I create the issue before the PR so that I can put the issue number in the Skip reason.

Looking at the test failures on the CI, the logs aren't very helpful because they were run before https://github.com/aspnet/Logging/pull/760, which means the logs for one framework overwrote the logs for a different framework. We should do some runs with the latest logging to see if that fixes things. I'll see about doing that.

Sounds good to me, I'll let you know if there's a new one to examine.

https://github.com/aspnet/SignalR/issues/1214

With the sync context disabled I'm seeing SSETransportStopsSendAndReceiveLoopsWhenTransportStopped fail repeatedly.

That test is fixed in https://github.com/aspnet/SignalR/pull/1279
But it has a null-ref on netcoreapp2.0

Ah hah, I disabled it for now.

I'm seeing this one fail randomly. I might have an idea why but I'll look into it after more refactoring.
ConnectionCanSendAndReceiveDifferentMessageSizesWebSocketsTransport_4096032
I have logs, it looks we might just need to extend the timeout.

In general I push back when people say "just increase the timeout". Why is it that things aren't ready when we think they should be? Could we just await the completion of some call or process instead? Is the timeout in this case just unexpectedly short?

I'm going to take these

I removed the timeout entirely. I'm going to close this issue @ryanbrandenburg, let me know if you're still seeing failures.

Failures continue.

@ryanbrandenburg all of them?


WIP - Keycloak pkce using angular-oauth2-oidc

angular-oauth2-oidc

@eformat  - I'll smash this in now and have a play with it

@eformat  - I'll smash this in now and have a play with it


ci: fix RCE vulnerability in file overwrite

Critical RCE Vulnerability:
An attacker can exploit this vulnerability by uploading files named scripts/size-report.ts, which will overwrite the existing file and subsequently execute the injected code.
This flaw grants the attacker the ability to execute arbitrary code in a secure environment that has write permissions to both pull-requests and issues.
Special thanks to @RedYetiDev

It was an honor to help :-)

out of curiosity what is wrong with ${{ github.event.number }} ?

size-data and size-report are two distinct workflows. The former runs in an untrusted environment (lacking permissions to write issue/PR comments), while the latter operates in a trusted environment with the necessary permissions.
To link these workflows, we need to pass the target PR number as an argument to the trusted environment via artifacts.

Sure yes the environments are different, but to me it looks like size-report still knows about the pr environment which triggered it because it still thinks it is a pr
    if: >
      github.event.workflow_run.event == 'pull_request' &&
      github.event.workflow_run.conclusion == 'success'

in my mind replacing
number: ${{ steps.pr-number.outputs.content }}

with
number: ${{ github.event.number }}

should have the same end result without having to upload the pr number in workflow a) and then downloading it in workflow b)

Did you try it? IIRC, it doesn't work. According to GitHub Docs, Webhook event payload is workflow_run object

I haven't that's why I first asked if i missed anything. I can give it a shot later, the webhook event payload also seems to have an array of objects named pull_requests these objects seem to have the pr number.. i wonder why it is returning an array though

As far as I know, pull_request[0] with the pull_request event will always be the submitted PR.


Adding build:es task

Fixes #665
Adding build:es npm script task for generating hyperapp.es.js (optimised ES6 module) for direct consumption via ES6 module import in modern browser like Chrome.
Note: babel-minify currently does not generate sourcemap file. In future if it does such mangling that need sourcemap, then it can be added accordingly at that point of time.

Thanks, @kumarabhishek. To clarify again, the use case here is using Hyperapp as an ES module directly from a CDN?

666 :smiling_imp:

@jorgebucaran Yes. You got it right.

@kumarabhishek And what is the disadvantage of using https://unpkg.com/hyperapp@1.2.5/src/index.js?module?

CDN:

Localhost Example:

If you see above pictures, notice the size. CDN(2.6 KB) and localhostexample(1.8 KB). Hope this clarify the motivation. I know it may sound crazzy to run for another 0.8 KB, but in future it may increase.

@kumarabhishek Would you consider this less relevant if unpkg had a way to minify modules on the fly? Just curious about your use case.

...but in future it may increase

Hyperapp? Maybe, but not for much.

@jorgebucaran If unpkg does what babel-minify is doing on the fly then its awesome and we can drop this change. In near future many browser will start supporting ES6/ES7/ES8 natively and whole concept of bundling and chunking that we do today using webpack and similar tools may go away. So its very usefull if we have some way to consume Hyperapp in a modern way (ES6 way). I know the current implementation of Hyperapp is also in ES5 but to save more space and keep up with the 1 KBish starting motto there may be need to write some part in ES6/ES7. My idea is just to bring to our notice that Hyperapp consumption as ES6 module is future and we should consider it. Which ever way we do is pure our choice (babel-minify or unpkg or something else).

Related issue directly in unpkg  - https://github.com/unpkg/unpkg/issues/57

@jorgebucaran So what is that we conclude on this matter ? Is it worth discussing further or shall we close this for now and revisit in future if required ?

@kumarabhishek Let's leave it open for now! 😄

Hey, just wanted to ping that jsdelivr offers minification on the fly. But it doesn't work for this file. Probably because of uglify+es6 trouble?

<EMAIL_ADDRESS>https://github.com/jsdelivr/jsdelivr/issues/18043

@oskarrough But it doesn't work for this file. Probably because of uglify+es6 trouble?

I think so too. 😓

@kumarabhishek, you do realize that the benefits of http2 will make the need to minifying a library the size of Hyperapp superfluous? Import of ES6 modules in browsers is still not entirely ironed out, but when it is done, you'll be able to easily import whatever part of Hyperapp you need without having to use a bundled version. That's still down the road. Therefore I don't think it warrants making changes to Hyperapp's build process to support this yet. My two cents.

@rbiggs You are correct. Http2 is really a nice thing but it is usually done at page load when server pushes more files as part of document response. In scenario where we need on demand loading upon page navigation, dynamic ES6 module import is what will be more relevant.

#665
Thank you, @kumarabhishek, but I am not going to take this.
People who want a non-UMD minified bundle can simply minify Hyperapp in their own dev environment.


Add NPM TOKEN env to publish workflow

Codecov Report

Merging #85 (c777353) into main (76b1727) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##              main       #85   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           19        19
  Lines          170       170
  Branches        26        26
=========================================
  Hits           170       170

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 76b1727...c777353. Read the comment docs.


ztest: add a delay between tests

Some testsuites dump lots of output very fast where some systems are not
able to capture the complete output from the tests. Add a slight delay
between each test in the suite.
Fixes #50455
Signed-off-by: Anas Nashif<EMAIL_ADDRESS>

I tested it on CAVS platform five times. And it would pass stably. So this PR will solve the issue of missing log.


Fix KFAC when running vmc-molecule with single ion

I realized my previous PR #114 introduced a bug where KFAC would throw an error when running vmc-molecule on a system with only one ion.
On debugging it seems like kfac_jax has a bug where it ends up dividing by zero and throwing a floating point error when a layer of the network involves only 1x1 kernels, so that every block of the block-diagonal part of KFAC for that layer is simply a scalar.  This was exactly what was happening in our exponential envelope code.
I didn't see an immediate way to fix it through the KFAC tagging, partly because I don't deeply understand how the kfac_jax library works. However, that particular layer doesn't need to be written that way and it can just as easily be written as an element-wise multiplication followed by a sum over the final axis.  As such I fixed the bug by adding an ElementWiseMultiply Module and using this approach instead.
PTAL @nilin or @JiahaoYao

Hi @ggoldsh , thanks for having an fast fix on this, and the fix looks good to me.

Thanks @JiahaoYao  for the review!


Viewing manage account shows a "Feed Source ID does not exist" error

 Issue by demory
Tuesday May 08, 2018 at 14:39 GMT
Originally opened as https://github.com/catalogueglobal/datatools-ui/issues/122

This happens on the current test server with my<EMAIL_ADDRESS>account

 Comment by landonreed
Tuesday May 08, 2018 at 15:42 GMT

This is due to subscriptions in the user profile to feed sources that no longer exist.  Rather than showing the error message, the requests should just silently fail.


Update installation instructions for Mac and Linux

The README.md says:
If you are using the MacOS or Linux operating system you can follow the instructions in wsl_setup.md starting at the Installing_Miniconda section.
But the section #installing-miniconda does not exist. Also, the setup seems to use pixie instead of conda now.  Can you update the installation instructions for Max and Linux. Likely an installation of pixi is required.

Thanks @pya for trying out the Linux/Mac route. I have updated the instructions, you should be able to follow them starting from "Clone the class repo" (we had renamed that section...)

Thanks @mjr-deltares. Worked like a charm and completed all steps. It says:
Host machine cpu family: x86_64
Host machine cpu: x86_64

But I am on an M1 Macbook. Looks like it did recognize that arm64 would be the better choice.

Excellent. Appreciate the feedback and maybe we can have a look at your arm64 issue next week? Looking forward.


Binary AND not supported for bitmasks

Description
I have an entity with a property of type integer which contains a converted bit-flag enum. I want to query entities by comparing that value to a given bit-mask which is an integer as well.
If I replace the given bit-mask with a static integer like 1, everything works fine. The exception thrown by Entity Framework indicates, that effort wrongly converts the given bit-mask parameter to a Nullable<int>.
Exception
Exception message: The binary operator AND is not defined for "System.Nullable`1[System.Int32]" and "System.Int32"

Stack trace:
at System.Linq.Expressions.Expression.GetUserDefinedBinaryOperatorOrThrow(ExpressionType binaryType, String name, Expression left, Expression right, Boolean liftToNull)
   at System.Linq.Expressions.Expression.And(Expression left, Expression right, MethodInfo method)
   at Effort.Internal.DbCommandTreeTransformation.CanonicalFunctionMapper.<>c.<AddBitwiseMappings>b__4_1(EdmFunction f, Expression[] args)
   at Effort.Internal.DbCommandTreeTransformation.CanonicalFunctionMapper.CreateMethodCall(EdmFunction function, Expression[] arguments)
   at Effort.Internal.DbCommandTreeTransformation.TransformVisitor.Visit(DbComparisonExpression expression)
   at Effort.Internal.DbCommandTreeTransformation.TransformVisitor.Visit(DbAndExpression expression)
   at Effort.Internal.DbCommandTreeTransformation.TransformVisitor.Visit(DbFilterExpression expression)
   at Effort.Internal.DbCommandTreeTransformation.TransformVisitor.Visit(DbProjectExpression expression)
   at Effort.Internal.CommandActions.QueryCommandAction.ExecuteDataReader(ActionContext context)
   at System.Data.Entity.Infrastructure.Interception.InternalDispatcher`1.Dispatch[TTarget,TInterceptionContext,TResult](TTarget target, Func`3 operation, TInterceptionContext interceptionContext, Action`3 executing, Action`3 executed)
   at System.Data.Entity.Infrastructure.Interception.DbCommandDispatcher.Reader(DbCommand command, DbCommandInterceptionContext interceptionContext)
   at System.Data.Entity.Core.EntityClient.Internal.EntityCommandDefinition.ExecuteStoreCommands(EntityCommand entityCommand, CommandBehavior behavior)

Project
public class MyEntity
{
    public int BitValues { get; set; }
}

public void Filter()
{
    DbSet<MyEntity> set = context.MyEntities;

    // Throws exception
    var bitMask = 1;
    var query = (from entity in set
                        where (entity.Bitvalues & bitMask) == bitMask
                        select entity).ToList();

    // Works fine
    query = (from entity in set
                        where (entity.Bitvalues & 1) == 1
                        select entity).ToList();
}

Further technical details

EF version: 6.3
EF Effort version: 2.2.6
Database Provider: NMemory 3.1.0

Hello @Toxantron ,
Thank you for reporting,
We will look if that's possible to do something on our side.
Best Regards,
Jonathan

Performance Libraries
context.BulkInsert(list, options => options.BatchSize = 1000);
Entity Framework Extensions • Entity Framework Classic • Bulk Operations •  Dapper Plus
Runtime Evaluation
Eval.Execute("x + y", new {x = 1, y = 2}); // return 3
C# Eval Function • SQL Eval Function

Hello @Toxantron ,
The v2.2.7 has been released.
Could you confirm for us that the issue is fixed on your side as well?

Hello @Toxantron,
Since our last conversation, we haven't heard from you.
Did you get the time to try the released version V2.2.7
Let us know if that issue is fixed on your side.
Best regards,
Jon

Hello Jon,
sorry I did not get back to you sooner. We had disabled the unit test and didn't work on that part of the code in the last weeks.
But I just updated the package, enabled the test and it runs without errors. Thank you very much for the fix.


Add Einride Vulnerability Disclosure Policy

Einride has a public Vulnerability Disclosure Policy.
This PR includes it into the list.

Thank you!


Make the codes python 3 compatible

codes now seems to work with python 3. only 6-7 failing unit tests remain


Select component: no border.

Installed tailwind into create vue app as per https://tailwindcss.com/docs/guides/vite#vue
Noted that hyperui uses tailwind v3.4.5 - so downgraded my version.

Diffed my version of v3.4.5 to hyperui.
Where is the select styles coming from:
select {
  background-image: url("data:image/svg+xml,%3csvg xmlns='http://www.w3.org/2000/svg' fill='none' viewBox='0 0 20 20'%3e%3cpath stroke='%236b7280' stroke-linecap='round' stroke-linejoin='round' stroke-width='1.5' d='M6 8l4 4 4-4'/%3e%3c/svg%3e");
  background-position: right 0.5rem center;
  background-repeat: no-repeat;
  background-size: 1.5em 1.5em;
  padding-right: 2.5rem;
  print-color-adjust: exact;
}

That does not exist in tailwind: https://play.tailwindcss.com/6ZjKV5OnC1
https://shottr.cc/s/WUel/SCR-20240805-kptp.png

Sorry @sidouglas, there is meant to be a comment at the top of the those components. It's using @tailwindcss/forms.


Is there a way to get submissions who have not been upvoted/downvoted by users?

As written above, I'm looking for a way to only get submissions that have not been upvoted or downvoted by a specific reddit user yet. Is that possible with the API? Thank you!

There is no direct way to do what you're asking. You can use the comment_stream functionality to get new submissions as they arrive and check its constraints. However all new submissions have 1 vote by the submitter.
Perhaps search exposes the number of votes on submissions, so that may be another avenue.
In the future, please direct PRAW usage or reddit API related questions to https://www.reddit.com/r/redditdev/.

Thanks for the quick reply, will try what you suggested.


Macro: Pressing Snake mode button causes exception

Steps to Reproduce

Switch to Macromolecules view
Create configuration as on the picture:

Pressing Snake mode button

Actual behavior
System throws exception into console: localhost-1702893317531.log

Expected behavior
App works as intended.
Desktop (please complete the following information):

OS: win10
Browser Chrome 120.0.6099.110 (Официальная сборка), (64 бит)

Ketcher version [e.g. v2.4.2].

Verified.


window10 delve is debugger.go:61: launching process with args: [./debug] could not launch process: Access is denied

As the title
my system is windows10
go version is go 1.7
Run go get github.com/derekparker/delve/cmd/dlv   in cmd
run delve debug --log main.go is error
that is information
delve is debugger.go:61: launching process with args: [./debug] could not launch process: Access is denied
Is not OSX has this problem?? window also have??
i am chinese,  English  is poor . Please help me with this problem
I will use Google translate view your answer，Thank you

@alexbrainman @lukehoban any thoughts on this?

any thoughts on this?

I cannot reproduce it here. But I use latest version of delve. You can get it if you run
go get -u github.com/derekparker/delve/cmd/dlv

I also use Windows 7, not Windows 10.
I see:
c:\Users\Alex\dev\src\a>type main.go
package main

import "fmt"

func main() {
        fmt.Printf("Alex\n")
}
c:\Users\Alex\dev\src\a>dlv debug --log main.go
ALEX: fp=c:\Users\Alex\dev\src\a\debug
2016/07/23 11:26:40 server.go:71: Using API v1
2016/07/23 11:26:40 debugger.go:61: launching process with args: [./debug]
Type 'help' for list of commands.
(dlv) c
2016/07/23 11:26:41 debugger.go:393: continuing
Alex
Process 288 has exited with status 0
(dlv) q
Process 288 has exited with status 0

c:\Users\Alex\dev\src\a>

Delve uses ./debug to launch executable. But all Windows executables have .exe extension. Maybe we should use delve.exe instead of delve for executable name. Perhaps Windows 10 refuses to run it because it has no .exe.
Perhaps you have some antivirus program that meddle here.
Also having ./ at the start of Windows executable path is wrong too. Windows uses \ for path delimiters, so it should be .\delve.exe. And Windows always look for executable in current directory first anyway, so instead of .\delve.exe it can be just delve.exe. Or you could just use absolute path instead. You calculate absolute path anyway, so you could delete executable on exit.
Unrelated to this problem, but I noticed that sometimes (quite often) delve file does not get deleted after dlv.exe exits. The problem is that you cannot delete executable while it is running. If your debugged app exits before dlv.exe, I see no delve file left behind. But if I, for example, quit immediately after startup, the debug file is left behind.
Also maybe it is good idea to create debug file in /tmp?
Alex

@alexbrainman
If I want to modify the code，Modify the generated file where the debug，Plus .exe suffix，I should be how to do, where to modify?
Be sure to help me look，Thank you

@dunmengjun you could compile manually with go build -gcflags='-N -l' main.go then run delve with: dlv exec <executable file>

I modify the code  Plus the .exe suffix，The resulting file is debug.exe
then I run
dlv debug --log main.go
Or the error
2016/07/23 12:13:29 server.go:71: Using API v1
2016/07/23 12:13:30 debugger.go:61: launching process with args: [./ebug.exe]
could not launch process: Access is denied.

then I modify the ./ to .\
Or the error
2016/07/23 12:13:29 server.go:71: Using API v1
2016/07/23 12:13:30 debugger.go:61: launching process with args: [.\ebug.exe]
could not launch process: Access is denied.

Next, I will debug.exe into tmp folder
However, I think we should still get this error
wait me
I guess, it is a permissions issue, but I still do not know the specific
Really going to cry！！！

@aarzilli
Let me try，Thank you

@aarzilli
I use
go build -gcflags="-N -l" main.go
then
 dlv exec main.exe
Or the error
2016/07/23 12:30:47 server.go:71: Using API v1
2016/07/23 12:30:47 debugger.go:61: launching process with args: [main.exe]
could not launch process: Access is denied.

I'm really going to cry！！

Make sure that the user you are using has the "Debug programs" right: https://technet.microsoft.com/en-us/library/dn221963(v=ws.11).aspx

@aarzilli
I do not know how I confirm that the user have the "Debug programs" right
Can you tell me how to do it? ?
and if the user is not have this
What can I do?
thank you very much

follow the instructions on the link and check that the user you are using or one of the groups that user belongs to is listed after "Debug programs", I don't have a simpler way to explain it.

thank you  @aarzilli
but I'm sure this is not a problem
I look at my group policy, Debug programs value is Administrators, It is a user group includes my current login user,Even so，I tried my login users to join Debug programs.
then I repeat dlv operation
But useless，Still errors
2016/07/23 13:48:46 server.go:71: Using API v1
2016/07/23 13:48:46 debugger.go:61: launching process with args: [./debug]
could not launch process: Access is denied.

@derekparker @alexbrainman  @aarzilli
After I carefully check the code，I found the error is generated here
proc/proc_window.go:98
err = sys.CreateProcess(argv0, cmdLine, nil, nil, true, DEBUGONLYTHISPROCESS, nil, nil, si, pi)

sys.CreateProcess generated the error
then I find this func at https://github.com/golang/sys/blob/master/windows/zsyscall_windows.go
func CreateProcess(appName *uint16, commandLine *uint16, procSecurity *SecurityAttributes, threadSecurity *SecurityAttributes, inheritHandles bool, creationFlags uint32, env *uint16, currentDir *uint16, startupInfo *StartupInfo, outProcInfo *ProcessInformation) (err error) {
	var _p0 uint32
	if inheritHandles {
		_p0 = 1
	} else {
		_p0 = 0
	}
	r1, _, e1 := syscall.Syscall12(procCreateProcessW.Addr(), 10, uintptr(unsafe.Pointer(appName)), uintptr(unsafe.Pointer(commandLine)), uintptr(unsafe.Pointer(procSecurity)), uintptr(unsafe.Pointer(threadSecurity)), uintptr(_p0), uintptr(creationFlags), uintptr(unsafe.Pointer(env)), uintptr(unsafe.Pointer(currentDir)), uintptr(unsafe.Pointer(startupInfo)), uintptr(unsafe.Pointer(outProcInfo)), 0, 0)
	if r1 == 0 {
		if e1 != 0 {
			err = error(e1)
		} else {
			err = syscall.EINVAL
		}
	}
	return
}

find
procCreateProcessW                     = modkernel32.NewProc("CreateProcessW")
CreateProcessW  is Access is denied
I do not know how to do it

What happens if you change:
err = sys.CreateProcess(argv0, cmdLine, nil, nil, true, DEBUGONLYTHISPROCESS, nil, nil, si, pi)

with
err = sys.CreateProcess(argv0, cmdLine, nil, nil, true, 0 nil, nil, si, pi)

?
Thank you.
Alex

I change with
err = sys.CreateProcess(argv0, cmdLine, nil, nil, true, 0 nil, nil, si, pi)
but
2016/07/23 15:23:10 server.go:71: Using API v1
2016/07/23 15:23:10 debugger.go:61: launching process with args: [./debug]
could not launch process: The handle is invalid.
chan is null
hello
hello
chan is null
hello
hellohello

the progrem is run
However, errors or

@alexbrainman

Thank you.
It seems Windows does not allow you to use DEBUG_ONLY_THIS_PROCESS flag with CreateProcess API. I do not know why. I need to think about it.
Alex

@alexbrainman If you have the answer, please be sure to tell me. thank you very much

If you have the answer, please be sure to tell me.

I will not forget you. But I really do not know what to do here. Sorry.
Maybe @lukehoban knows more.
Alex

@alexbrainman  Thank you
@lukehoban please help

I guess I found out why！
Now, I have been able to run delve
The reason is that I uninstalled the 360 security guards，Then restart the computer.
360 security guards is a security software, its implementation is invaded windows kernel.
The operation of the process debugger is very dangerous, because of the uncertainty whether the operator is a dangerous software, so the 360 block of this operation
For the above reasons is my guess, wrong place, please correct me, but it really is because 360 can not run led to delve
hanks to all who helped me
thank you very much @alexbrainman @aarzilli @derekparker

I was wrong
By chance, I found that when I play LOL, delve actually appeared Access is denied error
So, I go through repeated tests and found that safety procedures LZ actually banned CreateProcess DEBUG_PROCESS call the whole system
I have been to check how he did it, and whether there are counter-measures。
But still no gain
You, I was wrong, I was wrong 360
@alexbrainman @aarzilli @derekparker If you have any way, be sure to send me to tell
I now play games and write c procedures can not be performed at the same time，

Does LOL here stand for League of Legends?

Yes， LOL is the League of Legends
but I also was wrong
Because this problem is not caused by LOL.You know Tencent acquired LOL it?In China, LOL start at the same time will start the process QQ Safety,This is the cause of CreateProcess DEBUG mode is disabled.Because, Tencent do not want any other program to bundle its program.
Now, I have not take QQ and LOL, because I could not find a way to prevent QQ security process starts.
tragedy!!!!

@aarzilli

Good that you figured it out, I'm closing this issue since there's nothing we can do about it.


always save execution cache

This PR should setup github actions such that the notebook execution cache will always be saved (even in case of errors). To my understanding, jupyterbook will re-execute failed notebooks on the next build, but could reuse successfully executed notebook cells in that case. This should therefore give use a big improvement in average build time.

I just ran two builds in a row (first and second time).

The build is failing (due to an issue with the ICON-LES notebook), but the cache has been created successfully :+1:.
The second build could restore the cache :+1:.
The second build re-ran the failing notebook :+1:, but used all the successful ones from cache :+1:.

I just re-run the build, and as #102 is now fixed, the book was built successfully, and used the cache for execution.

How does the new save execution cache fit in here now?

The issue here is, the default caching step will only upload the cache if the job was successful. The extra save execution cache step also uploads the cache if the job failed (which I guess is what we need as failing jobs due to a single problematic notebook are frequent).


Default colors are unreadable

Sure we can disable colors with -c and .gdu.yaml, but why have default colors that are hard to read.
On my screen the default colors at the bottom look like this:

Even in the README, you have:

I don't know about your display, but for me the red on blue is not readable.
Changing colors for taste is one thing.  But having default colors that are readable really should be standard.

I will check the colors. It should be black and white, the grey is there by mistake.

I think I have found the cause. You probably have first color in your color palette mapped to something different then black. I have selected the color more explicitly now.
For future testing: which terminal, OS and colors theme do you use?

You're right. My bad.
I'm using iTerm on macOS, with the "Violet Light" theme from https://iterm2colorschemes.com/
The palette looks like:

Thanks, I will check that theme.


Increase default timeout to 30

I started getting the following error, which proved quite hard to diagnose. Not a very helpful error from the API wrapper.
Impossible to connect, please check your Algolia Application Id.

In short, at least in this instance, it’s a result of having the connect_timeout set to 1 second by default. Could we increase this to the default of 30 as per the official defaults?

@joshuabaker that link says the defaults are: "PHP, Ruby, JavaScript, Python, Android, Java, Go, and Scala: 2 seconds"
At any rate, it seems like maybe we shouldn't be setting them at all by default and just be relying on the defaults, unless explicitly passed.

@timkelty Quite right, it does say “2 seconds”.
The plugin does actually set that value as 1 second, which was enough to cause an issue in our scenario.
Could we set the default config to null and then run a check around the current setConnectTimeout call? I think that’s what you’re proposing here anyway, right?

@joshuabaker exactly!

Going to close as stale. If anyone wants this feature, please consider opening another issue.


Set variables in beforeAll hook, get undefined value in describe block

🐛 Bug Report
Set variables in beforeAll hook, get undefined value in describe block; I am not sure it's a bug or is there any purpose for designing jest like this?
To Reproduce
Steps to reproduce the behavior:
Here is my test code:
import { envVars, IEnvVars } from './envVars';

let env: IEnvVars;
beforeAll(async () => {
  env = await envVars;
  console.log('a - beforeAll env:', env);
});

describe('a test suites', () => {
  console.log('a - describe scope env:', env);
  it('Test', () => {
    console.log('a - test case scope env:', env);
    expect(1).toBe(1);
  });
});

Here is the stdout:
 PASS  src/multiple-beforeAll/a.spec.ts
  ● Console

    console.log src/multiple-beforeAll/a.spec.ts:10
      a - describe scope env: undefined
    console.log src/multiple-beforeAll/a.spec.ts:6
      a - beforeAll env: { apiKey: '123', baseUrl: 'https://github.com' }
    console.log src/multiple-beforeAll/a.spec.ts:12
      a - test case scope env: { apiKey: '123', baseUrl: 'https://github.com' }

Expected behavior
I can get env value in beforeAll hook and each test case. But can not get env value in describe block. As you can see the stdout -  a - describe scope env: undefined
Expect get env value in describe block.
Link to repl or repo (highly encouraged)
Here is the repo: https://github.com/mrdulin/jest-codelab/tree/master/src/multiple-beforeAll
Issues without a reproduction link are likely to stall.
Run npx envinfo --preset jest
Paste the results here:
npx: 1 安装成功，用时 3.068 秒

  System:
    OS: macOS High Sierra 10.13.6
    CPU: x64 Intel(R) Core(TM) i7-7567U CPU @ 3.50GHz
  Binaries:
    Node: 8.11.4 - /usr/local/bin/node
    Yarn: 1.6.0 - /usr/local/bin/yarn
    npm: 5.6.0 - /usr/local/bin/npm
  npmPackages:
    @types/jest: ^23.3.3 => 23.3.3
    jest: ^23.6.0 => 23.6.0

This is by design, see https://jestjs.io/docs/en/troubleshooting#defining-tests


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-target_438103ba-5d5f-4f6c-9914-cc1bc38de34c/pull/1 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-target_438103ba-5d5f-4f6c-9914-cc1bc38de34c.git

Docs Build status updates of commit ba3815b:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

:x:Error

Details

[Error-RuningBuildFailed] Some unexpected errors happened when running build, please open a ticket in https://aka.ms/SiteHelp and include the error report for our team to troubleshoot

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Provide pageable org users endpoint

Explained in #2489. Adds /v3/organizations/:guid/users.

[x] I have reviewed the contributing guide

[x] I have viewed, signed, and submitted the Contributor License Agreement

[x] I have made this pull request to the main branch

[x] I have run all the unit tests using bundle exec rake

[ ] I have run CF Acceptance Tests

Acceptance
`
cf curl /v3/organizations/15c8f550-9a9a-49f0-ac83-3b2a14891423/users
{
"pagination": {
"total_results": 2,
"total_pages": 1,
"first": {
"href": "https://api/v3/organizations/15c8f550-9a9a-49f0-ac83-3b2a14891423/users?page=1&per_page=50"
},
"last": {
"href": "https://api/v3/organizations/15c8f550-9a9a-49f0-ac83-3b2a14891423/users?page=1&per_page=50"
},
"next": null,
"previous": null
},
"resources": [
{
"guid": "f1639818-b0f7-4386-bd58-0f26fa5973c3",
"created_at": "2021-09-08T18:56:49Z",
"updated_at": "2021-09-08T18:56:49Z",
"username": "admin",
"presentation_name": "admin",
"origin": "uaa",
"metadata": {
"labels": {
    },
    "annotations": {

    }
  },
  "links": {
    "self": {
      "href": "https://api/v3/users/f1639818-b0f7-4386-bd58-0f26fa5973c3"
    }
  }
},
{
  "guid": "9bf359d3-82f7-431f-a6ef-b526a0090545",
  "created_at": "2021-09-09T19:00:10Z",
  "updated_at": "2021-09-09T19:00:10Z",
  "username": "cool",
  "presentation_name": "cool",
  "origin": "uaa",
  "metadata": {
    "labels": {

    },
    "annotations": {

    }
  },
  "links": {
    "self": {
      "href": "https://api/v3/users/9bf359d3-82f7-431f-a6ef-b526a0090545"
    }
  }
}

]
}
`
Checked this after assigning a second user to this org, LGTM, merging!


Can't sign in with Google Auth

Good day, please I'm having an issue with Peatio installation, can't "sign in with Google", display the following message "Wrong customer ID or password, please try again.", using the latest version. see the link below http://qybit.com, please how can i resolve this? Thanks

Put apikey from google auth and make sure everything is correct Or if u still struggling i can help u aswell chat me on whatsapp : +6285790752549

Thanks @ysv and @Regzaenal
Issue resolved, but I was faced with another issue, after authentication, it displays this => "Please, pass the corresponding verification steps to enable trading." under "fund" menu, please where can i get the verification steps.? Thanks

@samsoft00 Update
  MINIMUM_MEMBER_LEVEL_FOR_DEPOSIT:  '0'
  MINIMUM_MEMBER_LEVEL_FOR_WITHDRAW: '0'
  MINIMUM_MEMBER_LEVEL_FOR_TRADING:  '0'

In application.yml


Test Python source distribution installation

I've run it through my local machine (with both broken and not-broken packages); still need to shove it through Jenkins.

To follow:
https://grpc-testing.appspot.com/view/Artifacts/job/gRPC_build_artifacts/1249/

note to self: update release process documentation to emphasize checking the distrib test results on the release branch once this is merged...

The distribtests for Python are now failing as they should be because we're waiting on a fix we put on the x1.0.x branch to the source package to eventually be upmerged.
... Unless we just want to cherry-pick that now. Anyway, removing DNM as the distribtests appear to be behaving as they should both locally and remotely, via task_runner in Docker on Jenkins (where the fix isn't around) and out (where I'd used both the fixed source package and a broken one).

Luhguhtum.

This should be placed on top of the v1.0.x branch, since, y'know, distribution for GA should look green...


fix: crash on unix based systems without find

Summary
Closes #9329
Test plan
Unit tests added

Codecov Report

Merging #9579 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #9579      +/-   ##
==========================================
+ Coverage   65.07%   65.08%   +<.01%
==========================================
  Files         286      286
  Lines       12144    12146       +2
  Branches     3010     3011       +1
==========================================
+ Hits         7903     7905       +2
  Misses       3605     3605
  Partials      636      636

Impacted Files
Coverage Δ

packages/jest-haste-map/src/crawlers/node.ts
85.88% <100%> (+0.34%)
: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 63593a2...c3997c2. Read the comment docs.

Codecov Report

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

@@            Coverage Diff             @@
##           master    #9579      +/-   ##
==========================================
+ Coverage   65.07%   65.09%   +0.01%
==========================================
  Files         286      286
  Lines       12144    12150       +6
  Branches     3010     3011       +1
==========================================
+ Hits         7903     7909       +6
  Misses       3605     3605
  Partials      636      636

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 63593a2...a8f28fe. Read the comment docs.

Seems like the build is broken on Node v13.x on ubuntu-latest, but I couldn't understand why 🤔

Codecov Report

Merging #9579 into master will increase coverage by 0.03%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #9579      +/-   ##
==========================================
+ Coverage   65.07%   65.11%   +0.03%
==========================================
  Files         286      286
  Lines       12144    12151       +7
  Branches     3010     3011       +1
==========================================
+ Hits         7903     7912       +9
+ Misses       3605     3604       -1
+ Partials      636      635       -1

Impacted Files
Coverage Δ

packages/jest-haste-map/src/crawlers/node.ts
86.66% <100%> (+1.12%)
:arrow_up:

packages/expect/src/utils.ts
96.22% <0%> (+1.25%)
: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 63593a2...ee303a1. Read the comment docs.

Sorry to bug you, but just curious how the Jest releases work and when this fix would become available to the public?
Thank you for the fix!

We can make a minor release by the end of the month!

@DiZy would the minor release be in<EMAIL_ADDRESS>I am having this issue on Node12 on AWS, which is not even using Windows. It is using the Ubuntu Standard 3.0 Docker image.

@DiZy status of this?

+1 also waiting on a release here. Is there anywhere documented your clear release schedule?

I cannot even pull in the latest commits from the master branch:
$ npm i facebook/jest#master --save
npm ERR! Can't install github:facebook/jest#b5c7092687a265e3f4f2ba6f9787e47e8c6b9d5e: Missing package name

25.2.0 has been published with this fix

@lovefamilychildrenhappiness we don't support installing from github.


Compilation fails with older CMake

Hi,
CMAKE_CXX_STANDARD does not work with older cmake versions (not sure which one, but 2.4 does not support it); therefore, it builds with C++ < 11 and fails (e.g. "nullptr").
If version is less than 3.1, CXX flags should be set explicitly:
if (CMAKE_VERSION VERSION_LESS "3.1")
    if (CMAKE_CXX_COMPILER_ID STREQUAL "GNU")
      set (CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -std=gnu++11")
    endif ()
  else ()
    set (CMAKE_CXX_STANDARD 11)
  endif ()

Regards
Eugenio

Can you make a pull request?

fixed with cf6436111fb7636d5e8026178e26f3229e9ebf91


AmplePro Bonds

New bond type 'pro' on front end that can handle bonds w/ different payout tokens. The UX will be the same as our current bonds but will need the following to function:

payout token name will come from bond settings (needs to be added)
market price will come from getGoldPrice() (oracle.sol contract; 0x9c3ED1A6B079BA8a2a0F047ed69b9402c0347b16 on amber testnet)
will also need a variable in bond settings for 'treasury' address as these new bonds will use a different treasury. this will be needed to report on holdings in the dashboard

Proposed changes for pro bonds in src/helpers/bond/index.ts:
export const fusdGob = new ProBond({ //new bond type 'ProBond'- price coming from getGoldPrice() instead of LP reserves
name: "fusd_gob_lp",
displayName: "GOB-flexUSD LP",
payoutTokenName: "gBCH",  //new field 'payoutToken' - will show on /mints pages
bondToken: "fUSD",
bondIconSvg: FusdGobIcon,
bondContractABI: ProBondContract, //new abi for pro bonds
reserveContractAbi: LpReserveContract,
networkAddrs: {
[Networks.AMBER]: {
bondAddress: "0x9C122F45aE24767Df7cfAC347Cd96cF3d3655Bea",
reserveAddress: "0x9b9190C5392116a865Ceef53b57b0bb758256f74",
proTreasuryAddress: "0x9b9190C5392116a865Ceef53b57b0bb758256f74", //new treasury field for pro bonds
},
},
lpUrl: "https://app.mistswap.fi/add/0xEA48ca36933873507636e4aFD72a74f612aAf8CD/0x96917995f030f8fC3D8fDF40F3a6422547c877c1",
});

Ample and Pro Contracts.zip
testnet addresses.txt

oracle abi - used to get current token price - getGoldPrice()
oracle.txt

bond abi
bond.txt

Bond Purchase
Mint
prices - do they not show decimals? (on mint & redeem)
all "GOB" references should be replaced w/ payout token for bond
time til fully vested not working
Redeem
"Your balance" pulling bond name, not token name
Remove "claim and autostake"
Gold and market prices are flipped :)
"you will get" nan


[ENH]: Switching Ollama JS EF to ollama js client

Description of changes
Summarize the changes made by this PR.

Improvements & Bug fixes

Switching to ollama js client - https://www.npmjs.com/package/ollama
Default model is all-mini
Default URL is ollam's localhost url and port
Using test containers to test the EF
Conditional tests when ollama package is installed
Moved tests to test/embeddings/ollama.test.ts

Test plan
How are these changes tested?

[x] Tests pass locally with yarn test for js with and without ollama package

Documentation Changes
Updated integration example (simplified)

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#2948  👈
#2946
main

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

Join @tazarov and the rest of your teammates on  Graphite


Implement jump for IWMO and IWSV

will close for now, no progress, moving to issue https://github.com/larshp/abapGit/issues/3116


Dispatch sortenter

Motivation: be able to listen to sortenter event - useful when dragging to a different list and wish to update placeholder content.
Basically same PR as #488 - except :
// dispatch sortenter event on each element in group
      sortableContainer.dispatchEvent(new CustomEvent('sortenter', {
          detail: {
              origin: {
                  elementIndex: originElementIndex,
                  index: originIndex,
                  container: originContainer
              },
              destination: {
                  container: sortableContainer,
                  itemsBeforeUpdate: destinationItemsBeforeUpdate
              },
              item: dragging
          }
      }));

Coverage decreased (-1.06%) to 6.944% when pulling edcf477c67d561e2c21adb4a898b34113ccc2392 on christophe-g:dispatch-sortenter into 872e0ce8d226a77dd9166927fab9567b1560ecec on lukasoppermann:master.

Closing as not yet ready (sortenter event should only fire first time it enters a new sortableContainer) and duplicate code from #488