Website with given name func-aibx-mlw already exists - MLOps in-abox

Hi Team,
in the steps of  Create Azure ML Workspace - 01-create-workspace.yml
build faced keep showing the error or the below build
Process completed with exit code 1.
where i can add maul change of - pre-define "Website with given name func-aibx-mlw already exists." ??
B. Rgds
ChanJIn
ERROR: "status":"Failed","error":"code":"DeploymentFailed","target":"/subscriptions/xxxx/resourceGroups/aibx-ml-cj/providers/Microsoft.Resources/deployments/main","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-deployment-operations for usage details.","details":["code":"ResourceDeploymentFailure","target":"/subscriptions//resourceGroups/aibx-ml-cj/providers/Microsoft.Resources/deployments/func-aibx-mlw","message":"The resource write operation failed to complete successfully, because it reached terminal provisioning state 'Failed'.","details":["code":"DeploymentFailed","target":"/subscriptions//resourceGroups/aibx-ml-cj/providers/Microsoft.Resources/deployments/func-aibx-mlw","message":"At least one resource deployment operation failed. Please list deployment operations for details. Please see https://aka.ms/arm-deployment-operations for usage details.","details":["code":"Conflict","target":"/subscriptions//resourceGroups/aibx-ml-cj/providers/Microsoft.Web/sites/func-aibx-mlw","message":"\r\n  "Code": "Conflict",\r\n  "Message": "Website with given name func-aibx-mlw already exists.",\r\n  "Target": null,\r\n  "Details": [\r\n    ***\r\n      "Message": "Website with given name func-aibx-mlw already exists."\r\n    ***,\r\n    ***\r\n      "Code": "Conflict"\r\n    ,\r\n    \r\n      "ErrorEntity": \r\n        "ExtendedCode": "54001",\r\n        "MessageTemplate": "Website with given name 0 already exists.",\r\n        "Parameters": [\r\n          "func-aibx-mlw"\r\n        ],\r\n        "Code": "Conflict",\r\n        "Message": "Website with given name func-aibx-mlw already exists."\r\n      \r\n    \r\n  ],\r\n  "Innererror": null\r\n"]]]

@MarcoABCardoso  @Welasco what I did as fix or workaround is as below,
//2. Deploy Azure Function App (Used to handle Azure Alerts and Invoke GitHub Actions)
// https://docs.microsoft.com/en-us/azure/templates/microsoft.web/sites?tabs=bicep
module functionApp './modules/functionApp.bicep' = {
name: 'func-${amlworkspace}-# ${uniqueSuffix}'
params: {
functionname: 'func-${amlworkspace}-${uniqueSuffix}'
location: resourceLocation
existingStorageAccountName: stg.name
gitHub_FunctionDeploymentZip: gitHub_FunctionDeploymentZip
gitHub_PAT: gitHub_PAT
gitHub_repoName: gitHub_repoName
gitHub_repoOwnerName: gitHub_repoOwnerName
gitHub_workflowId: gitHub_workflowId
resource_group: resourceGroupName
aml_workspace: amlworkspace
aml_flow_deployment_name: aml_flow_deployment_name
aml_endpoint_name: aml_endpoint_name
aml_model_name: aml_model_name
}
}

That looks good! It follows the pattern in other accelerators and should take care of the issue for any future users.
We'll work to get that included in this accelerator, appreciate your input!


There is no way to get the state of the particles in particle effects.

Issue details
We want to store the state of each particle w.r.t time to replay the same effect later, but there is no way to access the particles in particle emitter.
Do let me know if there is a workaround to save/record the particle effect w.r.t.

Not really sure what you're tryin to do. if you whant an effect to be replayer you just spawn it again.
If you're trying to record a scene, record the event triggers for the particle effects instead and replay those.


Can you provide the pretraining log of UCF101?

Hi, I confronted a problem with your code but transfer to some other datasets that the contrastive loss could hardly drop. I wonder if it is a normal phenomenon that such loss could not demonstrate the results of the model or some other problems. If possible, can you provide your log with UCF101 or just provide some details of the loss curve? Thanks.

Well, the loss does drop slightly at the pretraining stage due to the harder objective in our setting. And the loss curve doesn't indicate whether the model is pre-trained well or not. Instead, the results of the downstream tasks can prove it.


require.js

