Polygon does not close, and 'equals' function does not work

When pushing 4 vertices in the the polygon function, though the output should be a tuple with 5 elements, It seems that the order of the input vertices passed to the function has impact the result: sometime the polygon is described with 5 vertices and sometimes with 4 - meaning, it`s not a closed polygon (or in other words - it is not a polygon at all) It must be some bug.
Example:
from shapely.geometry import Polygon

print ('poly1 = ', Polygon([(620, 420, 500), (620, 420, 0), (620, 40, 0),(620, 40, 500)]))
print ('poly2 = ',Polygon([(620, 40, 500), (620, 420, 500), (620, 420, 0), (620, 40, 0)]))

>>poly1 =  POLYGON Z ((620 420 500, 620 420 0, 620 40 0, 620 40 500, 620 420 500))
>>poly2 =  POLYGON Z ((620 40 500, 620 420 500, 620 420 0, 620 40 0))

Also, when using .equals function - it doesn`t always work:
poly1 =  Polygon(([220.0, 400, 500], [220.0, 20, 500], [220.0, 20, 0], [220.0, 400, 0], [220.0, 400, 500]))
poly2 =  Polygon(([220.0, 20, 500], [220.0, 400, 500], [220.0, 400, 0], [220.0, 20, 0], [220.0, 20, 500]))

print (poly1.equals(poly2))

gives a False

@YairSchwartz Something I haven't explained very well about Shapely is that it is a 2D geometry library where Z values are ignored by most of the constructive and analytic algorithms. Your polygons aren't strictly valid in 2D: (620, 420, 500) and (620, 420, 0) are considered to be the same point and so your Polygons don't have the required number of vertices. They each have only 2 unique vertices.
>>> Polygon([(620, 420, 500), (620, 420, 0), (620, 40, 0),(620, 40, 500)]).is_valid
Too few points in geometry component at or near point 620 420 500
False

I think it's clear Shapely is not doing the right thing here and should raise an error instead of leaving you with perplexing and invalid polygons.

We'll get this done for 1.6.3. Plan: make a list of coordinates with unique x, y values and check the length of that, raising a ValueError if it isn't 3 for linear rings and polygons and if it isn't 2 for line strings.

After some review of the code, this is a can of worms. Here's the deal:
Shapely currently allows you to work in "anything goes" mode. You can easily create invalid or pathological geometries like LineString([(0, 0), (0, 0)]) and get no exceptions or warnings. If you want to work in a more strict mode, you can check the validity of objects (geom.is_valid or shapely.validation.explain_validity(geom)). Our reasoning is that checking validity is an extra expense for programmers who are certain to have valid geometries.
I'm beginning to realize that checking for the xy-uniqueness of coordinates that have Z values can't fully guard against invalidity unless we check every coordinate tuple, and even then doesn't guard against accidental self-intersections. It's not an adequate substitute for calling is_valid post-creation. I think more documentation will be the solution to this issue.

Resolved by ae435c4.


Improve determination of compatible API version

Sync Endpoint API requires the client to provide an API version, known internally as odkClientVersion or odkClientApiVersion, currently the only valid version is 2. Its usage is documented on the Sync Protocol page.
ODK-X Services determines the API version by truncating off the last 2 digits of the release rev version, e.g. version 2.1.5 rev236 ⇒ 2. Services could use a more robust method to determine the API version for example storing the version in a constants file.
Relevant code in ODK-X Services:
https://github.com/opendatakit/services/blob/670be84a409c763ba21a104487542e52c740aebf/services_app/src/main/java/org/opendatakit/services/sync/service/SyncExecutionContext.java#L100

@linl33  Can I take this as my first issue?

Hi @IAmJaishree,
Yes, please go ahead. Thanks.

Closed by: odk-x/services#182


ENH: update package version

Hi! This is the friendly automated conda-forge-webservice.
I've started a version update as instructed in #172.
I'm currently searching for new versions and will update this PR shortly if I find one! Thank you for waiting!
Fixes #172

@conda-forge-admin, please rerender

Going to merge this on red, since its just ppc64le that's failing.


fix: check for provider if before calling Stripe to fetch payment met…

…hods

Pull Request Test Coverage Report for Build<PHONE_NUMBER>0
Details

0 of 6 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 9.506%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

billing/customer/service.go
0
6
0.0%

Totals

Change from base Build<PHONE_NUMBER>2:
0.0%

Covered Lines:
10358

Relevant Lines:
108966

💛  - Coveralls


fix: CWE-326: Inadequate Encryption Strength

Implement secure HTTPS communication TLSv1.3.
Consider using latest TLSv1.2 instead of TLS.
https://docs.oracle.com/en/java/javase/11/docs/specs/security/standard-names.html#sslcontext-algorithms

Thanks you for you PR (sorry for this late answer)
We have rework the MongoDB connection settings for the 3.18.0 and made this option configurable through the gravitee.yaml


feat: post props to loading Component

I met a situation that the loading part could not met my need. The situation is like that:
const Loading = (props) => {
    if ( props.condition) {
        return <A {...props} />;
    } else {
        return <B {...props} />;
    }
}

Maybe leave a channel to post props is needed.

@jamiebuilds what do you think about this? It would be great to merge it.

+1 on this.

I met a situation that the loading part could not met my need. The situation is like that:
const Loading = (props) => {
    if ( props.condition) {
        return <A {...props} />;
    } else {
        return <B {...props} />;
    }
}

Maybe leave a channel to post props is needed.

This author doesn't seem to be updating this component, but you can look at similar components that implement SSR PROMISE asynchronous loading，github address ：https://github.com/qq281113270/react-lazy-router-dom

这是来自QQ邮箱的假期自动回复邮件。你好，我最近正在休假中，无法亲自回复你的邮件。我将在假期结束后，尽快给你回复。


Example usage for template configurations

In addition to executing a specific check against a server (where the config file contains a value for ServerInstance), provide an example where a template configuration can be executed against multiple targets.
As an example, given the following config:
AgentEverywhere.config.json
{
  "AgentIsRunning": {}
}

I want to be able to target a number of servers - hardcoded in the script, pulled from a file, from a database table, central management server, etc.
Results should still be correctly reported with the target instance.
@andrewalumkal does this match your expectation of a template config?

I think something similar to the following may work...
$config = Get-ConfigFromSomewhere...
$servers = Get-ServerListFromSomewhere...

foreach($serverInstance in $servers) {
  $config.ServerInstance = $serverInstance
  Invoke-SqlChecks -Config $config # or Invoke-SqlChecksToLogAnalytics...
}

Documented in #36


Update development status to beta

It looks like trycast is pretty mature and well tested, so upgrade its status to "stable" to reassure potential users (at least, me).

I second to this as well.

I'm going through a rough patch in my personal life right now, so I will respond briefly.
I'd be okay with upgrading the status of this project to "beta", but I feel like "stable" would require some additional documentation. The roadmap is suggestive of my thoughts here.
Right now there is a quickstart but no reference or API documentation to speak of.

Thanks for the update @davidfstr
I've modified this PR to change to "beta" then.

Yeah beta is much better. At least, it gives me more confidence in trying it out. Thanks!


Issue with k8s.io/docs/tasks/administer-cluster/kubeadm/kubeadm-upgrade-1-14/

This is a Bug Report

When I run kubeadm upgrade plan the kubeadm doesn't list the containers that need to be updated:
kubeadm upgrade plan
[preflight] Running pre-flight checks.
[upgrade] Making sure the cluster is healthy:
[upgrade/config] Making sure the configuration is correct:
[upgrade/config] Reading configuration from the cluster...
[upgrade/config] FYI: You can look at this config file with 'kubectl -n kube-system get cm kubeadm-config -oyaml'
[upgrade] Fetching available versions to upgrade to
[upgrade/versions] Cluster version: v1.13.4
[upgrade/versions] kubeadm version: v1.14.0

Proposed Solution:

This can be resolved by specifying the version:

