new API to set multicast loop v4 for ServiceDaemon

This patch is trying to solve issue #278 . I'm curious if this new API can support what was asked:
/// Enable or disable the loopback for locally sent multicast packets.
///
/// When disabled, a local querier will not receive announcements from a local responder.
pub fn set_multicast_loop_v4(&self, on: bool) -> Result<()>

So I finally got the time to look into this. For some reason, I wasn't able to reproduce the effect. Here are the info logs that I collected and cleaned up.
Received other event: ServiceFound("_myservice._tcp.local.", "arijit-laptop._myservice._tcp.local.")

service resolved: ServiceInfo { ty_domain: "_myservice._tcp.local.", sub_domain: None, fullname: "arijit-laptop._myservice._tcp.local.", server: "arijit-laptop.local.", addresses: {<IP_ADDRESS>, fe80::dfed:529c:84ef:d24a}, port: 10116, host_ttl: 120, other_ttl: 4500, priority: 0, weight: 0, txt_properties: TxtProperties { properties: [] }, addr_auto: false, status: {}, requires_probe: true }

service resolved: ServiceInfo { ty_domain: "_myservice._tcp.local.", sub_domain: None, fullname: "arijit-laptop._myservice._tcp.local.", server: "arijit-laptop.local.", addresses: {fe80::dfed:529c:84ef:d24a, <IP_ADDRESS>}, port: 10116, host_ttl: 120, other_ttl: 4500, priority: 0, weight: 0, txt_properties: TxtProperties { properties: [] }, addr_auto: false, status: {}, requires_probe: true }

service resolved: ServiceInfo { ty_domain: "_myservice._tcp.local.", sub_domain: None, fullname: "arijit-laptop._myservice._tcp.local.", server: "arijit-laptop.local.", addresses: {fe80::dfed:529c:84ef:d24a, 2409:40e5:167:6011:3b95:d952:c425:3f82, <IP_ADDRESS>}, port: 10116, host_ttl: 120, other_ttl: 4500, priority: 0, weight: 0, txt_properties: TxtProperties { properties: [] }, addr_auto: false, status: {}, requires_probe: true }

service resolved: ServiceInfo { ty_domain: "_myservice._tcp.local.", sub_domain: None, fullname: "arijit-laptop._myservice._tcp.local.", server: "arijit-laptop.local.", addresses: {2409:40e5:167:6011:3b95:d952:c425:3f82, <IP_ADDRESS>, fe80::dfed:529c:84ef:d24a}, port: 10116, host_ttl: 120, other_ttl: 4500, priority: 0, weight: 0, txt_properties: TxtProperties { properties: [] }, addr_auto: false, status: {}, requires_probe: true }

service resolved: ServiceInfo { ty_domain: "_myservice._tcp.local.", sub_domain: None, fullname: "arijit-laptop._myservice._tcp.local.", server: "arijit-laptop.local.", addresses: {fe80::dfed:529c:84ef:d24a, 2409:40e5:167:6011:3b95:d952:c425:3f82, <IP_ADDRESS>}, port: 10116, host_ttl: 120, other_ttl: 4500, priority: 0, weight: 0, txt_properties: TxtProperties { properties: [] }, addr_auto: false, status: {}, requires_probe: true }

Hope this helps, if you want more verbose logs feel free to hit me.
Thanks

Oops my bad, I didn't look into the changes and assumed that this option should be enabled by default. Anyways, I disabled both the IPv4 and IPv6 loopbacks and everything seems perfect.
Thanks for working on this.

Not a problem, thank you for trying it out! That's very helpful! I will merge this API.


cilk_scope vs cilk_sync

When might we switch our tutorial paradigm from cilk_sync to the new keyword, cilk_scope? (I think that is the new keyword, anyway, for Cilk blocks.
To introduce OpenCilk with cilk_scope instead of cilk_sync, we need to patch the places that use cilk_sync. These include

[ ] Home page "Write fast code" image and "Teach parallelism" image
[ ] Convert a C++ program
[ ] OpenCilk keywords

But we probably would not change the PDFs and videos for the tutorials from SPAA and PACT.

I linked this issue to PR #142 because that will fix the home page images. The other documentation pages are in various stages of being updated, all with attention to cilk_scope.


Make the grace period configurable

A simple PR to make the grace period configurable:
EddystoneBeaconScanner.startScanning(true, 30000); // Increase the grace period to 30s

The default value is still of 5 seconds.

@ValentinGot thanks for submitting, I've added some suggestions above.

@sandeepmistry You're right, it's cleaner :)

@ValentinGot thanks!


hide lines of code - angular 7

Is there any way to hide code lines in the control in angular7? I could not implement setHiddenArea()

@davidndigitalware Can you provide a reproducible sample that executes at https://microsoft.github.io/monaco-editor/playground.html for us to investigate?

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


postgres: fix users and roles

The users and roles get confused between each other in initialization script

Yeah, thats is true


Delete database reference

