add new keys to the ocs-operator-config configmap

Signed-off-by: Nitin Goyal<EMAIL_ADDRESS>

/lgtm

/hold

Per conversation elsewhere, this is a temporary fix to unblock the CI due to a bug introduced with deploying rook-ceph-operator via operator bundle. The correct implementation of this is coming with PR #2475 (specifically here).
/hold cancel

/retest ci/prow/ocs-operator-bundle-e2e-aws

/test ocs-operator-bundle-e2e-aws

/test ocs-operator-bundle-e2e-aws

/hold

It is not required as #2475 was merged before this.


swaggerToCSharpClient, custom JsonInheritanceConverter and underscorein the object type

When the object type in the swagger file contains an underscore, the c# class name is created without an underscore, but the descriminator value is still using an underscore in the name.
I've added a quick fix in my local version (and also raise an error and do a case-insensitive search in the GetType(.., true,true)
var typeName = objectType.Namespace + "." + discriminator.Replace("_", string.Empty);
return System.Reflection.IntrospectionExtensions.GetTypeInfo(objectType).Assembly.GetType(typeName, true, true);

Can you create a PR with this fix so that I can review and merge it (it shouldn't break existing code)?

Confirmed this is still happening with version 11.17
If you C# property uses a JsonProperty attribute with an underscore, the value of that parameter is lost in the request.

Can you clarify?


Upgrade django and lxml in the dependency

As warned by tools/gae:
WARNING appinfo.py:1651 django version "1.9" is deprecated, use one of: "1.4", "1.11"
WARNING appinfo.py:1651 lxml version "2.3" is deprecated, use one of: "3.7.3"

Forgot to reference the issue from the PR, but lxml was upgraded to 3.7.3 in PR #512.

Upgraded Django to 1.11 in PR #613.


configuration/network_filters/tcp_proxy_filter.md

原文：https://github.com/servicemesher/envoy/tree/master/configuration/network_filters/tcp_proxy_filter.md

translate.


Sound.stop does not exist

via https://groups.google.com/forum/#!topic/gamejs/wFmkvKyiC6g
The Sound.stop() doesn't work as described in API.

function muteSound(sound) {
sound.stop();
}
bgMusic = new gamejs.audio.Sound(document.getElementById("bgMusic"));
muteSound(bgMusic);
So, it cause an error "audio.stop is not a function".
The solution is to use standard JavaScript methods.
bgMusic = document.getElementById("bgMusic");
muteSound(bgMusic);
function muteSound(sound) {
sound.pause();
}

fixed https://github.com/GameJs/gamejs/commit/3c279431da58e7e127f05a21272c14172b528aa8

Cool thanks, I got one more question. In my game I got multiple sounds played at almost the same time: fire and explosion (fast rocket launcher or machinegun). On ubuntu (using chrome) I got all the sounds queued so even if I stop firing, the sound for shot and for explode is still playing. On Firefox game crashes after Your last update (when stop() method is called, exaclty here: audio.currentTime = 0;) with:
InvalidStateError: An attempt was made to use an object that is not, or is no longer, usable


Release New Version Number

Can we update the version number and push it to npm, now that the chat color and chat emoji changes have been implemented?

This feature is still in development (see #124 and #198).  Feel free to sync to master if you want the latest features.

npm is updated.


Performance issue with font css file link

The style sheet files below seems is hosted on an unstable server, which may cause the web page taking more than 2min to load.
<link href="https://fonts.loli.net/css?family=Roboto:300,300italic,400,400italic,500,500italic" rel="stylesheet">
<link href="https://fonts.loli.net/css?family=Material+Icons">

By "unstable", I mean error code 500 seems to happen a lot.
BTW, after delete those code, there seems to be no problem on web page rendering.

This part of the code is related to fonts and icons. You can view the commit record to localize it.
这部分代码和字体和图标有关，你可以查看 commit 记录将其本地化


Provisioning without "pop" (MEGH-4873)

Answers checklist.

[X] I have read the Rainmaker documentation and the issue is not addressed there.
[X] I have updated my IDF branch (release/vX.Y) to the latest version and checked that the issue is present there. This is not applicable if you are using Rainmaker with Arduino.
[X] I have searched the Rainmaker forum and issue tracker for a similar issue and not found a similar issue.

General issue report
While provisioning over ble or wifi, rainmaker app asks for "proof of possesion key".
Is there a way to provision without proof of possesion key?
How to disable pop in arduino ide.
Would be really helpful.

Even though we recommend to not use it, considering the security implications, you can pass POP_TYPE_NONE to app_wifi_start for provisioning without PoP.

"POP_TYPE_NONE" results in undeclared error during compulation in arduino ide.
It works only with vscode / esp-idf.
Anyway of doing it with arduino ide?
Also nova home app does not discover devices over ble or softap, need to manually scan qrcode whereas rainmaker app has no problem in discovering devices on any method.

For Arduino, you can simply pass NULL as the pop to  WiFiProv.beginProvision call.
If the Nova Home app issue is the same as in https://github.com/espressif/esp-rainmaker/issues/277, we will check it and answer there separately.

Passing NULL as the pop to WiFiProv.beginProvision  call worked. Thanks.
Nova Home app issue is same as #277.
Is there a way to auto enter "pop" details over ble and wifi as it happens with scanning qrcode since it will increase the user experience and remove the hassle of finding the "pop" details in an event when "pop" is unavailable.

The QR code has the name, PoP (Proof of Possession pin) and transport info (BLE/SoftAP) which is the used to establish a secure session with the device for provisioning and that is akin to an "Auto enter". If the PoP is not available and yet you want to proceed for "better user experience" , it defeats the purpose of having a PoP in the first place. Having no PoP is the only option then.

Understood.
Thanks.


Release script doesn't update website

Version: 0.10.0
Module: quill-jdbc
Expected behavior
Example taken from official docs site should compile:
  case class Contact(personId: Int, phone: String)
  val a = quote(query[Contact].insert)
  ctx.run(a)(List(Contact(999, "+1510488988")))

Actual behavior
Compile-time error is thrown:
ambiguous reference to overloaded definition,
[error] both macro method insert in trait EntityQuery of type (value: malibu.Main.Contact)malibu.Main.ctx.Insert[malibu.Main.Contact]
[error] and  method insert in trait EntityQuery of type (f: malibu.Main.Contact => (Any, Any), f2: malibu.Main.Contact => (Any, Any)*)malibu.Main.ctx.Insert[malibu.Main.Contact]
[error] match expected type ?
[error]   val a = quote(query[Contact].insert)

@getquill/maintainers

@Rogach thanks for reporting this, the website was still on 0.8.0. I've fixed it manually and will leave this issue open to investigate why the release script didn't update the website.
Take a look at the documentation again, the insert method now always receive a parameter.

Thank you! Yes, I already read the docs on github and fixed my code.

@fwbrasil It seems now we don't have issues with website updates. Should we close this?


merge MONDO_0018708 'squamous cell carcinoma of the oral tongue' with MONDO_0000500 'tongue squamous cell carcinoma'

consider merge MONDO_0018708 'squamous cell carcinoma of the oral tongue' with MONDO_0000500 'tongue squamous cell carcinoma' (parent class)

Let's keep these separate: https://www.webmd.com/cancer/tongue-cancer-facts#1
Uberon has:
id: UBERON:0010032
name: anterior part of tongue
def: "The portion of the tongue in front of the terminal sulcus. At the apex, thin and narrow, it is directed forward against the lingual surfaces of the lower incisor teeth. It is derived primarily from the first pharyngeal arch." [Wikipedia:Anterior_tongue]
xref: Wikipedia:Anterior_tongue
property_value: development_notes "The lateral lingual swellings grow and fuse with eachother, encompassing the tuberculum impar to provide the ectodermally derived mucosa of the anterior 2/3 of the tongue" xsd:string
property_value: external_ontology_notes "check NCIT" xsd:string {external_ontology="NCIT"}
xref: FMA:229084 ! anterior part of tongue
xref: NCIT:C32130 ! Apex of the Tongue
synonym: "anterior 2/3 of the tongue" RELATED [Wikipedia:Anterior_tongue]
synonym: "anterior 2/3 of tongue" RELATED [Wikipedia:Anterior_tongue]
synonym: "anterior two thirds of the tongue" RELATED [Wikipedia:Anterior_tongue]
synonym: "apex linguae" RELATED [Wikipedia:Anterior_tongue]
synonym: "apex of the tongue" RELATED [Wikipedia:Anterior_tongue]
synonym: "buccal part of the tongue" RELATED [Wikipedia:Anterior_tongue]
synonym: "front of the tongue" RELATED [Wikipedia:Anterior_tongue]
synonym: "oral part of the tongue" RELATED [Wikipedia:Anterior_tongue]
intersection_of: UBERON:0000064 ! organ part
intersection_of: part_of UBERON:0001723 ! tongue
intersection_of: develops_from UBERON:0006260 {source="ISBN10:1607950324"} ! lingual swellings
synonym: "pars anterior dorsi linguae" RELATED LATIN [Wikipedia:Anterior_tongue]
property_value: http://xmlns.com/foaf/0.1/depicted_by http://upload.wikimedia.org/wikipedia/commons/4/4c/Illu04_tongue.jpg
xref: UMLS:C0226946 {source="ncithesaurus:Apex_of_the_Tongue"}
disjoint_from: UBERON:0010033 {source="lexical"} ! posterior part of tongue
xref: EMAPA:37437 {source="MA:th"} ! tongue anterior part
But FMA also has a separate class for "oral part of tongue". I'm not sure how much a strict division matters here

See also:
https://rarediseases.org/rare-diseases/tongue-cancer/
Tongue cancers are oral cancers that are differentiated by their location in the mouth and on the tongue ... The characteristics of these two cancers are quite distinct, and reflect the differences in their origins
But note if we do keep we need to keep it's sib

action item:

[ ] remove comment

But note if we do keep we need to keep it's sib

MONDO_0018708 'squamous cell carcinoma of the oral tongue' does not have a sibling


Account.getTransactionHistory():  Dash  sent to myself (.getUnusedAddress().address)) show as received instead of address_transfer

wallet-lib:
Account.getTransactionHistory():  Dash  sent to myself (.getUnusedAddress().address)) show as received instead of address_transfer

please provide some more details - package version, dashmate (local) / testnet etc. thanks

retest passed<EMAIL_ADDRESS>


Bad URL in stacktrace if urlRoot specified

Expected behaviour
Correct URL is printed in stacktrace
Actual behaviour
Bad URL is printed in stacktrace
Environment Details
// package.json
{
  "devDependencies": {
    "karma": "^1.7.1",
    "karma-chrome-launcher": "^2.2.0",
    "karma-jasmine": "^1.1.0",
    "jasmine-core": "^2.8.0"
  }
}

//karma.conf.js
module.exports = function (config) {
  config.set({
    urlRoot: '/__karma__',
    frameworks: ['jasmine'],
    files: ['tests/**/*.js'],
    reporters: ['progress'],
    colors: true,
    logLevel: config.LOG_INFO,
    autoWatch: true,
    browsers: ['Chrome'],
    singleRun: false
  });
};

// tests/sub/foo.js
describe('foo', function () {
    it('test', function () {
        fail();
    });
});

