Problem with "open button" in the gui.py example

I am new to the pymeasure package and I just tried the gui.py example. This runs fine (using Spyder) but when I hit the open button I get the following error:
Traceback (most recent call last):
File ".\Anaconda3\lib\site-packages\pymeasure\display\windows.py", line 325, in open_experiment
dialog = ResultsDialog(self.procedure_class.DATA_COLUMNS, self.x_axis, self.y_axis)
File ".\Anaconda3\lib\site-packages\pymeasure\display\widgets.py", line 364, in init
self._setup_ui()
File ".\Anaconda3\lib\site-packages\pymeasure\display\widgets.py", line 387, in _setup_ui
self.layout().addWidget(preview_tab, 0, 5, 4, 1)
AttributeError: 'NoneType' object has no attribute 'addWidget'
My current installation setup:

Anaconda version: 5.1.0; build py36_2
Pymeasure version: 0.5.1

Any idea what went wrong?

@tvervust, what version of Qt do you have? It appears that the layout method is missing in QtGui.QFileDialog for your version of Qt.

Hello Colin,
These are the versions:
pyqt: 5.6.0
qt: 5.6.2
Is qt5 supported? I will try with qt4...
Thanks,
Thomas
2018-05-04 15:31 GMT+02:00 Colin Jermain<EMAIL_ADDRESS>
@tvervust https://github.com/tvervust, what version of Qt do you have?
It appears that the layout method is missing in QtGui.QFileDialog for
your version of Qt.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ralph-group/pymeasure/issues/141#issuecomment-386602155,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AlLQHMyajNor8Bi0NC48P-xHGZoT3ae8ks5tvFg9gaJpZM4Tyorw
.

@tvervust, it looks like in PyQt5, the QFileDialog had it's methods renamed, so I suspect the issue is related. See the discussion in http://pyqt.sourceforge.net/Docs/PyQt5/pyqt4_differences.html#qfiledialog.
The simplest short term fix is to use Qt4 and PyQt4.
It would be great to support Qt5 as well. I certainly welcome pull requests, and will see if I can find time to work on this feature.

This should be solved with pymeasure 0.6


Pattern matching externals

