Variable List Correction

[Enter feedback here]
The %LOCALAPPDATA%  variable on this page is redirected to C:\Windows\System32\config\systemprofile\AppData\Local but when I use this variable on my system(s) it takes me to the C:\Users\USERNAME\AppData\Local folder instead.
Please make the correction.,
Thanks
TK
-- | --

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

ID: 87906607-83fd-8dc8-facb-89719390f824
Version Independent ID: 92f3359d-84ac-90c3-33d5-513feffc4173
Content: Configure and validate exclusions based on extension, name, or location
Content Source: microsoft-365/security/defender-endpoint/configure-extension-file-exclusions-microsoft-defender-antivirus.md
Service: microsoft-365-security
Sub-service: mde
GitHub Login: @denisebmsft
Microsoft Alias: deniseb

@mtariqkhanIT Thank you for your feedback. We have made changes to the document via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/9740
Once the author approves, the changes will be live.
Thanks
Sri

@msbemba Thank you very much for the contribution and updating the documentation with PR. @mtariqkhanIT Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Footer UI enhancement

Need some enhancement with the footer UI.

@ParamThakkar123 I'm interested in this enhancement. Could you please provide more details about it. Thank you.

Hello @ChetanDayapule , Thank you for your interest in contributing to our project.

@ParamThakkar123 I'm interested in this enhancement. Could you please provide more details about it. Thank you.

We have made the footer but it's UI is not that good. It's too rudimentary, we would like to have a better footer design for that

@ChetanDayapule also it would be really great and motivating for our team if you could star this repo so that this project reaches more people. Thanks again for your interest. I am assigning issue to you

@ParamThakkar123 I'm interested in this enhancement. Could you please provide more details about it. Thank you.

We have made the footer but it's UI is not that good. It's too rudimentary, we would like to have a better footer design for that

Okay, I will take a look and come up with a proposal soon. Thank you.

@ChetanDayapule, any updates?

In my view keeping the footer simple is ideal for any web application; my proposal is as follows (yet to adapt the current theme) and I'm open for suggestion -

Two queries -

Hope, it is okay to add "Terms of Use" page? If not, feel free to suggest any 1 or 2 more links.
If you have a channel/group in social media that can be linked here, please do share it. If no social links, will align the slogan "Experience..." to left and "Feedback", "About Us" buttons to right in 1-liner.

In my view keeping the footer simple is ideal for any web application; my proposal is as follows (yet to adapt the current theme) and I'm open for suggestion -

Two queries -

Hope, it is okay to add "Terms of Use" page? If not, feel free to suggest any 1 or 2 more links/pages.
If you have a channel/group in social media that can be linked here, please do share it. If no social links, will align the slogan "Experience..." to left and "Feedback", "About Us" buttons to right in 1-liner.

Looks great @ChetanDayapule , We neither have any terms of use page nor any links of social media. The footer looks good but It would be great if you could align it with the modern web site design UI with some glassy nature of opacity

In the links, you can add a link to the Github repo of the website.

i can improve the footer ui more as I have created many frontend projects


Browser prefix on clip-path

Hi, I'm having a hard time applying a clip-path animation on Safari, which currently needs a browser prefix in order to work. I've checked the issues and one of them suggests using webkitClipPath but this is not working for me. Please advise!
Pen: https://codepen.io/c0mrx/pen/qywQeg

Nevermind, I figured it out! I had to pass an initial value for it to work.


⚠ Bofrost.fr has degraded performance

In 3ddafe1, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 2071 ms

Resolved: Bofrost.fr performance has improved in df8e093 after 10 minutes.


Model inheritance

Given the following models, I would expect Planet to inherit name, moon, and sun but it does not. Moving those declarations to Planet works but I lose the value of shared code. Is this a bug or does ember-orbit recommend solving this in a different manner?
import { Model, attr, hasOne, hasMany, key } from 'ember-orbit';

export default class Planet extends Model {
  @attr('string') name;
  @hasMany('moon', { inverse: 'planet' }) moons;
  @hasOne('star') sun;
}

import Planet from './planet.js'

export default class Earth extends Planet {
}

orbit version<EMAIL_ADDRESS>

I have an idea of what's going on. We use reflect-metadata on the model prototype to store schema information. I am pretty sure it creates issues with inheritance.

Thanks @tchak. Let me know if there is a way I can help.

@tchak Thanks for the updates to ember-orbit today. I pulled down master to check this out and somethings are better... some are worse. I'm trying to create a test case for what I'm seeing but struggling to do so. Perhaps you could provide some guidance.
I have a set of model classes that look roughly as follows...
class NamedModel extends Model {
  @attr('string') name;
}

class Moon extends NamedModel {
  @hasOne('planet', { inverse: 'moons'}) planet;
}

class Ocean extends NamedModel {
  @hasOne('planet', { inverse: 'oceans'}) planet;
}

class Continent extends NamedModel {
  @hasOne('planet', { inverse: 'continents' } planet;
}

class Planet extends Model {
  @hasMany('moons', { inverse: 'planet' }) moons;
  @hasMany('oceans', { inverse: 'planet' }) oceans;
  @hasMany('continents', { inverse: 'planet' }) continents;
}

The issue is that orbit is throwing an IncorrectRelatedRecordType error with a message that is roughly Schema error: Relationship definition oceans for model definition planet does not accept record of type moons.
I can cause the error in two ways in my app:

When I load a planet via a jsonapi source with relationship identifiers for one of the hasMany relationships.
When I create a fork, then add an ocean (or moon or continent) record with the planet relationship.

I have a branch of ember-orbit where I've tried adjusting the test models to roughly match my setup but I can't make it fail. Any ideas?
One other observation: If I comment out the hasOne planet relationships on the models I'm not using at the moment, then things work. It appears that the inheritance is messing with how relationships get defined.

I know what is happening. Will fix soon.

https://github.com/orbitjs/ember-orbit/pull/310

#310 has been merged and looks solid. Thanks @tchak!
I'm pretty sure this has been resolved now, and we'll get another beta release out ASAP. Of course, please let us know if you continue to have problems @chbonser


Add Iterator#slice_before

Part of #7142
This also improves the documentation of slice_after, now the two docs are similar and make it more clear why they are called that way.
I thought about reusing the implementation of slice_before but it might not be worth it. The algorithms need different variables. Plus at least in Ruby I think they don't seem to reuse the algorithm either. (in Ruby it's also much, much longer, in Crystal the core algorithm is just 30 lines)

ary is short for Array. Ruby uses it in its examples. For example: https://ruby-doc.org/core-2.2.0/Range.html#method-i-bsearch
It also happens to be my name :-P

Might be good to hold off merging this until we reach consensus on what happens with non-empty reuse:  though.

But we have reuse in many other methods and they behave the same.
We can probably tackle the reuse "issue" in a separate discussion.

Plus we already merged slice_after


High Contrast and Abnormal Colors in Generated Images

Hello，I am currently attempting to fine-tune a network for image restoration using ControlNet. However, during the training process, I have noticed that the generated images exhibit extremely high contrast and abnormal colors. I am wondering how I can address this issue. Is it related to the CFG scale?
Thank you very much for your assistance.

Hello, may I ask if the content of your generated image is consistent with the original image? The image I generated is different from the original image.

Hello, may I ask if the content of your generated image is consistent with the original image? The image I generated is different from the original image.
The content of the generated image is basically consistent, but the color and contrast have changed

Hello, may I ask if the content of your generated image is consistent with the original image? The image I generated is different from the original image.

The content of the generated image is basically consistent, but the color and contrast have changed

Hello, may I ask if the content of your generated image is consistent with the original image? The image I generated is different from the original image.

The content of the generated image is basically consistent, but the color and contrast have changed

Have you solved the problem? My color has changed a bit, too.

Hello, may I ask if the content of your generated image is consistent with the original image? The image I generated is different from the original image.

The content of the generated image is basically consistent, but the color and contrast have changed

Have you solved the problem? My color has changed a bit, too.

I haven't solved it yet. I suspect this problem is related to CFG, but I am not very sure and I don't know why

Hello, may I ask if the content of your generated image is consistent with the original image? The image I generated is different from the original image.

The content of the generated image is basically consistent, but the color and contrast have changed

Have you solved the problem? My color has changed a bit, too.

I haven't solved it yet. I suspect this problem is related to CFG, but I am not very sure and I don't know why

I also have a problem with abnormal colors. After entering red as the prompt word, the picture turns out to be blue. Have you solved it?

Hello, may I ask if the content of your generated image is consistent with the original image? The image I generated is different from the original image.

The content of the generated image is basically consistent, but the color and contrast have changed

Have you solved the problem? My color has changed a bit, too.

I haven't solved it yet. I suspect this problem is related to CFG, but I am not very sure and I don't know why

I also have a problem with abnormal colors. After entering red as the prompt word, the picture turns out to be blue. Have you solved it?

same here, mine is green turn out to blue

Hello, may I ask if the content of your generated image is consistent with the original image? The image I generated is different from the original image.

The content of the generated image is basically consistent, but the color and contrast have changed

Have you solved the problem? My color has changed a bit, too.

I haven't solved it yet. I suspect this problem is related to CFG, but I am not very sure and I don't know why

Have you solved this problem？


Remove version: StrawberryMusicPlayer.Strawberry version 1.0.5

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://github.com/strawberrymusicplayer/strawberry/releases/download/1.0.5/StrawberrySetup-1.0.5-msvc-x64.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 911)


Add the Text output for GeminiMultimodalLiveLLMService

Description
Is this reporting a bug or feature request?
feature
Will you think about change the response_modalities of the output of Gemini, so we can use the output to other tts service.
It will output by text only if you change it to TEXT.

@fatwang2 can you clarify? Do you want audio input and text output?

