Ef core6.0 mysql create table

Update MySqlQueryBuilder.CreateTableCopy
Because "Create Table NewTableName Select * FROM OldTableName LIMIT 0;" MySql RDS is not supported in aliyun, so change it to "Create Table NewTableName LIKE OldTableName;"

Hi, thx for contrib.
Could you make new PR without changes of Spacing if files so that real Change is easily visible as only Diff.

Added LIKE syntax


Windows Script Host Error

When I open command prompt in the game directory and type in npx love.js game.love game -c
I get this error

I had love.js installed before and it worked, but now its just giving me this error, please help.

As suggested in https://github.com/Davidobot/love.js/issues/48, try using:
npx love.js.cmd


Additional load commands

MachEncryptionInfo
MachRunPath
MachSourceVersion
MachUuid

Thanks @filipnavara . I addressed your comments and added a roundtrip test using an executable which contains a LC_RPATH command.  I couldn't easily create one which contained LC_ENCRYPTION_INFO.
Let me know if you have more feedback.

Thanks a lot!


Minimal Mode

Adds -minimal flag
Adds template without additional css, service worker or svg
Switches to functional components and createRoot
React dependencies @next

@jimthedev No, in fact it can work - you'd just, as with many other tools, use @jimthedev/some-template - which would reference @jimthedev/create-react-app-template-some-template.
I think many users will publish templates... and this wouldn't stop you from simply referencing any local module.

I've given this some more thought, and I think we'll want to require that templates indicate the minimum compatible version of react-scripts - peerDependencies should do the job.
Another thought - should templates be listed as dependencies once used? I can't think of a good use for this, but it would be nice to keep track of which template was used to initialize a project.

Closing this in favour of the more general template approach in #6514. Thanks for kicking this all off in typical Ken Wheeler style. 😀


Testing vue-good-table produces so many warning that it exceeds stack size

Issue Type

[x] Bug

Specs
2.16.0
yarn test
Expected Behavior
clean console output when mounting vue-good-table in test cases
Actual Behavior
there are so many warnings that we get a stack overflow when running tests
Steps to Reproduce the Problem
Please detail your steps here

follow the documentation in "getting started"
write a test that actually mounts the component.
Component:

<template>
    <div>
      <vue-good-table
        :columns="[]"
        :rows="[]"
      >
      </vue-good-table>
    </div>
</template>

<script>
    import VueGoodTablePlugin from 'vue-good-table';
    import 'vue-good-table/dist/vue-good-table.css'
    import Vue from 'vue';

    Vue.use(VueGoodTablePlugin);
</script>

Test:
import {createLocalVue, mount} from '@vue/test-utils';
import ParentComponent from './ParentComponent';
const localVue = createLocalVue();

describe('ParentComponent', () => {
    let wrapper;

    describe('Mounting', () => {
        jest.resetModules();
        jest.clearAllMocks();

        describe('when loading the parent component', () => {
            beforeEach(() => {
                wrapper = mount(ParentComponent, {
                    localVue
                });
            });

            it('should render vue-good-table', () => {
                expect(wrapper.findAll('th').at(0).isVisible()).toBeTrue();
            });
        });
    });
});

run yarn test

==> output in console:
(much abbreviated... we get so many outputs of the first two warnings that we exceed the stack size)
 [Vue warn]: Avoid mutating a prop directly since the value will be overwritten whenever the parent component re-renders. Instead, use a data or computed property based on the prop's value. Prop being mutated: "typedColumns"

      found in

      ---> <VgtTableHeader>
             <VueGoodTable>
               <Anonymous>
                 <Root>
    console.error node_modules/vue/dist/vue.runtime.common.js:603
      [Vue warn]: Avoid mutating a prop directly since the value will be overwritten whenever the parent component re-renders. Instead, use a data or computed property based on the prop's value. Prop being mutated: "rows"

      found in

      ---> <VueGoodTable>
             <Anonymous>
               <Root>
    console.error node_modules/vue/dist/vue.runtime.common.js:603
      [Vue warn]: Error in nextTick: "RangeError: Maximum call stack size exceeded"
    console.error node_modules/vue/dist/vue.runtime.common.js:1821
      RangeError: Maximum call stack size exceeded

jsfiddle
unfortunately, jsfiddle does not offer to write tests
note
Thanks for building and maintaining this library. On the functional side we really ❤ it!

I am also getting so many warning in the console, please look into it, and please suggest a solution for this, thanks !

Not test-related but I get these warnings on Vue Dev Tools on Chrome:
[Vue warn]: Avoid mutating a prop directly since the value will be overwritten whenever the parent component re-renders. Instead, use a data or computed property based on the prop's value. Prop being mutated: "rows"

Any updates to this issue? Also getting these warnings whenever updating rows, not test-related either.


security issue in dependency

Is there an existing issue for this?

[X] I have searched the existing issues

Current behavior
Performing npm audit returns security issue inside axios.
# npm audit
# npm audit report

axios  >=1.3.2
Severity: high
Server-Side Request Forgery in axios - https://github.com/advisories/GHSA-8hc4-vh64-cxmj
fix available via `npm audit fix`
node_modules/axios

1 high severity vulnerability

To address all issues, run:
  npm audit fix

Minimum reproduction code
https://github.com/advisories/GHSA-8hc4-vh64-cxmj
Steps to reproduce

npm audit
npm audit fix
npm audit

Expected behavior
npm audit | grep "found 0 vulnerabilities"
Package version
3.0.2
NestJS version
10.4.0
Node.js version
20.12.2
In which operating systems have you tested?

[ ] macOS
[ ] Windows
[X] Linux

Other
VSCode devcontainer running Oracle Linux 8

Issue is related to Axios SSRF vulnerability. Here's more details on that https://security.snyk.io/vuln/SNYK-JS-AXIOS-7361793

Anyone working on this or do I open a pr for this?

We also facing issues in our Audit pipeline to.. Is there a fix planned ?

We also facing issues in our Audit pipeline too.. Is there a fix planned ?

https://github.com/axios/axios/pull/6539
We need to wait for this pr to merge.

We also facing issues in our Audit pipeline too.. Is there a fix planned ?

axios/axios#6539
We need to wait for this pr to merge.

Thanks 😃

We also facing issues in our Audit pipeline too.. Is there a fix planned ?

axios/axios#6539
We need to wait for this pr to merge.

PR already merged 😀

Yup but they need to release it as V1.7.4. I assumed that would happen after the merge but i seems like there's issues with it. Cause it changes the way some things are handled(not spec compliant I believe). But they are working on a fix.
https://github.com/axios/axios/issues/6463#issuecomment-2286538061. https://github.com/axios/axios/pull/6539#issuecomment-2286492862.
Once 1.7.4 the bot will automatically update it. (I think)

https://github.com/nestjs/axios/pull/1173


updates to untested nbs model_graph and panel_errors

Codecov Report

Merging #113 (0b2f95e) into main (8d2be2c) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #113   +/-   ##
=======================================
  Coverage   80.30%   80.30%
=======================================
  Files          38       38
  Lines        4459     4459
=======================================
  Hits         3581     3581
  Misses        878      878

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


package doesn't work with latest expo sdk 39.0.0

modal not showing

Upgraded to expo-sdk 40.0.0

works perfectly now!


Pasting images into markdown cells no longer adds a notebook attachment

I tested on Win11 & Win10.  It works as long as you start a new workbook and don't save the jupyter notebook.  Once you save the notebook, your pastes of images from the clipboard will no longer be treated as inline attachments.
Originally posted by @beckerben in https://github.com/microsoft/vscode-jupyter/discussions/14408#discussioncomment-7180960

Possibly getting overridden by a markdown file paste provider? @mjbvz

@amunger were you able to replicate my findings?

With the above investigation, I believe this is likely an issue with markdown paste providers, assigning @mjbvz to take a look at this.
@beckerben, to clarify, are you expecting the images to be created on-disk upon paste, and referenced via an image markdown link? Or are you looking for the attachment:filename experience, with images stored in the notebook itself. The latter can be achieved and still works properly with the setting "ipynb.pasteImagesAsAttachments.enabled": true

I prefer the later, where the images are stored in the notebook, this makes it easier to share with others in these scenarios.  This used to be working and then stopped working in the build I noted in the original question refenced, but I just tested in the latest build and it is back to working as expected, so must have been a bug that has now been fixed:
The version I just tested with is updated now and and it is working as expected with the "ipynb.pasteImagesAsAttachments.enabled": true
Version: 1.83.1 (system setup)
Commit: f1b07bd25dfad64b0167beb15359ae573aecd2cc
Date: 2023-10-10T23:48:05.904Z
Electron: 25.8.4
ElectronBuildId: 24154031
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19045

Works fine for me too with the default settings. Will disable pasting into untitled notebooks with the setting off


CORS error while flashing

While trying to flash ESP32-S3-Box-3 in console there is error whit cors. Device is succesfully connected with serial port.
Flasher: https://flash.heywillow.io/

Access to fetch at 'https://worker.heywillow.io/api/fetch?url=undefined' from origin 'https://flash.heywillow.io' has been blocked by CORS policy: No 'Access-Control-Allow-Origin' header is present on the requested resource. If an opaque response serves your needs, set the request's mode to 'no-cors' to fetch the resource with CORS disabled.
Browsers:
Edge: 117.0.2045.60
Chrome 118.0.5993.89

Might need a force refresh (ctrl-f5).

Locally it works. That was quick. Thanks.

Locally it works. That was quick. Thanks.

Very welcome. Glad it works!


Add Match Details for 1718 Season

Add the Match Details for the 1718 Season, right now it just shows blank

This was completed with commit 3f83b7c


[Bug] Frame doesn't respect CornerRadius in transition

The frame control seems to ignore its CornerRadius during a transition.
I ran the SampleApp with XF <IP_ADDRESS>9. I know that Frames are kinda f***ed within XF itself at the moment but i still wanted to raise awareness and see, if you can fix this issue rather easily.
If you cannot do that however, a future XF update might resolve this issue again.
It might also be helpful to report this issue to the XF issue tracker. I have not done this yet.
Bug Information
Version Number of Plugin: 2.3
Device Tested On: iPhone 11 Pro, iOS 14b5
Simulator Tested On: iOS, multiple
Version of Xamarin: <IP_ADDRESS>9
Steps to reproduce the Behavior
run sample app.
Expected Behavior
the frames CornerRadius animates to its chosen value during transition.
Actual Behavior
the frames CornerRadius seems to be 0 during transition and is snapping back to the chosen value after the transition.
Screenshots

probably they modified how the corner radius are managed in the Frame component. I'll take a look

Thanks for the quick answer!
I only see this behavior in iOS, which means you're probably right about the modification.

fixed in the next version


Add support for eeprom based SSID/PASSKEY

no way to update to new wifi without reflashing. Shoudl fix that.

Added in 84da30e886090e7f7b66171db1c445170b5e4e28


ArrowFunction should be a FunctionLikeDeclaration to be consistent with typescript

I don't see any reason why it we can easily add FunctionLikeDeclaration to ArrowFunction, am I missing something? This is typscript:
interface ArrowFunction extends Expression, FunctionLikeDeclarationBase