This adds support for deleting database references. Also fixes system db references being read from .sqlproj files as dacpac references (previously didn't care because just the name needed to be shown in the tree, but now it matters because the SSDT system db reference needs to get deleted too)

Pull Request Test Coverage Report for Build 265533223

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

Totals

Change from base Build 265522820:
0.0%

Covered Lines:
19153

Relevant Lines:
45789

💛  - Coveralls


will this paginate / have a scroll body?

will this table just endlessly grow with saved filters or will it be a fixed height scroll? doesn't look to be using dataTables

It will paginate: https://github.com/eastgenomics/trendyQC/blob/local_dev/trendyqc/trend_monitoring/views.py#L144


Grant Type customization

Hi just discovered the remix-auth package and was going to start digging deep, trying to get my grant (password based) I noticed that the grant_type is hard coded to be authorization_code while I'm using a password grant, is there any chance I can change the grant_type in the configuration?

Looking for the same.


we want web demo

pleeeeeeeeeeeeeeeeeeeeeeeeeeeeease!!

For now, a web demo is not possible, because it doesn't satisfy the OpenAI terms of service.

What terms does it break?


Multi-tenant Pelorus

Filter Pelorus dashboards such that users can see data for the projects in which they have read permissions.

Continue delegating to cluster authentication
Don't require cluster read
Populate dashboards based on current permission levels

Enhance (or create separate) dashboard to be able to drill down to metrics for an individual app


post order request

Could you add a function that allows one to place post order requests with ticker symbol, quantity, buy/sell, market/limit? I didn't see any such function listed, and I am not sure how to program such a function myself. I'm not familiar enough with the requests module to do this. I'd like to be able to use your code to place buy and sell orders.

@jr751 Hey, I haven't worked on this project in a while, but you're the second person who has taken an interest.  Since you've proposed a real feature, I'll take a look at implementing it this weekend.  Will update you here.

Thank you. I'd definitely use it if you could make it happen.

mskymoore, did you make any progress in adding this new feature?


Accordion

This is the new design, fresh off the baking table. We would like to see if it makes sense before we do an official update.

:tada: This issue has been resolved in version 3.0.0-pre.3 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 3.0.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


FAB list buttons color input not working since RC3

Short description of the problem:
Buttons inside an ion-fab-list should use color correctly. It's a css issue because the color specific styles are there, but are overridden. In RC.2 this was working fine.
<ion-fab right bottom>
  <button ion-fab color="primary"><!-- This button shows blue correctly -->
    <ion-icon name="add"></ion-icon>
  </button>

  <ion-fab-list side="top">
    <button ion-fab color="primary"><!-- This button shows grey instead of blue -->
      <ion-icon name="checkmark"></ion-icon>
    </button>
    <button ion-fab color="secondary"><!-- This button shows grey instead of green -->
      <ion-icon name="close"></ion-icon>
    </button>
  </ion-fab-list>
</ion-fab>

Which Ionic Version? 1.x or 2.x
2.0.0-rc.3
Plunker that shows an example of your issue
http://plnkr.co/edit/IgQLki?p=preview
Run ionic info from terminal/cmd prompt: (paste output below)
Cordova CLI: 6.4.0
Ionic Framework Version: 2.0.0-rc.3
Ionic CLI Version: 2.1.12
Ionic App Lib Version: 2.1.7
Ionic App Scripts Version: 0.0.45
ios-deploy version: 1.9.0
ios-sim version: 5.0.11
OS: macOS Sierra
Node Version: v6.9.1
Xcode version: Xcode 8.1 Build version 8B62

Fixed! thanks!


Move state outside of AddStoreVariableStatements.

This allows us to call AddStoreVariableStatements methods while holding a reference to variables in the state.

This change is

crates/cairo-lang-sierra-generator/src/store_variables/test.rs line 597 at r1 (raw file):

    let statements: Vec<pre_sierra::Statement> = vec![
        dummy_push_values(&db, &[("0", "100"), ("1", "101")]),
        dummy_push_values(&db, &[("100", "200"), ("101", "201"), ("2", "202"), ("3", "203")]),

Let me know if you want me to try to create a smaller PR that catches that issue.
I wasn't able to do it in 10 min :)
Code quote:
  dummy_push_values(&db, &[("100", "200"), ("101", "201"), ("2", "202"), ("3", "203")]),


FIX: Minor fixes to plotting

This allows both colormap and clim to be 'auto'. This way I get a plot using mne_analyze colormap with this:
stc.plot(lims=dict(pos_lims=(10, 50, 90)))

and a plot with hot colormap I get this:
stc.plot(lims=dict(lims=(10, 50, 90)))

@wronk please have a look and see if you're happy.

I also dropped the language about PySurfer 0.4 regarding views because it has been out for almost two years now.

cc @rkmaddox if you have time you could pull and test and see if it works for your code the way you expect

Open question: should the default for stc.plot() going forward be to use mne_analyze colormap with positive and negative limits, or hot colormap with only positive? I have more use cases and I think it's probably more common for mne_analyze to be the default, so I'd like to have it be that. This is a good opportunity to change it if we want to...

I think it comes down to what inverse method is used most often of MNE, dSPM, and sLORETA. Personally, mne_analyze would make more sense for me too.

Coverage decreased (-0.06%) to 90.24% when pulling 23204ac73cd88ef021ef3e121bc2338a7f9a7a20 on Eric89GXL:plot-fix into f2ab0641f087c63beab715ff5fdaa9399c0807fe on mne-tools:master.

does it affect any of our examples?

Yep @Eric89GXL, looks to do what it should do with properly interacting clim and colormap parameters. Thanks.

@agramfort the changes are designed to be backward-compatible, but the changes do allow us to simplify our examples. They have now been updated.

thanks @Eric89GXL !


Question: Docstring

How did you change the docstring at the top of your files to look different than the default?
// Kevin Li - 4:07 PM - 7/2/20

https://useyourloaf.com/blog/changing-xcode-header-comment/


added pd.Index with tests in common.pyi

[x] Closes #415
[x] Tests added: Please use assert_type() to assert the type of any return value

Sir I had one doubt should the script test/nighly should be changed so that it's upgrades automatically for newer version of pip

Sir I had one doubt should the script test/nighly should be changed so that it's upgrades automatically for newer version of pip

Just merged in #421 that fixes the nightly issue.  So you can merge with latest master to fix that.

One small change - you duplicated one test.  Otherwise looks good

Yes sir I'll fix it

Done the change sir

Thanks @ramvikrams

Thanks sir


VisibleOnScreenNotifier2D events aren't triggered while the window is minimized.

Godot version
4.1.2
System information
Windows 10, Godot 4.1.2.stable.official - Vulkan
Issue description
I have an object moving across the screen on a scene, which comes into the visibility of the screen and then disappears from the visibility area.
The events work correctly when the window is visible, but as soon as the window is minimized and disappears from the screen, the events from VisibleOnScreenNotifier2D stop working completely.
Perhaps it's because nothing is drawn when the window is minimized, but I don't understand how to replace these events.
Steps to reproduce
extends VisibleOnScreenNotifier2D

func _ready():
	connect("screen_entered", Callable(self, "_screen_entered"))
	connect("screen_exited", Callable(self, "_screen_exited"))

func _process(delta):
	position += Vector2.DOWN * delta * 500.0
	if position.y >= get_viewport_rect().size.y * 2:
		position.y = 0.0

func _screen_exited():
	print("on screen")

func _screen_entered():
	print("out of screen")

Minimal reproduction project
test_proj_gd4.zip

I'm not sure this is a bug, I'd say it's expected as the visibility checks here are rendering based so can't happen if the window isn't rendered

I think we should write in the documentation that these events are triggered only when the window is visible / renderer is active.
Maybe we need some other way to get such events that do not depend on the renderer?

That's not possible unless we add a lot of special case code that wouldn't be used in most projects
It should indeed be documented, but it does say it depends on the renderer already

What can you recommend as a workaround? Check the intersection of viewport rectangle and object rectangle on the scene?

No clue, that could work, but I'd suggest that you shouldn't try processing visual things when not visible, and that any workaround is risky, I don't see the logic in having this kind of thing work when the game is minimised, or any logic for that matter

Also related:

https://github.com/godotengine/godot/issues/83971

Thanks for your help. These events were not very critical, and the engine knows how to optimize rendering behind the screen anyway.
I think we can close this issue if nobody needs it anymore, and since it is expected behavior of the engine.

I'd suggest using an Area2D with the camera for this
Closing per the above
Thank you for discussing this 🙂


[Question] How to deal with BottomNavigation in compose?

I'm currently practicing with the ftue sample as reference. Already create unauthenticated navugation, but I want my authenticated first page contains BottomNavigation (compose), which every of the tab contains its own stack (like youtube or instagram).
How to achieve that? Is the nested-navigation sample can be user for that?

Hello, that is correct, you would create a BackstackHost to have a stack per each bottom nav tab, like in nested navigation example.

I'm running out ideas to make it work with 4 bottomNav and have history when navigate between 4 of them. Example bottomNav: HOME, EXPLORE, WHISHLIST, PROFILE.
How to keep history when user navigate: HOME -> EXPLORE -> WHISLIST -> PROFILE, after user press back, it will goes backwards like from PROFILE back to WHISLIST back to EXPLORE back to HOME and then Exit the app?
Do you have example for this?

I haven't had any time to make an example, even though technically your request is totally viable.
In our projects, this is why I still use fragments instead, but I do know it has become "uncool" to do so these days.


Add -stdin and -stdout command line options

Not sure if this fits in with your preferred CLI style, so I'd be happy to make any changes you'd prefer. For https://github.com/ditinc/linter-cflint, I needed to take input from stdin and output to stdout.
Opening pull request to discuss.

I'm good with the direction you went, with the following caveats.

I don't think you need the stdout CL parm, it defaults to std if you don't specify a textOutFile name
the CFLintMain file you patched is pretty out of date :-)
I'm inclined to assume stdin if there is no -folder or -file specified.

What do you think?

Agreed on all three points - is dev the current branch? I can patch that one instead.

yes you can start with what's in the dev branch
On Thu, Oct 8, 2015 at 9:23 AM, Stephen Matta<EMAIL_ADDRESS>wrote:

Closed #83 https://github.com/cflint/CFLint/pull/83.
—
Reply to this email directly or view it on GitHub
https://github.com/cflint/CFLint/pull/83#event-430298197.


feat: make vite work in Bun

Description
There are two small issues that are blocking Vite from working in Bun, which is much easier to fix in the Vite project itself.

Bun implements it's own version of ws, which gets overridden when doing import {...} from "ws". Vite bundles it's own copy of ws which attempts to use req.socket.write which we do not implement as we do not use a node stream for the underlying network sockets. Implementing the APIs needed for the JS implementation of ws would take too long and be much slower.

During config loading there is a directory caching bug, the workaround we are using is applying a query string parameter to the file:// url used when loading the config through esm.

That is all that it takes to get Vite working on Bun, though it depends on a handful of various fixes we recently made to our node compatibility. To test these changes, you need to use the canary version bun upgrade --canary (or 0.7 when released), and then use bun --bun run dev (--bun forces bun's runtime over node's).
Closes https://github.com/oven-sh/bun/issues/250

Additional context

What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[ ] Documentation update
[X] Other

Before submitting the PR, please make sure you do the following

[X] Read the Contributing Guidelines.
[X] Read the Pull Request Guidelines and follow the PR Title Convention.
[X] Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
[X] Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
[ ] Ideally, include relevant tests that fail without this PR but pass with it.

@paperdave thanks for the PR. Would you separate the directory caching bug part on a separate PR so we can better track both changes? (and revert this one later if we go forward with it). We would like to know for this part what are your plans, as it seems we are patching a bug that may be solved soon in Bun.
This PR could remain about Bun's own version of ws, which I think we can merge without much discussion.

Load config with Bun without deps restart trigger. Bun users have
slightly different experience.

In Bun, you can do Loader.getDependencyKeysIfEvaluated and it will call
this function which returns an array of module specifiers a module depends
on
https://github.com/WebKit/WebKit/blob/2f688c542e6be822158997052c6fe6f6f63c4c16/Source/JavaScriptCore/builtins/ModuleLoader.js#L593
I don’t feel good we expose this JSC internal api so directly because they
may change it randomly later, but we can expose a wrapper for this
Another thing in Bun is that require.cache can be used to invalidate ESM
modules. require.cache is implemented as a Proxy which merges the real
CommonJS module cache and the ESM registry cache into one
On Thu, Jul 20, 2023 at 6:02 AM Bjorn Lu @.***> wrote:

@.**** commented on this pull request.
In packages/vite/src/node/config.ts
https://github.com/vitejs/vite/pull/13901#discussion_r1269428864:

// Bun will cache file/directories, and since we just created the file it might throw a ResolveError

// Adding a query string with the ? will bust the cache.
// See https://github.com/oven-sh/bun/issues/3216
const fileUrl = ${pathToFileURL(fileBase)}.mjs?

That would lose slight parity with the current config loader, as it tracks
the dependencies of the config to trigger a server restart. Personally, I
think this "feature" has blocked us from simplifying a lot of things that
I'd rather not have it (around Vite 5? Not sure yet).
So I guess we have two options:

Use ? cache bust, but Bun should fix this eventually.
Load config with Bun without deps restart trigger. Bun users have
slightly different experience.

—
Reply to this email directly, view it on GitHub
https://github.com/vitejs/vite/pull/13901#discussion_r1269428864, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFNGS4MW66HCNIZNBTEYVLXRET5NANCNFSM6AAAAAA2QYJQQQ
.
You are receiving this because you commented.Message ID:
@.***>

thanks for the PR. Would you separate the directory caching bug part on a separate PR so we can better track both changes? (and revert this one later if we go forward with it). We would like to know for this part what are your plans, as it seems we are patching a bug that may be solved soon in Bun.
This PR could remain about Bun's own version of ws, which I think we can merge without much discussion.

I think that is reasonable. I'll split this into two PRs.

the ? thing has been fixed in bun with a less-than-ideal solution but it will work for this. this should also mean older versions of vite will get past config loading, but only break on hmr.

Thanks for moving the ? workaround to Bun's side @paperdave. We released this PR as part of<EMAIL_ADDRESS>

legendary. were gonna release bun 0.7 tomorrow (i think) so that will be when users can test it out.


Update client-libraries.md

Add community PHP client.

I signed it

/assign @kbarnard10

/lgtm
/approve


Switch the reactor to use a HashMap

Description
This converts the sinks to use a HashMap instead of a list, as described in the below issue. This also removes a few unnecessary clones and changes the formatting on some defaults, just for the hell of it.
Link to issue
https://github.com/RhizomeDB/rs-rhizome/issues/25
Type of change

[x] Refactor (non-breaking change that updates existing functionality)

Test plan (required)
I ran cargo test --workspace and got all green on my machine. I assume this is sufficient as there should be no observable API changes with this refactoring, just a slight speedup in on certain workloads.

Once the CI finishes I'll merge this, but this look great, thank you!

I just noticed that these commits aren't signed! Are you able to set commit signing up? The Github docs for that are here: https://docs.github.com/en/authentication/managing-commit-signature-verification/signing-commits

Thank you! 🎉🎉🎉


Display for Increase Rates

it would really help, if you would add a display of the production rate of every dimension and not just the first dimension

Closed by something in the last year or so.


Code Profiling Review (Days 3, 2022)

The notebook is brief and touches on all intended topics very clearly, in sufficient depth, and without fluff. There is absolutely no assumed knowledge, except possibly the one-time fleeting mention of MKL/BLAS.
"If you speed up something that takes 50% of the runtime by a factor of 100, you speed up the code by a factor of 2." I feel saying "speed up by a factor of 100" could lead to unrealistic expectations and aims, especially since this is the first time we are mentioning speeding up a part of the code. A more mellow example like "speeding it up by a factor of 2 will reduce runtime by 25%" will drive home the same point.
The example under "runtime" in "Code Performance Extrapolation" seems too idealized, so it may be helpful to mention that there will most likely be deviations, especially at lower values of N, and hence it is advisable to test a fairly broad range on N.

Thanks @Devagio! Just pushed some changes in commit 6da1c5 that incorporated your suggestions!


Get reviews

TC39-appointed reviewers:

[x] @ajklein
[x] @dherman
[x] @wycats

Also the editor:

[x] @bterlson

Why do you insist on randomly overloading language keywords with random things for your own pleasure?
First, let's make import a not-really-a-keyword-not-really-a-function abomination.
Now let's randomly add properties to it. Because reasons. What happened to sitting down and speccing out and designing things?

I've deleted a comment that was off-topic for this thread (and fairly disrespectful, to boot). Looking forward to getting reviews for this feature.

I realize that community input means next to nothing. Still, it would be really nice to hear the reaons for this, why it's already stage 2, and what's the rationale behind randomly extending keywords with random properties

Getting accepted in stage 1 and moving to stage 2 means:

The committee expects to devote time to examining the problem space, solutions and cross-cutting concerns

So. Problem space is defined. There are concerns. How are they going to be addressed?

I'm deleting more off-topic comments from @dmitriid, as he does not seem to understand this is a thread dedicated to getting reviews from the TC39-appointed reviewers.
@dmitriid, please email es-discuss with your question, as it is off-topic for this repository and are about more general issues.
I guess I'll lock this thread as it seems to be attracting more off-topic noise and the reviews will have comment privileges anyway.

:+1:

All done, stage 3, woo!! @bterlson, you mentioned some editorial issues, please feel free to file :)