yes, so that I can transfer to other TTS to support multi languages, I have
finished it on my own git
Mark Backman @.***>于2024年12月22日 周日22:49写道：

@fatwang2 https://github.com/fatwang2 can you clarify? Do you want
audio input and text output?
—
Reply to this email directly, view it on GitHub
https://github.com/pipecat-ai/pipecat/issues/892#issuecomment-2558480886,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A77NZSX4M6O6UI72XHABHXD2G3GODAVCNFSM6AAAAABT5N2KOSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDKNJYGQ4DAOBYGY
.
You are receiving this because you were mentioned.Message ID:
@.***>

I also wish for this feature. I believe it’s very powerful and represents the future.


error TS2688: Cannot find type definition file for '.vue-global-types/vue_3.5_false.d.ts'

Vue - Official extension or vue-tsc version
2.1.6
VSCode version
1.93.1
Vue version
3.5.6
TypeScript version
5.3.3
System Info
System:
    OS: macOS 15.0
    CPU: (10) arm64 Apple M1 Max
    Memory: 6.98 GB / 64.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 20.15.1 - /usr/local/bin/node
    npm: 10.7.0 - /usr/local/bin/npm
    pnpm: 9.10.0 - /usr/local/bin/pnpm
    bun: 1.0.13 - ~/.bun/bin/bun
  Browsers:
    Brave Browser: <IP_ADDRESS>
    Chrome: 128.0.6613.138
    Safari: 18.0

package.json dependencies
No response
Steps to reproduce
We're using a PNPM monorepo, each package has it's own type check. We run the these using Turborepo. The issue still happens randomly even type checking individual packages.
We're also using typescript references.
What is expected?
.vue-global-types/vue_3.5_false.d.ts is found
What is actually happening?
.vue-global-types/vue_3.5_false.d.ts is not found
Link to minimal reproduction
No response
Any additional comments?
We've tried adding
"plugins": [{ "name": "@vue/typescript-plugin" }],

directly to the tsconfig's and also tried adding
"paths": {
  "vue/__globalTypes_3.5_false": ["node_modules/vue/__globalTypes_3.5_false.d.ts"]
},

This didn't seem to help

.vue-global-types/vue_3.5_false.d.ts should exist directly in node_modules.

I have a similar problem, which might be related. In a PNPM monorepo I have one library and an app. The app includes some library files using relative paths, like ../../library/src/somefile.vue.
.vue-global-types/vue_3.5_false.d.ts is created inside app/node_modules, but not for the library or the repo root, which causes this error for files imported from library. I've previously used vue-tsc 1.8.27, which worked fine with this setup.
I tried to fiddle with vite-plugin-checker's vueTsc root/tsconfigPath options, but couldn't get the file to be generated to repo root node_modules.

pnpm + monorepo, vue-tsc check UnoIcon.vue Error at .pnpm/@unocss
 ERROR(vue-tsc)  Cannot find type definition file for '.vue-global-types/vue_3.5_false.d.ts'.
 FILE  /Volumes/...../node_modules/.pnpm/@unocss+nuxt@0.62.4_magicast@0.3.5_postcss@8.4.47_rollup@4.22.0_webpack-sources@3.2.3_webpack@5.94.0_esbuild@0.23.1_/node_modules/@unocss/nuxt/runtime/UnoIcon.vue:4:23

    2 |   <div />
    3 | </template>
  > 4 | /// <reference types=".vue-global-types/vue_3.5_false.d.ts" />

Having the same problem here within a Nuxt project. The issue persists for me at least as far back as 2.0.29

My workaround was to not point on the other package via relative path but via node modules mananged by pnpm.
So I've replaced:
../../packages/my-package
by
node_modules/@company/my-package
in my TypeScript alias

in my case, i downgrade the pnpm as follows,
but i am not sure if it is all pnpm reson,

✅ pnpm 9.9.0 + shared-workspace-lockfile=false
❌ pnpm 9.10.0
❌ pnpm 9.11.0

same problem in pnpm monorepo.
Error happens when trying import .vue file in another package.
For example, when executing vue-tsc in /apps/my-app, it will generate .vue-global-types/vue_3.4_false.d.ts in /apps/my-app/node_modules.
Then we import my-lib/src/some-comp.vue which locates in /lib/my-lib, it cannot find a vue_3.4_false.d.ts for some-comp.vue since they are in different folders.
/apps
  /node_modules
    /.vue-global-types
  main.ts
/my-lib
  /node_modules
    [missing .vue-global-types]
  /src
    some-comp.vue

https://github.com/vuejs/language-tools/blob/6916e42fb7109d9510e181de24f1a0f3d56da63b/packages/language-core/lib/utils/ts.ts#L289-L300
viewed source code, seems like it'll put .vue-global-types/vue_3.4_false.d.ts on the same level of vue in node_modules.
But it's not guaranteed this file will be resolved by all triple slash of files from anywhere:
yield `/// <reference types=".vue-global-types/${options.vueCompilerOptions.lib}_${options.vueCompilerOptions.target}_${options.vueCompilerOptions.strictTemplates}.d.ts" />${newLine}`;

Anyone got any idea to fix this?

Is there any workaround before the PR is merged? (btw nice work @depressedX!!!)

Is there any workaround before the PR is merged? (btw nice work @depressedX!!!)

You use use pnpm-patch (or other third party library patch tool) to add changes in this PR to your project. And that's exactly how I did in my project.


ORC-1205: Size of batches in some ConvertTreeReaders should be ensured before using

What changes were proposed in this pull request?
This fix introduces a new API for TreeReaders: ensureSize, which is supposed to be called by TreeReader clients before making a call for reading a new batch.
Why are the changes needed?
As described on jira, if there are batches of increasing size while reading, if they are in different stripes, we can hit an edge case where nextBatch doesn't force them to ensure the size of some batches.
How was this patch tested?
Unit tests added for all possible ConvertTreeReader, also tested in hive locally.

BTW, @abstractdog . Which ORC version are you using now?

The latest one is https://orc.apache.org/news/2022/06/16/ORC-1.7.5/
1.6.x will be EOL soon.

Thanks to @abstractdog  for the extensive test coverage. I verified that the patch works.
The current reader inheritance relationship looks like this
TypeReader +--> TreeReader +---> ConvertTreeReader +-> Various ConvertTreeReaderImpl
                    +
                    |
                    +--------------------------------> Various TreeReaderImpl

The new ensureSize interface is called recursively to ensure the size of the vector, which is a good solution to the out-of-bounds problem in Various ConvertTreeReaderImpl. However, in the Various TreeReaderImpl code path we had the ensureSize method being called repeatedly. I think this is a good refactoring design and I wonder if we should simply fix the out-of-bounds issue in the ConvertTreeReader first. Then refactor the ensureSize entry in another pr to be consistent.
Please let me know if I am not correct.

thanks for the comments so far!
@dongjoon-hyun: hive is on ORC 1.6.9, so I'm motivated to backport the patch to branch-1.6 too if it can make its way to at least a single release, but if it becomes EOL, I don't mind, we're fixing this on 1.7.x too, hive should upgrade eventually anyway :)
@guiyanakuang: can you please elaborate on this:
in the Various TreeReaderImpl code path we had the ensureSize method being called repeatedly

it's not clear where exactly this happens, and why it is a problem, but otherwise, I'm fine with fixing this on ConvertTreeReaders only: so did you mean that the ensureSize method can stay exposed in TypeReader interface (as it is in the current patch), and let it be a no-op in TreeReaderFactory (so ConvertTreeReader will implement it instead of calling super.ensureSize)?

What I mean is that this pr is more like a refactoring than a problem fix.
The ensureSize interface is defined in the TypeReader, and if this pr is treated as a refactoring, our modification looks only half done. There are still many temporary reuse vector calls to ensureSize in Various TreeReaderImpl, rather than using the new interface to control size (e.g. line 2131).
https://github.com/apache/orc/blob/3eac56761f48f3e315eea56dc63de9f2cbadcdbb/java/core/src/java/org/apache/orc/impl/TreeReaderFactory.java#L2126-L2134
So I think it would be better to use a separate pr to abstract the code and use the public ensureSize interface (both ConvertTreeReaderImpl and TreeReaderImpl use this interface to control the size of the vectors).

I see, makes sense @guiyanakuang as I'm now interested in fixing the issue ASAP for ConvertTreeReaders, the best way is what you proposed in Jira, right?
    if (doubleColVector == null) {
        // Allocate column vector for file; cast column vector for reader.
        doubleColVector = new DoubleColumnVector(batchSize);
        longColVector = (LongColumnVector) previousVector;
    }
+   else {
+       doubleColVector.ensureSize(batchSize, false);
+   }

without touching the interface
if so, let me refresh the PR soon with the fix

Yes I do think so.

thanks @guiyanakuang, just force pushed the fix

thanks for your comments and approving this PR @guiyanakuang
if patch looks good, I'm about to open some PR with the very same contents, I'll update jira

In addition,

Please revise the PR description. The current one is invalid.

This fix introduces a new API for TreeReaders: ensureSize

After revising the PR description, please copy and paste it into your new PRs. PR title and description become a commit title and message. So, the following is an invalid PR description in your backporting PRs in Apache ORC community.

same patch as main

finally, this doesn't introduce any APIs, updating descriptions


Change typing for masks

@jafingerhut any observations?

Ok, then I will merge this; during the LDWG this solution was suggested, so I assume we agree to push this into 1.2.1.


feat(ui): top-navigation - allow popover-based actions

Please read and mark the following check list before creating a pull request:

[x] I read and followed the CONTRIBUTING.md guide.
[x] I read and followed the New Feature Checklist guide.

Short description of what this resolves:

Closes #489

