Give absolute path to worker script

Starting workers fails (at least on slurm) if the template['workingDirectory'] is not the current working directory because:
/var/spool/slurm/d/job1177731/slurm_script: line 2: ./dask-worker-scriptz8gmf_cpsh: No such file or directory

SGTM. Care to try a PR?

Working on it ;)

See #90

Fixed in PR ( https://github.com/dask/dask-drmaa/pull/90 ).


Adding appender/3 with conn, schema_name, table_name

Previously initialising appender with "schema.table" appender/2 gave an error
Screenshot 2024-10-10 at 16 27 14
In this PR I added appender/3 definition with conn, schema_name, table_name, so that you can also specify the schema name

Hi! Thanks!


get distances from lidar

Hi Folks,
in my project I need to get distances of dynamic vehicle to the other (static-parked) vehicles beside it. However, I could not find enough explanation in documents so how can I see detected obstacle with lidar sensor? then how can I get the distance to them (with which attribute of lidar sensor?)
I used raw_data but it gives me a buffer and I could not extract anything from that. and how can I observe that sensor detecting what?(I mean how can I understand that it detects other car or pedestrian or ...)nothing is visible here. I also save detection of sensor with save_to_disk() but it gets me bunch of same files with format .ply!
I also used sensor.other.obstacles but all of the distances I get from it are the same!(like 50 all the time!)
Please help me what should I do?

@JohnEmad92 @nsubiron

+1


Updated vectorized settings in 20.2, 21.1

Fixes https://github.com/cockroachdb/docs/issues/9996.
Fixes https://github.com/cockroachdb/docs/issues/9099.

This change is

Online preview: http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/a8c9a9c536cd3679dbed32ae493be8064686fdf4/
Edited pages:

v21.1/vectorized-execution.md


Describe ruby gems

Link to bundler application guide. Mention why gems are awesome.

Is this some research you have to do? As in i need to learn and figure out the above?

No, it's based on slack input. In case it will be useful for someone. If we want an actual usecase first, this should be closed for now.

I'm thinking that praqma-rnd exist for work we have to do... so if you're writing a blog post on how to do bundler and gems it fit well.
Also if you want to form a small simple project around defining good practices.
But if it is just knowledge sharing I think it should go to our blog posts.
Then again, praqma-rnd is my intention this way... but we might settle for something else.

And it was meant for all of us to have a place to create an issue for research work, because obviously we don't do any work without an issue :-)

I will close this - I am not writing a blog post, I don't have work to do related to Ruby gems. For people wanting to learn, the Internet is already a pretty helpful place to getting started with Ruby gems.
FWIW, I started here: http://bundler.io/v1.13/guides/creating_gem.html


Support parsing and rendering admonitions in Markdown

Closes #279.

Syntax is similar to a regular code fence, with special languages:
Supported admonition types:

note
tip
info
warning
danger

e.g.
```note
This is a note with **bold** text.
```

Codecov Report

Merging #280 (5f1bf11) into master (33016dd) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #280   +/-   ##
=========================================
  Coverage     69.48%   69.48%
  Complexity      596      596
=========================================
  Files           117      117
  Lines          2078     2078
  Branches        220      220
=========================================
  Hits           1444     1444
  Misses          550      550
  Partials         84       84

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


Add unal - Universidad Nacional de Colombia

Added Universidad Nacional de Colombia, the biggest puplic university in Colombia,  website: https://unal.edu.co/

@fegonzalez7 I am sorry, the domain unal.edu.co is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Fixed order of hint placement

weird, aren't these changes in this commit here? f0ab4c906e7e68e9300a0036d336991d7e1dec5b

@defbyte they are, that's the commit I merged in


Pass environment variables to plugin during version build/installation

Hi
How are you?
Before anything, thanks for this amazing software.
I'm using an asdf plugin (https://github.com/tsuyoshicho/asdf-vim) to manage vim versions.
That plugin allows passing some variables to be used during vim build, such as ASDF_VIM_TYPE and ASDF_VIM_CONFIG, as you can see in the section Installation Variable in README.md.
There is any way that I can set this in .rtx.toml, so when I backup this file and restore in another PC it's install the same way ?
Thanks in advance.
Best regards.
Ricardo Seriani

duplicate of https://github.com/jdx/rtx/issues/702


Deprecate searchbird

The searchbird lesson does not work and the CLI suggestion to install a non-existent ruby lib is a potential security vulnerability to users who follow these instructions.
Remove all mention of cap utils and and highlight deprecated nature of the content.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@agargenta how about the slightly longer explanation in 0315196bd1f7656bda28243561114ea57f61ffbb


Youtube playlist links don't work within m3u playlist

If you put a YouTube playlist URL of the form yt:http://www.youtube.com/playlist?list=<listId> or yt:http://www.youtube.com/watch?v=<videoId>&list=<listId> in a M3U-playlist, it doesn't actually add anything to the queue when you try to play it.
The only log output this adds is these three lines:
Jan 05 18:46:01 radio mopidy[20098]: ERROR    WebSocket request error: Expected album to be a <class 'mopidy.models.Album'>, not {u'uri': u'x', u'name': u''}
Jan 05 18:46:01 radio mopidy[20098]: ERROR    WebSocket request error:
Jan 05 18:46:01 radio mopidy[20098]: ERROR    WebSocket request error:

The error message is due to some bad handling of playlist tracks in musicbox-webclient following the move to stricter model validation in Mopidy (ages ago) see https://github.com/pimusicbox/mopidy-musicbox-webclient/issues/131. I should have spotted that on IRC, sorry.

Well, it also doesn't work when using the mpd frontend. I actually disabled musicbox to make sure it doesn't interfere.
(Though those error messages are indeed caused by musicbox.)
I have also noticed, that it doesn't work even when making an M3U playlist with just a single video in it: https://gist.github.com/Xandaros/e2f4c546fba9c4f6d85b
I tried it both with and without the yt: prefix.

 Apparently you use load <playlist>, not playlist <playlist>.
Interesting how nobody called me out on that, though.


fix: Mock all bundler related modules

Description
See https://discord.com/channels/967097582721572934/1037724107069997129/1133047689827848193
Should resolve issue in #3060

[x] I have read and understand the CONTRIBUTING.md document in this repository

Type of change

[x] Bug fix (non-breaking change which fixes an issue)

Thank you for your efforts on this PR, but we've discovered an alternative approach that we'll be pursuing, so I'll be closing this for now.


Implementation in clojure js

Can any one suggest how this component can be implemented in clojure js.?
This would be really help us!!!!!!!!!
Thanks in advance

Hey, I just released a new alpha 2.0.0-alpha.0 of the library. It's rewritten using react-native-gesture-handler and react-native-reanimated addresses a many platform specific bugs and performance problems. The documentation is updated as well.
Please try the new version and see if it addresses your issue. If not, please open a new issue following the issue template.


empty space after breakpoint

Hi, I have this problem
http://codepen.io/sebastianpothe/pen/VvrjQd
I resized the viewport to show only one slide, then I swiped (or click on dots) to see slide 4, but, when I resized again to show the 4 slides, the slider stays on the left, showing only 1 slide.
I tried with 'slickGoTo' on 'setPosition' but nothing happens

Hi, I have tried this and seems to work:
this script checks if all the slides are visibles, then resets the transform property of the slider track/
$('.slider').on('setPosition', function(slider){
var slick = slider.currentTarget.slick;
var currentBreakpoint = slick.breakpointSettings[slick.activeBreakpoint];
var originalSettings = slick.originalSettings;
slick.currentSettings = undefined !== currentBreakpoint ? currentBreakpoint : originalSettings;
if ((slick.slideCount * slick.rowCount) >= slick.currentSettings.slidesToShow) {
slick.$slideTrack.css('transform', 'initial');
};
	})

Glad you found a solution! Closing.


flash does not work on coordinates that were imported

The flash does not seem to work with coordinates that were imported.  The points highlight but the flashing cross hairs does not occur.  The flash does occur with collected coordinates. To reproduce:

import coordinates from csv
select a coordinate from the list
right click and flash
the coordinate will highlight green but the flash will not occur

This will need to be logged as a known issue. Adding the Known Issue and Affects Documentation labels. Assigning to @NatalieCampos and @dfoll to document.

@NatalieCampos Please update this issue with what has been done.

The following statement was added to the CC Known issues for ArcMap.
" Using the Flash option on imported coordinates does not work.  "

I have removed the known issues from Coordinate Conversion and Military tools. I have also added the statement to issues addressed for the next release of Coordinate Conversion. @dfoll I am not aware of where this is for Military Tools so I have not updated the issues addressed, but I have removed this from known issues.

@dfoll please help in finishing how to close this issue.

documentation changes have been made by @topowright in cms.  This issue can be closed. As part of the release of Military Tools and Coordinate Conversion, all documentation will need to be republished.


Maybe don't run in debug builds and tests

The data is also sent in debug (development) builds
The data is also sent in intergration tests
does that not skew the results?
if you feel this is not something to fix in the lib (because you feel it's the clients responsibility, then at least add it to the example code in the README).

I guess we want this so we are able to test it properly!? If that is the case I would just add it to the README.

I think there is no possibility to do this inside the lib, because the lib is always a release-build when used as a library. All other libs (firebase) also leave it to the app to decide. I'll add it to the sample code and demo app

I've thought about this for 4 more minutes and I prefer to close this one. Perhaps 5 people per project (2 devs + 2 testers and their cat) will use debug builds. That is insignificant. It would add needless noise to the README imho.
Whoever reads this in 2025 or beyond: Feel free to close this if no objections are voiced.


support ES2018 object spread on java maps from javascript code

I'm using GraalVM's polyglot API to call javascript code from kotlin, I pass a Map to javascript function which needs to clone it using casual {...map} javascript idiom.
Unfortunately the result of {...map} is {get: function,remove: function,etc} that is java map's methods get cloned while its content doesn't.
The problem I have here is that I'm using GraalVM To Server-side render a react app which means that javascript code is shared between server and browser and I don't want to do if(isServer)... all over the place everytime I want to interact with kotlin/java map.
Information:
GraalVM Version: 20.1.0
JDK: GraalVM Itself, java 11 version.
OS: Windows 10 x64.

Hi @abnud1
thanks for your report. We are aware of that problem. It is discussed in #143. While we have support for the similar case for Arrays and Lists, we don't have that yet for Maps. This requires an additional primitive on the Truffle/Polyglot level, to be supported by all languages. Support is planned, but I cannot give you an ETA for it (@chumer ?).
Any chance you can simulate the behavior with a ProxyArray or a ProxyObject? What would require a wrapper implemented on the Java side, but could solve your problem for JavaScript.
Best,
Christian

@wirthi ProxyObject would work for a simple map, but my map can contain Lists in its value of maps of simple keys to lists, or map of string to map of string to list of...... you get The idea.
ProxyObject is almost impossible to use for such cases, in The end I convert the map to JSON and convert back at javascript side using JSON.parse.

Hi @abnud1
nested structures should not be an issue, you can convert them lazily on demand. But JSON is of course another option you can take.
Best,
Christian

Update: we're adding support for this in 21.1.0.


⚠ Cioos National CKAN Sitemap has degraded performance

In 3cb35ba, Cioos National CKAN Sitemap (https://catalogue.cioos.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 13317 ms

Resolved: Cioos National CKAN Sitemap performance has improved in 54aaf2f after 12 minutes.


Support for dynamic extensions

See https://lantern.dev/blog/pglite-lantern for examples of a fork that supports loading postgres extensions.

demo: https://pglite.lantern.dev/
code: https://github.com/lanterndata/pglite-demo
my backup: https://github.com/ouseful-backup/pglite-demo

I've made a start on supporting extensions.
Check availability as:
from jupyter_anywidget_pglite import AVAILABLE_EXTENSIONS

AVAILABLE_EXTENSIONS
>>>['fuzzystrmatch', 'pg_trgm', 'vector']

Then load via the extensions= parameter:
from jupyter_anywidget_pglite import pglite_panel

# Launch it
pg_panel = pglite_panel(extensions=["vector", "fuzzystrmatch", "pg_trgm"])

The required packaes are loaded dynamically:
  async function loadExtensions(extensions) {
    const extensionsObj = {};
    const extensionModules = [
      {
        name: "fuzzystrmatch",
        url<EMAIL_ADDRESS>      },
      {
        name: "pg_trgm",
        url<EMAIL_ADDRESS>      },
      {
        name: "vector",
        url<EMAIL_ADDRESS>      },
    ];

    // Prepare an array of promises for the imports
    const importPromises = extensionModules.map(async ({ name, url }) => {
      if (extensions.includes(name)) {
        try {
          const module = await import(url);
          extensionsObj[name] = module[name];
        } catch (error) {
          handle_error(error);
          console.error(`Failed to load ${name}:`, error);
        }
      }
    });

    // Wait for all import promises to complete
    await Promise.all(importPromises);

    return extensionsObj;
  }

So this approach should allow for us passing in additional, arbitrary extensions, e.g. via a (name, url) two tuple?


caret jumps backwards

I tried to create a mask for numbers with dots as thousands-separator. My mask is in the array-format, and I figured out that each mask entry needs be exactly one character longer:
    [
       '###',
        '####', // workaround to prevent '1.11'
        '#.###',
        '##.###',
        '###.###',
        '####.###', // workaround to prevent '1.000.00'
        '#.###.###',
        '##.###.###',
    ]

The mask itself works fine so far, but if the same character is entered repeatedly, the caret will not end up at the end but move one or two spaces backwards. This seems to happen when the two workaround mask entries are jumped over. If different numbers are entered, the caret stays at the end as intended.
1.0|0
10.000.0|00

I am so sorry, posted this in the wrong project!


README example tense is wrong

nlp.text("She sells seashells").negate()
// She didn't sell seashells

should be:
nlp.text("She sells seashells").negate()
// She doesn't sell seashells

or:
nlp.text("She sells seashells").negate()
// She isn't selling seashells

that's a good point.

so the logic is this -
[past tense] - "sold" -> "didn't sell"
[present] - "sells" -> "doesn't sell"
[future] - "will sell" -> "won't sell"
it may just be that the readme is wrong. I'll take a look this afternoon.

got it. thanks!


Migrate TestBrokerTriggersSink broker e2e test to new reconciler-test framework

PR to migrate the TestBrokerTriggersSink broker e2e test to new reconciler-test framework.
Proposed Changes

:broom: Migrate TestBrokerTriggersSink (0dbf48180916a09a6f4497bfa54c975644fa102c)

Hits for reviewers:
PR is currently WIP as:

requires knative-sandbox/reconciler-test#452 to be merged (got around it temporarily by bfafd744f5c25df1f44cadbdf278838880f9d2ae)

/test unit-tests
/test channel-reconciler-tests-ssl

/test channel-integration-tests-sasl-plain
/test upgrade-tests

/cc

/test upgrade-tests

/test upgrade-tests

/test channel-integration-tests-sasl-plain
/test channel-reconciler-tests-ssl
/test channel-reconciler-tests-sasl-plain
/test upgrade-tests

/test channel-integration-tests-sasl-ssl
/test upgrade-tests

/retest-required


[Docker Hub] Add ability to rename/delete user accounts

Tell us about your request
Users should be able to rename or delete their user accounts.
Which service(s) is this request for?
Docker Hub
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
See https://github.com/docker/hub-feedback/issues/11
Are you currently working around the issue?
NA
Additional context
NA

+1 ∞

Just played with Hub's SSO for the first time, and realized that having SSO choose a username for me makes this even more important (IMO). :see_no_evil:

I'd like to be able to change to Docker hub username too if it's possible. Can I contact anyone who can help me in the meantime? Thank you

This was asked for in June of 2015 (https://github.com/docker/hub-feedback/issues/11). Seven years later and still no resolution?

Is the thumbs up enough or should we all post +1 to increase prio? :p

+1

+1 please

+1

+1

+1

+1

+1

+1
No redirect needed.

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1
They don't even care about it, as time passed and they didn't even commented about it.
IMO, they MUST have this ability and should warn user that changing name could compromise push / pull and they must change all scripts, and warn their own users...

+1

Hi! I work at Docker (but not on the Hub team, so bear with me on details :see_no_evil:). I was actually asking about this issue last week (week before? need to check my Slack history) as I wondered if we made progress on this, and I know it’s a feature that comes up frequently.
I found some people that worked on this, and they gave me some details;
The short version:
There has been some progress, but no ETA yet, and some of this depends on services we integrate with.
The slightly longer version:
The team spent some sprints working on this feature; one of the known complications is situations where images also have to be migrated, so their first goal was to fix the “lower hanging fruit”; accounts that have no content (images) in the namespace. They implemented this feature (allow rename, but only if no content is present, otherwise to contact support). This feature was rolled out in a staging environment, but revealed new challenges for cases where single-sign-on integration is enabled; syncing accounts with those services turned out to be, erm, “challenging” :speak_no_evil:. One of the developers on the team became a victim of this, got locked out of their account, and it took literally months (:grimacing:) to have their account reinstated in those services.
So the “good news” is that the work gave us more insight into the scope of this work, and areas that need to be worked. The bad news is that, while we’re in talks with these providers to work on a solution, we don’t have an ETA yet on when this will be sorted.
TL;DR; This feature definitely remains on the backlog (and knowing myself, I’ll probably continue harassing those working on it :innocent: :joy:), but don’t (yet) know “when”.

Thanks for the information @thaJeztah, I can imagine how complicated this can be as something that was never planned from the beginning (we've been something similar recently 😂)

Yes, with complex, distributed systems, things are rarely "easy". More so, doing without interrupting services (half a billion daily pulls? 😬)

I can imagine how complicated this can be as something that was never planned from the beginning

Big nod.
Fun fact; this ticket was actually created by a former colleague who was Product Manager for Docker Hub. At the time we still had various older systems in service, some of which dated all the way back to "the Docker Index" (the old name of Docker Hub). Those parts were on the roadmap to be replaced "at some point", but also had quite some hard-coded assumptions (who would've thought things could change??), and had to be replaced. The replacement services have been running side-by-side for an extended period of time to verify consistency, and I think we recently were able to decomission the last one of the old services (whoop!).
So, yes, progress has been made, but (per my earlier comment) still lots of "fun" to address.

+1

This very well complies with the recent announcement of deleting Open Source organizations. So now, any public organization on Docker Hub cannot preserve its images naming without paying. We can create a separate personal account, but we cannot rename it to the former org name once all the images are transferred.
If you have a public org on Docker Hub with public images, you're doomed to:

either pay 420 USD / year;
or lost your public images;
or switch to another namespace name, or even container registry

+1

+1

+1

For those leaving a +1; please  use the "thumbs up" emoji on the first comment (as Github allows sorting tickets based on that, and it doesn't send out e-mail notifications to everyone on this thread 😅)

+1

+1

+1

+1

Hi there! I know this is very silly of me, but apparently I accidentally set my username identical to my password, so this feature is incredibly important to me 😬
+1 👍

@capitantoto While you cannot change your username (until this issue gets resolved!), you can change your password!

Yes, this is absolutely true, and luckily it's a unique password just relevant to Docker. That said, I went the easy way and having no images to my name, jsut deactivated the account and created a new one (with a new password!). Thanks for the prompt response.

+1

+1, I can't change email now

+1
when is this going to be done : (

+1
when?

+1

+1
Yesterday would have been great.

+1

+1

For those leaving a +1, please  use  the Thumbs-up emoji in the top-comment instead, which allows the team reviewing the roadmap to sort requests based on "upvotes".

For those leaving a +1, please use the Thumbs-up emoji in the top-comment instead, which allows the team reviewing the roadmap to sort requests based on "upvotes".

However, thumbs-up does not notify me that this issue still relevant to others. I guess relevance is more important where there are more comments than upvotes, because discussion provides engagement. After the idea of adding +1, this issue bacame quickly noticed.

So, I am leaving a +1 in a comment and thumbs up the first comment. This is an important feature.  +1

I always like a constant username across all profiles but typos and unavailability make it hard to stick to that rule.
I hope the devs look into rename/edit usernames possibility.

+1

+1

+1
The namespace value must be changeable. I didn't know that when creating a user, the username will appear in the namespace and the namespace name cannot be changed. I want to be able to change the namespace.

+1

+1

+1 🙇

+1

+1

+1

+1
surprised this isn't possible yet

+1

+1

+1 more please about adding that feature.

+1

+1

It's surprising that this feature request hasn't been resolved after almost a decade. The 2015 issue https://github.com/docker/hub-feedback/issues/11 was closed for this one from 2020, which doesn't make sense to me. Prioritizing some features for the Docker community, such as this one, in the future would be nice. Hopefully it will! Many thanks to the team for the ongoing efforts.

Yeah @daveology it's kind of unfortunate that this hasn't been implemented yet.
Even like, at worse, a compromise could involve reserving the old username and continuing to support pulls, similar to how github will redirect old repository links to the new one.

+1

+1

👍

1

+1

+1

+1

+1

I'm genuinely surprised that this feature STILL isn't possible. Since 2015 this has been requested and it's now 2024?!?! I'm sure it's not a "trivial" ask but even Github eventually solve this 😉
To look at all of the leaps and strides we've made in technology and see that most common of requests - renaming a username - is still a struggle for a $100 million dollar company is rather strange, no?
Anyways, the most frustrating thing a company can do is ignore their user base.
Confronting an issue head on goes such a long way, even if people are upset with the answer. It shows integrity.  But what do I know 🙃

I'm genuinely surprised that this feature STILL isn't possible. This feature has been requested since 2015, and it’s now 2024. I'm sure it's not a "trivial" ask but even Github eventually solve this 😉
I get that this might not be an easy change, but it’s a fundamental feature that so many people have asked for.
It would be great to at least have an update on whether this is being considered or if there are any plans to implement it...

+1

+1

+1

+1

+1

+1

Quite a needed feature


Container Registry on 5005 port

Hello,
I have deployed image-reflector-controller with version: ghcr.io/fluxcd/image-reflector-controller:v0.15.0
When I want to create ImageRepository with port 5005, eg. git.mydomain.com:5005/foo/bar I got message:
✗ .spec.image value should not start with URL scheme; remove 'git.mydomain.com://'
How to fix this?
Ps. Version v0.14.0 works fine.

I'm trying to run the automatic image update also with a private repository on a non-standard port, facing the same issue.

There is a resolution for this issue in process of being merged, see #222 👍

When can I expect new version release with this fix ?

This week we'll release flux 0.26 that will include this fix, you can subscribe to this issue https://github.com/fluxcd/flux2/issues/2308


Trying to use the Proxy for a POST request

I am trying to pass a POST request through the Proxy but get a statuscode 502 Bad Gateway.
Is there anything special that needs to be done to setup the Proxy for a POST request. The mapping is basically like what we use for the GET which works with out issues.

I do believe that a POST request just works, even on a Map() configuration.  Was there maybe some other issue that we resolved already?

@sfossum, can you paste the relevant code?  The number one problem I see with POST requests is caused by reading the RequestBody, usually by using [FromBody], or something like that.
Reading the request body before it is passed to the proxier causes the body to be fully consumed, so proxying usually fails.

I have been able to get it working. I am not sure what was done to make it work. It just started working.
Thanks for the replys.


Library is not in central LuaRocks repository

I couldn't find resty-redis-cluster library in a central LuaRocks repo, is there a reason why it is not uploaded yet?

@toredash @steve0511 can this be done? Would love to help maintain if required

I'm not the maintainer of the project so I cannot assist, sorry.


[AutoPR cost-management/resource-manager] Editing ReportConfigDefinition

Created to sync https://github.com/Azure/azure-rest-api-specs/pull/4788

(message created by the CI based on PR content)


Add WhitePaper in Chinese language

add WhitePaper zh-cn.md
hope that more people from China can get to know XDAG

Thank you.


Update to Frontend 0.0.27-alpha

Update package.json and package-lock.json to install latest GOV.UK Frontend
Created issues for documentation of new features that needs to be created for DS docs:

https://github.com/alphagov/govuk-design-system/issues/231
https://github.com/alphagov/govuk-design-system/issues/232
https://github.com/alphagov/govuk-design-system/issues/234

Trello card: https://trello.com/c/vPvNOxab/896-update-design-system-with-changes-from-frontend

You can preview this change here:
Built with commit 1809a1d82b5befa937c7d5ea0a948e3ff6193bd7
https://deploy-preview-235--govuk-design-system-preview.netlify.com

it's not uptodate with master but it will do :)


Feature: [menu] To do not show tail of long menu item

2 options to develop (for you choose):

replace tail with few ...
Split item to 2 text lines

Here exists workaround. I solved this with adding  into menu item text: Информационныесистемы (ИС).

Here exists workaround. I solved this with adding  into menu item text: Информационныесистемы (ИС).


npm install with latest react fails

npm install --save @react-native-community/masked-view
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@react-native-community/masked-view
npm ERR!   @react-native-community/masked-view@"*" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /Users/njoynjami/.npm/eresolve-report.txt for a full report.
npm ERR!      /Users/njoynjami/.npm/_logs/2021-03-15T13_30_30_753Z-debug.log

unable to start new project due to this

Facing the same issue.

Same problem here

+1

+1

Facing the same issue after upgrading to react 17. Any solution please?
Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@react-native-masked-view/masked-view
npm ERR!   @react-native-masked-view/masked-view@"*" from the root project

Seeing the slow response moved my project to 16.13.1.

Moved back to 16.13.1 by making the following changes in the package.json.
"dependencies": {
"react": "16.13.1",
"react-native": "0.63.4",
}
"devDependencies": {
"react-test-renderer": "16.13.1"
}

While waiting for #108 to be published, we fixed the peer dependecy issue with pointing react-native-community/masked-view to the commit from the PR. Like this:
{
  "dependencies": {
    "@react-native-community/masked-view": "https://github.com/react-native-masked-view/masked-view#a7a088dfffa9d4b35ee40aa3c5910126d442e90d",
  }
}

@react-native-masked-view/masked-view now is 0.2.3. Please use new version

With version 0.2.3, i can not install @react-navigation/stack, please recheck

I am still seeing the following error.
ive-safe-area-context @react-native-community/masked-view
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving<EMAIL_ADDRESS>npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/@react-native-community/masked-view
npm ERR!   @react-native-community/masked-view@"*" from the root project

@react-native-masked-view/masked-view now is 0.2.3. Please use new version

Now there's a dependency problem with @react-navigation/stack

What I did is

removed note modules
removed yarn.lock and npm.lock
closed the project
restarted it
installed node modules using yarn

and its worked for me


Ensure Sequential output components come first in the sorted list

This fixes the inconsistencies we discussed offline.
Also, very minor change, adjusting the rv32i model example to a breaking change in a dependency.

not sure why clippy fails here but not locally, on some untouched line of code no less. hopefully fixed now.


186844045 variable entry modal

Adds functionality to the ... (series) button.
When series button is clicked, modal to enter desired series should appear.
Enter a series in input box. If user clicks Cancel, no change is made to device. If user clicks OK or hits Enter, variable names in device changes to the series entered.

Skipping this review since @lublagg already reviewed it.


File::OpenUri on mac, linux doesn't properly handle %-encoded uris

Per https://github.com/flutter/flutter/pull/19044#issuecomment-403374283, this is breaking pub running in dart2 mode.

cc @zanderso

Does this need to be in Dart2Stable?

Does this need to be in Dart2Stable?

Yes, not only that, but it's also blocking dart roll.

To clarify, @aam is working on a fix here: https://dart-review.googlesource.com/c/sdk/+/64101.

Can this be closed?


[OpenAI GPT3] Bug with form input info tooltips when updating values

Extension – OpenAI GPT3
Author: @simicvm

Description
Steps To Reproduce

Open 'Query OpenAI GPT3' command
Scroll to change Frequency Penalty or Presence Penalty parameter?
Hover over the info icon to the right to observe a tooltip
Remove the existing value in on the parameter
Observe the error message
Add a value as a number
Info icon that activates the tooltip is now gone
Do the steps 4-6 in the other parameter
Observe that the icon for parameter #1 is back, but icon for parameter #2 is now gone

See video: https://share.cleanshot.com/Mt7VLj

The current behavior
Info icon disappears
The expected behavior
Info icon should stay
Raycast version
Version: 1.39.3

Thanks for the report @lazarjov. I am aware of this behavior. It’s a bug in Raycast and the team is working on it.


Unexpected behaviour for window.minimumSize

Describe the bug
window.minimumSize does not take system DPI into consideration. The actual minimum size gets smaller than expected. When converted Dp to physical pixels using LocalDensity, the minimum size works as expected, but the initial window size gets larger than expected.
Affected platforms
Select one of the platforms below:

Desktop

Versions

Kotlin version*: 1.8.20
Compose Multiplatform version*: 1.4.1
OS version(s)* (required for Desktop and iOS issues): Windows 11 Enterprise (22H2)
OS architecture (x86 or arm64):  x86
JDK (for desktop issues): OpenJDK 20.0.1

openjdk version "20.0.1" 2023-04-18
OpenJDK Runtime Environment (build 20.0.1+9-29)
OpenJDK 64-Bit Server VM (build 20.0.1+9-29, mixed mode, sharing)

To Reproduce
The following code gets correct initial window size, but a smaller minimum size on a HIDPI system
fun main() = application {
    Window(
        onCloseRequest = ::exitApplication,
        state = WindowState(width = 800.dp, height = 600.dp)
    ) {
        window.minimumSize = Dimension(800, 600)
    }
}

The following code gets correct minimum window size, but a larger window size on a HIDPI system
fun main() = application {
    Window(
        onCloseRequest = ::exitApplication,
        state = WindowState(width = 800.dp, height = 600.dp)
    ) {
        with(LocalDensity.current) {
            val dim = Dimension(
                800.dp.toPx().toInt(),
                600.dp.toPx().toInt()
            )
            window.minimumSize = dim
            // window.size = dim
            // this has completely no effect
        }
    }
}

Expected behavior
The second code fragment should result in a window with a initial and minimum size of 800dp by 600dp.
Screenshots
A smaller minimum window size (150% HIDPI scaling):

A bigger initial window size (150% HIDPI scaling):

window.minimumSize isn't part of Compose Multiplatform, it is part of JDK. Please, report it here
For minSize provided in Window argument, follow this.
Anyway, thanks for the issue, it should be taken into account when we will implement this.

Please check the following ticket on YouTrack for follow-ups to this issue. GitHub issues will be closed in the coming weeks.


Update link_config script due to moving currencies.yml (closes #642)

@yssavchuk Please, do the next:

Move all configs to lib/config_templates
They must respect actual structure: lib/config_templates/seed/currencies.yml.erb must be rendered into config/seed/currencies.yml. Paths must not be hardcoded, they must be calculated.

lib/config_templates
  deposit_channels.yml.erb
  withdraw_channels.yml.erb
  seed
    currencies.yml.erb


Fix a typo mistake in UI string (mount/fstab extroot advice)

Signed-off-by: Ran Bao<EMAIL_ADDRESS>

merged.


use npm scripts to run bower install

https://github.com/mozillascience/software-discovery-dashboard/pull/37


Fixed the bug of pages rendering at bottom

Fixed the bug of pages rendering at bottom
Solved issue #141

[ x] I have tested the changes locally
[ x] Changes are backward-compatible

https://github.com/VaibhavArora314/StyleShare/assets/154724111/7733d40d-2cf7-48d3-947d-50c3f23995cb

@VaibhavArora314 Pls review the PR!!!!

@VaibhavArora314 Made the changes


Can I use this source code?

Sir, I want to publish these app on Google play store in my account, can you give me a permission?

@lovebdsobuj this source code is already published as app in the play store:
https://play.google.com/store/apps/details?id=com.mikepenz.lollipopshowcase.sample
If you used the source as base for your app and modified it and created something new it's fine.


Add visionOS CI

Should catch regressions in visionOS builds.

That failure was expected. Just merged #2746 in to test the fix against this branch.

Thanks! Glad to see this CI pipeline added :)

This may also be an opportunity to try the new macOS-14 builders (M1-based) for Xcode 15, for faster builds.

This may also be an opportunity to try the new macOS-14 builders (M1-based) for Xcode 15, for faster builds.

Just saw that :smile: We'll give it a shot on another PR once this is merged.

@jshier Taking them for a test run on CasePaths and 15 minutes into a build still waiting for a runner to pick up the job: https://github.com/pointfreeco/swift-case-paths/actions/runs/7719232654/job/21042112183
Probably can't switch till they've been rolled out a bit more thoroughly.

@jshier Taking them for a test run on CasePaths and 15 minutes into a build still waiting for a runner to pick up the job: https://github.com/pointfreeco/swift-case-paths/actions/runs/7719232654/job/21042112183
Probably can't switch till they've been rolled out a bit more thoroughly.

Too bad. Also, since they're macOS 14 images, you likely won't be able to run Xcode 14 builds anyway, so those can stay on the older runners.
(Wish GHA had availability graphs like other providers, just so we could see how backed up they are.)

@jshier Yeah seeing everything queued here, too. I wonder if this is just affecting the @pointfreeco org or if it's site wide?

Too bad. Also, since they're macOS 14 images, you likely won't be able to run Xcode 14 builds anyway, so those can stay on the older runners.

Oh, and according to their runner images, macOS 14 does come installed with Xcode 14:
https://github.com/actions/runner-images/blob/main/images/macos/macos-14-Readme.md
Was just waiting to see if it actually worked.

Oh, and according to their runner images, macOS 14 does come installed with Xcode 14:
https://github.com/actions/runner-images/blob/main/images/macos/macos-14-Readme.md

Odd, given Apple specifically blocks you from running Xcode 14 on macOS 14. I wonder if their docs are just out of date. Guess we'll see.


Slack community

Maybe we will create a community on Slack? I saw it was but now the link is not available. 👍🏻

didn't see this, i made a similar request, would be good to have a community!

Yup. Would be much appreciated.

Let's use Github Issue with question tag for now - See https://github.com/awslabs/deequ/issues/450#issuecomment-1450877217


Use gocache in test and e2e-test Make targets

What this PR does / why we need it:
We started using gocache for the build job with #1557. Since everything seems fine, this PR makes test and e2e-test Make targets download the cache before running tests.
Does this PR introduce a user-facing change?:
NONE

/test pull-kubeone-e2e-aws-containerd-conformance-1.19


Create & save function pane

Under function create section, support creating and autosaving function

Subtasks

Extract our different functions and constants defined in the JS Class
Class name in code section should be uneditable


Is there any incompatibility when upgrading from 2.4 to 2.7 in cluster mode?

Is there any incompatibility when upgrading from 2.4 to 2.7 in cluster mode?

The issue had no activity for 30 days, mark with Stale label.

Closed as stale.
You can read the release policy for version compatibility https://pulsar.apache.org/contribute/release-policy.


Update requirements

@paularamo please approve when you have a chance to review. This should be good to merge tomorrow


[explorer] Epoch and checkpoint fixes

Description
Describe the changes or additions included in this PR.
Test Plan
How did you test the new or updated feature?

If your changes are not user-facing and not a breaking change, you can skip the following section. Otherwise, please indicate what changed, and then add to the Release Notes section as highlighted during the release process.
Type of Change (Check all that apply)

[ ] user-visible impact
[ ] breaking change for a client SDKs
[ ] breaking change for FNs (FN binary must upgrade)
[ ] breaking change for validators or node operators (must upgrade binaries)
[ ] breaking change for on-chain data layout
[ ] necessitate either a data wipe or data migration

Release notes

Current dependencies on/for this PR:

main

PR #10099   👈

This comment was auto-generated by Graphite.


Replace partial_alpha with alpha_mode field

As discussed in https://github.com/FraserLee/bevy_sprite3d/issues/12 there might be a need for more flexibility in which AlphaMode is applied to the sprite material. Having 2 parameters related to alpha would be confusing, so I decided to replace partial_alpha with an optional alpha_mode with docs matching in spirit.
Migration guide:

replace partial_alpha: false with either alpha_mode: None or alpha_mode: AlphaMode::Mask(0.5);
replace partial_alpha: true with alpha_mode: Some(AlphaMode::Blend).

Just a few small nitpicks, this looks great!

Thanks for the PR!

@FraserLee thanks for the quick responses and this amazing crate!


Upgrade bridge solidity version to ^0.8.0

@hujw77 There are conflicts.

merge to https://github.com/darwinia-network/darwinia-bridge-sol/pull/129


Last modified header getting added and removed

When applying a remote resource from another kapitan component (ie. managedset or mustachetemplate), the if-modified-since header that gets added by the remoteresource controller gets removed. This is a side effect of the mergePatch behavior (arrays are completely replaced in a mergPatch).
    spec:
      requests:
      - options:
          headers:
            If-Modified-Since: Mon, 08 Jul 2019 18:50:38 GMT
          url: https://github.com/alewitt2/samples/releases/download/0.0.3/good.yaml

Solutions

hard: add support to StrategicMergePatch code in CompositeController, if one of the kapitan components is being merged, get the live resource, do a strategic merge patch in code, then merge patch the resource
easier: move if-modified-since into status.children and check there to add the header to the request

upgrade to 0.2.4 to use fix


[patch] fixes global config bug introduced in 44c6d9b

@sgress454,
Commit 44c6d9b41e1b1ffd38150788085890c8c2cbe286 doesn't allow for attributes in config/globals.js to be commented out.  This patch changes exposeGlobals.js to make sure each attribute is defined as well as an acceptable data type before throwing an error.

Hi Nick, this is the expected behavior for Sails 1.0 -- if you define sails.config.globals at all, you have to define everything.  The default config/globals.js file in new v1 apps looks like:
module.exports.globals = {
  _: require('lodash'),
  async: require('async'),
  models: true,
  sails: true
};

Since we're working towards modularizing everything (lodash, async and the orm hook are all installed at the app level in v1), we were looking at having to make undefined keys default to false which would be a breaking change anyway.  We decided in this case it's best to just be as explicit as possible.  It's already been pointed out that those error messages are not as clear as they could be, so I'm about to update them.

GlobalGridline patch was fixed tuesday 1/2017morning personal individual pyramid


Fix notice in BoolQuery::getQueries when nothing is added to bool type

Fixes #158.

Thanks for PR!

You're welcome. 😃


Design Login View (UI Only)

This page should handle:

[x] Editing Users Button

[ ] Deleting
[ ] Gray Out Cell, and Edit Label Overlay

[ ] New User Button
[ ] Users Collection View

Once you get the selection of the user from the storyboard working, go ahead and create a new pr for that, so that we can get it linked with the rest of the project. We'll flesh it out in a different pr.


 Guessing the numbers 1-10 within 3 guesses

How do you play your game? A-D to move the arrow, W to guess, S to restart the game

Code
Check off the items that are true.

[x] The game was made using the Sprig editor.
[x] The game is placed in the in the /games directory.
[x] The code is significantly different from all other games in the Sprig gallery (except for games labeled "demo").
[x] The game runs without errors.
[x] The name of the file/game contains only alphanumeric characters, -s, or _s.
[x] The game name is not the same as the others from gallery

Image (If an image is used)

[ ] The image is in the /games/img directory.
[ ] The name of the image matches the name of your file.

Thanks for your PR!

Per @Cosmin-Mare 's comment above, please fix the bug he encountered to meet the bar for being awarded a Sprig device.  Perhaps a new mechanic could be that when people guess wrong, you can tell them if they're hot or cold? (just an idea)


Cleanup unneeded complexity in gardenlet Helm chart

How to categorize this PR?

/area dev-productivity
/kind cleanup technical-debt
What this PR does / why we need it:
This PR cleans up the following unneeded complexity in the gardenlet Helm chart:

Drop useless runtime subchart
Drop useless enabled value
Drop useless version inclusions
Drop useless global.gardenlet structure in values

Release note:

The values for the `gardenlet` Helm chart are no longer put below `.global.gardenlet`. For example, before this PR the replica count was controlled via the `global.gardenlet.replicaCount` value while it's now controlled via `replicaCount` directly. Please adapt your values files accordingly.

/assign

/milestone v1.59

/lgtm
/approve


CP hidden on closed non-binary questions

Current behavior
Two questions from the AIB tournament closed yesterday, but the community prediction is not being shown.
30927:

31010:

These are the first non-binary questions in the AIB tournament, so I suspect question type is a factor.
Expected/desired behavior
I expect to see the chart and details of number of predictors and community prediction, like I do for binary questions.

I concur that this is a bug.

a data problem with how the questions were launched

Yes, that's almost certainly what's going on. If @elisescu can't get to it today, I probably can.

will this be resolved for future non-binary questions on the AIB

Requires tweaks to the launcher (hence calling in Elis)

can these already launched questions have this resolved

Yes. I think all that needs to happen is to go on the admin panel, check the include_bots_in_aggregates field, and save. I believe this triggers a recalc of the CP. If it doesn't, let me know and I'll go in and manually trigger the recalculations.
If any of these questions have resolved, they will have to be unresolved and then re-resolved.

@lsabor I edited it in admin, so bots are included - but the CP didn't update. I didn't see a way to trigger that.
https://www.metaculus.com/questions/30927/what-will-be-the-domestic-earnings-of-a-complete-unknown/

Elis's upgrades to the admin tool work - they include bots in aggregate - but these questions were launched manually, since the tool wasn't ready at the time.


Popup window not closing (Facebook)

Tried to implement facebook authorization and noticed the following things.

The facebook popup window is not closing and stays there.
In facebook developers site when creating the token if I mention the url with sub-directories the token doesn't work and if I mention just as a localhost or www.example.com then it works but then it redirects to the mentioned url within the popup instead of redirecting in the main window.

Not sure how to solve this.
Thanks.

I have experienced the same thing after following the server example provided for PHP.
One thing I noticed is that when I try to do it in incognito the pop-up works fine.

Hello All

The facebook popup window is not closing and stays there in web-app.

have you got any solution?
I am using original satellizer.js file
Thanks

@Bmkaradia Your Site URL in facebook must match the server url (http://<IP_ADDRESS>:8000 !=  http://localhost:8000)


Create input service

Decouple the seed input from the graph service. The input service will be used by the seed form on the query page, the graph when a seed is selected and, if the table's rows are selectable, the table view.

Merged into develop.


To add an array for storing data.

Suggestion to add array to store the data rather than manual updating it on HTML, its to make data easily accessible for every use case.

Suggestion to add array to store the data rather than manual updating it on HTML, its to make data easily accessible for every use case.

The array will have to be made in JS?

Hello, can i work on this ?

Opened a PR #54 to add the database

@MayankBansal12 had started working on this from a long time since we spoke on Discord.

fixed in #54


[Feature][Plugin] Add Plugin Flink Source TiDB

Search before asking

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

Description
Using TiDB native source is batter than using jdbc source
Usage Scenario
No response
Related issues
No response
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

I have tried to implements flink-tidb-connector in flink-1.9.0. Type conversions are hard to solve when use row and dataset. These problems do not appear in the new version of flink such as 1.11/1.12/1.13/1.14. So I think this plugin should implement  in Flink-1.14. I am waiting for this roadmap task finished: https://github.com/apache/incubator-seatunnel/issues/870.

https://github.com/apache/incubator-seatunnel/pull/1089


Small changes in Subject agreement along the text.

Word Choice
-Subject Agreement between vos and tu

[ ] I have read freeCodeCamp's contribution guidelines.
[ ] My pull request has a descriptive title (not a vague title like Update index.md)
[ ] My pull request targets the master branch of freeCodeCamp.
[ ] None of my changes are plagiarized from another source without proper attribution.
[ ] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding.


Only compile HLSL shaders for RenderDemo on windows

Fixes #6 by only compiling HLSL Shaders for the RenderDemo if the project is being built on windows. I've tested this on arch linux and on a Windows 10 VM. Shaders are not built on arch and the project can be built and run successfully. On windows, it attempts to compile shaders, but I think I'm missing some tools so the build fails with the same errors I was getting in #6.

Hm, I think it needs a slightly different fix. With this patch, I don't think the shader files will get copied into the output directory at all, because that happens inside the target down here: https://github.com/mellinoe/veldrid/blob/master/build/ShaderCompilation.targets#L40.
I think what we should be able to do is just condition this definition on "'$(FxcExe)' != '':
https://github.com/mellinoe/veldrid/blob/master/build/ShaderCompilation.targets#L19
That would cause none of the shaders to get processed, but all of them to be copied to the output directory. This would make the sample program work even if you don't have fxc.exe installed on Windows.
I can take a look at making the fix described above soon, probably later today.

Hey @nlowe, I fixed the issue in the master branch. Sorry about that -- I neglected to test that recent change on Linux.

I'll close this, let me know if you are still seeing problems.


Added note about AddCertificateForwarding and certificate validation

Addresses https://github.com/aspnet/AspNetCore.Docs/issues/15974

@blowdart Please review.


Where is  VictoriaMetrics's Data Dir

Question and Steps to reproduce
我参考https://flashcat.cloud/docs/content/flashcat-monitor/nightingale-v7/install/binary/     部署单机版夜莺V7
具体为关系库mysql + 实时库redis + 平台n9e + 时序库VictoriaMetrics + 采集装置categraf
现在跑了有一段时间了
一直有一个疑问，VictoriaMetrics是个单可执行程序连配置文件都没的时序库
那么这段时间的采集数据都存在哪里了呢？
是不是n9e下的integrations目录？
Relevant logs and configurations
n9e
[[Pushgw.Writers]]
# Url = "http://<IP_ADDRESS>:8480/insert/0/prometheus/api/v1/write"
Url = "http://<IP_ADDRESS>:8428/api/v1/write"

categraf
[[writers]]
url = "http://<IP_ADDRESS>:17000/prometheus/v1/write"

Version
序号	组件	名称	版本号
1	实时库	Redis	6.0.8
2	关系库	MySQL	5.7.30
3	平台	Nightingale	7.0.0-beta.9
4	时序库	VictoriaMetrics	1.101.0
5	采集器	Categraf	0.3.69

可以看下 VictoriaMetrics 的官方文档 https://docs.victoriametrics.com/single-server-victoriametrics/#how-to-start-victoriametrics

Question and Steps to reproduce
我参考https://flashcat.cloud/docs/content/flashcat-monitor/nightingale-v7/install/binary/ 部署单机版夜莺V7 具体为关系库mysql + 实时库redis + 平台n9e + 时序库VictoriaMetrics + 采集装置categraf 现在跑了有一段时间了 一直有一个疑问，VictoriaMetrics是个单可执行程序连配置文件都没的时序库 那么这段时间的采集数据都存在哪里了呢？ 是不是n9e下的integrations目录？
Relevant logs and configurations
n9e
[[Pushgw.Writers]]
# Url = "http://<IP_ADDRESS>:8480/insert/0/prometheus/api/v1/write"
Url = "http://<IP_ADDRESS>:8428/api/v1/write"

categraf
[[writers]]
url = "http://<IP_ADDRESS>:17000/prometheus/v1/write"

Version
序号 组件 名称 版本号 1 实时库 Redis 6.0.8 2 关系库 MySQL 5.7.30 3 平台 Nightingale 7.0.0-beta.9 4 时序库 VictoriaMetrics 1.101.0 5 采集器 Categraf 0.3.69

ulric@ulric-flashcat tgz % ./victoria-metrics-prod --help|grep storage
    	The interval for guaranteed saving of in-memory data to disk. The saved data survives unclean shutdown such as OOM crash, hardware reset, SIGKILL, etc. Bigger intervals may help increasing lifetime of flash storage with limited write cycles (e.g. Raspberry PI). Smaller intervals increase disk IO load. Minimum supported value is 1s (default 5s)
    	The number of members in the cluster, which scrape the same targets. If the replication factor is greater than 1, then the deduplication must be enabled at remote storage side. See https://docs.victoriametrics.com/#deduplication (default 1)
    	Optional URL to push metrics exposed at /metrics page. See https://docs.victoriametrics.com/#push-metrics . By default metrics exposed at /metrics page aren't pushed to any remote storage
    	Whether to sort labels for incoming samples before writing them to storage. This may be needed for reducing memory usage at storage when the order of labels in incoming samples is random. For example, if m{k1="v1",k2="v2"} may be sent as m{k2="v2",k1="v1"}. Enabled sorting for labels can slow down ingestion performance a bit
  -storage.cacheSizeIndexDBDataBlocks size
  -storage.cacheSizeIndexDBIndexBlocks size
  -storage.cacheSizeIndexDBTagFilters size
  -storage.cacheSizeStorageTSID size
    	Overrides max size for storage/tsid cache. See https://docs.victoriametrics.com/Single-server-VictoriaMetrics.html#cache-tuning
  -storage.maxDailySeries int
    	The maximum number of unique series can be added to the storage during the last 24 hours. Excess series are logged and dropped. This can be useful for limiting series churn rate. See https://docs.victoriametrics.com/#cardinality-limiter . See also -storage.maxHourlySeries
  -storage.maxHourlySeries int
    	The maximum number of unique series can be added to the storage during the last hour. Excess series are logged and dropped. This can be useful for limiting series cardinality. See https://docs.victoriametrics.com/#cardinality-limiter . See also -storage.maxDailySeries
  -storage.minFreeDiskSpaceBytes size
    	The minimum free disk space at -storageDataPath after which the storage stops accepting new data
  -storageDataPath string
    	Path to storage data (default "victoria-metrics-data")

从最后一个参数 storageDataPath 可以看出来，没有指定的话默认是 victoria-metrics-data

victoria-metrics-data

额，找到了
就在根目录下


[Chia] Investigate wrong counting of missed challenges

The amount of missed challenges seems wrong. It is way to many and the logfiles state that it should be fine. The whole logic needs to be revised and corrected with solid tests. It is most probably best to take logfile scenarios as basis.

The problem is that once a missed challenge occurred there is a stau causing every following challenge to be missed too. This occurs since one list in case of multiple missed challenges is always behind the other list.

@tnt-manu would you be so kind to document your PR changes and the inherent problems we discussed here? Thx :heart:

Previously, the whole time logging was done with two lists on the harvester_info
One list captured all the send messages, the other all the recieved messages. We then compared the last elements to see the latest time delta and put some logic around it - no recieve before send and so on.
This is now changed in PR #22
First, I reduced the lists to variables - so time_last_incoming_msg  for the time of the last incoming, and time_last_outgoing_msg for the time of the last outgoing (farmer - harvester) message.
An additional line_check in the log file has been introduced: ActionFinishedSignagePoint - this comes up on every Signage Point in the logfile of the farmer and marks our entry point for checking if one of the harvester timed_out or missed a challenge.
So on every occurence of this finished sgn point in the log we check for each harvester if the time delta between time_of_end_of_last_sgn_point and time_last_incoming_msg is greater than a threshold (25s for missed challenge and 60s for timeout)
Why all the fuzz? It is because in the event of a cable pull on the harvester we don't get immediate disconnects and would love to see a notification on the discord. Also missed challenges are nice as well as they indicate, how good a harvester is working related to the challenges. But this is still TODO right now. The flags in the FarmerHarvesterLogfile (n_overdue_responses and timed_out ) are tracked with #22 but notification-wise there is no message. n_overdue_responses can be seen with the $farmers command and thats enough for now I guess, but time_outs currently are done by the db and the 60s metric which we know might be off as it tracks sent and recieved timestamps and in the case of a cable pull we have sends but no recieves.

The new counts seem reasonable - so this can be closed


Swift type aliases

Re-opening https://github.com/artsy/artsy.github.io/pull/235

More than 30 ppm of exclamations. Keep them under control.

Preposterous!


[WIP] testing SSR sizing for header icons

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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 62.625%

Totals

Change from base Build<PHONE_NUMBER>:
0%

Covered Lines:
1239

Relevant Lines:
1723

💛  - Coveralls


Added homepage and git repo

Thank you!
@gitcash tip 0.5 dollars to @MissMonacoin

Hey MissMonacoin, user cgcardona tipped you 623 bits in Bitcoin Cash ( ~ $0.5 ).
Click here to claim it!
You can also add the "thumbs down" reaction to cgcardona's comment above to prevent future tips.


neo4j cant mount volumes for /data an /logs works in minikube not in other environments

Is this a request for help?:
Yes
Is this a BUG REPORT or FEATURE REQUEST? (choose one):

Version of Helm and Kubernetes:
Client: &version.Version{SemVer:"v2.8.2", GitCommit:"a80231648a1473929271764b920a8e346f6de844", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.12.3", GitCommit:"eecf22f77df5f65c823aacd2dbd30ae6c65f186e", GitTreeState:"clean"}
kubectl version
Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.3", GitCommit:"2bba0127d85d5a46ab4b778548be28623b32d0b0", GitTreeState:"clean", BuildDate:"2018-05-21T09:17:39Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"windows/amd64"}
Server Version: version.Info{Major:"1", Minor:"12+", GitVersion:"v1.12.3-1+fee411aca93f0e", GitCommit:"fee411aca93f0e786995cc125cf8f9783484f958", GitTreeState:"clean", BuildDate:"2018-12-04T06:45:24Z", GoVersion:"go1.10.4", Compiler:"gc", Platform:"linux/amd64"}
Which chart:
neo4j
What happened:
volume mounts happen in minikube, but doesnt work in other environments
$ kubectl logs -l "app=neo4j,component=core"
Changed password for user 'neo4j'.
chmod: /data/.snapshot: Read-only file system
chmod: /data/.snapshot: Read-only file system
Changed password for user 'neo4j'.
chmod: /data/.snapshot: Read-only file system
chmod: /data/.snapshot: Read-only file system
Changed password for user 'neo4j'.
What you expected to happen:
its supposed work, help me identifying the issue
How to reproduce it (as minimally and precisely as possible):
just running helm install
Anything else we need to know:
as i am not that much experienced in kubernetes it can be simple issue

did you get answer on how to fix this? I have same problem.


No executable found matching command "dotnet-electronize"

open dos command prompt
dotnet add package ElectronNET.API
dotnet tool install ElectronNET.CLI -g

You can invoke the tool using the following command: electronize
Tool 'electronnet.cli' (version '0.0.10') was successfully installed.

dotnet tool list -g

Package Id            Version      Commands
----------------------------------------------
electronnet.cli       0.0.10       electronize

dotnet electronize

No executable found matching command "dotnet-electronize"

electronize

Electron.NET Tools (0.0.10)
Project Home: https://github.com/ElectronNET/Electron.NET
etc...

Is there anything wrong with what I did for electronize tool not working with dotnet command?

Try call it without dotnet key,
Or modify launchSettings.json
"Electron App": {
     "commandName": "Executable",
     "executablePath": "C:\\Users\\<user name>\\.dotnet\\tools\\electronize.exe",
     "commandLineArgs": "start <path to core app>"
   }

Works for me

solution:
C:\Users<username>.dotnet\tools and rename or clone the electronize.exe to be called dotnet-electronize.exe
PS C:\Users\ruohk> dotnet electronize
Keine ausführbare Datei zum Befehl "dotnet-electronize" gefunden.
PS C:\Users\ruohk> dotnet electronize
Electron.NET Tools (0.0.10)
Project Home: https://github.com/ElectronNET/Electron.NET

Commands to start the Electron Application:

        start                   Start your ASP.NET Core Application with Electron, without package it as a single exe. Faster for development.

Commands to build the Electron Application:

        build                   Build your Electron Application.

Commands to init the Electron Application:

        init                    Creates the needed Electron.NET config for your Electron Application.

Commands to see the current ElectronNET version number:

        version                 Displays the ElectronNET.CLI version

To get help on individual commands execute:
        dotnet electronize help <command>

@golyakho , @ruohki I already had the work around as I mentioned on the post. But still, I opened an issue for the pb to be solved at the next release so that either we can make a dotnet electronize by following what the document mentions, or so that the documentation can be updated. Anyway, the goal is that the behavior matches what is in the documentation :).

Ah - sorry for this misunderstanding. With the most recent version it is just "electronize". "dotnet electronize" was needed in the pre-tools .NET Core age.
Maybe checkout the PR #192
I hope this change will clear some questions.

The PR is merged - please let me know if you would like further changes.

Thanks a lot @robertmuehsig , actually having the issue #188 solved would be good as it is an almost blocking bug to build any package on a fresh project. Very easy solution as I mentioned in my analysis: https://github.com/ElectronNET/Electron.NET/issues/188#issuecomment-449859346


Set version to 0.32.0-b11

[Current coverage][cc-pull] is 68.73%

Merging [#1694][cc-pull] into [develop][cc-base-branch] will increase coverage by +<.01%

2 files in ...c/ripple/shamap/impl were modified. more

Misses -3
Hits +2

1 files in src/ripple/shamap were modified. more

Misses -2
Hits +2

3 files in ...ipple/app/paths/impl were modified. more

Misses +1
Hits -1

1 files in src/ripple/app were modified. more

Misses -12
Hits +12

1 files in src/ripple were modified. more

Hits +1

3 files (not in diff) in ...ripple/protocol/impl were modified. more

Misses -11
Hits +11

1 files (not in diff) in src/ripple/protocol were modified. more

Misses -3
Hits +3

2 files (not in diff) in ...ple/app/paths/cursor were modified. more

Misses -21
Hits +21

2 files (not in diff) in src/ripple/app were modified. more

Misses +2
Hits -2

4 files (not in diff) in src/ripple were modified. more

Misses -47
Hits +47

@@            develop      #1694   diff @@
==========================================
  Files           866        862     -4
  Lines         61435      61423    -12
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits          42178      42216    +38
+ Misses        19257      19207    -50
  Partials          0          0

[][cc-pull]

Powered by Codecov. Last updated by [36ac112...3d5d7bf][cc-compare]
[cc-base-branch]: https://codecov.io/gh/ripple/rippled/branch/develop?src=pr
[cc-compare]: https://codecov.io/gh/ripple/rippled/compare/36ac1124f4b4b96a50b1f596d4acb7880803e870...3d5d7bf557110bdc744e37267b1b15247977179d
[cc-pull]: https://codecov.io/gh/ripple/rippled/pull/1694?src=pr


How i can add multiple lines in the editor which can be visible in the preview section as well.

Describe your question
Currently, we are not able to add multiple lines in the editor which can be seen in the preview section. So, is there any way to add multiple lines in the editor that can be visible in the preview section as well?
Multiple Enter keys are not working in the editor.

This is a quirk (feature!) of Markdown, multi empty lines are treated as one line.
You can instead use a
 tag in Markdown to create an empty line. The
 will place an empty line in the preview.
So for example:
First line

Low text

Will result in:

https://developer.mozilla.org/en-US/docs/Web/HTML/Element/br


[WIP - GH-42] Add Service Bus Subscription Count Client

Allow for users of the metrics adapter to gather message counts from Azure Service Bus Topic Subscriptions to decide their scaling targets
Closes #42

Based on #39, this is the structure I'd see for configuring the CRD
apiVersion: azure.com/v1alpha2
kind: ExternalMetric
metadata:
  name: example-external-metric-service-bus-subscription
spec:
  type: servicebussubscription
  azure:
    resourceGroup: storage-rg
    namespace: sample-ns
    topic: sample-topic
    subscription: sample-subscription
  metric:
    # This would default to activeMessageCount, but could be updated to one of the counts from https://github.com/Azure/azure-sdk-for-go/blob/master/services/servicebus/mgmt/2017-04-01/servicebus/models.go#L1116
    field: activeMessageCount

@jsturtevant currently didn't link up the providers just yet as I didn't want to get too far into the weeds without agreement from your end. I'll keep it moving forward based on the initial feedback

Looks good to move forward on.  Let me know if there are any challenges or blockers.   Thanks again!

Just checking in to see of there are any blockers I can help with.

No blockers currently. I got backlogged on professional and personal commitments and will be getting back to this PR.

I am closing this as I did not have permissions to push changes to @marc-sensenich branch.  Thanks for your initial implementation!

Thanks for taking this over @jsturtevant!


[KO] Added Korean translation and index page

@rijgndqw012 and some of my friends found a great documentation of Elixir programming language which wants to be translated into Korean(#201), and started translating to it.
I'm glad to translate Elixir School in Korean! :rocket:

This is awesome :boom:!  Thank you @devleoper & @rijgndqw012


Dependabot batched updates

More dependabot updates.

@paul-sachs not sure why the browserstack stuff is failing, but the karma updates could be the culprit maybe?

@paul-sachs not sure why the browserstack stuff is failing, but the karma updates could be the culprit maybe?

Eh, who knows. I'll dig in.

So it seems the culprit is<EMAIL_ADDRESS>Not exactly sure where or how, but it's causing safari 11.1 to hang during testing.

So finally just hooked up browserstack and pointed it at the karma runner and it looks like we've got some syntax that Safari doesn't like:

No idea why upgrading these deps made it worse but it looks like Safari 11.1 may not have ever worked properly and it was just not getting hit properly before, or there's a flag I need to use to compile this syntax.


Bugfix/publish charging schedules on startup

Describe your changes
Issue ticket number and link
Checklist before requesting a review

[x] I have performed a self-review of my code
[x] I have made corresponding changes to the documentation
[x] I read the contribution documentation and made sure that my changes meet its requirements

Hi, have some question for this. Should it send charging schedules if the stations don't want to support smart charging?


Make tests working for us

Fix single test launch (no need to add --harmony). It should have been fix with dfa70e7 but it doesn't (see https://github.com/facebook/jest/pull/265 , need investigations)
Update jsdom (commits cherry-picked from the jsdom-update branch). No commit on this branch since january...

Tag v0.5.0


The 'Stop Generate Prompt' function is not working

CancelData? _cancelData;
final stream = OpenAIApp().I.onChatCompletionSSE(
            request: request,
            onCancel: (cancelData) {
              _cancelData = cancelData;
            },
          );
stream.listen(
  (it) async {
    if (_cancelData != null) {
      _cancelData?.cancelToken.cancel("canceled "); // It's not working
    }
    controllerGen.sink.add(sum += (it.choices?.last.message?.content ?? ""));
    isContinue = it.choices?.last.finishReason == "length" ? true : false;
  },....

@difeW your response is success before stop. you try  long question

I had the same issue with long answers. I can see the mCancel?.cancelToken.cancel("canceled "); executed but answers keep coming from openAI.onChatCompletionSSE method.

After some research, I've come to realize that if you update to the latest version of the chat_gpt_sdk, Dio in this version will be updated accordingly. The new version of Dio no longer allows canceling the connection after it's been established; the only way is to cancel the currently listening stream. My issue has been resolved.


api api import error

API-Manager CLI Version
1.2.1
API-Manager and Service-Pack Version
7.7.0
Question
When running the apim api import command on Linux, i'm getting the error message:
Using java given in JAVA_HOME

API-Manager CLI: 1.2.1
To report issues or get help, please visit:
https://github.com/Axway-API-Management-Plus/apim-cli
Module: API - I M P O R T (1.2.1)
0      INFO     APIManagerAdapter| Successfully connected to API-Manager (7.7.0) on: https://apimanager-hml.autoglass.com.br:443
7      ERROR         APIImportApp| Cant parse JSON-Config file(s)
8      ERROR         APIImportApp| Cant parse JSON-Config file(s)
com.axway.apim.lib.errorHandling.AppException: Cant parse JSON-Config file(s)
at com.axway.apim.apiimport.APIImportConfigAdapter.(APIImportConfigAdapter.java:163)
at com.axway.apim.APIImportApp.importAPI(APIImportApp.java:66)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at java.base/jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at java.base/jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.base/java.lang.reflect.Method.invoke(Method.java:564)
at com.axway.apim.cli.APIManagerCLI.run(APIManagerCLI.java:142)
at com.axway.apim.cli.APIManagerCLI.main(APIManagerCLI.java:78)
Caused by: com.axway.apim.lib.errorHandling.AppException: Unable to find given Config-File: './import.config-hml.json'
at com.axway.apim.apiimport.APIImportConfigAdapter.locateAPIConfigFile(APIImportConfigAdapter.java:178)
at com.axway.apim.apiimport.APIImportConfigAdapter.(APIImportConfigAdapter.java:147)
What I've tried so far
Configured correctly the JAVA_HOME environment variable
Tried to execute the script from various locations
Verified the config file permissions
Additional information

Can you please provide the complete usage?
The error you get, basically means, that the API-Configuration file cannot be found.
You need to provide the config file with the parameter -c

Sure!
The command i'm using is:
apim.sh api import -c ./import.config-hml.json -h apimanager-hml -u apiadmin -p ********
The configuration file is located in current directory

Are you sure, the file: import.config-hml.json really exists?

Yes! It does exists and is located in current directory. :)

Can you please rename that file like so:
import-config-hml.json
Perhaps the code gets “confused”, as the DOT is also used to find stage-files.

Got it
Done, but, with same result.

Hmm, let me check and I come back to you here.

The Java version haves some interference in this case?
I'm using the Open JDK <IP_ADDRESS>-2

Hi @RodrigoMachadoSilva, I don't think it is related to the Java-Version you are using. I have realized, that I don't throw the causing exception, which I have improved now. Can you please re-run with the version attached here, as it will trace the real cause, why the file can't be loaded/located.
axway-apimcli-1.3.0-SNAPSHOT.zip

Ok @cwiechmann , i will do that right away
Thanks

@cwiechmann, i tried with the given version of CLI:

I must add that on windows everything works properly.

Okay. I found the issue and it's caused by having an undiscovered difference between the Windows apim.bat and the apim.sh. The Windows-Version is saving & restoring the current path, which the Shell-Script doesn't. Interesting, that nobody came across this issue before.
It really helped to know, that it works on Windows!

I feel really glad to contribute with the tool, do you suggest moving forward with the windows version?

Hi @RodrigoMachadoSilva, I improved the apim.sh script to work in the same way, as apim.bat. Could you please try on your side as well to validate it is now working as expected?
You can download the latest version from here: https://github.com/Axway-API-Management-Plus/apim-cli/blob/develop/modules/apim-cli/assembly/scripts/apim.sh
Thanks!

Concerning your question: No, the Linux Version must work as well.
I'm really thankful for your help discovering the problem.

Worked perfectly @cwiechmann, thank you!

Hi @cwiechmann
I had the similar issue where it is working in windows and encountering the "Can't parse JSON Config file" in Linux.  I did updated the apim.sh with reference to link mentioned above.  I am still getting the same error.
Details:
API-Manager CLI Version
1.2.1
API-Manager
7.7 July update
Question
When running the apim api import command on Linux, i'm getting the error message: "Can't parse JSON Config file"

@cabhilash-axway, I'm confused by your apimCLIHome as it points to a relative path. And I'm asking myself: Relative to what?
When you run an ls to the constructed filename given in the yellow line - Does this file then exists?

@cwiechmann I am building a CICD pipeline in GitLab.  Yes file does exist, When i run an ls on apimCLIHome.

What I meant is: ls apim-cli-1.2.1/samples/basic..../minimal-api-definition-config.json
I think the apimCLIHome is wrong. I think it should be an absolute location and not something relative.

I run ls on that path

Thanks @cwiechmann, It worked with absolute path


MSID: 94698 Version: 2 DOI: 10.1101/2<IP_ADDRESS>5925

MSID: 94698
Version: 2
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>5925
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=94698
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>5925

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/94698v2
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>5925
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/94698/v2/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

warn: [p-all-bold] Content of p element is entirely in bold - 'MDAR checklist'. Is this correct?
error: [junk-character-presence-3] xref element contains a junk character '□' which should be replaced or deleted.
error: [junk-character-presence-3] fn element contains a junk character '□' which should be replaced or deleted.


Architecture: Schematic diagram is outdated

There is an outdated architecture diagram of the deno process.

According to Releases.md libdeno had been refactored and now deno uses rusty_v8 for V8's C++ API bindings.

v0.29.0 / 2020.01.09

refactor: Replace libdeno with rusty_v8 (#3556, #3601, #3602, #3605, #3611,
#3613, #3615)

We're aware of this, it's listed to be updated in https://github.com/denoland/deno/issues/6844

Ok, sorry, haven't seen this to be listed in the deno repository. Closing the issue.


[user-story] Support HTTP '409 Conflict' Errors when updating an entity

The backend server should return a 409 Conflict error when a user attempts to update a versioned entity (e.g., a work item) that was already updated by another user since her last GET request. This prevents erasing changes from other users who edited the same entity.
sub-tasks:

[x] Support HTTP '409 Conflict' Errors when updating a work item (#1247)
[x] Support HTTP '409 Conflict' Errors when updating a work item during reordering (#1363)
[x] Support HTTP '409 Conflict' Errors when updating a work item link (#1280)
[x] Support HTTP '409 Conflict' Errors when updating a work item link type (#1311)
[x] Support HTTP '409 Conflict' Errors when updating a work item link category (#1317)
[x] Support HTTP '409 Conflict' Errors when updating a space (#1319)

Other entities such as User, Comment or Iteration do not have a Version field. Other endpoints such as /areas do not support the PATCH method.

+1

Closing for now since all tasks/sub-issues have been addressed.


add #[pymodule] mod some_module { ... }

Implements using #[pymodule] directly on modules.
Allows importing other wrapped items with a #[pyo3] use foo;. (Builds on #2366 to do this.)
Allows having a #[pymodule_init] function to run arbitrary code after adding all wrapped items.
Still very unfinished, hopefully it's an interesting proof-of-concept.
See the declarative module in test_module.rs and also pytests/src/lib.rs for examples of how this could be used.

Superseded by #3294


Fullscreen not working in Chrome extension

Reported here:
https://github.com/readium/readium-js-viewer/issues/344

This was probably already fixed in #354, The report you link to is from may, that issue was fixed in June.

 Thank you Ryan.


Add Mono

Relevant discussion: https://github.com/mono/website/issues/199
I've also added instructions and a reference to the Mono windows install documentation. Fortunately, this also means that the "third Mono language" turbo works, but that's not relevant for this document.
Mono GUIs (Windows Forms) run with an X server running on the windows side.

Wow, that pretty awesome. Thanks so much!


[Possible Feature Request] LISTENING and WATCHING RPC's

Hi! On Discord, there are multiple types of RPC as in PLAYING, LISTENING, and WATCHING. Currently, from my research, you can only set your RPC to PLAYING using this library. Is there anyway you could allow users to set their RPC's to "LISTENING" or "WATCHING"? Thanks!

Closing after only being open for 2 minutes after finding out Discord has this locked down to only certain integrations.
To quote Discord, "ActivityType is strictly for the purpose of handling events that you receive from Discord; though the SDK/our API will not reject a payload with an ActivityType sent, it will be discarded and will not change anything in the client." - https://discord.com/developers/docs/game-sdk/activities#data-models-activitytype-enum


Add my blog into showcase

The URL of my blog is: https://surfzjy.ghost.io

try again when you have around then posts please :-)


file $ref in relative $ref always raises "property #/soandso/0 was not a valid schema"

Hi - I think this might be an instance of #190, but I'm not sure.  I apologize if it is a duplicate.
The gist is that when I have a $ref to somewhere within the file, if the object described has a $ref to another schema, then the same error is always thrown regardless of if the file is valid or not.
To demonstrate this, I have this script:
require 'json'
require 'json-schema'

schema = JSON.load <<-END_JSON
{
  "defs" : {
    "this_works_ok" : {
      "item" : "number"
    },
    "this_fails" : {
      "$ref" : "test_schema.json"
    },
    "some_contrived_schema": {
      "type" : "object",
      "properties" : {
        "entities" : {
          "type": "array",
          "items" : { "$ref" : "#/defs/this_works_ok" }
        }
      }
    }
  },

  "allOf" : [
    { "$ref" : "#/defs/some_contrived_schema" }
  ]
}
END_JSON

data = JSON.load <<-END_JSON
{ "entities" : [1,2,3,4] }
END_JSON

errors = JSON::Validator.fully_validate(schema, data, strict: true)
puts errors

And this external schema saved as "test_schema.json":
{
  "item" : "number"
}

The above runs ok, but if you change
"$ref" : "#/defs/this_works_ok"

to
"$ref" : "#/defs/this_fails"

Then the following error is produced:
The property '#/' of type Hash did not match all of the required schemas. The schema specific errors were:

- allOf #0:
    - The property '#/entities/3' was not a valid schema
    - The property '#/entities/2' was not a valid schema
    - The property '#/entities/1' was not a valid schema
    - The property '#/entities/0' was not a valid schema

Interestingly, it throws that error regardless of if the file path is valid or not.
The above is kind of contrived, but demonstrates the problem I'm running into - the schemas I'm trying to use 'allOf' to combine schemas together so that parts of them can be re-used elsewhere.  The ref within the 'allOf' then points within that file to any new behavior, and objects that would be list items are broken out into their own files as well.  If there is another way to express file paths that would validate, or if there is another approach to structuring this, that would be very helpful.
Thanks for taking the time to consider the above.

Wow that's an interesting example. It does look like it's getting confused while following refs doesn't it? The example is helpful and should allow me to reproduce the problem in a test and then I can fix it from there. I'll let you know how it goes...

I think I hit this. I was able to get around it by loading the shared / referenced schema file (using JSON::Validator.add_schema). I had to make sure the file reference was the same. In my case it is a long file:some_dir/dir2/file.json and had to make sure the URI as given when creating the schema to load was the same as what is used in the schemas that used it. But that made it work.

ludwick: my work around was to construct my schemas such that if you make a relative reference to an object, that object does not make a file reference.  References to files that themselves contain relative references work fine.


Add the option to set client.id to storm-kafka and storm-kafka-client

Add the ability to set the Kafka client.id property to storm-kafka and storm-kafka-client (https://issues.apache.org/jira/browse/STORM-2524).
storm-kafka example:
	SpoutConfig spoutConfig = new SpoutConfig(
		brokers,
		topicConfs.topic,
+		"client_id-" + confs.consumerGroupId + "-" + topicConfs.topic,
		"/consumers",
		confs.consumerGroupId + "-" + topicConfs.topic
	);

storm-kafka-client example:
	KafkaSpoutConfig<String,String> kafkaSpoutConfig = KafkaSpoutConfig.builder(KAFKA_LOCAL_BROKER, TOPIC)
		.setGroupId(confs.consumerGroupId + "-" + topicConfs.topic)
+		.setClientId("client_id-" + confs.consumerGroupId + "-" + topicConfs.topic)
		.build();

@carl34  We already have setProp method in KafkaSpoutConfig in storm-kafka-client that can be used to set any producer property. I dont think this adds any value. Can you elaborate on why you need that method? As far as the storm-kafka module is concerned that property is already public. I dont see a reason for adding one more constructor argument.

@priyank5485, setting client.id is useful for identifying requests Kafka protocol packets.  For instance, we have many topologies which run on a shared Storm cluster using storm-kafka and all of the fetch requests contain blank "Client ID:" values.  We would like to have a clear way for our developers to set client.id so that we can quickly determine the source of Kafka packets.
For storm-kafka-client, I get your point that setProp can be used, but we would like to encourage our users to set this parameter so we would appreciate if it was clearly available with the builder pattern via intellisense.
Here is an example fetch request from storm-kafka without a client.id:
Kafka (Fetch Request)
    Length: 54
    API Key: Fetch (1)
    API Version: 0
    Correlation ID: 0
    String Length: 0
    Client ID:
    Replica ID: -1
    Max Wait Time: 10000
    Min Bytes: 0
    Array Count: 1
    Fetch Request Topic (1 partitions)
        String Length: 6
        Topic Name: canary
        Array Count: 1
        Fetch Request Partition (Partition-ID=2, Offset=162438425)
            Partition ID: 2
            Offset: 162438425
            Max Bytes: 1048576

@carl34 https://github.com/apache/storm/blob/master/external/storm-kafka/src/jvm/org/apache/storm/kafka/KafkaConfig.java#L32 is already a public property. Are you saying passing it as a constructor argument is more clear than setting a public property? If yes, I dont agree with you on that. That extra constructor does not add any value. Plus, community is encouraged to use new spout. So unless there is a blocker or critical bug in the old spout i prefer not changing any code there.
For storm-kafka-client I plan to raise a PR to get rid of the builder pattern. Builder pattern relies on builder.build() and pass the returned object to the constructor. Unfortunately, that does not work well with flux. Hence setProp method, either on the builder (or may be in future directly on KafkaSpoutConfig) is encouraged. There are so many other properties on kafka consumer api. It does not make sense to expose each one of those as a setXXX java method. Without those setXXX methods it is a much cleaner api for spout and its config object. Intellisense should not be the top priority here. I would prefer a cleaner api.

Thanks for the additional context.  Our primary objective is to find a way to update our legacy topologies which use storm-kafka, but it looks like the clientId member is final in 1.1.0, correct?
https://github.com/apache/storm/blob/a4afacd9617d620f50cf026fc599821f7ac25c79/external/storm-kafka/src/jvm/org/apache/storm/kafka/KafkaConfig.java#L32
We were initially just going to submit a PR for storm-kafka, but we're hoping to make it clear to new topology creators that client.id is available because we feel that it is valuable.  If the builder pattern is going to be removed, would it be a big deal to add this change in the mean-time?

@carl34 Good point. I did not realize it was final until you brought it up. In that case i am okay with your change for storm-kafka module. For storm-kafka-client module i still prefer avoiding the method setCliendId. I would rather use setProp instead. Can you use setProp instead on the builder for now? And when i create a PR for refactoring storm-kafka-client you can comment about any issues that you might run in to because of the changes in the PR.

Also please upmerge there is a merge conflict.

Added javadocs and rebased

@priyank5485 sounds good, I went ahead and removed the storm-kafka-client change so that only storm-kafka is updated and we will plan to use setProp.

I can try rebasing again for the CHANGELOG.md updates, please let me know if I should add a new entry or to that file or if there are any other actions for me to take.


How to use ICMP modules

Hello:
How to use ICMP modules? I compile ICMP modules,but i don't know how to use it

Hi, if I remember correctly there are two steps:

Compile and run ./serve (./serve  )
ping <IP_ADDRESS> -p  -s 10
(Suterusu is basically casting the payload of the ICMP packet to https://github.com/mncoppola/suterusu/blob/master/icmp.c#L9 and checking against that.  It also verifies the payload is 10 bytes large, i.e. AUTH_TOKEN=4, IP address=4, port number=2)

Then Suterusu should receive the ICMP packet, detect that it's a special packet, and download + execute the binary.  If you're having issues getting it to trigger, I'd recommend turning on debugging and watching dmesg.

I got it,Thank you!


Setting NULL throws NRE

Hello.
Am I doing something wrong?
Both tests are failed.
        [TestMethod]
        public void Change_Property_From_Null_To_Null_Should_Not_Throw()
        {
            var trackable = new Order { Id = 321,  }.AsTrackable();

            trackable.Invoking(o => o.Address = null).ShouldNotThrow<NullReferenceException>();
        }

        [TestMethod]
        public void Change_Property_From_Value_To_Null_Should_Not_Throw()
        {
            var trackable = new Order { Id = 321, Address = new Address() }.AsTrackable();

            trackable.Invoking(o => o.Address = null).ShouldNotThrow<NullReferenceException>();
        }

Thanks for the bug report.
Fixed in https://github.com/joelweiss/ChangeTracking/commit/867e076cc4311360ca4da1d27e82826d4a58d8ee
Please update to version 2.0.6 on nuget


It would be nice to specify columns

One feature that would be useful is to specify the exact columns.  Any column not listed would be ignored by BulkExtensions which would allow the database to set the default values instead of EFCore.  For example, I am having an issue while upserting an Sql Server timestamp column where the default value is generated in the temp table and then fails when merging into the actual table because timestamps can't be explicitly inserted.
I would like to allow BulkConfig to override the list of columns to update.

I understand the issue.
However proper solution for the situation you mentioned is actually to omit Properties with default value from inserting if their Columns have default value in Sql.
This is how regular EFCore Insert behaves so it's logical to make the same behavior.
I will see to implement this fix soon.

@borisdj Can you please specify on how to contact you by email to continue our discussion. Big thanks

Added feature that allows specifying exact columns to update.
Published NuGet 2.0.5

I am trying the new thing but it throws an exception of:
"Dictionary key not found"
What am I doing wrong?
This is a snippet of my code:
_context.BulkUpdate(productsToUpdate , new BulkConfig {  BulkCopyTimeout = 500, PropertiesToInclude = new List<string> { nameof(WalmartProduct.BuyBoxPrice) } });

You need to add PrimaryKey to list as well.
For example if PK is WalmartProductId then it would be:
_context.BulkUpdate(
    productsToUpdate,
    new BulkConfig {
        BulkCopyTimeout = 500,
        PropertiesToInclude = new List<string> { nameof(WalmartProduct.WalmartProductId), nameof(WalmartProduct.BuyBoxPrice) }
    }
);

But now I see that I could implement this to be done automatically.
Add the moment just add this, and when I update the library I will write it here.
In addition regarding config UpdateByProperties purpose of it is when we want Update to be done not ny OK but by one other, or several combined columns, that should be effectively unique.
For example in your case if there would be another column say WalmartProductCode then:
_context.BulkUpdate(
                    productsToUpdate,
                    new BulkConfig {
                        BulkCopyTimeout = 500,
                        UpdateByProperties = new List<string> { nameof(WalmartProduct.WalmartProductCode) },
                        PropertiesToInclude = new List<string> { nameof(WalmartProduct.WalmartProductCode), nameof(WalmartProduct.BuyBoxPrice) }
                    }
                );

@nafberger when using PropertiesToInclude, PrimaryKey column now(from 2.0.6 version) does not need to be explicitly added to UpdateByProperties list.
If not found in PropertiesToInclude list it will be automatically added in it from UpdateByProperties list.

And that is even with a composite key. Also what happens if I do include the PK's? Thanks for your help

You can include it, it will still work the same, it's just not required any more.
What it now does in the background is that it compares 2 lists: PrimaryKey list (composite key supported) and PropertiesToInclude list, and if it finds elements in first list that are not in the second it will add them there as well.

Aha. Not sure if it's my problem, I upgraded to the latest version and it
threw an error when I didn't include the PK in the properties list, not
really sure if it's my problem or not.
On Wed, Mar 28, 2018 at 2:21 AM, Boris Djurdjevic<EMAIL_ADDRESS>wrote:

You can include it, it will still work the same, it's just not required
any more.
What it now does in the background is that it compares 2 lists: PrimaryKey
list (composite key supported) and PropertiesToInclude list, and if it
finds elements in first list that are not in the second it will add them
there as well.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/borisdj/EFCore.BulkExtensions/issues/25#issuecomment-376775091,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AY6XK-RPzT0c_stzlNfkMbOR5PFSKbwrks5tiyvzgaJpZM4ROZ7E
.

What was the error? the same as you would get on previous versions when PK is omitted from the list or something else now.

I will try to post a screenshot
On Wed, Mar 28, 2018 at 1:54 PM, Boris Djurdjevic<EMAIL_ADDRESS>wrote:

What was the error? the same as you would get on previous versions when PK
is omitted from the list or something else now.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/borisdj/EFCore.BulkExtensions/issues/25#issuecomment-376977257,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AY6XK8Yv-NFz3oEN4sLpOpq47WfMM_icks5ti844gaJpZM4ROZ7E
.

You are right, it still throws an error.
I will have to make another fix, expect 2.0.7. soon.