change behavior of multi-words search from OR to AND

change behavior of multi-words search from OR to AND.
/cc @dusmart

Thank you for taking advice! Thanks for your theme. It's pretty good.


MINOR: Adding kafka-storage.bat file (similar to kafka-storage.sh) fo…

…r windows.
kafka-storage.sh was added as part of pull request #10043. But in windows, the .bat file was not included and hence caused issue while starting server using kraft

Hi @mumrah Could this be merged?


MISP Action

Would be nice to be able to send alerts to MISP.

It would be awesome to have a new action which sends found pastes to a MISP instance.
To solve this issue there needs to be a new action added in the actions directory. The action must follow the example of the other actions in this directory. Don't forget to add the action to the action package file.
The most simple example for such an action is the GenericAction which executes a passed function.
The action must be able to send a new message to an existing MISP instance. Not using additional packages is highly preferred to keep the size of the application as small as possible.
Tests are not necessary but highly appreciated. If there are questions, don't hesitate to contact me.

I'd like to tackle this one. I've almost managed to get MISP to run in Docker, hopefully it works and then I can try out my code - automated tests won't be possible (by me at least) though.
I have questions regarding the data output:
{
    "Event":{
        "date": "2015-01-01",
        "threat_level_id": "1",
        "info": "testevent",
        "published": false,
        "analysis": "0",
        "distribution":"0",
        "Attribute": [{
            "type": "domain",
            "category":"Network activity",
            "to_ids": false,
            "distribution":"0",
            "comment":"",
            "value":"test.com"
        }]
    }
}

This is the sample event object in the MISP docs
that has to be passed to their API, but I have no idea what to specify for a lot of the values - I could use some input on that part :)

Work in progress in misp-action
I made the action have a transformer parameter which can be used to specify a function to transform a Paste into a MISP event object, still have to build a default one but it's on its way.

@Zeroji Hi, thanks for the work. I don't have time to check it currently. Will try to have a look tonight!

Hey, just seen your EDIT. Yes, you should extend BasicAction. The text does not say that you should extend GenericAction, I just wanted to say that it's the easiest (working) Action I built. But yes, I should have added what base class to extend :)
I will check out your PR tomorrow. Had a stressful day today. Thanks for participating.

No worries, take your time! However for your own sake try to find a running MISP instance, because it took Docker about an hour and a half to build mine :stuck_out_tongue:


Allow hiera config for dotfiles

Thanks @PuppetNinja
The tests that are failing are not related to your PR. I am working on fixing them in PR #345

Hi @shivapoudel @stankevich
Would one of you mind testing this PR to ensure it does what it is supposed to? I'd like to get confirmation from someone before merging.


Code is not working properly

Context: https://tour.golang.org/flowcontrol/10
The day that I ran the code was Friday but code block shows as Tuesday. Wrong server time  ?

Sorry @haggucan This issue was closed due to a bug in the tourbo-gopherbot.
The issue you're seeing is due to how the tour is run, I don't know the details but the environment in which these programs are executed is deterministic, so it's always Tuesday in the tour. :smile:

@robphoenix
Thanks for the reply mate :)


Switch to broker.serializer instead of JSON-only solution

The code works well but contains a dirty hack for parsing pipeline data from task labels.
Pipeline loads()/dumps() are loadb()/dumpb() now.
Step's loads()/dumps() are gone away, seems like they don't override anything actually but all have the same code.
I'm not sure if the PR should be merged as is.
@s3rius please consider it as the idea demonstration.
If you agree with the code in general, I'll make bytes support to message labels first, and remove the hack later.
P.S.  Sorry for two-week delay. Tests were hanging without any exception or log message. It took a while to figure out how to debug the problem.

I also had no time to review it, sorry. Was busy as a bee on work.

I also had no time to review it, sorry. Was busy as a bee on work.

Don't rush please, a few days don't change the university :)


[ Bug ] Strip Product seeding not working

so when you do:
npx prisma migrate dev --name init

it always skipping with 🏃♂ Skipping Stripe products creation and seeding.
not sure if this intended, will file a PR for this

Hello @marsch.
As far as I remember, that's intended, as it has a custom message for it. Ideally, we want to do one of the following things before migrating/re-seeding:

Clean up our Stripe data (including plans).
Delete our database plans and seed those again with the available Stripe data (plans).

As the project is still part of a previous code, the first option was selected. In order to properly do migrations again, we will need to clean our Stripe data, as new plans will be created and stored in our database.
Nowadays, I personally follow the second approach:

Stripe plans are not being re-created; rather, we clean up our Stripe database plans and seed those again with the available Stripe data (plans).

So yeah, in the current code, we are taking the first approach, which requires clean Stripe data in order to properly run the migrations. That's also specified in the docs.

With that said, I have in mind to come up with a new version of Remix SaaS, and the second approach will be the main one, as it simplifies the seeding/migration process a lot.


chore: test

Pull Request Test Coverage Report for Build 109297225

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

Totals

Change from base Build 109271940:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls

Pull Request Test Coverage Report for Build 109297225

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

Totals

Change from base Build 109271940:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls


Policy cannot be clicked after creating

This happened to me with build from 0911. When I created a policy, the tree was not updated so it did not appear there. And also the entry in table was not clickable, only those that existed before. When you go away and return, it works normally. Seems like a missing UI update.

Right, we're replacing a wrong div (policy_tree_tree_div instead of policy_tree_div)...
(Affects all 7 trees under Control/Explorer.)


Fix Rails 7.1 compatibility.

Fixes https://github.com/bullet-train-co/jbuilder-schema/issues/54
Rails 7.1 has sped up its internal OutputBuffer implementation (what handles << concatenation in templates), which we rely on tricking a bit. https://github.com/rails/rails/pull/45614
Then a follow up PR the to_s call was moved to after calling view._run https://github.com/rails/rails/pull/45756
However, we rely on the return value being a Hash (which I'm not sure how to fix yet).
So since target! is the return value of our templates, we can wrap that and ensure to_s doesn't actually to_s, which Rails 7.1 will call for us.
On Rails < 7.1 we must call the compatibility unwrap_target! method ourselves to get our inner Hash return value.
This also adds a Rails version matrix so we can detect these things before release in the future.

I'm updating our CI scripts so they're a little closer to Showcase's where I'm copying from https://github.com/bullet-train-co/showcase/blob/fd9bd359326facc003fa5dfb2587f4427522d308/.github/workflows/ci.yml


pull request for e2e test at 01/27/2017 09:15:17

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


DisableLinterReason needs to be whitespace/linebreak-aware

The following fails to trigger the DisableLinterReason linter.
// Some Sectioning Comment
// ---------------------------------------------

// scss-lint:disable DeclarationOrder
.Example {}

Note the format of the preceding comment is irrelevant. The point is that this comment isn't explaining the reason behind using the linter comment, and this should be clear given the whitespace/linebreak.
scss-lint 0.42.2

Check disable_linter_reason_spec.rb
  context 'when a reason precedes a disabling comment, at a distance' do
    let(:scss) { <<-SCSS }
      // We like using `border: none` in our CSS.

      // scss-lint:disable BorderZero
      p {
        margin: 0;
      }
    SCSS

    it { should_not report_lint }
  end

It seems that it is valid to give explanation separated by line break.

Thanks for pointing out that spec, @sonalkr132.
Yes, it is allowed to separate the reason with a blank line.

FWIW I think this is not desirable behaviour. It would make more sense to assume the comment is not broken, otherwise any unrelated comment will case the linter to pass it.


More precise image downloading.

Adding option to be more precise when downloading image:
For example: getting only image with resolution bigger than 1000x1000, resolution is exactly 1920x1080, filename contains "IMG_" etc...

I think being able to tell 4scanner to download only a certain file extension could be useful. Example: download only webm.

You can now download images by width and height and file extension in #27

Searching by filename was also added in 1.2.3


Add max pods

Hi team!
This PR includes max-pods attribute on each node pool. This way you can configure the maximum number of pods per node in Azure AKS.
As this is part of the shared interface, wait for the rest of cloud installers :)
If you specify a value, it will be used, otherwise, specify null value to use the default behaviour witch in our offering was 250 pods per node.

Beware that a change on max_pods in the default_node pool cause (i think reading the docs) a recreation of the kubernetes cluster. Is default_node_pool mandatory?

Beware that a change on max_pods in the default_node pool cause (i think reading the docs) a recreation of the kubernetes cluster. Is default_node_pool mandatory?

Ye. You have to be careful about doing this modification. Before applying anything, review carefully the terraform plan output.


Backport "HBASE-26765 Minor refactor of async scanning code (#4121)" to branch-2

Signed-off-by: Duo Zhang<EMAIL_ADDRESS>

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 43s
Docker mode activated.

-0 :warning:
yetus
0m  7s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
3m 26s
branch-2 passed

+1 :green_heart:
compile
0m 21s
branch-2 passed

+1 :green_heart:
shadedjars
4m 40s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 19s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
3m  0s
the patch passed

+1 :green_heart:
compile
0m 20s
the patch passed

+1 :green_heart:
javac
0m 20s
the patch passed

+1 :green_heart:
shadedjars
4m 13s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 16s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
3m 20s
hbase-client in the patch passed.

22m 44s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4134/1/artifact/yetus-jdk8-hadoop2-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4134

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux fd82e66867a4 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 4cdb380ccc

Default Java
AdoptOpenJDK-1.8.0_282-b08

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4134/1/testReport/

Max. process+thread count
142 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4134/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 50s
Docker mode activated.