Hello, really nice plugin!
I'm doing a server side render of blog posts and was hoping to cache the server side render as externals, but your documentation says I cannot pattern match externals. What would be the recommended solution here? I'm really trying to avoid having to list each possible blog post path as an external.
    new OfflinePlugin({
      caches: {
        main: [
          '/',
          ':rest:',
        ],
        additional: [
          '/post/*',
        ],
      },
      externals: [ '/', '/post/*' ],
      scope: '/',
      updateStrategy: 'all',
      version: 'v1',
      ServiceWorker: {
        output: 'sw.js',
      },
      AppCache: {
        directory: 'appcache/',
      },
    }),```

Thanks, @tomatau
Yes, externals should have pattern matching, do not know why it's not there. I will add. By the way, using externals isn't mandatory, it just prevents warnings from being displayed in console. i.e. if some assets are not listed in externals and are listed in caches then they will be added to output anyway, just with warnings to console.
So you can probably use it as is right now and once I update offline-plugin then warnings will go away.

Thanks for the reply NekR, I've just been listing them in externals anyway and the warnings are good to help catch assets that aren't being generated when you expect them.

Fixed in 2.1.0. Already on npm. Thanks for reporting this :+1:

Oh, I just realized that you wanted pattern matching for externals inside caches section too.
This will require pattern matching at runtime :-(

Thinking more about it, it won't be possible to implement even runtime pattern matching for main and additional cache section since plugin simply do not know what to cache, i.e. it doesn't what means '/post/*'. To match pattern for assets, we have to do matching agains something, e.g. agains generated assets or URLs requested from page in runtime. Given that, pattern matching at runtime could be implemented only for optional cache section since it's to cache assets only when they were requested.

That makes a lot of sense.
Is there no way main and additional can do a pattern match on activate or install based on the current window path?

They need to know what to download. Say you have 10 posts on a server and
use this pattern '/posts/*'. To match and download them, SW has to know
URLs to all posts on server to download them at install/activate time. In
other words, that is impossible.
What you need is a some custom they to handle it. E.g. make a request to
the server, get list of all posts you want to cache/match and then cache
them.
Unfortunately this isn't possible with this plugin, at least yet.
On Apr 3, 2016 11:46 AM, "Thomas"<EMAIL_ADDRESS>wrote:

That makes a lot of sense.
Is there no way main and additional can do a pattern match on activate or
install based on the current window path?
—
You are receiving this because you modified the open/close state.
Reply to this email directly or view it on GitHub
https://github.com/NekR/offline-plugin/issues/26#issuecomment-204915934

I understand the need for a URL.
What I'm asking about is when an install / activate occurs, you can check against the current page URL (from window.location) and add the URL the cache if it's a match. The URLs can be added to the cache dynamically during these events. It wouldn't need to know them all on every install -- just update the cache when it finds a match.

Oh, I see now. Sounds interesting. Let me think about it a bit :+1:

Here is possible API design for it: https://github.com/NekR/offline-plugin/issues/117
Can you take a look? Would you be okay with it?


feat: artifacthub badge

Signed-off-by: Gaius<EMAIL_ADDRESS>
Description

Add artifacthub badge.

Related Issue

Motivation and Context

Screenshots (if appropriate):
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Documentation Update (if none of the other choices apply)

Checklist:

[x] Code compiles correctly.
[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes.
[x] All new and existing tests passed.

Codecov Report

Merging #524 (12192be) into main (c0bc4a9) will decrease coverage by 0.26%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #524      +/-   ##
==========================================
- Coverage   39.28%   39.01%   -0.27%
==========================================
  Files          85       85
  Lines        6443     6443
==========================================
- Hits         2531     2514      -17
- Misses       3547     3565      +18
+ Partials      365      364       -1

Flag
Coverage Δ

e2etests
?

unittests
39.01% <ø> (-0.27%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

client/daemon/peer/peertask_stream_callback.go
44.30% <0.00%> (-20.26%)
:arrow_down:

client/daemon/peer/peertask_base.go
32.24% <0.00%> (-0.71%)
:arrow_down:

client/daemon/peer/peertask_stream.go
57.60% <0.00%> (+0.72%)
:arrow_up:


config-file tag removed from config.xml is not removed from appxmanifest

Bug Report
Problem
What is expected to happen?
Removing a <config-file> element from the config.xml should be reflected in the AppxManifest.xml / package.*.appxmanifest file after running the build command.
What does actually happen?
Having for example this configuration in the main config.xml:
<platform name="windows">
        <config-file parent="/Package/Capabilities" target="package.appxmanifest">
            <Capability Name="allJoyn" />
        </config-file>
    </platform>

And then removing this  element and building the application with cordova build windows -- --archs="x64" will:

Update the platforms\windows\config.xml correctly
Not remove the according element from the platforms\windows\windows.json file
Not remove the according Capability element from the package.*.appxmanifest files

Information

Environment, Platform, Device

Windows 10
Version information

Cordova 8.1.2
Cordova-windows 6.0.1 and 7.0.0
Might be related to #336.
Not sure if this is a bug or if I am expected to readd the platform after making changes to the config.xml file... in case of the latter, feel free to close these two issues...
Checklist

[x] I searched for existing GitHub issues
[ ] I updated all Cordova tooling to most recent version
[x] I included all the necessary information above

Will try to reproduce and report back.

Similar to https://github.com/apache/cordova-windows/issues/336 I can also not reproduce this: https://github.com/apache/cordova-windows/issues/336#issuecomment-508737522
platforms/windows/windows.json has some leftovers of the munges, but that seems harmless:
    "files": {
      "package.phone.appxmanifest": {
        "parents": {
          "/Package/Capabilities": []
        }
      },
      "package.windows.appxmanifest": {
        "parents": {
          "/Package/Capabilities": []
        }
      },
      "package.windows10.appxmanifest": {
        "parents": {
          "/Package/Capabilities": []
        }
      }
    }

(Again, using Cordova CLI 9 for testing.)

Ping @dominic-simplan

We are archiving this repository following Apache Cordova's Deprecation Policy. We will not continue to work on this repository. Therefore all issues and pull requests are being closed. Thanks for your contribution.


bipedism, dinarzade

This pull request was generated by the 'mq' tool
[test]
flake rate: 0.4
logical conflict every: 100
sleep for: 300s
[pullrequest]
requests per hour: 30
deps=[b,d]

/trunk merge


Large csv input used with rmlmapper

Hi! Im trying to use rmlmapper over my csv files (1.400.000 rows), but it doesnt give any output because it keeps thinking. Is there another rmlToRdf conversor which could make that job?
Thank you!

RML streamer gives me an error, and I dont know how to solve it. I leave the error here!
Exception in thread "main" java.util.NoSuchElementException: None.get
at scala.None$.get(Option.scala:347)
at scala.None$.get(Option.scala:345)
at io.rml.framework.Main$.main(Main.scala:124)
at io.rml.framework.Main.main(Main.scala)

Could you provide the mapping file and cli arguments you're using to try
the rmlstreamer? It should be able to handle this amount of data :)
Op ma 15 mei 2023 19:48 schreef titoarrabal @.***>:

RML streamer gives me an error, and I dont know how to solve it. I leave
the error here!
Exception in thread "main" java.util.NoSuchElementException: None.get
at scala.None$.get(Option.scala:347)
at scala.None$.get(Option.scala:345)
at io.rml.framework.Main$.main(Main.scala:124)
at io.rml.framework.Main.main(Main.scala)
—
Reply to this email directly, view it on GitHub
https://github.com/RMLio/rmlmapper-java/issues/203#issuecomment-1548286355,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFDHJNAFNYBI7DXS7KZOFTXGJT63ANCNFSM6AAAAAAYCQYI7A
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Meteorologicos.zip
I am using yarrrml-paser to obtain map1.rml.ttl. Then, i use this command:
java -jar rmlstreamer.jar toFile --mapping-file C:/Users/Usuario/Desktop/TFG/Mapping/ComunidadMadrid/Meteorologicos/map1.rml.ttl --output-path  prueba.nt
It gives me this error

@ghsnd ?

Hi! Did you try the standalone version of RMLStreamer? It looks like this is the version that has to be deployed on a Flink cluster.

No response for a while, re-open if needed


Swift fails to correctly select between async and regular overload

Description
I have two functions; one that takes an async closure and one that doesn't. Then they invoke it, so the first is async itself and the second is not. I would expect that passing in an async closure or not would allow the compiler to select between them, but this does not seem the case.
Steps to reproduce
func foo(_ expression: @autoclosure () -> Int) {
	_ = expression()
}

func foo(_ expression: @autoclosure () async -> Int) async {
	_ = await expression()
}

foo(1)
await foo(await 1)

Expected behavior
This should compile without issues

Environment

Swift compiler version info swift-driver version: 1.87.3 Apple Swift version 5.9.2 (swiftlang-<IP_ADDRESS>.51 clang-15<IP_ADDRESS>.2)
Target: arm64-apple-macosx14.0
Xcode version info
Xcode 15.1
Build version 15C5042i
Deployment target: macOS 14.2

Additional context
$ swiftc test.swift
test.swift:11:11: error: 'await' in an autoclosure that does not support concurrency
await foo(await 1)
          ^
test.swift:11:1: warning: no 'async' operations occur within 'await' expression
await foo(await 1)
^
error: fatalError

As far as my problems with this bug go, it's as simple as, "synchronous overloads are not available in an async context."
func ƒ() async {
  func foo() { }
  func foo() async { }

  await foo()
  foo() // ❌ Expression is 'async' but is not marked with 'await'
}


[Epic/META] Ability for managers to offer an honorarium

[ ] Needs issue template filled @Erioldoesdesign

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Is your feature request related to an existing functionality? Please describe.
A clear and concise description of what or where the existing page or function is. Include URLs.
Describe the solution you'd like
A clear and concise description of what you want to happen. Include ideal URLs, if they don't already exist.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Priority

Low priority (annoying, would be nice to not see)

Affected roles

Staff
Applicants

Ideal deadline
Recommend review in Q2 rebasing for priority level.

Below are a historic list of functions, capabilities, ideas and acceptance criteria for the over 'Epic/META' of this feature.
Admin/manager

[ ] Option to enable somewhere in admin/settings or env var
[ ] Creates a new apply > menu item
[ ] List of existing providers
[ ] Ability to add/edit provider (Name, URL, help URL, status [enabled/disabled])
[ ] For each provider, the ability to add/load tokens (upload a csv file)
[ ] Ability to see information about each provider (tokens available, tokens issued, date added, status)
[ ] Option to select which "Screening" statues receive an honorarium automatically
[ ] A threshold setting for warning when tokens are low

Application editors

[ ] New "Honorarium" form option is available to be included for all applications, the applicant has the option to choose any one of the enabled providers

Applicant

[ ] Ability to choose an honorarium from the list of enabled providers when applying (see https://apply.reset.tech/reset-our-future-fund/)
[ ] On first determination, the applicant receives a new email letting them know they have an honoraria to claim
[ ] On the application page, a new box "Honorarium", listing their chosen provider with the option to change provider or claim their honorarium.
[ ] Change, presents an option for them to chose from any enabled provider
[ ] Claim, presents them a token, URL, and help URL, a confirm
[ ] When confirmed, they no longer see the option change but have the option to view their confirmed selection again

Backend considerations

[ ] Mapping confirmed token to the applicant, no double issuing

Ahh, gonna refine this to make use of https://docs.voucherify.io/reference

Q1 2022 Rebasing updates:

Removed from milestones 'backlog'https://github.com/HyphaApp/hypha/milestones/Backlog
Added the issue template to body without content yet and assigned Eriol to issue to fill at a later date.
This is not a single issue, it's an Epic/META. - PM needed to build out Epic/META details
Converted to an EPIC in Zenhub extension.
Size estimated at XL
Removed priority low label
Recommend review in Q2 rebasing for priority level.


Support for One to Many API using Ent and ogent

Team,
Thanks for creating this framework.
Trying to create  one to many relationship API to be  auto generated from ogent.
Didn't find any docs for this as such requested here.
The example is taken from gorm project to explain the scenario
We have one order struct and item struct . Order to Item it is having One to Many relation.
The details are given below:
type Order struct {
// gorm.Model
OrderID      uint      json:"orderId" gorm:"primary_key"
CustomerName string    json:"customerName"
OrderedAt    time.Time json:"orderedAt"
Items        []Item    json:"items" gorm:"foreignkey:OrderID"
}
type Item struct {
// gorm.Model
LineItemID  uint   json:"lineItemId" gorm:"primary_key"
ItemCode    string json:"itemCode"
Description string json:"description"
Quantity    uint   json:"quantity"
OrderID     uint   json:"-"
}
The expected payload for the API in such case will be as follows:
'{"orderedAt":"2019-11-09T21:21:46+00:00",
"customerName":"Tom Jerry",
"items":[{"itemCode":"123","description":"IPhone 10X","quantity":1}]}
How do I achieve using ent and if API to be autogenerated through ogent

Hey @madugula007, first of all thank you for the kind words.
Setting up a One-to-Many relation with Ent, is really straight forward and is covered very well in our docs.
Both entoas/ogent are extensions to Ent, you can find a good tutorial in the announcing blog post.
However, the API you are describing does not exist in ogent. The call you have there creates both an entity alongside its children and that is not covered by the autogenerated CRUD. You'd have to set up that endpoint by yourself.
Closing, but don't hesitate to re-open if things are not clear. Note, that our community in Discord is quite active and we have many users who will answer any type of question that might arise. Feel invited to join!


IPTT > Sector filter: Update sector filter to reflect new options (WORK IN PROGRESS)

Note: Make sure we only show sectors actually used by the program.

Looks like this one was fixed.


NZBGet: ping only works with FormAuth true

Environment
Docker
Version
0.10.4
Describe the problem
When using the nzb module. The ping fails if you are using the default auth setting with nzbget. Users have to use to set FormAuth: Yes.

Can we add support for the basic auth on ping?
If not, I can open a PR tp document that users must use FormAuth yes for ping to work correctly.

Additional info
No response
Please tick the boxes

[X] You've read the docs
[X] You've checked for duplicate issues
[X] You've tried to debug yourself

Hi @codezninja 👋,
thank you for your issue.

Version 0.10.4 is quite old. We usually don't provide any support for outdated versions and we recommend you to upgrade to 0.10.7.
The ping module actually has quite a lot of issues sometimes. If you would like to implement this, you would have to tell us how we need to send the credentials to NZBGet.
If you would like mention this in the documentation, you can create a pull request here. Make sure to use the 0.11 branch as a base, as we recently made many changes!

Thanks for the response. Love the product btw. Response to the first two bullet points

I have updated my image to the latest version 0.10.7 and confirmed the same behavior
So the current workaround I found is that in the Service Url if you add your basic auth information ping will work correctly. Looking through the code base the library being used supports that under the hood. So instead of http://localhost:6789 users can add http://{username}:{password}@localhost:6789 so my proposal might be that we add a checkbox saying use the username and password field for auth.

 url = app.url
 if(app.use_auth_for_ping):
   url =<EMAIL_ADDRESS>
  const response = await fetch(`/api/modules/ping?url=${encodeURI(url)}`);

I'll open a PR in the meantime to document the current workaround for nzbget

Thanks for the response. Love the product btw. Response to the first two bullet points

I have updated my image to the latest version 0.10.7 and confirmed the same behavior
So the current workaround I found is that in the Service Url if you add your basic auth information ping will work correctly. Looking through the code base the library being used supports that under the hood. So instead of http://localhost:6789 users can add http://{username}:{password}@localhost:6789 so my proposal is that we add a checkbox saying use the username and password field for auth.

 url = app.url
 if(app.use_auth_for_ping):
   url =<EMAIL_ADDRESS>
  const response = await fetch(`/api/modules/ping?url=${encodeURI(url)}`);

I'll open a PR in the meantime to document the current workaround for nzbget

I think this is an extremely edge case and it doesn't need to be implemented in the code.
If setting the internal address to http://{username}:{password}@localhost:6789 works, let's just add a mention about it in the docs. Adding a whole new set of conditional options isn't worth it for the number of people who will encounter the same issue. If you want to open a PR please do so on the ajnart/homarr-docs repository 😉 (you can browse the docs for a nice place to put it and press the "edit page" at the bottom)

I think this is an extremely edge case and it doesn't need to be implemented in the code.

I agree. Feel free to document this though

https://github.com/ajnart/homarr/issues/559#issue-1527939422
Thank you so much for this! I was going crazy why i was getting a 401 from nzbget


Turn on -Wc++20-compat

This will help us make sure to avoid incompatibilities for C++20.

Repurposing this to just say we need to migrate to C++20, since apparently we can do that now.
Will need to work on Dart as well.


Display latest statistics first and some UX/UI suggestions

Hi,
First of all, thank-you for putting that statistics page up for all of us Montrealers. Merci d'avoir mis en ligne cette page d'information et de statistiques pour nous-autres montréalais.
However, I believe the default display of data on the animated cases map and age-group barchart is misleading as, instead of showing the latest statistics, the earliest data from the datasets is displayed instead of the latest.
To prevent unintentionally misleading people, and to match the rest of the page already displaying the latest data, I believe the default vue should be of the latest data until a user clicks/taps on the play button.
As to other suggestions:

Perhaps adding more emphasis on the current date displayed during playback might help better/faster reading
Main numbers should be put in bold (or at least bolder)
The map automatically grabs the mouse preventing page scrolling: perhaps doing it the Google Maps-way and blocking zoom on the map display unless CTRL key is pressed could enhance usability
On mobile, the same thing happens when I tap inside a chart or map while wanting to scroll, perhaps a tap-and-release should be expected before engaging with these views, so as to not block scrolling.
On mobile, I am unable to properly interact with the map – can't zoom in or out, and the map itself is stuck in a small square inside the map view.

I would attempt doing the changes myself, but as I have stayed away from web technologies and git for 5 years now and am only getting back into it now, I'll limit myself to suggestions.
P.S.: This project may attract many contributors and people who may want to fork it to adapt it to their cities; it may therefore be beneficial to introduce some contribution guidelines.

Thanks for the comments and suggestions :)
I agree that it would be better for the choropleth map and animated bar plots to show the latest date by default. I actually tried to implement this once, but while I managed to change the default value on the slider, this didn't update the map values until the slider was moved. I don't think there's currently a way to set a default frame in the plotly express API (https://plotly.github.io/plotly.py-docs/generated/plotly.express.choropleth.html), but it might be possible to do something with the lower-level Graph Objects API. I'll take another look into this.

Perhaps adding more emphasis on the current date displayed during playback might help better/faster reading

I could try making the current date bold. Not the highest priority on the list of things to do right now, though.

Main numbers should be put in bold (or at least bolder)

I just tried switching them to bold, but not sure I like how it looks (subjective, I know). Maybe a medium-weight font would look better, but then I'd need to add a web font that supports more weights. Won't change for now. Thousands separators would also be nice.

The map automatically grabs the mouse preventing page scrolling: perhaps doing it the Google Maps-way and blocking zoom on the map display unless CTRL key is pressed could enhance usability
On mobile, the same thing happens when I tap inside a chart or map while wanting to scroll, perhaps a tap-and-release should be expected before engaging with these views, so as to not block scrolling.

I agree that the zooming behaviour is problematic, particularly on mobile. Plotly, however, currently only gives the option to either enable or disable zooming, so that's all I can do for now. It's a compromise, but I've opted to disable zooming since it makes the experience better on mobile (and regular pinch to zoom still works). On desktop, it's still possible to zoom using the +/- buttons in the modebar. Note that for the map there appears to be a bug that prevents the +/- buttons from working unless the user first pans the map (will need to wait for upstream fix). This does not affect the other figures.

On mobile, I am unable to properly interact with the map – can't zoom in or out, and the map itself is stuck in a small square inside the map view.

Yes, the map is very suboptimal on mobile currently (and used to be even worse, see #5). My options in styling the map differently on mobile are however limited because the entire map view is rendered as an SVG. I've tried making the "bounding box" of the map view resize dynamically to fill all the available space, but haven't figured out how to get that working yet. I might investigate using plotly.express.choropleth_mapbox instead of plotly.express.choropleth to see if that offers a better experience on mobile.
In regards to contribution guidelines, I'll add them if/when there are more active contributors.

I switched the map to use tile-based instead of outline-based maps (see b94f92716613075aa2564540ff7723b1d17283d0) and it works much better on mobile now! I re-enabled zoom for the map, however, since I can't set different default zoom levels on desktop and mobile (the default on mobile is a bit too zoomed in).

Changed choropleth map and age plot to display the latest date by default in 5f54d01c31159095c192105203b7c8a4849dab20, thanks to solution here: https://community.plotly.com/t/defining-default-starting-frame-with-plotly-express-animation/38636
Closing this issue for now.


dynamic-import-chunkname doesn't work with latest babel-eslint

That's because it traverses CallExpressions while import() is now an ImportExpression:
https://github.com/babel/babel/pull/10828

We’ll need to be sure to support both.

It seems like there's the same issue with @typescript-eslint/parser. I guess that's for the same reason.


Proposal: Remove multi-way if

Proposed Change
Remove the multi-way if syntax.
Currently the Elm Syntax docs include this example:
if | key == 40 -> n+1
   | key == 38 -> n-1
   | otherwise -> n

This would no longer work. Instead, this example would be rewritten to the following:
if key == 40 then
    n + 1
else if key == 38 then
    n - 1
else
    n

This would be a breaking compiler change.
Motivation
The multi-way if syntax has these problems:

It can currently lead to non-exhaustive matches.
It adds complexity to the compiler and learning curve to new users, in exchange for conciseness but no additional capabilities.
Expanding from a normal if/else to a multi-way if requires rewriting the entire expression, and failing to do so when a third case arises leads to stylistic inconsistencies across the code base.
It is a rarely-used feature in a language with pattern matching.

I thought I wouldn't like chaining 2-way ifs but now that I see it, it looks really good. We go from a hard-to-remember, little-used syntax, to one that extends so naturally from the 2-way if that it already works. Implementation should be almost entirely removal. +1

I was actually thinking in the other direction, remove the two-way one. That opens more syntax options to improve the multi-way one. I guess I'll open a counter-proposal, if I can find the time for it.
My thoughts on your motivation:

That's just a bug that needs fixing.
Conciseness doesn't have to add to the learning curve of new users. Elm syntax is alien for most people anyway, so removing this won't really help there.
Yes, but removing the two-way if will result in the same effect.
Debatable. Some code is heavy on the union types and therefore pattern matching. This may be an argument for eliminating one of the options since having two ways to write an if is silly, but not for removing the multi-way if in particular.

What I conclude is that those arguments that hold are not specific to removing the multi-way if but more to removing one of the two if options.

Conciseness doesn't have to add to the learning curve of new users. Elm syntax is alien for most people anyway, so removing this won't really help there.

The way I look at it is that Elm syntax being alien is a generally negative thing, and changes that make it less alien are generally positive things unless they sacrifice important language characteristics.
People are used to if / else if / else from a multitude of mainstream industry languages, so there's nothing really alien there. The pipe syntax exists in no mainstream industry languages, which makes it feel alien to many.
There's a cost to that, and whatever benefits it brings don't seem to outweigh the cost.

The thing I like about the two-way syntax is that it's impossible to have an non-exhaustive analysis. I don't think a multi-way construct could make that guarantee. Using the chained form, it's now impossible in the multi-way as well. This is cited as a benefit in the proposal, but it does mean you may have some cases where you know the pattern match is exhaustive but the conditions don't line up. I suppose this is code smell, so it's good to force people away from it. One fewer runtime error.
Also, would we drop otherwise from Basics?

@rtfeldman The pipes is something I was going to suggest removing once the two-way if is eliminated, since the multi-way if wouldn't be ambiguous with it any more then.
@mgold If you change the multi-way if to enforce the otherwise branch, you get basically the same.
By the way: the case where you see that the last else branch can't happen, you can add a Debug.crash. Though I'd prefer to have a different function, perhaps a value unreachable : a in Basics that prints file, line and column when it gets hit. That's better documentation than doing that by hand in the Debug.crash string.

Good catch! I assume so. There's several low hanging issues in Core. Anyone can PR if it's overlooked.

@Apanatshka I'd like to see that added behavior added to Debug.crash but I don't think we should have  a similar function in Basics. It seems contrary to the non-runtime-errors thing, and like I said this is probably code smell anyway.
@dnalot You're referring to removing otherwise, right? We can do that PR once the compiler one is ready. It seems the only use of multi-way if in core is in Color.elm, so the same PR can fix that.

I agree with @Apanatshka on all points.
I think the multi-way if is much easier to read and write, and I do use it even though I don't write too much Elm code. IMO that both guards/multi-way-ifs and pattern matching are important and serves different causes.

The reason I initially brought this idea up with @rtfeldman was that I have been trying to get to a uniform indentation style for Elm. This is particularly problematic for the multi-way if:
if  | x < 10 ->
      x + 100
    | x > 20 ->
      x + 200
    | otherwise ->
      x

Needing the two spaces after the if seems kind of sucky. After that I'd say point 1 and 3 are the ones that matter to me most.
Anyway, just wanted to say: show code examples!!! If you think one way is better, show us a specific example!

Also, if folks are interested in guarded patterns in case, multi-way if does not address that. I think that can be useful at times, but it seems like a separate issue to me.

Ok, I wrote a counter-proposal in #6. It surprised me that the syntax I came up with is very close to normal nested ifs, so it could be seen as removing multi-way if and adding some syntactic sugar...

I was thinking of using indentation to avoid then:
if a <= 0
    Nothing
else if a == 1
    Just x
else
    nthElement xs (a-1)

But then I realised that is not the idiomatic way for functional programmers (Haskell, F#, Clojure), see [F# control-flow-expressions] (http://fsharpforfunandprofit.com/posts/control-flow-expressions/).
Current Elm's conditionals are really great, because of the similar syntax for if and case of, and the fact we can drop support for Haskell guards (the preferred way in Haskell, see [guards-vs-if-then-else-vs-cases-in-haskell] (http://stackoverflow.com/questions/9345589/guards-vs-if-then-else-vs-cases-in-haskell)) without impact in the elegance.
I think the issue "It can currently lead to non-exhaustive matches" could be addressed with a warning. So for me, the relevant issue is "Expanding from a normal if/else to a multi-way if requires rewriting the entire expression, and failing to do so when a third case arises leads to stylistic inconsistencies across the code base". For that case, the normal if/else could better change to:
if | key == 40 -> n+1 | else -> n
The arrow is similar in the use of case of so I think it is idiomatic. The pipe is not alien, it is used normally in Haskell guards and in the |> <| operators.
New conditions doesn't require rewriting the entire expression:
if | key == 40 -> n+1
   | key == 38 -> n-1
   | else       -> n

Look how similar is this example compared to the haskell's normal guard showed in the last link:
if | a <= 0 -> Nothing
   | a == 1 -> Just x
   | else   -> nthElement xs (a-1)

Not sure how I feel about it, but a possible option could be to make if be
a synthetic function of type forall a. Bool -> a -> Lazy a -> Lazy a.
With the following example use case:
ex0 x =
  if (x > 0)
    "pos" <|
  if (x < 0)
    "neg" <|
  else
    "zero"

ex1 x = if (x == 0) "zero" "non-zero"

A working (non-lazy) implementation:
iF : Bool -> a -> a -> a
iF b l r = if b then l else r

eLSE : a -> a
eLSE = identity

ex0 x =
  iF (x > 0)
    "pos" <|
  iF (x < 0)
    "neg" <|
  eLSE
    "zero"

ex1 x = iF (x == 0) "zero" "non-zero"

@ryepesg guards on pattern matches are quite different when you involve more patterns, it's not always as easy to translate between that and Elm's multi-way if.
We're not optimising the syntax for Haskell users, the common user we expect and try to cater is more from the front-end web world of HTML and JavaScript.
@LiamGoodacre I like the idea of going back to basics and use functions, because this is usually so powerful. But in this case I'm hesitant because of the syntactic overhead of writing lazy functions, and the cognitive overhead of explaining laziness when all a beginner wants is a simple if.

I was reminded about another issue. If we allow expressions in record update syntax, multi-way ifs are ambiguous. Perhaps this is bad enough to make it impossible. { if | x == 1-> point | x <- 4 }

Ouch...

Yeah, that's actually a part of why Elm does not allow arbitrary expressions in that position. I didn't know how to deal with this case (and others), so I decided to answer the question later.

Then I think we should drop multi-way if and see if we can rework record update syntax, possibly as part of removing add and remove, to accept arbitrary expressions.

+1 to everything @mgold said!

I'm a web technology adult educator with an interest in introducing Elm into upcoming curriculum (because it's amazing and solves a big problem on the web). As I've been learning Elm myself, I keep thinking "how would I teach this?" The most common reason why students falter and give up is not because of one confusing moment, it's because of a "death by papercuts" series of small misunderstandings.
I think the else if is the easiest one explain to a new person. If a goal of Elm is to be inclusive of Ruby and JS developers (which is what I've inferred) then I think it's pretty important to err on the side of consistency with those languages over Haskell, especially for something as basic as a branching if.
As a non-Haskell knowing person, Elm seems to have a bit of "operator-itis". It's particularly confusing when operators have different meanings depending on context (like |, ->). I'm a fan of removing the multi-way if syntax and falling back on else if, because 1) it's clear what it's doing, and 2) that's one less single-use edge case for |.
As for white space importance in an if statement, I think the "sometimes white space matters, sometimes it doesn't" part of Elm is also one of the more confusing aspects. Internal consistency is really important for learning—people look for patterns relatable to what they already know when learning something new. Any changes that reduce those inconsistent edge cases through (simple) explicit language constructs is a big win for adoption in my opinion.
Here's what I'm proposing in case that wasn't clear:
if key == 40 then
    n + 1
else if key == 38 then
    n - 1
else
    n

-- or what I would expect is more "elm like"
if key == 40
then n + 1
else if key == 38
then n - 1
else n

@bbugh Thanks for your perspective! The first style you mentioned (with then at the end of the line instead of at the start) is what I've been doing, and I'm definitely a fan.

@bbugh, thanks for you perspective! The style you recommend was the one I initially showed to @rtfeldman that initiated this issue. That style is what I was planning to recommend if this feature was removed for pretty much exactly the reasons you outline.
Also, the one you theorized is "more elm like", not in my code I hope, that looks awful to me! :)

For reference, I think we'll do a bit of syntax removal in 0.16 for this and for record extension and deletion. That's what I have in my mind at least.

As for style, I like the first way as well. I've been known to put if, then, and else on three lines but only for single way if.
You can also take the opposite perspective: if we only had the single way syntax but it worked just fine for multiway, would we feel the need to add syntax for the multi way? I doubt it. And the thing I really like about this proposal is that the single way already works for multiway.
Although I might be beating a dead horse given that Evan seems to have tentatively accepted this for 0.16.

Since this is planned for 0.16, perhaps it's time to close this issue and mark it as accepted?

Closing down this repo. This is something I plan to do.


Docker pull error (ghcr.io/abbas-gheydi/radotp:main)

Hi, I am receiving an error when trying to pull the image.
Please advise.
Pulling radotp (ghcr.io/abbas-gheydi/radotp:main) . . .
edaedc954fb5: Pulling fs layer
fcfb138e1122: Pulling fs layer
79183bbdbf3e: Pulling fs layer
8c84ec5d4ada: Waiting
8b6e0f48469d: Waiting
4f4fb700ef54: Waiting
ERROR: error pulling image configuration: download failed after attempts=1: unknown blob

Issue resolved on its own.


String interpolation handler for WorkbookXml

Codecov Report
Attention: Patch coverage is 80.30303% with 13 lines in your changes are missing coverage. Please review.

Project coverage is 87.86%. Comparing base (4e4a728) to head (55476b8).

Files
Patch %
Lines

SpreadCheetah/MetadataXml/WorkbookXml.cs
77.77%
0 Missing and 6 partials :warning:

...llValueWriters/Number/NumberCellValueWriterBase.cs
50.00%
2 Missing :warning:

SpreadCheetah/SpreadsheetBuffer.cs
71.42%
1 Missing and 1 partial :warning:

...alueWriters/Boolean/FalseBooleanCellValueWriter.cs
50.00%
0 Missing and 1 partial :warning:

...ValueWriters/Boolean/TrueBooleanCellValueWriter.cs
50.00%
0 Missing and 1 partial :warning:

...eadCheetah/CellValueWriters/NullValueWriterBase.cs
66.66%
0 Missing and 1 partial :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #46      +/-   ##
==========================================
+ Coverage   87.84%   87.86%   +0.02%
==========================================
  Files         139      139
  Lines        4672     4689      +17
  Branches      899      891       -8
==========================================
+ Hits         4104     4120      +16
- Misses        253      254       +1
  Partials      315      315

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


Sort accented characters appropiately

When having links that start with an accented character, e.g., á, è, they appear at the bottom of the list instead of being at the same level as their non-accented letters.

Does Obsidian get this correct? I may need to request the addition of a locale-aware string compare to the API, so I can use that. I think JavaScript's default string comparison is dumb and doesn't do well for non-English characters.

I've updated string comparisons to be locale aware; I'm not super confident in how good JS locale-aware string handling is, but at the very least I do see better behavior for accented characters like á.

String comparisons are now locale aware - hopefully this is a moderate improvement.


Error: ran out of query data

i suffer from the question about when i use resperf to test DNS performmance.
there is the console:
$ resperf -d datafile.txt -s xxx
DNS Resolution Performance Testing Tool
Version 2.2.1
[Status] Command line: resperf -d a.txt -s xxx
[Status] Sending
Error: ran out of query data
there is my datafile:
dns1.ikang.com.      A
s-static.ak.fbcdn.net.  A
lachicabionica.com.     A
www.mediafire.com.      A
can you help me, thanks！

Use -l or -n also, read about all options in man dnsperf.


ENH: Descending order of x in PPoly

Hi!
Currently PPoly requires x to be in ascending order, which is typically sufficient. But if you computed coefficients for breakpoints in descending order (say integrated ODE back it time), then it will be awkward  to recompute coefficients to the required format (it's not very difficult, but not obvious).
On the other hand, ascending and descending orders are not different at all from a mathematical point of view. So it required only few modifications to support both.
@ev-br @pv please take a look.

@pv, appreciate that you checked.
I restored right=None and added warnings. I used simple warnings.warn, because DeprecationWarning is not issued by the default.
Also I didn't keep the previous substantial description, just said that it is deprecated and not used.
Right decisions?

Current coverage is 78.44% (diff: 42.30%)

Merging #6315 into master will decrease coverage by 0.35%

@@             master      #6315   diff @@
==========================================
  Files           244        242     -2
  Lines         45661      45469   -192
  Methods           0          0
  Messages          0          0
  Branches       8505       8508     +3
==========================================
- Hits          35982      35670   -312
- Misses         7091       7155    +64
- Partials       2588       2644    +56

Powered by Codecov. Last update b4aaee3...20dfb0c

All fixed now. Tests are passing, but one configuration on travis failed to wget something.

It looks like all of gnu.org is down.


#2803 Support custom script for chart widget on dashboard

Description
There are currently needs for limited event control or some chart reshaping in chart widgets placed on structured dashboards. It is an external alarm transmission when used as a dashboard, especially for monitoring purposes. It may be necessary to add event display control in the chart.
Related Issue : #2803

How Has This Been Tested?

Create a chart on the dashboard.
In the view screen, click the Event Trigger button at the top to enter the function.
     example )

function main(event) {
  if( event.name === 'InitWidgetEvent' ) {
     event.data.series[0].markPoint = {
                data : [
                    {type : 'max', name: '최대값'},
                    {type : 'min', name: '최소값'}
                ]
            };
     event.data.series[0].markLine = {
                data : [
                    {type : 'average', name: '평균값'}
                ]
            };
     return event.data;
  } else if( event.name === 'SelectionEvent' ) {
    switch( event.data.charAt(0).toLowerCase() ) {
        case "a" :
        case "b" :
            alert( event.data );
            return true;
        case "c" :
            window.open( 'http://www.naver.com' );
            return true;
        default :
            return false;
    }
  } else if( event.name === 'SelectionFilterEvent' ) {
      console.info( event.data );
      return event.data.some( item => {
          return 'Category' === item.field;
      });
  }
}

Check if it works as you entered.

Need additional checks?
Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have read the CONTRIBUTING document.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have added tests to cover my changes.

Additional Context

deploy to 1

@eltriny I want to implement the function only in the edit mode (except the read mode) and also add animation when the pop opens.

@AnnieHwang
There was no discussion about saving, so I worked on the view mode
Animation effects took time to work on, so I added the priority feature first.

deploy to 4

@eltriny

When it is black theme, we want to be the same design as white theme. The screen in the current theme moves when mouse hovering.

When a popup will be shown, the mouse cursor will be in the input box.

@eltriny
I added it as a custom script in the example.
I clicked on the max value, but Central was selected, not West.
And no values were selected for other charts.

deploy to 4


BUG #2762

BUG Fix for Building v5.2.1 fails on itkDCMTKImageIO.cxx #2762
Add missing includes to Modules/IO/DCMTK/wrapping/itkDCMTKImageIO.wrap:
itk_wrap_include("itkDCMTKImageIOFactory.h")
itk_wrap_include("itkDCMTKSeriesFileNames.h")

The commit message needs to be changed. Perhaps COMP: Fix Python wrapping of DCMTKImageIO. Closes #2762.

The commit message needs to be changed. Perhaps COMP: Fix Python wrapping of DCMTKImageIO. Closes #2762.

@dzenanz Done.


The connection to jolokia failed!

I am using fabric8-maven-plugin:4.2.0 and fabric8/s2i-java:2.3 image, also vertx-maven-plugin as generator, and i'm trying to open java console in openshift but cannot connect
Here's my jolokia.properties:
host=*
port=8778
discoveryEnabled=false
user=jolokia
password=jolokia
useSslClientAuthentication=true
extraClientCheck=true
protocol=https
caCert=/var/run/secrets/kubernetes.io/serviceaccount/ca.crt
clientPrincipal=cn=system:master-proxy

Also i set AB_JOLOKIA_PASSWORD_RANDOM = false
But i get authentication request, tried with the jolokia user and password, openshift user, even with admin but couldn't connect

Could you provide more info on your OpenShift cluster?
There is a known issue related to the Jolokia endpoint authentication that affects OpenShift 4, but also Minishift 3.11.
To work-around the issue, you can set useSslClientAuthentication to false. Otherwise, the new Hawtio console has to be used and configured to use a client certificate as documented here: https://github.com/hawtio/hawtio-online#openshift-4.

I noticed about, i'm running minishift 3.11, after googling it more and more i disabled useSslClientAuthentication and worked well, thanks


[Issue]: Failed to download subtitles

Confirmations

[X] I have checked the issues tab, and couldn't find an existing open issue for the issue I want to report.

OS Type
Windows
Python Version
3.12.2
Package Version
2.5.4
Description
Hi,
I'm facing an issue since version 2.5.3 (the same for 2.5.4) and I can't download my subtitles anymore. It just says "Failed to download xxxx", where xxxx is the sub's language. It seems to be a connection (timeout) issue so I tried a lot of things (DNS change, VPN, etc), but it didn't help... Maybe my connection is too slow but I don't know.
If I rollback iSubRip to version 2.5.2 on my computer, it works again !
I know this issue is weird because 2.5.4 or 2.5.3 works fine for everyone, but not for me :)
Can you help me ?
Output Log
[2024-05-17 16:24:29 | DEBUG | MainThread | __main__.py::download_subtitles::380] Debug information:
Traceback (most recent call last):
  File "......................AppData\Local\Programs\Python\Python312\Lib\site-packages\httpx\_transports\default.py", line 69, in map_httpcore_exceptions
    yield
  File "......................AppData\Local\Programs\Python\Python312\Lib\site-packages\httpx\_transports\default.py", line 373, in handle_async_request
    resp = await self._pool.handle_async_request(req)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "......................AppData\Local\Programs\Python\Python312\Lib\site-packages\httpcore\_async\connection_pool.py", line 216, in handle_async_request
    raise exc from None
  File "......................AppData\Local\Programs\Python\Python312\Lib\site-packages\httpcore\_async\connection_pool.py", line 196, in handle_async_request
    response = await connection.handle_async_request(
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "......................AppData\Local\Programs\Python\Python312\Lib\site-packages\httpcore\_async\connection.py", line 99, in handle_async_request
    raise exc
  File "......................AppData\Local\Programs\Python\Python312\Lib\site-packages\httpcore\_async\connection.py", line 76, in handle_async_request
    stream = await self._connect(request)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "......................AppData\Local\Programs\Python\Python312\Lib\site-packages\httpcore\_async\connection.py", line 122, in _connect
    stream = await self._network_backend.connect_tcp(**kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "......................AppData\Local\Programs\Python\Python312\Lib\site-packages\httpcore\_backends\auto.py", line 30, in connect_tcp
    return await self._backend.connect_tcp(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "......................AppData\Local\Programs\Python\Python312\Lib\site-packages\httpcore\_backends\anyio.py", line 114, in connect_tcp
    with map_exceptions(exc_map):
  File "......................AppData\Local\Programs\Python\Python312\Lib\contextlib.py", line 158, in __exit__
    self.gen.throw(value)
  File "......................AppData\Local\Programs\Python\Python312\Lib\site-packages\httpcore\_exceptions.py", line 14, in map_exceptions
    raise to_exc(exc) from exc
httpcore.ConnectTimeout

The above exception was the direct cause of the following exception:

Config
[downloads]
folder = "C:\\subrip"
languages = ["en-US", "fr-FR"]
overwrite-existing = true
zip = false

[subtitles]
convert-to-srt = true
fix-rtl = true

[subtitles.webvtt]
subrip-alignment-conversion = true

Hey @boulderdash09, these type of issues (networking & timeouts) are usually difficult to debug and are related to something with user's configuration, but I'll try my best to help.
Looking at the diffs between v2.4.2 and v2.4.3, I see two changes that might be related:

Replace aiohttp with httpx (https://github.com/MichaelYochpaz/iSubRip/commit/bb0f6c76c3e20ca3243e74e22c628954a9c44d4f)
Add proxy support  (https://github.com/MichaelYochpaz/iSubRip/commit/da5795affc867ebe2a1f16492898195b90bffcf3)

Could you check if you have any of the environment variables httpx supports is set to a value, and if so, what the value is?
You can list your current environment variables by running ls env: on a PowerShell terminal.
I specifically suspect one of HTTP_PROXY, HTTPS_PROXY, and ALL_PROXY might be set to a proxy server that isn't active / working.
Did you perhaps download subtitles for a big amount of movies over a short period, causing a temporary block by the server?
Other than these checks - please attach the full log and not just the error message, as there's a lot of relevant information being logged that can help debug this issue.
Note that you can attach the log file in a new comment by dragging & dropping it into the comment box (should be more convenient). If you just paste in a comment / edit the original post, please make sure to use the following format:
```Text
<LOG GOES HERE>
```

Hi @MichaelYochpaz and thank you for your feedback !
With your explanations, I think I found the problem but I changed the code to fix it.
In the class AsyncScraper (in scraper.py) I changed this :
    self._async_session = httpx.AsyncClient(
        headers={"User-Agent": self._user_agent},
        proxy=(httpx.Proxy(url=self._proxy) if self._proxy else None),
        verify=self._verify_ssl
    )

by this :
    self._async_session = httpx.AsyncClient(
        headers={"User-Agent": self._user_agent},
        proxy=(httpx.Proxy(url=self._proxy) if self._proxy else None),
        verify=self._verify_ssl,
        timeout=10.0
    )

And it works !
So I just added "timeout=10.0", as I read it's 5 by default. I think 5 is a bit short for my computer and/or configuration, I don't really know why however.

@boulderdash09 Nice find, thanks for reporting.
Looks like aiohttp had 300 seconds (5 minutes) timeout by default, which explains the difference.
5 seconds is plenty of time for the simple requests isubrip makes (and seems like it works for most users), as I don't believe it ever fetches more than a few KB in a single request (subtitles on iTunes are split into many very small chunks of something like 10-30 text lines. For most users, even 2 seconds is probably more than enough).
So regardless to isubrip, I'd suggest checking if you have a firewall, a very slow VPN / proxy in-use, or any other network-related software that might cause big delays. Could also just be your ISP being terrible (in which case there isn't really much you can do besides switching).
I'll have to check and consider increasing the default timeout value, as if there is a network issue, the program will wait for the full timeout to pass before failing, so it might seem unresponsive (For example, if you have no internet connection, and there's a 30 seconds timeout, the program will wait 30 seconds for a response with nothing happening, and only then fail with an error), so I can't guarantee I'll update that, I'll have to think about it.
But in any case, adding a config setting to update the timeout seems like a good idea to me, and will allow you to just add something like:
[scrapers]
timeout = 10

to your config to resolve this issue.
Keeping this issue open until a solution is implemented.

Thank you @MichaelYochpaz
I think my VPN is the root problem here after some more tests. However, having a config for the timeout is an excellent idea, it's easy (because the config file doesn't change when a new version is installed) and can solve the issue in just on line. Also useful if someone else has a similar problem.

Closing as timeout configuration options have been added on the latest 2.5.5 version :)
(Check the release-notes / example config for more information).


Snowflake: DESCRIBE TABLE not working

Description
In DBeaver, the DESCRIBE TABLE command does not work with Snowflake.
Steps to reproduce

Use Snowflake browser GUI or Snowsight:
exec DESCRIBE TABLE tablename
result: works fine.
Use DBEAVER
connect to Snowflake database
exec DESCRIBE TABLE tablename
result:

SQL Error [1003] [42000]: SQL compilation error:
syntax error line 7 at position 9 unexpected 'table'.
DBeaver Version
Community Edition v22.1.1
Operating System
Windows 10 Enterprise 21H2 64-Bit0

Hello @JoMrx
Unfortunately, I can't reproduce it.

Can you please provide a screen from your SQL Editor? And your database and driver versions (you can check it in the Connection Settings -> Test Connection). Thanks.

Sorry, was my error. Previous command wasn't terminated while not visible in the window.

I'm glad that this is ok now.


Not able to add a new page to the resume

Hi,
How can I add a new page to the resume? Which file to change and how ?

This is possible by editing index.html and adding another <section class="sheet" data-page-number="2"> element. You might also need to edit the CSS to format the new page appropriately.


python310Packages.goodwe: 0.2.23 -> 0.2.24

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.goodwe is: Python library for connecting to GoodWe inverter
meta.homepage for python310Packages.goodwe is: https://github.com/marcelblijleven/goodwe
meta.changelog for python310Packages.goodwe is: https://github.com/marcelblijleven/goodwe/releases/tag/v0.2.24
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.2.24 with grep in /nix/store/dnr2ir1cqij6rb13c4glhq6ycpc70sv9-python3.10-goodwe-0.2.24
found 0.2.24 in filename of file in /nix/store/dnr2ir1cqij6rb13c4glhq6ycpc70sv9-python3.10-goodwe-0.2.24

Rebuild report (if merged into master) (click to expand)

3 total rebuild path(s)

3 package rebuild(s)

First fifty rebuilds by attrpath
home-assistant-component-tests.goodwe
python310Packages.goodwe
python311Packages.goodwe

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/dnr2ir1cqij6rb13c4glhq6ycpc70sv9-python3.10-goodwe-0.2.24 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.goodwe https://github.com/r-ryantm/nixpkgs/archive/b9a500adce7d21a78d3b463534bba3daf7f25548.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/dnr2ir1cqij6rb13c4glhq6ycpc70sv9-python3.10-goodwe-0.2.24
ls -la /nix/store/dnr2ir1cqij6rb13c4glhq6ycpc70sv9-python3.10-goodwe-0.2.24/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

3 packages built:

home-assistant-component-tests.goodwe
python310Packages.goodwe
python311Packages.goodwe

Maintainer pings
cc @fabaff for testing.

@ofborg build home-assistant-component-tests.goodwe


Clarify a proper WSFC supported solution

As is, this does NOT create a supported solution. Cluster validation is required to be run and pass (even if it is with warnings) to have a supported solution. This article has to reflect that or be pulled until such time as that is supported. Thanks.
Here is the documentation:
https://docs.microsoft.com/en-us/windows-server/failover-clustering/create-failover-cluster#validate-the-configuration
"Microsoft supports a cluster solution only if the complete configuration passes all validation tests and if all hardware is certified for the version of Windows Server that the cluster nodes are running."
Allan

@allanhirt : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @allanhirt , thanks for your feedback! Let me verify... :)
Thanks!
Masha from the SQL Docs team

No worries :) THis also applies to other things as well ... I just happened to stumble on this one.

To be clear, anything with DSC has this issue. I verified this with John Marlin as validation isn't in there.

Hi @allanhirt , sorry, just to make sure we're on the same page, can you please clarify what DSC is?
Thanks!
Masha from the SQL Docs team

Desired State Configuration which is what I believe this does behind the scenes. Even if it isn't, this does affect DSC for SQL Server vis a vie its AG stuff.
Not running validation or ignoring the results of a bad validation test (or at least interpreting one ...) is the equivalent of using -Force in PowerShell for New-Cluster and in the GUI, accepting a dialog which is basically  the same thing.

Hi @allanhirt, just wanted to update you that we're still looking into this :)
Thanks for your patience!
Masha from the SQL Docs team

Hi @allanhirt , thanks again for bringing this to our attention! Instead of adding this note to the doc, we're instead going to add a step to run a manual cluster validation so that the cluster is supported.
I am tracking this work internally  (1698908) so will close out this PR in the meantime. However, please feel free to comment should you have any questions or need anything further from us.
Thanks again for improving our documentation and I hope you have a wonderful rest of your day! Masha from the SQL Docs team
#please-close


Ubuntu support?

Any plans to support this on ubuntu?
Or perhaps a docker container?

Some github users are helping me to migrate it to Linux. So there will hopefully a support in near future.

Could you please link to where the development for that is happening?

Alien can now be build in Linux. The changes including cmake script files have been merged to the developer branch (see https://github.com/chrxh/alien/pull/24).


Image is dimmed when there's a title.

Describe the bug
Image is dimmed when there's a title. I tried a slide with layout: image and it retains its brightness.
To Reproduce
Steps to reproduce the behavior:
Create a slide with a title and image.
---
layout: center
---

# Artificial Neural Networks

![neuron](images/neuron.png)

Desktop (please complete the following information):

OS: Linux 5.15 (Manjaro)
Browser: Opera
Slidev version: 0.28.9

As a workaround, I discovered that adding an element after the title fixes it.
---
layout: center
---

# Artificial Neural Networks

![neuron](images/neuron.png)

Not a bug if image is inside public/images, without  Only used slidev today 😅, sorry for the trouble.

@a-h yeah I should have dug deeper into the underlying code, it's Slidev after all :sweat_smile: At least that's something we can easily override.

I think it's a genuine bug. I can't believe that it's right that the images are dimmed in this case.
This appears to be the culprit: https://github.com/slidevjs/themes/blob/main/packages/theme-default/styles/layouts.css#L3


Add UTC parsing to cumulativeByMonthFromEvents

Addressing https://travis-ci.org/studentinsights/studentinsights/builds/155916314, originally from https://github.com/studentinsights/studentinsights/pull/602.

@erose trying to help fix the test failure, which looks like a real edge case failure.


Trying to reset the user passwords with cloud code. Function works fine windows 7 based server but gives an error on the linux based server.

AQs the title sais, I am trying to reset user passwords using a cloud code function. On my windows 7 based parse server running parse version 2.2.21 the function works fine but on my other server, ubuntu based also running parse version 2.2.21, it gives an error.  Both are running Node version 4.4.3.
Here is the function I am attempting to call:
`Parse.Cloud.define("assignPasswordToUser", function(req, res){
Parse.Cloud.useMasterKey();
var query = new Parse.Query(Parse.User);
query.equalTo("username", req.params.username);

query.first({
    success: function(theUser){
        var newPassword = req.params.password;
        console.log("New Password: " + newPassword);

        console.log("set: " + theUser.set("password", newPassword));
        console.log("setPassword: " + theUser.setPassword(newPassword));

        theUser.save(null,{
            success: function(theUser){
                // The user was saved correctly
                res.success(1);

            },
            error: function(SMLogin, error){
                res.error("No se pudo guardar la contraseña");
            }
        });
    },
    error: function(error){
        res.error("error and stuff" + error);
    }
});

});`
Here is the error I am geting:
error: Failed running cloud function assignPasswordToUser for user LODC5xe5LS with:   Input: {"username":"Administrator","password":"apple"}   Error: {"code":141,"message":"error and stuff[object Object]"} functionName=assignPasswordToUser, code=141, message=error and stuff[object Object], username=Administrator, password=apple, user=LODC5xe5LS error: Error generating response. ParseError { code: 141, message: 'error and stuff[object Object]' } code=141, message=error and stuff[object Object] [object Object]
And here is me c# code for calling the function with RESTsharp just incase it proves usefull:
`var request = new RestRequest(Method.POST);
        request.Resource = "/functions/assignPasswordToUser";

        request.AddParameter("Username", "NotARealUserName");
        request.AddParameter("password","NotARealPassword");
        IRestResponse response = client.Execute(request);`

Hi, this question belongs more on StackOverflow or the likes but the problem is most likely due to you not sending the users sessionToken or not using the masterKey when saving the user.
On another note, this is really insecure, don't do this. Anyone can set anyones password like this.

I made a post on stack overflow wich has yielded no responses. I don't see how it could be the sessionToken as this EXACT code runs on one server but not the other.  I am using Master key. It has been suggested it could be that my code is not formatted correctly for the new cloud code syntax, but I can't find any documentation for the new documentation. I'm completely at a loss here and i need to fix this soon. Any help would be super helpful.

You are not using the masterKey correctly.
https://github.com/ParsePlatform/Parse-Server/wiki/Compatibility-with-Hosted-Parse#master-key-must-be-passed-explicitly
It's still also a very insecure way of setting a user's password since anyone can change any users password.


WaitForAngular and WaitForjQuery support

Added browserFactory configuration attribute waitForPendingAjaxCallsVia.  Recognized values are angular and jQuery (case-insensitive).  If set, will implicitly wait until pending Ajax calls are complete after every step.

Added Gherkin steps to wait for angular or jQuery, as an alternative to implicitly waiting after every step via config file setting.

Removed experimental application configuration attribute ActionRetryLimit.  Wasn't helpful.

Added application configuration attribute RetryValidationUntilTimeout (defaults to false).  Helps in odd timing cases where browser seems to be updating the DOM.  Helps in cases where you are checking for a particular number of items in a list, or that list number X has certain values.  Added because it seems to be a race condition between the test and the browser's foreground thread -- i.e. outside of pending Ajax calls.

Added diagnostic output of the Gherkin to the console when running in debug mode.  Includes messages where it's waiting or retrying.

Cleaned up issues with Dispose() logic in BrowserBase and SeleniumBrowser that need to dispose the driver.  Also removed the "true" flag (to dispose) when registering the browser with the IoC container, as it interfered with the reuseBrowser option.  The corrected Dispose() logic should remove the need for that.

Current coverage is 80.54%

Merging #103 into master will decrease coverage by -0.88% as of 416f89b

@@            master    #103   diff @@
======================================
  Files          111     113     +2
  Stmts         2546    2616    +70
  Branches       296     298     +2
  Methods          0       0
======================================
+ Hit           2073    2107    +34
- Partial         46      48     +2
- Missed         427     461    +34

Review entire Coverage Diff as of 416f89b

Powered by Codecov. Updated on successful CI builds.

Anything in particular I can do to move this along?  Thanks

Hi Nels,
Sorry I've just been really busy. I'll merge this now and push a minor update for you


⚠ Bofrost.lu has degraded performance

In 9f9d4f5, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1122 ms

Resolved: Bofrost.lu performance has improved in cbc85b7.


Issue#150

Fixes #150 .
I added all the actions and bots to the code base and their yaml files etc.

@HarshKumarChoudary Thank you for your contribution. Looking forward to more such amazing contributions 🥂


tree viewer feature request: lines, arrows, etc.

I'd love to be able to draw a line (or, better, an arrow) from point A to point B. This should be easy to do in Director; it just requires some thought about the JSON format and a little bit of implementation.
I'll take a crack at it myself.

also polyline


Ingesting Tensorflow in larger (Bazel) project

System information

OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux 16.04
TensorFlow version: 1.10
Python version: 3.5
Installed using virtualenv? pip? conda?: Custom
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: 9.2/7
GPU model and memory: compute 6.1

Describe the problem
I am trying to use Tensorflow in a repository which also uses Bazel as build system.
Currently, Tensorflow C++ API library is pre-compiled and ingested as cc_library; however, I am trying to ingest it and build it from source directly since I need to support a number of configurations (and pre-compiling them is a pain)
I have been successful with a CPU only version. I did this by following the method used by Tensorflow Serving. I included the following in my WORKSPACE file
load("//repo.bzl", "tensorflow_http_archive")

tensorflow_http_archive(
    name = "org_tensorflow",
    git_commit = "355cc566efd2d86fe71fa9d755ceabe546d577a7",
    sha256 = "a19b64d5c7aecb8487302843b27c8fde916dc08df8d9c2fdded20d94e0947c34",
)

load("@bazel_tools//tools/build_defs/repo:http.bzl", "http_archive")

# TensorFlow depends on "io_bazel_rules_closure" so we need this here.
# Needs to be kept in sync with the same target in TensorFlow's WORKSPACE file.
http_archive(
    name = "io_bazel_rules_closure",
    sha256 = "a38539c5b5c358548e75b44141b4ab637bba7c4dc02b46b1f62a96d6433f56ae",
    strip_prefix = "rules_closure-dbb96841cc0a5fb2664c37822803b06dab20c7d1",
    urls = [
        "https://mirror.bazel.build/github.com/bazelbuild/rules_closure/archive/dbb96841cc0a5fb2664c37822803b06dab20c7d1.tar.gz",
        "https://github.com/bazelbuild/rules_closure/archive/dbb96841cc0a5fb2664c37822803b06dab20c7d1.tar.gz",  # 2018-04-13
    ],
)

# Specify the minimum required bazel version.
load("@org_tensorflow//tensorflow:version_check.bzl", "check_bazel_version_at_least")

check_bazel_version_at_least("0.20.0")

load("@org_tensorflow//tensorflow:workspace.bzl", "tf_workspace")

tf_workspace(
    path_prefix = "",
    tf_repo_name = "org_tensorflow",
)

where I defined repo.bzl as
""" TensorFlow Http Archive
Modified http_archive that allows us to override the TensorFlow commit that is
downloaded by setting an environment variable. This override is to be used for
testing purposes.
Add the following to your Bazel build command in order to override the
TensorFlow revision.
build: --action_env TF_REVISION="<git commit hash>"
  * `TF_REVISION`: tensorflow revision override (git commit hash)
"""

_TF_REVISION = "TF_REVISION"

def _tensorflow_http_archive(ctx):
    git_commit = ctx.attr.git_commit
    sha256 = ctx.attr.sha256

    override_git_commit = ctx.os.environ.get(_TF_REVISION)
    if override_git_commit:
        sha256 = ""
        git_commit = override_git_commit

    strip_prefix = "tensorflow-%s" % git_commit
    urls = [
        "https://mirror.bazel.build/github.com/tensorflow/tensorflow/archive/%s.tar.gz" % git_commit,
        "https://github.com/tensorflow/tensorflow/archive/%s.tar.gz" % git_commit,
    ]
    ctx.download_and_extract(
        urls,
        "",
        sha256,
        "",
        strip_prefix,
    )

tensorflow_http_archive = repository_rule(
    implementation = _tensorflow_http_archive,
    attrs = {
        "git_commit": attr.string(mandatory = True),
        "sha256": attr.string(mandatory = True),
    },
)

However, once I try using the GPU version, I run into problems... I have tried this by adding the following to my .bazelrc file, which are the same as the onces generated by a manual ./configure step in a standalone Tensorflow build
# Options for CUDA enabled Tensorflow
build --action_env TF_NEED_OPENCL_SYCL="0"
build --action_env TF_NEED_CUDA="1"
build --action_env CUDA_TOOLKIT_PATH="/usr/local/cuda"
build --action_env TF_CUDA_VERSION="9.2"
build --action_env CUDNN_INSTALL_PATH="/usr/local/cuda-9.2"
build --action_env TF_CUDNN_VERSION="7"
build --action_env TF_NCCL_VERSION="1"
build --action_env TF_CUDA_COMPUTE_CAPABILITIES="6.1"
build --action_env LD_LIBRARY_PATH="/usr/local/cuda-9.2/lib64:"
build --action_env TF_CUDA_CLANG="0"
build --action_env GCC_HOST_COMPILER_PATH="/usr/bin/gcc"
build --define=using_cuda=true --define=using_cuda_nvcc=true

I get the following error:
/cuda_configure.bzl", line 423, in cuda_toolkit_path
                auto_configure_fail("Cannot find cuda toolkit path.")
        File "/home/ferronfr/.cache/bazel/_bazel_ferronfr/fb0eee7f7437f761aece7d670e37cf54/external/org_tensorflow/third_party/gpus/cuda_configure.bzl", line 342, in auto_configure_fail
                fail(("\n%sCuda Configuration Error:%...)))

Cuda Configuration Error: Cannot find cuda toolkit path.

Does anybody know how to solve this?
Cheers!

Please check the tested build config. For  TF1.10, CUDA/cuDNN 9.0/7.0 is supported. Did you had a chance to migrate to TF1.12? Thanks!

Hi,
Actually the reason why the CUDA toolkit is "missing" is that in my Bazel project, the CUDA toolkit is itself a Bazel library target, CUDNN library is another target, etc so that's why the Tensorflow configuration doesn't find it, because it lives in some crazy long path in Bazel cache like
/home/ferronfr/.cache/bazel/_bazel_ferronfr/fb0eee7f7437f761aece7d670e37cf54/cuda_pc_amd64
Digging a bit, it seems Tensorflow only uses the environment variables defined by the --action_env to create a @local_config_cuda target in cuda_configure.bzl. This is over a thousand lines of scripting so I'm a bit lost...but I am just wondering if with the current targets @cuda and @cudnn library targets I have available in my Bazel project, I can somehow hand these over to Tensorflow? It seems plausible since there is a function called _create_remote_cuda_repository() ?
I tried also TF1.12 but the underlying issue is the same... thanks!

Could you try running configure.py from your main repository root?
@r4nt may also have some ideas.

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

Closing due to lack of recent activity. Please update the issue when new information becomes available, and we will open a new issue. Thanks!


[Bigtable] Split client into API client and user client

This is part 1 of a refactoring to eventually allow for a AppProfileId specifications, multi-channel support, and smart retries for ReadRows and MutateRows. The latter two are not included with this change.
Recommend to review one commit at a time.

Oh and to address your initial comment: all tests are still passing.

Not sure where the comment I received by email about it not building on Travis has gone, but I'd prefer to remove the hack if possible... but I'm okay with it going in as-is for now, just to unblock you.

Well the other possibility is to modify build.sh. Would you prefer that instead?

Without knowing why it didn't build, it's hard to know for sure. I'd rather we just got it to build under Travis :) Note that over the last 24 hours we've updated the SDK version - maybe it will build on Travis now anyway? I'm happy to look into it later though.

Here is the build log: https://travis-ci.org/GoogleCloudPlatform/google-cloud-dotnet/builds/344500517
The 4.6.1 assemblies could not be found. I just assumed that version was not supported on Travis.

Ah, does it rely on some part of Roslyn that doesn't work on .NET Core yet? If so, that makes sense. Will have a think about a better way to go long-term, but that's fine to start with.

Yes, the MSBuildWorkspace to load the csproj file. That only exists in the full framework.


mount crash

$ ./gidevice-cli mount ../tmp/14.0/DeveloperDiskImage.dmg ../tmp/14.0/DeveloperDiskImage.dmg.signature
panic: runtime error: index out of range [2] with length 2

goroutine 1 [running]:
github.com/electricbubble/gidevice-cli/cmd.glob..func6(0x174db60, 0xc00005c640, 0x2, 0x2)
	/Users/prife/workplace/iOS/gidevice-cli/cmd/mount.go:59 +0x9c5
github.com/spf13/cobra.(*Command).execute(0x174db60, 0xc00005c5e0, 0x2, 0x2, 0x174db60, 0xc00005c5e0)
	/Users/prife/workplace/iOS/gidevice-cli/vendor/github.com/spf13/cobra/command.go:856 +0x2c2
github.com/spf13/cobra.(*Command).ExecuteC(0x174dde0, 0x10014d0, 0x0, 0x0)
	/Users/prife/workplace/iOS/gidevice-cli/vendor/github.com/spf13/cobra/command.go:960 +0x375
github.com/spf13/cobra.(*Command).Execute(...)
	/Users/prife/workplace/iOS/gidevice-cli/vendor/github.com/spf13/cobra/command.go:897
github.com/electricbubble/gidevice-cli/cmd.Execute()
	/Users/prife/workplace/iOS/gidevice-cli/cmd/root.go:26 +0x2d
main.main()
	/Users/prife/workplace/iOS/gidevice-cli/main.go:8 +0x25
FAIL: 2

mount.go
59: dmgPath, signaturePath = args[1], args[2]

should be
dmgPath, signaturePath = args[0], args[1]

😂Sorry, I didn't do enough testing on this project
v0.0.4  fixed it


Fix used instrumentations shown on shutdown

What does this PR do?
Closes #2970.

Made the output deterministic
Added correct handling for experimental
Ensured that no used instrumentations are missing

Checklist

[x] This is a bugfix

[ ] I have updated CHANGELOG.asciidoc
[x] I have added tests that would fail without this fix

:grey_exclamation: Build Aborted

There is a new build on-going so the previous on-going builds have been aborted.

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2023-02-02T13:43:41.222+0000

Duration: 6 min 28 sec

Steps errors
Expand to view the steps failures

mvn install

Took 1 min 58 sec . View more details here
Description: ./mvnw clean install -DskipTests=true -Dmaven.javadoc.skip=true

mvn install

Took 0 min 24 sec . View more details here
Description: ./mvnw clean install -DskipTests=true -Dmaven.javadoc.skip=true

mvn install

Took 0 min 24 sec . View more details here
Description: ./mvnw clean install -DskipTests=true -Dmaven.javadoc.skip=true

:robot: GitHub comments
Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run benchmark tests : Run the benchmark tests.

run jdk compatibility tests : Run the JDK Compatibility tests.

run integration tests : Run the Agent Integration tests.

run end-to-end tests : Run the APM-ITs.

run windows tests : Build & tests on windows.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)


Add scheduled standups

Add a schedule command which schedules the next standup.
Members should be reminded of the standup 15 minutes before it starts. The bot should run the standup by itself. When the standup starts, it should let people know they can end the standup with the @bot end command.
We will also need a way to cancel a scheduled standup.
Parsing the schedule could be tricky, so we should see if we can leverage slackbot to do this. Maybe the standup bot can take the schedule message, dm it to slackbot to set it's own reminder? When that reminder comes up, the standup bot should then remind each member in a DM?
Or maybe slack has an api that would allow us to parse a reminder.

The more I think about this, the more I think using slackbot is a good idea.

Turns out bots won't respond to commands from another bot :(

Oooh, we might be able to listen to bot_message events to get messages from slackbot


implement fix for recording latency - MVP

The approach that was used on this branch is being abandoned. A new filx will be produced on a different branch


WebCore::FloatPoint Exception

SwiftR.useWKWebView = false results in an exception in WebCore::FloatPoint::FloatPoint: assembly.
SwiftR.usWKWebView = true results has no issues.
Occurs immediately after calling SwiftR.connect(url) {...}

Using the latest from 'master' branch.
This is a new issue, as it wasn't occurring before Swift 3 release.

Let me know what other information I can provide.

Can you provide some information about the server you're using (version, URL, etc.)? Since this doesn't happen with the demo project/server, I'm going to guess it's somehow server-related.

I'm fearing it might be server too, just wanted to see if this provoked any "aha" moments.
Server: version 2.2.0
Any way to get a console output of what the SignalR library or SwiftR is up to? Can I leverage Safari at all?

Yeah, you can open a Safari Web Inspector instances when running in the simulator or on the device. In there, you can debug SwiftR.js and also see console logs from SignalR.
FWIW, the usual culprit is CORS when it works with UIWebView but not WKWebView. First make sure you have CORS enabled.

Looks like the issue was that it SwiftR wasn't connecting from the main thread. UIKit, and its UIWebView, was rather displeased with this scenario. Thanks again for your help.


TQDM error in Multi_Script_CLI.py

Before submitting an issue, make sure you have:

[x ] Updated to the latest version
[ x] Read the README and docs
[ x] Searched the bugtracker for similar issues including closed ones

Purpose of your issue?

[ x] Bug report (encountered problems/errors)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections requests more details for particular types of issues, you can remove any section (the contents between the triple ---) not applicable to your issue.

For a bug report, you must include the Python version used, code that will reproduce the error, and the error log/traceback.
Paste the output of python -V here: Python 3.7.0
Code:
# Example code that will produce the error reported
from instabot import Bot

bot = Bot()
bot.login(username="user", password="pass")
followers = bot.get_user_followers(bot.user_id)

Error/Debug Log:
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
ZeroDivisionError: integer division or modulo by zero

Describe your issue
When starting the bot using the command: "python multi_script_CLI.py"
I get the following result:
Traceback (most recent call last):
File "multi_script_CLI.py", line 7, in
from tqdm import tqdm
ModuleNotFoundError: No module named 'tqdm'
How can i solve this?

@Sprellex  try pip install -U instabot


bug report: inputs is not valid! BadRequestType('error when building IndexRequestProto from

Running:

OS.X 11.1 (M1 chip)
jina==1.1.0

While running the object detection search, I hit the following error.
index |█                   | 📃      0 ⏱ 0.0s 🐎 0.0/s      0      batchindex ...	           JINA@7502[C]:inputs is not valid! BadRequestType('error when building IndexRequestProto from (b\'\\xff\\xd8\\xff\\xe0\\x00\\x10JFIF\\x00\\x01\\x01\\x01\\x00H\\x00H\\x00\\x00\\xff\\xdb\\x00C\\x00\\x01\\x01\\x01\\x01\\x01\\x01\\x01\\x01\\x01\\x01\\x01\\x01\\x01\\x02\\x02\\x03\\x02\\x02\\x02\\x02\\x02\\x04\\x03\\x03\\x02\\x03\\x05\\x04\\x05\\x05\\x05\\x04\\x0

Step to reproduce:

Clone and download the repo.
Run the requirement script
Download the kaggle data
Run: python app.py -task index -n 1000 -overwrite True
Index fails, with the above error.

Full error message can be found here.

I guess this is due to the fact that the hub module in your local environment is out of date. I've added hub requirements

Thanks, @nan-wang , I pulled your branch but it's still not working. The same occurs.

@cristianmtr PR #522 didn't fix that issue for me, did you try it?

yes, it worked. But I see this was in reference to a specific PR, not to master

If you get it working let me know, I can jump on a quick call with you in the next 20 minutes to see if it a local issue for me


Manual approval required for workflow run<PHONE_NUMBER>: Release t0rn refs/tags/v1.10.1-rc.0

Please approve runtime upgrade with github release t0rn v1.10.1-rc.0
Runtime upgrade dry run output:
💈 Script started at 1743545 block in t0rn chain

🐡 Running with dryrun flag!

🐙 checking out v1.10.1-rc.0...
✅ tag checked out

🔎 making sure runtime version got updated...
✅ runtime versions updated

🫧 check WASM artifact...
🔢 WASM blake2_256 hash: 0xad7eb66a4802e92aeecbde38f24593716319ed2c593eb42dbffcc785336e9c96
🔢 WASM blake2_256 hash fetched from release: 0xad7eb66a4802e92aeecbde38f24593716319ed2c593eb42dbffcc785336e9c96
✅ WASM blake2_256 hash is matching
⚙ set_code runtime upgrade... dry
✅ runtime upgrade executed... dry

Workflow is pending manual review.
URL: https://github.com/t3rn/t3rn/actions/runs/4697476007
Required approvers: [AwesomeIbex MaciejBaj]
Respond "approved", "approve", "lgtm", "yes" to continue workflow or "denied", "deny", "no" to cancel.

Workflow cancelled, closing issue.


Animated Sprite for user images

It would be great if we could use an animated sprite for the player avatar in the dialogue.
I'm imagining if it would be possible to load an animated sprite that was saved as an independent scene for the character avatars.
Perhaps enabling the game maker to use Godot's own sprite nodes would eliminate some other issues with your implementation sprites.

It's already possible, you can load custom scenes in the character's portrait, All you have to do is create your Animated spite node and save it as a scene.
I'm using it for my own game and it works like charm.
The only down side is that you can't see the preview in the editor, but at least it works.


runtime, x/mobile: runtime.raise_trampoline.abi0 crash

What version of Go are you using (go version)?

$ go version
go version go1.20.5 darwin/amd64

Does this issue reproduce with the latest release?
Yes.
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE="on"
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/user/Library/Caches/go-build"
GOENV="/Users/user/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/user/go/pkg/mod"
GONOPROXY="*"
GONOSUMDB="*"
GOOS="darwin"
GOPATH="/Users/user/go"
GOPRIVATE="*"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/usr/local/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.20.5"
GCCGO="gccgo"
GOAMD64="v1"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/user/Work/gomobile-experiments/go.mod"
GOWORK=""
CGO_CFLAGS="-O2 -g"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-O2 -g"
CGO_FFLAGS="-O2 -g"
CGO_LDFLAGS="-O2 -g"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/7v/1f7g4jw1337ggmqlxxgq74g00000gn/T/go-build1425140998=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
I have an app in production which uses library created with gomobile bind. Sometimes iOS crashes with runtime.raise_trampoline.abi0. There is no dependence on the user's device type and iOS version.
What did you expect to see?
No crash.
What did you see instead?
Crash.

This function is only called when the program is already about to crash or terminate, usually when it receives a signal that it doesn't want to handle. Does this program or other program send it signals that it doesn't want to handle?

@cherrymui I don't send any signals and don't wait for them. The crash is pretty rare and nobody from our team had it but according to reports it's about 80% of all crashes in the application. Users said that the crash can occur at any time: on the app start or even when user is doing nothing.
Full crash report is here. FIRCLSMatchException* is from Firebase library and it's the only suspect.

FIRCLSMatchException* is from Firebase library and it's the only suspect.

That is possible.
Unfortunately the stack traces for Go threads are stopped at runtime.asmcgocall. Could you try Go 1.21rc2 and see if that makes any difference? Thanks.

Could you try Go 1.21rc2 and see if that makes any difference?

I will try 1.21 after it will be released and notify on results.

@NightBlaze  I also encountered the same problem, any follow-up?

@jiangzhiguo1992 Nope. Will make next release with go 1.21 and check if the issue still present.

go version go1.21.1 darwin/arm64
I get this crash all the time，how to slove it？
help me

I have the same issue with a library built with the latest gomobile (v0.0.0-20231127183840-76ac6878050a) and go version (1.21.5).
I am not able to find a root cause that triggers the crash.


refactor(CategoryTheory/MorphismProperty): turn StableUnderBaseChange into a class

This allows for less tedious extra arguments.
A consistent renaming of StableUnderBaseChange to IsStableUnderBaseChange and StableUnderCobaseChange to IsStableUnderCobaseChange can be done in a follow-up PR.

Thanks for doing this!
maintainer merge

Thanks very much!
bors merge


Fixes #12689

Fixes #12689

Is this an issue with ios and wp as well?

@AmitMY I don't think so (the original input css blames from https://github.com/ionic-team/ionic/pull/9679, which notes this is not an issue for ios/wp).

Hello and thank you for contributing to Ionic! We have been working on porting all of the Ionic components to web components and have recently updated master to reflect this. This significant change has caused this pull request to break. While we really appreciate the time and effort you put into creating this, we are not able to merge it because of the newly introduced conflicts. We are extremely sorry about this. We will not be merging any more features in to v3. If this is a feature and you have the time, please resubmit this PR against the master branch. If this is a critical security issue in v3, we would greatly appreciate it if you would resubmit the PR against the new v3 branch. Thanks so much for your time!