The main consequence I found because of this is that TypeGuard.isFunctionLikeDeclaration() will return false for ArrowFunction which is inconsistent with function isFunctionLike(node: Node): node is SignatureDeclaration;. If not possible, then perhaps can we create a new TypeGuard isFunctionLike to cast to SignatureDeclaration like typescript's ?
I'm using TypeGuards.isSignaturedDeclaration but IMO that's not obvious I think we should use same TypeScript naming if possible...
I would be able to attack the issue, but first I wanted to ask you, thanks

Hi @cancerberoSgx, thanks for reporting this! I will take a look at this later because this is a breaking change and it requires some changing of where the mixins are placed.
For now, please continue to use isSignaturedDeclaration.

is not critical, more like a question I think - IMO you should take it as low priority and if you decide this should be tackled I would like to help you -  perhaps a little documentation note is enough to alleviate the problem...


Update README with Access-Control headers for development

Adds required configurations to run the WebUI on port 3000 so that it can properly make request to the running daemon.

Thanks


Include a test case that lambda arguments are covered by the indexer [phab:D1877]

Created by shahms at 2017-08-15 21:26:28:

zarko accepted this revision at 2017-08-15 22:08:13

shahms closed via commit 911f0f01c7cf59e970e899994d667f40362a7281 at 2017-08-15 22:09:52


Failed to install

I forked and tried to get the code running but I get an error when running npm install

npm ERR! Windows_NT 10.0.10586
npm ERR! argv "C:\Program Files\nodejs\node.exe" "C:\Users\user\AppData\Roaming\npm\node_modules\npm\bin\npm-cli.js" "install"
npm ERR! node v4.4.7
npm ERR! npm  v3.10.2
npm ERR! code ELIFECYCLE

npm ERR<EMAIL_ADDRESS>install: (node-gyp rebuild 2> builderror.log) || (exit 0)
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script '(node-gyp rebuild 2> builderror.log) || (exit 0)'.
npm ERR! Make sure you have the latest version of node.js and npm installed.
npm ERR! If you do, this is most likely a problem with the websocket package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     (node-gyp rebuild 2> builderror.log) || (exit 0)
npm ERR! You can get information on how to open an issue for this project with:
npm ERR!     npm bugs websocket
npm ERR! Or if that isn't available, you can get their info via:
npm ERR!     npm owner ls websocket
npm ERR! There is likely additional logging output above.

Here's the log file:npm-debug.zip

Hum, it's something related to the websocket dependency. Do you think you can find the builderror.log file that it mentions?
Also, what happens if you just run npm install websocket ?
(I thought maybe it was a windows issue, but I just tried on a clean install of windows 10 and it worked fine :/

So, looking for that file with procmon I notched it was being deleted calling a command that fired TCC  (my command line).
So I opened a regular command line and it worked! I guess it's just a bug with TCC.
Thanks for your time!

BTW, @theturtle32 - not sure if this merits an issue on websocket or not, but apparently it fails to install for node 4.4.7 on Windows using the TCC shell. Just thought I'd give you a shout out in case you see this come up elsewhere.


API inconsistency: ProcessInfo.getPid vs. JProcesses.kill

While ProcessInfo.getPid() returns a String JProcesses.kill requires an integer parameter.

Hello,
Thanks, you are right. I will fix that in a future version.


KAFKA-12284: increase request timeout to make tests reliable

The MirrorConnectorsIntegrationTests recently failed with
TimeoutException: The request timed out.

quite frequently. After investigation, the reason is the createTopic in the server doesn't complete within request.timeout.ms (default to 30 seconds). It makes sense that the server is pretty slow during 2 connector cluster (primary and backup) started at the same time. What I did are:

increase the timeout value to 60 seconds to make the tests reliable.
wait for internal topics created after connector cluster started, and then try to create our test topics.

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

@mimaison @ning2008wisc , I've updated the PR, please help review it. Thank you.

Failed tests are all un-related. (Not in MirrorMakerIntegrationTest)
)

    Build / JDK 8 and Scala 2.12 / kafka.server.RaftClusterTest.testCreateClusterAndCreateAndManyTopicsWithManyPartitions()
    Build / JDK 15 and Scala 2.13 / org.apache.kafka.connect.integration.ConnectorClientPolicyIntegrationTest.testCreateWithNotAllowedOverridesForPrincipalPolicy
    Build / JDK 15 and Scala 2.13 / kafka.admin.DeleteOffsetsConsumerGroupCommandIntegrationTest.testDeleteOffsetsNonExistingGroup()
    Build / JDK 15 and Scala 2.13 / kafka.admin.ResetConsumerGroupOffsetTest.testResetOffsetsByDurationToEarliest()
    Build / JDK 15 and Scala 2.13 / kafka.admin.ResetConsumerGroupOffsetTest.testResetOffsetsExistingTopicSelectedGroups()
    Build / JDK 15 and Scala 2.13 / kafka.admin.ResetConsumerGroupOffsetTest.testResetOffsetsToZonedDateTime()
    Build / JDK 15 and Scala 2.13 / kafka.server.RaftClusterTest.testCreateClusterAndCreateAndManyTopicsWithManyPartitions()
    Build / JDK 15 and Scala 2.13 / org.apache.kafka.streams.integration.KTableKTableForeignKeyInnerJoinMultiIntegrationTest.shouldInnerJoinMultiPartitionQueryable

thanks for the PR :) change seems reasonable and lgtm (if consistent test results can be verified by multiple runs). The final review will be done by the committer.

Thanks @ning2008wisc !
@mimaison , see if you have any comments. Thanks.

@showuon Could you merge trunk to trigger QA again?

Done. Let's wait and see :)

Failed tests are
    Build / JDK 8 and Scala 2.12 / org.apache.kafka.connect.mirror.integration.MirrorConnectorsIntegrationSSLTest.testReplication()
    Build / JDK 8 and Scala 2.12 / kafka.server.RaftClusterTest.testCreateClusterAndCreateAndManyTopicsWithManyPartitions()
    Build / JDK 15 and Scala 2.13 / org.apache.kafka.connect.integration.RebalanceSourceConnectorsIntegrationTest.testDeleteConnector
    Build / JDK 15 and Scala 2.13 / org.apache.kafka.streams.integration.KTableKTableForeignKeyInnerJoinMultiIntegrationTest.shouldInnerJoinMultiPartitionQueryable

The failed MirrorConnectorsIntegrationSSLTest.testReplication() is not request timeout anymore.
org.opentest4j.AssertionFailedError: Condition not met within timeout 20000. Offsets not translated downstream to primary cluster. ==> expected: <true> but was: <false>

Failed tests are all un-related. (Not in MirrorMakerIntegrationTest)
    Build / JDK 11 and Scala 2.13 / kafka.network.ConnectionQuotasTest.testListenerConnectionRateLimitWhenActualRateAboveLimit()
    Build / JDK 11 and Scala 2.13 / kafka.server.RaftClusterTest.testCreateClusterAndCreateAndManyTopics()
    Build / JDK 8 and Scala 2.12 / org.apache.kafka.streams.integration.StoreQueryIntegrationTest.shouldQueryOnlyActivePartitionStoresByDefault
    Build / JDK 15 and Scala 2.13 / kafka.api.TransactionsTest.testCommitTransactionTimeout()
    Build / JDK 15 and Scala 2.13 / org.apache.kafka.streams.integration.KTableKTableForeignKeyInnerJoinMultiIntegrationTest.shouldInnerJoinMultiPartitionQueryable


Why do I need to specify the token for open-source projects?

From here, it sounds like the token should not be required:

Note: public Travis-CI repos do not need this config file since Coveralls can get the information via their API (using access token exchange).

But if I don't specify the token, I get this error when running on Travis:
coveralls.exception.CoverallsException: Not on TravisCI. You have to provide either repo_token in .coveralls.yml or set the COVERALLS_REPO_TOKEN env var.

Our project is https://github.com/RaRe-Technologies/smart_open

In order for us to detect that we're running in TravisCI (and process the job accordingly, eg. without a token), we need access to the TRAVIS environment variable -- it looks like your tox.ini file does not pass through the values we need here (since tox blacklists all environment variables by default). Adding the following to the relevant tox job config should do the trick:
passenv = TRAVIS TRAVIS_*

Please see this page in our documentation for more info!

Thank you for the reply!
May I suggest you put this information in your README? It's buried a little too deep for someone unfamiliar with coveralls.io and your particular project (I realize they're separate). Here's the path I took when looking for docs:

https://docs.coveralls.io/ (has links to various languages)
https://docs.coveralls.io/python (has links to two projects, one of them yours, "Instructions in README’s.")
https://github.com/coveralls-clients/coveralls-python
Couldn't find any other helpful links to click, so went to https://github.com/coveralls-clients/coveralls-python/issues/new

A link from your README to https://coveralls-python.readthedocs.io would be helpful. The coveralls.io docs should probably point there too, as opposed to your github repo.

@mpenkov good call! The bottom line of our README is

For more information and usage instructions, see our documentation.