-0 :warning:
yetus
0m  5s
Unprocessed flag(s): --brief-report-file --spotbugs-strict-precheck --whitespace-eol-ignore-list --whitespace-tabs-ignore-list --quick-hadoopcheck

_ Prechecks _

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
3m 24s
branch-2 passed

+1 :green_heart:
compile
0m 19s
branch-2 passed

+1 :green_heart:
shadedjars
4m 48s
branch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 19s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 55s
the patch passed

+1 :green_heart:
compile
0m 19s
the patch passed

+1 :green_heart:
javac
0m 19s
the patch passed

+1 :green_heart:
shadedjars
4m 47s
patch has no errors when building our shaded downstream artifacts.

+1 :green_heart:
javadoc
0m 16s
the patch passed

_ Other Tests _

+1 :green_heart:
unit
3m 26s
hbase-client in the patch passed.

23m 31s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4134/1/artifact/yetus-jdk11-hadoop3-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4134

Optional Tests
javac javadoc unit shadedjars compile

uname
Linux aaaf14e049dc 5.4.0-1025-aws #25~18.04.1-Ubuntu SMP Fri Sep 11 12:03:04 UTC 2020 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 4cdb380ccc

Default Java
AdoptOpenJDK-11.0.10+9

Test Results
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4134/1/testReport/

Max. process+thread count
157 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4134/1/console

versions
git=2.17.1 maven=3.6.3

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
1m 41s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
hbaseanti
0m  0s
Patch does not have any anti-patterns.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ branch-2 Compile Tests _

+1 :green_heart:
mvninstall
3m 28s
branch-2 passed

+1 :green_heart:
compile
0m 55s
branch-2 passed

+1 :green_heart:
checkstyle
0m 23s
branch-2 passed

+1 :green_heart:
spotbugs
0m 54s
branch-2 passed

_ Patch Compile Tests _

+1 :green_heart:
mvninstall
2m 36s
the patch passed

+1 :green_heart:
compile
0m 59s
the patch passed

+1 :green_heart:
javac
0m 59s
the patch passed

+1 :green_heart:
checkstyle
0m 25s
the patch passed

+1 :green_heart:
whitespace
0m  1s
The patch has no whitespace issues.

+1 :green_heart:
hadoopcheck
8m 39s
Patch does not cause any errors with Hadoop 3.1.2 3.2.1.

+1 :green_heart:
spotbugs
0m 49s
the patch passed

_ Other Tests _

+1 :green_heart:
asflicense
0m 11s
The patch does not generate ASF License warnings.

26m  0s

Subsystem
Report/Notes

Docker
ClientAPI=1.41 ServerAPI=1.41 base: https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4134/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/hbase/pull/4134

Optional Tests
dupname asflicense javac spotbugs hadoopcheck hbaseanti checkstyle compile

uname
Linux 5f7405f83217 5.4.0-90-generic #101-Ubuntu SMP Fri Oct 15 20:00:55 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev-support/hbase-personality.sh

git revision
branch-2 / 4cdb380ccc

Default Java
AdoptOpenJDK-1.8.0_282-b08

Max. process+thread count
60 (vs. ulimit of 12500)

modules
C: hbase-client U: hbase-client

Console output
https://ci-hbase.apache.org/job/HBase-PreCommit-GitHub-PR/job/PR-4134/1/console

versions
git=2.17.1 maven=3.6.3 spotbugs=4.2.2

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


fix: cl-checkbox css specificity

Hopefully this fixes the inline-block issue

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feature: added Android colors and styles to the prebuild modifier preview

Failure looks like a false error, retrying now


trigger maxlength validation error

I have this input field
<input type="text" name="name1" id="name1" maxlength="10">
I'm loading data to this field using:
$("#name1").val("Data with length more than 10")
It's not showing any error, I tried to do this after setting the field value:
$("#name1").trigger("input")
And also no error.
The only way to validate the field is to try to edit the field directly
Also, I tried to do simulate spacebar keypress
http://stackoverflow.com/questions/1468384/simulate-keypress-with-jquery
any suggestion?

There isn't actually a maxlength validator because the browser just prevents the user from typing in more than n characters. In your case since you're programmatically setting the value you'd need to add a custom validator for maxlength.
$('#myForm').validator({
  custom: {
    'maxlength': function ($el) {
      var maxlength = $el.data('maxlength')
      return !$el.val() || $el.val().length <= maxlength
    }
  },
  errors: {
    'maxlength': "Too long"
  }
})

<input type="text" name="name1" id="name1" maxlength="10">

Or you could just truncate the value to 10 characters when you set it on the field and the user won't be able to input past that.

it's working great.
I just named the new custom function 'maxcheck' and kept the  'maxlength' attribute:
<input type="text" name="name1" id="name1" data-maxcheck="maxcheck" maxlength="10">
I'm receiving now two error messages and for me there's no problem with that.
Thanks you,
Sam


React example

React example
It is a simple example of react consuming flakey service through opossum.
Details

In the root of folder example there is server.js script and it is a copy of jQuery example server but it is not serving static files.

Inside the client folder there is a react app that was bootstrapped by  create-react-app and it is using the default structure of react apps.

Also to avoid problems with CORS this example uses the Webpack  Proxy.

Concurrently package is used to run two servers simultaneously in the same console.

Running
Install dependencies.
$ npm i && cd client && npm i && cd ..

Start the servers.
$ npm start

Coverage remained the same at 98.413% when pulling 794850cb3dfaa67962ca167528226196f757f1b2 on tiagofabre:react into a4c2408a1d942fa9c43449a034318ff930eeaa20 on bucharest-gold:master.

Coverage remained the same at 98.413% when pulling 794850cb3dfaa67962ca167528226196f757f1b2 on tiagofabre:react into a4c2408a1d942fa9c43449a034318ff930eeaa20 on bucharest-gold:master.

Coverage remained the same at 98.413% when pulling 794850cb3dfaa67962ca167528226196f757f1b2 on tiagofabre:react into a4c2408a1d942fa9c43449a034318ff930eeaa20 on bucharest-gold:master.

Hi @tiagofabre thanks for doing this. I'm having some difficulty getting it to work, however. After following the instructions you note above, I see the following output.
~/s/o/e/react git:tiagofabre-react ❯❯❯ npm start                                            ◼

><EMAIL_ADDRESS>start /Users/lanceball/src/opossum/examples/react
> concurrently "npm run server" "npm run client"

[1]
[1] ><EMAIL_ADDRESS>client /Users/lanceball/src/opossum/examples/react
[1] > node start-client.js
[1]
[0]
[0] ><EMAIL_ADDRESS>server /Users/lanceball/src/opossum/examples/react
[0] > node server.js
[0]
[0] Server: http://localhost:3001
[0] Endpoints:
[0] get /flakeyService
[1]
[1] ><EMAIL_ADDRESS>start /Users/lanceball/src/opossum/examples/react/client
[1] > react-scripts start
[1]
[1] Starting the development server...
[1]
[1] Failed to compile.
[1]
[1] Error in ./src/App.js
[1] Module not found: 'opossum' in /Users/lanceball/src/opossum/examples/react/client/src
[1]
[1]  @ ./src/App.js 14:15-33
[1]
[1] Error in ./src/App.js
[1] Module not found: 'jquery' in /Users/lanceball/src/opossum/examples/react/client/src
[1]
[1]  @ ./src/App.js 18:14-31
[1]
[1] Compiling...
[1] Failed to compile.
[1]
[1] Error in ./src/App.js
[1] Module not found: 'opossum' in /Users/lanceball/src/opossum/examples/react/client/src
[1]
[1]  @ ./src/App.js 14:15-33
[1]
[1] Error in ./src/App.js
[1] Module not found: 'jquery' in /Users/lanceball/src/opossum/examples/react/client/src
[1]
[1]  @ ./src/App.js 18:14-31
[1]

Can you figure out what's going on here?

Coverage decreased (-0.8%) to 97.638% when pulling cf9b8d19881cf99622323a599329a686848ec526 on tiagofabre:react into a4c2408a1d942fa9c43449a034318ff930eeaa20 on bucharest-gold:master.

Coverage decreased (-0.8%) to 97.638% when pulling cf9b8d19881cf99622323a599329a686848ec526 on tiagofabre:react into a4c2408a1d942fa9c43449a034318ff930eeaa20 on bucharest-gold:master.

Coverage decreased (-0.8%) to 97.638% when pulling cf9b8d19881cf99622323a599329a686848ec526 on tiagofabre:react into a4c2408a1d942fa9c43449a034318ff930eeaa20 on bucharest-gold:master.

Sorry for that, some dependencies wasn't in the package.json
Now your output should be just like this:
`~/Projetos/opossum/examples/react$ npm start

<EMAIL_ADDRESS>start /home/tiagofabre/Projetos/opossum/examples/react
concurrently "npm run server" "npm run client"

[0]
[0] ><EMAIL_ADDRESS>server /home/tiagofabre/Projetos/opossum/examples/react
[0] > node server.js
[0]
[1]
[1] ><EMAIL_ADDRESS>client /home/tiagofabre/Projetos/opossum/examples/react
[1] > node start-client.js
[1]
[0] Server: http://localhost:3001
[0] Endpoints:
[0] get /flakeyService
[1]
[1] ><EMAIL_ADDRESS>start /home/tiagofabre/Projetos/opossum/examples/react/client
[1] > react-scripts start
[1]
[1] Starting the development server...
[1]
[1] Compiled successfully!
[1]
[1] The app is running at:
[1]
[1]   http://localhost:3000/
[1]
[1] Note that the development build is not optimized.
[1] To create a production build, use npm run build.
[1]
`


Refactor TWHDWalletGetAccountKey and switch case inside HDWallet::getNode

Maybe we can have something like TWHDWalletGetKey(struct TWHDWallet *wallet, TWCoinType coin, DerivationPath path)

It is going to be TWHDWalletGetKey(struct TWHDWallet *wallet, TWCoinType coin) since we can deduce the derivation path form the coin.

Yes, TWHDWalletGetKey(struct TWHDWallet *wallet, TWCoinType coin) should be enough.
And we have at least 3 types:

BTC m/44'/0'/0'/0/0
XLM m/44'/148'/0' (NANO / Nimiq)
AION m/44'/425'/0'/0'/0' (Tezos)

The differences between 1/2 is level, 1/3 is hardened, we have 3 getNode methods in HDWallet, maybe we can do just a simple for loop + curve check in one HDWallet::getNode.

👍，it's fixed in https://github.com/TrustWallet/wallet-core/pull/120


Feature/6.3/tcomp 372

Initial PR from wwang