kubeadm upgrade plan v1.14.0
[preflight] Running pre-flight checks.
[upgrade] Making sure the cluster is healthy:
[upgrade/config] Making sure the configuration is correct:
[upgrade/confhttps://kubernetes.io/...ig] Reading configuration from the cluster...
[upgrade/config] FYI: You can look at this config file with 'kubectl -n kube-system get cm kubeadm-config -oyaml'
[upgrade] Fetching available versions to upgrade to
[upgrade/versions] Cluster version: v1.13.4
[upgrade/versions] kubeadm version: v1.14.0

Components that must be upgraded manually after you have upgraded the control plane with 'kubeadm upgrade apply':
COMPONENT   CURRENT       AVAILABLE
Kubelet     2 x v1.13.4   v1.14.0

Upgrade to the latest version in the v1.13 series:

COMPONENT            CURRENT   AVAILABLE
API Server           v1.13.4   v1.14.0
Controller Manager   v1.13.4   v1.14.0
Scheduler            v1.13.4   v1.14.0
Kube Proxy           v1.13.4   v1.14.0
CoreDNS              1.2.6     1.3.1
Etcd                 3.2.24    3.3.10

You can now apply the upgrade by executing the following command:

        kubeadm upgrade apply v1.14.0

Docs should be updated and specify that version should be provided or make the version mandatory field

Page to Update:
https://kubernetes.io/docs/tasks/administer-cluster/kubeadm/kubeadm-upgrade-1-14/

tracking issue:
https://github.com/kubernetes/kubeadm/issues/1469
fix should be available in 1.14.1
thanks for the report.
/close


gapi revoke some scope ?

Is there a way to "revoke" some scope without having to drop all token and having to re-authorize the all thing ?

No.  In fact, last time I checked whenever you ask for any scope for a client ID, it gives all scopes that the client ID is authorized for.  E.g. if you have already given a client ID drive.file and drive.metadata.readonly scope, there is no way to get a token with just one of these scopes.

No. In fact, last time I checked whenever you ask for any scope for a client ID, it gives all scopes that the client ID is authorized for. E.g. if you have already given a client ID drive.file and drive.metadata.readonly scope, there is no way to get a token with just one of these scopes.

That's annoying.
Is there at least a way when you re-authorise an app after detaching it from drive that it still have access to old files created by it ?

Can you confirm you lose access to old files when you detach from Drive (and not just the files you lost when we change the client ID)?
If so, that is annoying but probably unfixable.  Hopefully real user's detach from drive very rarely.


Add 6.0-rc (and associated new PGP key)

This PGP key is what's causing the failure in https://github.com/docker-library/official-images/pull/12271, so I might pull that out into a separate commit if 6.0-rc proves to be tricky.

Closing this for now -- as noted in my edit above, the key was added in 5cca057. :+1:


kubectx and kubens no longer work on macOS due to missing readlink -f option

Hi Ahmet, first of all, excellent tool!
This morning, I noticed that the current versions of both kubectx and kubens no longer (since https://github.com/ahmetb/kubectx/pull/6) work on macOS (and possibly other BSD-based systems), because readlink on these systems don't know about the -f option (to follow symlinks, and canonicalise the name).
$ kubens

readlink: illegal option -- f
usage: readlink [-n] [file ...]
/usr/local/bin/kubens: line 28: ./../include/utils.bash: No such file or directory

We could either require the GNU version of readlink (which supports -f/--canonicalize), and add a case to use greadlink on macOS, or add our own function that behaves like GNU's readlink, i.e. iterate through symlinks, and return the canonical name.

@koenrh I just ran into this myself.  The following resolved my local instance:
brew install coreutils
I then added gnubin to my path by appending
export PATH="/usr/local/opt/coreutils/libexec/gnubin:$PATH" to $HOME/.bash_profile, and restarting my terminal.
I agree that the additional case requirement would be a helpful addition, but until that point, if others are using this fantastic tool, these three steps might help some folks.

My bad. I keep forgetting gnu coreutils is installed on my Mac. I wonder if we can do without -f and without checking for greadlink.

In the brew recipe you might be able to make coreutils a dependency and then use greadlink? Seems like a hard sell.

Since readlink -f fails only on BSD coreutils which is on mac, I think we can safely assume Python is installed. So I am changing the way to determine SCRIPT_DIR to this, which falls back to Python:
SCRIPT_DIR="$(dirname $( readlink -f "${0}" 2>/dev/null || \
  python -c "import os,sys; print(os.path.realpath(sys.argv[1]))" "${0}" ))"

Every method I found without introducing GNU coreutils dependency is just a hack, including this one. I’ll submit a PR soon and make a release. Sorry for the disruption.

@ahmetb What do think of this? https://github.com/kubernetes/kubernetes/blob/b353700bfc1ee95b8f9f827c9cc818cf2ab3f943/hack/lib/init.sh#L111-L128

@koenrh it's nice but we would be copying it to 2 files, both kubens/kubectx scripts. So I prefer a one-liner more.

Fixed, try with brew update && brew upgrade.

Works for me, thanks!


opencv/hdf5/Iconv

Operating system
Windows
Compiler
MSVC
Steps to reproduce the behavior
-cd build
-cmake ..

Failure logs
CMake Error at C:/src/vcpkg/installed/x64-windows-static/share/opencv4/OpenCVModules.cmake:109 (set_target_properties):
  The link interface of target "opencv_hdf" contains:

    hdf5::hdf5-static

  but the target was not found.

CMake Error at CMakeLists.txt:32 (target_link_libraries):
  Target "IMO" links to:

    hdf5::hdf5-static

  but the target was not found.  Possible reasons include:

    * There is a typo in the target name.
    * A find_package call is missing for an IMPORTED target.
    * An ALIAS target is missing.

CMake Error at C:/src/vcpkg/installed/x64-windows-static/share/opencv4/OpenCVModules.cmake:369 (set_target_properties):
  The link interface of target "opencv_wechat_qrcode" contains:

    Iconv::Iconv

  but the target was not found.  Possible reasons include:

    * There is a typo in the target name.
    * A find_package call is missing for an IMPORTED target.
    * An ALIAS target is missing.

Additional context
I am really unsure if i should ask here or in a cmake forum since i dont know if my issue is with the installation via vcpkg, or if its a problem with cmake. i did search a bunch of forums and i dont understand most of it. From my understanding the target "IMO" (my project name) cannot link to the target hdf5-static and iconv.
cmakelists.txt

Thanks for posting this issue. If you use manifest mode, could you also provide your vcpkg.json file?

i cant find a vcpkg.json file, but besides that, i think i solved it somewhat. my problem was, that i was not able to call my toolchain file (vcpkg.cmake) after i call my project in my cmakelists.txt file. the error message disappeared after i changed it.
i still have an issue tho. i suspect that the issue is cmake but i will describe it as good as possible.
right now cmake is confusing the dynamic and static library of hdf5. i wasnt aware of the difference between the two and just installed the dynamic library. after trying to build my project, i figured out that it does not work. i then installed the static library and uninstalled the dynamic library. right now cmake is looking for a file in the dynamic installation but it should search for it in the static installation. long story short i think there is an issue with my hdf5 config file but it could also be an issue with vcpkg not installing it correctly.
CMake Error at C:/src/vcpkg/installed/x64-windows-static/share/hdf5/hdf5-config.cmake:150 (include):
  include could not find requested file:

    C:/src/vcpkg/installed/x64-windows/share/hdf5/hdf5-targets.cmake

as you can see, its searching for a file in the dynamic directory.
i think the best thing i can do is ask if my installation is correct. this is my directory of hdf5-static:
packages\hdf5_x64-windows-static
			├───debug
			│   └───lib
			│       ├───pkgconfig
			│       │   ├───hdf5.pc
			│       │   └───hdf5_hl.pc
			│       ├───libhdf5.settings
			│       ├───libhdf5_D.lib
			│       ├───libhdf5_hl_D.lib
			│       └───libhdf5_tools_D.lib
			├───include
			│       ├───
			│       ├───
			│       ├───
			│       ├───
			│       ├───
			│       ├───
			│       ├───
			├───lib
			├───share
			└───tools

I link all files in question down bellow (i had to change vcpkg.cmake to a .txt file since github does not accept cmake files)
cmakelists.txt
vcpkg.cmake.txt
i want to mention again that the issue is most likely cmake and not vcpkg.
thanks for your help anyway.


bad link in readme

Just noticed that the "Mixed Reality H&S Guidelines" link in the documentation isn't valid.

seems fine now!


support old resources

I don't understand this yet, and I think it's probably not worth anyone's time at this point as we're deprecating sqlite support in favor of formats with more first class support for objects (msgpack, JSON).


package:rapidinho/network/keys.dart not found

package:rapidinho/network/keys.dart not found

It is the file where contains the Google Maps key, you need to generate your own keys

What will be the content/format of the dart file once we have the keys ?

add a keys.dart file in your lib/network/ with this content
const googleMapsApiKey = 'YOUR_GOOGLE_MAPS_API_KEY';

Replace 'YOUR_GOOGLE_MAPS_API_KEY' text with... you guessed it, your api key from Google Maps


No folder: simple file dilaog should show home directory as default

Open vscode with no folder (purple status bar)
Have 	"files.simpleDialog.enable": true,
New untitled file, save as file
Notice how the simple dialog offers me the root path. This is wrong imho, you should offer the home directory, or at least something wich I choose the last time

@aeschli Could we keep the Untitled URI scheme here? https://github.com/Microsoft/vscode/blob/master/src/vs/workbench/services/textfile/common/textFileService.ts#L907
All file scheme URIs are absolute. Which means that the remote file picker would have to use a heuristic like checking for "Untitled-" in the path to tell that the user home should be used.

Feel free to change. @bpasero is the expert on the untitled URIs...

@alexr00 the URI only seems to be used in combination of the dialog itself (both native and custom - here), so I think we can preserve the scheme if you deal with the untitled case in both worlds.


Offsets drop during partition rebalance

Version of Ruby: 2.5.0
Version of Kafka: 0.11.02, but I've also experience this on 1.0
Version of ruby-kafka: 0.5.5

I've reported this before but haven't had enough logging or understanding to fully explain. This time I have more detail and a log file to share.
Problem:
When network errors occur and consumers can't be reached, the consumer group rebalances the partitions and when that happens, the offset for some of the partitions drop precipitously:

Setup
Partitions: 32
Compression: gzip
Consumers: 9 in 9 containers on 5 hosts
Kafka Provider: Heroku running <IP_ADDRESS>, but I previously saw it using Confluent Cloud with 1.0.0
Offset Commit: After every batch
Description:
All consumers were running normally for several days until a network hiccup in container 9f29db205855 caused the problem. Previously I've experienced it on deploys when existing consumers are stopped and new ones are started
Before the rebalance, all partitions had an offset of ~4.6m.
host      | container   | partitions before | partitions after
consumer1 | db2e98932f5a | 31,4,13,22  | 7,16,25
consumer1 | 6a2b29b21aa6 | 8,17,26 | 11,29,20,2
consumer2 | 0f135bafefa0 | 7,16,25 | 10,1,28,19
consumer2 | fcfcb4b9b2bc | 6,15,24 | 15,24,6
consumer3 | 3c2b60361ee7 | 11,29,20,2 | 4,13,22,31
consumer3 | ceafbb18fde7 | 18,27,0,9 | 18,27,9,0
consumer4 | b1796a07f06d | 14,23,5 | 14,23,5
consumer4 | 9f29db205855 | 28,19,10,1 | 17,8,26
consumer5 | 971b7bae879e | 3,21,12,30 | 21,12,3,30

After the rebalance, partitions 7,8,16,17 and 19 had their offsets dropped to between 1.4m and 2.3m
Timeline:
2018-05-30T09:47:47.884959 | ERROR -- : Timed out while waiting for response 1579349
2018-05-30T09:47:48.251902 | ERROR -- : Error sending heartbeat: Kafka::UnknownMemberId
2018-05-30T09:47:47.885698 | ERROR -- : Connection error while fetching messages: Connection error Errno::ETIMEDOUT: Connection timed out
2018-05-30T09:47:48.251902 | ERROR -- : Failed to join group; resetting member id and retrying in 1s
2018-05-30T09:47:48.333235 | ERROR -- : Error sending heartbeat: Kafka::UnknownMemberId
2018-05-30T09:47:52.015814 | On the other consumers, first occurrence
of Error sending heartbeat: Kafka::RebalanceInProgress
2018-05-30T09:47:58.194873 | Last Error sending heartbeat: Kafka::RebalanceInProgress
2018-05-30T09:47:59.008608 | partition 7 drops to OFFSET: 1419967
2018-05-30T09:47:59.467819 | partition 17 drops to OFFSET: 1410587
2018-05-30T09:48:01.683178 | partition 19 drops to OFFSET: 2033048
2018-05-30T09:48:26.157260 | partition 8 drops to OFFSET: 2339618
2018-05-30T09:49:23.001463 | partition 16 drops to OFFSET: 1401632
partition 25  Is never consumed again after the rebalance. Over 12 hours later, it still has yet to have a single message consumed from it
The log is here: https://nofile.io/f/cUdtxxQvYAn/full.log

This looks like a bug.

I just deployed these consumers and it happened again. Interestingly, I have 3 consumer groups running. The log file I attached above is filtered to just one of the consumer groups. The others 2 did not experience this bug. Now, when I deployed again, the bug only affected partitions in the same consumer group (new_new_relic_production). It's like once a consumer groups gets fouled, it's permanently that way.

@bradurani I'm experiencing the same issue from time to time – it could be due to differences in throughput, with high throughput correlating with this issue.
I'll try to find time to look into it soon.

@dasch we've experienced same issue so this seems like not an uncommon pattern. @bradurani for us that happened also when we didn't send heartbeat often enough and the  consumer got kicked out of the group.

Can you try running against this branch and see if it helps? https://github.com/zendesk/ruby-kafka/pull/594
Also, try to record the logs.

I've also been experiencing this issue in a new Kafka consumer/producer we've recently put in production.
I've observed this on ruby-kafka version 0.6.1. I haven't seen this behaviour in other Kafka topics we have so given the other comments I've seen here it might be related to throughput.
This happens very often in this particular service after any deploy. For example: I've observed a worker committing to partition 1 the offset 171_822 prior to being shutdown and then seeing another worker created after the deploy consuming/committing to partition 1 the offset 169_687

@dasch We ran into this issue with v0.6.7 again over the weekend 😿 so it looks like #595 doesn't fix the problem 100%.  Let us know if there's any information we can provide to help debug further.
cc @bai @maxboisvert

@andrewjamesbrown I'd greatly appreciate some kind of deep debugging with extended logs – I don't have the time for that right now.

@andrewjamesbrown Could you provide more information about this? And it would be great if you could provide some logs or code snippet to re-produce the problem?

delivery: inline_batch and Phobos::BatchHandler help me with
Error sending heartbeat: Kafka::RebalanceInProgress

delivery: inline_batch and Phobos::BatchHandler help me with
Error sending heartbeat: Kafka::RebalanceInProgress


IOError: [Errno socket error] [Errno -3] Temporary failure in name resolution

santosh@santosh-desktop:~/First$ python $HOME/First/New3.py Traceback (most recent call last):   File "/home/santosh/First/New3.py", line 188, in      tf.app.run(main=main, argv=[sys.argv[0]] + unparsed)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/python/platform/app.py", line 44, in run     _sys.exit(main(_sys.argv[:1] + flags_passthrough))
File "/home/santosh/First/New3.py", line 169, in main     train()   File "/home/santosh/First/New3.py", line 26, in train     fake_data=FLAGS.fake_data)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/contrib/learn/python/learn/datasets/mnist.py", line 227, in read_data_sets     SOURCE_URL + TRAIN_IMAGES)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/contrib/learn/python/learn/datasets/base.py", line 208, in maybe_download     temp_file_name, _ = urlretrieve_with_retry(source_url)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/contrib/learn/python/learn/datasets/base.py", line 165, in wrapped_fn     return fn(*args, **kwargs)
File "/usr/local/lib/python2.7/dist-packages/tensorflow/contrib/learn/python/learn/datasets/base.py", line 190, in urlretrieve_with_retry     return urllib.request.urlretrieve(url, filename)
File "/usr/lib/python2.7/urllib.py", line 98, in urlretrieve     return opener.retrieve(url, filename, reporthook, data)   File "/usr/lib/python2.7/urllib.py", line 245, in retrieve     fp = self.open(url, data)   File "/usr/lib/python2.7/urllib.py", line 213, in open     return getattr(self, name)(url)
File "/usr/lib/python2.7/urllib.py", line 350, in open_http     h.endheaders(data)
File "/usr/lib/python2.7/httplib.py", line 1053, in endheaders     self._send_output(message_body)
File "/usr/lib/python2.7/httplib.py", line 897, in _send_output     self.send(msg)
File "/usr/lib/python2.7/httplib.py", line 859, in send     self.connect()
File "/usr/lib/python2.7/httplib.py", line 836, in connect     self.timeout, self.source_address)
File "/usr/lib/python2.7/socket.py", line 557, in create_connection     for res in getaddrinfo(host, port, 0, SOCK_STREAM):
IOError: [Errno socket error] [Errno -3] Temporary failure in name resolution
Please help me sort out this prob.