but that's definitely a bit small and hard to find. I'll make sure to highlight this a bit more (or feel free to submit a PR, if you're so inclined!).
Unfortunately, I don't have control over any of the coveralls.io docs directly; I'm not actually sure they keep that up to date and I see they link to all other project's Github repos instead of docs as well... ah well, just gives us an even better reason to make sure this is all included in the README directly!

The bottom line of our README is

Wow, I totally missed that.
I'll make a PR.


Feature/flow improvement

throw new \ErrorException("cURL handle does not contain correct headers (user-agent, accept, accept-language)");
the error is missing "Upgrade-Insecure-Requests":
const DEFAULT_HEADERS =
[
"Upgrade-Insecure-Requests",
"User-Agent",
"Accept",
"Accept-Language"
];


Fixes remove-demo test failures

Fixes test failures due to remove-demo issues.

Since this is causing issues with the other PRs, I'm going to merge without review, but I still want @joshuayoes to take a look tomorrow to make sure I didn't screw something up.


When active is activated

Issue in ./help/destinations/ui/activate-batch-profile-destinations.md
"If the segmentMembership.status field is selected, exported files include Active members in the initial full snapshot and Active and Expired members in subsequent incremental exports."
Should this line rather say, "... and NEWLY Active and Expired members in subsequent incremental exports."
Meaning it does not activate every profile in active or expired state, but only the profiles that are "Newly" active or expired as of the last incremental activation.

Thank you for submitting this issue. We will review the documentation and make the necessary updates as quickly as possible. Tracking internally in PLAT-137523.

Closing this issue, as the documentation was updated.


In case using chat.postmessage with python pg. watson assistant doesn't receive mention.

I make the python PG which send mention to slack. But by using python PG. mention doesn't transfer to Watson assistant,I know it by checking improve tab. Are there any settings bot kit?

Please provide a set of reproducible steps.

This issue is fairly old and there hasn't been much activity on it. Closing, but please re-open if it still occurs.


Do you have any plans to support SDL2 and Vulkan?

Hi! I was wondering if you were going to also add Vulkan support later. Preferably, using the ash crate.

Apologies for the late reply. No, I don't have plans to support Vulkan. There is a reason why gl is in the name of this project. :) You may want to check out this project if you wish to plugin a renderer other than gl.


Upgrade to support 8.0 Mio ATXs

Motivation
Upgrade limits to allow for up to 8.0 Mio ATXs
Description
Increase encoding limits to allow for up to 8.0 Mio ATXs.
Test Plan

existing tests pass

TODO

[x] Explain motivation or link existing issue(s)
[x] Test changes and document test plan
[x] Update documentation as needed
[x] Update changelog as needed

bors merge

Flaky test https://github.com/spacemeshos/go-spacemesh/issues/6119
bors merge

Flaky test https://github.com/spacemeshos/go-spacemesh/issues/6120
bors merge

GH failed to download an action during the workflow so build on one of the runners failed
bors merge


Add Custom Commands application as a skill in Bot framework composer application

The documentation (https://docs.microsoft.com/en-us/azure/cognitive-services/speech-service/how-to-custom-commands-integrate-remote-skills) gives an idea about how to export the Custom Commands application as a skill, however, there is no skills tab available in the Speech studio under Custom commands settings.
Is there a way to export our commands app as a skill inside the bot framework application?

Closing as duplicate of https://github.com/microsoft/BotFramework-Composer/issues/9212.


Doesnt work on Members

Hi There, awesome editor. Though i can't get it to work with any datatypes on a member?
I found the following code returns null.
vortoResources.getDataTypeByAlias(currentSection, nodeContext.contentTypeAlias, $scope.model.alias).then(function (dataType2) {
                $scope.model.value.dtdGuid = dataType2.guid;

At this point dataType2 is "null"

Vorto is currently only built to work on Content / Media. I'd need to check what else we need to do, but it may just need us to add another option to this switch statement for member:
https://github.com/umco/umbraco-vorto/blob/master/src/Our.Umbraco.Vorto/Web/Controllers/VortoApiController.cs#L44
If you wanted to pull the code and compile a custom version to check, and create a Pull Request if that does do the trick, that would be awesome. Otherwise I'll try and take a look as soon as I can.

Okay, i was going to make a property editor either anyways. Will look into it.

Thanks Rasmus, any help would be appreciated.

Updated the package, and now it breaks again.
the returned datatype on:
https://github.com/umco/umbraco-vorto/blob/master/src/Our.Umbraco.Vorto/Web/UI/App_Plugins/Vorto/js/vorto.js#L313
Is null again.


Submission.getDuplicates no longer gives 404s

'duplicates/' request needed just the submission id, not the fullname, now corrected

Thanks!


Maximum Monthly API Request Reached.

While the API is running fine on the local development server and not in the production server.
It throws this exception :
The chain resulted in 2 exception(s): Exchanger\Exception\Exception: The maximum allowed API amount of monthly API requests has been reached. Exchanger\Exception\Exception: The currency is not supported or Google changed the response format

If you use a free plan, that simply means that you've reached the limit of 1,000 requests per month.


ng-pristine remains on form after interaction

Using Angular 1.4.7, when a page with a form on it loads, both the form and input fields within the form have the ng-pristine class on them.
After tabbing into and away from the first input, this input loses the ng-pristine class and gains the ng-touched class, but the form element retains the ng-pristine class and does not gain ng-touched.
Surely the class names on the form and its inputs should remain in sync?

I can't reproduce this: http://plnkr.co/edit/1HjZ8YeiBU9Uxgp48zLu?p=preview
The input get $touched = true, which the form doesn't have.

Sorry, I'm talking about the CSS class names that get applied to the form fields. In your Plunk, if you open up the DOM explorer in your browser and inspect the form, you will see that the input gets the class ng-touched after tabbing into and out of the field, but the outer <form> element does not

That's right, $touched is not propagated to the parent form, and that's why ng-touched isn't set on the form. It has nothing to do with ng-pristine, though.
This issue is already tracked here: https://github.com/angular/angular.js/issues/10813

I checked the linked issue and I didn't see any reference to the class names that appear on either the form or the form fields.
I'm sorry, but regardless of the technical implementation, I don't see how the form can be pristine if any of the fields have been touched. It's making styling things, like a label, based on the state of form impossible using pure CSS.
I really feel that this is a genuine issue that the form retains the ng-pristine class when it is clearly no longer pristine. It's unintuitive and feels like a bug.

Like I said, I cannot reproduce what you wrote in your first post. In the plnkr I linked, ng-pristine is not removed in either the input or the form when simply tabbing through the input. Pristine is changed to dirty when something is entered in the input (tabbing doesn't count), and this also changed the classes. This is intended behavior.
Untouched is changed to touched when you tab out of an input. This is currently not propagated to the parent form, and this is tracked in the other issue.

Many apologies, you're right, the ng-pristine class does not change on the form or the input after only tabbing.


Obtain git hash from git_version crate

Use the git_version crate for obtaining the git hash, instead of the custom build-script we used so far. Similar to https://github.com/knurling-rs/probe-run/pull/221

@japaric We discussed the possibility to completely get rid of build.rs with this change. But since we still want to alter the linker-script and insert the version, this isn't really possible is it?

@Urhengulas correct, we need the build script to put the defmt.x linker script in the project's target directory otherwise the linker won't find it. So we want to keep this line even if we don't modify defmt.x in the build script.
https://github.com/knurling-rs/defmt/blob/fde2aab0bc541081d7def42fb049a36fa793a2bc/build.rs#L38

@Urhengulas correct, we need the build script to put the defmt.x linker script in the project's target directory otherwise the linker won't find it. So we want to keep this line even if we don't modify defmt.x in the build script.
https://github.com/knurling-rs/defmt/blob/fde2aab0bc541081d7def42fb049a36fa793a2bc/build.rs#L38

But we want to modify it, don't we? To insert the version into it?

But we want to modify it, don't we? To insert the version into it?

yes, we want to continue doing that. (the last line of my last comment was referring to an hypothetical scenario)


Fix float32 JSON serialization

Fix Error:
Traceback (most recent call last):
  File "/home/vscode/.local/lib/python3.10/site-packages/shiny/session/_session.py", line 665, in _run
    await flush()
  File "/home/vscode/.local/lib/python3.10/site-packages/shiny/reactive/_core.py", line 260, in flush
    await _reactive_environment.flush()
  File "/home/vscode/.local/lib/python3.10/site-packages/shiny/reactive/_core.py", line 178, in flush
    await self._flushed_callbacks.invoke()
  File "/home/vscode/.local/lib/python3.10/site-packages/shiny/_utils.py", line 528, in invoke
    await fn()
  File "/home/vscode/.local/lib/python3.10/site-packages/shiny/session/_session.py", line 1021, in _flush
    await self._send_message(message)
  File "/home/vscode/.local/lib/python3.10/site-packages/shiny/session/_session.py", line 960, in _send_message
    message_str: str = json.dumps(message) + "\n"
  File "/usr/local/lib/python3.10/json/__init__.py", line 231, in dumps
    return _default_encoder.encode(obj)
  File "/usr/local/lib/python3.10/json/encoder.py", line 199, in encode
    chunks = self.iterencode(o, _one_shot=True)
  File "/usr/local/lib/python3.10/json/encoder.py", line 257, in iterencode
    return _iterencode(o, 0)
  File "/usr/local/lib/python3.10/json/encoder.py", line 179, in default
    raise TypeError(f'Object of type {o.__class__.__name__} '
TypeError: Object of type float32 is not JSON serializable
**Object of type float32 is not JSON serializable**

The issue was resolved by globally converting all possible numerical values into a format that is compatible with JSON. This ensures that both existing and future values are always JSON-serializable, preventing any non-JSON-compatible values from causing errors.

The proposed solution is a symptomatic solution rather than a root-cause solution. I understand the problem only occurs in some AnnData objects; otherwise, we would have noticed it during earlier testing. This means that the structural differences should be solvable by converting the datatypes already when reading the input.
I understand that the monkey-patching looks rather elegant, but since json.dumps is a core functionality in networking, there might be negative side effects that we are unaware of - potentially just shifting the problem from one location to the other. Thus, I would not be comfortable merging this solution.

"I understand the problem only occurs in some AnnData objects; otherwise, we would have noticed it during earlier testing"
The problem occurs in all of the AnnData object which we used before for testing too. It also occurs in the commits beforehand for the same AnnData objects, which worked well before.
I will try to fix the issue by converting the datatypes beforehand.


Added Civilshare to Showcase

[x] I have followed (at least) the PR section of the contributing guide.

@UrbanChrisy Well done with this project, thank you for sharing. Unfortunately, it doesn't qualify for the showcase. We are looking for projects that are well ranked in these dimensions: site traffic volume, non-material design look, open source, full usage of the library.


Some basic unit tests for the accuracy benchmarks

Coverage is light, but I focused on things that were relatively clean to test, and that I felt were
the more fragile parts. Future bugs should inform future extensions to the test suite :)
Fixes #601

Looks good!


i want to generate audio using srt file

i want to generate audio using srt file , is there any way of doing that using this lib?

+1


Scripttag not working

Hello,
I seem to have really big trouble adding script tag to the store front. after successful installation nothing seems working. I checked the store page source after installation and I don't see the script that supposed to be appended. I already setup the shopify-app.php
`'scripttags' => [
    [
        'src' => env('SHOPIFY_SCRIPTTAG_1_SRC', 'https://local-app.test/js/shopify-entry.js'),
        'event' => env('SHOPIFY_SCRIPTTAG_1_EVENT', 'onload'),
        'display_scope' => env('SHOPIFY_SCRIPTTAG_1_DISPLAY_SCOPE', 'online_store')
    ],

],`

I also tried having the src link to laravel route like so:
`'scripttags' => [
    [
        'src' => env('SHOPIFY_SCRIPTTAG_1_SRC', 'https://local-app.test/shopify-entry'),
        'event' => env('SHOPIFY_SCRIPTTAG_1_EVENT', 'onload'),
        'display_scope' => env('SHOPIFY_SCRIPTTAG_1_DISPLAY_SCOPE', 'online_store')
    ],

],`

can someone help me explain what supposed to happen to scriptag after installation? I already check thed shopify docs regarding the CRUD for the scriptags.
Thank you

I am having same problem, can you please let me know how did you solve it? @jesephan


Mount test

validate the runtime mount information by using docker/pkg/mount
and validate the rootfs readonly setting

Updated, PTAL @mrunalp @wking

On Thu, Jun 02, 2016 at 02:13:36AM -0700, 梁辰晔 (Liang Chenye) wrote:

Updated, PTAL @mrunalp @wking

3f334dcb → 49ecdec0 is forking and vendoring docker/pkg/mount?  Can't
we get the Docker maintainers to agree on a more neutral place for
shared development?  It seems inefficient to fork.

3f334dc → 49ecdec is forking and vendoring docker/pkg/mount?  Can't
we get the Docker maintainers to agree on a more neutral place for
shared development?  It seems inefficient to fork.

Yes, it is forking and vendoring docker/pkg/mount and does not take much time actually.
A neutral place is an elegant solution, the only disadvantage is it still brings unnecessary code.

The check for mount options doesn't work for strictatime. It is passed as MS_STRICTATIME to the /dev mount syscall as it errors out when passed as an option. We should probably skip checking mount options or only check for ones that actually are visible in the mount output.

On Mon, Jun 06, 2016 at 02:38:39PM -0700, Mrunal Patel wrote:

It is passed as MS_STRICTATIME to the /dev mount syscall as it
errors out when passed as an option…

Just to clarify, that distinction is ‘mountflags’ vs. ‘data’ in
mount(2).  Does the “doesn't work in ‘data’” restriction apply to only
a subset of filesystem-independent mount options 1?

@wking Yes, it mountflags vs data. And yes I would think that the filesystem-independent options may or may not be visible in the output.

Remove 'option' checking for now for three reasons:

some flags like 'strictatime' are invisible but not all flags are not invisible.
I did not see any rule about this in the mount page or document.
options like 'newinstance' is invisible too
Also I did not see any explanation about it.
data like 'a=b' is could be detected, but the output format is a little different (mode=666 vs mode=0666). It will be complicated and may not cover all the differences.

LGTM. We need to remove cgroups mount from runc config which is non-standard. I'll create a follow-on PR for that.


chores(deps): update indirect dependencies

[!WARNING]
Rate limit exceeded
@amery has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 8 minutes and 43 seconds before requesting another review.

⌛ How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.
We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization.
Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.
Please see our FAQ for further information.

📥 Commits
Reviewing files that changed from the base of the PR and between 9f53d80f164334476357e1bae074533372343a79 and adc59845ad5a8ed422721cb05c434abb1568b4a6.

⛔ Files ignored due to path filters (5)

go.sum is excluded by !**/*.sum
x/groupcache/go.sum is excluded by !**/*.sum
x/memcache/go.sum is excluded by !**/*.sum
x/protosink/go.sum is excluded by !**/*.sum
x/simplelru/go.sum is excluded by !**/*.sum

📒 Files selected for processing (5)

go.mod (1 hunks)
x/groupcache/go.mod (1 hunks)
x/memcache/go.mod (1 hunks)
x/protosink/go.mod (1 hunks)
x/simplelru/go.mod (1 hunks)

Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤ Share

X
Mastodon
Reddit
LinkedIn

🪧 Tips
Chat
There are 3 ways to chat with CodeRabbit:

Review comments: Directly reply to a review comment made by CodeRabbit. Example:

I pushed a fix in commit <commit_id>, please review it.
Generate unit testing code for this file.
Open a follow-up GitHub issue for this discussion.

Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:

@coderabbitai generate unit testing code for this file.
@coderabbitai modularize this function.

PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:

@coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
@coderabbitai read src/utils.ts and generate unit testing code.
@coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
@coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.
CodeRabbit Commands (Invoked using PR comments)

@coderabbitai pause to pause the reviews on a PR.
@coderabbitai resume to resume the paused reviews.
@coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
@coderabbitai full review to do a full review from scratch and review all the files again.
@coderabbitai summary to regenerate the summary of the PR.
@coderabbitai generate docstrings to generate docstrings for this PR. (Beta)
@coderabbitai resolve resolve all the CodeRabbit review comments.
@coderabbitai configuration to show the current CodeRabbit configuration for the repository.
@coderabbitai help to get help.

Other keywords and placeholders

Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
Please see the configuration documentation for more information.
If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

Visit our Documentation for detailed information on how to use CodeRabbit.
Join our Discord Community to get help, request features, and share feedback.
Follow us on X/Twitter for updates and announcements.

Why @coderabbitai? I'm on a paid seat


Missing builtins module for RHEL/CentOS/SL (concreate 1.3.0)

I hit following problem during osbs engine build on Scientific Linux 7.3 x86_64:
concreate build --build-engine=osbs
2017-11-24 19:15:12,754 concreate    INFO     Initializing image descriptor...
2017-11-24 19:15:18,966 concreate    INFO     Handling modules...
2017-11-24 19:15:18,975 concreate    INFO     Handling additional repository files...
2017-11-24 19:15:20,056 concreate    INFO     Handling additional repository files...
2017-11-24 19:15:20,671 concreate    INFO     Handling artifacts...
2017-11-24 19:15:21,756 concreate    INFO     Rendering Dockerfile...
Traceback (most recent call last):
File "/usr/bin/concreate", line 9, in
load_entry_point('concreate==1.3.0', 'console_scripts', 'concreate')()
File "/usr/lib/python2.7/site-packages/concreate/cli.py", line 187, in run
Concreate().parse().run()
File "/usr/lib/python2.7/site-packages/concreate/cli.py", line 154, in run
builder = Builder(self.args.build_engine, self.args.target)
File "/usr/lib/python2.7/site-packages/concreate/builder.py", line 23, in new
from concreate.builders.osbs import OSBSBuilder as BuilderImpl
File "/usr/lib/python2.7/site-packages/concreate/builders/osbs.py", line 7, in
from builtins import input
ImportError: No module named builtins
It seems that this issue could be resolved by installing future package for python 2.7.

maybe related to #139, @goldmann  can you take  a look?

Merged, thanks!


 from react-native-svg and from react-native

I used  from react-native, but i need text and from react-native-svg. But conflict arises when they are added through imports. Whether it is possible to rename the  from react-native-svg in  or <Svg.Text>?

import { Text as SvgText } from 'react-native-svg';

Thanks @gorangajic for answering this.
I'm closing this now.


Test hybrid parallelism (OpenMP+MPI)

Currently running this experimentally, but needs more thorough testing.


npm run build not working on windows

Hi, im trying to get your example working. I cloned your git repository, and tried to run mvn clean install, which does not work on my system. I looked at the other issue and changed the npm version to 5.0.4 but it is still failing.
Here is my error log
https://hastebin.com/ifegujapar.tex
npm debug log
https://hastebin.com/xiwuhahatu.tex
Any ideas?

 I´am experiencing the same error with Windows 10. I think, there´s a problem with the npm version on Windows - I thought, that should be gone with the help of a downgrade of npm (see https://github.com/jonashackt/spring-boot-vuejs/issues/1). But I also get that error with the 5.0.4 on Windows 10, so I guess that needs further investigation.
Please feel free to submit pull requests for that - I don´t have much time for that atm.

Ok, I´ll close this for now. Please open again, if the issue remains.


NOISSUE - Add insecure param to cli

Signed-off-by: drasko<EMAIL_ADDRESS>

Codecov Report

Merging #356 into master will decrease coverage by 0.4%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #356      +/-   ##
==========================================
- Coverage   89.17%   88.76%   -0.41%
==========================================
  Files          43       43
  Lines        1469     1469
==========================================
- Hits         1310     1304       -6
- Misses        110      115       +5
- Partials       49       50       +1

Impacted Files
Coverage Δ

ws/api/transport.go
80% <0%> (-7.06%)
:arrow_down:

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 1683d17...172950f. Read the comment docs.


Updates Release 2.15

[x] Gulpfile 2.15.1
[x] Privacy notice 2.15
Internal Tracking ID: EXPOSUREAPP-10950
[ ] FAQ

[ ]  #voluntary_self_quarantine
https://github.com/corona-warn-app/cwa-website/pull/2188
Internal Tracking ID: EXPOSUREAPP-10856
[ ]  #quarantine_measures
https://github.com/corona-warn-app/cwa-website/pull/2188
Internal Tracking ID: EXPOSUREAPP-10857
[x]  #val_service_no_valid_dcc
https://github.com/corona-warn-app/cwa-website/pull/2186
Internal Tracking ID: EXPOSUREAPP-10266
[x]  #val_service_result
https://github.com/corona-warn-app/cwa-website/pull/2186
Internal Tracking ID: EXPOSUREAPP-10266

[x] Screenshots
https://github.com/corona-warn-app/cwa-website/pull/2189
[ ] Blog

@dsarkar Will https://www.coronawarn.app/en/faq/#vac_cert_red_warning be also updated?
See https://github.com/corona-warn-app/cwa-app-ios/pull/3932#discussion_r762852706

@Ein-Tim thanks for pointing this out!


Gotcha in new jcr_version_d rule

With the current ABNF, I think we have a problem if we have a jcr-version directive followed by a number of basic type root rules. e.g.:
#jcr-version 0.6
integer
string

The integer and string root types will be assumed to be part of jcr-version directive.
We can either say this is nothing to worry about as it's a bit of a corner case.  Or we can introduce a DSP type (for directive space) that doesn't allow \n in one-line directive mode, but does in multi-line directive mode.  We'd probably only be able to represent that as a comment.

Or we could put the extensions in brackets (or something similar):
#jcr-version 0.6 (co-constraints-1.0 doc-1.2)
integer
string

Actually, with the ( at the start of (co-constraints-1.0 doc-1.2) we'd probably like the parser to start looking for a group :-(.  Back to the DSP option at the moment.

EOL is in the one line directive, so I don't think this is a problem. I tested this with jcrvalidator and it works.

If I put the following into my tree printer:
#jcr-version 0.6
# ruleset-id my_awesome_rules
integer
string

I get:
[{:directive=>
   {:jcr_version_d=>{:major_version=>"0"@17, :minor_version=>"6"@19}}},
 {:directive=>{:ruleset_id_d=>{:ruleset_id=>"my_awesome_rules"@38}}},
 {:primitive_rule=>{:integer_v=>"integer"@59}},
 {:primitive_rule=>{:string=>"string"@71}}]

which is good.
Whereas, if I put in:
#jcr-version 0.6
integer
string

I lose the integer root items and get:
[{:directive=>
   {:jcr_version_d=>{:major_version=>"0"@17, :minor_version=>"6"@19}}}]

which misses the integer etc.
The following rspec test will demonstrate it:
  it 'should parse jcr-version directive major and minor numbers followed by integer root rule' do
    ex5a = <<EX5a
# jcr-version 4.0
integer
string
EX5a
    tree = JCR.parse( ex5a )
    expect(tree[0][:directive][:jcr_version_d][:major_version]).to eq("4")
    expect(tree[0][:directive][:jcr_version_d][:minor_version]).to eq("0")
    expect(tree[1][:primitive_rule][:integer_v]).to eq("integer")
    expect(tree[2][:primitive_rule][:string]).to eq("string")
  end

which I've added to https://github.com/codalogic/jcrvalidator/tree/jcr-version-gotcha

This is now fixed. The issue was the spaces rules that would eat \r\n. I created a non_eol_spaces rule.

FUDGE! That doesn't work for multi-line. Back to the drawing board. I think perens are the way to go.

The syntax has been changed to have extension ids inside parenthesis characters.

I'm not too sure how much sleep we should lose over this, but...
...entering pedantic mode, with parenthesis it's now ambiguous whether it's the start of a group.
My solution in my C++ code was to introduce a DSPs ABNF rule like:
DSPs =     ; Directive spaces
       1*WSP /     ; When in one-line directive
       sp-cmt      ; When in muti-line directive

Then use it as:
jcr_version_d = jcr-version-kw DSPs major_version "." minor_version
    #!                 [ DSPs extension_id *( DSPs extension_id)  ]

(Side note - I've used sp-cmt rather than just spaces in DSPs.  This allows comments in directives like:
#{ jcr-version 4.0 ; May upgrade later
       foo_1 bar_4 }

That seems worth changing whatever we do.)
If we're not happy with the context sensitive DSPs rule, I think we could safely support the following syntax with ambiguity:
# jcr-version 1.0 + co-constraints-1.2 + jcr-doc-1.0

Once we've decided which route to take, I'm happy to update the ruby code and ABNF accordingly.

Well this small feature has blown up on me, huh?
I like the comments and I like the plus syntax even more than the parens. I welcome your help with the Ruby code and ABNF since I've made a complete hash of things so far.

Pull request submitted.
It's good that it's highlighted something that could have caused us grief in the future.
But hopefully we're all out of good ideas for the time being :-)

this seems to have been resolved


[Request]Show hidden cvars in console

There are so many commands that hidden by VALVE. There are so many cheats that can unlock hidden cvars.

you can still spoof them with cmdunrestricted just make a input box and a button

lol otc v3 have button to show hidden cvars in console


kasts: Add version 24.05-3241

Add manifest for Kasts, desktop & mobile podcast application from KDE.
Closes #11379

[x] I have read the Contributing Guide.

/verify

/verify

/verify


[SPARK-33928][TEST][CORE] Fix flaky o.a.s.ExecutorAllocationManagerSuite - "SPARK-23365 Don't update target num executors when killing idle executors"

What changes were proposed in this pull request?

Use the testing mode for the test to fix the flaky.
Why are the changes needed?

The test is flaky:
[info] - SPARK-23365 Don't update target num executors when killing idle executors *** FAILED *** (126 milliseconds)
[info] 1 did not equal 2 (ExecutorAllocationManagerSuite.scala:1615)
[info] org.scalatest.exceptions.TestFailedException:
[info] at org.scalatest.Assertions.newAssertionFailedException(Assertions.scala:530)
[info] at org.scalatest.Assertions.newAssertionFailedException$(Assertions.scala:529)
[info] at org.scalatest.FunSuite.newAssertionFailedException(FunSuite.scala:1560)
[info] at org.scalatest.Assertions$AssertionsHelper.macroAssert(Assertions.scala:503)
[info] at org.apache.spark.ExecutorAllocationManagerSuite.$anonfun$new$84(ExecutorAllocationManagerSuite.scala:1617)
...

The root cause should be the same as https://github.com/apache/spark/pull/29773 since the test run under non-testing mode.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Manually checked. Flaky is gone by running the test hundreds of times after this fix.

cc @tgravescs @cloud-fan  Please take a look, thanks!

Test build #133464 has started for PR 30956 at commit ba0a4bc.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/38055/

Test build #133464 has finished for PR 30956 at commit ba0a4bc.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/133464/

thanks, merging to master/3.1!


azcopy copy problem with  empty path resources

Which version of the AzCopy was used?
10.18.1
Which platform are you using? (ex: Windows, Mac, Linux)
Linux
What command did you run?
azcopy copy 'https://blobjoylearning.blob.core.chinacloudapi.cn/report//mnt/resource/ap/report/考试成绩查询120181121113950.xlsx' 'https://uattest.file.core.chinacloudapi.cn/globalx-file-shares/report//mnt/resource/ap/report/考试成绩查询120181121113950.xlsx'
What problem was encountered?
dears
when i execute azcopy copy command to  transfer my resources from blob container to file shares, i got http code 400 failure status.when i changed target path "//" in to "/",then it runs with ok state.
the problem is i run with command to transfer all  my blob resources, the "//" path is auto generated, would you please help to  provide a solution? thanks for your great support.
I have posted the log file below for your reference
How can we reproduce the problem in the simplest way?
Have you found a mitigation/solution?
ab4453fe-1223-594f-57a0-9c03345deb65.log

thanks a lot

@gapra-msft thanks for your contribution.

@xuejinx911 apologies for the lack of comment. I must have missed this. Our suggestion would be to fix the user application that is generating the paths to strip the slashes appropriately before passing it to AzCopy.


edgeDNSServer is not used for remote k8gb NS record resolution

We are contacting NS nameserver for cross-cluster communication directly at
https://github.com/k8gb-io/k8gb/blob/master/controllers/providers/assistant/gslb.go#L231-L232
The issue is that does not respect edgeDNSServer configuration option for the NS server fqdn resolution and uses the local one.  It might prevent proper k8gb function in cases when local nameserver is not matching the nameserver under edgeDNS support

Fixed by #516


Use Microprofile BOM/POM #35

Successfully switched java-ee-api 7.0 to microprofile 1.0.0. Everything builds as it did before.
For consistency I changed the groupId to io.microprofile for 1.0.0, the packages are as they were but we cannot rewrite history so it should be fine.
Task-Url: https://github.com/eclipse/microprofile-samples/issues/35

I thought we'd decided that samples and conf demo weren't going to be released with a 1.0 tag as it was deemed too complicated as they're already part of Eclipse

That was also my understanding from the discussions during the Microprofile hangout.
Plus who does really need it? Let's rather focus on 1.1

Do we want to stick with just a release tag and the BOM in one or the other repo?
This is proof, the BOM works. If we don't want to make it available, never mind but except for the groupId the dependency change is like it must be in the future. Whether using this PR or rewriting it, not a big difference. It works using the BOM. If PR rather hides the BOM, well it is out now, so that part of the hangout is done and if it is not meant to be used so widely, I am pretty sure having it in JCenter only makes little difference.

So the 3rd commit puts the version number to a 1.1 Snapshot. As long as no stable 1.1 BOM exists somewhere, this is a working sample (until Java EE 8 API is Final it also had to use EE 7 ;-) so unless somebody wants to completely eliminate these samples by Antonio it works now, CI passes, so we skip tagging them until a 1.1 release may some time be possible (it certainly won't be delayed by those samples as you know)

@kwsutter @jclingan I leave it to you what to do about samples.
The way I use them is at DemoCamps, Conferences, etc. showing people how something works, but also allowing them to use and try it themselves. This has been quite succesful at Eclipse DemoCamps or similar events. Might speak to @ivargrimstad @gunnarmorling @atsticks or others next week during JavaLand how they would like to use them, or if they have their own. Either way the demos should work and at least pass in some CI server (Jenkins in future if we get it running? ;-) even if they may not officially be offered to download. If MP 1.0 BOM exists IMO they should have been tagged accordingly, but I don't really mind if we did it under "org.eclipse" as the last state of the PR. It would only require a 1.0 version number but if we don't tag it till 1.1.x I'd be fine too, but then please tag things that match together.

Werner, now that we're making progress on the MicroProfile 1.1 BOM (in the eclipse/microprofile-bom repo), how about if we update this PR to reference the 1.1.0-SNAPSHOT and get that working?  That would be a good validation of the new bom contents and it would make these Samples more usable.  Thanks.

Will check it out today or tomorrow (it is a long holiday, so a bit of time;-)

Almost there. Ideally I also plan to use some of them for the DWX '17 talk I got with @ivargrimstad next week: http://www.developer-week.de/Programm/Veranstaltung/(event)/21467

The problem is, at least the README file was changed in https://github.com/eclipse/microprofile-samples/pull/40, so I would rather close this and open a new PR after https://github.com/eclipse/microprofile-samples/pull/40 was merged.   @kwsutter when do you think it could be merged?


Revert "fix slido logo color"

Reverts momentum-design/momentum-ui#1601

not merging, since revert is not necessary


Latest version of ComfyUI Front end breaks combine tool node

The latest version of the ComfyUI Front end (v1.2.64) breaks the combine tool node.

Not sure if it's a griptape comfyUI node problem, or a bug with their code - will need to dive in.

In the meantime, launching comfyUI this way works:
python main.py --front-end-version<EMAIL_ADDRESS>

Tried working my way back - looks like things broke in ComfyUI_frontend v 1.2.61.

Fixed by the latest ComfyUI update


Feat: #99 Implement flexible obs columns

In addition to the 'extract flexible columns' feature, type hints, and docstrings have been added to existing functions.

Added a casp.bq_scripts.helpers file with column name preprocessing functions.

Added a comment to #94 based on one of the suggestions during this PR.


Allow Mic and Camera Permissions in websites

Task/Issue URL: https://app.asana.com/0/0/1202399817058484/f
Description
Allow users to enable mic or camera permissions if website requests it
Steps to test this PR
Microphone dialog

Install from this branch
Go to www.mictest.cc
Tap on "Test my cam" button
[ ] Check rationale dialog appears ask you only for mic permissions and allow
[ ] If system permission not granted before -> check system permission dialog appears
Allow permission
[ ] Check microphone is working for site

Camera dialog

Install from this branch
Go to www.webcamtest.cc
Tap on "Test my mic" button
[ ] Check rationale dialog appears ask you only for camera permissions and allow
[ ] If system permission not granted before -> check system permission dialog appears
Allow permission
[ ] Check camera is working for site

Mic & Camera dialog

Install from this branch
Go to webcamera.io
[ ] Check rationale dialog appears ask you for mic and camera permissions and allow
[ ] If system permissions not granted before -> check system permission dialogs appears
Allow permissions
[ ] Check microphone and camera is working for site

Current dependencies on/for this PR:

develop

PR #2035   👈

This comment was auto-generated by Graphite.

Current dependencies on/for this PR:

develop

PR #2035   👈

PR #2165

This comment was auto-generated by Graphite.

Left some comments on the PR.
I was not able to test the PR because it automatically crashes on my device 🤷, could you double check if it works on your side?

Yes, it works fine to me 🤔

Changed SitePermissionsDialogActivityLauncher scope to be single instance in Fragment instead Activity scope. Thanks for the suggestion @cmonfortep 💯

@nalcalag there's a conflict produced by https://github.com/duckduckgo/Android/pull/2322


Import and Frontmatter?

Apologies if I'm just missing something in the docs, I cannot seem to get this to work.
I have a post that contains frontmtter that I also would like to use a component in. Doing this gets the following error: SyntaxError: import declarations may only appear at top level of a module
---
title: Markdown Post Processing (MDX-ish)
date: 2020-05-04
---
<script>
  import Button from './button.svelte'
</script>

And doing this will render the frontmatter to the page with two <hr/>s
<script>
  import Button from './button.svelte'
</script>
---
title: Markdown Post Processing (MDX-ish)
date: 2020-05-04
---

Is it possible to do both?

Just tested with the default svelte template+svelte-preprocess-markdown. Your first example works great. See this example in attachment.
testmd.zip


Upgrade 0.7 to 0.8: deriving Default on enums is experimental

Bevy version
cargo.toml

[package]
name = "tetris"
version = "0.1.0"
edition = "2021"

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

[dependencies]
bevy = "0.8.0"
rand = "0.8.5"

[profile.dev.package."*"]
opt-level = 3

[Optional] Relevant system information
➜  ~ cargo --version
cargo 1.61.0 (a028ae42f 2022-04-29)
OS: macos, Apple M1
What you did
Upgrade 0.7 to 0.8
What went wrong
If it's not clear, break this out into:

what were you expecting?
Compilation complete
what actually happened?

✗ cargo run
   Compiling bevy_reflect_derive v0.8.0
   Compiling bevy_render_macros v0.8.0
   Compiling bevy_encase_derive v0.8.0
   Compiling taffy v0.1.0
   Compiling hashbrown v0.12.3
   Compiling smallvec v1.9.0
   Compiling glam v0.21.2
   Compiling uuid v1.1.2
error[E0658]: deriving `Default` on enums is experimental
  --> /Users/cf/.cargo/registry/src/github.com-1ecc6299db9ec823/bevy_reflect_derive-0.8.0/src/container_attributes.rs:27:17
   |
27 | #[derive(Clone, Default)]
   |                 ^^^^^^^
   |
   = note: see issue #86985 <https://github.com/rust-lang/rust/issues/86985> for more information
   = note: this error originates in the derive macro `Default` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0658]: deriving `Default` on enums is experimental
  --> /Users/cf/.cargo/registry/src/github.com-1ecc6299db9ec823/bevy_reflect_derive-0.8.0/src/field_attributes.rs:25:10
   |
25 | #[derive(Default)]
   |          ^^^^^^^
   |
   = note: see issue #86985 <https://github.com/rust-lang/rust/issues/86985> for more information
   = note: this error originates in the derive macro `Default` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0658]: deriving `Default` on enums is experimental
   --> /Users/cf/.cargo/registry/src/github.com-1ecc6299db9ec823/bevy_render_macros-0.8.0/src/as_bind_group.rs:485:10
    |
485 | #[derive(Default)]
    |          ^^^^^^^
    |
    = note: see issue #86985 <https://github.com/rust-lang/rust/issues/86985> for more information
    = note: this error originates in the derive macro `Default` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0658]: deriving `Default` on enums is experimental
   --> /Users/cf/.cargo/registry/src/github.com-1ecc6299db9ec823/bevy_render_macros-0.8.0/src/as_bind_group.rs:589:10
    |
589 | #[derive(Default)]
    |          ^^^^^^^
    |
    = note: see issue #86985 <https://github.com/rust-lang/rust/issues/86985> for more information
    = note: this error originates in the derive macro `Default` (in Nightly builds, run with -Z macro-backtrace for more info)

error[E0658]: deriving `Default` on enums is experimental
   --> /Users/cf/.cargo/registry/src/github.com-1ecc6299db9ec823/bevy_render_macros-0.8.0/src/as_bind_group.rs:790:10
    |
790 | #[derive(Default)]
    |          ^^^^^^^
    |
    = note: see issue #86985 <https://github.com/rust-lang/rust/issues/86985> for more information
    = note: this error originates in the derive macro `Default` (in Nightly builds, run with -Z macro-backtrace for more info)

For more information about this error, try `rustc --explain E0658`.
error: could not compile `bevy_render_macros` due to 3 previous errors
warning: build failed, waiting for other jobs to finish...
error: could not compile `bevy_reflect_derive` due to 2 previous errors

You need to update your Rust version :)


Feature possiblity: add stdout option to analyzerMode

Not sure if you're open to requests, but I'm mainly looking for a decent summary of the build output in the terminal. This makes most sense on a build server as well because the final build does not need to have a fancy result. Something like the following is more than enough: 16 chunks of 876 kB (parsed: 432 kB | gzipped: 123 kB)

Looks well but I would like to know what is the format of output to stdout?

Hello! 👋 I'm not sure, but are you requesting a summary log for the serve process or the build? If it's for the serve process, it doesn't make sense right now because Vite generates chunks that are not similar for both processes. If it's for the build, I believe this request should be redirected to the Vite repository because it exhibits a similar behavior out of the box

For the build. You triggered me to think about the build, and I found doesdev/rollup-plugin-analyzer that works for me.
Rollup plugins for inspiration:

https://github.com/oscarmarina/rollup-plugin-total-bundlesize (close to my request)
https://github.com/yousifalraheem/rollup-plugin-summary

I apologize for any confusion. I still don't understand why you are hesitant to open an issue in the Vite repository. By default, Vite already provides information about every generated chunk, and it would be beneficial to enhance it with the option to view summary information.

It will be working more nativity and have a more strongest support instead of third-party plugins

off topic if only generate a simple dependencies tree to stdout i think it's interesting feature.

@a3mitskevich Its handled by rollup so they will tell me to take it up with them or to make a plugin. I feel this plugin is there for 98% already.

Alright, no objections from my side. I'm simply sharing my thoughts.

@Jogai @nonzzz Intermediate result. If it's okay, I will proceed to open a pull request.

0.6.0 has been released enjoy it.

Awesome, thanks!


Split up narrative page

From Lauren McNicoll,
Consider splitting the narrative page into three as it is quite long e.g: key assumptions and parameters, sector narratives, and narratives for sovereign exposures

see #104


Is there an alarm message send level configuration?

Set alarm rules through the template and enabled, regardless of the alarm threshold is exceeded, alarm information is sent regularly, which seems very messy.

I guess you missed the .stateChangesOnly property in your alarm definition.
See: https://docs.influxdata.com/kapacitor/v1.4/nodes/alert_node/#statechangesonly
It's purpose to avoid multiple alarms being triggered even without a state change.
Example (and I guess this is what you describe):

OK
CRIT
CRIT
CRIT
OK
...

It will look like this if you use .stateCHangesOnly():

OK
CRIT
OK


Change working directory back to original directory

[~]pwd
/home/jae/tmp
[~]gore
gore version 0.4.1  :help for help
gore> :import os
gore> os.Getwd()
"/tmp/gore-349575424"
<nil>
gore>

When gore starts up it has the working directory of a temp directory. It'd be nice if it switched back to the original directory after startup.
Maybe check the environment for PWD and change to that after startup?  Both bash and zsh set it, and it keeps the pre-gore value.
Thanks.

I believe moving to the temporary directory is necessary not to mess up the go.mod file in the current project.

Hadn't thought of that. I tend to use gore outside of projects as a quick way to test something. This came up as I was wanting to test filesystem calls and had created a set of dirs/files to test under the original directory.
How about a new :command, ":cd "?

Found the issue/PR about why the temp directory... #147

I'm going to close this issue for now as there is a good reason for the existing behavior and there is a workaround... import os; os.Chdir(<path>).

Can this be re-opened?  The workaround doesn't work if the temp directory doesn't have execute permissions, and the reason for changing directory would be that (or if this should be a new bug, I can open one for that too):
gore> os.Chdir("C:\\Users\\myusername\\go\\temp")
fork/exec C:\Users\myusername\AppData\Local\Temp\go-build2952805507\b001\exe\gore_session.exe: This program is blocked by group policy. For more information, contact your system administrator.


Some external links in "Installing as a Windows Service" page is broken

Description:
In the page [1] some external links such as Secure vault link is broken. Please fix those and revisit the content of the page.
[1] https://docs.wso2.com/display/SP400/Installing+as+a+Windows+Service

@RukshiW this is there for more than a month whats the update on this ?

Corrected the link. Changed "If the configurations are set properly for YAJSW, you can see a console output similar to the following, and you can now access the Management Console of your product from your web browser via the relevant hosts and ports." as follows:
"If the configurations are set properly for YAJSW, you can see a console output similar to the following, and you can now access the UIs of your product from your web browser via the relevant hosts and ports."


Kebechet.Maui.RevenueCat.iOS - <IP_ADDRESS> package is broken

If I try to install the package
Kebechet.Maui.RevenueCat.iOS - <IP_ADDRESS>
I got the error
Severity	Code	Description	Project	File	Line	Suppression State	Details
Error		Could not find a part of the path 'C:\Users\diman.nuget\packages\kebechet.maui.revenuecat.ios\<IP_ADDRESS>\lib\net8.0-ios17.2\Maui.RevenueCat.iOS.resources\RevenueCat.xcframework\ios-arm64_x86_64-simulator\RevenueCat.framework\Modules\RevenueCat.swiftmodule\arm64-apple-ios-simulator.private.swiftinterface'.

Hello @dimangulov
thanks for the report. In following days I will take a look.
Deploy on real devices works well ?

The package cannot be installed, because of the error

This seems to be caused by: https://developercommunity.visualstudio.com/t/Allow-building-running-and-debugging-a/351628
I have tried to remove redundant files to shorten final paths and we will see if you will be able to install latest version when I release it

Try https://www.nuget.org/packages/Kebechet.Maui.RevenueCat.InAppBilling/4.5.1
and then let me know if now you can install the package

Looks like the package Kebechet.Maui.RevenueCat.InAppBilling still uses -> Kebechet.Maui.RevenueCat.iOS - <IP_ADDRESS> as a dependency.
If I try to install manually Kebechet.Maui.RevenueCat.iOS - <IP_ADDRESS>, I get the same issue as before
Restoring packages for C:\w\suretint\Dispense\src\Mia.Mobile.Shared.Maui\Mia.Mobile.Shared.Maui.csproj...
GET https://api.nuget.org/v3-flatcontainer/kebechet.maui.revenuecat.ios/index.json
GET https://pkgs.dev.azure.com/hexagonmi/e2c20100-24d5-47f1-976a-24d5ada56982/_packaging/d9925a07-94d0-4765-8831-9bf2590311fb/nuget/v3/flat2/kebechet.maui.revenuecat.ios/index.json
OK https://api.nuget.org/v3-flatcontainer/kebechet.maui.revenuecat.ios/index.json 303ms
GET https://api.nuget.org/v3-flatcontainer/kebechet.maui.revenuecat.ios/<IP_ADDRESS>/kebechet.maui.revenuecat.ios.<IP_ADDRESS>.nupkg
NotFound https://pkgs.dev.azure.com/hexagonmi/e2c20100-24d5-47f1-976a-24d5ada56982/_packaging/d9925a07-94d0-4765-8831-9bf2590311fb/nuget/v3/flat2/kebechet.maui.revenuecat.ios/index.json 359ms
OK https://api.nuget.org/v3-flatcontainer/kebechet.maui.revenuecat.ios/<IP_ADDRESS>/kebechet.maui.revenuecat.ios.<IP_ADDRESS>.nupkg 278ms
System.IO.DirectoryNotFoundException: Could not find a part of the path 'C:\Users\diman.nuget\packages\kebechet.maui.revenuecat.ios\<IP_ADDRESS>\lib\net8.0-ios17.2\Maui.RevenueCat.iOS.resources\RevenueCat.xcframework\ios-arm64_x86_64-simulator\RevenueCat.framework\Modules\RevenueCat.swiftmodule\arm64-apple-ios-simulator.private.swiftinterface'.
at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)

Sorry, my bad I forgot to bump iOS package and link it to main wrapper. Try v4.5.2 (it is in process of publishing):
https://www.nuget.org/packages/Kebechet.Maui.RevenueCat.InAppBilling/4.5.2
+try the simulator run as well, I didnt get the build error for iOS simulator with this version

Works, thanks a lot!

This issue is resolved but potentially it can happen if the main repo has really long path. Workaround for that should be using:
<UseLegacyPathHandling>false</UseLegacyPathHandling>
<UseLongPaths>true</UseLongPaths>

in .csproj
reference:
https://developercommunity.visualstudio.com/t/Allow-building-running-and-debugging-a/351628#T-N10757314


Support for Indexes

Is your feature request related to a problem? Please describe.
Currently we have no ability to set Indexes in the auto migration.
Describe the solution you'd like
The @Column annotation should have a boolean parameter index which by default is set to false.

From my view it could be better to add a @Index annotation to the class itself where a list of the columns included in the index can be defined. Otherwise it would not be possible to handle an index containing more than one column. What do you think about it?

This sounds like a good approach. We should make the @Index annotation repeatable so we can create multiple indexes with multiple columns.


shortedPath(s): use fixed length List where possible

@natebosch PTAL – added another commit and update messages to include perf improvement
Net 1174 -> 1784 iterations/ms: 52% faster!


Integration test using docker compose

Integration test (defined in .github/workflows/test.yml) works calling the script imagemagick-files.py from agent2 communicating with agent1, using the current compose file.

Tests passing: https://github.com/ORNL/zambeze/actions/runs/3888182470

Please close the other PR that is opened against main when you have merged this.


create default storageclaim corresponding to storageclient

at all times we want to ensure storageclaims which creates storageclass that are same as internal mode.
however, if there are more that one storageclient connected we need to provide a way for user to choose which storageclient should host default storageclasses and as such we don't reconcile storageclaims we created once they are deleted by user.
Example:
scenario 1: only one storageclient is connected
controller creates a storageclaim automatically which eventually creates storageclasses with already established names
user can delete this storageclaim and controller will not reconcile providing a way for user to manually create storageclaim

scenario 2: two storageclients are connected
controller creates a storageclaim for one of the storageclients i.e, whichever reconciles first
user can delete that automatically created storageclaim and from then controller will behave exactly as scenario 1

part of [RHSTOR-5553]
Depends on #116

only review last commit.

Minimal testing (only 1 client):
Initial:

--- There are no storageclaims
# kc get storageclient,storageclaim
NAME                                           PHASE       CONSUMER
storageclient.ocs.openshift.io/storageclient   Connected   5a061036-e38a-4358-86bc-4ca4e66f34c8

w/ PR content:

# date --utc; kc get storageclient,storageclaim
Tuesday 19 March 2024 01:27:46 PM UTC
NAME                                           PHASE       CONSUMER
storageclient.ocs.openshift.io/storageclient   Connected   5a061036-e38a-4358-86bc-4ca4e66f34c8

NAME                                                        STORAGETYPE        STORAGEPROFILE   STORAGECLIENTNAME   STORAGECLIENTNAMESPACE     PHASE
storageclaim.ocs.openshift.io/ocs-storagecluster-ceph-rbd   blockpool                           storageclient       openshift-storage-client   Ready
storageclaim.ocs.openshift.io/ocs-storagecluster-cephfs     sharedfilesystem                    storageclient       openshift-storage-client   Ready

--- delete default block claim
# date --utc; k delete storageclaim ocs-storagecluster-ceph-rbd
Tuesday 19 March 2024 01:28:32 PM UTC
storageclaim.ocs.openshift.io "ocs-storagecluster-ceph-rbd" deleted

--- block claim wasn't reconciled
# date --utc; kc get storageclient,storageclaim
Tuesday 19 March 2024 01:28:51 PM UTC
NAME                                           PHASE       CONSUMER
storageclient.ocs.openshift.io/storageclient   Connected   5a061036-e38a-4358-86bc-4ca4e66f34c8

NAME                                                      STORAGETYPE        STORAGEPROFILE   STORAGECLIENTNAME   STORAGECLIENTNAMESPACE     PHASE
storageclaim.ocs.openshift.io/ocs-storagecluster-cephfs   sharedfilesystem                    storageclient       openshift-storage-client   Ready

--- delete default file claim
# date --utc; k delete storageclaim ocs-storagecluster-cephfs
Tuesday 19 March 2024 01:29:24 PM UTC
storageclaim.ocs.openshift.io "ocs-storagecluster-cephfs" deleted

--- file claim wasn't reconciled
# date --utc; kc get storageclient,storageclaim
Tuesday 19 March 2024 01:29:48 PM UTC
NAME                                           PHASE       CONSUMER
storageclient.ocs.openshift.io/storageclient   Connected   5a061036-e38a-4358-86bc-4ca4e66f34c8

--- restart operator
# date --utc; kc get po ocs-client-operator-controller-manager-7d68c875d6-8qccs
Tuesday 19 March 2024 01:30:10 PM UTC
NAME                                                      READY   STATUS    RESTARTS   AGE
ocs-client-operator-controller-manager-7d68c875d6-8qccs   2/2     Running   0          4m12s

# kc delete po ocs-client-operator-controller-manager-7d68c875d6-8qccs
pod "ocs-client-operator-controller-manager-7d68c875d6-8qccs" deleted

# date --utc; kc get po ocs-client-operator-controller-manager-7d68c875d6-s28mq
Tuesday 19 March 2024 01:31:38 PM UTC
NAME                                                      READY   STATUS    RESTARTS   AGE
ocs-client-operator-controller-manager-7d68c875d6-s28mq   2/2     Running   0          65s

--- default claims wasn't recreated
# date --utc; kc get storageclient,storageclaim
Tuesday 19 March 2024 01:31:55 PM UTC
NAME                                           PHASE       CONSUMER
storageclient.ocs.openshift.io/storageclient   Connected   5a061036-e38a-4358-86bc-4ca4e66f34c8

/lgtm


how to inject artoojs to be used in casperjs

Hi
I'd like to use artoo to be used in casperjs (or phantom js) to scrape information. I am able to inject other script and tested ok with it, but have no idea which script I should inject or how to inject in casperjs.
I have got code here:
var casper = require('casper').create();
casper.start("http://mikeyaworski.com/IP", function() {

    // casper.page.injectJs('js/jquery.min.js'); note this one works, i am able to get ip
    // casper.page.injectJs('chrome/content.js'); this one is the one used by chrome extension, not work in this case
    casper.page.injectJs('./artoo_inject.js'); // this one is the bookmarklet I copied to a local file and change the reference of latest.min.js(also saved to local), it does not work for this case
    var ip = this.evaluate(function() {
    var $ = artoo.$;
        return $('#ip').text();
        // return artoo.$('#ip').text();
    });

    this.echo("\nYour public IP address is: " + ip);
});

casper.run();

It doesn't print out the ip but just a null value, I don't know how to debug it. Please can someone from the team give me some advice.
Thanks very much!

some thing to add up. i managed to get true value from
     return artoo._enabled

but any of below will just return null value:
return $('#ip').text();
return artoo.$('#ip').text();

I was thinking if this is  because that the there are quite a few artoo scripts, like artoo.*.js, when inject the artoo.js, it can't actually reference to all linked other scripts,  in this  case, is there a way to just compile all the scripts to one big file?

Hello @stephencheng. It is possible indeed to compile the scripts in one big file. Just take a look at the gulpfile to see what can be done. Concerning the injection, you should take a look at how I do it when using phantomJS here.


Supported Browsers

What browser+version should we support?
Vue.js seems to target Chrome, Firefox, Safari and IE > 8. We should also support Android stock browser, Chrome, Android and Safari on iOS and Edge on Win10.
CI+Fronted Testing:

Vue uses Saucelabs but they don't have free plans.
Phantom.js has headless testing but only for Webkit.
Selenium looks also nice, especially their Node.js binding WebdriverIO

Hm. My priorities would be (from most important to less important):

Safari on iOS

Chrome on Android
Stock Android browser
Firefox on Android
Chrome on Windows 7/10
Firefox on Windows 7/10
Safari on macOS
Edge on Windows 10

I wouldn't bother about IE at all.
As for the CI: Let us discuss it in person on Monday or Tuesday.

Works surprisingly well on IE 11, only the radio buttons/checkboxes are not clickable if they contain math,

Karma!

Uhm, SauceLabs does have free plans: https://saucelabs.com/open-source

Using dist: trusty together with Xvfb, travis is able to start karma and execute the test scripts for

Mozilla Firefox 38.4.0
Chromium 53.0.2785.143

I've found the open source sign-up for Saucelabs. Together with karma-sauce-launcher I was able to test for all platforms except Firefox on Android. I think, Travis can trigger the saucelabs build automatically for us.

Great! Where was it actually?
Quite a pity about Firefox on Android. What's the problem exactly?

The sign-up button on bottom of https://saucelabs.com/open-source goes to https://saucelabs.com/beta/signup/OSS/None which is another one than for the 'Free Trial' sign-up https://saucelabs.com/signup/trial . It's also impossible to enable the open-source plan for accounts created with a Github login.
Saucelabs doesn't tell us which browsers they support on Android, only which emulators they have. https://saucelabs.com/platforms
I think they don't support stock browser either, only 'Chrome Mobile'

Saucelabs browser-matrix is working as well:


RenderDoc installation on Ubuntu 18.04 LTS

Hi there. I'm new to github and I've never compiled anything from source before.  I checked my dependencies were up to date, so there should be no issues there.
I downloaded the renderdoc_1.7.tar.gz tarball but I get lost at the next step:
cmake -DCMAKE_BUILD_TYPE=Debug -Bbuild -H.
make -C build
First, where should I put the tarball for installation?
Second, do I have to extract it first, or do those compilation commands work on a zipped tar?
RenderDoc version: 1.7
Operating System: Ubuntu 18.04 LTS
Graphics API: I have no idea. Yes, I googled it. I tried to find a definition clear enough to be useful to me in the Ubuntu documentation, and I failed.
Any help for a struggling newbie would be greatly appreciated! My ultimate goal is to compile a Blender add-on from source which uses data extracted from RenderDoc to build 3D models. It's open source but it's only available for Windows, so I thought I'd give it a go, but the first step is to get RenderDoc working!
Cheers.

Hi there, your issue has been closed due to not correctly following the issue template. Please read the documentation about filing issues and open a new issue which does follow the issue template. Do not reply to this issue with more information or with a correctly formatted issue, as that will only lead to confusion.
I try to address all github issues and fix bugs quickly, but badly formatted issues or poorly described bug reports especially are a significant drain on project resources. Taking a few minutes to file a good issue makes a huge difference and will get it addressed faster - everybody wins!
In github when you first create a new issue the form is pre-populated with a template which you should read over and follow. As linked above there is documentation going into more detail about what makes a good issue.


Added xSPWeb resource.

I added a resource to provision subwebs to site collections with the most common parameters.

This change is

Reviewed 4 of 5 files at r2.
Review status: all files reviewed at latest revision, 11 unresolved discussions.

Modules/xSharePoint/DSCResources/MSFT_xSPWeb/MSFT_xSPWeb.psm1, line 8 [r2] (raw file):
Can you add the parameter attribute here to specifically mark it as not required like the other properties?

Modules/xSharePoint/DSCResources/MSFT_xSPWeb/MSFT_xSPWeb.psm1, line 74 [r2] (raw file):
It looks like $result is not ever used here so we don't need to have the "$result =" bit at the front of this line

Modules/xSharePoint/DSCResources/MSFT_xSPWeb/MSFT_xSPWeb.psm1, line 78 [r2] (raw file):
A couple of issues with this part - if you are using PsDscRunAsCredential in PowerShell 5 instead of InstallAccount (which is our recommendation) then this will fail. Also I've got something on my radar to add this to xSPWebApplication to list the accounts that should have the process identity rights, so lets take this out of this resource and I'll prioritise up that other extension to cover this scenario.

Modules/xSharePoint/DSCResources/MSFT_xSPWeb/MSFT_xSPWeb.psm1, line 85 [r2] (raw file):
Ensure is optional so this will throw an exception if it isn't specifically provided. In some of our other resources we are adding it to the $PSBoundParameters collection before we hit the Invoke-xSharePointCommand line so we always know it's there inside that block. We do that with:
$PSBoundParameters.Ensure = $Ensure

Modules/xSharePoint/DSCResources/MSFT_xSPWeb/MSFT_xSPWeb.psm1, line 91 [r2] (raw file):
Same comment here about Ensure being optional - fixing the above reference will cover you here as well though

Modules/xSharePoint/DSCResources/MSFT_xSPWeb/MSFT_xSPWeb.psm1, line 97 [r2] (raw file):
Have you tested this where you see two or more of your options changed in one go? I'm sure I've seen scenarios where calling $web.Update() more than once could lead to issues. Either way couldn't we call it once after the if statements so we only write back to SQL once instead of potentially 4 times?

Modules/xSharePoint/DSCResources/MSFT_xSPWeb/MSFT_xSPWeb.psm1, line 143 [r2] (raw file):
You should be testing Ensure as well to see if the web should be removed. Also you'll need to do that same trick to add it to $PSBoundParameters as mentioned above to ensure that the test method we use always sees a value, even when one isn't supplied.

Modules/xSharePoint/DSCResources/MSFT_xSPWeb/MSFT_xSPWeb.schema.mof, line 25 [r2] (raw file):
You need to add descriptions to each of these properties - have a look at all of our other resources to see the formatting.

Tests/xSharePoint/xSharePoint.xSPWeb.Tests.ps1, line 36 [r2] (raw file):
When mocking something generic like New-Object, can you add the parameters check to it so that we ensure we only return that object when a specific type is asked for?
Mock New-Object { ... } -ParameterFilter { $TypeName -eq "Microsoft.SharePoint.SPSite" }

Tests/xSharePoint/xSharePoint.xSPWeb.Tests.ps1, line 113 [r2] (raw file):
We can move this mock to the top of the describe block where you mock the New-Object call as we won't change this mock at all between the contexts

Tests/xSharePoint/xSharePoint.xSPWeb.Tests.ps1, line 135 [r2] (raw file):
Can you move this declaration of $web in to the Mock statement rather than leave it in the context here? Then it will stay at its correct scope

Comments from Reviewable

Reviewable termintes with an exception. Can I so something more?

Just the one point about the install account left to handle on this one

Reviewed 4 of 4 files at r3.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable

I checked in the 'Invoke-xSharePointCommand' function. Eather the credential is present or the user name is correct:
if ($null -eq $Credential) {
    if ($Env:USERNAME.Contains("$")) {
        throw [Exception] "You need to specify a value for either InstallAccount or PsDscRunAsCredential."
        return
    }
}

I now check if the credential is null and use the $env:UserName instead.

This is still labeled "Waiting on author". Anything more for me to do?

@wulfland Sorry dude - we had a holiday weekend and I've been busy. Finalising this review is on my list for later today.

Reviewed 2 of 2 files at r4.
Review status: all files reviewed at latest revision, 1 unresolved discussion.

Comments from Reviewable


NTR : B cell–NK cell precursors

Hi,
I would like to request new ontology term for cell type 'B cell–NK cell precursors'
Label: B cell–NK cell precursors
Synonyms: B-NK
Text definition: A lymphoid lineage restricted progenitor cell with the phenotype Lin– CD34+ CD38– CD45RA+ CD90+ CD7- CD10+
develops_from : CL-0000838: multi lymphoid progenitors (lymphoid lineage restricted progenitor cell)
Definition source: PMID:  23708252 (http://www.nature.com/ni/journal/v14/n7/full/ni.2615.html)
thanks
Avik

Hi,
I would like to close this term request as we have received more updates regarding this cell type. I will submit a new request with the updated cell type name and marker definition.
Apologies for any inconvenience.
thanks
Avik


Fix sub-second output when using colonNotation option

Fix sub-second formatting when applying colonNotation together with secondsDecimalDigits. Closes #52
Ping @Mageer

Thanks for fixing this 🙌🏻


Sync-Dc issue with creating a new dc process with missing prefix

Trying to create a new dc agent using pe instance of cmwell using the following qp configuration in config:
"qp" : "system.parent.parent_hierarchy:/data.thomsonreuters.com,type.rdf:DealsJVContent,ISJOINTVENTURE.JV:true"
Due to the JV prefix being missing on the target environment the dc process fails.
Error in local dc/application.log:
dc.log
Error in ws/application.log
ws.log

No, it's related to dc-sync with qp.
Dc-Sync does a simple check to find out what the destination currently have. When using qp with missing prefix, this check fails and the sync doesn't start at all. It's a catch-22.
Should be documented somehow as a limitation of the dc-sync with qp, or to make some small change to dc-sync manager that only in the case of a missing prefix will consider the target system as not having any of the requested infotons to be sync and start from the beginning.


Second tutorial branch pull request to master

I hope this works

oh this was because of an issue, #2


Make sure file:// is added before source/dest pfn in gfal2 plugin

Created a new function createFinalPFN, which will take care of preparing pfn and appending file:// if needed
a) Also added a check if pfn starts with '/' and in that case append file://
any function createStageOutCommand or createRemoveCommand or removeFile uses new function for preparing pfn. In any case if WMA or CRAB or whatever would call this Stageout plugin without correct pfns, it enforces pfn modification by calling createFinalPFN function.

These 3 new methods don't use protocol at all, so I'd suggest to remove it

you cant from two, only from one new one which is createFinalPFN.

Btw, @amaltaro , please merge #6944 before this since there are likely conflicts otherwise.

yes, missed that. I will rebase after that, just ping me.

Testing it on vocms0230

Justas, can you test it in CRAB as well?
I've just ran a few workflows at Purdue and everything looks good. Thanks.
@ticoann good to merge from my side

test this please

Justas, as foreseen before, can you rebase this PR. Please, also get rid of the protocol unused argument on your 3 new methods.

@juztas can you rebase this PR please?

yes, today. I was on holiday


engine: check iter status when scanning

What have you changed? (mandatory)
Check iter status when scanning. This is a supplement of https://github.com/tikv/tikv/pull/4470
What are the type of the changes? (mandatory)

Bug fix (change which fixes an issue)

How has this PR been tested? (mandatory)
No
Does this PR affect documentation (docs) or release note? (mandatory)
No
Does this PR affect tidb-ansible update? (mandatory)
No

cc @kennytm for importer change.

Please fix the compiling error
error: unused `std::result::Result` that must be used

   --> src/import/stream.rs:247:13

    |

247 |             iter.next();

    |             ^^^^^^^^^^^^

    |

/run-all-tests


Check that the "tools" are accessible before running

Check that the tools are accessible before trying to execute a toolflow.

A more useful error message has been added if the toolflow commands fail.


feat(sbb-header): introduce size s

Draft - to display the logo without text a new property has been added to the sbb-logo component.
If the API change is fine, the final visual result should be validated by @mcilurzo, otherwise a different solution (new SVG? new component?) should be find.
EDIT:

API change is fine (signetOnly is the property proposed name)
no need of a size prop on the header-button/link

ux review:
https://www.figma.com/design/9r6xSfNmEfCFxl1yFYedrj?node-id=43777-147301#935958908
https://www.figma.com/design/9r6xSfNmEfCFxl1yFYedrj?node-id=43777-147442#935950489
https://www.figma.com/design/9r6xSfNmEfCFxl1yFYedrj?node-id=43910-264561#936057771

ux review: https://www.figma.com/design/9r6xSfNmEfCFxl1yFYedrj?node-id=43777-147301#935958908 https://www.figma.com/design/9r6xSfNmEfCFxl1yFYedrj?node-id=43777-147442#935950489 https://www.figma.com/design/9r6xSfNmEfCFxl1yFYedrj?node-id=43910-264561#936057771

@mcilurzo
I added the application name/version in stories size=s, applying some base styling class
may I ask to you to recheck the height? i see it correctly
16px

20px from medium

As discussed via MS Teams:

no changes on the sbb-logo component
add a new padded prop to the sbb-signet to display the logo without text
use the full logo or the signet for the header in size=s is a consumer responsibility, so no changes on header component but only in stories

ux review:
https://www.figma.com/design/9r6xSfNmEfCFxl1yFYedrj?node-id=44464-179452#946527963
https://www.figma.com/design/9r6xSfNmEfCFxl1yFYedrj?node-id=44464-179452#946820512

LGTM. Thank you!


influxdb-python approximates nanoseconds to microseconds when calling write_points in Json format

This is my first time creating the issue. Please forgive for any mistakes

influxdb-python version: 5.2.3
When points are passed to write_points function of InfluxDBClient as JSON data, even if we mention time_precision='n', it approximates the time to microseconds. Same does not happen when passed as line protocol.

tested by inserting timestamps 2019-09-24T03:31:00.100000001Z and 2019-09-24T03:31:00.100000000Z with one replacing the other

This should all be addressed in recent PRs. Please reopen if you still see an issue.


Declaring minimum versions for optional dependencies

We have a large number of optional dependencies which include:

pandas
dask
xarray
datashader
scipy
iris
others that I have forgotten about

For all of these we should declare the minimum versions if necessary. I thought I'd open this issue to note down which versions we should require for each.

pyctdev allows to test packages when installed into various environments with whatever combinations of optional dependencies there might be. You can't test everything because of the time it would take (although you don't have to run all the tests all the time), but if you want to describe one or more such combinations, it should be possible to do (assuming you have tests that would exercise the optional environments, e.g. unit tests, notebooks, etc).

Related to https://github.com/ioam/holoviews/issues/798


vitest should be compatible with webpack.definePlugin

Clear and concise description of the problem
I’m migrating from jest to vitest on an existing webpack based project.
The project is generating 2 outputs : one for the browser and one for node.js
It’s using webpack DefinePlugin to create a few expressions resolved at compile time, like __BROWSER__, set to true for browser output, and false to server output.
Vitest fails when encountering that expression and I wasn’t able to find a way to make it work.
ReferenceError: __BROWSER__ is not defined
Note that using global.__BROWSER__ is not a solution since it won’t be replaced by webpack DefinePlugin.
Suggested solution
It would be great to allow the definition of some global trough config, like we did with jest : https://jestjs.io/docs/configuration#globals-object
Alternative
No response
Additional context
I’ve built a minimal project reproducing the problem here : https://github.com/paulgreg/bug-defineplugin-vitest
You can build it via npm run build and run server code via npm run start retourning « isBrowser: false ».
It has a simple unit test which will fail launching vitest :
ReferenceError: __BROWSER__ is not defined
I’ve also installed jest which will run test fine by overloading globals configuration.
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Use define option: https://vitejs.dev/config/shared-options.html#define
// vitest.config.mjs
export default {
  define: {
    __BROWSER__: true
  }
}

Thank you.
Sorry I’ve missed that in the doc :/