[x] The commit message follows Talend standard
[x] Tests for the changes have been added (for bug fixes / features)

What is the current behavior? (You can also link to an open issue here)
https://jira.talendforge.org/browse/TCOMP-372
What is the new behavior?
https://jira.talendforge.org/browse/TCOMP-372
Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information:
https://jira.talendforge.org/browse/TCOMP-372

I have merge master onto this branch and updated the getSample code from the DatasetRuntime to match the new interface method signature.
I have also added a helper class to adapt any Source Reader to a getSample new behavior, see ReaderDataProvider (for the lack of a better name).

Pull request test result: success :star:

LGTM

Pull request test result: failure :x:


add missing cpl_vsi_error.h

@conda-forge/libgdal the absence of cpl_vsi_error.h is preventing us from building the gdal python bindings. Not sure if the absence of this file is by design or a mistake upstream.

Not sure if the absence of this file is by design or a mistake upstream.

Seems like a mistake upstream: https://github.com/OSGeo/gdal/issues/674

@gillins we have a new error that is causing a segfault :-/
But I did notice that we got OpenJPEG support: no and that is probably the reason for the lack of this driver in the python bindings.

@ocefpaf Maybe related: https://lists.osgeo.org/pipermail/gdal-dev/2018-June/048657.html

@kmuehlbauer that did the trick, thanks! But I'm still puzzled by the segfault :-/

Here are the erros:
libtool:   error: you must specify an output file
/bin/sh: line 3: 30257 Segmentation fault      (core dumped) /opt/conda/conda-bld/libgdal_1529500577956/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_place/lib/libhdf5.so
/bin/sh: line 3: 30258 Segmentation fault      (core dumped) /usr/lib64/librt.so
/bin/sh: /usr/lib64/libpthread.so: Permission denied
/bin/sh: line 3: 30260 Segmentation fault      (core dumped) /opt/conda/conda-bld/libgdal_1529500577956/_h_env_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_placehold_place/lib/libz.so
/bin/sh: line 3: 30261 Segmentation fault      (core dumped) /usr/lib64/libdl.so
/bin/sh: line 3: 30262 Segmentation fault      (core dumped) /usr/lib64/libm.so -o libgdal.la ./ogr/gml2ogrgeometry.lo ./ogr/ogr2gmlgeometry.lo ./ogr/ogr_api.lo ./ogr/ogr_expat.lo ./ogr/ogr_fromepsg.lo ./ogr/ogr_geo_utils.lo ./ogr/ogr_geocoding.lo ./ogr/ogr_opt.lo ./ogr/ogr_srs_dict.lo ./ogr/ogr_srs_erm.lo ./ogr/ogr_srs_esri.lo ./ogr/ogr_srs_ozi.lo ./ogr/ogr_srs_panorama.lo ./ogr/ogr_srs_pci.lo ./ogr/ogr_srs_proj4.lo ./ogr/ogr_srs_usgs.lo ./ogr/ogr_srs_validate.lo ./ogr/ogr_srs_xml.lo ./ogr/ogr_srsnode.lo ./ogr/ogr_xerces.lo ./ogr/ograpispy.lo ./ogr/ograssemblepolygon.lo ./ogr/ogrcircularstring.lo ./ogr/ogrcompoundcurve.lo ./ogr/ogrct.lo ./ogr/ogrcurve.lo ./ogr/ogrcurvecollection.lo ./ogr/ogrcurvepolygon.lo ./ogr/ogrfeature.lo ./ogr/ogrfeaturedefn.lo ./ogr/

Not sure yet what is going on, this was passing yesterday though.

@ocefpaf
Can you explain, why some of the compile artifacts are linked with relative paths (./ogr/gml2ogrgeometry.lo) and others with full path (/feedstock_root/build_artefacts/libgdal_1529500300408/work/gdal-2.3.0/alg/contour.lo)?

This seems related: https://bitbucket.org/chchrsc/kealib/issues/3/kea-config-hdflibs-wrong-format. I'll see about patching kealib tomorrow to workaround it, but disabling it for Unix seems to fix the problem for now...

Thanks @gillins!


Allow custom ResourceIterators

Hi there, my application uses a combination of "normal" and generated class files.  Class data are provided as plugins which the base application does not know about until the plugin arrives.  Because of the default implementation of ResourceIterator relies on the URL to determine how to create an InputStream, it can't handle my "special" cases such as classfiles whose bytecode is never written to a file system.  By making these classes/methods public instead of package-private I am able to write my own ResourceIterator which handles all kinds of classes, which I think makes the annotation-detector much more flexible especially in "special cases" like mine.  I wonder if you would consider pulling this change in case others out there have the same situation as me?  Thanks

Thanks for your contribution. I will take a look at it next week.

Thanks for your contribution.
IMO this is a valid use-case.


login command reports "system error" on newest macos 15.0 (24A335)

My bad, fix it by restarting the service.


it does not work in IE 11