@Santosh110892 Have you already find the problems? I got this problem too. I am a beginner of tensorFlow. Thanks for your answer!


Add Docker Image CI support

Signed-off-by: Carson Yang<EMAIL_ADDRESS>

Oh nice, thank you!

@yangchuansheng, how to configure this CI?

Ill just leave it to run on push to main, will remove it on pull requests
  push:
    branches:
      - main


Update dependencies for 2024-01 release train

Update all dependencies using ./gradlew versionCatalogUpdate once Spring Framework 6.1.3 is available on 2024-01-11.
Ensure the following libs are updated:

[x] spring framework
[x] micrometer*
[x] reactor

Closing via https://github.com/spring-projects/spring-pulsar/commit/1f260841e81c98c354e45fde3bf8a9fbdb346f2a


sqlFile throws IOException when relativeToChangelogFile is used (from an included databaseChangeLog file)

Environment
Liquibase Version: 4.5.0
Liquibase Integration & Version: spring boot (webflux)
Liquibase Extension(s) & Version: no
Database Vendor & Version: postgres 14.4
Operating System Type & Version: MacOs (M1)
Infrastructure Type/Provider: local
Description
When using relativeToChangelogFile=true for sqlFile liquibase throws IOException.
This is because the getPhysicalFilePath contains also the changelog filename. ( /db/changelog/./install/main.yaml )
Workaround: start the path from parent directory
  sqlFile:
-   path: sql/sql-file.sql
+   path: ../sql/sql-file.sql
    relativeToChangelogFile: true
    splitStatements: false

Steps To Reproduce
resources/db/changelog/db.changelog-master.yaml
databaseChangeLog:
  - include:
      file: ./install/main.yaml
      relativeToChangelogFile: true

resources/db/changelog/install/main.yaml
databaseChangeLog:
  - changeSet:
      id: create-schema
      changes:
        - sqlFile:
            path: sql/sql-file.sql
            relativeToChangelogFile: true
            splitStatements: false

Actual Behavior
Exception is thrown
Caused by: java.io.IOException: The file sql/sql-file.sql was not found in
    - Spring resources

Expected/Desired Behavior
File is found and executed
Additional Context
Maybe in liquibase.change.core.SQLFileChange FilenameUtil.getDirectory should be used (to remove the filename):
        try {
            String relativeTo = null;
            if (ObjectUtil.defaultIfNull(isRelativeToChangelogFile(), false)) {
                relativeTo = getChangeSet().getChangeLog().getPhysicalFilePath();
            }
-            inputStream = Scope.getCurrentScope().getResourceAccessor().openStream(relativeTo, path);
+            inputStream = Scope.getCurrentScope().getResourceAccessor().openStream(FilenameUtil.getDirectory(relativeTo), path);
        } catch (IOException e) {
            throw new IOException("Unable to read file '" + path + "'", e);
        }

it looks like it works with : path: sql/sql-file.sql

Found that it does not work if liquibase is used with spring-boot-starter-webflux.
If we I use spring-boot-starter-web, the path works like this: path: sql/sql-file.sql

I think that as of Liquibase version 4.23.1 the issue is already resolved. Please, see #4249 for reference.

Closing, since it has been tested with 4.32.1 and it couldn't be reproduced it. Please, feel free to reopen if you see this issue still happening.
Thanks,
Daniel.


To much used item box

if u have more same items they will show in separate box, not in 1 box with x22 redline
and float / stickers view .

Not a bug, also not adding floats & stickers.


closed loop feedback?

Hi @tlambert03
I really like the idea of useq-schema!
Over the last couple of days I've been working on putting together a basic C++ library that uses a combination of:

MMCore I/O handling.
OpenCV basic image processing.
TensorRT  inference on NVIDIA GPUs and deep learning accelerators.
Imgui bloat-free Graphical User interface for C++ with minimal dependencies.
Polyscope viewer for 3D data like meshes and point clouds extracted by OpenCV and TensorRT.

I primarily work in in situ sequencing method development (https://www.biorxiv.org/content/10.1101/722819v2).
One of the things we will see this year in in situ sequencing is an explosion of different commercial vendors offering "hardware boxes" that are essentially just a box with automated fluidics and an epifluorescent microscope to perform some iterative smFISH.
My idea with embarking on the above C++ library is rather that any microscope and hardware can be turned into a sequencing machine operating in a similar fashion to the Illumina Local Run Manager (keeping similar file conventions, user interface etc.).
I really want to use useq-schema to specify and make acquisition runs reproducible. Before embarking on this and digging deeper into the useq-schema I had one question:
Question: Do you have any current plan on how closed-loop events could be handled within the useq-schema. In situ sequencing is full of closed-loop events. Let me give you an example:
Closed-loop histogram equalization: Traditionally each nucleotide can be represented by a single non-overlapping fluorophore. For example A, T, C, G could be represented by Alexa488, Alexa555, Alexa594, Alexa647. Before each sequencing cycle is started the microscope first goes to a region not imaged and takes some snapshots with a given exposure time and light power. The images from these snapshots are then used to compute histograms and extract statistics that are then used to inform the exposure time and light power etc. of each channel setting such that the histograms of each channel are as closely equalized as possible before starting the true acquisition.
There are tons of similar events like this (checking that fluorophores have cleaved before starting the next cycle, adjusting laser power across z-axis etc.).
Do you have any good idea if events like this should already be represented at the level of useq-schema?

Hi @tractatus, thanks for your note.
I agree completely, closed loop sequence modification is super important, and definitely something that I would want to do in my own stuff as well.  Short answer is: it's still an open question, and I'd love to hear your thoughts.    here are some of my first thoughts
as i'm sure you've seen,useq-schema is kind of playing with two main objects: useq.MDAEvent, which contains the single-timepoint instructions, and useq.MDASequence, which is a more amorphous object that ultimately boils down to a iterable of useq.MDAEvent objects.   my immediate thoughts are that MDAEvent – the thing the actual acquisition engine (be it mmcore, or whatever) will "consume" – probably shouldn't have any representation of dynamic properties (i.e. "set the exposure so as to satisfy these criteria").  I can, however, imagine MDASequence acquiring some closed-loop/dynamic instructions... we'd just need to think about how one would represent the parameters of something like histogram equalization in a declarative format such that it could go into a YAML file.
One idea: I would like to see MDASequence turned into a bit more of an abstract iterable-of-iterables.  Currently, MDASequence has those four hard-coded TCZP attribute, and an axis order.  Each of the time, channel, z, attributes though is really just an iterable for that specific axis.  So MDASequene could/should look a bit more like
class MDASequence
    axis_order: Sequence[str]
    axes: Dict[str, Iterable]

The closed loop logic then would need to be captured in the individual axis Iterable.  For instance, take the time axis, which is currently captured in the MDASequence.time_plan attribute here.  TimePlans are all ultimately just iterables of floats, where the float represents the time that the instrument should wait before starting the next MDAEvent.  If one wanted to create a dynamic time plan (for instance, change the inter-frame interval depending on some event in the experiment, like mitosis or something), then we'd need to
A) have some additional stateful logic in TimePlan.__iter__ that is capable of changing the yielded time-delta based on input
B) actually provide that input (in the form of the previous frame or something) to the TimePlan
part B there is more up to the actual acquisition engine.  pycro-manager, for instance, has the concept of acquisition hooks that would perform the actual logic of updating the event sequence.  So I guess the challenge for useq-schema would be to come up with a declarative nomenclature that captures the "intention" of something like a hook, while the actual implementation is left to the engine consuming the MDASequence iterable...
anyway, that's obviously not a complete answer.  but I definitely agree that it's important ... and it would be nifty if we could capture the concept of dynamic closed loop logic in a declarative schema, without being totally dependent on the actual engine performing the acquisition.  I very much welcome your thoughts!
btw... I worked with Je a little bit when he was here at HMS, say hi! :)