Answered in: ### [#489](# https://github.com/akveo/react-native-ui-kitten/issues/489
I) propose to "freeze" this issue until clarification of the new occasion.


test(search-lib): add unit test for search component.

Description
Unit test cases added for search-lib ie. Search Component
Fixes # (issue)
#867
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Intermediate change (work in progress)

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[ ] Test A
[ ] Test B

Checklist:

[ ] Performed a self-review of my own code
[x] npm test passes on your machine
[x] New tests added or existing tests modified to cover all changes
[ ] Code conforms with the style guide
[ ] API Documentation in code was updated
[ ] Any dependent changes have been merged and published in downstream modules

Add coverage script and share coverage percentage in the PR.

@Surbhi-sharma1 the coverage should be atleast 75%.


Suggestion: change the ** **  to

Hey luke/Hey Community,
big fan of the project, please consider changing the ** markdown thing to  html-statement.
Thanks!

Why?

Why?

Why not have a bold theme-ish thing?

**text** already becomes text.

The literal point of markdown is to make our life slightly less miserable by not writing HTML, as well as the possibility to compile it to different formats like man pages.
Putting HTML in a markdown file is like putting an asm() directive in C code, or soy milk in your coffee. We simply don't do it here.

The literal point of markdown is to make our life slightly less miserable by not writing HTML, as well as the possibility to compile it to different formats like man pages.
Putting HTML in a markdown file is like putting an asm() directive in C code, or soy milk in your coffee. We simply don't do it here.

The only reason you would want to compile a man page is when you have many many formats, docs. html-docs and etc. Writing a man page shouldnt be an issue every once in a while when there is a new future.

soy in your coffee
haha I am better than you
linar thinking like you are in a cult
This shame lack of creativety is why the modern web is so bloat, so do me a favor have some critical thinking, and have some creativity. But actually I am not sure about
Its liking using asm() in your C code
while in many cases it wouldnt make sense sometimes it would make in rare edge cases where you might want to optimise a certain issue with your compiler? regarding our case I really dont think its the same thing and that we can compare that to our case because its literally changing something you woul;d write in X language lets say you use python and you write your line which is "xValue = 'string' " and then you regex and replace it to look like something you would see in a C code lets say ? its the same thing I think think your argument is too strong, I disagree but I guess this is just my opnion not here to attack last time I touched code was in a SVN in 2008 and that was a helping an opengl demo for a friend written in C.

to different formats like man pages.
why not regexing it("") and literally replacing it with   do we have an issue regexing between two different strings and put html tags between the two?

How is this an hassle?
Literally all you need to do is A , find all test between ** ** and then put the HTML tags around the regex text once that done remove all the * from the text and you should have it
Maybe I am missing here something and I sound like a complete idiot but I dont understand where is the issue here?

The recipes are written in markdown and markdown syntax for bold text is **text you want emboldened**.
The site generator then converts the markdown syntax to HTML, in which the syntax for bold text is text you want emboldened.
I'm not quite sure what the issue here is.

The recipes are written in markdown and markdown syntax for bold text is **text you want emboldened**.
The site generator then converts the markdown syntax to HTML, in which the syntax for bold text is text you want emboldened.


[MacOS] Shows windows controls on macos when platform is not specified

I was trying to debug why Windows controls were showing up on my project, but trying to run the SolidJS example yields similar results. As seen in the image, the first title bar, which, as per the code, should auto-detect the platform <WindowTitlebar class="border">content</WindowTitlebar> , detects the platform as windows.

I am on an M1 Pro Macbook.
Additionally, in my own project, I also get an error regarding a 400 error from the OS plugin. Note: I have the exact same cargo.toml, tauri.conf.json, and package.json versions for the dependencies.

Checking tauri-plugin-os, the rust code has no kind handler...

Thanks for the issue. Releated to #14

I had the same probably but it was caused by the platform being undefined. I fixed this by adding this to src-tauri/capabilities/default.json
"os:allow-arch",
"os:allow-hostname",
"os:allow-os-type",


Implement LockFreeExponentiallyDecayingReservoir

This implementation has several advantages over the existing
ExponentiallyDecayingReservoir:

It exclusively uses the precise system clock (nanotime/clock.tick)
instead of a combination of nanotime and currentTimeMillis so it's
not vulnerable to unexpected NTP clock jumps.
Lock free for substantially better performance under concurrent
load[1] and improved performance in uncontended use[2]
Allows the rescale threshold to be configured programatically.

Potential trade-offs:

Updates which occur concurrently with rescaling may be discarded if
the orphaned state node is updated after rescale has replaced it.
In the worst case, all concurrent threads updating the reservoir may
attempt to rescale rather than a single thread holding an exclusive
write lock. It's expected that the configuration is set such that
rescaling is substantially less common than updating at peak load.

[1] substantially better performance under concurrent load
32 concurrent update threads
Benchmark                                 (reservoirType)  Mode  Cnt
Score      Error  Units
ReservoirBenchmarks.updateReservoir            EXPO_DECAY  avgt    5 8235.861 ± 1306.404  ns/op
ReservoirBenchmarks.updateReservoir  LOCK_FREE_EXPO_DECAY  avgt    5 758.315 ±   36.916  ns/op

[2] improved performance in uncontended use
1 benchmark thread
Benchmark                                 (reservoirType)  Mode  Cnt
Score    Error  Units
ReservoirBenchmarks.updateReservoir            EXPO_DECAY  avgt    5 92.845 ± 36.478  ns/op
ReservoirBenchmarks.updateReservoir  LOCK_FREE_EXPO_DECAY  avgt    5 49.168 ±  1.033  ns/op

I'm not sure if this should have gone toward the 5.x branch instead, I defer to your judgement.
This could be considered as an update to the existing ExponentiallyDecayingReservoir type, however it doesn't have a good way to support the public void update(long value, long timestamp /* epoch seconds */) method without additional cruft -- maybe that could be removed in 5.x?

@carterkozak Thanks for this interesting contribution!
Could you please rebase this PR on the release/4.2.x branch? I would try to avoid putting this change into Metrics 4.1.x but into 4.2.x instead.
In your opinion, would this PR replace #1638?

Could you please rebase this PR on the release/4.2.x branch? I would try to avoid putting this change into Metrics 4.1.x but into 4.2.x instead.

Of course, I'd be happy to.

In your opinion, would this PR replace #1638?

I believe so, however that PR replaces the ExponentiallyDecayingReservoir where this one adds a new Reservoir implementation -- I can update this design to replace the ExponentiallyDecayingReservoir with the exception of an update(value, timestampSeconds) method if you like, but that might make more sense in a 5.x release. What do you think?

I agree this change could replace #1638 , they seem to solve the same problem.
I am not sure which is best, but I think it was a good idea to create a new reservoir instead of changing the old one.
My only concern is that this solution might lose more data points during a rescale than in #1638
If you generate data points at a high rate, it seems likely to trigger multiple rescale operations before the first rescale has completed.
If this PR is merged, I can rebase my change on it and see if it still makes sense in some form.

my only concern is that this solution might lose more data points during a rescale

In both implementations we cannot guarantee events are not lost. Given that this is a sampling implementation I opted for simplicity in the update path (single volatile state read) with the risk that updates may be ignored if the state read occurs prior to a rescale clock tick but commits the update after another thread has begun a rescale. When two threads race a rescale, the failed compare-and-swap results in a new volatile read to update the fresh state object.

If you generate data points at a high rate, it seems likely to trigger multiple rescale operations before the first rescale has completed