Steps to reproduce the behaviour

Run tests with ./node_modules/karma/bin/karma start
Output:

9 11 2017 19:49:40.487:WARN [config]: urlRoot normalized to "/__karma__/"
29 11 2017 19:49:40.540:WARN [karma]: No captured browser, open http://localhost:9876/__karma__/
29 11 2017 19:49:40.546:INFO [karma]: Karma v1.7.1 server started at http://<IP_ADDRESS>:9876/__karma__/
29 11 2017 19:49:40.546:INFO [launcher]: Launching browser Chrome with unlimited concurrency
29 11 2017 19:49:40.560:INFO [launcher]: Starting browser Chrome
29 11 2017 19:49:40.980:INFO [Chrome 62.0.3202 (Linux 0.0.0)]: Connected on socket 3ny3gbgd6AeYqWRDAAAA with id 79121705
Chrome 62.0.3202 (Linux 0.0.0) foo test FAILED
        Failed
            at UserContext.<anonymous> (http://localhost:9876/__karma__tests/sub/foo.js:3:9)
Chrome 62.0.3202 (Linux 0.0.0): Executed 1 of 1 (1 FAILED) ERROR (0.003 secs / 0.003 secs)

Unexpected output part:
  at UserContext.<anonymous> (http://localhost:9876/__karma__tests/sub/foo.js:3:9)

Expected:
  at UserContext.<anonymous> (tests/sub/foo.js:3:9)

The issue came from https://youtrack.jetbrains.com/issue/WEB-29940

Looks like urlRoot should be taken into account when composing URL_REGEXP: https://github.com/karma-runner/karma/blob/v1.7.1/lib/reporter.js#L28

I tried to fix it.


FR - have renamer_cleanarr support multiple assets dir

I use renamer to copy posters to an output dir and then run fix_border and have that copy the posters to the PMM assests dir.
To clean them both, I now have 2 renamer cleanarr scripts and config  entries.
It would be nice if we could combine this into one.
My current setup works, so no rush, just wanted to drop this here.

Thanks for the suggestion. That's a fairly easy fix. I'll put it on my todo list


Getter Setter CodeActions provides code actions when getter and setter are already defined

Description:
Getter Setter CodeActions provides code actions when getter and setter already defined
Steps to reproduce:
consider following ocde example
class Person {
    private int age = 5;

    public function getAge() returns int {
        return self.age;
    }

    public function setAge(int  age) {
        self.age = age;
    }
}

Follow below steps

edit field private int age = 5 to private int age
perform getter setter codeaction again

Here the issue is .text() is missing in the getter setter code action implementation. Need to add .text() as follows in code actions.
String fieldName = String.valueOf(objectFieldNode.fieldName().text());


Anuglar

Hello, what is the story for angular2 here are you planning to upgrade or stay on 1.x?

Hi!
I've some ideas to switch from bootstrap to material design including angular2, but currently the backend is more important, so there won't be any radical changes to the frontend by me any time soon.


Bump kafka_protocol to 2.2.5

latest version has been bumped to 2.2.7 :)

Let me merge this one first, and I'll update to 2.2.7 in another branch. I actually forgot that I already had this one pending. :)


Can I overwrite function name for the swagger ui?

Hi.
I have a question about if I can overwrite the function name to add '/' separator. By default the extension take the function name as the title.
I tried to change the code but it won't allow me that function name with '/'. I want the function route as the title.

Not sure how "correct" this is but you can put '[ApiExplorerSettings(GroupName = "exampleName")]' on the method to rename it.

Closing this, as not relevant, please reopen if needed


LSP: Incorrect Content-Length for dart/textDocument/publishFlutterOutline response

I've encountered a potential issue with the dart/textDocument/publishFlutterOutline method where the Content-Length specified in the header doesn't match the actual content length. This mismatch is causing my LSP stream parser to crash at the JSON decoder due to an extra 2 characters.
Steps to reproduce:

Receive data from the dart/textDocument/publishFlutterOutline method.
Parse the Content-Length from the header.
Check the length of the content based on the header.
Compare the parsed Content-Length with the actual length of the content.

Observed behavior:
For the attached IO dump, the specified Content-Length in the header was 8928. However, upon examining the content, its length was found to be 8926, leading to a difference of 2 characters.
lsp_bug_quinine.log
Expected behavior:
The Content-Length specified in the header should match the actual content length.
I've attached a snippet of the string buffer during the time of the issue for further insight.
You can observe that the header's Content-Length and the actual length of the content differ by 2 characters.
I believe there might be a discrepancy in how the Content-Length is being computed or set for this specific method. It would be great if you could take a look.
Thanks for your assistance!

Dart SDK version: 3.1.0 (stable) (Tue Aug 15 21:33:36 2023 +0000) on "macos_arm64"

Here is my LSP client parser:
5hirish/quinine/lib/services/lsp/buffer.dart#L59-L70

I will try to reproduce this in a smaller project.

My guess is the file URI format is causing this issue.
My LSP client creates URI like this -
file:///Volumes/Macintosh%20HD/Users/felix/StudioProjects/tonic-ide/lib/main.dart
Where as VSCode -
file::///Users/felix/StudioProjects/tonic-ide/lib/main.dart
in the key uri could it be the white space being encoded messes up the content length?

I think I can reproduce this - taking a look. I don't think it's the %20 as if the length was treated that as decoded, the content-length would be 2 characters less, but it seems like it's 2 characters more.
It's possible there's a trailing \r\n included in the payload that I'm not logging (in which case it might add up), but I don't think that would explain your issue.

Ok, I found the difference here is in the number of bytes vs the number of characters. If I log message.length and message.codeUnits.length I see:
Length:: 8916
Code Units:: 8914

The LSP spec describes content-length as:

The length of the content part in bytes. This header is required.

So my feeling is that the server is doing the right thing. Looking at your code, I think you are using content-length to read numbers of characters (by using substring):
var rawMessage = bufferString.substring(messageStart, messageLength);

Whereas in the servers code, we keep the buffer as raw bytes until we have the correct length and then decode using utf8:
https://github.com/dart-lang/sdk/blob/4d4a922638cda394c59605936a56874f75819b9e/pkg/analysis_server/lib/src/lsp/lsp_packet_transformer.dart#L51-L58
So I think if you change your code to keep the buffer of bytes and take a slice of that once you have the length, that will solve the issue (it might also prevent some other issues like this which I had in VS Code when multi-byte characters were split by the buffering in stdout 🙃).

Oh let me try switching to a bytes buffer and see what's up. Thanks for explaining this.

It looks like this is WAI. Feel free to re-open if you feel otherwise?  Thanks for the thoughtful replies @DanTup!

@DanTup just switched to bytes buffer, works like a charm. Thanks a bunch!

Excellent, thanks for confirming!


Container-fluid inside container issue

http://getbootstrap.com/css/#overview-container It says containers shouldn't be nested here. I see it's difficult to avoid for radix_layouts. The workaround in _layout.scss hardcodes -15px margins to offset the padding. I'm using variables to set the gutter width so this breaks things and throws the layout off.
Would the ideal solution not be to take containers out of radix_layouts and leave it to the theme to add the necessary containers? Or could the bootstrap variable be factored in instead?

In previous versions of Radix, this is how it used to be. No container-wrapping in layouts.
But we moved it inside a few months ago. See the issue thread here: https://www.drupal.org/node/2334871
Another workaround this is to use a css fix as follows:
.container .container {
    width: auto;
}


new commit

Alright, based on the given Coding Standards Document, here's the review of the diff:

Variable Renaming: WrapperID is renamed to a , app is renamed to b, songInfoList is renamed to SONGinfoLIST and the loop counter i is renamed to j. As per the coding standards, variable names should be descriptive and single letter variable names should be avoided except for loop counters. Hence, this breaks the standards.

Another variation is the change from document.getElementById(WrapperID) to document.getElementById(Wrapper-ID). It seems like it is trying to refer the variable a previously renamed. However, the hyphen may cause an issue as it might be interpreted as a minus operation.

In the slide function, there are changes made in the formula calculations such as slider.clientWidth -> slider.client-Width. This change may lead to an error as it is trying to subtract Width from client object which may not exist.

In conclusion, the revisions provided in the diff unfortunately do not follow the standards outlined in Coding Standards Document, particularly in terms of variable naming and apparent introduction of potential syntax errors.
Please verify the above changes and correct them as per the coding standards document to maintain the quality of the source code.


All steps pass but returns with "errored after 14 min" error.

I am running my test case using Gulp, Webdriver IO, Cucumber JS.
All my steps are passing but only this (check attached image)

@kapilchokhawala do you have a reproducible example?


Minify css file, add it to project

Added a minified CSS file to the project. More smol!

Sorry for the late reply!
Github pages has http compression enabled, which results in minification not always being that effective, e.g. from accessing the raw files from github:

x.css: 1.13KB transferred (compressed with headers), file size: ~590B
x.min.css: 1.10KB transferred (compressed with headers), file size: ~400B

so while the filesize savings are over 20% (190B), the actual savings are ~30B
And while a resource is under ~700B, it should ideally fit into a single TCP packet (~1400B total) without issue, so saving size on the css file can not speed up the transfer of the website

So I'm probably going to close this pull request, sorry! The work of keeping a CSS file in sync with its minified version would be a bit much (not that it isn't something worth looking into, but I would prefer it as a build step or similar (through netlify or some other static site service))
Thank you for your time, however! :smiley:


Assignment 1

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
using bash to add/edit/remove/copy directories, files
What did you learn from the changes you have made?
bash code
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Nope
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
yes. not familiar with the code. Overcome by searching online regarding the usage of the codes
How were these changes tested?
At visual studio code
A reference to a related issue in your repository (if applicable)
@mentions of the person or team responsible for reviewing proposed changes (At least 2 people)

@dtxe
Checklist

[V ] I can confirm that my changes are working as intended

Hi there, glad to see you have began so early and soon on your assignment! That being said, you have unfortunately made a pull request to the wrong repo. You shouldn't be making a pull request to uoft-dsi/shell, but rather your own forked shell repo.
Please read the instructions on your assignment (Submission Information) more carefully (specifically the Assignment Submission Guide)

Hi Daniel,
Thank you for your reminder. I have tried to submit my assignment again
according to the submission guide. May I have your help to double check if
I have submitted it correctly this time?
Thank you very much.
Moniz
On Wed, 24 Apr 2024 at 09:12, Daniel Razavi @.***>
wrote:

Hi there, glad to see you have began so early and soon on your assignment!
That being said, you have unfortunately made a pull request to the wrong
repo. You shouldn't be making a pull request to uoft-dsi/shell, but rather
your own forked shell repo.
Please read the instructions on your assignment (Submission Information)
https://github.com/UofT-DSI/shell/blob/main/02_assignments/assignment.md#submission-information
more carefully (specifically the Assignment Submission Guide
https://github.com/UofT-DSI/onboarding/blob/main/onboarding_documents/submissions.md
)
—
Reply to this email directly, view it on GitHub
https://github.com/UofT-DSI/shell/pull/33#issuecomment-2074917947, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/BHXLA2IXZW45ZTDA4CKOJMLY66VUNAVCNFSM6AAAAABGWBLVJKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZUHEYTOOJUG4
.
You are receiving this because you authored the thread.Message ID:
@.***>