one more side note...
for me, this is all very much still work in progress... but if you'd like to see how I've been using useq-schema along the lines of what you're also doing:

pymmcore-remote is my wrapper around pymmcore, it adds a couple conveniences, and has a run_mda method that actually consumes the useq.MDASequence object and drives the hardware.
napari-micromanager is a gui interface for pymmcore-remote, that builds the MDASequence based on user input, and passes it to pymmcore-remote to run it.  It then receives the frames as they are generated and displays them in napari
image processing would then happen on the napari layer object...

Wow yeah sounds that we are very much on the same page then. I will directly go in and checkout your run_mda method and simply translate it to C++ and start from there and as the process gets more mature the full aspects of closed-loop probably have been planned out.
 my immediate thoughts are that MDAEvent – the thing the actual acquisition engine (be it mmcore, or whatever) will "consume" – probably shouldn't have any representation of dynamic properties (i.e. "set the exposure so as to satisfy these criteria").

I agree with this. The specifics of the dynamic properties should probably not be contained in the schema representation but rather just a human-readable way of quickly identifying the loops that exist and then further information is specified in the class (or even compiled program) that is the controller.
Im aware of the napari-micromanager and before embarking on this I quickly went through both pymmcore, pycromanageretc. to see what was available. But in my case Im having increasing issues with deploying inference of trained networks in a performant way and the solution really is TensorRT and then Im back at C++ so I figured I just make a minimal viable solution in C++ and then start thinking about hooks and other things to napari.

But in my case Im having increasing issues with deploying inference of trained networks in a performant way and the solution really is TensorRT and then Im back at C++ so I figured I just make a minimal viable solution in C++ and then start thinking about hooks and other things to napari.

awesome.  makes sense.

I will directly go in and checkout your run_mda method and simply translate it to C++ and start from there

I would also point you to the two other existing micromanager "engines" that I know about:

the clojure engine built into micro-manager.  The main "sequence-consuming" function is run-acquisition ...  and the main "event-consuming" function is make-event-fns
The java-based acquisition engine AcqEngJ that was written (as I understand it) specifically to consume the event dicts produced by pycro-manager, where submitEventIterator is the thing that receives the sequence (the event iterator) and executeAcquisitionEvent is the thing that drives the hardware for each event object (and, it also calls the hooks that would support this sort of closed loop use case)

I'll also cc @nicost ... who I believe has also toyed with the idea of writing a core acquisition engine in C/C++  (which would be fantastic!).  honestly, if you write an mmcore acquisition engine in C++, I suspect others might want to use it?


Added some unit tests

Tested methods for SampleState class and removed some unused code.

 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.:x: qnuw sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


Fix sdk "in" list filter query param generation

It looks like we're not handling in filters correctly:
https://github.com/freshbooks/api-nodejs-sdk/blob/master/packages/api/src/models/builders/SearchQueryBuilder.ts#L27
This would generate a query_string in the for of: ?search[userids]=1&search[userids]=23
When the correct form for lists in evolve is: ?search[userids][]=1&search[userids][]=2&search[userids][]=3

Fixed


Update fix-problems-in-windows-search.md

Added "Solution 5" to "Fix problems in Windows Search" article
Pull request guidance
Thank you for submitting your contribution to our support content! Our team works closely with subject matter experts in CSS and PMs in the product group to review all content requests to ensure technical accuracy and the best customer experience. This process can sometimes take one or more days, so we greatly appreciate your patience.
We also need your help in order to process your request as soon as possible:

We won't act on your pull request (PR) until you type "#sign-off" in a new comment in your pull request (PR) to indicate that your changes are complete.

After you sign off in your PR, the article will be tech reviewed by the PM or SME if it has more than minor changes. Once the article is approved, it will undergo a final editing pass before being merged.

#sign-off

@i3v The changes are live. This PR will be closed.


Initial volume doesn't match volume bar

The volume seems to start out at a certain level, but the initial value for volume slider in the app doesn't match up with that value, so as soon as you move the slider, the music gets a lot louder.

Can you handle it @dkniffin ? It's not a complicated one, it's juste some maths:

The range input is from 0 to 1
When we update it, we set the volume value to Math.pow(value, 4) to be more natural and we store that value to the config file
At startup, the defaultValue of the range input is wrongly calculated
See https://github.com/KeitIG/museeks/blob/master/src/js/components/Header/VolumeControl.react.js

Yep, I'll take care of it.


Clean-up config resources in ktlint-cli

Resources in location ktlint-cli/src/main/resources/config and ktlint-cli/src/main/resources/project-config have to be removed. They are not used in the project anymore.

Hey @paul-dingemans, can i work on this issue ?

Sure


Metrics: changed port from 8080 to 8082

Description
This PR changes the ports used for metrics from 8080 to 8082. This is useful to avoid using port 8080 with host network pods, as port 8080 should be used by another service on the host.

/rebase test=true

/rebase test=true

@cheina97 this is probably breaking something since both tests (overlapped and not) are broken while waiting for the connection to be set up in the peer command

/build

/rebase test=true

/rebase test=true

/rebase test=true

/rebase test=true

/rebase test=true

/merge


Capability conflict regression fix and bump of SSH versions

This PR resolves two issues for 8.8 RC2:

Fixes #29208 by reverting #28218
Fixes #28823 (was already fixed for 8.9, moving the fix up)

Merging directly since this is a revert and a cherry-pick


No Options Present in Grafana 7.2.0

Hi,
First: Thanks for developing this plugin. It would be perfect for my needs.
However, after I installed version 7.0.4 of the plugin for grafana 7.2.0 I noticed that the "Options" dialog is missing after selecting "Button Panel" as visualization. (Also see the screenshot below).
Without this, configuration is impossible.
Can you please have a look at this behaviour?
Thanks in advance!

Can confirm.
It works in Grafana 7.0.x and 7.1.x
Looking into it.
Thank you for reporting

Hey @derjust ,
this is my console output when trying to add it:
[Deprecation warning] PanelPlugin: setDefaults is deprecated. Use setPanelOptions instead deprecationWarning.ts:14:12
Hope this helps, but i think you found that too.


RKE2 not pulling all images from private registry

Rancher Server Setup

Rancher version: 2.6.3
Installation option (Docker install/Helm Chart):

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): RKE2

Information about the Cluster

Kubernetes version:  1.21.7
Cluster Type (Local/Downstream): Local

Describe the bug
When configuring an RKE2 cluster with a private registry, most images are pulled using this registry, but fleet-agent, rancher-agent, system-agent, and system-upgrade-controller are not.
To Reproduce

Create a private registry and load all images defined at https://github.com/rancher/rancher/releases/download/v2.6.3/rancher-images.txt
In Rancher, create an RKE2 custom cluster. Enter the endpoint for the registry created. All other default options are fine.
Get the registration command (all 3 roles) and run the command on the node.
Wait about 10 minutes for the cluster to get created.
Check the images downloaded by running: export CRI_CONFIG_FILE=/var/lib/rancher/rke2/agent/etc/crictl.yaml
/var/lib/rancher/rke2/bin/crictl images

Result
> crictl images
IMAGE
docker.io/rancher/fleet-agent
docker.io/rancher/rancher-agent
docker.io/rancher/system-agent
docker.io/rancher/system-upgrade-controller
<PRIVATE_REGISTRY_FQDN>/rancher/hardened-cluster-autoscaler
...

Expected Result

> crictl images
IMAGE                                                                                TAG
<PRIVATE_REGISTRY_FQDN>/rancher/fleet-agent
<PRIVATE_REGISTRY_FQDN>/rancher/rancher-agent
<PRIVATE_REGISTRY_FQDN>/rancher/system-agent
<PRIVATE_REGISTRY_FQDN>/rancher/system-upgrade-controller
<PRIVATE_REGISTRY_FQDN>/rancher/hardened-cluster-autoscaler
...

Screenshots

Additional context

SURE-3856

For the fleet-agent image, this is blocked by https://github.com/rancher/fleet/issues/711

Root cause
The RKE2 controllers in Rancher were ignoring the cluster's private registry.
What was fixed, or what changes have occurred
The controllers will now take into account the private registries configured for the RKE2 cluster, if it exists.
Areas or cases that should be tested