That is correct, by design (not to suggest my design is ideal :smile:). We may spend more cycles rescaling under concurrent load, but we don't have to worry about failures leaving the reservoir in a degraded state. Rescale is both infrequent and inexpensive enough that I expect the cost to be negligible.
Related thought (probably best for another PR if it's worthwhile at all): Most applications I work on create the vast majority of Histograms and Timers on startup, around the same time, and in the worst case have constant use. This results in a concurrent rescale of every reservoir at the same time, perhaps we should add some jitter to prevent load spikes?

@carterkozak I made this commit with my idea: https://github.com/spkrka/metrics/commit/08f16ac166cd8ccf76dbe7127d6fe458f04dcaf5
(Didn't manage to figure out how to make a PR for your branch - if you like it you can just take it, no need to preserve author)

I need to update the benchmark results. Single-threaded results have improved ~20% and concurrent load results have improved by an order of magnitude from ~700ns/op to ~70ns/op!

I've updated the benchmark results in the PR description.

Would it be possible to also get this into 4.1.x and get a release?
Alternatively release 4.2.x (Seems like no release on that branch exists yet)
No urgency from my side, would just be nice to clean up some manual inlining on my end.

@spkrka I'd like to keep it in Metrics 4.2.x.
This being said, we'll probably publish a first beta of Metrics 4.2.0 soon.


Update SearchBar help text

Summary of Changes
We should update the help text along with #228 if we are also supporting 'database' for advanced search syntax.
Tests
N/A
Documentation
N/A
CheckList
Make sure you have checked all steps below to ensure a timely review.

[x] PR title addresses the issue accurately and concisely. Example: "Updates the version of Flask to v1.0.2"

In case you are adding a dependency, check if the license complies with the ASF 3rd Party License Policy.

[x] PR includes a summary of changes, including screenshots of any UI changes.
[x] PR adds unit tests, updates existing unit tests, OR documents why no test additions or modifications are needed.
[x] In case of new functionality, my PR adds documentation that describes how to use it.

All the public python functions and the classes in the PR contain docstrings that explain what it does

[x] PR passes all tests documented in the developer guide
[x] I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message"

Codecov Report

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

@@           Coverage Diff           @@
##           master     #229   +/-   ##
=======================================
  Coverage   70.02%   70.02%
=======================================
  Files         124      124
  Lines        3219     3219
  Branches      390      390
=======================================
  Hits         2254     2254
  Misses        934      934
  Partials       31       31

Impacted Files
Coverage Δ

...ic/js/components/SearchPage/SearchBar/constants.ts
100% <ø> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 35fa2c3...eec53fc. Read the comment docs.


adapt functions for diagnostic_variables.jl

@maximilian-gelbrecht Adapt functions for the structs in diagnostic_variables.jl in the way they were also implemented in the lower_triangular_matrices.jl file. I haven't tested them yet, but wanted to know if this is basically the right way to do it.

I've just resolved the conflicts that arose because I changed all gridded fields from Matrix to Grid<:AbstractGrid as #127 now introduces a grid-flexible spectral transform. Quick note: I usually write {NF} ... where NF in two cases.

When I want to constraint the method of a function to input arguments with the same parametric type. That means in

DiagnosticVariables(G::Geometry{NF},S::SpectralTransform{NF}) where NF = zeros(DiagnosticVariables,G,S)

Both G and S need to have the same NF. But NF isn't actually used in the function body.

If I want to use NF (here T) in the function body. Example

function clip_negatives!(A::AbstractArray{T}) where T
    @inbounds for i in eachindex(A)
        A[i] = max(A[i],zero(T))
    end
end

Otherwise I don't think there's a need to write where NF.

@milankl why do you want this to merge directly into main? I would rather not do this until the kernelabstractions version is actually working

@milankl why do you want this to merge directly into main? I would rather not do this until the kernelabstractions version is actually working, we agreed on a separate branch for this version at first. Also please give me the chance for a code review into the kernelabstractions branch as well and don't merge it immediatly, even if it is not the main branch. Now, this is a bit messy with adding to this PR which shouldn't go into the main.

Sorry my bad! Misunderstood what was supposed to go into which branch and just incorrectly merged two PRs that pointed into different branches. I'll let you merge anything kernelabstractions-related from now on! Sorry 😬

Ok to sort this out, I guess I would just close this issue and @katharinamaetschke does the corrections as another PR into the kernelabstractions_adtest branch
And for the next PRs, everything as planned also as PRs that point to the kernel abstractions_adtest

The last commit addresses the compat issues, the unpacking in the tests and using/import statements as we discussed in the reviews

I'll close this PR now. @katharinamaetschke please do another PR for the corrections into the kernelabstractions_adtest branch

I'll close this PR now. @katharinamaetschke please do another PR for the corrections into the kernelabstractions_adtest branch

Yes, I willl, I am very sorry for taking so long!


Importing order is not working consistently across OSs (MacOs)

Hello Dear,
Its similar to this issue:
https://github.com/benmosher/eslint-plugin-import/issues/1692
Running the lint here on macs it complains about the importing order.
Running on mohave.
nodejs 12.18.2
"eslint": "^7.3.1"
"eslint-plugin-import": "^2.18.2"

Can you provide the code that you're linting, and the error that you're seeing?

@ljharb,
Sure thing.
MacOS

windows

same here

This would be easiest to debug with either a PR with a failing test case, or with a repro repo.

This is still an issue, right?

@caiokoiti yes, i still haven't found a way to reproduce it. A failing test or repro repo would help tremendously.

Experiencing the same problem. I get import errors on Mac and get no errors on remote development server with Linux


added a new file

thanks
On Fri, Feb 27, 2015 at 12:37 PM, Dan Kotowski<EMAIL_ADDRESS>wrote:

Closed #533 https://github.com/jmmastey/level_up_exercises/pull/533.
—
Reply to this email directly or view it on GitHub
https://github.com/jmmastey/level_up_exercises/pull/533#event-243463746.


[DOCS] Create Example Sections

Create Example Sections
This is an open-ended issue and we welcome multiple PRs.
I would like to add a few example sections to the documentation. Each example should be self-contained and should demonstrate a potential usage of the Graaf library.
If you are interested, you can pick an example from the proposals below, or (even better) suggest your own example. The scope of a PR is restricted to a single example.
Definition of Done
Each example should contain the following:

[ ] A complete and self-contained example under examples/EXAMPLE_NAME/main.cpp.

To verify that the example compiles and runs.
This can be used to generate any graphics (via dot files) for the public documentation.
See the dot serialization example for inspiration.

[ ] An entry on our public documentation.

A short description of the example, what is the use case and why is it typical.
A list of the Graaf algorithms and data structures which are demonstrated in the example. Here you can link to the respective algorithms page in the documentation.
Code pointers (in the form of code blocks) to the important sections of the example. It is not necessary to cover the entirety of the added main.cpp, but the most important concepts should be captured.
Non-trivial parts in the code blocks should be explained in-text.
Graphics (i.e. in the form of visualized dot files) can be used. Any graphics should go under `docs/static/img/examples/.
Instructions on how to build the documentation locally can be found on the wiki.
An example entry can be found here.

Proposed Examples

[x] Dot serialization
[x] Shortest path (directed graph)
[x] Shortest path (undirected graph) [@Hromz]
[ ] Cycle detection
[ ] Social network use case
[ ] Minimal CNN implementation
[ ] ...

Hey @bobluppes, i would like to work on examples for documentation. Can i try?

Hi @Hromz, of course, looking forward to your contribution!
Please note that we have slightly changed the format we use for the examples. There is now a single source file with the example code under examples/YOUR_EXAMPLE_NAME/. Additionally, there is a docusaurus entry explaining it. I have updated this in the original issue and added links to the wiki on how to build and host the documentation locally.

Hey @bobluppes, I made PR #47. Can you take a look? Simple example that uses graph traversal and the shortest path. In the future, I will add better examples.

@bobluppes can I work on cycle detection example?

@bobluppes can I work on cycle detection example?

Hi @Hromz, of course! Looking forward to your contribution :)

Hey There. I would like to contribute in your repository so kindly grant me permission to do so.

Hey There. I would like to contribute in your repository so kindly grant me permission to do so.

Hi @arnav1203, welcome to Graaf!
Of course! Would you be interested in working on the social network use case?

can u plz assign me this issue , I would like to contribute to this issue .


Merge NLog 4.7.7 to dev

@304NotModified Think I pressed the wrong button on merge. Became a squash instead of merge. Should we "uncommit" and re-merge ?

@304NotModified Think I pressed the wrong button on merge. Became a squash instead of merge. Should we "uncommit" and re-merge ?

And please remove with force push. Then a new pr or a push from local.

And please remove with force push. Then a new pr or a push from local.

The "merge when passing" wasn't triggering?

No because the build-servers required "restart".

The "merge when passing" wasn't triggering?

No because the build-servers required "restart".

Yes please remove with force push.

Not sure I can force push agains the dev-branch. Says "Cannot force-push to this protected branch"

Yes please remove with force push.

Not sure I can force push agains the dev-branch. Says "Cannot force-push to this protected branch"

Ow yes force push is only for admin I guess, will do that

Ow yes force push is only for admin I guess, will do that

The squased commit (4dbc737b61e3a8fb97adc551c43719342fcb4d91) has been removed from dev and I'm busy with a new PR

The squased commit (4dbc737b61e3a8fb97adc551c43719342fcb4d91) has been removed from dev and I'm busy with a new PR


April Docs Merge & CPM

Docs Build status updates of commit c6a9009:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/consume-packages/Central-Package-Management.md
:warning:Warning
View
Details

docs/TOC.md
:bulb:Suggestion
View
Details

.openpublishing.publish.config.json
:white_check_mark:Succeeded

docs/api/catalog-resource.md
:white_check_mark:Succeeded
View

docs/api/search-autocomplete-service-resource.md
:white_check_mark:Succeeded
View

docs/consume-packages/Package-References-in-Project-Files.md
:white_check_mark:Succeeded
View

docs/consume-packages/Package-Source-Mapping.md
:white_check_mark:Succeeded
View

docs/consume-packages/managing-the-global-packages-and-cache-folders.md
:white_check_mark:Succeeded
View

docs/create-packages/Select-assemblies-referenced-by-projects.md
:white_check_mark:Succeeded
View

docs/create-packages/Sign-a-Package.md
:white_check_mark:Succeeded
View

docs/hosting-packages/Overview.md
:white_check_mark:Succeeded
View

docs/reference/Errors-and-Warnings.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU1005.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU1301.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU1504.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU1505.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU1506.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU3004.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5133.md
:white_check_mark:Succeeded
View

docs/reference/errors-and-warnings/NU5501.md
:white_check_mark:Succeeded
View

docs/reference/media/NuGetProperties.png
:white_check_mark:Succeeded
View

docs/reference/media/NuGetUnblock.png
:white_check_mark:Succeeded
View

docs/reference/msbuild-targets.md
:white_check_mark:Succeeded
View

docs/reference/nuget-config-file.md
:white_check_mark:Succeeded
View

docs/reference/packages-config.md
:white_check_mark:Succeeded
View

docs/release-notes/NuGet-6.1.md
:white_check_mark:Succeeded
View

docs/visual-studio-extensibility/nuget-api-in-visual-studio.md
:white_check_mark:Succeeded
View

docs/consume-packages/Central-Package-Management.md

Line 19, Column 15: [Warning: file-not-found - See documentation] Invalid file link: '..\release-notes\NuGet-6.2.md'.

docs/TOC.md

Line 68, Column 5: [Suggestion: preserve-view-not-set - See documentation] Did you mean to link to a specific version? It's usually better not to specify a version, so the link always goes to the canonical version. If you really need to link to a specific version, add &preserve-view=true to the URL: /azure/devops/artifacts/nuget/publish?view=azure-devops. Otherwise, remove the view parameter.

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


Trimmer for illumina adapter

In GATK there is going to be a new ReadTransformer for trimming illumina adapters. Maybe we can use it as a trimmer for our framework too, but that will require to change the interface a bit.
We can also add ReadTransformer plugin once it is implemented in GATK, to apply either before/after proper trimming...

Unblocked because GATK already includes AdapterTrimTransformerin the latest version. We require to add a wrapper around it to include parameters and be a sub-class of the trimmer.

Related with #66


How to get HTML content

Any idea how I could get the innerHTML of an element? I want to select an id and get the html of that element.

I think you would need to parse it into a string for that.

I think you would need to parse it into a string for that.

Perfect that worked.
let document = kuchikiki::parse_html().one(html_string);
let selector = document.select_first("#my-content").unwrap();
println!("{:?}", selector.as_node().to_string());


After uninstalling and reinstalling, the NPU cannot be invoked. Can't even see the npu in the panel

Describe the bug

Steps to reproduce

Go to '...'

Result

Expected behavior

Environment

GPUStack version:
OS:
GPU:

Additional context
Add any other context about the problem here.

Could you provide more details about this? Including:

What's your OS and CPU arch?
How did you install and uninstall GPUStack? By installation script or pip directly?
There are some improvements in GPU detection in version 0.4.0. Could you try 0.4.0 and let us know the result?


Suggestion: how about using Google's Maven mirror in maven-example/pom.xml?

How about using Google's Maven mirror (https://storage-download.googleapis.com/maven-central/index.html) in maven-example/pom.xml? The proximity from Google Cloud Build to the mirror within GCP should be good for cloud build users for the following reasons:

It's fast.
It reduces unnecessary traffic that crosses the boundary of Google Cloud.

According to the link provided, "This is not an officially supported Google product."
With the release of Assured OSS, there's documentation there for how to use alternative mirrors https://cloud.google.com/assured-open-source-software/docs/download-java-packages


Fantastic work! When the code and dataset will be released？

We will release the dataset in this week.


dssp version argument

Hello!
https://github.com/a-r-j/graphein/blob/3d7af1fd07dc16707068b2d63a2f3668fc02c632/graphein/protein/features/nodes/dssp.py#L110C71-L110C71
Here, the dssp version is not specified, which leads to Bio.PDB.DSSP uses the default version 3.9.9 for dssp. For people using dssp version >=4.0.0 this leads to the output format by default as mmcif.
I included Bio.PDB.DSSP line 135 for reference
def dssp_dict_from_pdb_file(in_file, DSSP="dssp", dssp_version="3.9.9"):
... ...
    if version(dssp_version) < version("4.0.0"):
                DSSP_cmd = [DSSP, in_file]
            else:
                DSSP_cmd = [DSSP, "--output-format=dssp", in_file]

Because I'm using dssp version 4.0.4 and without specifying the version number, it leads to an empty dssp_df, and adding an arg     of dssp_version works.
dssp_dict = dssp_dict_from_pdb_file(pdb_file, DSSP=executable, dssp_version="4.0.4")

Maybe add a version attribute in DSSPConfig? or automatically detect it from running subprocess dssp --version which returns mkdssp version 4.0.4 for me.

Hi @biochunan, appreciate you raising this issue. I'm currently loathe to support DSSP 4 as it requires CRYST1 records for PDB files, which while sensible, will break compatibility for a lot of people who use Graphein for predicted structures. As a result, I'd suggest the conda distribution of DSSP provided by the Sali lab. I realise this is not an ideal strategy in the longterm but am not able to prioritize this at the moment, if you'd like to make a PR, I'm happy to assist - as you might be able to tell from this PR I've already tried to setup a more universal solution. Automatically detecting the version seems like the best approach.

Thanks for getting back to me! I've opened a PR #355 .


Adding full mailaddress as alias-target is not working

Cookbook version
Latest
Chef-client version
Latest
Platform Details
Debian 8
Scenario:
Adding a full emailaddress<EMAIL_ADDRESS>to the postfix aliases doesn't
work. The template (aliases.rb) keeps adding double-quotes around it, whereas
postfix then doesn't recognize the domain-part and threats it as a local
address. The manpage (http://www.postfix.org/aliases.5.html) doesn't state that
an "address" needs to be quoted. But once an address is quoted, postfix will
append the domain and send mails to "sysadmin<EMAIL_ADDRESS>- which
obviously doesn't work.
Can anyone else confirm this behaviour?
Steps to Reproduce:
node.override['postfix']['aliases'] = { "www" =><EMAIL_ADDRESS>}
Expected Result:
$ cat www /etc/aliases

#
# This file is generated by Chef
#
# (...)
www<EMAIL_ADDRESS>
Actual Result:
$ cat www /etc/aliases

#
# This file is generated by Chef
#
# (...)
www<EMAIL_ADDRESS>

This has been fixed with the merge of PR #130 and can therefore be closed.


nifi: 1.28.0 -> 2.0.0

Fixes #351389
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
25.05 Release Notes (or backporting 24.11 and 25.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Error:
  l7hxi6pw00i4dmdspg0py1s7z57bvx44-nifi-pre-start.sh[859]: cp: cannot stat '/nix/store/n6civkg1a6asdvn5b7g8nq3bv9y33v2d-nifi-2.0.0/share/nifi/conf/bootstrap-hashicorp-vault.conf': No such file or directory

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/4920


How do I use the two models？

I am a communication engineering student who is new to python, I got the best and final two models after training, and I would like to know how exactly to use these two models?How do I use them for encoding, decoding and modulation?

Currently my project looks like this


fix(TDI-46380): CVE: update daikon to 0.31.12

What is the current behavior? (You can also link to an open issue here)
https://jira.talendforge.org/browse/TDI-46380
What is the new behavior?
https://jira.talendforge.org/browse/TDI-46380
Please check if the PR fulfills these requirements

[x] The commit message follows Talend standard
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features) ?
[ ] The code coverage on new code >75%
[ ] The new code does not introduce new technical issues (sonar / eslint)

What kind of change does this PR introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build / CI related changes
[x] Other... Please describe: CVE

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Pull request test result: failure :x:


Open themes .css files

My obsidian vault does not recognize the .css files. How can I enable the .css themes you attach in your github? Should I install some complementary plugin? Think that my vault has only Marp's plugin installed as a test.
https://samuele-cozzi.github.io/obsidian-marp-slides/23.SlidesCustomTheme.html
Regards

I use vscode to edit the css files. After edit the css, I reload the preview of the deck


add magic actions

Add kind of "energy" that allows special actions:

speed up all chips
mass invert all attackers

perhaps not a good idea. Too complicated.


TypeError: can't pickle Environment objects

I'm getting this error when trying to train on default data, any information is helpful!

It seems that this is a windows specific issue, and has to do with the number of workers > 0. If I set num workers = 0, it runs fine.

It seems that this is a windows specific issue, and has to do with the number of workers > 0. If I set num workers = 0, it runs fine.

Thanks! Windows is always trolling...


Update leaflet.dbpedialayer-src.js

Hi,
I think it will be good and very useful if this plug-in resets the visitedBounds every time we need to disable and enable again the DBpediaLayer.
In my opinion I consider a bug the fact we are keeping the bundle of visited bounds when we disable the plug-in and clean the markets, because when we turn it on again, the array still has the old coordinates and the query will discard to show the poi's markers again for areas we had seen before.
I think it makes sense reset the visiting bounds every time we do reload the label, and I believe this will help many guys out there..
It's my suggestion.
cheers
José Gomes

Hi José,
it is a good addition.
Thanks for the patch!
I will merge it. I think it would be great to add this also to
src/dbpedia_layer.js
Christian


setGlobalLogLevel, setLogLevel, muteLoggers don't work

Describe the bug
setGlobalLogLevel, setLogLevel and  muteLoggers don't change the logging behaviour.
To Reproduce
Steps to reproduce the behavior:

use setGlobalLogLevel, setLogLevel or muteLoggers to get fewer log messages
Execute executeHttpRequest
See a lot of 'info'-level log messages

Expected behavior
It should only show the appropriate log messages
Used Versions:

node version via node -v: v16.14.0
npm version via npm -v: 8.3.1
SAP Cloud SDK version you used as dependency: "@sap-cloud-sdk/core": "^1.54.0", "@sap-cloud-sdk/util": "^2.1.0"

Impact / Priority

This makes the logs harder to read during development.
Affected development phase: Development
Impact: Inconvenience, I

Hi brodo,
@sap-cloud-sdk/core should be removed as it has been split into different modules, on of such is @sap-cloud-sdk/util. After having fixed the imports, you should be able to influence logging behavior as it was before.
What I have seen after updating is, that some messages appeared e.g. for loggers in destination context, that should have been muted by a less detailed log level but appeared anyway. It seems, that messageContexts win over loggers in precedence - so you might need to modify the log level for some messageContexts in addition.
register-destination messageContext for example is one of those candidates, that appeared in the logs also their logger was set to error-level.

Good morning,
@nocheintobi is right and you have a mix of version 1 and version 2 of the SDK. We recommend to use the latest version and do not use core but the smaller pacakges like connectivity or OData depending on you use case. Here is a migration guide in case you have already some code in the old version. The breaking changes are not so large.
I will create a backlog item about the log level issue. If you set one specific logger to error and do not touch the level on the messageContext this should block it.
Best
Frank

@nocheintobi could you perhaps give the steps to reproduce the behavior with the logger?

Thanks a lot @nocheintobi @FrankEssenberger for helping so quickly! It worked, but I still have a very verbose error output when an error happens. I'll open another issue for that though.

@FrankEssenberger, I noticed it for the following messageContexts - all of which are logged via the sap-cloud-sdk-logger:

jwt
register-destination
env-destination-accessor
destination-accessor-vcap
destination-accessor-service
proxy-util (will only show up when using destination service instance instead of destinations env variable)

The following snippet will show when adding proper configuration to VCAP_SERVICES and filling in an existing destination name:
process.env.VCAP_SERVICES = `{"xsuaa": [...], "destination": [...]}`;

const { getDestination } = require("@sap-cloud-sdk/connectivity");
const { setLogLevel } = require("@sap-cloud-sdk/util");

setLogLevel('error', 'sap-cloud-sdk-logger');
// setLogLevel('error', 'proxy-util');
// setLogLevel('error', 'destination-accessor-service');
// setLogLevel('error', 'env-destination-accessor');
// setLogLevel('error', 'jwt');
// setLogLevel('error', 'register-destination');

(async() => await getDestination("<ValidDestinationName>")) ();


Update for latest EnderIO (Fixes #21)

Hello, I updated the mod to work with the latest EnderIO version.
I hope this can be pushed to CurseForge.
Thanks!

I cannot get this project to compile on my environment, so if you want to build a jar and submit it to me such that I can push it to CF, I can do that.
Otherwise this is likely to not ever be uploaded, as I don't have the desire to fiddle with this until it compiles or install java 8 again.

Hello, here is the compiled jar of the mod with the version updated to 1.1.2.
EnderTweaker-1.12.2-1.2.2.zip

Hello,
Thanks ElektroKill. I download your .jar and try this.
I have no errors or crashes while loading the modpack and the world. The new recipes are displayed in JEI. But when I try the new recipe in AlloySmelter, nothing happens. And after putting the input item once in one of the top solts, I can't put it back. As if the recipe was no longer known.
My very simple test script:
mods.enderio.AlloySmelter.addRecipe(<minecraft:iron_nugget> * 9, [<exnihilocreatio:item_ore_iron:1>], 4000);
Thank's for your help.


update readme: additional app using framework

Really enjoyed using this framework.
Just adding my app to the readme as one that uses StatusAlert.

Thanks for your feedback! Glad to see you're using this framework :)

No worries, feel free to approve & merge the pull request :)
On Tue, 9 Jun 2020 at 10:06, LowKostKustomzDL<EMAIL_ADDRESS>wrote:

Thanks for your feedback! Glad to see you're using this framework :)
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/LowKostKustomz/StatusAlert/pull/12#issuecomment-641144112,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAT6CWS6RMTUA43ORCFG7UTRVX3SZANCNFSM4NYYQS6Q
.


cantata: use Qt mkDerivation

Motivation for this change
Fix runtime error:
qt.qpa.plugin: Could not find the Qt platform plugin "xcb" in ""
This application failed to start because no Qt platform plugin could be initialized. Reinstalling the application may fix this problem.

Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[x] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @fuuzetsu, @peterhoeg

Merged in 9a9dc03cf49ab1672611ec07aead6130fd65b2be


kway --input

This is not trivial since you need to keep many files open. It shares many similarities with partition, but what's more you need to remember where the file was read before. I will close this until it becomes a true need.


Python 3.12 removed asyncore module

This library is broken with Python 3.12 because of removed asyncore module
27f4772d425a ~ # python3.12 pyinotify-0.9.6/python3/pyinotify.py -h
Traceback (most recent call last):
  File "/root/pyinotify-0.9.6/python3/pyinotify.py", line 71, in <module>
    import asyncore
ModuleNotFoundError: No module named 'asyncore'

Python 3.11 prints this warning:
27f4772d425a ~ # python3.11 pyinotify-0.9.6/python3/pyinotify.py -h
/root/pyinotify-0.9.6/python3/pyinotify.py:71: DeprecationWarning: The asyncore module is deprecated and will be removed in Python 3.12. The recommended replacement is asyncio
  import asyncore
Usage: pyinotify.py [options] [path1] [path2] [pathn]

Maybe this project is no longer maitained.
So I forked it and renamed to pyinotify-elephant-fork.
Try:
pip install pyinotify-elephant-fork

https://pypi.org/project/pyinotify-elephant-fork/


Search and show without sudo like apt

On a standard Ubuntu install, you can open a terminal and type "apt search foo" and it will query and return results.
It would be wonderful if deb-get did the same.
This is also true when using "apt show foo".

Duplicate of #33


Add "ParsedMethodView" and example

#220
@rochacbruno I don't kwon if the file/class name/structure is suitable, feel free to tell me if I'm wrong.

Coverage increased (+0.5%) to 91.056% when pulling f856d1276ba2cc9e4d4d194e725c2b0b6ea1a5dd on strongbugman:feature/add_parsed_view into a4340b799847886fcc637db15eba095062a452e5 on rochacbruno:master.

This is a great improvement, I'll try to test and review by this weekend!
If you can also take a look @javabrett :)

I Like the implementation with the ParseMethodView as a base class.
But I also think we can provide the same functionality for when only function based views are used, and also when no viewa at all are used. (this would address the issues #202 and #216
Example:
Swagger(.....parse_spec_path="path/to/docs")

Then we just parse the YAML and generate the swagger UI even without any view defined.

Thanks your suggestion, you are right, I will update this PR soon

@strongbugman this is awesome! I'll review ASAP!

@javabrett  Thx for your review, I fixed the errors and rebased the origin/master

Changes look good, 🍰 .  I don't think your rebase picked up the latest master but never mind - you might have forgotten to fetch or use origin/master.

@strongbugman  awesome!
@javabrett first, thanks for the awesome contribution to Flasgger reviewing and helping maintaining this project :sparkles:
@javabrett I am with little time this week to review OSS projects (due to work + FlaskConf which is this weekend). So I would like to ask if you can take care of this PR! this is awesome improvement and I do not want to hold it. As long as the code is good, working and we have some kind of doc (readme) about how to use it please decide if you want to merge it or request more changes! I'll let this one for you to review/merge!
:) thanks

@javabrett Thx for your suggestions  and patience, I changed the code according to your suggestions

Thanks @strongbugman for this contribution! 🍰


MW2019 SWEPs Why is the gun invisible?

I've made a gun but I can't see

I checked the sanbox, the gun works fine, but when it's in place, the Lambda doesn't see it.

How would I go about testing this myself? Please provide the steps required to reproduce this.

TFA Weapons ARC9 Weapons Compatible with Lambda

I think some details are still missing, are you using the workshop addons or did you install them manually? Also which gun is invisible, all of them or just a specific one? You need to be a bit more concrete on how I would do this, which workshop addons or normal addons do I need to install and from where. Do I need to toggle any specific options or set specific convar values? The more information you provide the better

https://steamcommunity.com/sharedfiles/filedetails/?id=2526457139
This add-on where i have a problem

I'm using the workshop add-on

I've had the same issue, for me every MWB weapon was invisible, functionality was just fine, and they didn't appear only in
the Lambda gamemode, in sandbox they worked fine. As for replicating the issue, just spawning the weapons with sv_cheats 1 should give you an idea, In my case, I used an addon which replaced NPC weapons with weapons that of MWB. Also, the world models loaded, it seems to not load the weapon model in firstperson only.

Would you update it or tell me how to fix it?

I've been wondering, have you attempted to recreate this issue? I've done it using an addon replacing default HL2 weapons with MWB weapons (and of course all weapons from Modern Warfare Base). I've loaded into d1_canals_01 and killed the metrocops, after that any weapon they dropped I could normally pick up but the viewmodel would be invisible, functionality was fine tho.

I've found the issue and will commit the fix asap.

I've pushed a smaller release which should solve the issue with SWEPs.


wptide.org domain has lapsed ERR_NAME_NOT_RESOLVED

The domain name didn't get renewed

this is the www. subdomain that's linked from the Make .org blog

The www should redirect is it not currently doing that?

@derekherman nope, it doesn't find a DNS record for the www subdomain so there's nothing to perform the redirection

www.wptide.org appears to be resolving now, so closing this.  Thanks @tomjn for flagging this!

URL working for me too, thanks!


Added awesome-environment-variable-names.txt and an auto-updater github action

The list comes from here: https://github.com/Puliczek/awesome-list-of-secrets-in-environment-variables

Thank you!
The github action is a nice touch :)


Optimize Snort image size

Optimize Snort image size and faster image creation by compiling source code with parallelization (4 concurrent jobs).
The image size shrunk from 1.45GB to 630MB.

Would it be logical to use something else than Alpine for this? E.g. could it be made smaller with some other OS? We could build out another SDK to make it easier/better.
Either way; looks good - good job :D

Yes, I agree creation of non-Alpine SDK image, I would suggest Debian based image.


Waiting gets longer for release of 78 version

Dear dev i am using iridium browser for privacy concerns in windows nd debian @ work. Windows version gets updated to 78 but debian is still 74 & using it like daily torture because missing features of chromium 78. Chrome:/flags. "Force Dark Mode for Web Contents" so its a humble request to you developer that please release new version based on chromium 78 asap. Please asap

@sahilneal  You can try Dark Reader addon while you wait. :)

@sahilneal You can try Dark Reader addon while you wait. :)

I dont wanna use extension because doesn't trust those extensions can see all data of my surfing.

@sahilneal  Do not worry it has been verified by Firefox in their version, so probably safe and doesn't send back data.


Allow disabling color

Related to #169. Reported in https://github.com/Azure/azure-cli/issues/6080. Also caused side effect of https://github.com/Azure/azure-cli/issues/9903.
If stdout is redirected and CLI displays a colored error message, the color in the console can't revert back to the default which is caused by https://github.com/tartley/colorama/issues/200.
Without proper a fix from Colorama, Knack should have the ability to disable color. This can be done by extending _should_enable_color by supporting env var like KNACK_ENABLE_COLOR or KNACK_DISABLE_COLOR.
https://github.com/microsoft/knack/blob/44ee1799c891213c90916f186ab9aee7b9ff8344/knack/log.py#L57-L64

Completed in https://github.com/microsoft/knack/pull/181.


[Fix](Row Store) all filter should match key columns condition (#36400)

Queries like select * from tbl will pass
LogicalResultSinkToShortCircuitPointQuery rule in the previous. Introduced by #35823
Proposed changes
Issue Number: close #xxx

run buildall


Voice Acting

From @aschaeffer on September 29, 2014 21:20
Recording voice samples: Killing Spree, ...
Voice: TOMYC
Ideas: @movabo
Copied from original issue: inexor-game/data#9

From @ObaniGemini on October 10, 2015 13:55
Do the old sounds need to be made also? (e.g: quad_damage annoucements,etc...)

From @Croydon on October 10, 2015 15:44
@ObaniGemini I will soon contact the creator (or one of them not sure without looking in it)  and ask for relicensing (he made also other kind of content). If he is up for relicensing we wouldn't need replacement, but for some reason I doubt that he will agree.

Please create a list of voices and voice acts (text) that are needed and how they are used

@MartinMuzatko This issue had the label question tagged.
I just renamed this label, since it was confusing from the very beginning. So it is now named needs validation/discussion/decision, exactly the way we always used it (this label is essential for our kanboard but important for a good organising in general).
Means in this case: This is not 100% planned out or fully decided.
The only thing which is clear right now is that we need replacements for the already existing ones, since it seems to be impossible to contact the creators and the license is unknown and therefore assumed as non-free.
That means we need the following voice acts:
* base captured
* base lost
* fight
* health boost
* health boost in 10 seconds
* lets fight
* quad damage
* quad damage in 10 seconds
* respawn point set


Create self-contained  module build

The previous PR, which aimed to fix requiring this component without a transpile step, was unsuccessful - https://github.com/ubilabs/react-geosuggest/pull/5
A transpiled, but non-packed distribution is required. The current distribution files include global variables  and wiring for use of require where it is not supported. This conflicts with a build system which supports require and not global variables.
The existing build has been set up solely for browserify output. This PR adds a new gulp task build-module, which uses babel to transpile the files into plain Javascript, but without packing the files together. These are output into the react-geosuggest.module directory within dist and the package.json has been updated to point to that directory. An index.js file is created as the entry point.
Additional changes:

license field added to package.json
google variable can be provided via props, but will default to the global variable (if set)
build task can be run via npm run build

Thanks for you PR!
For the additional changes – can you split them as own PRs to not mix up and have a descriptive history?
I’m not sure if checking in the react-geosuggest.module is the way to go… It is only needed for an npm publish. Maybe we should build and then publish? Doing this in another project and it is working fine. Need to check whether the react-component-gulp-tasks have this. Also I was thinking on switching to npm scripts. What do you think?

As we switched to Travis CI @Ubilabs, I configured it here, too. Only checking eslint with our code styles. A merge of the current master is appreciated. :)

@ro-ka I've just rebased against master. I'll look at splitting out PRs and updating this tonight.

For the additional changes – can you split them as own PRs
Will do

I’m not sure if checking in the react-geosuggest.module is the way to go
Fair enough - I'm not sure how you manage your build files and publish.

I was thinking on switching to npm scripts
I'm a big fan of using npm run x as it abstracts away what the underlying tool is and means that I don't have to install it globally :)

I've split off the additional changes and updated the description for this PR to reflect.
New PRS are #9 and #10
I've also left off the built files that I'd included before; I'll leave you to decide
a) if this is the best approach
b) if you want to generate the build files as a separate commit, on publish, etc.
@ro-ka While I remember - I don't think your dist directory is included in your npm publish at the moment; when I do an npm install and check the output react-geosuggest directory, it's missing.

Thanks for the update! I will review this later – still thinking about switching to pure npm based tasks.

Removed gulp and switched to npm scripts.


Problem on ref in typescript

Check that this is really a bug

[X] I confirm

Reproduction link
https://codesandbox.io/s/swiper-default-react-forked-scddn?file=/src/App.tsx
Bug description
i want to use ref in typescript but i got this error on IDE and build time
this is my code:
const swiperRef = useRef() as React.RefObject<any> <Swiper ref={swiperRef} className="mySwiper">
this is error message:
Type '{ children: Element[]; ref: RefObject<any>; className: string; }' is not assignable to type 'IntrinsicAttributes & SwiperProps & { children?: ReactNode; }'.   Property 'ref' does not exist on type 'IntrinsicAttributes & SwiperProps & { children?: ReactNode; }'
i'm not sure i'm doing something wrong or this is a bug.
Expected Behavior
No response
Actual Behavior
No response
Swiper version
7.3.1
Platform/Target and Browser Versions
chrome
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.
[X] Make sure this is a Swiper issue and not a framework-specific issue

Would you like to open a PR for this bug?

[ ] I'm willing to open a PR

Just came across the same issue and my current hack is to do the following:
  const swiper = useRef() as any;

<Swiper
          onInit={(core: SwiperCore) => {
            swiper.current = core.el
          }}
/>

Just came across the same issue and my current hack is to do the following:
  const swiper = useRef() as any;

<Swiper
          onInit={(core: SwiperCore) => {
            swiper.current = core.el
          }}
/>

Thanks
I'm doing this right now but I think this is not the right way

Yup its for sure just a workaround hack to get the ref. I haven't been able to find a solution for working with the ref in another way

you could create your own union type using:
const RefSwiper: React.FunctionComponent<
  SwiperProps & RefAttributes<SwiperCore>
> = Swiper;

@farhadf246
@nolimits4web
i think this issue can be closed

Where does SwiperCore comes from? It is not exposed by swiper/react

@KevinKreps it is exported from swiper package itself not swiper/react

the correct type for Swiper instance is SwiperClass. So just do:
import SwiperClass from 'swiper'
const swiperRef = useRef<SwiperClass>()

Hi! Looking to solve this issue through this PR: https://github.com/nolimits4web/swiper/pull/6186

Is there a solution on this case? I tried almost all the solutions but non of them worked. I'm working on create-react-app --template typescript. When I was working without --template typescript there was no problem like this.

Am currently having same issue using this library. am working on react project with typescript. non of the solution suggested here seem to workout for me.

import { Swiper, SwiperSlide } from 'swiper/react';
import SwiperCore from 'swiper';

const swiperRef = useRef<SwiperCore>();

      <Swiper
        onSwiper={(swiper) => {
          swiperRef.current = swiper;
        }}
      >
...
</Swiper>

I solved the typescript issue by using SwiperRef type imported from swiper/react and use swiper functions by accessing them from current.swiper like this:
import { Swiper, SwiperRef, SwiperSlide } from "swiper/react";

const swiperRef = useRef<SwiperRef>(null);

const slideNext = (): void => swiperRef?.current?.swiper.slideNext();

<Swiper ref={swiperRef}

I solved the typescript issue by using SwiperRef type imported from swiper/react and use swiper functions by accessing them from current.swiper like this:
import { Swiper, SwiperRef, SwiperSlide } from "swiper/react";

const swiperRef = useRef<SwiperRef>(null);

const slideNext = (): void => swiperRef?.current?.swiper.slideNext();

<Swiper ref={swiperRef}

Thanks this solution worked for me

Just came across the same issue and my current hack is to do the following:
  const swiper = useRef() as any;

<Swiper
          onInit={(core: SwiperCore) => {
            swiper.current = core.el
          }}
/>

I tried it, It's work


Add support for go modules

Hi @muesli,
first of all, thank you for this great package. It eases working with systemd and their unit system a lot for users coming from init.
While building this project I've created go.mod and go.sum files to manage the dependencies, to avoid errors like:
# github.com/coreos/go-systemd/dbus
/home/pyrox777/go/pkg/mod/github.com/coreos/go-systemd@v0.0.0-20190321100706-95778dfbb74e/dbus/dbus.go:127:33: cannot use dbus.SystemBusPrivate (type func() (*dbus.Conn, error)) as type func(...<T>) (*dbus.Conn, error) in argument to dbusAuthHelloConnection
/home/pyrox777/go/pkg/mod/github.com/coreos/go-systemd@v0.0.0-20190321100706-95778dfbb74e/dbus/dbus.go:136:33: cannot use dbus.SessionBusPrivate (type func() (*dbus.Conn, error)) as type func(...<T>) (*dbus.Conn, error) in argument to dbusAuthHelloConnection
/home/pyrox777/go/pkg/mod/github.com/coreos/go-systemd@v0.0.0-20190321100706-95778dfbb74e/dbus/dbus.go:146:42: undefined: dbus.ConnOption
/home/pyrox777/go/pkg/mod/github.com/coreos/go-systemd@v0.0.0-20190321100706-95778dfbb74e/dbus/dbus.go:204:48: undefined: dbus.ConnOption
/home/pyrox777/go/pkg/mod/github.com/coreos/go-systemd@v0.0.0-20190321100706-95778dfbb74e/dbus/dbus.go:224:53: undefined: dbus.ConnOption

If you would like to, you can use the associated PR to have a known-good starting point for other users trying to build this project.

Thank you for your contribution!


refactor of stream typeahead order

Changed algorithm for sorthing the streams type ahead suggestions
Fixes: https://github.com/zulip/zulip/issues/20618
I encountered an issue where sometimes an unmuted unsubscribed stream would be suggested ahead of a subscrubed muted stream. To solve that issue I gave all subscribed streams a +2 to their score. Every other case will only give them +1. This way the typeahead will always suggest subscribed streams over unsubscribed streams.

**Self-review checklist**

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

Communicate decisions, questions, and potential concerns.

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

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

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

Completed manual review and testing of the following:

[x] Visual appearance of the changes.

[x] Responsiveness and internationalization.

[x] End-to-end functionality of buttons, interactions and flows.

[x] Corner cases, error conditions, and easily imagined bugs.

@barrysnow1 could you comment on how this PR differs from #20823? Thanks!

Closing in favor of https://github.com/zulip/zulip/pull/20823, which has had a more active contributor.


HistomicsStream sometimes freezes

@cooperlab says: On cooperlabgpu5, you can access my ~/histomics_stream.ipynb. The slide TCGA-A2-A0T1-01Z-00-DX1.CD1E9C46-18A3-466B-AA0B-24F3055FA851.svs is located there. This is a large 40X magnification slide.

@cooperlab says: On cooperlabgpu5, you can access my ~/histomics_stream.ipynb. The slide TCGA-A2-A0T1-01Z-00-DX1.CD1E9C46-18A3-466B-AA0B-24F3055FA851.svs is located there. This is a large 40X magnification slide.

Closed by #106.

Re-opening pending the merging of Pull Request #106.


Character Level CNN based features concatenation with Word Embeddings

I am trying to implement the Character CNN + LSTM model presented in this paper (http://arxiv.org/abs/1511.08308) for Named Entity Recognition using Keras. However, I am facing issue with including the Character based CNN features along with the word embedding for the model.
The basic code for my model using Bidirectional LSTM using Keras Graph model is presented here: https://github.com/napsternxg/DeepSequenceClassification
However, for each word I also want to add CNN based features on the character embedding to the model input. I know I can do this by creating a new input node and adding a CNN layer to it and then merging the Convolution + Pooling output with the Embedding features, but I am facing issues with defining the model input. For each word in the sequence I will have a sequence of characters which I can pad to make it say max_word_char_size long, but this will make the input layer consist of 3 dimensions (samples, maxlen, max_word_char_size) then this layer will pass through an character embedding layer which will output a layer of size (samples, maxlen, max_word_char_size, char_embedding_size) and I am not sure how to go about doing this.
I have come up with the following hack using different dim_ordering modes and Reshape layers to ensure I get 1 vector per word in each sequence. These vectors will then be concatenated with the embedding of each word.
import theano, keras
from keras.models import Sequential, Graph
from keras.layers.core import Dense, Dropout, Activation, TimeDistributedDense, Flatten, Merge, Reshape
from keras.layers.embeddings import Embedding
from keras.layers.recurrent import LSTM, GRU
from keras.layers.convolutional import Convolution1D, MaxPooling1D, Convolution2D, MaxPooling2D
from keras.preprocessing.sequence import pad_sequences

import numpy as np

max_len, max_char_len = 5, 4
char_vocab = 100
char_embedding_size=50
nb_filters = 10
# Sequence words represented as sequence of sequence of characters
X_c = np.array([[[0,1,2,3], [0,1,2,3],[0,1,2,3],[0,1,2,3], [0,1,2,3]],
               [[0,1,2,3], [0,1,2,3],[0,1,2,3],[0,1,2,3], [0,1,2,3]]])
# corresponding sequence of words
X_w = np.array([[1,1,1,1,1], [1,1,1,1,1]])
print X_c.shape, X_w.shape
# Output: (2, 5, 4) (2, 5)

model = Sequential()
model.add(Embedding(char_vocab, char_embedding_size, input_length=max_len*max_char_len))
model.add(Reshape((max_len, max_char_len, char_embedding_size)))
model.add(Convolution2D(nb_filters, 1, 2, dim_ordering='tf', border_mode='same')) # Hack to ensure I get output per word
model.add(MaxPooling2D((2, 2), dim_ordering='th')) # Hack to get output per word
model.add(Reshape((max_len, 10)))

model.compile(optimizer='adam', loss='mse')

input = X_c.reshape(2,max_len*max_char_len) # Hack to pass all chars to the embedding layer.
output = model.predict(input)
print output.shape
# Outputs: (2, 5, 10)

I look forward to your feedback if this is the correct way to go about it or if there is any other better way to do it.

I have implemented a basic version of it at https://github.com/napsternxg/DeepSequenceClassification in the model.py file and am able to run it. I will close this issue as of now and if anyone can suggest a better way then feel free to comment on it.

super
thank a lot
On Tue, Jan 5, 2016 at 4:17 PM, Shubhanshu Mishra<EMAIL_ADDRESS>wrote:

I have implemented a basic version of it at
https://github.com/napsternxg/DeepSequenceClassification in the model.py
file and am able to run it. I will close this issue as of now and if anyone
can suggest a better way then feel free to comment on it.
—
Reply to this email directly or view it on GitHub
https://github.com/fchollet/keras/issues/1400#issuecomment-169136425.

Nice, Thanks a lot!

@napsternxg Thanks for sharing. For word embedding, are we able to used pre-trained word vectors (from word2vec, e.g.), and how?

Thank for this wonderful post.
Suppose, i have an input string say "cu 2morr, gf" and the expected output is "see you tomorrow, girlfriend".  The corresponding sequence of the input is my expected output. I am very confused on how i should validate the model or predict the output. Please help.
Abbey

@dupsys your problem is probably an instance of seq2seq prediction where the input sequence is "cu 2morr, gf" and the output sequence is "see you tomorrow, girlfriend". In this case the input and output seq lengths will be different, also you might want to use an attention based model (used mostly in machine translation literature).
@ShuaiW right now my code doesn't support this, but you can edit the embedding layer to take the pre-trained embeddings.

Thank you napstenxg. Will look at the implementation of attention model

Hi napsternxg,
Please, can you give me a hint with the attached code segment. I am trying to implement Tweet2Vec from the scratch and add attention model on it.
codesegment.txt
I got this error:
Error when checking model input: expected input_1 to have 2 dimensions, but got an array with shape (307551, 180, 68).

Hi @napsternxg
this is a great feature, thanks for implementing it!
I am trying to understand your code - but I am quite confused about how you cope with padding.
You pad/cut you sentences to maxlen_sent AND all words inside of the sentence to maxlen_char, isn't it?
But you are not padding in the embedding layer (with mask_zero= True), I guess because it would not work with the CNN layer.
You are also not using weights, or do I understand something wrongly?
So how can you ensure padding not to interfere while backprop/calculating the metrics?
Thank you in advance for your answer :)

@napsternxg have you passed number of words as number of channels in CNN?

Thank you napstenxg,
It's great !!
Do you find a way to implement the CRF ?

Hi @napsternxg! Thanks for your great work!
There is a thing i don't understand very well: for the input to the character embedding layer, do you have to padd any word with a fixed length?
Thanks


[Enterprise Bot Template] Several QnA Maker instances created while it can be hosted in one?

In the Enterprise Bot Template I see that now several instances of Cognitive Services and QnA Maker are provisioned when running msbot clone services, including one Azure Search (Standard) instance per QnA Maker instance.
I believe this would not be needed if you use one QnA maker instance with multiple knowledge bases rather than multiple QnA maker instances.
Is there a rationale behind this choice? The costs of having multiple Azure Search (Standard) instances are high, and I do not believe those costs are justified well - does chitchat functionality really need to be served by a separate instance???

ARM template deployment is coming soon.


[FEATURE] Pin Columns to the Filters Pane

https://sharepointmaven.com/how-to-add-custom-metadata-columns-to-the-filters-pane-in-sharepoint-online/
Refer to step 5 of this webpage.
I'm not sure if it's already an existing feature, but I can't find it on google search.
Can add the [Pin Columns to the Filters Pane] function?

Hi @hunandy14, this has been added in PR https://github.com/pnp/powershell/pull/2632. Once merged you can use Set-PnPField -Identity "Internal name of field" -ShowInFiltersPane <Auto\Removed\Pinned> to set it to the desired value.

Thank you very much, I will update the version tomorrow to try this function

We need @gautamdsheth to buddy check my changes and merge the PR first :) Once that is done, after the first 2.30 AM Central European Time after the PR is merged, it will be available in the nightly build.


[ALLUXIO-2167] Add a debug message in SwiftUnderFileSystem.java::list

https://alluxio.atlassian.net/browse/ALLUXIO-2167

Can one of the admins verify this patch?

ok to test

LGTM, thank you for your contribution to Alluxio!

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/11046/
Test PASSed.


fix dataflow bug when cutting layout key

This doesn't seem to do the right thing...
zed compile -C -P 2 "from 'pool-ts' | cut x=ts,ts=1"

which gives...
from (
  G2eDzBUfU6IEmUSGCa5kHyXMhoO =>
    cut x=ts,ts=1;
  G2eDzBUfU6IEmUSGCa5kHyXMhoO =>
    cut x=ts,ts=1;
)
| merge ts

The merge here should be on x.  I believe this bug was in the original parallelization code so we will put the hazmat PR up and fix this later.

I've verified that as of Zed commit 874ac74 this is now showing a merge on x as expected. The updated Zed language syntax also now requires := to be used for assignment.
$ zed -version
Version: v0.29.0-358-g874ac748

$ zed compile -C -P 2 "from 'pool-ts' | cut x:=ts,ts:=1"
from (
  G2eDzBUfU6IEmUSGCa5kHyXMhoO =>
    cut x:=ts,ts:=1;
  G2eDzBUfU6IEmUSGCa5kHyXMhoO =>
    cut x:=ts,ts:=1;
)
| merge x:asc

Thanks @mccanne!


searchClass and sortClass  .. actual element support along with classes

Due to my design,
my search input and my sort buttons are not inside main div.
It will be very helpful if I can pass and element as selector as well or
searchClass  could search inside main div if cant find it then it should try to find in global scope and get first element as search input.

I've just added a FAQ and included that question: http://listjs.com/faq/#search-outside 🎉