This question is available on Nuxt community (#c7720)

你需要polyfill

I've opened a stack overflow question about this here if anyone cares to answer. I really wish this was documented somewhere - it's a very common question.

See https://github.com/nuxt/nuxt.js/issues/2260#issuecomment-367114722

@galvez I'm currently using your solution. It doesn't work in nuxt 2.0, however.

@dweldon
You can try to remove all config for build.babel
Default preset will introduce polyfill as needed.
Or they can use:
babel: {
  presets: [
    [ '@nuxtjs/babel-preset-app', {
        targets: { ie: 11 }
      }
    ]
  ]
}

@clarkdo Thanks for the suggestions. In both cases I see that Nuxt renders the default error page and informs me that Object.entries doesn't exist. Additionally, typing Object.entries into the IE 11 dev tools gives me undefined.
Note that when I build -a, I can see that core-js is included.
Perhaps the build process is actually attempting to do the right thing, but core-js isn't being hoisted / executed first. Is that possible?

@dweldon Could you provide a reproducible repo for me, I can try to see if I can figure it out.

@clarkdo thanks for taking the time to look into this - I really appreciate it!
I created a minimal reproduction here (along with a readme explaining what to try):
https://github.com/dweldon/nuxt-polyfill-issue
I was able to get the issue to appear when I included the @nuxtjs/apollo package. I don't know if that's because there's something broken with that package or if the execution errors before core-js is loaded.
Note this is the same package and version that I use in Nuxt 1.4.2, where I don't see the error in IE11.

I've opened an issue here as well. I'm still unsure where this is coming from.

@clarkdo
You can try to remove all config for build.babel
Default preset will introduce polyfill as needed.
Or they can use:
babel: {
  presets: [
    [ '@nuxtjs/babel-preset-app', {
        targets: { ie: 11 }
      }
    ]
  ]
}

I've tried your suggestions but the babel config won't make my app run in IE 11 after upgrading to nuxt 2. It seems like the polyfills never take effect with errors like:
webpack-hot-middleware's client requires EventSource to work. You should include a polyfill if you want to support this browser: https://developer.mozilla.org/en-US/docs/Web/API/Server-sent_events#Tools
[nuxt] Error while initializing app ReferenceError: 'WeakSet' is undefined

I'd like to try useBuiltins: 'entry' but not sure where to import @babel/polyfill. If I roll back to<EMAIL_ADDRESS>and add back @babel/polyfill in vendor option, everything works again on IE11.  Is there any special point that I might have missed?

Having the same issue:

Note that when I nuxt build -a, I can see that core-js is included.

@yenesey Yes but core-js alone may not be enough. You can see the details of what I did here. The short version is that I ended up using polyfill.io with some app-specific arguments.
I'd be curious to know if there's a way to not include core-js for those of us using other polyfills.


Improve type resolving

Since tartiflette allow us to deal only with dict. Resolving the __typename field for an UnionType is difficult and is delegate to the Resolver implementation which need to return the __typename in the resolved value.
Maybe we could add an optional resolve_type argument to the Resolver decorator which could take a callable in charge of determining the __typename to return for the resolved value? This resolve_type parameter could be required for an UnionType resolver which returns a value where the __typename couldn't be resolved by the default resolve_type function (a dict without a __typename key for instance).
@Resolver(
    "Query.catOrDog",
    resolve_type=lambda result: "Dog" if is_dog(result) else "Cat",
)
def query_cat_or_dog_resolver(*_args, **_kwargs):
    return {
        "name": "Dogo",
    }

I kind of like this idea @Maximilien-R.
@tsunammis what do you think of this idea?
I think it's great to enable our users to override the current default behavior.


Add servicerouter Python Script to Marathon

Merge of mesosphere/servicerouter with mesosphere/marathon.

Thanks @lloesche
@kolloch @aquamatthias Do you think the Marathon project is the right place for this, or should it live in its own project?

Original request to merge into mesosphere/marathon came through @guenter
bin/servicerouter.py is a replacement for bin/haproxy-marathon-bridge that's been in use mesosphere internally since December 2014.

If this makes the life easier for a lot of people, I think it is a good idea.
I would also like to have some documentation / best practices.
Can we add this?
BTW: Once PR #1436 is merged, you can attach to the events by an http stream.
Perhaps that makes this script even simpler?

@aquamatthias Cool that sounds super useful! The script already has some preliminary support for the Marathon Http Event Callback function (see def MarathonEventSubscriber) but we never got to the point where it was really usable. The function is currently more of a proof of concept but lacks in things like detecting which local IP to tell marathon to connect back to, tracking weather we're already registered (we ended up registering the same callback url multiple times in marathon). Also there were issues with speed that we never got around to debug further. But I remember when we build the prototype at the off-site it sometimes took Marathon on the order of minutes before it would trigger our callback after an event happened. So simply polling every 60s ended up being faster.
However once SSE support is merged I think it makes a lot of sense to support that. Is there any advantage of using WebSockets though? Like, is there anything we can communicate to Marathon at runtime?

@lloesche Cool! Regarding WebSockets vs SSE there is nothing you can do better with one or the other (the forward channel in the WebSocket case is not used at the moment). In the browser/javascript case it might be easier to choose WebSockets.

:+1:


Partial replication failure on ClueWeb12-B13 for ax models

According to #590 @matthew-z wasn't able to replicate results with ax models on ClueWeb12-B13. We should look into this...

Somehow the AX results matched after indexing again with oracle JVM (1.8.0_201)....

Interesting.
Hypothesis is that pseudo-random number generation works differently in OpenJDK vs. Oracle JVM. This would explain the differences in ax results, which rely on sampling docs.
@matthew-z Just to be clear, with Oracle JVM you are able to replicate our figures exactly, while with OpenJDK you come close? (If so, how close is "close" in the latter case?)

Did you mean no-AX algorithms?   They matched exactly I remember. sorry, the openJDK results have been covered by the oracle ones. I will try openjdk again.

Okay, thanks! I'll send a PR to capture this discussion and ask you to review.

@matthew-z please look at https://github.com/castorini/Anserini/pull/594

Resolved with #594, closing.

Hypothesis is that pseudo-random number generation works differently in OpenJDK vs. Oracle JVM.

I am not sure, but this is unlikely IMO.
I am going to test it again by myself

Whatever the issue, Oracle vs. openJDK isn't the issue. All regressions run fine on damiano:
[jimmylin@damiano ~]$ java --version
openjdk 11.0.4 2019-07-16 LTS
OpenJDK Runtime Environment 18.9 (build 11.0.4+11-LTS)
OpenJDK 64-Bit Server VM 18.9 (build 11.0.4+11-LTS, mixed mode, sharing)


Issue with PySide/matplotlib 2.1.0rc1

When matplotlib 2.1.0rc1 is used, tests are not passing with PySide (at least on Windows).
I reproduced locally and started looking at it but without success so far...

Here is my small test:
import logging
logging.basicConfig()
from PySide import QtGui

logger = logging.getLogger("test")

app = QtGui.QApplication([])

class Foo_super(QtGui.QWidget):

    def __init__(self, parent=None):
        super(Foo_super, self).__init__(parent)

class Foo_oldschool(QtGui.QWidget):

    def __init__(self, parent=None):
        QtGui.QWidget.__init__(self, parent)

import matplotlib
matplotlib.use('Qt4Agg')
from matplotlib.backends.backend_qt4agg import FigureCanvasQTAgg as FigureCanvas
from matplotlib.figure import Figure

fig = Figure(figsize=(600,600), dpi=72, facecolor=(1,1,1), edgecolor=(0,0,0))
ax = fig.add_subplot(111)
ax.plot([0,1])

try:
    a = Foo_super()
except Exception as e:
    logger.error("Backtrace 1", exc_info=True)

try:
    a = Foo_oldschool()
except Exception as e:
    logger.error("Backtrace 2", exc_info=True)

canvas = FigureCanvas(fig)

try:
    a = Foo_super()
except Exception as e:
    logger.error("Backtrace 3", exc_info=True)

try:
    a = Foo_oldschool()
except Exception as e:
    logger.error("Backtrace 4", exc_info=True)

Here is the result:
ERROR:test:Backtrace 3
Traceback (most recent call last):
  File "test_matplotlib_vs_pyside.py", line 47, in <module>
    a = Foo_super()
  File "test_matplotlib_vs_pyside.py", line 14, in __init__
    super(Foo_super, self).__init__(parent)
TypeError: PySide.QtCore.QObject isn't a direct base class of Foo_super
ERROR:test:Backtrace 4
Traceback (most recent call last):
  File "test_matplotlib_vs_pyside.py", line 52, in <module>
    a = Foo_oldschool()
  File "test_matplotlib_vs_pyside.py", line 19, in __init__
    QtGui.QWidget.__init__(self, parent)
TypeError: PySide.QtCore.QObject isn't a direct base class of Foo_oldschool

Then I created that issue on matplotlib: https://github.com/matplotlib/matplotlib/issues/9162

@t20100
I think this can be closed with the acceptance of your patch and the release of matplotlib 2.1.0

Yes


ollama create not working

Following the Modelfile tutorial in the readme, I can't get ollama create to work.
My modelfile is as follows:
FROM codellama:13b-instruct

SYSTEM """
You are Mario from super mario bros, acting as an assistant.
"""

When I attempt to create from the modelfile, I get the following error:
transferring model data
pulling model
pulling manifest
Error: pull model manifest: file does not exist

I have pulled the codellama:13b-instruct model to my machine and succesfully run it.
I'm using a M2 Macbook running ollama 0.1.13

I think "create" is used for models you have already downloaded, i.e. it is a file you specify, not model name.
otherwise you just do
ollama run codellama2
with whatever name on Hugging Face.

@phalexo the docs imply that you can reference a model in this way https://github.com/jmorganca/ollama#customize-a-prompt

This is a bug. There appears to be a space in the FROM line so it's looking for codellama:13b-instruct<space>. As a workaround, removing the space should work as expected.


Folder With Spaces

Hi.
When a try upload something like that
./uploads/cool_image.jpg in /SERVER/Demo/Cool Folder/Hydrangeas.jpg
The application give the next error:
Error: /SERVER/Demo/Cool Folder/Hydrangeas.jpg: Permission denied.
at makeError (PATH\node_modules\ftp\lib\connection.js:1067:13)
at Parser. (PATH\node_modules\ftp\lib\connection.js:113:25)
at Parser.emit (events.js:110:17)
at Parser._write (PATH\node_modules\ftp\lib\parser.js:59:10)
at doWrite (_stream_writable.js:301:12)
at writeOrBuffer (_stream_writable.js:288:5)
at Parser.Writable.write (_stream_writable.js:217:11)
at Socket.ondata (PATH\node_modules\ftp\lib\connection.js:273:20)
at Socket.emit (events.js:107:17)
at readableAddChunk (_stream_readable.js:163:16)
at Socket.Readable.push (_stream_readable.js:126:10)
at TCP.onread (net.js:538:20)
error: Forever detected script exited with code: 1
error: Script restart attempt #2
I think is for the space in folder name, but in the documentation i Couldn't find how to solve it.
Is this a Bug?

I know this is veeery late but I got stuck on this for far too long so I figured I'd reply here to try to help others:
The solution is remarkably simple, make it recursive so that node-ftp navigates to the path to run the 'MKD' command

mkdir(< string >path, [< boolean >recursive, ]< function >callback) - (void) - Creates a new directory, path, on the server. recursive is for enabling a 'mkdir -p' algorithm and defaults to false. callback has 1 parameter: < Error >err.
(From Readme)

example:
var Client = require('ftp');

var ftpDetails = {};
var ftpCon = new Client(),

function ftpErr(err) {
    if (err) {
        console.log(err);
        ftpCon.end();
    }
}

ftpCon.on('ready', function() {
    var path = "/path/you want/with spaces";
    console.log("addDir:", path)
    ftpCon.mkdir(path, true, ftpErr);
    ftpCon.end();
});
ftpCon.connect(ftpDetails);

Command structure for recursive:
addDir: /path/you want/with spaces // console.log
PWD
CWD /
CWD path
CWD you want
CWD with spaces
MKD with spaces

Command structure for non recursive:
addDir: /path/you want/with spaces // console.log
MKD /twitch/alerts/New folder

If you want to know why this works I cannot help you but it works :stuck_out_tongue:
Hope this helps
scagood

Try backslashing the spaces in the folder name, as I mention it in the referenced issue.


chore(CPlusPlus): add stack using linked list

PR Checklist:

[x] My submission is formatted according to the guidelines in the contributing guide
[x] My addition is on refer on the language README.md file
[x] My addition does not have a spelling problem
[x] My submission has a proper and user-friendly description of the algorithm
[x] My submission has the time complexity of the algorithm
[x] My submission has sample input-output of the program (NOT FOR PYTHON)

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] New algorithm
[ ] Optimization in previous algorithms
[ ] Code style update
[ ] Refactor
[ ] Documentation
[ ] Other, please describe:

Briefly describe the changes in this PR

Implemented stack using linked list

Done


error msg [SSL: CERTIFICATE_VERIFY_FAILED] when execute plex-tvst-scrobbler.py script

python plex-tvst-scrobbler.py
== Requesting tvshowtime.com auth ==
Traceback (most recent call last):
File "plex-tvst-scrobbler.py", line 98, in
tvst.tvst_auth()
File "/share/CACHEDEV1_DATA/.qpkg/Python/lib/python2.7/site-packages/plex_tvst_scrobbler/tvst.py", line 100, in tvst_auth
auth_infos = self._get_auth_infos()
File "/share/CACHEDEV1_DATA/.qpkg/Python/lib/python2.7/site-packages/plex_tvst_scrobbler/tvst.py", line 56, in _get_auth_infos
res = self._do_tvst_post(url, urllib.urlencode(args))
File "/share/CACHEDEV1_DATA/.qpkg/Python/lib/python2.7/site-packages/plex_tvst_scrobbler/tvst.py", line 40, in _do_tvst_post
res = urllib2.urlopen(f, data)
File "/share/CACHEDEV1_DATA/.qpkg/Python/lib/python2.7/urllib2.py", line 154, in urlopen
return opener.open(url, data, timeout)
File "/share/CACHEDEV1_DATA/.qpkg/Python/lib/python2.7/urllib2.py", line 431, in open
response = self._open(req, data)
File "/share/CACHEDEV1_DATA/.qpkg/Python/lib/python2.7/urllib2.py", line 449, in _open
'_open', req)
File "/share/CACHEDEV1_DATA/.qpkg/Python/lib/python2.7/urllib2.py", line 409, in _call_chain
result = func(*args)
File "/share/CACHEDEV1_DATA/.qpkg/Python/lib/python2.7/urllib2.py", line 1240, in https_open
context=self._context)
File "/share/CACHEDEV1_DATA/.qpkg/Python/lib/python2.7/urllib2.py", line 1197, in do_open
raise URLError(err)
urllib2.URLError: <urlopen error [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed (_ssl.c:581)>

Anyone knows why I get this error when I execute the script?
please help me

help me please

finally I solved myself this error but now I have another another one : this time in log file
please help me!

If you have this error msg  [SSL: CERTIFICATE_VERIFY_FAILED]
here is the solution :

edit plex_tvst_scrobbler/tvst.py file
add line import ssl at the top
in _do_tvst_post() function add just after the line try: add line context = ssl._create_unverified_context()
in the line just below change to res = urllib2.urlopen(f, data, context=context)
run python setup.py install the changes to take effect


Kanbanize v1.5

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[x] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[x] I attest that the connector works and I verified by deploying and testing all the operations.
[x] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[x] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[x] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[x] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff.

We have applied some small fixes and changed titles and descriptions for the actions and triggers. Please review the changes.

Hello @kostadin-kanbanize,
Thank you very much for an update. Our Certification Engineers will review it within 1-2 weeks.
Could you please check if your fork is up-to-date? There's a conflict between branches.
Please let us know if you have any questions.
Thank you very much again for working with us.

We have verified that the correct version of the files (apiDefinition.swagger.json and icon.png) is uploaded in Kanbanize-v1.5 branch. Please overwrite the conflicting files.

Hi @kostadin-kanbanize,
Unfortunately, we cannot overwrite the conflicting files. Please update your fork and branch, it's many commits behind our dev branch and it causes conflicts.

Hi, I have created new branch from dev and uploaded our new changes to it. This pull request will be closed and you can process the new one.
Regards


Fix typo in fastify logger type declaration

Issue #, if available: #587
Description of changes: Fixes typo in import and type of FastifyLoggerInstance in aws-xray-sdk-fastify package
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #589 (90efaa7) into master (eebda75) will not change coverage.
The diff coverage is n/a.

:exclamation: Current head 90efaa7 differs from pull request most recent head ae73cda. Consider uploading reports for the commit ae73cda to get more accurate results

@@           Coverage Diff           @@
##           master     #589   +/-   ##
=======================================
  Coverage   83.38%   83.38%
=======================================
  Files          37       37
  Lines        1794     1794
=======================================
  Hits         1496     1496
  Misses        298      298


Fix caching of small inline images in Parser.makeInlineImage (issue 8790)

Follow-up to PR #5445.
Using the PDF file from issue #2618, i.e. http://bugzilla-attachments.gnome.org/attachment.cgi?id=226471, with the following manifest file:
[
    {  "id": "issue2618",
       "file": "../web/pdfs/issue2618.pdf",
       "md5": "",
       "rounds": 50,
       "type": "eq"
    }
]

I get the following results when comparing master against this patch:
browser | stat         | Count | Baseline(ms) | Current(ms) |  +/- |     %  | Result(P<.05)
------- | ------------ | ----- | ------------ | ----------- | ---- | ------ | -------------
firefox | Overall      |    50 |         4694 |        3974 | -721 | -15.35 |        faster
firefox | Page Request |    50 |            2 |           1 |    0 | -22.83 |
firefox | Rendering    |    50 |         4692 |        3972 | -720 | -15.35 |        faster

So, based on these results, it seems like a fairly clear win to fix this broken caching :-)
Fixes #8790.