Update RGB templates repository

Task to do
Update template_rgb_plot within the templates repository to reflect the recently updated AgPypeline template_rgb_plot repo. Also ensure that documentation is up to date and get rid of unnecessary files
Reason
Increase usability of the templates repo
Result
Repository will be cleaner and updated
This differs from template_rgb_plot at https://github.com/AgPipeline/template-rgb-plot because it uses a separate templates folder to run tests rather than doing it all through the separate algorithm_rgb.py and testing.py files. The idea at the moment is to create a versatile testing file that can handle multiple types of image data
The base files contained in this repository and template_rgb_plot are the same, but the reason that this repo is different is because of the testing folder

Just to be clear, the testing file is for users to test their algorithms out with?
Would it be better if the testing code was in a library and the testing.py file (the file doesn't need to have that name) makes use of the library and handles any differences between what's in the library and the template code? For example, loading RGB data instead of Lidar.
If this isn't done, won't the testing.py file need to contain ways to load every different type of
supported data file, even when it's not needed? For example, the RGB testing file would now need to support lidar as well, even if the developer of the RGB algorithm never uses Lidar?

I think I see. Would that mean having a testing.py script as a library in and of itself and then having templates import that library and then use it. The templates could have their own testing.py files if needed/wanted for extra methods, but could use the library one for the default testing

I don't think the testing scripts should in a library. I think that there should be more than one script to handle the differences between Image data and Lidar data

Moved to Ready pipeline
@jvanderleeuw Can you comment on the status of this?

Issue closed due to vagueness and overlap with #87, #42, and #44


Add brief clarification question numbers

#115944077
Numbers are needed in the buyer and public view of the clarification questions so that later questions can easily refer to earlier questions.
The fact that summary tables are done with macros makes it more difficult to just use Jinja2's loop.counter variable. The simplest way of getting a counter actually seems to be to pass it in to the template. While this number could be generated in the API it is just for display, ideally it would be generated in the view but that is not easy because of how we use macros.

We could add the index to the call(question, index) in the summary macro later, but that's quite a bit more work, so this is good :+1:

Yes, I looked at doing that. It would mean a major version bump on the toolkit and changing all the call sites. Something like that is probably the right thing to do ultimately though.


Cannot View Files Added as Part of Content

I've created a custom content type 'Data' with field 'Video', and created content where I uploaded an .mp4 file. However, I'm never actually able to view the video. The furthest path set up is ('/node/:nodeid/assets/:filename') which returns video "url": as "http://localhost:3000/57dac1ab543dd1044dcbd6ed/180178618.hd.mp4" in Postman and a 401 in browser. So, I'm able to see where the video is but never able to reach it in order to view it. Any help on if there is a node where I'd be able to watch the video or if there are a few steps I'm missing would be greatly appreciated!

Hello, are you saying that your video doesn't display in the administration site? Or when you request it through the grasshopper API?

The video doesn't display in either. In the admin site if I click 'View Selected File' and then 'Open' it opens a new tab with the same url given above which displays File not Found. If I click 'View Selected File' and then 'Download' it fails and says no file. If I change the given url to the format ('/node/:nodeid/assets/:filename') I receive a 401 error. If I try to reach the file through the API with format ('/node/:nodeid/assets/:filename') in Postman via GET with Authorization passed in the header,  I receive the same url: http://localhost:3000/57dac1ab543dd1044dcbd6ed/180178618.hd.mp4, along with size & lastmodified.

It looks like the problem is that the url is given in the format: /nodeid/filename & the route set up to view it is /node/nodeid/assets/filename.

Oh, I see. It really sounds like your public folder is not linked correctly.
You should have set up a public path for all of your assets to be uploaded and served through express.
Something like
app.use(express.static(__dirname + '/public'));

And for the configuration see the config section in the documentation:
https://solid-interactive.github.io/grasshopper-core-nodejs/documentation.html#gettingstarted
Now, your assets should not need the API to serve them. It also sounds like your API route and your asset route might be going to the same place causing a conflict. For the assets config if it is set to '/' as the path try changing it to '/assets' or something different than the default API routes.

I set up my project using grasshopper fly, and so I had app.use(express.static(__dirname + '/public')); in the index.js file already. In the config file the "urlbase" was set to "http://locahost:3000" when I started. I changed it to "http://localhost:3000" to get the admin running, then just now I added /assets/ to make it "http://localhost3000/assets/" which fixed the issue I was having. Thanks!

Great, glad that fixed the issue.


fix for invalid byte sequence in UTF-8 (ArgumentError)

fix for issue #340

will create new pull request


License

Fantastic work! Is there an open source license for tagger or any plan to add one?

Hey!
Yes, I just added one. Sorry for the delay!


GUVNOR-2290 - Move Find menu into Authoring perspective context menu

Depends on https://github.com/uberfire/uberfire/pull/274

@cristianonicolai Is it possible to add the "Asset Search" menu item to drools-wb-webapp too? I use this webapp a lot during development and not having it mirror kie-drools-wb as closely as possible becomes a PITA (and I normally submit PRs to "add back" missing features)

Otherwise, +1 to merge.

@manstis unless I missed something, the update on AuthoringPerspective should had added it.

@cristianonicolai OK, I guess I was looking for the other "Find" item (see .newTopLevelCustomMenu( iocManager.lookupBean( SearchMenuBuilder.class ).getInstance() ).endMenu() in kie-wb-distributions)?

oh, ok. done :)

@cristianonicolai Thanks
+1 to merge.

master: baca96221bb864b6fad5985e9573bc2b141d2468


Use redis in "standalone" mode instead of "replication"

Currently we use the default mode for bitnami/redis, which is "replication". By default we don't use any of the replication features. This results in us having only one "master" node with no "slaves". We're essentially mimicking standalone.
We should set the "architecture" to "standalone".

We are already using standalone mode.


Its not able to use python in the backend and next.js in the frontend

Hi Developers.
Great product to use. I was looking for a project using python as the backend and next.js with react framework as the frontend as a non-techie.
I tried running it with the help of bolt, but it failed to build the same saying it can't use python frameworks in the web environment.
Can you look into the same and allow people to use both python and next.js as backend and frontend respectively.
Thanks!

I agree with your point of view. I am more familiar with Python and hope to support Django backend. Now it only supports node-related backends.


Allow custom version of npm

Fixes https://github.com/paketo-buildpacks/nodejs/issues/270
based on https://github.com/paketo-buildpacks/npm-install/pull/752
Summary

Instead of always using the npm installed by the node-engine buildpack, the user can specify BP_NPM_VERSION to change to the specified version.
Use Cases

See https://github.com/paketo-buildpacks/nodejs/issues/270 for details, but this allows users to use a specific version of npm rather than the one packaged with nodejs, e.g. in case of a known bug.
Details
If BP_NPM_VERSION is provided, it will install this version of npm first before installing the node_modules. It will be installed within node_modules (NOT globally).
Unfortunately, there are shells that do not allow binaries to the PATH, but only directories. So in order to only prepend this specific binary (see comment), we have to create a separate folder .bin_local and symlink the npm binary from .bin. So that .bin_local can be prepended to PATH in order to use it instead of the already installed npm. In order to make it accessible, it

Modifies the PATH env of the current process, so that the following npm call (e.g. npm install) will use this instead.
Modifies the PATH in the BuildEnv, so that other buildpacks use the same npm
Modifies the PATH in the LaunchEnv, so that other using npm at launch will use the same version.

Checklist

[X] I have viewed, signed, and submitted the Contributor License Agreement.
[X] I have linked issue(s) that this PR should close using keywords or the Github UI (See docs)
[x] I have added an integration test, if necessary.
[X] I have reviewed the styleguide for guidance on my code quality.
[X] I'm happy with the commit history on this PR (I have rebased/squashed as needed).

Tests are still missing: Reason that the current BuildFunc is kind of hard to test and we didn't want to add any more parameters.
Idea: We would refactor BuildProcessResolver to ProcessResolver and instead of Resolve, there would be ResolveBuildProcess,ResolvePruneProcess and ResolveUpdateNPMProcess.
This would not only keep the BuildFunc testable, but also simplify the signature by reming the need for the PruneProcess.
@paketo-buildpacks/nodejs-maintainers WDYT?

@mhdawson We changed it to not install the global version. Hence we had to prepend the .bin to the PATH instead of append. Could you have another look?