In JSRootCore.js
      // first configure all dependencies
      requirejs.config({
       baseUrl: dir,
       paths: {
          'd3'              : 'd3.v3.min',
          'helvetiker_bold' : 'helvetiker_bold.typeface',
          'jquery'          : 'jquery.min',
          'jquery-ui'       : 'jquery-ui.min',
          'touch-punch'     : 'touch-punch.min',
          'MathJax'         : 'https://cdn.mathjax.org/mathjax/latest/MathJax.js?config=TeX-AMS-MML_SVG&delayStartupUntil=configured',
          'THREE'           : 'helvetiker_regular.typeface'
       },

can you use baseUrl with 'context' like this:
var reqOne = require.config({
  context: "version1",
  baseUrl: "version1"
});
```

(see: http://requirejs.org/docs/api.html#multiversion). When using only 'baseUrl' without 'context', it would overwrite and destroy any other configurations. With the 'context' option, jsroot would be more plugin-like.

Hi
You mean I should provide code like:
 requirejs.config({
   context: dir,
   baseUrl: dir,
   paths: {

Or you prefer to use actual JSROOT version like:
 requirejs.config({
   context: "3.8",
   baseUrl: dir,
   paths: {

Hi,
a version is not necessary, you can use e.g.
 requirejs.config({
   context: "jsroot",
   baseUrl: dir,
   paths: {

Are you sure?
According to :
http://requirejs.org/docs/api.html#config-context
one should use unique identifier for context value.
Let use version number.
Is it ok with you?

You are right. A version number is better.

I check new code with require.js - it does not work :(
I revert changes back.

But still the problem is that baseUrl is a global variable which might lead to errors when adding jsroot to an existing project or framework.
May be the usage of relative paths instead of baseUrl would be a solution. I will test it.

I test usage of require.js with following page:
http://jsroot.gsi.de/dev/indexr.htm?nobrowser&file=../files/danilo6.root&item=canvas;1&opt=
You could try your variants based on it.

This seems to work for me:
https://github.com/asseldonk/jsroot

I will see how your approach can be used.
You propose to avoid baseUrl at all and always use full path to the scripts.
I should verify that such modification does not affect existing code.

Could you check latest code?
I use similar approach as yours, but keeping all kind of special configurations in JSRootCore.js
One benefit now - one could use minified versions of JSROOT scripts with require.js

I think
'rawinflate': dir + 'rawinflate' + ext

is missing in paths of the require config.

Thanks. I add it (including minified version).
By the way, it was working?
Seems to be require.js tries to load script from the same location.

Yes, it works. Thanks a lot.

I have another suggestion for require.js.
      var paths = {
          'd3'                   : dir+'d3.v3.min',
          'jquery'               : dir+'jquery.min',
          'jquery-ui'            : dir+'jquery-ui.min',
          'touch-punch'          : dir+'touch-punch.min',
          'rawinflate'           : dir+'rawinflate'+ext,
          'MathJax'              : 'https://cdn.mathjax.org/mathjax/latest/MathJax.js?config=TeX-AMS-MML_SVG&delayStartupUntil=configured',
          'THREE'                : dir+'three.min',
          'THREE_ALL'            : dir+'jquery.mousewheel'+ext,
          'helvetiker_regular'   : dir+'helvetiker_regular.typeface',
          'helvetiker_bold'      : dir+'helvetiker_bold.typeface',
          'JSRootCore'           : dir+'JSRootCore'+ext,
          'JSRootInterface'      : dir+'JSRootInterface'+ext,
          'JSRootIOEvolution'    : dir+'JSRootIOEvolution'+ext,
          'JSRootPainter'        : dir+'JSRootPainter'+ext,
          'JSRootPainter.more'   : dir+'JSRootPainter.more'+ext,
          'JSRootPainter.jquery' : dir+'JSRootPainter.jquery'+ext,
          'JSRoot3DPainter'      : dir+'JSRoot3DPainter'+ext
       };
       // check if modules are already loaded
       for (var module in paths) {
         if (requirejs.defined(module)) {
           delete paths[module];
         }
       }

      // first configure all dependencies
      requirejs.config({
       paths: paths,
       shim: {
         ...
'''

All the paths are saved in an object called paths. The for loops checks if a plugin is already loadad and removes it in this case from paths.

I implement fix you provided.

Thanks a lot :-)


Maximize a window to center with aspect ratio 4:3

My whish
Maximize a window to center with aspect ratio 4:3

What is the use case here?  You could do this with FancyZones

closing out due to lack of clarity, happy to reopen


[Feature][Resource Center] Slice and buffer write for large data

Search before asking

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

Description
Slice and buffer for large data upload and download
Use case
No response
Related issues
No response
Are you willing to submit a PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

I'd like to design in few days if others think it meaningful. Any suggestions are welcome.


No message on IPC when the monitor layout is changed

Description
When a monitor is layout is changed (eg: with the "monitor" keyword in hyprctl) there is no notification on the socket.
This would enable fixing some corner-case scenarios in scripts (eg: Pyprland's scratchpads) without impacting performances for 99.9% of the use-cases.
In case we have a way to query a single monitor property at the time of this ticket implementation,  a list of monitors that changed their properties should be provided in the message (to enable the "micro optimization").
Different options/ideas:

monitorlayoutchanged
monitorchanged NAME
monitorschanged
monitorschanged NAME1,NAME2

or even more specific:

monitormoved NAME
monitorscaled NAME
monitortransformed NAME
monitorupdated NAME (for everything else like frequency... ?)

monitorlayoutchanged should be enough no?

Yes

I guess


#16 allow case override at individual TokenType level

This is the implementation for https://github.com/TwitchBronBron/brightscript-formatter/issues/16. In my config I have:
"brightscript.format.keywordCaseOverride": {
        "string": "title",
        "float": "title",
        "double": "title",
        "object": "title",
        "boolean": "title",
        "dynamic": "title",
        "integer": "title"
},

and all seems to be working well. Let me know if you see any problems or if I did something wrong. Thanks.
This PR relies on https://github.com/TwitchBronBron/brightscript-formatter/pull/17 for updates in the formatter.

@triwav<EMAIL_ADDRESS>includes the necessary change. Can you update your to include that new version?

@TwitchBronBron updated to 1.5.0 and realized I'd accidentally messed up brightscript.format.compositeKeywords so reverted that. Tested and seems to be working well.

Everything looks good. Thanks for your work on this!

This got deployed in version 1.3.1. Take a look and let me know if that works for you!

Working great. Thanks!

Oh good! Thanks for the quick turnaround on the testing!
On Thu, Dec 6, 2018, 17:21 triwav<EMAIL_ADDRESS>wrote:

Working great. Thanks!
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/TwitchBronBron/vscode-brightscript-language/pull/45#issuecomment-445051204,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACbTbXdQdd4iclws807bTl5RAci_V2xsks5u2ZhkgaJpZM4Y9u8J
.


Failed to download pytorch zip

After I run in Windows 10:
export TORCH_CUDA_VERSION=cu113

And try to run anything using tch-rs, I get this error:
   Compiling torch-sys v0.13.0
error: failed to run custom build command for `torch-sys v0.13.0`

Caused by:
  process didn't exit successfully: `D:\dev\stable-diffusion-burn\target\release\build\torch-sys-9321d07214d78619\build-script-build` (exit code: 1)
  --- stdout
  cargo:rerun-if-env-changed=LIBTORCH_USE_PYTORCH
  cargo:rerun-if-env-changed=LIBTORCH
  cargo:rerun-if-env-changed=TORCH_CUDA_VERSION

  --- stderr
  Error: https://download.pytorch.org/libtorch/cu113/libtorch-win-shared-with-deps-2.0.0%2Bcu113.zip: status code 403

When I try to visit that URL, I get AccessDenied error.

It is because there exists no libtorch > 2.0.0 that supports cuda 11.3.
I guess tch-rs generates the wrong link for the binary. Afaik only torch up to 1.12.1 is supported with cuda 11.3. you can get the binary here, but I have no clue if this will work with pch-rs (probably not if it uses 2.0 feature). https://download.pytorch.org/libtorch/cu113/libtorch-win-shared-with-deps-1.12.1%2Bcu113.zip


Merge r1.1 CHANGELOG and README changes into main

What type of PR is this?
Add one of the following kinds:

subproject management

What this PR does / why we need it:
Merge the CHANGELOG.md create and README.md update from #106 into main, as base for the preparation of r1.2.
Which issue(s) this PR fixes:

Fixes # none
Special notes for reviewers:

closed, branch will be re-used to prepare r1.2


Update request_list.html

#42 Fix

This change fixes the symptoms not the issue. The real issue is bootstrap overflow property being manually overwritten.
Please refer to #93 raised to fix this issue.

we don't need table-responsive class, by default table class and html5 will adapt to this, anyhow #93 💯  👍

@JijinJayakumar Currently, the table-responsive class is not used. It is recommended in bootstrap 3 docs though.
http://bootstrapdocs.com/v3.0.3/docs/css/#tables-responsive
Please open further discussion on this subject at #93 and close this pr. Else clarify.
Cheers.


Fine-tune configuration

Issues #61 and #69 describe how to setup procs generically.
Fine-tune configuration could also be suggested, for example:

Mage's Heating Up can proc during a Hot Streak; currently, this "heating up during hot streak" is not shown because it matches the reference client (Retail) but it could be displayed, should the user want to
Warrior's Overpower currently procs when the spell is usable and the warrior is in the correct stance; but players may want to see the button glow even if not in the good stance, or the other way around (make the button glow only if already in the good stance)
Paladin's Art of War currently procs when the buff is active but, if Exorcism is on cooldown, Retribution paladins may not want to see the spell alert nor the glowing button

The above list is not exhaustive.
As a matter of fact, the goal of this issue is not to implement them, but rather to design a foundation for allowing them to even exist. Once this system will be in place, new issues could be created for each desired configuration.
Issues #61 and #69 should be both implemented first. They are much more demanded and their code will most certainly allow class-based configurations, which would be useful for (and used by) the current issue.

#112 added an option to have both Heating Up and Hot Streak effects at the same time. This option is disabled by default.

Closing this issue which is too vague. It was a good starting point for brainstorming the future of the addon when development was just beginning, but nowadays it looks like a wish list written on a napkin.
For the record, all mentioned points have had their share of resolutions:

Mage's Heating Up "option" has been implemented as the Hot-Streak-Heating-Up option in #112
Warrior's Overpower now has an option to glow either in Combat stance or in any stance, since #121
Paladin's Exorcism now has a dedicated glowing option for the ability itself (on top of talent+ability) since #211

PS. oddly enough, these pull requests are all numbered with 1, 1 and 2 🤯


Implemented a Mini Chatbot

Issues Identification
I have created the PR https://github.com/ML-Fusion-Lab/ML-Fusion-Lab-Website/pull/236 ..for the issue https://github.com/ML-Fusion-Lab/ML-Fusion-Lab-Website/issues/186 that was assigned to me.
Description
This issue involves the development of a mini chatbot for the platform. The chatbot will provide users with an interactive experience, helping them explore ML courses, hands-on projects...
Types of Changes
Please check the boxes that apply

[ ] Bugfix (non-breaking change that fixes an issue)
[x] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update (Documentation content changed)
[ ] Other (please describe):

Checklist
Please check the boxes that apply

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] My changes do not break the current system and pass all existing test cases
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Screenshots

@DivyaJyothi9 pls resolve the conflicts.

Implement a Mini Chatbot for this website #186 raised a PR #236
kindly assign Gssoc extd, Hactoberfest and lavel-3
Thank you @vivekvardhan2810

@DivyaJyothi9 pls resolve the conflicts.

Just now resolved sir,kindly check it.
Thank you

@DivyaJyothi9 can you pls share the screenshot in which you have implemented.

@DivyaJyothi9 can you pls share the screenshot in which you have implemented.

@vivekvardhan2810 code files or final chatbot which was embedded in website?

@vivekvardhan2810

@DivyaJyothi9 the conflicts have arised, pls resolve it and check the deployment preview.

@vivekvardhan2810 sir, Kindly check it now!!

@vivekvardhan2810 kindly assign the labels--Gssoc-extd, Hactoberfest accepted and Level 3
Thank you

@DivyaJyothi9 it's coming like this

chatbot is near adjacent to scroll bar button,
Can you update that

And you can check the above image it's coming like this, pls update that also.

@vivekvardhan2810 should I make chatbot to move little above from it?

yes.

@vivekvardhan2810 can u assign the labels please.

@DivyaJyothi9 i will assign the labels, once you have made changes.

@DivyaJyothi9 have you done.

@DivyaJyothi9 in home page i can't see the chatbot icon, whereas i can see it in remaining pages.

Yes, Now it is coming. @DivyaJyothi9, Good.

yes sir,,i have checked it...now it's visible..u can check it now @vivekvardhan2810

Thank you @vivekvardhan2810


Why is noopContentParser needed?

const noopContentParser: FastifyContentTypeParser = (
  _request,
  payload,
  done,
) => {
  done(null, payload);
};

app.addContentTypeParser('application/json', noopContentParser);
app.addContentTypeParser('*', noopContentParser);

Just wondering what purpose this serves

we don't want fasitfy to parse any formData from incoming requests, so we're just passing it through as is. the reason for both json and * being set is because * doesn't handle json for whatever reason
https://github.com/mcansh/remix-fastify/issues/116#issuecomment-1694540545

@mcansh Maybe the solution is to make remix-fastify work with npmjs.com/package/fastify-raw-body ?

the server is fully in your control, so you're able to use that already if you'd like.
Remix just needs the raw request body in order to parse it. i'm not sure if fastify has a way to only set the content parser for specific routes or not.

without telling fastify to not process the json, it does, and that messes up things down the line. if you want to test out other ways to go about, i'm all for it. in the interim this works
app.addContentTypeParser("application/json", (req, payload, done) => {
  if (req.url === "/api/echo") {
    let body = "";
    req.raw.on("data", (chunk) => {
      body += chunk.toString();
    });
    req.raw.on("end", () => {
      let json = JSON.parse(body);
      return done(null, json);
    });
  }

  return done(null, payload);
});

app.addContentTypeParser(["text/plain", "*"], (request, payload, done) => {
  done(null, payload);
});

app.post("/api/echo", async (request, reply) => {
  let body = request.body;
  return reply.send(body);
});

I think that the ideal setup (given that this is a plugin for Fastify) is that remix-fastify should expect that parsing is already done at the Fastify level and Remix request is constructed using the parsed request.

this hasn't been a plugin for quite a while to allow for situations like this where you need more control over the server.


[Patron Orders] Spam clicking next button sometimes crafts item without order

see discord, maybe due to the orderid pointing to a non existent order and the api defaulting to a non order craft

#740


Error on Create

Describe the bug
It is saying that the method doesn't exist, and errors out.
NPCLib usage option
1
NPCLib JAR name
npclib-plugin-2.0.1-SNAPSHOT.jar
Server version
Paper version git-Paper-641 (MC: 1.13.2) (Implementing API version 1.13.2-R0.1-SNAPSHOT)
You are running the latest version
To Reproduce
Create an NPC using the Lib.
Expected behavior
It would create the npc w/ no errors.
Screenshots
[06:14:04 WARN]: Exception in thread "Thread-51" java.lang.NoSuchMethodError: net.jitse.npclib.NPCLib.createNPC(Ljava/util/List;)Lnet/jitse/npclib/api/NPC;
[06:14:04 WARN]: at net.vast.pvp.modules.npc.DJCreepz.lambda$onEnable$1(DJCreepz.java:31)
[06:14:04 WARN]: at net.jitse.npclib.api.skin.MineSkinFetcher.lambda$fetchSkinFromIdAsync$0(MineSkinFetcher.java:47)
[06:14:04 WARN]: at java.lang.Thread.run(Thread.java:748)

I saw https://github.com/JitseB/NPCLib/issues/28 and saw that, so I tried the plugin, instead of the api, but that didn't work either. I'm probs doing something wrong, idk.

I was looking at the examples too, I basically did the same thing, besides.. what is ids? https://github.com/JitseB/NPCLib/blob/master/examples/example-3.java#L42

Strange issue. NPCLib does indeed load all NMS classes corresponding to your server’s version...
Could you post a snippet of the code you’re using to create the NPC?

        npclib = new NPCLib(VastPvP.getInstance());
        MineSkinFetcher.fetchSkinFromIdAsync(1132172857, skin -> {
            npc = npclib.createNPC(Arrays.asList(new Message("&a&lDJ Creepz").getColored(), new Message("&7How's it Creepin?").getColored(), new Message("&7Ready to get your").getColored(), new Message("&7groove on?").getColored()))
                    .setSkin(skin)
                    .setLocation(new Location(Bukkit.getWorld("world"), 5.5, 183, -6.5, 42, 0))
                    .create();
            Bukkit.getScheduler().runTask(VastPvP.getInstance(), () -> {
                for(Player p : Bukkit.getOnlinePlayers()) npc.show(p);
            });
        });

If it helps i'm using my own API, I export (extract) the plugin (npclib-plugin) into the api (Using IntelliJ's Compile), and then I add the api via gradle. My api is built in maven, and plugin in gradle.

I replicated the issue and no errors appeared. I believe the issue is related to how you build your plugin and stuff. Have you checked to see if your compiled plugin contains the NPCLib class(as well as everything that comes with it)?

I replicated the issue and no errors appeared. I believe the issue is related to how you build your plugin and stuff. Have you checked to see if your compiled plugin contains the NPCLib class(as well as everything that comes with it)?

I've double checked and NPCLib does indeed package everything correctly.
Any chance you're using minimizeJar in your pom.xml build settings?

I don't believe I am. Everything worked fine before 2.0.

So, I tried removing it from my api, and directly shadowing it via gradle, and that didn't do anything.


deletion from array

I added a c program on deletion from an array.
Please accept my pull request.

Contribute more and more projects


maybe a bug with subroutes in hot-reload

With nested routers, modify the component which contains the subroute's outlet — <router-view>, the hot-reloaded page will reproduce the content of <router-view>. If I refresh this page, the behaviour is correct.
I found some others encountered the same problem. Check the second issue have a look at the screen talk.
https://github.com/vuejs/vue/issues/2550
https://github.com/vuejs/vue-loader/issues/217

Same problem here :/

Closing this in favour of #662


How to Recover Deleted Photos from Samsung S7 Edge

"Photos in my Samsung S7 Edge are lost," says Mary,"I was trying to export all the photos from my Samsung S7 Edge to my desktop. After the selected photos were transferred, I have deleted them from my phone before checking the folder on my desktop. When I have actually done that I realized that not all pics were transferred only 60 out of 400. How can I recover my photos now?" Of course there are solutions to this problem such as using an Samsung data recovery program.
To recover deleted photos from Samsung S7 Edge/S7, Samsung Data Recovery is your best choice.
Samsung Data Recovery is the world's No.1 data recovery tool that can directly Recover Deleted Text Messages, Contacts, Videos, Photos, Call logs, Whatsapp Messages and other Document Files from your Samsung Galaxy Phone and SD card.

More info:
Recover Deleted Photos Videos on Galaxy S7/S7 Edge
Recover Deleted Text Messages from Galaxy S7/S7 Edge

Maybe you can have a try of data recovery methods, I think that is a good choice for data recovery. Anyway, if you really want the deleted data back, you should try every possible method. Those recovery tools can also have the possibility to recover deleted data I think.

You can use software to recover you data back. But do remember act fast over it and don't try writing something more over it otherwise the chances of getting your things back will decrease.


Undertow 1.2.8.Final

Core - Full Integration Build 64 is now running using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38

Linux Build 77 is now running using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38

Windows Build 64 is now running using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38

Windows Build 64 outcome was SUCCESS using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38
Summary: Tests passed: 3102, ignored: 68 Build time: 0:20:43

Linux Build 77 outcome was SUCCESS using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38
Summary: Tests passed: 3102, ignored: 68 Build time: 0:22:48

Core - Full Integration Build 64 outcome was FAILURE using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38
Summary: Tests failed: 2 (2 new), passed: 2868, ignored: 375, muted: 1 Build time: 0:51:31
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ws.wsse.signencrypt.EJBSignEncryptTestCase.encryptedAndSignedRequest: <no details avaliable>

org.jboss.as.test.clustering.cluster.sso.ClusteredSingleSignOnTestCase(SYNC-tcp).testNoAuthSingleSignOn: <no details avaliable>

#797 is for master, moving to 1.3.0.Beta1

retest this please

I keep testing this, along with 780 and 811, on 1.x-ignore and keep seeing that EJBSignEncryptTestCase failure. So I'm being cautious.

Core - Full Integration Build 70 is now running using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38

Linux Build 84 is now running using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38

Windows Build 71 is now running using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38

Windows Build 71 outcome was SUCCESS using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38
Summary: Tests passed: 3102, ignored: 68 Build time: 0:21:38

Linux Build 84 outcome was SUCCESS using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38
Summary: Tests passed: 3102, ignored: 68 Build time: 0:24:48

Core - Full Integration Build 70 outcome was FAILURE using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38
Summary: Tests failed: 4 (2 new), passed: 2866, ignored: 375, muted: 1 Build time: 0:54:15
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ws.wsse.signencrypt.SignEncryptMultipleClientsTestCase.encryptedAndSignedRequestFromJohn: <no details avaliable>

org.jboss.as.test.clustering.cluster.web.authentication.FormAuthenticationWebFailoverTestCase(SYNC-tcp).test: <no details avaliable>

org.jboss.as.test.integration.ws.wsse.signencrypt.EJBSignEncryptTestCase.encryptedAndSignedRequest: <no details avaliable>

org.jboss.as.test.clustering.cluster.sso.ClusteredSingleSignOnTestCase(SYNC-tcp).testNoAuthSingleSignOn: <no details avaliable>

I will investigate

Retest this please

Windows Build 75 is now running using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38

Core - Full Integration Build 75 is now running using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38

Linux Build 88 is now running using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38

Windows Build 75 outcome was SUCCESS using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38
Summary: Tests passed: 3102, ignored: 68 Build time: 0:20:43

Linux Build 88 outcome was SUCCESS using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38
Summary: Tests passed: 3102, ignored: 68 Build time: 0:22:44

Core - Full Integration Build 75 outcome was FAILURE using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38
Summary: Tests failed: 5 (3 new), passed: 2864, ignored: 375, muted: 2 Build time: 0:51:09
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ejb.remote.client.api.interceptor.EJBClientInterceptorTestCase.testEJBClientInterceptionFromInVMClient: <no details avaliable>

org.jboss.as.test.integration.ejb.remote.jndi.RemoteEJBJndiBasedInvocationTestCase.testRemoteSLSBInvocation: <no details avaliable>

org.jboss.as.test.integration.ejb.remote.jndi.RemoteEJBJndiBasedInvocationTestCase.testRemoteSFSBInvocation: <no details avaliable>

org.jboss.as.test.integration.ws.wsse.signencrypt.EJBSignEncryptTestCase.encryptedAndSignedRequest: <no details avaliable>

org.jboss.as.test.clustering.cluster.web.authentication.FormAuthenticationWebFailoverTestCase(SYNC-tcp).test: <no details avaliable>

EJBSignEncryptTestCase seems to be failing consistently in other PR's as well. I don't think this is undertow related

Retest this please

Linux Build 89 is now running using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38

Core - Full Integration Build 76 is now running using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38

Windows Build 76 is now running using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38

Windows Build 76 outcome was SUCCESS using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38
Summary: Tests passed: 3102, ignored: 68 Build time: 0:20:44

Linux Build 89 outcome was SUCCESS using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38
Summary: Tests passed: 3102, ignored: 68 Build time: 0:23:04

Core - Full Integration Build 76 outcome was FAILURE using a merge of f9b6e32f4b5b12421b84348917334c1a8a9add38
Summary: Tests failed: 2, passed: 2867, ignored: 375, muted: 2 Build time: 0:51:42
Build problems:
Failed tests detected
Failed tests
org.jboss.as.test.integration.ws.wsse.signencrypt.EJBSignEncryptTestCase.encryptedAndSignedRequest: <no details avaliable>

org.jboss.as.test.clustering.cluster.web.authentication.FormAuthenticationWebFailoverTestCase(SYNC-tcp).test: <no details avaliable>


Update easy-move-plus-resize to 1.1.0

After making all changes to the cask:

[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.
[x] The commit message includes the cask’s name and version.

Thank you for the contribution. It was merged directly as abc9cdcaaa178c2c9b404091815c7e05e95af54e. Your contribution is still credited to you.


Feature request: Please add support of MySQL

Hi
You've wrote:

Supported back-end is at this time PostgreSQL only, the most popular back-end for Zabbix.

I think the most popular back-end for Zabbix is MySQL...

@banzayats, support of MySQL is now OK.
Could u please check and let me know ?


edupont04 from OPS portal: Merge from master to live

Docs Build status updates of commit a5d8c2e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

dynamics-gp/distribution/purchase-order-processing-part3-receipts.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Feature.sps.removable particle

New feature for the expandable SPS : removable particles.
Usage :
    // expandable sps creation with 1000 boxes
    var sps = new BABYLON.SolidParticleSystem("sps", scene, {expandable: true});
    var model = BABYLON.MeshBuilder.CreateBox("m", {}, scene);
    sps.addShape(model, 1000);
    sps.buildMesh();

     // ... further in the code
    sps.removeParticles(700, 999);  // removes the last 300 particles
    sps.removeParticles(0, 9);      // then removes the first 10
    sps.buildMesh();                // like for adding particles after the sps creation,
                                    // the call to buildMesh() is required

[EDIT] : please don't merge now, I think there's still a little bug to fix about the particle 0.

There is a missing comment for doc ;)


Widgets do not get downloaded in CodeSpaces

Waiting for codespaces/vscode to resolve CORS issue.

We should check if abovementioned CORS issue has been fixed.

Looks good. Got the third party warning and it worked:


Feature/issue 1057 generalize indexing

Summary:
Allow multi-indexing in expressions and assignment statements.
Intended Effect:
Adds indexing functions, types, and extends language to allow multiple indexes (lower bounds, upper bounds, lower and upper bounds, array of int, all) to both expressions and statements.
The API code is in stan/model/indexing contains the following functionality:

index.hpp: structures for single and multiple indexing
index_list.hpp: sequence of indexes
deep_copy.hpp:  deep copy arbitrary Stan data types (avoids aliasing)
rvalue_index_size.hpp: metaprogram for size of result container given index and size of underlying container
rvalue_at.hpp: map index object plus result position to underlying position
rvalue_return.hpp: metaprogram for calculating return type of indexing
lvalue.hpp: assignment value to variable given an index list

The structures in index lists are mirrored in the abstract syntax tree (stan/lang/ast.hpp) and it also contains code for type inference.  There's also a new program to compute whether a variable shows up in an expression to detect the possibility of aliasing.
The C++ code generator changes are surprisingly straightforward and in the usual place (stan/lang/generator.hpp) --- the only hitch is checking if the left-hand variable occurs in the right-hand side expression, which causes the right side to be wrapped in a deep copy.  The harder part is generating neat code for user-facing and code-generation purposes --- there is separate logic for expressions printed for users and for C++.
The parser code is in `stan/lang/grammars/':

indexes_grammar...: grammar for sequence of single and multiple indexes
statement_grammar...: adds multiple assignment case with semantic actions
term_grammar...: adds case for multiple indexed expression with semantic actions

In the statement and term grammar, the original logic remains, but now no loger has expectation operators > but just concatenation >> so that failure to match can backtrack to the multiple indexing.  This means multiple indexing has to deal with informative error messages.
How to Verify:
There are full unit tests for the boundary conditions in the API and template metaprograms.  There are also exhaustive unit tests for up to three dimensions of indexes both at the API level and the grammar/parser level that verify values and sizes are all correctly assigned.  (There's a repeated tested idiom with helper functions for expressions and statements and a whole lot of cut and paste and emacs macros in the grammar examples.)
Side Effects:
Error messages are slightly different for indexing errors because they are handled in the multi-indexing blocks.  Tests have been updated to reflect this change.
Expressions and assignment statements with no multiple indexing (that is, no indexing or just single integer indexes) will be parsed and code-generated as before.
Documentation:
New manual chapter for user's guide and new chapter for reference manual;  updated BNF description and added some cross-references.
API functions all documented.
Reviewer Suggestions:
Daniel (he'll be pickiest and this one should be looked at pretty closely)

@bob-carpenter, this failure isn't the fault of this pull request. It looks like we snuck in some compiler warnings prior to this and Jenkins wasn't smart enough to pick it out. I think the warnings have come from the new clang++ compiler on Mac.

I take it back... Bob, does this run on your machine:
make src/test/test-models/good/indexing/good-ints.hpp-test
It first compiles src/test/test-models/good/indexing/good-ints.stan to src/test/test-models/good/indexing/good-ints.hpp. Then it tries to compile the header file.
Mine doesn't compile.

Nope, broken here, too.  I'll fix and push a patch.
Is there a make target for the integration tests?  I don't see
it with "make help" --- shouldn't it be under "Tests:"?

Bob

On Nov 10, 2015, at 10:47 PM, Daniel Lee<EMAIL_ADDRESS>wrote:
I take it back... Bob, does this run on your machine:
make src/test/test-models/good/indexing/good-ints.hpp-test
It first compiles src/test/test-models/good/indexing/good-ints.stan to src/test/test-models/good/indexing/good-ints.hpp. Then it tries to compile the header file.
Mine doesn't compile.
—
Reply to this email directly or view it on GitHub.

It's just:
./runTests.py src/test/integration
(or ./runTests.py src/test/integration/compile_models_test.cpp)
I did some make magic to get all the good models to be run through stanc
then compiled with the C++ compiler.
On Tue, Nov 10, 2015 at 11:19 PM, Bob Carpenter<EMAIL_ADDRESS>wrote:

Nope, broken here, too. I'll fix and push a patch.
Is there a make target for the integration tests? I don't see
it with "make help" --- shouldn't it be under "Tests:"?

Bob

On Nov 10, 2015, at 10:47 PM, Daniel Lee<EMAIL_ADDRESS>wrote:
I take it back... Bob, does this run on your machine:
make src/test/test-models/good/indexing/good-ints.hpp-test
It first compiles src/test/test-models/good/indexing/good-ints.stan to
src/test/test-models/good/indexing/good-ints.hpp. Then it tries to compile
the header file.
Mine doesn't compile.

—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/stan-dev/stan/pull/1681#issuecomment-155657016.

Cool,

./runTests.py -j4 src/test

seems to kick everything off.  May I update the make doc while
I'm at it to make this more visible?

Bob
./runtests.py

On Nov 10, 2015, at 11:26 PM, Daniel Lee<EMAIL_ADDRESS>wrote:
It's just:
./runTests.py src/test/integration
(or ./runTests.py src/test/integration/compile_models_test.cpp)
I did some make magic to get all the good models to be run through stanc
then compiled with the C++ compiler.
On Tue, Nov 10, 2015 at 11:19 PM, Bob Carpenter<EMAIL_ADDRESS>wrote:

Nope, broken here, too. I'll fix and push a patch.
Is there a make target for the integration tests? I don't see
it with "make help" --- shouldn't it be under "Tests:"?

Bob

On Nov 10, 2015, at 10:47 PM, Daniel Lee<EMAIL_ADDRESS>wrote:
I take it back... Bob, does this run on your machine:
make src/test/test-models/good/indexing/good-ints.hpp-test
It first compiles src/test/test-models/good/indexing/good-ints.stan to
src/test/test-models/good/indexing/good-ints.hpp. Then it tries to compile
the header file.
Mine doesn't compile.

—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/stan-dev/stan/pull/1681#issuecomment-155657016.

—
Reply to this email directly or view it on GitHub.

Yes, please.

On Nov 11, 2015, at 12:14 PM, Bob Carpenter<EMAIL_ADDRESS>wrote:
Cool,

./runTests.py -j4 src/test

seems to kick everything off. May I update the make doc while
I'm at it to make this more visible?

Bob

./runtests.py

On Nov 10, 2015, at 11:26 PM, Daniel Lee<EMAIL_ADDRESS>wrote:
It's just:
./runTests.py src/test/integration
(or ./runTests.py src/test/integration/compile_models_test.cpp)
I did some make magic to get all the good models to be run through stanc
then compiled with the C++ compiler.
On Tue, Nov 10, 2015 at 11:19 PM, Bob Carpenter<EMAIL_ADDRESS>wrote:

Nope, broken here, too. I'll fix and push a patch.
Is there a make target for the integration tests? I don't see
it with "make help" --- shouldn't it be under "Tests:"?

Bob

On Nov 10, 2015, at 10:47 PM, Daniel Lee<EMAIL_ADDRESS>wrote:
I take it back... Bob, does this run on your machine:
make src/test/test-models/good/indexing/good-ints.hpp-test
It first compiles src/test/test-models/good/indexing/good-ints.stan to
src/test/test-models/good/indexing/good-ints.hpp. Then it tries to compile
the header file.
Mine doesn't compile.

—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub
https://github.com/stan-dev/stan/pull/1681#issuecomment-155657016.

—
Reply to this email directly or view it on GitHub.

—
Reply to this email directly or view it on GitHub.

@bob-carpenter, this all looks great! Thanks for adding the doc and the tests.


GitHub: Invite member

Who should be invited to Catena-X ng
GitHub User:
@SCHLANT (Anton Schloesser (SAP)) - works on the Demand and Capacity Management Use Case
Vouching person:
evgenyvinokurov

Hi @evgenyvinokurov  could you please verfiy the new user @schlant
Thanks in advance

Hi @evgenyvinokurov could you please verfiy the new user @schlant
Thanks in advance

Hello @FaGru3n, what exactly do you need from me for the verification of Anton Schloesser?

In gernerall we just need any form from validation that the user is related to Catena-X Member. 😉

User @schlant (Anton Schloesser (SAP)) was invited.
@schlant Please Update your GitHub profile with clear names, makes us easier to validate you. Thanks in advance.

@evgenyvinokurov  Please verify the succeeded invitation and close the issue. 😉

@FaGru3n Anton Schloesser was added to the standardization repository

User @schlant (Anton Schloesser (SAP)) was invited.
@schlant Please Update your GitHub profile with clear names, makes us easier to validate you. Thanks in advance.

Hi Fabian, jo hab mein Profil geupdated
Danke fürs hinzufügen!
VG;
Anton

Hi Fabian, jo hab mein Profil geupdated
Danke fürs hinzufügen!
VG;
Anton

Sure no problem 👍🏼  FYI Mails will redirected directly to GitHub


more code reuse/base classes in the *Managers would help

NetworkManager, StorageManager, etc share an awful lot of the same code that's asking to be genericized, perhaps...

done enough for now


Embedding fail on Graph

I failed to add an Embedding layer on Graph model.
Here's the code that works:

from keras.models import Sequential, Graph
from keras.layers.core import Dense
from keras.layers.embeddings import Embedding
lookup = Sequential()
lookup.add(Embedding(2, 1))
lookup.compile(loss='binary_crossentropy', optimizer='sgd')

And here's the code that fails:

from keras.models import Sequential, Graph
from keras.layers.core import Dense
from keras.layers.embeddings import Embedding
model = Graph()
model.add_input('input')
model.add_node(Embedding(2,1), input='input', name='emb')
model.add_output(name='output', input='emb')
model.compile('sgd', {'output':'binary_crossentropy'})

The error message is:

Traceback (most recent call last):
  File "./test.py", line 16, in
    model.compile('sgd', {'output':'binary_crossentropy'})
  File "/usr/local/lib/python2.7/dist-packages/keras/models.py", line 525, in compile
    y_train = output.get_output(True)
  File "/usr/local/lib/python2.7/dist-packages/keras/layers/embeddings.py", line 62, in get_output
    out = self.W[X]
  File "/usr/local/lib/python2.7/dist-packages/theano/tensor/var.py", line 474, in __getitem__
    return self.take(args[axis], axis)
  File "/usr/local/lib/python2.7/dist-packages/theano/tensor/var.py", line 506, in take
    return theano.tensor.subtensor.take(self, indices, axis, mode)
  File "/usr/local/lib/python2.7/dist-packages/theano/tensor/subtensor.py", line 2290, in take
    return take(a, indices.flatten(), axis, mode).reshape(shape, ndim)
  File "/usr/local/lib/python2.7/dist-packages/theano/tensor/subtensor.py", line 2268, in take
    return advanced_subtensor1(a, indices)
  File "/usr/local/lib/python2.7/dist-packages/theano/gof/op.py", line 509, in __call__
    node = self.make_node(*inputs, **kwargs)
  File "/usr/local/lib/python2.7/dist-packages/theano/tensor/subtensor.py", line 1608, in make_node
    raise TypeError('index must be integers')
TypeError: index must be integers

Bleeding edge theao is used.

add_input(name, ndim=2, dtype='int')

will tell the Graph to expect an integer matrix input.

One problem is when ndim=3. Then you get:
Exception: Type "int" can only be used with ndim==2 (Embedding).
I encountered this when trying to reproduce examples/lstm_text_generation.py using a Graph model instead of Sequential, and then adding an Embedding node before the LSTM node. I don't know how to make this work, so I've asked for help in a post to keras-users.
Do you have an idea Francois?
Thanks in advance.
Cyril.

@ipod825 hi, did you work out the "TypeError: index must be integers" problem? I have that quenstion too.

@baobao7766 , yeah, as @fchollet answered.
add_input(name, ndim=2, dtype='int')

@ipod825 OK! Thands!

@ipod825 OK, Thanks!

When I do this, I am unable to store the graph using to_json, getting a TypeError: <type 'int'> is not JSON serializable.
(I tried to use the solution at https://stackoverflow.com/questions/3488934/simplejson-and-numpy-array/24375113#24375113 to address, but then I started getting RuntimeError: maximum recursion depth exceeded. Raising max rec to 10k didn't fix that so I suspect something's inf-looping.)
Why wouldn't to_json work just because you switch to an int? I'm confused.


About resources

Best resources for learning react js.

Hey Mihir,
One of the best place to learn React is the React docs
After that there are a lot of free tutorials on youtube, which you can follow. I would recommend the following two channels:

Codedamn
Codevolution

Freecodecamp also has a free thorough curriculum for react, do check that out.
Hope this helps you 😇

Thanks..


Home page doesn’t render on Firefox mobile

This is what a new user sees when loading on firefox mobile. Requesting the desktop site fixes this.
iPhone 8
iOS 14.3
Firefox Mobile Daylight 31.0

Some further testing reveals that no pages render without requesting the desktop sitr

Maybe it's iOS only problem? Because it works fine on Firefox Android 9, MIUI 12, Xiaomi Redmi Note 8T.

try it without the slash at the end and see if that is the problem

could you see what is in your console

The console isn't available on firefox mobile for iOS.

I opened the console in Edge for android, and got this error:

It works if i request the desktop website, although that is a bad experience on mobile.

This appears to be fixed? Can someone else confirm? If so I’ll close it.

This appears to be fixed? Can someone else confirm? If so I’ll close it.

Yes this is fixed for me at least! :)


Angular CLI/Webpack not working with Phosphor

We've had great luck using Angular2 and Phosphor together, using the SystemJS loader. But the Angular industry seems to have moved to Angular-CLI and Webpack, so we are trying to do the same change.
Unfortunately, I'm running into a show-stopping problem. From what I've learned about Angular CLI, it looks like I need to add the Phosphor scripts I use to the .angular-cli.json file, like so:
      "styles": [
        "styles.css",
        "../node_modules/phosphor/styles/base.css"
        ],
      "scripts": [
        "../node_modules/phosphor/lib/core/messaging.js",
        "../node_modules/phosphor/lib/algorithm/iteration.js",
        "../node_modules/phosphor/lib/ui/widget.js",
        "../node_modules/phosphor/lib/ui/boxpanel.js",
        "../node_modules/phosphor/lib/ui/dockpanel.js",
        "../node_modules/phosphor/lib/ui/widget"
      ],

Adding these .js files made a lot of my browser errors go away, but I'm still getting the error Uncaught Error: Cannot find module '../algorithm/iteration'.   That relative path is in the Phosphor code, in a number of places. Has anyone had any luck getting Webpack and Phosphor working together?
Thanks,
John

We use webpack exclusively for Phosphor example and tests, and also for JupyterLab.
Be advised, that we recently split phosphor up into @phosphor/* namespace packages, all managed under this repo.
@blink1073 can maybe link you to an example webpack config which consumed phosphor as an npm-installed package, rather than a local package.

I'm going to close this because it's not relevant for the Phosphor code base, but feel free to keep exchanging ideas.

Here's an example.

Notice that there is nothing phosphor-specific in that config. Just treat phosphor like any other package.

Thanks for the quick response. Note though that I'm using Angular CLI, which provides a wrapper around WebPack, including the configuration file.  Thus, I don't have a webpack.conf.js file in my project. So, I'm trying to figure out what needs to be in my .angular-cli.json  file.
(See<EMAIL_ADDRESS>I've posted a similar question the Angular CLI team to see if they have any ideas: https://github.com/angular/angular-cli/issues/5084
Has anyone gotten Phosphor to work with CLI?

I'm surprised it doesn't "just work" out of the box, since the phosphor files should be treated like any other npm module. I'm not familiar at all with Angular or CLI, but unless they just dont allow 3rd party libs at all, I don't see why it would cause any trouble.

They definitely support third-party packages; that's why the "styles" and "scripts" sections are in their configuration file. I was surprised too that it didn't work. Hopefully the CLI team can provide an answer. It would be nice if you could provide a search path somehow in their configuration file, not explicitly list each .js file as I seem to need to. I'll keep fiddling...

So, it appears that uninstalling 'phosphor' and then installing '@phosphor/widgets', and making some changes for how things moved around, has fixed this problem. Phosphor appears to be loading fine now, although the stylesheets for the docking don't seem to be working right. I'm guessing the stylesheet, that I took from one of your examples, is no longer valid. Is there a newer example around?

The required styles are included with the packages in the /style dir. There are currently two: @phosphor/dragdrop/style/index.css and @phosphor/widgets/style/index.css, unfortunately webpack doesn't yet handle the style key in package.json, so you'll need to use those full paths: https://github.com/webpack-contrib/css-loader/issues/51
This example dir contains the updated example styles, which are in addition to the required CSS above:
https://github.com/phosphorjs/phosphor/tree/master/example/style
Here's where I import the required CSS:
https://github.com/phosphorjs/phosphor/blob/master/example/style/index.css#L8-L9
And here's where I pull in the final stylesheet for the example:
https://github.com/phosphorjs/phosphor/blob/master/example/src/index.ts#L20
And here's how webpack is configured to bundle those styles:
https://github.com/phosphorjs/phosphor/blob/master/example/src/index.ts#L20

Great, thanks @sccolbert , that helped.


fix: update to eye v4.5.1

fix: update to eye v4.5.1

:tada: This PR is included in version 5.5.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v5.5.1

Your semantic-release bot :package::rocket:


Bucket notifications not being created

Bucket notifications are not being created on an already existing bucket. I'm currently in the process of migrating my CDK app over to SST. One thing I've noticied is that my bucket notifications aren't being created using sst dev or sst deploy.

Nevermind, not sure what I changed but it's working now.


Move SetTestPipelineVersion.ps1 to eng/common

Move SetTestPipelineVersion.ps1 to eng/common.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


Ng2EmojiModule is not an NgModule

when I try to use this library in my angular 2.3.1 project, I manage to run npm install ng2-emoji --save successfully, but then ng serve fails with:
chunk    {0} main.bundle.js, main.bundle.map (main) 187 kB {2} [initial] [rendered]
chunk    {1} styles.bundle.css, styles.bundle.map, styles.bundle.map (styles) 4.74 kB {3} [initial] [rendered]
chunk    {2} vendor.bundle.js, vendor.bundle.map (vendor) 3.21 MB [initial] [rendered]
chunk    {3} inline.bundle.js, inline.bundle.map (inline) 0 bytes [entry] [rendered]

ERROR in Ng2EmojiModule is not an NgModule
webpack: bundle is now VALID

I restarted ng serve after I installed the ng2-emoji, but the problem persists.

Same error after installing and try to build it in IONIC2 RC4.  Why  Ng2EmojiModule is not not an NgModule whereas the document say that we can import it in our module?

Hi @stchoukeu , @petervojtek . Really sorry for the delay. Couldn't get time before.
I'm gonna look into it today.
Do let me know if you're still facing the issue

Hi @stchoukeu , @petervojtek .
I've just installed and tested in an Angular 2.4.1 project.
Works like a charm.
Do let me know if you're still facing issues.

Hi Ahsan,
Thank you for your reply. i want to know if your module is compatible with
IONIC2.  because i complete the doc as you explained in your Github but it
seem that noting has change there since my last test. i make a test again
but notting.
2017-02-26 18:33 GMT+01:00 Ahsan Ayaz<EMAIL_ADDRESS>
Hi @stchoukeu https://github.com/stchoukeu , @petervojtek
https://github.com/petervojtek .
I've just installed and tested in an Angular 2.4.1 project.
Works like a charm.
Do let me know if you're still facing issues.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AhsanAyaz/ng2-emoji/issues/2#issuecomment-282572153,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIvkplDiLVhnRWOo9VGtvM5V19brRXTWks5rgbdlgaJpZM4MAPIf
.

--
Avec mes sincères salutations

TCHOUKEU Simplice Aimé
Architecte des Systèmes d'informations
tel: +237 75 38 35 76
skype: stchoukeu

the FACT is that a module is not recognized after and instalation at the
import
command in the module
2017-02-27 13:29 GMT+01:00 Simplice TCHOUKEU<EMAIL_ADDRESS>
Hi Ahsan,
Thank you for your reply. i want to know if your module is compatible with
IONIC2.  because i complete the doc as you explained in your Github but it
seem that noting has change there since my last test. i make a test again
but notting.
2017-02-26 18:33 GMT+01:00 Ahsan Ayaz<EMAIL_ADDRESS>
Hi @stchoukeu https://github.com/stchoukeu , @petervojtek
https://github.com/petervojtek .
I've just installed and tested in an Angular 2.4.1 project.
Works like a charm.
Do let me know if you're still facing issues.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AhsanAyaz/ng2-emoji/issues/2#issuecomment-282572153,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIvkplDiLVhnRWOo9VGtvM5V19brRXTWks5rgbdlgaJpZM4MAPIf
.

--
Avec mes sincères salutations

TCHOUKEU Simplice Aimé
Architecte des Systèmes d'informations
tel: +237 75 38 35 76
skype: stchoukeu

--
Avec mes sincères salutations

TCHOUKEU Simplice Aimé
Architecte des Systèmes d'informations
tel: +237 75 38 35 76
skype: stchoukeu

I get same error and the solution turned out to be simple. This is error in readme, just replace Ng2EmojiModule to Ng2EmojiModule.forRoot() in imports section of @NgModule.
@NgModule({
  declarations: [
    AppComponent
  ],
  imports: [
    BrowserModule,

    // Include the awesome
    Ng2EmojiModule.forRoot() // here
  ],
  providers: [],
  bootstrap: [AppComponent]
})
export class AppModule { }

this is weird xD  i got output

@Dj-jom2x , I guess you forgot to include the css file.

its fix now i mod it a bit..   i  reuse emoji pipe css img and service and declare them to my app.module.ts ....   angular2 throws xss and html sanitation thingy  in console if i dont add this :(  { SafeHtml , DomSanitizer , bypassSecurityTrustHtml }

import { Pipe, PipeTransform } from '@angular/core';
import { EmojiService } from '../services/emoji/emoji.service';
import { DomSanitizer, SafeHtml } from "@angular/platform-browser";
@Pipe({
  name: 'emojis'
})
export class EmojiPipe implements PipeTransform {
  constructor(private _sanitizer:DomSanitizer) {
  }
   transform(value: string, args: any[]):SafeHtml {
    value = value + '';
    return this._sanitizer.bypassSecurityTrustHtml(value.replace(EmojiService.emojisRegex, function (match, text) {
        return "<i class=\"emoji icon-ng2_em_" + text + "\" title=\":" + text + ":\" style=\"display:inline-block;\"></i>";
    }));
  }

}

a emoji material2 selector would be great if you have time XD

@Dj-jom2x
haha. thanks for putting much efforts into this brother 😄. Will surely look into your suggestions.
I'm updating the library to Angular4 and creating a sample project too to refer to. Hopefully it will resolve the issue :)

Please do you use it wih Ionic2 ?  I get the same error after installation.
Ng2EmojiModule  is not reconized
Le 13 avr. 2017 08:51, "Oleg Krasavin"<EMAIL_ADDRESS>a écrit :

I get same error and the solution turned out to be simple. This is error
in readme, just replace Ng2EmojiModule to Ng2EmojiModule.forRoot() in
imports section of @NgModule.
@NgModule({
declarations: [
AppComponent
],
imports: [
BrowserModule,
// Include the awesome
Ng2EmojiModule.forRoot() // here

],
providers: [],
bootstrap: [AppComponent]
})export class AppModule { }
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/AhsanAyaz/ng2-emoji/issues/2#issuecomment-293818745,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AIvkppT2KKfimVDu1tCneHe0lgv003wnks5rvdQNgaJpZM4MAPIf
.

@Dj-jom2x , @stchoukeu .
I've pushed the latest commit to the repo and updated the release.
Kindly update your npm package ng2-emoji, follow the readme and let me know if the problems are resolved :)

I've included the fix for html sanitization too. Tested on both Angular4 using angular cli and on Ionic2 with webpack.
Kindly re-open the issue if it persists.

please i'm also facing problem with a module . error:   metadata mismatch for modules  ng2-emoji

please i'm allways facing problem with a module.  error:   metadata mismatch for modules  ng2-emoji

@stchoukeu . Can you try uninstalling and re-installing the plugin?
Also can you share the error logs?

I have /ng2-emoji/ng2-emoji.d.ts found version 3 expected 1 too.
You find unswer how 2 fix it ?

@StepPointer can you please share the error logs so I have a better understanding of the issue?


Fixed access permission to read voting principles for normal users.

Required to show decimal places for poll result values for motions and
elections.

@jwinzer Do you want to review this pull request?

Fine with me.


Add note about exclusive_caps to v4l2 documentation

Thank you :+1:
I reworded to move the note to a separate paragraph: 74f11b096a0283d3cc0619318ee74949811ea8f1

Merged into master: bd9292931e868621294ec64ea85179a2803976ef


IGAPP-411-fix-dashboard-navigation-items-on-small-screens

This pull request belongs to an issue on our bugtracker.
You can find it there by looking for an issue with the key which is mentioned in the title of this pull request.
It starts with the keyword IGAPP.

This PR does fix this issue, right? https://issues.tuerantuer.org/browse/IGAPP-367

This PR does fix this issue, right? https://issues.tuerantuer.org/browse/IGAPP-367

This PR does fix this issue, right? https://issues.tuerantuer.org/browse/IGAPP-367

Yeah, seems to be the same issue as 411, if i get it right.

This PR does fix this issue, right? https://issues.tuerantuer.org/browse/IGAPP-367

Yeah, seems to be the same issue as 411, if i get it right.

This PR does fix this issue, right? https://issues.tuerantuer.org/browse/IGAPP-367

Yeah, seems to be the same issue as 411, if i get it right.

Nice, then I'll reject it.

This PR does fix this issue, right? https://issues.tuerantuer.org/browse/IGAPP-367

Yeah, seems to be the same issue as 411, if i get it right.

Nice, then I'll reject it.


Cannot modify header information - headers already sent

Bug description
Warning: Cannot modify header information - headers already sent by (output started at /var/www/html/plugins-enabled/AdminerThemeSwitcher.php:129) in /var/www/html/adminer.php on line 1447

Steps to reproduce

Add AdminerThemeSwitcher.php to plugins-enabled
Open adminer in Browser

Expected behavior
No response
Device info
Docker container adminer:latest
Additional context
No response

Hi, @philharmonie.
I'm sorry I didn't clarify in the Readme that AdminerThemeSwitcher was not meant to run as an Adminer Plugin.
It's a standalone PHP script that downloads an adminer.css file based on your selected theme.
So there are two ways to run it after placing it in the same folder as adminer.php:

php adminer-theme-switcher.php
php -S localhost:8000 and then access http://localhost:8000/adminer-theme-switcher.php on your browser


Add small documentation on using forks. Fixes #10594

Adds documentation as described in https://stackoverflow.com/questions/57404087/how-does-one-package-a-fork-of-wagtail-for-installation/57407742#57407742
Fixes #10594

May I work on this issue?

@anshikavashistha please stop posting requests to work on tasks apparently without putting any work into the actual task.

sorry for the inconvenience
I will make sure this for next time

Yes no worries :)
On Mon, 1 Jan 2024 at 09:34, Anshika Vashistha @.***>
wrote:

May I work on this issue?
—
Reply to this email directly, view it on GitHub
https://github.com/wagtail/wagtail/pull/11107#issuecomment-1873246391,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BDNNV4NYRYSTBWWFVONL6A3YMJ7LLAVCNFSM6AAAAAA6JFZEBSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZTGI2DMMZZGE
.
You are receiving this because you authored the thread.Message ID:
@.***>

@NixAsteri  I have added you to the contributors list as 'Nix Asteri', let me know if you wanted a different name listed.
Thank you for this PR & thanks @zerolab for the tweaks.


[MINOR] Include isolationLevel in toString of FetchRequest

Include isolationLevel in toString of FetchRequest
This is a follow-up to https://issues.apache.org/jira/browse/KAFKA-4818.

Thanks @apurvam for the comments. Changes applied. What are the next steps? Who's to merge it to master?

cc @ijuma @hachikuji
They might be able to merge this.

retest this please


Conditional execution of test steps

What would you like to be added:
I would like to run a test step only if a condition is met. This would allow me to install and test cluster specific resources.
For example, Helm has a way to check if a certain API is available in the cluster by querying the capabilities object and install custom resources of the corresponding kind.
Here is an snippet  that creates a SCC resource only when the security.openshift.io/v1 API is available:
{{ if .Capabilities.APIVersions.Has "security.openshift.io/v1" }}
---
apiVersion: security.openshift.io/v1
kind: SecurityContextConstraints
metadata:
  name: airflow-scc
...
{{ end }}

As you'd guess, the custom resource above is only created if the target cluster of the Helm chart is an OpenShift cluster.
In a similar fashion, I would like to be able to run a kuttl test step that creates  the same SCC only if the tests are running against OpenShift or to be more general, only if the API security.openshift.io/v1 is available.
Why is this needed:
We write many tests and would like them to be reusable without modification across kubernetes implementations.
Currently, to run the same tests on KinD and OpenShift for example, we'd have to maintain two largely redundant code bases for each target cluster. This duplicates the effort to create and maintain these tests.

Duplicated by #400?

Duplicated by #400?

I am not sure if the issues are duplicates. This issue seems to address the need to conditionally disable test steps, while #400 asks for selectively disable tests.

We would also find it useful if kuttl had a way to conditionally skip files or steps. For exactly the same reasons as @razvan mentioned.
My best (but still horrible) idea for a workaround is to cp -a the test tree into a temporary directory and massaging it with a find -name '*not-this-plaftorm* -deletebefore invokingkuttl` there.

I guess you can use helm to template your test manifests.

There's always this possibility to treat a test suite as a template for the actual test suite.
But I like the idea sketched in https://kubernetes.slack.com/archives/CG3HTFCMV/p1687173186335739?thread_ts=1687170514.079399&cid=CG3HTFCMV

My best (but still horrible) idea for a workaround is to cp -a the test tree into a temporary directory and massaging it with a find -name '*not-this-plaftorm* -deletebefore invokingkuttl` there.

It's basically what we are doing now in an automated way around kuttl. Our tests are made of two things:

A set of of Jinja templates
A matrix of variables that we build a cartesian product from.

Then we use a simple Python tool to generate the test suite.

We do something similar than @razvan but with Gomplate


Add couple rating to heat entries when no results

Prior to scoresheets being entered, display the couple's rating (not ranking) on the heat

Fixed with pull request #107


Native TextInput does not work properly when wrapped with Factory

Describe the bug
When using Factory creator for native text input. When the input gets focused it automatically gets unfocused
To Reproduce
1 - Use the text input factory example. Click on the text input and the input get unfocused automatically.
Expected behaviour
Text input shouldn't be unfocused.
CodeSandBox/Snack link
I am using the example provided in the docs. https://docs.nativebase.io/next/nativebase-factory#using-ref
Screenshots
If applicable, add screenshots to help explain your problem.
https://user-images.githubusercontent.com/9849229/145554317-910b0ce2-3d58-424d-8e33-0b380ceae586.mp4
Platform

NativeBase version [3.2.2]
Device: [Simulator]
OS: [15.0]
Browser [e.g. stock browser, safari]
Browser version [e.g. 22]
React Native version [0.66.3]
Expo sdk version [e.g. 41]
node [14.15.1]
yarn [1.22.11]

Additional information

I was able to make it work, mimicking part of the implementation of Factory
`import React, {memo, forwardRef} from 'react';
import {TextInput} from 'react-native';
import {
usePropsWithComponentTheme,
useStyledSystemPropsResolver,
} from 'native-base';
import inputTheme from '../theme/components/input';
const MaskedInput = (props, ref) => {
const calculatedProps = usePropsWithComponentTheme(inputTheme, props);
const [style, restProps] = useStyledSystemPropsResolver(calculatedProps);
return <TextInput ref={ref} style={style} {...restProps} />;
};
export default memo(forwardRef(MaskedInput));
`
I think that there is something not working properly on the implementation of Factory that is making components unmount and mount again. I did not have time to investigate further. But it also happened something similar to other components

We have similar issue, on Android the Native Base Input wrapped in HOC with Factory
export default Factory(ContainerWithButton);
results in the keyboard being automatically dismissed after typing a character.
Although this seems to be more happening on Android than on iOS
Simply removing the Factory wrapper from the HOC component resolves issue so agree it has to be some problem with Factory itself.
<ContainerWithButton
            onPress={() => navigation.navigate('onboarding')}
            buttonText={t('next', {ns: 'common'})}
            isButtonDisabled={!selectedLanguage}>
     <Input/>
</ContainerWithButton>

This example works only if we remove the Factory wrapper from the HOC component given above.

I also saw problems after using Factory in wrapper and then Input from Native Base

@acollazomayer I tried to reproduce the issue but was not able to.
It'll be really helpful if you'll share a snack, as the issues are quite visible in the video.

I have same issue, my component example
import { TextInputMask } from 'react-native-masked-text';
import { Factory } from 'native-base';

export const MaskedInput = Factory(TextInputMask);

And component usage code example
<MaskedInput
    w="80%"
    type='custom'
    keyboardType="numeric"
    options={{ mask: '+7 (999) 999-99-99' }}
    value={phone}
    onChangeText={onChangeText}
/>

When i enter number, i have focus out how in video. Without factory method, all right

Hi @MD-REHMAN ,
I have a snack of the issue.
When using onChangeText and / or value, this is when the input automatically unfocuses after each keypress.
https://snack.expo.dev/@apptractive/factory-rn-textinput-broken

Not stale


Application is vulnerable to XSS (Reflective)

The input doesn't seem to be sanitized properly leaving room for malicious XSS payloads.

How to patch
Input sanitization is the most straightforward solution to patch an XSS vulnerability.

@CopernicusPY I'm not sure how this could be exploited, can you successfully demonstrate that this is dangerous?

Not really. But this could possibly interfere with the functionality of the users website and cause problems.
Therefore in order to address possible security issues, why not just sanitize the input properly.

this could possibly interfere with the functionality of the users website and cause problems

Again, I would like to see this being demonstrated. The website doesn't store any user data, the only possible issue that can come from leaving the input "unsanitized" is, like how you showed previously, breaking your own instance.
Like any website, you can break your own instance by going to the devtools and do all sorts of things.
I don't see the point of adding more code for unnecessary security, or following best practices blindly.
Also, feel free to create a PR 😉


Unable to start via docker-compose

Hi, I tried with docker-compose, but it did not work!

Try the latest version, it is working for me.


Windows: Command line shortcut doesn't work in Git Bash

Open Git Bash (Msysgit) on Windows
Type brackets

Result:
sh.exe": brackets: command not found

even though it works just fine in cmd (both admin and non-admin)
Expected:
Brackets launches
Problem:
As the Git Bash emulates a Linux/*NIX-like bash, it doesn't interpret .bat files, but Brackets only has one of those. That's probably also an issue with cygwin and similar environments.
Possible solution:
Also include a brackets file (extensionless) so a bash-like environment can run those.
That's also what node does for npm and ruby does for gem.

We should consider fixing this for 1.3, especially since it seems like the fix might be easy (just add another file in the command subfolder that gets installed).
Note: I think it would also work if the file has a .sh extension (without the user needing to type ".sh").  This might be a little more clear since the type of the file is more obvious from its name.

@MarcelGerber @peterflynn Sure! Surprisingly issuing brackets in GitBash works fine on my system. I will look into this. And thanks for the pointers @MarcelGerber.

@nethip Perhaps you still have the old path (pointing to the installation directory directly rather than the command subfolder) included in your PATH?

@MarcelGerber I think I still have the Path to 'command' folder on my system. I can check this tomorrow when I get to the office.

Yeah, that's the folder the current installer adds to the PATH, so it's fine.
What I meant was the part added by the old installer.

You are right @MarcelGerber. This does not work in GitBash. I am looking into this.

This is fixed now with PR https://github.com/adobe/brackets/issues/10938

Closing as the PR is now merged.


Fix/coverity issues 0.21.3

Removed dead code regions identified by Coverity.

No. We want to cherry-pick not merge main into gold/2021


Settings-Page

As an editor
I want to have a setting page to set title, telegram bot-name and onboarding text for my project
in order to

in oder to...?

In order to customize the content in my system without the help of a developer


Bluetooth: Mesh: Add initial implementation

Add an initial implementation for the Bluetooth Mesh Profile
Specification. The main code resides in subsys/bluetooth/host/mesh and
the public API can be found in include/bluetooth/mesh.h. There are a
couple of samples provided as well under samples/bluetooth and
tests/bluetooth.
The implementation covers all layers of the Bluetooth Mesh stack and
most optional features as well. The following is a list of some of
these features and the c-files where the implementation can be found:

GATT & Advertising bearers (proxy.c & adv.c)
Network Layer (net.c)
Lower and Upper Transport Layers (transport.c)
Access Layer (access.c)
Foundation Models, Server role (health.c & cfg.c)
Both PB-ADV and PB-GATT based provisioning (prov.c)
Low Power Node support (lpn.c)
Relay support (net.c)
GATT Proxy (proxy.c)

Notable features that are not part of the implementation:

Friend support (initial bits are in place in friend.c)
Provisioner support (low-value for typical Zephyr devices)
GATT Client (low-value for typical Zephyr devices)

Jira: ZEP-2360
Change-Id: Ic773113dbfd84878ff8cee7fe2bb948f0ace19ed
Signed-off-by: Johan Hedberg<EMAIL_ADDRESS>

Rebased to see if CI would finally complete. (i.e. the "dismissed stale reviews" isn't really valid as nothing changed in the patch)

i am looking into the CI issues

I recreated this pull request since CI seems to have some bugs properly building it. The copy PR can be found here: https://github.com/zephyrproject-rtos/zephyr/pull/850

The duplicate PR (which CI was able to successfully build) has been merged, so closing this one.


Use File.exist instead of File.exists

This pull request applies style fixes from code review by CodeFactor.

For more information, click here.

Got deprecated, see https://ruby-doc.org/core-2.5.0/File.html#method-c-exists-3F


#1724 Reverting back HttpClient full buffering

Follows up #1724

#1724

After testing the new http message invoker pool on a test environment, we noticed that some resources were not being released. After further checks, we realised that some streams were not being closed correctly.
This doesn't appear in the application with the standard http client, but I made a mistake by adding the HttpCompletionOption.ResponseHeadersRead property when calling the SendAsync method, which could potentially cause the same problems as with the new message invoker pool.
A revert should therefore be performed for the time being, and this problem will be addressed in the new message invoker pool.
🙈 🙉 🙊
Proposed Changes

Reverting back HttpClientWrapper keeping the full buffering for now. Usage of the new option is out of scope and hazardous!!!

@raman-m please, asap ;-)

HttpCompletionOption.ResponseHeadersRead property

I cannot get how does the property influence to disposing internal resources?
If Yes then .NET framework has bugs... I cannot believe in that.
More realistic scenario is...Ocelot with DI has hidden services which consumes and produces and behaves badly.

I'm not going to approve till Raynald's report if he provides some tests in prod.
@RaynaldM FYI  We need your report and approve here.

HttpCompletionOption.ResponseHeadersRead property

I cannot get how does the property influence to disposing internal resources? If Yes then .NET framework has bugs... I cannot believe in that. More realistic scenario is...Ocelot with DI has hidden services which consumes and produces and behaves badly.

As soon as you decide to handle the stream yourself, then you are responsible of releasing it. It's logical no?

I'm not going to approve till Raynald's report if he provides some tests in prod.
@RaynaldM FYI We need your report and approve here.

@raman-m Raynald has tested a version without these specific changes

we noticed that some resources were not being released.

were not being disposed!

freed willy

I need Ray's input here... There is no problem to revert. The release is still active. This fix is a part of the Nov'23 milestone.

I need Ray's input here... There is no problem to revert. The release is still active. This fix is a part of the Nov'23 milestone.

I won't be able to validate these last changes in prod until January 3 or 4.

@RaynaldM  🆗
In this case, it is better to wait new NuGet packs of Nov'23 release and deploy them and test...
So, Dec'23 version will not be ready on January 3...


MAPREDUCE-7468: Change add-opens flag's default value from true to false

Description of PR
How was this patch tested?
For code changes:

[x] Does the title or this PR starts with the corresponding JIRA issue id (e.g. 'HADOOP-17799. Your PR title ...')?
[ ] Object storage: have the integration tests been executed and the endpoint declared according to the connector-specific documentation?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE, LICENSE-binary, NOTICE-binary files?

Thanks @tomicooler for the review, merging to trunk.


cloud-nuke: 0.30.0 -> 0.31.1

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/cloud-nuke/versions.
meta.description for cloud-nuke is: A tool for cleaning up your cloud accounts by nuking (deleting) all resources within it
meta.homepage for cloud-nuke is: https://github.com/gruntwork-io/cloud-nuke
meta.changelog for cloud-nuke is: https://github.com/gruntwork-io/cloud-nuke/releases/tag/v0.31.0
Updates performed

Golang update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.31.0 with grep in /nix/store/rvnpx10h7fb1n7llhmzxdf622a5n912v-cloud-nuke-0.31.0
found 0.31.0 in filename of file in /nix/store/rvnpx10h7fb1n7llhmzxdf622a5n912v-cloud-nuke-0.31.0

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cloud-nuke

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/rvnpx10h7fb1n7llhmzxdf622a5n912v-cloud-nuke-0.31.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cloud-nuke https://github.com/r-ryantm/nixpkgs/archive/211f94f7393448a40d1f2d73dbfae46bed83c92b.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/rvnpx10h7fb1n7llhmzxdf622a5n912v-cloud-nuke-0.31.0
ls -la /nix/store/rvnpx10h7fb1n7llhmzxdf622a5n912v-cloud-nuke-0.31.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cloud-nuke

Maintainer pings
cc @marsam for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/cloud-nuke/versions.
meta.description for cloud-nuke is: A tool for cleaning up your cloud accounts by nuking (deleting) all resources within it
meta.homepage for cloud-nuke is: https://github.com/gruntwork-io/cloud-nuke
meta.changelog for cloud-nuke is: https://github.com/gruntwork-io/cloud-nuke/releases/tag/v0.31.1
Updates performed

Golang update

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.31.1 with grep in /nix/store/46qfandf4sqwfqkjpzw7bm88w0ynf4il-cloud-nuke-0.31.1
found 0.31.1 in filename of file in /nix/store/46qfandf4sqwfqkjpzw7bm88w0ynf4il-cloud-nuke-0.31.1

Rebuild report (if merged into master) (click to expand)

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
cloud-nuke

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/46qfandf4sqwfqkjpzw7bm88w0ynf4il-cloud-nuke-0.31.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A cloud-nuke https://github.com/r-ryantm/nixpkgs/archive/6a7c8a7c51719094183478636e19bcf6d9fe408d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/46qfandf4sqwfqkjpzw7bm88w0ynf4il-cloud-nuke-0.31.1
ls -la /nix/store/46qfandf4sqwfqkjpzw7bm88w0ynf4il-cloud-nuke-0.31.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package built:

cloud-nuke

Maintainer pings
cc @marsam for testing.


DuckDBException when inserting some Unicode characters using appender

When using the DuckDBAppender, I am receiving exceptions when inserting some Unicode characters. I believe this is a DuckDB.NET issue, as similar code works fine with Python.
Example code (uses the new .NET6 console application template):
using DuckDB.NET.Data;

var USE_APPENDER = true;

var DB_FILE = "csharptest.duckdb";
var TABLE_NAME = "TestTable";

File.Delete(DB_FILE);
File.Delete($"{DB_FILE}.wal");

using var db = new DuckDBConnection($"DataSource={DB_FILE}");
db.Open();

var dbCommand = db.CreateCommand();

dbCommand.CommandText = $"CREATE TABLE {TABLE_NAME}(index INTEGER, words VARCHAR);";
dbCommand.ExecuteNonQuery();

var words = new List<string> { "hello", "안녕하세요", "Ø3mm CHAIN" };

if (USE_APPENDER) {
    using var appender = db.CreateAppender(TABLE_NAME);

    for (int i = 0; i < words.Count; i++) {
        Console.WriteLine($"Creating row {i}...");

        var row = appender.CreateRow();
        row.AppendValue(i).AppendValue(words[i]);

        row.EndRow();
    }

    appender.Close();
} else {
    for (int i = 0; i < words.Count; i++) {
        Console.WriteLine($"Creating row {i}...");

        dbCommand.CommandText = $"INSERT INTO {TABLE_NAME} VALUES ({i}, '{words[i]}');";
        dbCommand.ExecuteNonQuery();
    }
}

db.Close();

If USE_APPENDER = true, the program will crash when inserting the 3rd word with the following log:
Creating row 0...
Creating row 1...
Creating row 2...
Unhandled exception. DuckDB.NET.Data.DuckDBException (0x80004005): INTERNAL Error: Invalid unicode (byte sequence mismatch) detected in segment statistics update
   at DuckDB.NET.Data.DuckDBAppender.ThrowLastError(DuckDBAppender appender)
   at DuckDB.NET.Data.DuckDBAppender.Close()
   at Program.<Main>$(String[] args) in C:\Users\jmyers\projects\duckdb-test\Program.cs:line 33

If USE_APPENDER = false, the program will complete successfully. Opening up the database with Tad reveals that all data was inserted successfully, with no encoding issues.
The only way I have found to insert strings containing Ø has been to convert the string to ASCII like Encoding.ASCII.GetString(Encoding.UTF8.GetBytes(s)), however this results in the Ø being replaced with ?? - not exactly desirable.
Other information:

Windows 11 22H2 x64
dotnet --version is 7.0.102
Using package DuckDB.NET.Data.Full, version 0.6.1

This should now be fixed in develop branch.

@JackMyers001 You should be able to get the fix from GitHub Packages

I can confirm this patch has fixed my issue! Thanks so much for fixing it so quickly :)
The only oddity I have noticed (and I need to do more testing to confirm if it's related to this patch) is it appears that insert time has taken a hit; inserting some 1.3 million rows used to take ~280 seconds, but now takes more like 370. Again, need to do more testing to confirm.

I made some changes to how strings are marshaled to native code. Can you check if the latest preview package is faster?

Apologies for the slow response.
I had run some testing a week or so ago with DuckDB.NET.Data.Full version 0.6.1-alpha.9 and I could have sworn that I remember seeing no performance change (i.e. still ~370 seconds). However, I have just run two runs over 1.3 million rows (the same data as before) and they both completed in ~255 seconds. I also tried out 0.6.1-alpha.10, and that appears to have within margin-of-error performance gains.
The difference in my memory vs testing now could be a slight coincidence as I have changed a couple of settings for my test VM, however I didn't change the amount of RAM / CPU cores... Maybe I'm just losing it 🤣
Anyhow, it does appear that the second patch has fixed that performance degradation. I really appreciate your help!

Nice to hear that it runs fast again. As 0.7.0 was released yesterday, I'll push a new build in a day or two for these fixes and include the 0.7.0 build too.
Consider Sponsoring the Project if you want to support it.


Add remaining time iformation for the whole batch

Add the calculation of the remaining time before the whole job batch is expected to finish.
Consider logging this information in case of a connection loss or program crash to be able to retrieve the last known info and aid in recalculation.

Technically, this is implemented, but perhaps it is poorly implemented. See Issue #20


Error when I executed command "drogon_ctl create" on Windows

I have compiled the source code on windows
this is my drogon_ctl version
     _
  __| |_ __ ___   __ _  ___  _ __
 / _` | '__/ _ \ / _` |/ _ \| '_ \
| (_| | | | (_) | (_| | (_) | | | |
 \__,_|_|  \___/ \__, |\___/|_| |_|
                 |___/

A utility for drogon
Version: 1.8.0
Git commit: 6a397efefe9bf62de807a71d00ed48b8609d4a81
Compilation:
  Compiler: C:/Program Files/Microsoft Visual Studio/2022/Community/VC/Tools/MSVC/14.33.31629/bin/Hostx64/x64/cl.exe
  Compiler ID: MSVC
  Compilation flags: /MDd /Zi /Ob0 /Od /RTC1 -std=c++20 -IC:/Users/Yoga/.conan/data/jsoncpp/1.9.4/_/_/package/d057732059ea44a47760900cb5e4855d2bea8714/include -ID://include
Libraries:
  postgresql: no  (pipeline mode: no)
  mariadb: no
  sqlite3: yes
  openssl: yes
  brotli: yes
  boost: no
  hiredis: yes
  c-ares: no

I executed command drogon_ctl create and this happen

I don't have any idea to fix this error

What did you create? please paste the complete command that leading errors.

already solved
but thanks anyway


Possible issue when misspelled word has 2 non consecutive deletes

I successfully implemented symspell using a mysql memory table and with an odbc driver and some post processing to filter candidates the performance is absolutely amazing! But I ran across a case today that I thought might be an issue.
I was looking for corrections to some abbreviated words and I had this abbreviation: IMPROVMNT and expected the suggested correction to be :IMPROVEMENT which is only 2 deletes away but instead got no suggestions.
In my prepared symspell database I have nearby candidates in my index such as IMPROVEMNT and IMPROVMENT but since my precalculation iteratively deletes one letter at a time to populate the index I would never have words where the 2 deletes are non-consecutive. Am I missing something?

You want to match IMPROVMNT to IMPROVEMENT
The two terms have a Damerau–Levenshtein edit distance distance of 2.
This means, you have to recursively delete 2 two characters both from all dictionary terms (only once in the pre-calculation phase) and from the (misspelled) input term.
Lets have a look at your example:
First delete iteration of Dictionary term: IMPROVEMENT
MPROVEMENT
IPROVEMENT
IMROVEMENT
IMPOVEMENT
IMPRVEMENT
IMPROEMENT
IMPROVMENT   *
IMPROVEENT
IMPROVEMNT
IMPROVEMET
IMPROVEMEN
Second delete iteration of first iteration delete IMPROVMENT   (do this for each of the first iteration deletes)
MPROVMENT
IPROVMENT
IMROVMENT
IMPOVMENT
IMPRVMENT
IMPROMENT
IMPROVENT
IMPROVMNT   -> this is the match for your input term
IMPROVMET
IMPROVMEN
The above method does generates non-consecutive deletes.
And don't forget that you do need also to generate deletes from the input term, not only for the dictionary term.
While for your specific example this is not necessary, in many other cases it is!

You want to match IMPROVMNT to IMPROVEMENT
The two terms have a Damerau–Levenshtein edit distance distance of 2.
This means, you have to recursively delete 2 two characters both from all dictionary terms (only once in the pre-calculation phase) and from the (misspelled) input term.
Lets have a look at your example:
First delete iteration of Dictionary term: IMPROVEMENT
MPROVEMENT
IPROVEMENT
IMROVEMENT
IMPOVEMENT
IMPRVEMENT
IMPROEMENT
IMPROVMENT   *
IMPROVEENT
IMPROVEMNT
IMPROVEMET
IMPROVEMEN
Second delete iteration of first iteration delete IMPROVMENT   (do this for each of the first iteration deletes)
MPROVMENT
IPROVMENT
IMROVMENT
IMPOVMENT
IMPRVMENT
IMPROMENT
IMPROVENT
IMPROVMNT   -> this is the match for your input term
IMPROVMET
IMPROVMEN
The above method does generates non-consecutive deletes.
And don't forget that you do need also to generate deletes from the input term, not only for the dictionary term.
While for your specific example this is not necessary, in many other cases it is!

Thanks. I need to review if my pre-calculated dictionary does the second round of deletes. Otherwise this is working great..

Thanks. I need to review if my pre-calculated dictionary does the second round of deletes. Otherwise this is working great..


how could I make a larger kernel for motion blur?

I read the code of linedictionary, but failed  to figure out what it means? is there anything to support it?

LineDictionary defines the coordinates wrt the kernel matrix of the two points identifying each possible line, these are then used in LinearMotionBlur.LineKernel to "draw" the line on the kernel matrix, setting the relevant values to 1. The keys of the different dict values are the angles of the lines each set of point coordinates refers to
If you want to make a larger linear blur kernel, just follow this logic and add what you need to LineDictionary

@HoiweCheng Hope this help
def createNxNLines(self,n):
    lines={}
    assert: (n-1)%2==0,"n must be a odd number!!!"
    Num=2*n-2
    angle_unit=180.0/Num
    cnt=0
    for i in range((n-1)/2,n):
        j=0
        lines[cnt*angle_unit]=[i, j, n-1-i, n-1-j]
        cnt+=1
    for j in range(1,(n+1)/2):
        i=n-1
        lines[cnt*angle_unit]=[i,j,n-1-i,n-1-j]
        cnt+=1
    for j in range((n+1)/2,n):
        i=n-1
        lines[cnt*angle_unit]=[n-1-i, n-1-j, i, j]
        cnt+=1
    for i in range(1, (n-1)/2):
        j=0
        lines[cnt*angle_unit]=[i,j,n-1-i,n-1-j]
        cnt+=1
    self.lines[n]=lines
    return


Mediacore replacement

Finished review. Back to @sallakarppinen for follow-up

Thanks for figuring out the CSS issue, @nicolaasmatthijs. I've made some fixes and deployed them to qa2 server.

Finished another round of review. Back to @sallakarppinen

Added some more review. Back to @sallakarppinen

Finished another round of review. Back to @sallakarppinen

Left some final comments. Back to @sallakarppinen

Made some changes (including tidying up repetition as brought up by @nicolaasmatthijs)... Had some weird behaviour on qa2, it seems that using position: unset caused the video to not play properly so I've swapped back to width: 98%.

Looks like the "play" icon for MP3 is a bit out of wack:

There are also issues when you play it with the player UI, and the file only seems to play the first half a second or so of the file then stops:

The following functional issues were identified during the bugbash:

The Processing this file screen is not correctly updated once the video has finished transcoding and requires a refresh of the screen before the video can be seen. We expect a push notification to inform the UI that it has finished processing and update the content preview. This is in place for other types of content already.

The video player needs to align on both ends with the comments widget at all widths. That's why I was pushing back on the 98% width as I can't see how that can ensure the alignment. In the screenshot below, you'll see red lines where the video is expected to align:

@nicolaasmatthijs: For issue 2, could you specify the browser and OS that were used, since I'm  not able to reproduce (on Firefox/Linux Mint)?

@sallakarppinen : It appears to be any browser (Firefox / Chrome / Safari). Maybe try changing the width of your browser window as there's a single "perfect" width where they do roughly align (because of the percentage width).

@nicolaasmatthijs You're right, I can see it if I resize to something other than the 'standard' sizes... I'll try to find a way around it.

Have not been able to fix the resizing, but the first issue (file not refreshing when processed) is fixed by the latest changes to Hilary branch for this task which is deployed on qa2.


Copyright check: add logic to handle empty files and files less than 3 lines.

@dilijev Did you consider using the zip function instead? I think we could do:
linecount = 0
with open(file_name, 'r') as sourcefile:
    for (copyright_line, line) in zip(copyright_lines, sourcefile):
        linecount += 1
        # ...
if linecount == 0:
    # PASS
elif linecount != len(copyright_lines):
    # FAIL

@goyakin possible, but I don't want to step on PR #367 so I'll wait until that gets merged in to make the change you're suggesting and use regexes instead of copyright lines, in line with that change.

Alright, I made a few modifications based on @goyakin's suggestion. I also wanted to make the output easier to read.
The output will now look like this:
$ sh jenkins.check_copyright.sh
Check Copyright > Begin Checking...
Check Copyright > Checking jenkins.check_copyright.py
--------------
--- ERRORS ---
jenkins.check_copyright.py ... does not contain a correct Microsoft copyright notice.

     #-------------------------------------------------------------------------------------------------------
     # Copyright (C) Microsoft. All rights reserved.
     # icensed under the MIT license. See LICENSE.txt file in the project root for full license information.
     #-------------------------------------------------------------------------------------------------------

--------------

I tested that this script works on the four scenarios:

blank file (success)
too short (failure)
long enough but incorrect (failure)
long enough and successful (success)

@goyakin How does it look now?

LGTM :smile:


Update scodec-bits to 1.1.15

Updates org.scodec:scodec-bits from 1.1.14 to 1.1.15.
GitHub Release Notes - Changelog - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.scodec", artifactId = "scodec-bits" } ]

labels: library-update, semver-patch

Codecov Report

Merging #281 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #281   +/-   ##
=======================================
  Coverage   62.35%   62.35%
=======================================
  Files          38       38
  Lines        1283     1283
  Branches        7        8    +1
=======================================
  Hits          800      800
  Misses        483      483

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 b9259e0...f023fc0. Read the comment docs.


remove unnecessary return nil check

I was looking for nil checks in save inventory and stumbled across this.
no need for if x.nil? return nil else return x

:+1:


How to generate labels from PASCAL VOC dataset

Hello Raj,
You code is awesome and really helpful. But I am unable to generate labels. I have downloaded Pascal VOC dataset from this link: http://host.robots.ox.ac.uk/pascal/VOC/voc2012/. I am not able to understand what should be in the "path to labels" in the code create_labels.py. Also what is the value 220 in this file. How the contour pixel value is decided.?
Hope to hear from you soon.
Thanks in advance.

Hi , can you visualize one of the labels with %matplotlib notebook and hover your cursor to the boundaries ?
What value does it say ?

Ok I will Surely try . Thanks for the response