/botio test


automatically create Product Type

fixes #97

Hi @tomaszn, thank you for your contribution. It has been released with 1.0.11.


Top filter not cleared when filtering with sidebar

When clicking on "User Story" or "Bug" in the sidebar, the filtering is done correctly but the filters in the top (e.g. "Assigned to Me") are not cleared.

Sidebar filters for "user story", "bug", etc should be moving to the top filtering bar, following the designs as specified here:  https://redhat.invisionapp.com/d/main#/projects/prototypes/10295470


tour: Program exits without printing

Context: https://tour.golang.org/basics/11
The program exists without printing if fmt is imported using the parentheses import.

There was a caching issues with the playground (which stores the tour programs); which is now fixed. Closing here.


connect using URI connection string support

As mentioned in #73.
Typescript doesn't allow spreading arguments into function https://github.com/Microsoft/TypeScript/issues/4130 and doesn't allow conditional super() call https://github.com/Microsoft/TypeScript/issues/945.
One way to achieve this is to instantiate Sequelize in a static factory function.
Old constructor must be removed to allow arguments to be exactly relayed to OriginSequelize constructor. Thus, this change will break backward compatability.
Developers currently using "new Sequelize(config)" have to change the code to "Sequelize.init(config)" or "Sequelize.init(URI, config)".
Should this be merged into next major version ?

Codecov Report

Merging #79 into master will decrease coverage by 0.18%.
The diff coverage is 87.5%.

@@            Coverage Diff             @@
##           master      #79      +/-   ##
==========================================
- Coverage    95.6%   95.41%   -0.19%
==========================================
  Files          65       65
  Lines         750      763      +13
  Branches      103      105       +2
==========================================
+ Hits          717      728      +11
- Misses         11       12       +1
- Partials       22       23       +1

Impacted Files
Coverage Δ

lib/models/BaseSequelize.ts
90% <100%> (ø)
:arrow_up:

lib/models/v3/Sequelize.ts
95.16% <86.66%> (-2.8%)
:arrow_down:

Continue to review full report at Codecov.

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

Hey @kukoo1 thanks for contributing :)
Why is spreading necessary here, from your point of view?
Did you try to resolve this with overloading the constructor like:
export class Example {

  constructor(connection: string);
  constructor(options: Options);
  constructor(optionsOrConnection: Options|string) {
    /*...*/
  }
}

?

I've tried this.
constructor(config: ISequelizeConfig, uri?: string) {
    // Sequelize constructor signature:
    //// new (database: string, username: string, password: string, options?: Options): Sequelize;
    //// new (database: string, username: string, options?: Options): Sequelize;
    //// new (uri: string, options?: Options): Sequelize;
   if (uri) {
      super(uri, BaseSequelize.prepareConfig(config));
   } else {
      super(
         (preparedConfig = BaseSequelize.prepareConfig(config), preparedConfig.name),
         preparedConfig.username,
         preparedConfig.password,
         preparedConfig
      );
   }
   this.init(config);
}

The compiler complains
A 'super' call must be the first statement in the constructor when a class contains initialized properties
So I tried solving with this.
constructor(config: ISequelizeConfig, uri?: string) {
   super(
      uri || (preparedConfig = BaseSequelize.prepareConfig(config), preparedConfig.name),
      uri ? BaseSequelize.prepareConfig(config) : preparedConfig.username,
      uri ? undefined : preparedConfig.password,
      uri ? undefined : preparedConfig
    );
    this.init(config);
}

Now it works with traditional connection options as an object.
But by using connection string, it won't work.
Because originalSequelize will recognize that super(uri, config, undefined, undefined) contains 4 arguments, so "uri" would be db name, "config" would be username, "undefined" would be password and so on.
Any idea to this? :)

@kukoo1 Thank you for your input :)
Or much simpler:
new Sequelize({uri: '...'})

constructor(config: ISequelizeConfig | ISequelizeUriConfig) {}

Since sequelize-typescript already uses one object literal for options instead of multiple parameters, I think this will be the most consistent approach. What do you think?

hmm it's just a string, do we really need to make it object literal?

@BruceHem I think it is most likely, that the configuration will look like this
new Sequelize({
  uri: '...',
  modelPaths: ['...']
})

But in addition, we could overload the constructor like:
  constructor(connection: string);
  constructor(options: ISequelizeConfig | ISequelizeUriConfig);
  constructor(optionsOrConnection: ISequelizeConfig | ISequelizeUriConfig | string) {
    /*...*/
  }

I found the root of this mess.
I've just looked into Sequelize code.
It accepts
new Sequelize({ ... options })
new Sequelize(URI, { ... options })
new Sequelize(database, username, password, { ... options })

So we could call super(uriOrOption) properly.
At first, when I coded this PR, I only looked at Sequelize.d.ts definition (@types/sequelize).
The definition accepts only
new (uri: string, options?: Options)
new (database: string, username: string, options?: Options)
new (database: string, username: string, password: string, options?: Options)

So we've to open PR in DefinitelyTyped to add this beforehand
new (options: Options): Sequelize;

Now I get into another trouble. I can't pass both URI and options at the same time.
current constructor looks like this
constructor(configOrUri: ISequelizeConfig | ISequelizeUriConfig | string) {
   super(
      (typeof configOrUri === "string") ?
        configOrUri : // URI connection string
        BaseSequelize.prepareConfig(configOrUri) // config object
   );
}

Sequelize checks arguments using this code
if (arguments.length === 1 && typeof database === 'object') {
   // new Sequelize({ ... options }) <-- in order THIS to WORKS, we must ONLY pass 1 argument !!
   // ...
} else if (arguments.length === 1 && typeof database === 'string' || arguments.length === 2 && typeof username === 'object') {
   // new Sequelize(URI, { ... options })
   // ...
}  else {  // <-- with this, URI string will NOT WORKING !!
   // new Sequelize(database, username, password, { ... options })
   // ...
}

So we couldn't do this.
super(
   (typeof configOrUri === "string") ?
      configOrUri : // URI connection string
      BaseSequelize.prepareConfig(configOrUri), // config object
   (configOrUri.uri) ?
      configOrUri : // ISequelizeUriConfig
      undefined // <-- THIS will BREAK arguments.length === 1
);

There're 2 options.