Integration tests are failing, so I guess I have to make them run on ARM :(

@c0d1ngm0nk3y made one suggestion otherwise it looks good to me.

I think some of the failures were related to the removal of support for 16.x, restarted the tests, although it might need a rebase to pick up the change I landed today for that.

@c0d1ngm0nk3y actually this was not one of the repos where I saw a failure and fixed. If you search for 16 in the tests and replace with 18 that may fix up some of the test failures.
Across a number of the repos the tests were coded to request Node.js 16 and since that was removed from node-engine the request needs to be updated to 18.

needs to be updated to 18

I have opened https://github.com/paketo-buildpacks/npm-install/pull/713 for this and picked the commit in the meantime.

The integration tests are driving me nuts. They are still failing for an unrelated reason. Seem that they are very fragile :(

The integration tests are driving me nuts. They are still failing for an unrelated reason. Seem that they are very fragile :(

I think this error is due to too many requests on the github registry for fetching the buildpack images. Try rerunning them after 10-20 minutes

The logger was fixed in packit, we have to wait for a release > 2.14.0 to be released and consumed.

@c0d1ngm0nk3y  thanks for the update.

https://github.com/paketo-buildpacks/packit/pull/581#issuecomment-2238670896

@mhdawson Is there any way to get the ubi extension run on arm? Normally, I just skip the ubi builder locally, but the current error in the tests only occur for ubi.
When I run it locally, it fails with
            [extender (build)]   Resolving Node Engine version
            [extender (build)]   Node no longer requested by plan, satisfied by extension
            [extender (build)]   Setting up launch layer for environment variables
            [extender (build)]   Configuring build environment
            [extender (build)]     NODE_ENV     -> "production"
            [extender (build)]     NODE_HOME    -> ""
            [extender (build)]     NODE_OPTIONS -> "--use-openssl-ca"
            [extender (build)]     NODE_VERBOSE -> "false"
            [extender (build)]
            [extender (build)]   Configuring launch environment
            [extender (build)]     NODE_ENV     -> "production"
            [extender (build)]     NODE_HOME    -> ""
            [extender (build)]     NODE_OPTIONS -> "--use-openssl-ca"
            [extender (build)]     NODE_VERBOSE -> "false"
            [extender (build)]
            [extender (build)]     Writing exec.d/0-optimize-memory
            [extender (build)]       Calculates available memory based on container limits at launch time.
            [extender (build)]       Made available in the MEMORY_AVAILABLE environment variable.
            [extender (build)]     Writing exec.d/1-inspector
            [extender (build)]
            [extender (build)] Paketo Buildpack for NPM Install 1.2.3
            [extender (build)]   Resolving installation process
            [extender (build)]     Process inputs:
            [extender (build)]       node_modules      -> "Found"
            [extender (build)]       npm-cache         -> "Found"
            [extender (build)]       package-lock.json -> "Found"
            [extender (build)]
            [extender (build)]     Selected NPM build process: 'npm ci'
            [extender (build)]
            [extender (build)]   Executing launch environment install process
            [extender (build)]     Running 'npm ci --unsafe-perm --cache /layers/paketo-buildpacks_npm-install/npm-cache'
            [extender (build)]       npm ERR! code 1

Should the ubi builder work with arm?

@c0d1ngm0nk3y introduction of arm support is new on some stacks and none of the buildpacks are built for arm either. So short answer is that I don't think any of the components for the Node.js buildpack support arm.
There is no reason that if you build the buildpacks they would not build on arm, but you would have to build all of them.
I'm surprised that you can run any of the tests with arm? Is it that you are trying to use a M based OSX machine?

I'm surprised that you can run any of the tests with arm? Is it that you are trying to use a M based OSX machine?

I am normally running them with the env variable DOCKER_DEFAULT_PLATFORM=linux/amd64 which seems to work fine. Also for the ubi builder.

The tests are fine locally...

@mhdawson It was "only" the node dependencies which could not be downloaded. I have the same problem locally from time to time. Seems not to be too reliable.

I checked the posix standard

PATH
This variable shall represent the sequence of path prefixes that certain functions and utilities apply in searching for an executable file. The prefixes shall be separated by a  (':'). If the pathname being sought contains no  ('/') characters, and hence is a filename, the list shall be searched from beginning to end, applying the filename to each prefix and attempting to resolve the resulting pathname (see 4.16 Pathname Resolution ), until an executable file with appropriate execution permissions is found. When a non-zero-length prefix is applied to this filename, a  shall be inserted between the prefix and the filename if the prefix did not end in . A zero-length prefix is a legacy feature that indicates the current working directory. It appears as two adjacent  characters ("::"), as an initial  preceding the rest of the list, or as a trailing  following the rest of the list. A strictly conforming application shall use an actual pathname (such as .) to represent the current working directory in PATH . If the pathname being sought contains any  characters, the search through the path prefixes shall not be performed and the pathname shall be resolved as described in 4.16 Pathname Resolution . If PATH is unset or is set to null, or if a path prefix in PATH contains a  character ('%'), the path search is implementation-defined.

ubi does it a bit differently and also accepts binaries, but to make it compatible I guess the approach with the extra directory containing only the npm binary was needed.

@paketo-buildpacks/nodejs-maintainers Is there any chance to get this merged? Or is there anything missing?
I checked again and there should be no possibility for any regression since the folder .bin_local is only created if BP_NPM_VERSION is set and adding a non-existing path to PATH should not have any influence.

All checks have passed


Windows XP support? "Application failed to initialize properly"

On Windows XP, Telegram does not work with FontMod. The following error is displayed:

The application failed to initialize properly (0xc0000142). Click on OK to terminate the application.

Maybe this is because FontMod.dll is not compiled for Windows XP? I would be very thankful if you could make it work for Windows XP.

Maybe it's fixed in v2.1. Can you help me test it?


Job /job/ExampleWorkspace/job/ExampleProject/job/Create_Environment fails

Job /job/ExampleWorkspace/job/ExampleProject/job/Create_Environment doesn't work:
[EnvInject] - Inject global passwords.
Started by user Admin User
[EnvInject] - Loading node environment variables.
[EnvInject] - Preparing an environment for the build.
[EnvInject] - Keeping Jenkins system variables.
[EnvInject] - Keeping Jenkins build variables.
[EnvInject] - Injecting as environment variables the properties content
WORKSPACE_NAME=ExampleWorkspace
PROJECT_NAME=ExampleWorkspace/ExampleProject

[EnvInject] - Variables injected successfully.
[EnvInject] - Injecting contributions.
Building remotely on Swarm_Slave-68f7e58b (swarm java8 ldap aws docker) in workspace /workspace/ExampleWorkspace/ExampleProject/Create_Environment

Deleting project workspace... done

[ssh-agent] Using credentials jenkins (ADOP Jenkins Master)
[ssh-agent] Looking for ssh-agent implementation...
[ssh-agent]   Java/JNR ssh-agent
[ssh-agent] Started.
Cloning the remote Git repository
Cloning repository ssh://jenkins@gerrit:29418/ExampleWorkspace/ExampleProject/adop-cartridge-java-environment-template
 > git init /workspace/ExampleWorkspace/ExampleProject/Create_Environment # timeout=10
Fetching upstream changes from ssh://jenkins@gerrit:29418/ExampleWorkspace/ExampleProject/adop-cartridge-java-environment-template
 > git --version # timeout=10
using GIT_SSH to set credentials ADOP Jenkins Master
 > git -c core.askpass=true fetch --tags --progress ssh://jenkins@gerrit:29418/ExampleWorkspace/ExampleProject/adop-cartridge-java-environment-template +refs/heads/*:refs/remotes/origin/*
 > git config remote.origin.url ssh://jenkins@gerrit:29418/ExampleWorkspace/ExampleProject/adop-cartridge-java-environment-template # timeout=10
 > git config --add remote.origin.fetch +refs/heads/*:refs/remotes/origin/* # timeout=10
 > git config remote.origin.url ssh://jenkins@gerrit:29418/ExampleWorkspace/ExampleProject/adop-cartridge-java-environment-template # timeout=10
Fetching upstream changes from ssh://jenkins@gerrit:29418/ExampleWorkspace/ExampleProject/adop-cartridge-java-environment-template
using GIT_SSH to set credentials ADOP Jenkins Master
 > git -c core.askpass=true fetch --tags --progress ssh://jenkins@gerrit:29418/ExampleWorkspace/ExampleProject/adop-cartridge-java-environment-template +refs/heads/*:refs/remotes/origin/*
 > git rev-parse refs/remotes/origin/master^{commit} # timeout=10
 > git rev-parse refs/remotes/origin/origin/master^{commit} # timeout=10
Checking out Revision 86f2e0b110d7569d7933d77c6478d7e8e425c483 (refs/remotes/origin/master)
 > git config core.sparsecheckout # timeout=10
 > git checkout -f 86f2e0b110d7569d7933d77c6478d7e8e425c483
 > git rev-list 86f2e0b110d7569d7933d77c6478d7e8e425c483 # timeout=10
[Create_Environment] $ /bin/sh -xe /tmp/hudson3927040214662628747.sh
+ set +x
CI, tomcat.conf
TLS configuration is invalid - make sure your DOCKER_TLS_VERIFY and DOCKER_CERT_PATH are set correctly.
You might need to run `eval "$(docker-machine env default)"`
Build step 'Execute shell' marked build as failure
[ssh-agent] Stopped.
Warning: you have no plugins providing access control for builds, so falling back to legacy behavior of permitting any downstream builds to be triggered
Finished: FAILURE

I tried defining DOCKER_TLS_VERIFY and DOCKER_CERT_PATH within the shell script that is embedded in the job, but it does not seem to make any difference. Where exactly are the TLS certs for the engine, and how should they be transferred to the node?
Please note that this instance is a single node ADOP server, created from OS X, where the gen-certs step was skipped because it won't work on OS X (see related issue #73)

This issue is directly related to the fact that gen-certs was skipped.
You can manually follow the script to generate the certificates and upload them to Jenkins slave (correct directory should be //root/.docker/)

Can this be closed here and raised in the correct repository please if it is still an issue?
https://github.com/Accenture/adop-cartridge-java

where is the script that is mentioned in: "You can manually follow the script to generate the certificates and upload them to Jenkins slave (correct directory should be //root/.docker/)"

Hi @teamschwarz
The certificate generation function is part of a cli command in adop compose init found here: https://github.com/Accenture/adop-docker-compose/blob/master/cmd/compose#L181

I think an important point here is that when the certificate generation fails as part of running quickstart, it kind of fails silently (the script does stop, but it does not show an error to the user) and then when you go on and try the platform, the certificates are not generated, and it fails. PR #104 should help avoiding the certificate generation error itself, but if it does fail on the cert gen, it is also changed so that it shows an error to the user


Remove libmagic dependency

Reasons for removing libmagic are numerous, but the main ones are:

It's very slow
It lies about file types
It doesn't work on Windows

This PR replaces file type detection using libmagic with a file-extension based lookup (see file_types.rs). The lookup table is from file extension to mime type and friendly name, and uses phf for efficient compile-time construction of the hash table.

I like it, +100.

I worry that the list is a bit dated.  Maybe it'd be worth writing a quick script to scrape https://en.wikipedia.org/wiki/List_of_filename_extensions_(A–E)#A,  etc. to use that?  We can keep it up to date that way.
Let's do another PR after this to just replace "libmagic" with filetype or something like that.

I like it, +100.

I worry that the list is a bit dated.  Maybe it'd be worth writing a quick script to scrape https://en.wikipedia.org/wiki/List_of_filename_extensions_(A–E)#A,  etc. to use that?  We can keep it up to date that way.
Let's do another PR after this to just replace "libmagic" with filetype or something like that.

There are extension collisions in that wiki list. Definitely need some manual screening there.

@hoytak, @seanses: how about we merge this PR, then open separate issues for (1) renaming libmagic, and (2) updating the file type list using the Wikipedia data (with some manual effort)?

Sounds good to me!


[Feature][CI] Add plugin_config and seatunnel-dist/pom.xml check for connectors

Search before asking

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

Description
All connectors need to add themselves to config/plugin_config and seatunnel-dist/pom.xml and seatunnel-dist/src/main/assembly/assembly-bin-ci.xml file. We need add check job in github workflow job to check this.
Usage Scenario
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

Yes I am willing to submit a PR!

I want try improvement this issue


⚠ RPC server - Optimism has degraded performance

In 9a2c5c9, RPC server - Optimism ($OPTIMISM_RPC_URL) experienced degraded performance:

HTTP code: 502
Response time: 222 ms

Resolved: RPC server - Optimism performance has improved in 4cba2d4 after 35 minutes.


The Path is wrong, 'MFA' does not exists: "Service settings can be accessed from the Azure portal by browsing to Azure Active Directory > MFA"

The described Path
"Service settings can be accessed from the Azure portal by browsing to Azure Active Directory > MFA"
does not exists. There is no "MFA" in "Azure Active Directory"

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

ID: 84b26750-880c-8f14-1628-436293de915e
Version Independent ID: 518b727f-79e8-b5c2-bcb3-ff4de3444c6f
Content: Configure Azure Multi-Factor Authentication - Azure Active Directory
Content Source: articles/active-directory/authentication/howto-mfa-mfasettings.md
Service: active-directory
Sub-service: authentication
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@schittli thanks for this. I'm closing this out as it is a duplicate of https://github.com/MicrosoftDocs/azure-docs/issues/42331

Sorry, just realized the other one is different. I do agree that the screenshot could be updated on this so I'm assigning the issue to the author, @MicrosoftGuyJFlo

Sorry, I realized this is a separate issue. I made a pull request to remove the outdated screenshot and am assigning this to the content author @MicrosoftGuyJFlo to add the new one.
https://github.com/MicrosoftDocs/azure-docs-pr/pull/95192

#please-close resolved in private PR


Image cannot be rendered when using RepaintBoundary screenshots in the FlutterEngineGroup

Image cannot be rendered when using RepaintBoundary screenshots in the FlutterEngineGroup

@Rempage
Can you provide flutter doctor -v and a complete minimal reproducible code sample that shows this issue ?
Thanks.

I have solved this problem in other ways, thanks

Closing this as resolved based on above comment.


REDESIGN OF RESOURCES PAGE

Follow the given design below.
Responsiveness is needed.

I would like to design this page. Please kindly assign.

yeah @Devil1205, have at it.


How to run this code ?

Hi Can you please help me which file to run and where is the final output of the code ?

As the README.md file says you can just run one of the python scripts (i.e python sigrecogtf.py) and you will get just one single output, which is the score of the network on the specified author samples. Currently, the author is defined within the source file.

@guilhermefloriani I'm on the same boat as @basant01 I've found the parameter you refer to. But it seems something is hardcoded or maybe you missed some file on the commit?

Well, as I said, the parameter that specifies the author is in the source file.
https://github.com/guilhermefloriani/signature-recognition/blob/5fe360116a1e58792464a0b47191cd7afe4eed35/sigrecogtf.py#L12
https://github.com/guilhermefloriani/signature-recognition/blob/5fe360116a1e58792464a0b47191cd7afe4eed35/sigrecog.py#L13
The training and test subsets will be selected using this author id.

Yes I've noticed that but it seems form the tests I did that the evaluation is only considering the suffix of the files and not the NN

It does use the file prefix to train the NN and to assert the results as well.

Do i need the prefix on my training data?

Yes, it should follow the same pattern. Otherwise, there's no way to know which one whether an example if genuine or forged.

So I cannot classify a  sample having as input only valid ones?
Em sex, 9 de ago de 2019 17:44, guilhermefloriani<EMAIL_ADDRESS>escreveu:

Yes, it should follow the same pattern. Otherwise, there's no way to know
which one whether an example if genuine or forged.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/guilhermefloriani/signature-recognition/issues/3?email_source=notifications&email_token=AAGVSFXTYBCKCVKPCHZKURLQDXJL5A5CNFSM4IJJ6ZRKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD37XHJY#issuecomment-520057767,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAGVSFQ2M3NVEUK5MZFDEVTQDXJL5ANCNFSM4IJJ6ZRA
.

Actually, it does not make much sense. How could the NN learn to identify a forged signature while you only train it w/ genuine samples?
Anyway, that's not the approach of this experiment.

@guilhermefloriani Oki I was replying on mobile and that sucked, sorry.

What I did expect: Based on a set of valid signatures to be able to identify a forged one.

How: based on a given difference threshold. So I have a set of valid signatures and will consider invalid the ones that differ too much.

So the NN would say: xyz% sure. I still don't get how this code works at the moment, as I did not read it properly. I'm just space exploring and was looking for implementations. Yours has the biggest number of stars :)

@vfbsilva @guilhermefloriani given the popularity of this repository I've been thinking for a while in approaching this problem differently. When we came up with this solution, we were not aware of the one-shot learning techniques.
Now that we have a better understanding of these techniques, it looks promising.
In summary, we'll generate an embedding of the anchor signature, and use a norm to evaluate if new samples are real or forged. That's roughly how SOTA facial recognition algorithms work, and maybe if we take advantage of a convolution neural network trained for handwritten characters we could leverage transfer learning and reduce training time.
Let me know what you think of these ideas. This work would be published in a different repository.

Perfect!

@vfbsilva  I think we should develop something like this because I had tried lots of signature verification repository but none of us performing well?

Cool! Sounds great @anisiomarxjr, let me know if I can help with something.
I’m sorry @vfbsilva, but we made this experiment a couple of years ago and this approach worked well in that controlled scenario. Hopefully the new approach mentioned by @anisiomarxjr works for you.

@basant01 yes. I've found many repositories all of the seem to present some issue or having missing parts. Did you had any luck finding a proper one?


Support for GFPGAN?

First off thank you for VSGAN!
I was trying to see the results of this model: https://github.com/TencentARC/GFPGAN/releases/download/v0.2.0/GFPGANCleanv1-NoCE-C2.pth from https://github.com/TencentARC/GFPGAN but received this error: 'model.0.weight'.
Is this model set incompatible or is there some other issue? Thank you.

That is a model for GFPGAN, not ESRGAN. VSGAN currently only supports ESRGAN at the time of writing.

Support for GFPGAN most likely won't happen, at least not soon. This project is focused on general image and video inputs, but GFPGAN is focused specifically on faces. You need to crop the images to only the faces for it to really work.


Which platform is this bot native? Just curious.

I would like to know on which platform is this developed? Is it Mac?

this is a dumb question, and it is not an issue. Please refrain from creating issue for this kind of questions.

I asked because I want to use the same platform. At home, I have a Windows box and a Mac box. I'm deciding which one to run my bot.
I want to be on the same side of the developers.

It's platform agnostic. As long as you have Python installed. Follow the instruction.


Change offerAmount to pricePerToken

Re: v2/MarketWithAuction
To maintain a consistent interface throughout the contract, we’ll change the Offer struct code to use pricePerToken instead of totalOfferAmount, and implicitly calculate the ‘total offer amount’ as price per token offered * quantity of tokens wanted.

Updated the  Offer struct code to use pricePerToken instead of totalOfferAmount, and implicitly calculate the ‘total offer amount’ as price per token offered * quantity of tokens wanted. Updated the tests to reflect this change as well. See 9552b1dddff42ceaf0cf5daa916977b6e6873019


Add extensionswebhook.EnsureUnitWithName func to add unit to OSC units

How to categorize this PR?

/kind enhancement
What this PR does / why we need it:
Right now various extensions have different funcs for adding an unit to the OSC units. By adding, I mean "upsert" - insert if not present, update - if present.
The existing func extensionswebhook.AppendUniqueUnit does only insert but does not do update.
IMO, extensionswebhook.AppendUniqueUnit does not make much sense as we usually want "upsert" behaviour. I cannot thing of scenario where "insert only" behaviour is the desired one.
Examples of extensions having such func inline:
registry-cache: https://github.com/gardener/gardener-extension-registry-cache/blob/d701d3b4fcef53d8cb4a44281e4f9407e2723cba/pkg/webhook/operatingsystemconfig/ensurer.go#L154-L165
shoot-rsyslog-relp: https://github.com/gardener/gardener-extension-shoot-rsyslog-relp/blob/c09737d2ab8780c87463e5a685cc2ed89988a5e8/pkg/webhook/operatingsystemconfig/ensurer.go#L97-L99
Which issue(s) this PR fixes:
N/A
Special notes for your reviewer:
N/A
Release note:

There is now new `github.com/gardener/gardener/extensions/pkg/webhook.EnsureUnitWithName` func that can be used to add/update unit to OperatingSystemConfig units.

/approve


fix(docker): Updating docker image

:tada: This PR is included in version 3.4.1 :tada:
Your semantic-release bot :package::rocket:


Disappearing tab content

On DDG <IP_ADDRESS> and iPadOS 15.8.3 with an iPad Air 2, tabs in the background will regularly lose their content.
To reproduce:

Go to any website in tab A
Open two new tabs B and C and go to any website
Switch back to tab A or close tabs B and C

Result:

As you can see, the tab still has the name of the original website but it shows the DDG new tab page rather than the actual website.
This can be refreshed by creating a new tab again and clicking back on tab A, which then reloads with the actual website, but it's still obviously not the correct behavior.

Hi @SkylerMime - thanks for this.  We're tracking it internally at  https://app.asana.com/0/392891325557410/1208801015356280/f
I've been trying to replicate today and have been unable to.   Does it happen every time or just intermittently?
Can you let me know what your settings are in our app, in the General section (in particular "keyboard") and Data Clearing > Autoclear, please?
Thanks!

Thanks for looking into this!
Keyboard settings: show keyboard on "New Tab" is checked, "App Launch" is unchecked.
Data clearing: Automatically clear data is off, no sites are fireproof.
The bug may not be as consistent than I thought, I tried to reproduce the bug this morning and everything worked fine, which is good. I'll let you know if it starts happening again!


Raise exception when num circuits exceeds backend max_circuits

Summary
Raise and exception when the number of circuits passed to IBMBackend.run() exceeds IBMBackend._max_circuits.
Details and comments
Fixes #667

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 51.966%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
3146

Relevant Lines:
6054

💛  - Coveralls


Sorting the Result

Any chance that adding a sort option for the resulting checklist is feasible?
I'm surprised that sorting checklists isn't in native Trello, but based on my searching, doesn't seem any PowerUps offer it either.
Thanks for the great product, BTW. Very useful :)

Hey Geoff,
Sorry for the late response on this one, I've been a bit flat out.
I'm happy to add a sorting option but I wouldn't want to force people into any one sorting method. Do you think a drop-down under the list selection would be okay?
I'm thinking the drop-down will read:
Sort by

Original order
Alphabetically (Ascending)
Alphabetically (Descending)

As a secondary thought, I don't think it will be too challenging to create a PowerUp that sorts checklists. I'm assuming nobody else has done this because Trello doesn't offer us an API to move checklist items. We could potentially get around this by deleting the original checklist items and re-adding them in sorted order.
Let me know if you think this would be useful and I can take a look at adding it to this suite.

No worries for the delay, thanks for looking into this! I think your idea with the dropdown would be great, although as you said it might be best to just offer it as a separate powerup.

Too easy, I think I'll release both as I'm not sure about the repercussions of deleting checklist items at the moment.
I've got a basic implementation of the sorting option ready here so I'll add the finishing touches, test and let you know once it's deployed.

This is deployed and should now be available after a refresh

This works wonderfully! Thanks so much :)


bfloat16 value type support

TFRA supports many value types including float16, but bfloat16 type is not there. Are there any specific reasons?
I tried to add a BF16 value type kernel for cuckoo hashtable op and it seems to work. And it is easier to converge than FP16 in training.

Thank you @xiangzez , there're no specific reasons, you can commit a PR for this, thank you~

@xiangzez
Additionally, at the beginning, we can't confirm if it works well that TF supports bf16 on CPUs, so we didn't open the data type for bf16. Since you have already made enough testing, I believe we can try to open it, could you commit a PR?

Yes TF now supports bf16 well on CPU. I'll submit a PR for this.


[sil-opaque-values] Fix SILGen and ownership of open_existential_box_…

…value.
Handle open_existential_value the same regardless of whether it's an opaque or boxed representation. In both cases, the opened value is a borrowed subobject.

@swift-ci smoke test.

@swift-ci smoke test.

@swift-ci smoke test.


bug: Make sure coverage report gets printed when people doing PR from a fork

We did not saw this problem anymore.


Update setup wiziard to show the instruction how to use the previous installed firmware

I have a firmware that I got with a hacked PS4 for a long time, but now this PS4 is not there, please add this function.

As I would like to contribute to this project, I will try to solve this one.

As I would like to contribute to this project, I will try to solve this one.

But it... Already has a solution? After you select your system folder on the Wizard... You drag and drop these 3 folders.

I think that  Dani

As I would like to contribute to this project, I will try to solve this one.

But it... Already has a solution? After you select your system folder on the Wizard... You drag and drop these 3 folders.

this includes firmware?

from what I know is that you need a ps4 to extract the firmware or you cant pass the wirzard part in other case

Err, no, you just close the wizard, put the folders into the System folder you selected earlier, and tada.

Unless if we got some check that doesn't- @ultimaweapon How... Do we check if the System folder the user selects actually contains firmware?

You only need the system folder from the PS4.  Then choose the folder that contains this folder for the system location on the wizard. It should not ask for the PS4 if your system is copied correctly from the PS4.

Wait, in order for the system folder to be usable all of the sprx files must be decrypted. That means if that folder was copied without SELF decryption it cannot be used.

Maybe we should adding a note in the system location page to tell the user that they can choose the location where the firmware was manually extracted.

@Danik2343 @AlexsanderDamaceno Will this new page work? https://github.com/obhq/obliteration/wiki/Firmware-Setup-Guide#firmware-setup-via-local-files

@VocalFan Sorry for long waiting, not working page.

Yeah we're uh, in the middle of converting it to be part of the website

@VocalFan If you still have the text, can you quote here what was written in that topic?

Recently we have changed how Obliteration works so the previously dumped firmware is no longer usable. Right now the tool to dump a new firmware is currently under development so you need to wait for this tool.

Recently we have changed how Obliteration works so the previously dumped firmware is no longer usable. Right now the tool to dump a new firmware is currently under development so you need to wait for this tool.

Right now I just use a blank file called no.obf lmao

Fixed by #1160.


Add addresses in Muskogee, OK

Status:

[ ] Research sources and propose best source (and it's data license)
[ ] Review proposed source
[ ] Approve source based on review comments
[ ] Add to OpenAddresses

NOTE: Proposed source notes should include basic column mapping to OpenAddress data model.

anyoneone know how to convert autocad to reality!? http://www.muskogeeonline.org/departments/planning/city_maps.php#revize_document_center_rz269
e-mailed asking for gis data ;)
<EMAIL_ADDRESS>

Eek, that is some crazy!
On Thu, Mar 2, 2017 at 5:06 AM, Justin Meyers<EMAIL_ADDRESS>wrote:

anyoneone know how to convert autocad to reality!?
http://www.muskogeeonline.org/departments/planning/city_
maps.php#revize_document_center_rz269
e-mailed asking for gis data ;)
<EMAIL_ADDRESS>—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/openaddresses/openaddresses/issues/2004#issuecomment-283648488,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA0EO_b8ZurI_S3UU__tC5_ScnAjAxrbks5rhr7agaJpZM4J-FNX
.

yes - just got off the phone w the city. county only has in cad... I'm calling to confirm this. the city has data and will be sending to us (*heavily outdated), but the guy said he would never convert the data from cad to gis ever again!?! stay tuned...

Sarah with the county assessor's office will be back tomorrow and should call me.

*uploading to dropbox now and reaching out to e911:
Justin,
Here is the address shapefile, the latest updates I believe to this was in 2014.  I will send what parcels I have in another email, I was exceeding the email attachment limit size with both.   If you need the addresses for entire county maybe Vicki over at 911 might be able to help you out.
<EMAIL_ADDRESS>Jimmy

just e-mailed Vicki - city data here: https://www.dropbox.com/sh/vlybcs3sjt5nw3b/AADRst1-LEOmXVZ5bicfMFWTa?dl=0

Sorry Justin, I thought I had sent everything for that shapefile.  Here is that dbf file.  I tried sending that parcel but I got a return email.   IT said it looked like you guy’s server denied the email  because the attachment size was too large 18.7MB. That was just the dbf file alone.   IT said we could do a FTP file transfer for the parcel shapefile and also for this, if this dbf file doesn’t go through.
I should mention for this address shapefile, I’m not sure what it is, I made a GIS ID for making “Joins” and it seems to work good as a shapefile, but when it’s in a geodatabase, some of the records don’t seem to join correctly.
Jimmy

https://www.dropbox.com/sh/vlybcs3sjt5nw3b/AADRst1-LEOmXVZ5bicfMFWTa?dl=0


Visual Studio Code Open 3 times in a row  when using cmd

Issue Type: Bug
When I go to the path of the project that I wanted to work on using cmd and I enter " code ."  the Visual Studio Code opened 3 times in a row.
1- the project i wanted to work on
2- the last project i worked on
3- An empty Visual Studio Code from the root of my C: \  computer
VS Code version: Code 1.31.1 (1b8e8302e405050205e69b59abb3559592bb9e60, 2019-02-12T02:20:54.427Z)
OS version: Windows_NT x64 6.3.9600

You might want to tweak the settings files.hotExit, window.openFoldersInNewWindow and window.restoreWindows which drive this behavior.


Minimap and/or Minimap Slider isn't correct

Issue Type: Bug
"editor.minimap.enabled": true
"editor.minimap.renderCharacters": true
The minimap slider, more specifically the highlight area it shows of where you are on the code is not the correct position.
I assume that highlight should go from the top line to the bottom line of the visible code   in the editor, but it's off by quite a lot.
You can see what I'm talking about in the following picture.
https://i.imgur.com/z3a1jVO.jpg
VS Code version: Code 1.23.1 (d0182c3417d225529c6d5ad24b7572815d0de9ac, 2018-05-10T17:11:17.614Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz (8 x 4008)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwarevideo_decode: enabledvideo_encode: enabledvpx_decode: unavailable_softwarewebgl: enabledwebgl2: enabled

Memory (System)
14.94GB (8.04GB free)

Process Argv
C:\Program Files\Microsoft VS Code\Code.exe

Screen Reader
no

VM
0%

Extensions (5)

Extension
Author (truncated)
Version

gitlens
eam
8.3.0

material-icon-theme
PKi
3.4.0

todotasks
san
0.5.0

wow-bundle
Sep
1.2.5

vscode-lua
tri
0.12.4

ok, this is really strange.
Today I open vscode, on the same project, same file, and the minimap highlight is on the correct place.
At first I was completely confused but then I found something strange.
When the "Side Bar" is open, the minimap highlight is correct, when it's closed it's not correct anymore.
Gif example of this happening.
This is making me think that it has something to do with "word wrap" / "max columns".

@m4xc4v413r4 I see the problem in your GIF, but I could not reproduce.
I am pressing Ctrl+B and Alt+Z to toggle the sidebar or to toggle word wrap, but I don't observe the slider being off:
I wonder what other settings you have, if this reproduces when extensions are disabled and if this reproduces for any file or only for a specific file.

@alexandrudima Can you do me a favor and try that with "editor.minimap.renderCharacters": true ?
To be honest I'm not sure of anything anymore.
I just did some more testing, with and without extensions, hiding and showing the Side Bar with mouse and keyboard shortcut, enabling and disabling word wrap, and I'm now more confused than ever.
I don't see consistency, some times the problem is there, some times it's not, other times the problem exists but goes away after closing and opening VSC without changing the settings.
I literally did the exact same thing as my previous gif, with same extensions and same settings, and now there's no problem when toggling the Side Bar.
I tried toggling the wordwrap after that because you menioned it and that breaks it for me again. I never touched it when I originally noticed the problem though, I don't think I ever touched wordwrap since I first turned it on and I've been using VSC since 0.10 or something so a couple of years.
I'mgoing to leave some more gifs here for different thing I tried.
If there's anything specific you want me to try, a specific test or any information I can provide you, feel free to ask for it.
With extensions. With minimap.renderCharacters on:
Toggling Side Bar - Same test i had done on the other gif but not it works fine :/
Toggling WordWrap - Breaks it pretty much the same way I was experiencing before, even though I never toggled wordwrap before. I even toggle the Side Bar at the end and it doesn't change anything, only the wordwrap does.
With extensions. With renderCharacters off:
Toggling Side Bar and WordWrap - The same two tests don't cause problems if renderCharacters is not enabled.
Without extensions every test results in the exact same thing as before:
Toggling Side Bar and WordWrap with and without renderCharacter
So the problem might be between minimap.renderCharacter and word wrapping, maybe even their word wrapping limits (max columns) since we can put a limit on both the editor and the minimap.
My current settings are as following:

{
"terminal.integrated.shell.windows": "C:\Program Files\Git\bin\bash.exe",
"window.zoomLevel": 0,
"editor.wordWrap": "on",
"window.menuBarVisibility": "toggle",
"zenMode.fullScreen": false,
"zenMode.hideTabs": false,
"zenMode.hideStatusBar": false,
"workbench.iconTheme": "material-icon-theme",
"editor.minimap.enabled": true,
"editor.minimap.renderCharacters": true,
"editor.minimap.maxColumn": 120,
"editor.insertSpaces": false,
"editor.tabSize": 4,
"editor.detectIndentation": false,
"editor.acceptSuggestionOnEnter": "smart",
"editor.renderWhitespace": "none",
"editor.showFoldingControls": "always",
"extensions.autoUpdate": false,
"editor.quickSuggestions": {
"other": true,
"comments": false,
"strings": true
},
"editor.multiCursorModifier": "alt",
"git.enableSmartCommit": true,
"workbench.startupEditor": "newUntitledFile",
"editor.smoothScrolling": false,
"gitlens.advanced.messages": {
"suppressShowKeyBindingsNotice": true,
"suppressWelcomeNotice": true
},
"files.trimTrailingWhitespace": true,
"gitlens.blame.ignoreWhitespace": true,
"gitlens.gitExplorer.files.layout": "auto",
"gitlens.keymap": "chorded",
"markdown.preview.scrollEditorWithPreview": false,
"markdown.preview.scrollPreviewWithEditor": false,
"markdown.preview.linkify": true,
"[markdown]": {
"editor.wordWrap": "on",
"editor.quickSuggestions": true
},
"lua.format.lineWidth": 500,
"lua.preferLuaCheckErrors": true,
"editor.fontLigatures": true,
"editor.fontFamily": "Fira Code, Consolas, monospace",
"editor.fontSize": 13,
"editor.fontWeight": "500",
"gitlens.historyExplorer.enabled": true,
"gitlens.codeLens.enabled": false,
"gitlens.hovers.currentLine.over": "line",
"gitlens.mode.statusBar.alignment": "left",
"gitlens.statusBar.alignment": "left",
"editor.minimap.side": "right",
"editor.minimap.showSlider": "mouseover"
}

This includes the extension related settings (the ones in bold), those obviously are irrelevant on the no extensions tests.

Do you see any problems in the console? F1 > Developer: Toggle Developer Tools or in the logs F1 > Developer: Show Logs... > Window (VS Code) ?
Do you perhaps have CJK characters in that specific file. It might be a dup of #49378

No errors, some Violations though (I think the JS ones happened while recording the gif that I have linked at the end here), and found out some more things.
So far all my tests were done with the window using half the screen (I use a 21:9 2560x1080 screen) using Windows Snap feature.
Because I opened the console to test what you wanted I noticed that when I go above a certain width on the VSC window size, the behavior changes (you can actually see when this happens by looking at the minimap), wordwrap doesn't cause a problem anymore, but simply pressing Alt (which toggles the Menu Bar as you probably know) creates the same problem, but on the previous size I was using, pressing Alt doesn't create the problem.
Example of this happening

@m4xc4v413r4 I see it happening for you and I have tried and tried, but I am still unable to reproduce. I see that there is a clear point for your setup (when you enlarge the width of VS Code) after which the minimap is off.
But after trying and trying, I could still not reproduce... I wonder if you can perhaps share the file that reproduces it, or if it reproduces on any file for you...
I've even reviewed the source code that computes the minimap layout to look for a possible division by 0 or some other corner case, but I failed to find anything obvious: https://github.com/Microsoft/vscode/blob/7a557d12c961511039d3906c26c76c8556c30909/src/vs/editor/browser/viewParts/minimap/minimap.ts#L216-L296
To rule out some things, can you please disable all extensions, and slowly comment out your settings (including font settings, etc) to try to narrow down what exactly causes it to reproduce...


Versions import

How about adding a way to import a Black Pixels Versions app .plist bookmarks file into GitHawk?

Bug Report Dump (Auto-generated)

Version 1.27.2<PHONE_NUMBER>)
Device: iPad6,11 (iOS 12.2)
TestFlight: false

I'm not sure what you mean. Can you elaborate further? 😇

@BasThomas I believe the request here is to allow importing of Bookmarks from external sources such as the Versions app's Plist format.
Would be a case of just pulling out the repository URLs, working out which ones are on GitHub and then pulling them into the DB.

@BasThomas I believe the request here is to allow importing of Bookmarks from external sources such as the Versions app's Plist format.
Would be a case of just pulling out the repository URLs, working out which ones are on GitHub and then pulling them into the DB.

Yes. That was what I had in mind. On macOS, I beleive the file name in question is "com.blackpixel.versions.plist" but I could be wrong - Versions may store the repo bookmark list in some other location in the filesystem.


UnmatchedTest tutorial should be on an unmatched example

I don't like how the exact same example I used for the MatchedTest tutorial was used for Unmatched also. We should:

[ ] explain when one would use unmatched
[ ] generate an example where the two graphs are not matched and use that for the tutorial
[ ] make sure all plots have axes labels

@bvarjavand

once #222  is done we can do this

i can do this also when pr'ing size fix, if you want
@bdpedigo @bvarjavand

That would be great!
On Mon, Apr 20, 2020 at 12:09 PM alyakin314<EMAIL_ADDRESS>wrote:

i can do this also when pr'ing size fix, if you want
@bdpedigo https://github.com/bdpedigo @bvarjavand
https://github.com/bvarjavand
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/neurodata/graspy/issues/177#issuecomment-616652748,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGEF4H4MSG2AXUDWHOE5SK3RNRXRZANCNFSM4HOMR3XA
.

i feel like it makes most sense to only do this after #375 ?
cc: @bdpedigo

@alyakin314 either order makes sense to me, up to you if you wanna do it

@bdpedigo don't really mind delegating this to ndd as well (and would be happy to consult whoever will take this).

@kareef928 wanna do this one too? We finally have the tools to do so... I think you'd be able to do this issue and the other I tagged you in this semester

I'd be down to do this too

Dod: Modifications to the latent distribution test tutorial will be made so that the following will be included:

An explanation of when to use the latent distribution test over the latent position test

Examples of the latent distribution test being used on two unmatched graphs (m /= n) in the cases we expect the null hypothesis to be true and false - generate labeled figures of null distribution, test statistic, p-value for both cases


added unit tests for shipping free module - FreeShippingServiceProvider

added tests for #1037 #1069 #1076
SimplCommerce.Module.ShippingFree -> Services- FreeShippingServiceProvider

Thanks for your contribution


[das/#106] Change patterns and templates indexes to store handles only rather than handles+targets

We use 5 indexes in AtomDB. Names, incoming set, outgoing set,  type templates and patterns.
All of them are hash tables mapping from handles to values. Values are different deppending on the type of the index. The former 3 indexes map handle->List[Handle] (list of handles) while the later 2 map handle->List[Tuple[Handle, [Tuple[Handle]]]] (list of tuples (handle, targets) with the handle and the targets pod the mapped links.
Mapped link targets are stored in the last two indexes for the sake of performance in the pattern matcher query algorithm.
In the scope of the integration of the cache in the query engines, this optimization is no longer required, thus we want to simplify the API and store handle->List[Handle] for ALL indexes. This is what this PR is about.
This PR changes the AtomDB to consider a mapping of Handle->List[Handle] in the templates and in the pattern indexes.
This change is coupled with changes in the meta parser and the query engines in order to keep compliance.
SIDE EFFECT CHANGE: as a side effect of these changes, ORDERED_LINKS references (as well as is_ordered() calls) have been removed.

Also, it seems that outgoing_set could be annotated as dict[str, set[str]].
Currently it is:
    outgoing_set: dict[str, Any] = dc_field(default_factory=dict)


Unable to Change Plans

Unable to get the app to recognize created subscription plans from the test stripe account.  It only every says - No such plan: "name of plan"

Do the names match in server/config/secrets.jsL#16?

+1 make sure to update plans and planData https://github.com/eddywashere/node-stripe-membership-saas/blob/master/server/config/secrets.js#L16-L33 is
I'd eventually like to clean that up into just 1 list and pluck the names when needed as an array of strings.

like this? What's involved?
plans: [
      {'free': {}},
      {'silver': {}},
      {'gold': {}}
    ]

Not sure if you resolved your issue but let me tell you what happened to me:
I wasn't able to update plans so what I found out was; we have to have a plans on stripe.com portal.

Login https://dashboard.stripe.com/
And https://dashboard.stripe.com/test/plans (Dashboard, Plans)
Add new plan: in this case, I added 3 plans: free, silver, gold, platinum
Try Update Plan button under http://localhost:3000/billing (or your server url)

hope this helps.

@mayoralito  Hello, Thanks but.
I can see the Product and Product has the plan.
What info I need to copy. there isn't the link like https://dashboard.stripe.com/test/plans.
and there is the link like https://dashboard.stripe.com/test/subscriptions/products.


go/worker/compute: Start batch resolution early for backup worker

Sure, are you going to fix/implement the TODO items?

Those have been transferred over from existing code and should be taken care of in a separate PR IMO.

Sure, are you going to fix/implement the TODO items?

Those have been transferred over from existing code and should be taken care of in a separate PR IMO.

Alright


Added About us page

This pull request solves #100,  which adds an About Us page to the project.

@DurgaPrasad-R kindly work upon the orientation the about us page is not properly oriented there is unevenness between images and text.

Work on it!

@Mahitej28 Actually, I was in doubt regarding this. I need a suggestion on this, should I put the text similar to the section above it?

Hii, after reviewing your Pr we came to a conclusion that we need to see the Figma for the design before you can start working on it. Do share the Figma in the issue itself. @DurgaPrasad-R


Should I use in Production?

Is it possible to use hotel in production? If yes, how should I go about it?

My recommendation is no. Because, it only works locally and setup it on network would give a lot of deal on dns settings. That's my tough.

Thanks for you input @indrakaw, but I wanted to use it just to manage (start and stop) all my servers (node and rails) in one place. If Hotel is not the most suitable for production, what may you recommend?

No do not use it in production
Supervisor (python)
PM2 (node)
Would be better options


Resolve #13

I replaced the force-unwrap of newSuperView with a guard clause to fix issue #13 .
Additionally, I replaced the series of if thing != nil with optional unwrapping.

Hey after i change the code same like you i still receive same issue

Looks like issue #13 was closed, but this branch has not yet been merged to master.


[6.5.x] GUVNOR-2503: Asset screen Validate-button should use incremental builder

See https://issues.jboss.org/browse/GUVNOR-2503
Mirror from: https://github.com/droolsjbpm/kie-wb-common/pull/421 (using JDK 7)

+1


Paymium : incomplete API implementation

The paymium API is missing the User part.
Only market data are available.

Maybe offer a bounty?


Your requirements could not be resolved to an installable set of packages.

Your requirements could not be resolved to an installable set of packages.
Problem 1
- teamtnt/tntsearch v1.0.6 requires ext-sqlite3 * -> the requested PHP extension sqlite3 is missing from your system.
- teamtnt/tntsearch v1.0.5 requires ext-sqlite3 * -> the requested PHP extension sqlite3 is missing from your system.
- teamtnt/tntsearch v1.0.4 requires ext-sqlite3 * -> the requested PHP extension sqlite3 is missing from your system.
- teamtnt/tntsearch v1.0.3 requires ext-sqlite3 * -> the requested PHP extension sqlite3 is missing from your system.
- teamtnt/tntsearch v1.0.2 requires ext-sqlite3 * -> the requested PHP extension sqlite3 is missing from your system.
- teamtnt/tntsearch v1.0.1 requires ext-sqlite3 * -> the requested PHP extension sqlite3 is missing from your system.
- teamtnt/tntsearch v1.0.0 requires ext-sqlite3 * -> the requested PHP extension sqlite3 is missing from your system.
- Installation request for teamtnt/tntsearch ^1.0 -> satisfiable by teamtnt/tntsearch[v1.0.0, v1.0.1, v1.0.2, v1.0.3, v1.0.4, v1.0.5, v1.0.6].
To enable extensions, verify that they are enabled in your .ini files:
- D:\xampp\php\php.ini
You can also run php --ini inside terminal to see which files are used by PHP in CLI mode.

It should be fixed. At this stage i am not doing Index based product search so yes i don't need this package.

Here i went to php.ini and uncommend extension=php_pdo_sqlite.dll it statrted working


Add support for python3 jobs

Current 1.0 RC 11 only supports python 2 jobs.

I think it depends only on system python version. Can you check python version on spark slaves and mist itself?
Also, can you provide code that is causing the error?

mist docker image is running with python 2.7 and this is not configurable.
But we found out that our worker execution code is not compatible with py3.6 version.
And also there is the issue in spark 2.1+. You can not run python 3.6 code withing spark 2.2 it will raise next error:
cls = _old_namedtuple(*args, **kwargs)
TypeError: namedtuple() missing 3 required keyword-only arguments: 'verbose', 'rename', and 'module'

SPARK-19019 shows as resolution fixed in Fix Version/s: 1.6.4, 2.0.3, 2.1.1, 2.2.0

I believe there are still issues with python3 support.  After applying the fix above, I tried running a job on a system where the system python is python3 and received the error message below.  My understanding is that iteritems was removed from python3.
"Error running job with JobParams(python_version.py,SimpleContext,Map(),execute). Type: java.lang.Exception, message: Error in python code: Traceback (most recent call last):\n  File \"/usr/share/mist/mist-worker.jar/__main__.py\", line 97, in <module>\n    result = instance.execute(**to_python_types(parameters))\n  File \"/usr/share/mist/mist-worker.jar/__main__.py\", line 28, in to_python_types\n    for key, value in any.iteritems():\n  File \"/usr/share/spark/python/lib/py4j-0.10.4-src.zip/py4j/java_gateway.py\", line 1133, in __call__\n    answer, self.gateway_client, self.target_id, self.name)\n  File \"/usr/share/spark/python/lib/py4j-0.10.4-src.zip/py4j/protocol.py\", line 323, in get_return_value\n    format(target_id, \".\", name, value))\npy4j.protocol.Py4JError: An error occurred while calling o29.iteritems. Trace:\npy4j.Py4JException: Method iteritems([]) does not exist\n\tat py4j.reflection.ReflectionEngine.getMethod(ReflectionEngine.java:318)\n\tat py4j.reflection.ReflectionEngine.getMethod(ReflectionEngine.java:326)\n\tat py4j.Gateway.invoke(Gateway.java:272)\n\tat py4j.commands.AbstractCommand.invokeMethod(AbstractCommand.java:132)\n\tat py4j.commands.CallCommand.execute(CallCommand.java:79)\n\tat py4j.GatewayConnection.run(GatewayConnection.java:214)\n\tat java.lang.Thread.run(Thread.java:748)\n\n\n, trace io.hydrosphere.mist.worker.runners.python.PythonRunner.run(PythonRunner.scala:57); io.hydrosphere.mist.worker.JobStarting$class.io$hydrosphere$mist$worker$JobStarting$$runJob(WorkerActor.scala:65); io.hydrosphere.mist.worker.JobStarting$$anonfun$1.apply(WorkerActor.scala:38); io.hydrosphere.mist.worker.JobStarting$$anonfun$1.apply(WorkerActor.scala:36); scala.util.Success$$anonfun$map$1.apply(Try.scala:237); scala.util.Try$.apply(Try.scala:192); scala.util.Success.map(Try.scala:237); scala.concurrent.Future$$anonfun$map$1.apply(Future.scala:237); scala.concurrent.Future$$anonfun$map$1.apply(Future.scala:237); scala.concurrent.impl.CallbackRunnable.run(Promise.scala:32); java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149); java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624); java.lang.Thread.run(Thread.java:748)"

from mist.mist_job import MistJob
import sys
import socket

class SimpleContext(MistJob):

    def execute(self):

        # foo = 'bar'
        # print(f'foo = {foo}')

        ret = dict()
        ret['version'] = sys.version_info[0]
        ret['interpreter'] = sys.executable
        ret['hostname'] = socket.gethostname()

        return ret

fixed in https://github.com/Hydrospheredata/mist/blob/v1.0.0-RC12/mist/worker/src/main/resources/main.py#L28

Thanks @blvp - will check now.  I made a mistake when refreshing my sources...


Upgrade Readiness blog url points to old URL

A link in the article Getting Started with Upgrade Readiness for blog entry at the current address details that the location of the blog has changed to https://aka.ms/blog/WindowsAnalytics (or https://techcommunity.microsoft.com/t5/Windows-Analytics-Blog/bg-p/WindowsAnalyticsBlog for the direct link).

@officedocsbot assign @jvsam

Hi @mm1nIPOVdDsu thank you for bringing this to our attention and we appreciate your patience on this matter. You are correct, the link redirects to the new Windows Analytics Blog site. I will forward this issue to the Windows writing team to get the link updated.
Thank you for engaging with the community!
Content: https://docs.microsoft.com/windows/deployment/upgrade/upgrade-readiness-get-started
Content source: https://github.com/MicrosoftDocs/windows-itpro-docs/blob/master/windows/deployment/upgrade/upgrade-readiness-get-started.md

Hi again @mm1nIPOVdDsu, the suggested update to this documentation via Pull Request (PR) #4224 has been approved and merged. It will be displayed at docs.microsoft.com in the next scheduled publishing run. You can verify the changes at this URL: https://docs.microsoft.com/windows/deployment/upgrade/upgrade-readiness-get-started. We will now close this issue but we'll provide an update here to inform you when the changes are live.
Thank you for your feedback to help improve the contents of this document and for being part of the Microsoft Docs community!

@officedocsbot close


Consider complimentary "build_filter" commit message parsing

Currently we have a big pain point that we cannot re-run a single "build" within a job on Appveyor.  I found the same true on Travis with parallel stages.  We can do it for Travis through the UI, but not appveyor.

Now that we are parsing commit messages with CPT, and applying filtering of which builds to run as a result, it seems feasible to extend it to be able to specify something like the following:
[build_filter=<query>]
where <query> uses the syntax of the conan search command for example:
[build_policy=os=Windows AND compiler.version=15 AND build_type=Release]
With this, we would have precision control over triggering rebuilds any CI system, regardless of it's internal featureset.
The code and unit testing to be added for query parsing and the filtering layer is non-trivial, so a lot of people would have to vote for this feature to justify it.  While non-trivial, it is certainly feasible, and we could potentially borrow some code from Conan client.
Note: I originally wrote this as an extension of build_policy, however I have since updated the suggestion to use a new keyword build_filter because the two are probably orthagonal.

I like the idea.

Also, another helper function that i am working with locally that others might use.  Again, don't have time to make a nice PR with tests yet.  Will make PR when i do.
def update_builds(builder, new_settings=None, new_options=None, new_env_vars=None, new_build_requires=None):
    updated_builds = []
    for settings, options, env_vars, build_requires, reference in builder.items:
        if new_settings:
            settings.update(new_settings)
        if new_options:
            options.update(new_options)
        if new_env_vars:
            env_vars.update(new_env_vars)
        if new_build_requires:
            build_requires.update(new_build_requires)
        updated_builds.append([settings, options, env_vars, build_requires])
    builder.builds = updated_builds


[BUG] ImportError: cannot import name 'QuestionAnswerPrompt' from 'llama_index' (/home/m1nd3xpand3r/Workspace/M1NDB0T-GPTDiscord/venv/lib/python3.9/site-packages/llama_index/init.py)

Successfully installed aiostream-0.5.2 dataclasses-json-0.5.14 deprecated-1.2.14 fsspec-2023.10.0 llama-index-0.9.4 nest-asyncio-1.5.8 pandas-2.1.3 pytz-2023.3.post1 tzdata-2023.3 urllib3-1.26.18 wrapt-1.16.0
(venv) m1nd3xpand3r@M1NDB0T:~/Workspace/M1NDB0T-GPTDiscord$ python3 gpt3discord.py
Loading environment from .env
Loading environment from /home/m1nd3xpand3r/Workspace/M1NDB0T-GPTDiscord/etc/environment
Loading environment from None
Attempting to retrieve the settings DB
Retrieved the settings DB
Traceback (most recent call last):
File "/home/m1nd3xpand3r/Workspace/M1NDB0T-GPTDiscord/gpt3discord.py", line 15, in
from cogs.search_service_cog import SearchService
File "/home/m1nd3xpand3r/Workspace/M1NDB0T-GPTDiscord/cogs/search_service_cog.py", line 49, in
from models.search_model import Search
File "/home/m1nd3xpand3r/Workspace/M1NDB0T-GPTDiscord/models/search_model.py", line 15, in
from llama_index import (
ImportError: cannot import name 'QuestionAnswerPrompt' from 'llama_index' (/home/m1nd3xpand3r/Workspace/M1NDB0T-GPTDiscord/venv/lib/python3.9/site-packages/llama_index/init.py)
(venv) m1nd3xpand3r@M1NDB0T:~/Workspace/M1NDB0T-GPTDiscord$
Not sure what I am missing installing everything needed

You somehow installed a too-high version of llama-index, did you install using the requirements.txt? Do python3.9 -m pip install -r requirements.txt in the directory of the project.
The latest llama-index version we support is llama-index==0.8.69.post1, you have installed 0.9.4


Untranslated Strings on Arcade Homepage

Some of the category titles are not translated on the homepage (see "Skillmap" and "Multiplayer Games" in screenshot below):

These are translated in crowdin.


Remove the deprecated pandas_wraps

This PR proposes to remote pandas_wraps as it's deprecated.
There are two more deprecated functions, map_in_pandas, and get_dtype_counts. The former is referred more publicity for example in https://databricks.com/blog/2020/03/31/10-minutes-from-pandas-to-koalas-on-apache-spark.html. I would like to delay the removal.
The latter is used in DataFrame.info internally when pandas < 1.0.0 is used. We can remove it when we use minimum pandas version as 1.0.0.

Codecov Report

Merging #1529 into master will decrease coverage by 2.00%.
The diff coverage is 87.50%.

@@            Coverage Diff             @@
##           master    #1529      +/-   ##
==========================================
- Coverage   93.97%   91.97%   -2.01%
==========================================
  Files          36       36
  Lines        8500     8409      -91
==========================================
- Hits         7988     7734     -254
- Misses        512      675     +163

Impacted Files
Coverage Δ

databricks/koalas/__init__.py
78.43% <ø> (-13.88%)
:arrow_down:

databricks/koalas/frame.py
92.25% <ø> (-3.20%)
:arrow_down:

databricks/koalas/generic.py
96.67% <ø> (ø)

databricks/koalas/typedef/typehints.py
85.26% <86.66%> (+4.18%)
:arrow_up:

databricks/koalas/typedef/__init__.py
100.00% <100.00%> (+15.78%)
:arrow_up:

databricks/koalas/usage_logging/__init__.py
24.32% <0.00%> (-70.28%)
:arrow_down:

databricks/koalas/usage_logging/usage_logger.py
50.00% <0.00%> (-50.00%)
:arrow_down:

databricks/conftest.py
88.67% <0.00%> (-7.55%)
:arrow_down:

... and 6 more

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 c972da8...73abe76. Read the comment docs.

Thanks! merging.


dead function

whyyy
const threadFeed = ig.feed.directThread({ thread_id: threadId });

return only 11 feeds

Please take a look at how to use feeds in the readme.


languages: add terraform language support

Missing the documentation entry, but I'll add it while I initialize the v0.6 changelogs.


Improve user documentation

Is your feature request related to a problem? Please describe.
The schema is difficult to approach as a new user.
Describe the solution you'd like
Improve the sphinx docs to outline major concepts. For example:

[ ] what is the difference between rig/instrument and acquisition/session?
[ ] ...

Looks like it will be tracked here: https://github.com/AllenNeuralDynamics/aind-data-schema/issues/624


Wouldn't you also need to define an ApplyLeft and ApplicativeLeft to enable Flip to have a MonadLeft instance?

Ran across this today as I was looking for a bindLeft in a project. Would be useful!

Maybe this would be better moved to the 'qualified do' package?


Updating to latest of strimzi, via upstream

Signed-off-by: Matthias Wessendorf<EMAIL_ADDRESS>Update once upstream PR 1039 is merged
/hold
/assign @pierDipi

/unhold


xilem_masonry: Fix Flex splice (increment ix by 2 because of spacer widget)

While adding filters to the TodoMVC example, I produced a crash. Which seems to originate because there was a spacer added, which also increments the underlying element child count.
So in case we want to have that spacer, we need to increment by 2 instead...
(Since mutate and delete skip spacers, because child_mut returns none, and the index is only incremented for concrete children it's not necessary to change the index incrementation code there)

Just gut-checking a policy thing here - I think I would have expected @Philipp-M to activate the merge here, as the PR author.

Generally yes but I think we can speed things along a bit right now.

Right, I sped this one along because we agreed that bug blocked other work, but maybe I overreached a bit.


Simon Paul

Email
<EMAIL_ADDRESS>Bio
I'm an COBOL analyst programmer with an experience of 8.8 years. Looking forward to learn new things and showcase my expertise to the best of the community.
Projects within OMP you would want to be an ambassador for
COBOL related project's.
List any other topics you would be able to speak about
Cloud and COBOL integration.
Why do you want to be an ambassador?
COBOL is an age old language, but itd legacy code is still dominating the business. So, always wanted to learn more and explore new things.
Attach a headshot
Simon

Application NOT accepted by the TAC