From Rancher, provision an RKE2 cluster with a private registry for the Rancher images. Can be a single node cluster.
After the cluster is active, SSH to the node.
Run the following command: CRI_CONFIG_FILE=/var/lib/rancher/rke2/agent/etc/crictl.yaml /var/lib/rancher/rke2/bin/crictl images
The only image that should not be pulled from the private registry is fleet-agent (see issue https://github.com/rancher/fleet/issues/711 for the issue there). This includes rancher-agent, system-agent, and system-upgrade-controller images.

What areas could experience regressions?
Provisioning RKE2/K3S clusters from Rancher.
Are the repro steps accurate/minimal?
Yes.

Blocked by https://github.com/rancher/rancher/issues/36939; Unable to provision RKE2 Custom cluster w/ private registry
This ticket is also related to blocker - https://github.com/rancher/rancher/issues/37364

Ticket #36137 - Test Results
With Helm via HA Jenkins job:
Verified on rancher v2.6.5-rc1:

Fresh install of rancher v2.6.5-rc1
Create a standard user
As standard user, provision RKE2 Custom cluster with a private registry
Verified - With the exception of fleet-agent, all other images successfully pulled from private registry

Screenshot:

@Josh-Diamond Why was the issue closed if you can still reproduce the issue? If fleet-agent is not using the private registry, it's going to fail in an air-gapped environment.

@dnoland1 `Areas or cases that should be tested - #4'
https://github.com/rancher/rancher/issues/36137#issuecomment-1089567007

@dnoland1 The fleet-agent being pulled from a private registry is not in Rancher's control. That is handled by fleet-controller and there is nothing Rancher can do to change this at this point.
The issue I linked above from the fleet repository is the one being used to track the fleet-agent changes.


Show exact distance in meters

Hello
on the online BRouter Web Client there is the distance (de: "Entfernung") in km on the bottom left.
The km is rounded to the first digit after the dot, e.g. 0.5 km for 549 meter or 0.6 for 551 meter.
I did not find any setting or switch to show meter instead of km.
Frank

You can see the not rounded value with meter precision in the tooltip when hovering over the value.


Update deps and CI action

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.17%. Comparing base (693b7b7) to head (31b832c).

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #11      +/-   ##
==========================================
+ Coverage   90.70%   91.17%   +0.47%
==========================================
  Files           6        6
  Lines         430      362      -68
==========================================
- Hits          390      330      -60
+ Misses         31       23       -8
  Partials        9        9

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Update PyYaml Version

Cython 3.0 was released. This breaks PyYaml 5.4.1 (a dependency of discord-logger). PyYaml<=5.4 has a bug which can lead to ACE (https://cve.mitre.org/cgi-bin/cvename.cgi?name=CVE-2020-14) set version to 6.0.1 because discord-logger depends on PyYaml >5.4, <6.0

@chinnichaitanya This is also relevant for me so if you could merge this that'd be super :)


QR code enhancements

Release criteria

Add App User name and Project name
Add "temporary" (translated) in the middle of a draft QR code

Note to self when I come back to this issue: I could definitely do some canvas tricks to add things to the QR image, but I ran into some issues last time I tried. Would need to do something different with the Popover component (that's the only place it is used) or maybe use a modal?

Let me know once you start taking a look at this, I have ideas about how to address the issues you encountered before!


Feature/logger updates

if(DEBUG) statements are not necessary anymore
Logs are colorized
winston-util.js is added to easily deal with logging options

Nice to see these improvements! Could you remove the if (DEBUG) {...}'s in ./server/index.js as well?

@liayoo I missed them! just removed them all! Thanks for the review!

I am merging it just right after leaving this comment! Thanks for the reviews!


Turn off module header hint by default

Seemingly too annoying, especially with lens based template haskell.

Done in v2.0.9.


⚠ Samsung has degraded performance

In 5e49072, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2072 ms

Resolved: Samsung performance has improved in 947ffea.


Configure via env vars, not sed

This PR changes the entrypoint.sh script so that it doesn't need to change airflow.cfg to customize the settings.
The main reason we want this is that we have a repo which is our AIRFLOW_HOME, it contains a dags/ folder, and custom plugins in  plugins/, some library functions/python modules and an airflow.cfg.
Because we have more than just the dags folder in our repo I'd prefer to not have to manually volume mount each thing individually and would prefer to just docker run -v .:/usr/local/airflow. Which works, except that our config is no longer default so some of the sed commands fail, and others end up changing the file which is tracked in git. All just a bit messy.
So since Airflow has built in feature to read config settings from environment variables we can just export some vars. No sed needed, no changes I need to revert every time in git (and an added bonus no changing to the base docker image at run time)

@puckel Anything I can do to get this merged? (other than inventing a magic machine that gives everyone more time)

Hi @ashb
Thanks for your PR, I've tested your changes. It would be great if you update the readme too according to your changes.
Thanks

Hi @puckel I've finally gotten around to adding some docs to the readme about configuration via environment. Let me know if you'd like any more detail.

Thanks @ashb I'll release it at the same time with 1.9.0

Great, 1.9.0 should be soon - RC8 has been accepted by the Airfow team so we're just waiting on Apache governance process now. (Not sure if you follow along on the mailing list or not.)

I'm following the mailing-list too. I'll release as soon as possible when 1.9.0 is out


Make error message (for invalid project name) work with Test PyPI

Right now, when someone tries to upload a package to PyPI but the project name they want to use is not a project name we allow, the error message we send them includes -- hardcoded -- the URL https://pypi.org. Instead, we should use Pyramid's URL routing to refer to the URL of the PyPI in question, thus ensuring that we send error messages that, as appropriate, refer to pypi.org, test.pypi.org, or any other future PyPI.
To fix this, look at warehouse/forklift/legacy.py and tests/unit/forklift/test_legacy.py (the test is test_fails_with_invalid_names), and make changes similar to the changes I made in #3183. If you're used to string substitution/string formatting in Python, this should be a relatively approachable first Warehouse issue.

Good First Issue: This issue is good for first time contributors. If you've already contributed to Warehouse, please work on another issue without this label instead. If there is not a corresponding pull request for this issue, it is up for grabs. For directions for getting set up, see our Getting Started Guide. If you are working on this issue and have questions, please feel free to ask them here, #pypa-dev on Freenode, or the pypa-dev mailing list.

I'm working on this issue.

Hi @pgadige! Saw your question in IRC about logging in within your local developer environment. As Dustin said, check https://warehouse.readthedocs.io/development/getting-started/#logging-in-to-warehouse . Glad to have you joining us!
You mentioned you're interested in the larger process of building something like Warehouse. I recommend you take a look at:

the PyPA's list of presentations and articles
PyPA's history of Python packaging
the Warehouse roadmap
blog posts, mailing list messages, and notes from our core developer meetings
Warehouse application overview


[repo add] StarBreach

would like to add my personal repository for StarBreach in the new gallery so to allow other user to download the data file directly from the app
https://github.com/andreanespeca/StarBreach
would be also great to have some sort of visual interface in the app to make the adding process easier.
Thanks

Hi. I'm not sure I'll be able to do the visual/tutorial thing, but:
Please make a PR to include your repository in registry. As an example, see https://github.com/BSData/gallery/blob/master/registry/entries/wh40k.catpkg.yml
You should add a starbreach.catpkg.yml in the same folder, with the following content:
location:
  github: andreanespeca/StarBreach

We're going to redirect old AppSpot endpoint onto this Gallery's, so when this happens, your StarBreach repo will be visible in the same list as currently users get when they open a new BattleScribe installation.

@andreanespeca - mention for additional attention 👆👆👆

Will look into it as soon as possibile thanks for letting me know!
Il Ven 18 Set 2020, 18:17 Amadeusz Sadowski<EMAIL_ADDRESS>ha
scritto:

@andreanespeca https://github.com/andreanespeca - mention for
additional attention 👆👆👆
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/BSData/gallery/issues/11#issuecomment-694958735, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ANDKCACOCNQKBDKFDJGS3TTSGOBX3ANCNFSM4OSYRIIQ
.

@amis92 i have created a pull request to include star breach in the gallery let me know if i have done anything wrong!
THanks


stage-1: fix ZFS root mount

Fixes #9904, although stage-2-init.sh still tries to remount / rw, then complains about not being able to do so.
Signed-off-by: Hajo Möller<EMAIL_ADDRESS>

Cc @wkennington

I need to take a better look at this because I don't think this is quite what you want to do with mountPoints.

This is slightly similar to what the Ubuntu PPA does, see https://gist.github.com/dasjoe/b7d1f7b5e8b0957eede0#file-zfs-L134-L141

Right, but why would you need the zfsutil option if you had the mountpoint set to legacy?

I don't set mountpoint=legacy, this is how I create rpools:
HOST=interzone
DISKS="/dev/disk/by-id/ata-SAMSUNG_SSD_830_Series_S0WKNYAC302936"

zpool create \
  -f \
  -o ashift=13 \
  -O normalization=formD \
  -O atime=off \
  -O canmount=off \
  -O compression=lz4 \
  -O mountpoint=/ \
  -R /mnt \
  "$HOST"-rpool \
  "$DISKS"

for disk in $DISKS; do
  parted -s "$disk" mkpart GRUB2 48s 2047s
  parted -s $disk set 2 bios_grub on
done

zfs set sync=disabled "$HOST"-rpool
zfs create -o canmount=off -o mountpoint=none "$HOST"-rpool/ROOT
zfs create -o canmount=noauto -o mountpoint=/ "$HOST"-rpool/ROOT/15.09
zfs mount "$HOST"-rpool/ROOT/15.09
zpool set bootfs="$HOST"-rpool/ROOT/15.09 "$HOST"-rpool

So what's wrong with our current behavior? you shouldn't be using an explicit mount on datasets without legacy mountpoints? I'm not against having an override but it should be explicit.

As explained in #9904, a non-working /etc/fstab line gets built for setups that boot off datasets with explicit or inherited non-legacy mountpoints, leading to a system that gets stuck during stage-1.
When not using mountpoint=legacy you need to mount the required datasets using the (undocumented) zfsutil flag:  mount -t zfs -o zfsutil pool/root-dataset /mnt-root/, otherwise the mount will fail.
Also, if mount -t zfs pool/root-dataset /mnt-root/ would work, it would set pool/root-dataset's mountpoint property to /mnt-root/, causing all descendant datasets to inherit a wrong mountpoint.

I need to use -o zfsutil on my system with mountpoint=legacy, which should be noted was not required before.

Try <IP_ADDRESS> and let me know if the problem persists.
On Sat, Sep 19, 2015, 18:36 Carter Charbonneau<EMAIL_ADDRESS>wrote:

I need to use -o zfsutil on my system with mountpoint=legacy, which
should be noted was not required before.
—
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/pull/9905#issuecomment-141724397.

@wkennington that fixes it with mountpoint=legacy, but will still break boot without mountpoint=legacy afaict, which worked before.

ZoL <IP_ADDRESS> fixes booting off rpools created in the way I documented above, so this PR is not required.


[BUG] For some users the tweets are not displayed, but an error.

Describe the bug
For some users (e.g.: @2ContextPpl) the tweets are not displayed, but an error: "Null check operator used on a null value".
To Reproduce
Steps to reproduce the behavior:
Search the user, e.g. @2ContextPpl and click to see the tweets.
Screenshots

Device
LOS 17.1, Fritter 2.11.2

This one should be fixed now, and going out in v2.12.0. It looks like that profile has a pinned tweet, but the tweet no longer exists, so Fritter was barfing on it 😅


sinter plot_error_rate plot_args_func doesn't apply to fill_between

eg when using
plot_args_func=lambda index, curve_id: {'color': 'red'
                                                        if curve_id == 'pymatching'
                                                        else 'blue'}

the lines are colored according to the decoder, but the fill_betweens are not

This appears to be fixed now, probably during https://github.com/quantumlib/Stim/commit/e5172a480505fea9bc05ebe506c9bc14cf3d6f2a


Memory leak issue in swagger-jaxrs2 on a Wildfly app server

Hello there.
I have been experiencing a very strange memory leak on my application server. This memory leak happens when deploying an application without restarting the server from time to time. It eventually leads to an OutOfMemory error after some deployments.
Here is a little POC repository that demonstrate the memory leak: https://github.com/creekorful/wildfly-jee-swagger-memory-leak
Any help would be appreciated.
Regards

Thanks for the detailed reproduction, @creekorful. Unfortunately, I'm not sure how soon we'll be able to further investigate the issue.

Thanks for your response @webron. This bug looks really critical to me, I'll try to collect more details & fix the issue if possible, but any help would be really welcome when it's possible for you.


Support events in print_event

The print_event action currently only supports a single event and fails when invoked with multiple events. This PR adds support for multiple events.
https://issues.redhat.com/browse/AAP-4829

@benthomasson Please review

@mkanoor Added multiple events test

@benthomasson @mkanoor This is outside the scope of this PR, but I've noticed that this test_examples.py file is getting pretty large and seems a bit unwieldy to me. We already have two PRs both adding tests to this and one of them will have to change its numbering depending on whose goes in first. Not that that's a huge deal, but it feels odd that we would have to do that any time two PRs adding examples are in flight.
I'm not sure what the best solution is, but it feels to me like these should be contained within the examples folder with their corresponding yaml files and separated out into separate test files. I think this would make it much easier to maintain and (perhaps more importantly) lookup/search which ones you may care about later at a glance instead of digging through a 600+ line file.
Maybe I'm wrong, would like to hear your opinions.

@eclarizio the examples directory is a place to simplified examples of different features of the rule language that we want to support.   It should not be used a place to put all the unit tests.  We should use other test files for normal unit tests.   For these two PRs they are good examples of language features and can go into this directory.    They do need to be renamed as they already conflict with other tests that I added in the meantime.

@eclarizio the examples directory is a place to simplified examples of different features of the rule language that we want to support. It should not be used a place to put all the unit tests. We should use other test files for normal unit tests. For these two PRs they are good examples of language features and can go into this directory. They do need to be renamed as they already conflict with other tests that I added in the meantime.

Yeah, that's fair, I wasn't sure if organizationally my idea was feasible or good, but ultimately running into issues with "conflicting" names seemed to poke at a potentially deeper issue that could be solved fairly easily. Thanks! 👍

@benthomasson Please review I have fixed the example file names


[FEATURE REQUEST] Allow resuming interupted dreambooth lora training

Feature Request
I would like to be able to save in-progress dreambooth LoRA training weights to disk or s3 every n steps, such that if the training is interrupted, it can be resumed from the latest checkpoint.
Motivation
GPU prices are much lower if you use spot instances, or interruptible community cloud instances like SaladCloud. An obstacle to using interruptible infrastructure is the potential to lose training progress. This drawback could be mitigated by being able to easily resume interrupted training jobs.
Additional Context
No response

I think this may really just be a documentation issue. I found corresponding options in the diffusers train_dreambooth_lora_sdxl.py that explained the options. Solution is:
autotrain dreambooth \
--num-steps 500 \
--checkpointing-steps 50 \
--resume-from-checkpoint latest \
--checkpoints-total-limit 1 \
...


document error: getting_started.rst

In the section Building without the Zephyr SDK
is there lost cd $ZEPHYR_BASE before make ?
$ cd $ZEPHYR_BASE
$ mkdir build && cd build
$ cmake $ZEPHYR_BASE/scripts
$ make
$ echo "export PATH=$PWD/kconfig:\$PATH" >> $HOME/.zephyrrc
$ source $ZEPHYR_BASE/zephyr-env.sh

I believe it is correct as written; make is supposed to run in the build directory, right @SebastianBoe

@dbkinder But there is no such file named Makefile in build directory.

@dbkinder  sorry, it's my fault. the right way is cmake $ZEPHYR_BASE/scripts, but i forgot to type scripts

OK.  We'll close this issue then.


Add dry run

Pull Request Description
Summary
This pull request introduces a new --dry-run flag to the CLI application. This flag allows users to see what would be sent to the model without actually sending the request. The changes include updates to the README.md for documentation purposes and modifications to the CLI code to implement the dry-run functionality.
Files Changed

README.md

Added a description for the new --dry-run flag under the Application Options section.

cli/cli.go

Implemented the logic for handling the --dry-run flag. This includes fetching the pattern and context content (if provided) and displaying the formatted request that would be sent to the model.

cli/flags.go

Added the DryRun field to the Flags struct to accommodate the new command-line option.

Code Changes
README.md
 Application Options:
   -u, --url=              Choose ollama url (default: http://<IP_ADDRESS>:11434)
   -o, --output=           Output to file
   -n, --latest=           Number of latest patterns to list (default: 0)
+      --dry-run           Show what would be sent to the model without actually sending it

cli/cli.go
 func Cli() (message string, err error) {
 		}
 	}

+	if currentFlags.DryRun {
+		var patternContent string
+		var contextContent string
+
+		if currentFlags.Pattern != "" {
+			pattern, patternErr := fabric.Db.Patterns.GetPattern(currentFlags.Pattern)
+			if patternErr != nil {
+				fmt.Printf("Error getting pattern content: %v\n", patternErr)
+				return "", patternErr
+			}
+			patternContent = pattern.Pattern // Assuming the content is stored in the 'Pattern' field
+		}
+
+		if currentFlags.Context != "" {
+			context, contextErr := fabric.Db.Contexts.GetContext(currentFlags.Context)
+			if contextErr != nil {
+				fmt.Printf("Error getting context content: %v\n", contextErr)
+				return "", contextErr
+			}
+			contextContent = context.Content
+		}
+
+		systemMessage := strings.TrimSpace(contextContent) + strings.TrimSpace(patternContent)
+		userMessage := strings.TrimSpace(currentFlags.Message)
+
+		fmt.Println("Dry run: Would send the following request:\n")
+		if systemMessage != "" {
+			fmt.Printf("System:\n%s\n\n", systemMessage)
+		}
+		if userMessage != "" {
+			fmt.Printf("User:\n%s\n", userMessage)
+		}
+		return "", nil
+	}

cli/flags.go
 type Flags struct {
 	YouTube                 string  `short:"y" long:"youtube" description:"YouTube video url to grab transcript, comments from it and send to chat"`
 	YouTubeTranscript       bool    `long:"transcript" description:"Grab transcript from YouTube video and send to chat"`
 	YouTubeComments         bool    `long:"comments" description:"Grab comments from YouTube video and send to chat"`
+	DryRun                  bool    `long:"dry-run" description:"Show what would be sent to the model without actually sending it"`
 }

Reason for Changes
The --dry-run flag is introduced to provide users with a way to preview the request that would be sent to the model. This can be useful for debugging and ensuring that the correct data is being sent without making an actual request.
Impact of Changes

Functionality: Adds a new feature allowing users to perform a dry run of their request.
Usability: Improves the user experience by providing a way to verify requests before sending them.
Codebase: Introduces additional conditional logic to handle the --dry-run flag.

Test Plan

Manually test the --dry-run flag with various combinations of patterns, contexts, and messages to ensure the correct output is displayed.
Verify that the application behaves as expected when the --dry-run flag is not used.

Additional Notes

Ensure that any existing automated tests that may interact with the CLI are updated to account for the new --dry-run functionality.
Consider adding automated tests specifically for the --dry-run feature in future updates.

Thank you, for the implementation.
Please refactor and integrate it to the Chatter struct:
...
var chatter *core.Chatter
if chatter, err = fabric.GetChatter(currentFlags.Model, currentFlags.Stream, currentFlags.DryRun); err != nil {
return
}
....
type Chatter struct {
db *db.Db
Stream bool
DryRun bool

model  string
vendor vendors.Vendor

}
And use the field DryRun in the methods of the Chatter struct to print out instead of sending the request.

@eugeis I've refactored the dry run to a DryRun vendor as requested and ensured that the dry run model is excluded when listing models.
Let me know if there's anything else you'd like to adjust.


JSONPath template language

@brendandburns Thanks. This is the rebased version of #9296

[x] "" quote string
[x] . key operator
[x] [] array operator
[x] start : end : step slice operator
[x] @ current object
[x] * wildcard operator
[x] ?() array filter
[x] .. parent operator
[x] [,] array indices as a set
[x] range, end keyword

LGTM.  Please squash the commits. and then I will add 'lgtm' label and get this merged.

@brendandburns Thanks. I squashed them into 8 commits.

Sorry, I wasn't clear.  Please squash to 1 commit.  If there really are multiple conceptual things, they should be broken out as separate PRs (but I think it's fine for there to be one commit for this)
Shippable appears to be a flake, I'm re-running.
@k8s-bot test this please
--brendan

@brendandburns Thanks. There is only one commit now.

LGTM, shippable is a flake, re-running.

There is something severely borked in shippable.  Its complaining about v1beta3, but there's nothing about v1beta3 in this PR.
@k8s-bot test this please
If this passes jenkins, I'm just going to merge it.

@k8s-bot test this please

This is strange. The code of 8dcd077 and 949704a are the same, but GCE e2e build/test passed for commit 8dcd077 and failed on 949704a.

@daizuozhuo I think it's a flake, I'm just going to merge this, and we'll see what happens...

@daizuozhuo now that this is submitted, are you working on the integration with kubectl?  We should create a new output format printer.
Thanks!
--brendan

@brendanburns Yes, I will send pull request soon.

/cc @pmorie re: expressiveness.

Awesome work here guys, looking forward to this. Given this work, I'm going to stick to Go templating for now, instead of searching for some 3rd-party json pre-processor to fill in variables and the likes.


SharePoint PnP Provisioning Service

Fail repeated saying I need an App Catalog in the Tenant and one already exist.  Please advise.

I’m sorry I don’t have enough information to help you.  What exactly is failing, what are you trying to do exactly?

Setup the SharePoint learning portal.
Thanks,
jbh

From: Julie Turner<EMAIL_ADDRESS>Sent: Thursday, April 25, 2019 1:11 PM
To: pnp/custom-learning-office-365
Cc: Harvey, Jermaine; Author
Subject: Re: [pnp/custom-learning-office-365] SharePoint PnP Provisioning Service (#44)
I’m sorry I don’t have enough information to help you. What exactly is failing, what are you trying to do exactly?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpnp%2Fcustom-learning-office-365%2Fissues%2F44%23issuecomment-486759232&data=02|01|jharvey%40shm.net|e3309857ced4466fbdac08d6c9a11936|e2cbf95a41ca4a3c8268fbfe6502b1a6|1|0|636918091092274862&sdata=sJXHfrdx2DxFSay7FMYkv2AZJVZvznluQ%2BXL1PuxHrI%3D&reserved=0, or mute the threadhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAL5NIQHDQZZ5MD5RFVMAQK3PSHQ5BANCNFSM4HIO4GBA&data=02|01|jharvey%40shm.net|e3309857ced4466fbdac08d6c9a11936|e2cbf95a41ca4a3c8268fbfe6502b1a6|1|0|636918091092284858&sdata=xpGezEEtfZOyt1f2b4V82aGdrKGudBR3Vde%2BqIC6g5s%3D&reserved=0.
This e-mail may contain confidential or privileged information. If you are not the intended recipient, please advise the sender by reply e-mail and then delete this e-mail immediately.

  There are two ways to get the custom learning solution into your tenant... by using the pnp provisioning service, which is what we recommend or by a manual method using powershell and other manual steps.  You need to provide me detailed information about what steps you've already taken and where you're seeing the error message.

I've attempted to use the service (as my subject line would suggest) w/ this link...
https://aka.ms/CLO365.
I get an e-mail saying "Unfortunately, request to provising "Custom learning for Office 365" template failed."
If that same e-mail it states...
"In order to provision the template you need to have an App Catalog in your tenant.  Please create one, wait 30 mins, and try again"
We already have one (see below).  Please advise.
[cid:e45c5ec5-4b77-4287-af57-9a1726bcf85b]
Thanks,
jbh

From: Julie Turner<EMAIL_ADDRESS>Sent: Thursday, April 25, 2019 1:59 PM
To: pnp/custom-learning-office-365
Cc: Harvey, Jermaine; Author
Subject: Re: [pnp/custom-learning-office-365] SharePoint PnP Provisioning Service (#44)
Again, i'm happy to try and assist you if you give me some detailed explanation of what steps you've taken and where you are. There are two ways to get the custom learning solution into your tenant... by using the pnp provisioning service, which is what we recommend or by a manual method using powershell and other manual steps. You need to provide me detailed information about what steps you've already taken and where you're seeing the error message.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpnp%2Fcustom-learning-office-365%2Fissues%2F44%23issuecomment-486776735&data=02|01|jharvey%40shm.net|e84c5fba54104a4d582c08d6c9a7d606|e2cbf95a41ca4a3c8268fbfe6502b1a6|1|0|636918120019861342&sdata=rdD6LBuxvxWf5dqkP3JqKd7AGY4zO22AjnVDieX%2FPzQ%3D&reserved=0, or mute the threadhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAL5NIQFTN7LNQIYUIJGXEO3PSHWR7ANCNFSM4HIO4GBA&data=02|01|jharvey%40shm.net|e84c5fba54104a4d582c08d6c9a7d606|e2cbf95a41ca4a3c8268fbfe6502b1a6|1|0|636918120019871356&sdata=AdwAJ9Wz4flxDUifkOjufITabM86pmFlF0HN7UBCIFY%3D&reserved=0.
This e-mail may contain confidential or privileged information. If you are not the intended recipient, please advise the sender by reply e-mail and then delete this e-mail immediately. NOTE: This message contains an attachment that cannot be scanned for sensitive information. Please discard any attached files containing private health information.

Ok thank you hat helps a lot and explains my confusion as support for the provisioning service needs to go to their support issues list. Unfortunately I cannot offer you any support for the provisioning service itself only with the custom learning solution.  I can however transfer this issue to that site or you can submit it there yourself. https://github.com/SharePoint/sp-dev-docs/issues

Please transfer it.
Thanks,
jbh

From: Julie Turner<EMAIL_ADDRESS>Sent: Thursday, April 25, 2019 2:46 PM
To: pnp/custom-learning-office-365
Cc: Harvey, Jermaine; Author
Subject: Re: [pnp/custom-learning-office-365] SharePoint PnP Provisioning Service (#44)
Ok thank you hat helps a lot and explains my confusion as support for the provisioning service needs to go to their support issues list. Unfortunately I cannot offer you any support for the provisioning service itself only with the custom learning solution. I can however transfer this issue to that site or you can submit it there yourself. https://github.com/SharePoint/sp-dev-docs/issueshttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FSharePoint%2Fsp-dev-docs%2Fissues&data=02|01|jharvey%40shm.net|f6e0751aaf274bec6b3208d6c9ae649e|e2cbf95a41ca4a3c8268fbfe6502b1a6|1|0|636918148177625089&sdata=UbSacTmFJjmMT5YpWaO9vO9i61tCMFHm9jtaOrKrMSE%3D&reserved=0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fpnp%2Fcustom-learning-office-365%2Fissues%2F44%23issuecomment-486793382&data=02|01|jharvey%40shm.net|f6e0751aaf274bec6b3208d6c9ae649e|e2cbf95a41ca4a3c8268fbfe6502b1a6|1|0|636918148177635098&sdata=aXfAPmToqEMg1lAloFl3lk%2FRpIEWEC59kZEvlw41LIk%3D&reserved=0, or mute the threadhttps://nam01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAL5NIQADOMH2FAMBMIIX5V3PSH4CBANCNFSM4HIO4GBA&data=02|01|jharvey%40shm.net|f6e0751aaf274bec6b3208d6c9ae649e|e2cbf95a41ca4a3c8268fbfe6502b1a6|1|0|636918148177635098&sdata=Xw8gFOlmfDH3q6Tky1BenQYzs5pJuWrKzDptn9HCRuA%3D&reserved=0.
This e-mail may contain confidential or privileged information. If you are not the intended recipient, please advise the sender by reply e-mail and then delete this e-mail immediately.

I will and link to it here but be aware since I’ll be entering it for you you will not be notified of updates to it unless you specifically follow the issue.

Transferred to so-dev-docs for support
https://github.com/SharePoint/sp-dev-docs/issues/3842


Upgraded a v7.0 to v7.1 User->Change Password gives "Invalid salt version"

Can't recreate this in 7.1.  Successfully changed the password when the password had been set in a 7.0 instance.
Is this happening every time for you?


Functions on the global scope

Hey @brandonroberts, I'm sorry to abuse your github issues like this, but I'm following your great example that you've given in this repo, and wanted to ask if you know why the standard convention for Angular code emerging in last year or so is to declare the controller functions on the global scope?  Eg as in your code
angular.module('app.admin', []).component('admin', {
   ...
	controller: AdminController,
   ...
});

function AdminController() { // function is on the global scope
}

As far as I've ever known, this is bad practice.  Any idea how has it become widespread practice in Angular?

@poshest I do agree thats its bad practice. I think its best practice to organize your angular code this way though.  One if not the most well known style guide follows this, but he recommends avoid the functions being on the global scope by wrapping them in a closure. I just haven't done that here as this is meant to be an example of using the router, not necessarily building an entire angular app.
Style guide: https://github.com/johnpapa/angular-styleguide

OK, thanks for clarifying and for the link @brandonroberts. It's good to revisit that after a while.  It's a pretty comprehensive styleguide now.  :)


Feature/add puppeteer loader

Hi there,
I would like to propose some changes to the project:
I added a loader using Puppeteer to improve web scraping performance. I was previously using Cheerio for this task, but I decided to use Puppeteer because of its additional features and ability to interact with sites and access content protected by login forms and scripts. For more information, please refer to this blog post by Scraper API: link to the blog post and this comparison by AIMultiple: link to the comparison.

[x] implement loader
[x] added a test:watch script to the project to ensure that our tests are always up-to-date and working correctly. This script runs automatically whenever a file is saved.
[x] added some documentation to the code to make it easier to understand and maintain.

Please feel free to review and comment on these changes.
Best regards,

@CarlosZiegler thanks for this! the test doesn't seem to finish on windows (see CI run). Would you have any idea why / how to fix?


feat: basic Vector.inductionOn API

Also make the case names slightly nicer for when using the syntax:
induction v using Vector.inductionOn with
| nil => sorry
| cons ih => sorry

bors r+


Doesn't compile against lts-7.14

$ stack build

Error: While constructing the build plan, the following exceptions were encountered:

In the dependencies for sext-<IP_ADDRESS>:
    base-<IP_ADDRESS> must match >=4.7 && <4.9 (latest applicable is <IP_ADDRESS>)
    template-haskell-<IP_ADDRESS> must match >=2.9 && <2.11 (latest applicable is <IP_ADDRESS>)
needed due to risk-types-<IP_ADDRESS> -> sext-<IP_ADDRESS>

Plan construction failed.

Hey, thanks for the tip. This should be sorted now.

Thanks!


Remove old secret v1 from UI

As we have the Kubernetes secrets as part of configuration management, we should remove the secret v1 from argo.

done


verdaccio-htpasswd v0.1.4

Type: Fix/Improvement
The following has been addressed in the PR:

Unit or Functional tests are included in the PR

Description:

Bumps verdaccio-htpasswd to v0.1.4

This close "again" #441


Live update pinned / unpinned chats

<If necessary, assign reviewers that know the area or changes well. Feel free to tag any additional reviewers you see fit.>
~On hold until https://github.com/Expensify/Web-Expensify/pull/30612 is merged. This PR shouldn't break if merged before the Web-E PR, but its functionality won't work -> the reportTogglePinned event will never get triggered.~
No longer on hold! Web-e PR Merged
Details

Previously, if a user toggled a pin in one device, they'd have to refresh other devices to get pin / unpin updates.
This isn't best for end users, so I added live pinning / unpinning across devices.
Fixed Issues

Fixes https://github.com/Expensify/Expensify/issues/159214
Tests

git checkout this & related web-e branches on your machine (or pull from master, if web-e PR was merged already)
Run the app on multiple local devices (ex: mobile and web)

Sign in to the same account on both

Pin a chat on one device, see the pin update on the other device

QA Steps

Log in to e.cash on multiple devices (ex: desktop & web) with the same account
Pin a chat on one device, see the pin update live on the other device without needing to refresh
Repeat on the other device - pin or unpin, see the change update live on the original device

Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[x] iOS
[x] Android

Screenshots

Web

This is showing sync between web & Android:
https://user-images.githubusercontent.com/3885503/113706271-c65fb800-96de-11eb-9f12-729cf1464ba6.mov
Mobile Web

This is showing sync between Mobile Web (iOS - Safari) & web:
https://user-images.githubusercontent.com/3885503/113707376-1a1ed100-96e0-11eb-9ace-b4c52216ca0e.mov
Desktop

This is showing sync between desktop & iOS:
https://user-images.githubusercontent.com/3885503/113706840-6c132700-96df-11eb-9be4-4cd77f8c88d1.mov
iOS

See Desktop video ⬆
Android

See Web video ⬆

@yuwenmemon I'm going to add testing details tomorrow, but feel free to review today if you'd like :)

Ready for review :)

Taken off hold (since web-e PR was merged a bit ago) - ready for review :)

Added a test + replied to comments. Ready for another review :)

Updated 👍 - added comments to clarify pusher scriptions

Fixed merge conflict + responded to your comments @stitesExpensify :)

Updated & ready for final review! (hopefully)

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging in version: 1.0.32-1🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


collate sequences

Collate sequence outputs into a sub-directory for each seed being rendered. This would help better organize the outputs.

I no longer think this is necessary; naming conventions ensure no filename clashes, so scripts can target a specific sequence easily.


Should ExtData have an option to set an offset of the clock DT rather than ISO duration

When trying to trace problem in GAAS (which we eventually traced to the fact that no one ever updated the files in the AMIP directory there for GAAS to correctly function) I remembered I have to set an offset in the ExtData file for GAAS that is the model heartbeat like so as in this example:
Collections:
  aod_a.sfc.%y4%m2%d2_%h2%n2z.nc4:
   template: das.aod_a.sfc.%y4%m2%d2_%h2%n2z.nc4
   freq: PT3H

Samplings:
  GAAS_sample_0:
   update_offset: PT450S
   exact: True

Exports:
  aod_a:
    collection: aod_a.sfc.%y4%m2%d2_%h2%n2z.nc4
    sample: GAAS_sample_0
    variable: AOD
    fail_on_missing_file: false

The reason for this is that GAAS runs 1 heartbeat before the start of the hour. So if you start at 21z, GAAS runs 1 heartbeat before 0z, 3z, 6z, etc. But the files GAAS needs are timestamped on the hours, so essentially GAAS needs the file timestamped 1 heartbeat in the future.
Notice that I have to set an offset, and that offset needs to be the heartbeat of the clock and apparently when this file was made it was for a 7.5 minute heartbeat. So this is an accident waiting to happen, if the user changes the heartbeat in CAP.rc, this needs to change too.
I'm thinking we have some notion of a HEARTBEAT you specify for that offset frequency. So rather than an ISO duration you could alternatively specify a number of hearbeats of the clock. That fixes this accident waiting to happen.

I think I understand.   I'm wondering if it should be in terms of DT of the component as opposed to the heartbeat itself?   Current case is presumably degenerate, but you might want to consider what happens if GAAS runs at a different time step size than the rest of the model.

In MAPL2 every component has the same heartbeat as far as the actual clock

Yes, but they do not all have the same DT.  Hence my point.  Bill runs a number of components at a different frequency.
Presumably you are saying that even if GAAS were to run say twice as often, you still want the offset to be heartbeat.  Which is fine, just wanted to be certain that you'd thought it through.   I can imagine vaguely analogous situations where we'd want to use DT as a multiplier/offset.

@tclune,  nothing can run a finer timescale than the heartbeat of the clock as created by Cap in MAPL2 as far as ExtData is concerned. So GAAS would never running 2x more than this. Besides ExtData does not have access to any of the components run DT's which once again whether or not they are used at the mercy of the component writers in MAPL2. It doesn't matter what they are doing, if it say knew the DT of component foo, that could mean because the component writer, didn't use the timer that created in any way. ExtData can see the only clock provided, that's it.
Also remember it is far worse. GAAS actually does work based on if the fields provided by ExtData are not all MAPL_UNDEF which only happens when you are exactly the file time, that's what the exact: true is for in the sampling definition. Because was pretty much the only way forward when we had that emergency when GAAS was modified to use ExtData without consultation.

@tclune case in point, someone added a GAAS_DT to AGCM.rc , but GAAS never checks the run alarm, so not sure why one bothered adding this to the AGCM.rc. sigh

Just as a note. I talked with @vbuchard and if need be, I don't think it would be too hard to do run-time detemplating of the GAAS control yamls where we grab the DT: from CAP.rc and then shove that into the yaml.
But my guess is @bena-nasa when he has the time could add a toggle in ExtData2G with like offset_by_heartbeat: true or whatever.

Unfortunately this might be worse, apparently @wmputman was running chemistry at a different heartbeat from the model as reported to me by @vbuchard so it may not be a single heartbeat of the model clock. This apparently was breaking GAAS since now if the yaml file has the cap heartbeat and chemistry running at a longer heartbeat, I need to see what the offset needs to be in the yaml file. Is 2 hearbeats if that is the frequency of chemistry, I don't know I will find out if I can figure out how to run GAAS.
Of course the real issue is that GAAS SHOULD NEVER HAVE BEEN CONVERTED TO USE EXTDATA IN THE FIRST PLACE. The logic GAAS was using to decide to run (do I have a file) was simply something that ExtData had no way to transmit so there kluge upon kluge I needed to do to get this to work.

@wputman @vbuchard  Yep, looks like you need to set the offset in the GAAS extdata.yaml file to the CHEMISTRY_DT rather than the heartbeat. Unfortunately since ExtData has no way to ever detecting this, this will have to be a scripting rather than code solution

@wputman @vbuchard Yep, looks like you need to set the offset in the GAAS extdata.yaml file to the CHEMISTRY_DT rather than the heartbeat. Unfortunately since ExtData has no way to ever detecting this, this will have to be a scripting rather than code solution

This shouldn't be too hard to do. I have a quick possible code change here:
https://github.com/GEOS-ESM/GEOSgcm_App/pull/647
We can test it on Monday


Feature: Apply good font for the Navbar

Feature description
the font of the Navbar is not looking good so , I want to change the font of the Navbar
👀 Have you spent some time to check if this issue has been raised before?

[x] I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

[x] I have read the Code of Conduct

can you please assign this issue to me?

/assign


Requiring 3rd-party Library Manually Doesn't Work

Describe the bug
If environment is set to LOVE but there's local love = require("love") in the code, the LOVE API won't appear.
This may be not LOVE-specific however. LOVE here is just an example.
To Reproduce
This code explains everything:
local love = require("love")
love.graphics.draw() -- hover your cursor to "draw", no hints.

Expected behavior
require loads 3rd-party libraries (in this case, require("love") should load LOVE definitions).
Screenshots
With require("love")

Without require("love")

Environment (please complete the following information):

OS: Windows
Is WSL remote? No
Client: VSCode

Additional context
We have strict local love = require("love") as part of my and our coding style, so if require("love") does not load LOVE definitions then we have to resort back to our external definition.
Provide logs
file_d%3A_Data_Development_love-11.3-win64_nlay.log

Due to the example code in the specific, I mistakenly thought that love is a global variable.

LOVE is a global variable indeed, but we have strict local love =
require("love") where the require fails to resolve.
Pada tanggal Kam, 22 Jul 2021 13.03, 最萌小汐 @.***>
menulis:

Due to the example code in the specific, I mistakenly thought that love
is a global variable.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/sumneko/lua-language-server/issues/614#issuecomment-884657519,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABZHFFT22KF4Q6UVTNU2HHTTY6RD3ANCNFSM5AZGH7OA
.


Hudi Sink Connector Upsert records

Motivation
The Hudi Sink Connector currently only inserts record to the Hudi Table, I would like the sink connector to also be able to upsert records to the Hudi table.
Modifications
Provided a flag for the Hudi sink connector hoodieUpsertRecords in the sink connector configs, which if set to True uses the upsertPreppedRecords instead of bulkInsertPreppedRecords in the BufferedConnectWriter.
Verifying this change

[ ] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
Documentation
Check the box below.
Need to update docs?

[ ] doc-required
(If you need help on updating docs, create a doc issue)

[ ] no-need-doc
(Please explain why)

[x] doc
(If this PR contains doc changes) lakehouse-sink has been updated to reflect the new config

@hangc0276 @horizonzy can you please review this?


Parity break with OpenAI API: /models

Reproduction:
curl -X GET https://infinity.semanticallyinvalid.net/models

Expected: List of model dictionary:
{
  "data": [
    {
      "id": "/thenlper/gte-small",
      "stats": {
        "queue_fraction": 0.0,
        "queue_absolute": 0,
        "results_pending": 0,
        "batch_size": 4096
      },
      "object": "model",
      "owned_by": "infinity",
      "created":<PHONE_NUMBER>,
      "backend": "torch"
    }
  ],
  "object": "list"
}

Actual: Dictionary:
{
  "data": {
    "id": "/thenlper/gte-small",
    "stats": {
      "queue_fraction": 0.0,
      "queue_absolute": 0,
      "results_pending": 0,
      "batch_size": 4096
    },
    "object": "model",
    "owned_by": "infinity",
    "created":<PHONE_NUMBER>,
    "backend": "torch"
  },
  "object": "list"
}

I have worked around this here, but for this commit to be accepted in the upstream, it would need to adhere to the expected list of models.

Make sense - thanks for the input and pointing this out.

PR welcome this should not have a large effect on the integrations.

Batch_size is 4096 - I would not recommend setting this that high - you will likley OOM and there is no speedup beyond 128 IME.

Thank you. Are you able to educate me on why there is no speedup after 128?

You could check out e.g. https://github.com/michaelfeil/infinity/tree/main/docs/benchmarks - for long enough requests, a batch_size of 32 will saturate the GPU usage up to a decent amount, where further vectorization does not bring any benefit when adding more items in the batch dimension. On CPU, even if avx-512 instructions are used, you might not see a decent speedup beyond 4/8.
Feel free to benchmark it (with the included benchmark scripts and report the results here - I would love to learn from them as well!

@MichaelMcCulloch Do you want to PR a fix?


feat: add FromString

:tada: This PR is included in version 1.12.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Explorer: undo file operations in Explorer

Refs: https://github.com/microsoft/vscode/issues/9390

[x] Windows @meganrogge
[x] macOS @rebornix
[x] Linux @aeschli

Complexity: 4
Authors: @alexdima @jrieken @isidorn
Issues created assign only to @isidorn please
Create Issue

File explorer now supports undoing file operations. For this we changed the underlying service which we use for file operations, so we need to verify older Explorer functionality as well. Open a large repository, open VS Code Explorer and verify:

You can rename files
You can drag and drop files from within Explorer. You can drag and drop files from outside of VS Code into the Explorer
You can copy files and folders
You can Move files to Trash and you can permenantly delete them (alt + context menu)
You can create new files and folders in the Explorer
Optional: connect to remote and verify you can download files via context menu action
All of the above operations can be Undone, both via Ctrl/Cmd + Z and Main Menu > Edit > Undo when the focus is in the Explorer.
When the focus is in the editor that the Undo and Redo are applied on the Editor. And that the Explorer is Undone only when the focus is on the Explorer
Redo works as expected
Delete a very large file > 10mb. Try to bring it back. Make sure VS Code does not freeze.

Thanks for testing!