ignore config when URI is passed (ISequelizeUriConfig will be a junk and we'll lose flexibility when use URI connection string)
open PR in Sequelize to add const argLength = arguments.filter(val => val !== undefined).length and use if (argLength === 1) instead.

@kukoo1 Ahh ok, I see. I didn't get it in the first place.
From my understanding, typescript makes it impossible to use overloads in constructors in combination with inheritance in general. Cause it is very common to check arguments length. Or have we overseen something?

waiting for review ...
https://github.com/DefinitelyTyped/DefinitelyTyped/pull/18896

@RobinBuschmann  I committed the new one.
constructor(config: ISequelizeConfig | ISequelizeUriConfig | ISequelizeDbNameConfig);
constructor(uri: string);

I suggest to deprecate "name" property in options, and use "database" instead.
Because, in Sequelize documentation, "database" is used.
In order to use the same config when switching from ordinary sequelize, we should use "database".
What's your opinion ?

Looks good to me. Deprecating "name"  is reasonable as well. But instead of adding "@Todo", you could write "@deprecated" - this will be noticed by many IDEs and therefore be notice the user that "name" is deprecated.

Fixes #73


Port Standardization for Glance, Nova, NoVNC, Cinder and Neutron

Port Standardization for glance, nova, novnc, cinder and neutron. Customer facing port has been changed to standard openstack ports. Keystone, heat etc will be included in the next PR

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

ok to test

Since this is changing configuration, we need something in upgrade.yml to alter the endpoints. Otherwise if we run this on an existing cloud we'll get some problems. I haven't taken much look on this other than that, review is still needed.

I would love to see the openstack project itself listening on <IP_ADDRESS> on the correct port and then have haproxy listen on the 10.x IP on the controller nodes.   We would have to solve the fact that we also need haproxy to serve across the external VIP as well for systems that don't have a external firewall/LB doing it for them.
we should probably make both the listen_ip and port a variable in each role so that they can be easier reconfigured if they ever need to be.

ok to test

Gonna move these commits to a new branch to unblock those trying to use the updated wip_kilo branch


Make search take focus on opening pokedex; add cmd-f shortcut

I added some convenience functionality because I wanted it; I think it makes the website a little bit nicer to use.
This adds some convenience behavior:

Cmd/Ctrl-F changes page to the Pokedex page
The search input is focused by default, or if you press cmd-f again on the pokedex page
The input text is selected by default, so you don't have to delete the contents before typing a new search

Basically, this makes the pokedex search page behave roughly like the browser find-in-page.

Hey Phoenix, thanks for working on this. Here's some thoughts about the PR:

Cmd/Ctrl-F changes page to the Pokedex page

This seems good to me. I think you could remove the OS check and just bind to Ctrl-F and Cmd-F though. It's simpler and helps out people who are less familiar with whichever OS they happen to be using right now. Also, / is a pretty common search shortcut that would be worth binding.

The search input is focused by default, or if you press cmd-f again on the pokedex page

Fun fact, it used to be focused by default. But this is really annoying on phones/tablets since it forces the keyboard to pop up and cover half the screen. We could get into the game of guessing if the device is being used with a touch keyboard, but it's further complicated by stuff like touch screen laptops...

The input text is selected by default, so you don't have to delete the contents before typing a new search

This is handy, but I don't think we should do it given the above^

FYI, you're welcome to add yourself to the "Special Thanks" part of the Info page.

Thanks for the quick response!

Fun fact, it used to be focused by default. But this is really annoying on phones/tablets since it forces the keyboard to pop up and cover half the screen. We could get into the game of guessing if the device is being used with a touch keyboard, but it's further complicated by stuff like touch screen laptops...

How do you feel about a "good enough/mostly right" solution here, using something like heuristics based on screen size? Maybe if the first pokemon result would be at all covered by a hypothetical keyboard (that is, fits in ~the top half of the screen), don't autofocus?
In lieu of that, we could make it autofocus only if the user navigated with cmd-f, and not if they use the mouse / tap the screen.

Yeah, making Cmd-F do (1. Take you to Pokdex; 2. Focus search field; 3. Select all text) sounds perfect to me. Quick shortcut to help folks out, but no chance of bothering folks who don't care.
Maybe you could add another query param, or use some global state in App, so that the Pokedex knew what it should do on load.

I'm closing this due to the branch being out of date and no recent activity. Feel free to make a new PR if you still want to work on this feature. Thanks!


Add C to IL

:D

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


https://github.com/BotolMehedi/bvai/issues/3

في الأربعاء، ٢٧ أكتوبر، ٢٠٢١ ١:٢٠ م aminmuhamed @.***>
كتب:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/BotolMehedi/bvai/issues/3, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AROOJRXPKVNGBGGRVJQU2SLUI7N73ANCNFSM5G2CUC7A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.


[SMALLFIX] Update master UI to show in-Alluxio files

We already updated API to return in-Alluxio status instead of in-Memory. The UI label is also updated to In-Alluxio data. This change updated the UI to actually display files in Alluxio as opposed to files only in memory.

@aaudiber @ifcharming PTAL.

Jenkins, test this please.

Merged build finished. Test PASSed.

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

Merged build finished. Test PASSed.

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


Panics generated from langlib string cause 'ballerina run' command to break

Description:
public function main(string... args) {
    var k = "k".substring(4, 6);
}

This result in a bad sad.
ballerina-internal.log
Steps to reproduce:
Affected Versions:
Ballerina 1.0.0-alpha
OS, DB, other environment details and versions:
Mac
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

This fixes the same issue https://github.com/ballerina-platform/ballerina-lang/pull/18151


Added DSA-Courses subcategory in DSA Section

DSA section original-
DSA articles
DSA tutorials
Added- DSA Courses
Closes #2007
Changes proposed

Included Courses for DSA section (dsa-courses.json)
For example- Coding Ninjas Online 4-5 month course etc
A total of 8 courses included

Screenshots
BEFORE

AFTER

Hey @thatdamncoder we're currently not adding any paid resources, could you please remove the paid courses?

Done @rupali-codes

@thatdamncoder Please add a description.

I think we don't have descriptions for any of the subsections in any of the categories. Only the main sections/categories are provided with the description.

There's some merge conflicts too

@thatdamncoder Please add a description.

I think we don't have descriptions for any of the subsections in this category. Only the main sections/categories are provided with the description.

I've tried but could you please tell me what to include to make the
changes? I've written according to the guidelines many times but the result
is not showing up.
On Wed, 25 Oct 2023, 17:26 Anmol Baranwal, @.***> wrote:

@thatdamncoder https://github.com/thatdamncoder Please add a
description.
[image: image]
https://user-images.githubusercontent.com/74038190/277194504-199c9a00-5390-4d36-a617-32e6bd3bdd6a.png
I think we don't have descriptions for any of the subsections in any of
the categories. Only the main sections/categories are provided with the
description.
Just go the official website, see how many subcategories in Frontend
section has descriptions. Each has a description.
Simply add it, it is clearly mentioned in the contributing guidelines.
If you're not willing to do so, let me know.
—
Reply to this email directly, view it on GitHub
https://github.com/rupali-codes/LinksHub/pull/2066#issuecomment-1779105089,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/BCLXNCOJ3WWYWYDOXIJUM2DYBD45ZAVCNFSM6AAAAAA6KLZB5OVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTONZZGEYDKMBYHE
.
You are receiving this because you were mentioned.Message ID:
@.***>

@thatdamncoder
What is the problem, you just need to take care of the name, which will be same as json file. Check it locally.

Hello @rupali-codes
Could you please guide me on where is the issue arising in the conflicting files?

@thatdamncoder Please add a description.

I think we don't have descriptions for any of the subsections in any of the categories. Only the main sections/categories are provided with the description.

Just go the official website, see how many subcategories in Frontend section has descriptions. Each has a description.
Simply add it, it is clearly mentioned in the contributing guidelines.
If you're not willing to do so, let me know.

@Anmol-Baranwal
Do you want me to add descriptions in other categories too?

Descriptions not showing up.

@Anmol-Baranwal
Apologies for the description not showing
please check now and let me know
FOR ARTICLES

FOR COURSES

FOR TUTORIALS

@CBID2 @rupali-codes please do check and review for any more changes
apologies for the chaos


Create Italics variant of the font

Make the changes necessary to create the italics version of the font.  Note that we will only italicize the letter forms themselves and not the symbols.

I think this looks good, so I am merging it.


Support Curlib

Had a question on this in the review
 |
 |
 |Edmund Reinhardt
 |
 |Rational Developer for i Architect
 |
 |IBM
 |p:
 |<PHONE_NUMBER>
 |w:
 |  ibm.biz/rdi_hub  e:
 |<EMAIL_ADDRESS>
 |

From:	"Tongkun Zhang" @.>
To:	"IBM/ibmi-bob" @.>
Cc:	"Edmund Reinhardt" @.>, "Review
requested" @.>
Date:	10/01/2021 09:50 AM
Subject:	[EXTERNAL] Re: [IBM/ibmi-bob] Support Curlib (#36)
@TongkunZhang requested your review on: #36 Support Curlib. — You are
receiving this because your review was requested. Reply to this email
directly, view it on GitHub, or unsubscribe. Triage notifications on the go
with GitHub Mobile
@TongkunZhang requested your review on: #36 Support Curlib.
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.


Change usages of new path configs

Substitute projectBasePath correctly with viur.instance.project_base_path
Cast Path to str before .replace()
Make use of the new Path and use context managers for I/O

@ArneGudermann this pull request is intended to extend your pull request #582. Please add your review.


Add custom GatedRepoError when trying to access gated repo

Related to discussions happening around gated repo (example : https://github.com/huggingface/huggingface_hub/issues/1198#issuecomment-1325030852). Follow server-side PR (internal link).
This PR takes advantage of the new "GatedRepo" error code that is returned by server when a user tries to access a repo that is gated (and user is not in authorized list). It raises a  GatedRepoError in python (cc @sgugger).
Note: GatedRepoError inherits from RepositoryNotFoundError to avoid breaking changes.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Query Name Rewriter

First of all this is not an issue, I need a little bit of help.
I didn't find a rewriter for the query name.
Example:
Let say we have query like this which is being used in many clients
query query getUserById($id: String!) {   userById(id: $id) {     ...   } }
I want to change my query to this
query getCurrentUserByID($id: String!) {   userById(id: $id) {     ...   } }
So I want that whenever client query getUserById, it should map to getCurrentUserByID. I think currently there is no option for query name. So is there any problem with writing a rewriter for query name? Have you tried this?  Because I am thinking to write a query rewriter for this.

This was also requested in https://github.com/ef-eng/graphql-query-rewriter/pull/14. Currently there's no rewriter for this since you can deprecate the old field and make a new field with the new name using normal graphQL deprecationReason, but I think it's fine to add if it's something people want to use. As discussed in the existing PR, it will require changing some internals to make this rewriter possible.

@chanind Thanks for the reply. I checked out the PR. I went through code as well of rewrite a response. I found out that its more related to work with rewriteScalarFieldToObjectField  and  NestFieldOutputsRewriter. To make it work FieldNameRewrite won't be straight away. I am trying to make it work with it.
Any suggestions would be appreciated.

@chanind did you already have something in mind for updating the rewriter? I could submit a pull request for it. I've created a version of the field name rewriter but the only thing missing is the ability to rewrite the response.

Thanks @gregbty! Sorry for the delay getting back on this! Just a minor naming nit on the PR and should be good to merge.


"Holey" behavior on differently-typed getter/setter properties

I was asked to provide an example of separate read/write types that doesn't necessarily use classes, and here's what I ended up finding. 😅
Take the following:
interface Thing {
    get name(): string
    set name(value: string | null | undefined);
}

function makeThing(): Thing {
    let name = "";
    return {
        get name() {
            return name;
        },
        set name(value) {
            name = value.toString();
        }
    }
}

In this example, you might think that you're writing type-safe code, but in fact you've got a bug. Our get/set accessors don't have type annotations, so TypeScript assumes the name setter has the type string. Because only the reading type of the name property is validated against Thing, TypeScript thinks that the given object conforms to the type Thing; however, a user can pass write a null or undefined, and a user won't know that they haven't handled it correctly.
This was one of the known tradeoffs when we implemented this feature - we acknowledged this. But I hit this surprisingly quickly. It seems like if you're using this pattern without classes, it's pretty easy to run into this issue.

That would help, for sure. It's not the full solution, but maybe it's good enough?

But it doesn't really "fall out" of how we usually do contextual typing, where you only look at types (not their declarations).

I'm not sure what else to do; this is only as unsound as we already are with union types.

I definitely think we should properly contextually type the setter parameter, but beyond that, I agree that feature doesn't make us any more unsound than we already are. I mean, we permit a property of type unknown in the target to be matched by a property of type string in the source, at which point to can pass any value you want to the setter.

Maybe I'm missing some context on what's already unsound, but this does appear to be specific to getters and setters and not generally about subtyping? The nearly identical example below without getters/setters gives an error as expected
interface Thing2 {
	getName(): string;
	setName(value: unknown);
}

function makeThing2(): Thing2 {
	let name = "";
	return {
		getName() {
			return name;
		},
		setName(value) {
			name = value.toString(); // Error: 'value' is of type 'unknown'. ts(18046)
		},
	};
}


change was made to check 'k' func using valid handle

handle should be previously known to c api via the various connect calls (not a handle created by the users own system calls).

change made, thanks