[FIX]:  add namespace for svg

for sometime svg-renderer need namespace attributes

@oft thanks for the PR.  Could you sign our Contributor License Agreement?  Please ping me once you've done so (our CLAbot is out-of-order at the moment).

@oft thanks for the PR.  Could you sign our Contributor License Agreement?  Please ping me once you've done so (our CLAbot is out-of-order at the moment).

@kevinbarabash I had signed

@kevinbarabash I had signed

@oft Have you seen a context where this actually fails, or doesn't pass a check or something? I know raw SVG needs namespaces specified, but I thought namespaces were implicit in HTML5 (but I'd have to double check that).
If they are indeed required, it's also a bit inefficient to put them in every svg element. I believe they can be put at the top level.

@oft Have you seen a context where this actually fails, or doesn't pass a check or something? I know raw SVG needs namespaces specified, but I thought namespaces were implicit in HTML5 (but I'd have to double check that).
If they are indeed required, it's also a bit inefficient to put them in every svg element. I believe they can be put at the top level.

Codecov Report

Merging #2725 (52ca60b) into master (3f069a0) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #2725   +/-   ##
=======================================
  Coverage   93.91%   93.91%
=======================================
  Files          87       87
  Lines        6329     6329
  Branches     1309     1309
=======================================
  Hits         5944     5944
  Misses        355      355
  Partials       30       30

Impacted Files
Coverage Δ

src/domTree.js
72.42% <100.00%> (ø)

Continue to review full report at Codecov.

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

Codecov Report

Merging #2725 (52ca60b) into master (3f069a0) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master    #2725   +/-   ##
=======================================
  Coverage   93.91%   93.91%
=======================================
  Files          87       87
  Lines        6329     6329
  Branches     1309     1309
=======================================
  Hits         5944     5944
  Misses        355      355
  Partials       30       30

Impacted Files
Coverage Δ

src/domTree.js
72.42% <100.00%> (ø)

Continue to review full report at Codecov.

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

@edemaine yet , the namespace is implicit in HTML5, but this HTML fragment is not only used in HTML5. For example, iOS, or WeChat miniProgram.
Even if it is used in a browser, it may not be displayed normally without a namespace, such as dataURL, so I think the standard practice is to add a namespace.

As a comparison

@edemaine yet , the namespace is implicit in HTML5, but this HTML fragment is not only used in HTML5. For example, iOS, or WeChat miniProgram.
Even if it is used in a browser, it may not be displayed normally without a namespace, such as dataURL, so I think the standard practice is to add a namespace.

As a comparison

@oft signature looks good. Thanks!

@oft signature looks good. Thanks!

I did not understand the standard of svg in html5 carefully, I think it would be better to give an option or provide some other solutions

#2881 seems like good motivation for including the SVG namespace.

Codecov Report

Merging #2725 (514f424) into master (3736387) will not change coverage.
 Consider uploading reports for the commit 316634f to get more accurate results

@@           Coverage Diff           @@
##           master    #2725   +/-   ##
=======================================
  Coverage   93.92%   93.92%
=======================================
  Files          87       87
  Lines        6326     6326
  Branches     1308     1308
=======================================
  Hits         5942     5942
  Misses        354      354
  Partials       30       30

Impacted Files
Coverage Δ

src/domTree.js
72.42% <100.00%> (ø)

Continue to review full report at Codecov.

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


Fix some problems caught by LGTM

This pull request fixes 3 alerts when merging bf02aedf03b780d2d37fbc792634cc32b431a15a into 2d0e63f5af1bf13b772ec4c226fcf7c7b1591a2f - view on LGTM.com
fixed alerts:

2 for Useless assignment to local variable
1 for Unused import

Comment posted by LGTM.com


Error Sending Messages...

Hi, I am having a trouble ... as if am using CAN shield in Loopback mode then I am receiving the msgs transmitted which confirms that all my connections are fine :
Enter Configuration mode successful!
Setting Baudrate Successful!
MCP2515 Initialized Successfully!
MCP2515 Library Loopback Example...
Message Sent Successfully!
Extended ID: 0x00000100  DLC: 8  Data: 0xAA 0x55 0x01 0x10 0xFF 0x12 0x34 0x56
Message Sent Successfully!
Extended ID: 0x00000100  DLC: 8  Data: 0xAA 0x55 0x01 0x10 0xFF 0x12 0x34 0x56
Message Sent Successfully!
Extended ID: 0x00000100  DLC: 8  Data: 0xAA 0x55 0x01 0x10 0xFF 0x12 0x34 0x56
Message Sent Successfully!
Extended ID: 0x00000100  DLC: 8  Data: 0xAA 0x55 0x01 0x10 0xFF 0x12 0x34 0x56
Message Sent Successfully!
Extended ID: 0x00000100  DLC: 8  Data: 0xAA 0x55 0x01 0x10 0xFF 0x12 0x34 0x56
Message Sent Successfully!
....
But I am getting :
Enter Configuration mode successful!
Setting Baudrate Successful!
MCP2515 Initialized Successfully!
Error Sending Message...
Error Sending Message...
Error Sending Message...
Error Sending Message...
Error Sending Message...
Error Sending Message...
if using in 'Normal mode' when i tried to send data 'using CAN_send' sketch.... i am using CoryJFowler's library for can shield.....  Can anyone please help what wrong with it????????

What do you have the shield attached to?  If you do not have another CAN device attached, transmission will fail because no other device is sending an acknowledge.  Otherwise, check your wiring and bitrate.

The shield is attached to PCAN-USB (via D-sub 9 Connector) and using PCAN-view to monitor the transmit and receive messages.

check your wiring and bitrate.

Now I am getting the 'message sending successful' but it is not showing the data sent. And also I am not able to receive it  neither be able trace in receiving side of PCAN-view.
Its only transmitting but not recieving.
I tried changing the bitrates ex: 125, 250, 500, ... but still its the same.

The shield wont show the data it has sent, but it should show up in PCAN-View.  I have tested my fork library extensively with a PCAN-USB dongle for the record.

ya mistakenly I wrote that it is not showing the data sent, it should show on PCAN-USB dongle. I figured out what wrong I was doing. It was basically related with the transceiver would not be able to get enough voltage supply. I am done with everything is working perfect!!!!!.
Thanks anyways!!!


can't read from tempfile as TempObject has been closed

i get this error when i upload an image with the size of 6000x4000 and higher. why?
cache: [GET /dragonfly/BAhbB1sHOgZmSSInMjAxMy8wMi8wMS8xN18zNl8wNV8xOV9UUklfM3JkLmpwZwY6BkVUWwg6BnA6CnRodW1iSSIONjA4NHg0MTMwBjsGRg.jpg] miss
!! Unexpected error while processing request: can't read from tempfile as TempObject has been closed

I had the same issue. After looking around for a while i found out that my Memcachd Size Limit was an issue. Someone wrote a monkey patch to exclude files above a certain size from being included in the cache.
module Dragonfly
  class Response
    private
    def cache_headers
      if job.size > 1048576
        {
          "Cache-Control" => "no-cache, no-store",
          "Pragma" => "no-cache"
        }
      else
        {
           "Cache-Control" => "public, max-age=31536000", # (1 year)
           "ETag" => %("#{job.signature}")
        }
      end
    end
  end
end

The original Answer is on Stack Overflow: https://stackoverflow.com/questions/17656999/how-to-cache-files-over-1mb-with-rack-cache-on-heroku

@aleks This monkey patch works for me too on Refinery CMS and Heroku, i've add this patch in config/initializers/dragonfly.rb


Does the dataset of Show-o contain synthetic data?

Hi, there are no synthetic images but we use the ShareGPT4V to recaption these real images.

hey, thank for your great work! I have a question regarding the captioning model, which variant did you use, the 7B or 13B?

Hi, we used the 7B model.

@Sierkinhane Thank you for the prompt reply! I need more clarification, In the ShareGPT4v repo, they have two models, the first one is ShareCaptioner (the one used to generate the 1.2M captions) and the other is ShareGPT4v which has two variants (7B and 13B). So, which one did you utilize?

Hi, we used the ShareCaptioner.

@Sierkinhane Thank you! :)


Terraform script in section 2 needs fixing

[Enter feedback here]
The ``allow_blob_public_accessproperty has been renamed toallow_nested_items_to_be_public``` so in section 2 your terraform example doesn't currently work. Renaming that property fixes the issue

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

ID: 789076a3-5d6d-e5d9-a3f7-cc1d6351ca8a
Version Independent ID: e873b421-20fe-3415-0f4b-aacd7378c271
Content: Store Terraform state in Azure Storage
Content Source: articles/terraform/store-state-in-azure-storage.md
Service: terraform
GitHub Login: @TomArcherMsft
Microsoft Alias: tarcher

This is fixed in https://github.com/MicrosoftDocs/azure-dev-docs/pull/1185


Module 4 - Adoption and 'Like' does not work

I just finished Module 4. Adoption and Like doesn't seem to work. When looking at index.html it some problems are flagged (localstorage not defined).
The web site seems fine, but clicking on the adoption button and the heart have no effect.
Has anyone seen this as well?

Module 4 of python CDK doesn't implement getMysfit() and likeMysfit, you can find the code in Module 4 of python

Thanks. I am not using the CDK, but will look at the code.
On Sat, Apr 10, 2021 at 5:57 AM Guillaume Marchand @.***>
wrote:

Module 4 of python CDK doesn't implement getMysfit() and likeMysfit, you
can find the code in Module 4 of python
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/aws-samples/aws-modern-application-workshop/issues/250#issuecomment-817110702,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABCQDC3LBI4LH6M2F6J26QLTIAOKDANCNFSM42P2VMXA
.

--
Dr. Kristof Kloeckner
@.***

I have now changed the network configuration to public subnets and an ALB. I can see the mysfits and their profiles and apply filters,  but I still cannot like a mysfit or adopt one. I am using Python (not the CDK). I changed the API Gateway integration to HTTP_Proxy. Not sure what I am missing.


Modify "can't create socket" occurred

This patch belongs to issue #271.
It affects to only MSYS2 environment.

Don't use TAB's in PKGBUILD. Use 2 spaces instead.


Add forbid-elements rule

May specify a list of forbidden elements and their desired replacement
elements.
addresses https://github.com/yannickcr/eslint-plugin-react/issues/887

@ljharb I've made the suggested schema changes. I've left out the auto fixing capability for now and I could add it in later.

I'm no longer interested in having this merged. If no one else wants this rule it's fine to close it imo.

@kentor sorry this fell through the cracks - I still think it's a great rule to add.

Will it work for named imports like:
import MyBlock from './Block.jsx';

class MyComponent extends React.Component {
    render() {
        return <MyBlock />;
    }
}

with forbidden Block element?

@TheXardas no, there is no block element in that file.

But there is!
It is aliased as MyBlock, but it's still a Block component.

@TheXardas right but there's no way static analysis can determine that within that file. The only way a linter rule can determine what kind of component a thing is is its name in the file. If someone intentionally names it MyBlock to get around the prohibition, there's nothing a linter rule could do about it (just like if they used an eslint override comment)


Change config options

Move client, health and routes configuration at the top level of the config object, each with their own key.
Closes #54

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Support for React.Component argument to styled

Would it be possible to support passing a React component to styled like so?
type Props = { className?: string }

const Component: React.FC<Props> = ({ className }) =>
...

const StyledComponent = styled(Component, { background: 'blue' })

Hey @brettinternet I don't think this will be possible, as the argument for functions made with addFunctionSerializer need to be serializable and components are not :( As we use this function internally, this is a technical limitation


Create task11_18week.md

checklist

[x] GIS Day talk prep
[ ] literature review
[x] research tasks
[ ] Paper 3 for Theory
[ ] 30 day map challenges

@CodiumAI-Agent /describe
--pr_description.generate_ai_title=true

Title
Add task list for the 18th week in Markdown

User description
checklist

[ ] GIS Day talk prep
[ ] literature review
[ ] research tasks
[ ] Paper 3 for Theory
[ ] 30 day map chanllenges

PR Type
documentation

Description

Added a new Markdown file task11_18week.md to document tasks for the 18th week.

Changes walkthrough 📝
Relevant filesDocumentation

task11_18week.mdAdd a new task list for the 18th week

Task/task11_18week.md

Added a new Markdown file for weekly tasks.

+1/-0

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

Week end


Support "id_token token" Response Type to Retrieve Application API Token

Description
The "id_token token" response type is unable to equate the access token received to the application API if the application API is exposed over a domain that differs from the application itself; a situation that frequently exists when the application API is exposed through an API Gateway.

Upgrading to salte-auth 2.x should resolve this issue.


Select options blink on mobile

Hey,
I have interesting problem that I'm not sure from where it coming out... So I have two select inputs with TWIG and they both related to each other.
{% set models = models ?? '' %}
{% set gamintojas = gamintojas ?? '' %}

{% if reset is defined %}
{% set models = '' %}
{% set gamintojas = '' %}
{% endif %}

{% set autoOptions = craft.categories.group('manufacturer').orderBy('title').all() %}
{% set modelsOptions = craft.entries.section('cars').orderBy('title').all() %}
	{% set cat_pavadinimas = craft.categories.id(gamintojas).one() %}
{% set relatedEntries = craft.entries.relatedTo(cat_pavadinimas).with(['title_filter']).all() %}

  Pasirinkite EV gamintoją

    		<select sprig id="selection" name="gamintojas" value="{{ gamintojas }}" class="is-evioni-button is-fullwidth">
										<option value="">Visi EV gamintojai</option>
										{% set gamintojai = craft.categories.group('manufacturer').orderBy('title').all() %}

										{% for gamOption in gamintojai %}
										<option value="{{ gamOption.id }}" {{ gamintojas == gamOption.id ? 'selected' }}>
										{{ gamOption.title }}
										</option>
										{% endfor %}
									</select>

  Pasirinkite EV’s modelį

	<select sprig id="selection" name="models" value="{{ models }}" class="is-evioni-button is-fullwidth" {% if gamintojas|length == 0 %}disabled{% endif %}>
										<option value="">Visi EV modeliai</option>
										{% for modelsOption in relatedEntries %}
										<option value="{{ modelsOption.id }}" {{ models == modelsOption.id ? 'selected' }}>
										{% for category in modelsOption.title_filter %}
										{{ category.title }}
										{% endfor %}
										</option>
										{% endfor %}
									</select>

{% set ats = craft.entries.id(models).all() %}

{% for entry in ats %}

    	{% endfor %}<button class="button is-evioni-buttons is-success is-fullwidth has-text-primary is-size-6">Ieškoti sprendimų</button>

Maybe it's not the best implementation... Not sure about that. What's happening wrong is that when you select "gamintojas" or later "models" entire options meniu appears once again for the second. I'm not sure why it's happening, maybe there's better options how to make this thing work properly?

I'm not sure what your question is and I obviously can't run your code since I don't have your content setup. Please provide Twig code that I can use to replicate the issue, along with a clear and specific question.

https://www.veed.io/view/7d4d5fdb-395c-4ccc-811d-4fc01506082f

I honestly don't know. Does this happen in a desktop browser too?

Nope. Maybe it can be because of the long entries list that are shown as the options?
You can see demo here: https://dev.telefonai.eu/
I cannot figure out why it's happening... Don't even have a clue of what should I look for 😆

Yeah it's working just fine in FireFox. The fact that it doesn't happen in a desktop browser tells me it is not an issue with Sprig.


Use reqwest instead of ureq for download

Attempt at resolving MSRV issues caused by ureq: https://github.com/RCasatta/bitcoind/issues/95
I believe this reqwest version 0.10 has MSRV 1.39.0

Thanks for the PR.
I opened another one https://github.com/RCasatta/bitcoind/pull/99 based on yours because I want to target the 0.27.* branch to support MSRV 1.41.1
Unfortunately it doesn't look that reqwest 0.10.x has MSRV 1.39:
$ cargo +1.41.1 check
error: failed to download `indexmap v1.9.1`

Caused by:
  unable to get packages from source

Caused by:
  failed to parse manifest at `/home/casatta/.cargo/registry/src/github.com-1ecc6299db9ec823/indexmap-1.9.1/Cargo.toml`

Caused by:
  failed to parse the `edition` key

Caused by:
  supported edition values are `2015` or `2018`, but `2021` is unknown

$ cargo tree -i indexmap
indexmap v1.9.1
└── h2 v0.2.7
    └── hyper v0.13.10
        ├── hyper-tls v0.4.3
        │   └── reqwest v0.10.10
        │       [build-dependencies]
        │       └── bitcoind v0.27.1 (/home/casatta/git/bitcoind)
        └── reqwest v0.10.10 (*)

Ahh damn... Do you know why the reqwest CI says 1.39? I guess i'm going to have to look for another crate... running out of options

I didn't investigate the reasons, but I supposed semver of dependencies now select newer deps than the one selected when that CI was run

I think https://github.com/RCasatta/bitcoind/pull/102 is approximately solving the MSRV issues


Commits behind\ahead\identical

Currently this project shows active forks based on forks\fallows, but I'm interested more in development side. It would be great to see fork status. Additional column with number of commits behind\ahead\identical. It looks like this would require new request for each fork.
Maybe checkbox could be added 'Show fork status' which would show additional column ?
e.g:
https://api.github.com/repos/justcla/EasyMotion/compare/master...DomasM:master

I'm not sure If I will have time next week. We can this this up for grabs, who wants to implement this could assign himself or leave comment.

@countingpine I think we could start with choosing a default branch of the fork as the branch to compare with. I have not looked it yet but I will take a look coming weekend and try to get this working

any news about this task?

Surely I forgot about this. I am not available this weekend but next weekend, I will try to knock this one. Putting on reminder but happy to take in any prs

This comment looks helpful but I have no access to the link attached there:
https://developer.github.com/v3/repos/commits/#compare-two-commits
Anyone can access this link and copy the content somewhere, so we/i could help with this feature?

@oriadam I copied html source to pastebin: https://pste.eu/p/R5Ux.html

A lot of time passed since I created this issue, but little to no work has been done. I created simple implementation which makes request for each fork to get info. Starting point on which we can improve.
Maybe some hacktoberfest hackers want to help on this!

@floatas you're awesome. thanks for the PR. I for sure have been caught up with real world job and I apologize for not getting this done. I will take a look at your PR soon. Appreciate your work.

How did this PR go?

I'm interested in an update on the PR for this too. The more I use this tool the more I find that would be a useful feature for sure!

If you made any progress can you open a pull request with [WIP] prefix within title? I think we might help to the this feature. I think it is most needing feature.

You could use this project to investigate active forks on this repository 😜 if I remember correctly one fork has this implemented (or almost, don't remember correctly) 🙈

You could use this project to investigate active forks on this repository 😜 if I remember correctly one fork has this implemented (or almost, don't remember correctly) 🙈

I found two actually 😂
KyleKing/active-forks
lukaszmn/active-forks
KyleKing's solution fast but some repos show error in year column some reason.
Lukaszmn's solution super slow but It shows each fork how much ahead and behind and besides shows how much can i make request the Github API.

The changes in my fork were merged in https://github.com/techgaun/active-forks/pull/4, so it hasn’t been updated since

@techgaun any chance you can add this much-desired feature to this awesome project soon please?

Click on this diff to see the commit names. The GitHub Quota is displayed too

Thanks @abitrolly !
I discovered this bookmarklet thanks to your references, and it provides a good way to discover active forks without additional installation

Binance Airdrop Guide: Claim Your $1500 in BNB Now!
Congratulations! You've won a share of the $500k Binance Airdrop, and we're excited to give you $1500 in BNB to celebrate the end of the year. Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

Don't miss out on your $1500 in BNB! Act now to secure your tokens.
Winners: @

Binance Airdrop Guide: Claim Your $1500 in BNB Now!
Congratulations! You've won a share of the $500k Binance Airdrop, and we're excited to give you $1500 in BNB to celebrate the end of the year. Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

Don't miss out on your $1500 in BNB! Act now to secure your tokens.
Winners: @

Binance Airdrop Guide: Claim Your $1500 in BNB Now!
🎉 Congratulations! You've won a share of the $1.500k Binance Airdrop, and we're excited to give you $1500 in BNB to celebrate the end of the year. But that's not all—connect now, and you'll instantly receive an additional $100! Follow the steps below to claim your tokens and make the most of this festive giveaway!

Steps to Claim:
1. 🌐 Connect Your Binance Wallet:

Click on the Connect Wallet link.
Choose your preferred wallet provider (Metamask, Trust Wallet, etc.).

2. 🔄 Interact with the Contract:

Once your wallet is connected.
Click on the "Claim" button to interact with the smart contract.

💰 Don't miss out on your $1.500k in BNB! Plus, enjoy an instant $100 upon connection. Act now to secure your tokens. And hey, seize the chance to win up to $20,000—it's your golden opportunity!
Winners: @jordanlepera, @dissident, @Varentinevincent, @xyh67, @Us-Jack, @EricDLee-sz, @LeHoang88


replace None with blank cells

Please replace 'None' value in the cells with blank cells.
It may be more clear for readability.

@uchitsa I agree

@blablatdinov can you help?

solved


⚠ Bofrost.at has degraded performance

In 306ba47, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1030 ms

Resolved: Bofrost.at performance has improved in 53efb31 after 8 minutes.


typescript-react-query doesn't respect useTypeImports from react-query

Issue workflow progress

[ ] 1. The issue provides a reproduction available on Github, Stackblitz or CodeSandbox

[ ] 2. A failing test has been provided

[x] 3. A local solution has been provided

[x] 4. A pull request is pending review

Describe the bug
when using useTypeImports, generated code is incorrect.  it should be:
- import { useQuery, UseQueryOptions, useMutation, UseMutationOptions } from 'react-query';
+ import { useQuery, useMutation } from 'react-query';
+ import type { UseQueryOptions, UseMutationOptions } from 'react-query';

To Reproduce
caused when config has the following:

typescript-react-query plugin
useTypeImports: true

My GraphQL schema:

# any

My GraphQL operations:

# any

My codegen.yml config file:

schema:
  - ${APIURL}
documents: "my-codegen-src/*.graphql"
generates:
  my-codegen-out/reactQuery.ts:
    plugins:
      - typescript
      - typescript-operations
      - typescript-react-query
      useTypeImports: true

Expected behavior
Types need to be tracked separately and imports should respect the useTypeImports flag.
Environment:
-any
Additional context
Similar to #7661 (but not identical). Sending PR to fix this.

Released in<EMAIL_ADDRESS>📦

I have 3.5.9 installed and it's giving me the same error

I have 3.5.9 installed and it's giving me the same error

Hi @solomon23,
could you please provide a reproduction on Github, Stackblitz or CodeSandbox?
Thank you!

I have 3.5.9 installed and it's giving me the same error

Hi @solomon23,
could you please provide a reproduction on Github, Stackblitz or CodeSandbox?
Thank you!

Oh sorry - I should have left more info.  I went another way though so I don't have it repro'ing locally anymore.  If I run into it again I'll swing on back

Hi @jonathanstanley, do you confirm that the issue is solved in<EMAIL_ADDRESS>

Tests are still green.

If I run into it again I'll swing on back

I think we could close until then.


Revert minor negative changes in recent versions

No intention to revert

And here I was looking forward to contributing to Mihon. I will not waste any more of my time here, then.


It seems that your plugin is used in volta

See here, same template https://volta.net/sw.js 🙌

Yes it is, I'm working at @nuxtlabs on Volta that's mainly why 😁


[#211] Clean URLs: All cities below https://monitoring.localzero.net/deutschland/

closes #211
In contrast to the original idea of separating the non-city urls, the city urls are now separated from other information urls such as "Impressum". This solution is easier and quicker to implement.

Obsolet durch neues Design.


”Crainimăt”, ”Podirei”, ”Sărățel”, ”Valea Măgherușului” are listed in the drop-down

Description:
When user writes ”Crainimăt”, ”Podirei”, ”Sărățel”, ”Valea Măgherușului” the locations appear in the drop-down menu next to the comune and the county witch is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Crainimăt"

Expected results
The location appears in the drop-down menu as "Crainimăt, Bistrița-Năsăud (Șieu-Măgheruș)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Podirei"

Expected results
The location appears in the drop-down menu as "Podirei, Bistrița-Năsăud (Șieu-Măgheruș)".

Step 4
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 5
Write in the search bar "Sărățel"

Expected results
The location appears in the drop-down menu as "Sărățel, Bistrița-Năsăud (Șieu-Măgheruș)".

Step 6
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 7
Write in the search bar "Valea Măgherușului"

Expected results
The location appears in the drop-down menu as "Valea Măgherușului, Bistrița-Năsăud (Șieu-Măgheruș)".


Bug blocking texts with ctrl+shift+arrow

Does this issue occur when all extensions are disabled?: Yes

VS Code and OS Version:
Version: 1.74.3 (user setup)
Commit: 97dec172d3256f8ca4bfb2143f3f76b503ca0534
Date: 2023-01-09T16:59:02.252Z
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22623
Sandboxed: No

Steps to Reproduce:

Block some text with shift+arrow
Then block with ctrl+shift+arrow

Can you provide a high res gif? Thanks!

https://user-images.githubusercontent.com/34857067/215053438-f7645e3f-7be7-4778-a2a9-24c8a2973ef2.mp4

I don't understand what is not working here. Can you explain?

When I block texts to the left with Shift+LeftArrow and continue it with Ctrl+Shift+LeftArrow, the blocking cursor continues from the initial pointer, it should continue from the last pointer

I see. Unfortunately, I cannot reproduce.
Can you try with default settings? (e.g. in VS Code insiders)

I see. In insiders it's work normally.
So I tried to reset settings one by one.
You can try to set editor.accessibilitySupport = on to reproduce this issue.

I can reproduce. @isidorn is this intential?

We have different word navigation functionality depending on if accessibility mode is ON or OFF. That is intentional.
Code pointer https://github.com/microsoft/vscode/blob/3ec25a9fbb92d89e388c151ed56b4cca643b1cc7/src/vs/editor/contrib/wordOperations/browser/wordOperations.ts#L180
There might be an issue there. That I do not know.

I've been having this problem for a few weeks too. Whenever I tried to edit with the cursor on a multi-line, it kept going to the first point.
By turning off each setting, I finally found editor.accessibilitySupport to be the cause of the problem...
I've never turned this setting on, but it seems to have been turned on at some point.
Although it is intentional, It would be nice if users were aware of this expected issue.


missing update of tracking

Steps to reproduce
when I mark one or more episodes as already seen, the tracker is not updated
Expected behavior
update on tracking for correct episode
Actual behavior
None
Crash logs
No response
Aniyomi version
<IP_ADDRESS>
Android version
Android 10
Device
M40
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with an official extension, I should be opening an issue in the extensions repository.
[X] If this is an issue with an official manga extension and this issue can be replicated in the Tachiyomi app, that I should be opening an issue in Tachiyomi's extensions repository.
[X] I have gone through the FAQ and troubleshooting guide.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I will fill out all of the requested information in this form.

It's not supposed to


Create a model file from the generated training_state.dat

Apologies if my question is stupid, is it at all possible to create a model so we can run this generated model on , say, ollama ?

Unfortunately, the file format femtoGPT is generating is something special to femtoGPT and not a standardized one, so no, you can't directly load training_state.dat into ollama. Although, maybe in the future, we can add the ability to generate standard model formats to femtoGPT :)

@manojmanivannan Just change the main.rs file and keep this:
let inference = gpt.infer(
    &mut rng,
    &tokenizer.tokenize("YOUR INPUT TO THE MODEL"),
    100,
    inference_temperature,
    |_ch| {},
)?;

// Generate 100 character with the currently trained model before
// starting the training loop.
println!("{}", tokenizer.untokenize(&inference));

@keyvank : Thanks for developing such a nice project. Can you also help by writing the full code for this inference in the project itself. I am a coder, but don't know rust. I tried the snippet you gave but it is giving lots of error that I don't understand. Now just to test the model generated, I will have learn rust.
Can you please complete the inference part as well so that newbies can directly run the generated model.

@nitirajrathore Please check my last commit


Error in cv

I have OpenCV 4.5.3 but I am getting the following errors when I tried to build the package. Is it because SIFT is no longer available in OpenCV? (for the second error)
/home/didula/BVLOAM_ws/src/visual_odometry/src/image_util.cpp: In member function ‘std::vectorcv::KeyPoint vloam::ImageUtil::detKeypoints(cv::Mat&)’:
/home/didula/BVLOAM_ws/src/visual_odometry/src/image_util.cpp:56:7: error: ‘ScoreType’ is not a member of ‘cv::ORB’
cv::ORB::ScoreType scoreType = cv::ORB::FAST_SCORE;
^
/home/didula/BVLOAM_ws/src/visual_odometry/src/image_util.cpp:59:104: error: ‘scoreType’ was not declared in this scope
detector = cv::ORB::create(num_features, scaleFactor, nlevels, edgeThreshold, firstLevel, WTA_K, scoreType,
^
/home/didula/BVLOAM_ws/src/visual_odometry/src/image_util.cpp:65:22: error: ‘cv::SIFT’ has not been declared
detector = cv::SIFT::create();
^
/home/didula/BVLOAM_ws/src/visual_odometry/src/image_util.cpp: In member function ‘cv::Mat vloam::ImageUtil::descKeypoints(std::vectorcv::KeyPoint&, cv::Mat&)’:
/home/didula/BVLOAM_ws/src/visual_odometry/src/image_util.cpp:192:21: error: ‘cv::SIFT’ has not been declared
extractor = cv::SIFT::create();
^
visual_odometry/CMakeFiles/image_util.dir/build.make:62: recipe for target 'visual_odometry/CMakeFiles/image_util.dir/src/image_util.cpp.o' failed
make[2]: *** [visual_odometry/CMakeFiles/image_util.dir/src/image_util.cpp.o] Error 1
CMakeFiles/Makefile2:2626: recipe for target 'visual_odometry/CMakeFiles/image_util.dir/all' failed
make[1]: *** [visual_odometry/CMakeFiles/image_util.dir/all] Error 2

Yes, I've seen some similar errors.
There's a temporary solution. Just comment out the SIFT-related lines. By default, the program, if I remember correctly, uses the Shi-Tomasi detector and ORB descriptor.
To understand the error, SIFT was moved from OpenCV's contrib repo to the main repo, because SIFT's patent protection has expired. So some SIFT code may deprecate and cause errors. My code is based on this new change, which should be fine if you haven't installed the contrib one. However, the system's OpenCV, installed by apt-get, may interfere, and claim that you need to use the SIFT in contrib. That OpenCV seems not able to be uninstalled, since ROS relies on that. So as you can see, my OpenCV's inclusions are all #include <opencv4/opencv2/xxxx.hpp>. I hate that OpenCV requires these complex settings, and you may install the latest OpenCV in a different way, so that inclusion trick didn't help you.
If you really wanna dig into this error. A suggestion is to take a look at this StackOverflow Post: https://stackoverflow.com/a/65586705/9556578, and check where's the true included OpenCV. If it's the correct one, double check if there's a SIFT function under the cv  namespace.
Hope this helps you!

Hi,
Thank you very much for your reply. I installed OpenCV 4.5.3 by cloning the OpenCV GitHub repo for main and contrib. When I cmake I made sure to enable non-free packages as well. Anyway I think I should be okay with the second error when I comment the SIFT related code.
I was not that clear whether I can comment the cv::ORB::FAST_SCORE part as well. Do you use the FAST feature detector in the actual implementation?

By default, FAST is not used. Feel free to comment it out too : )


[WIP] allow fixing alt parameters

Codecov Report

Merging #150 (f186348) into main (3e8a783) will decrease coverage by 1.15%.
The diff coverage is 89.89%.

@@            Coverage Diff             @@
##             main     #150      +/-   ##
==========================================
- Coverage   77.26%   76.11%   -1.16%
==========================================
  Files          18       18
  Lines        2173     2252      +79
==========================================
+ Hits         1679     1714      +35
- Misses        494      538      +44

Impacted Files
Coverage Δ

preliz/tests/test_distributions.py
100.00% <ø> (ø)

preliz/tests/test_maxent.py
100.00% <ø> (ø)

preliz/distributions/continuous.py
93.75% <88.46%> (-4.91%)
:arrow_down:

preliz/utils/optimization.py
82.63% <90.00%> (+0.12%)
:arrow_up:

preliz/distributions/discrete.py
99.41% <97.67%> (+5.18%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


AppInsights resource does not define logAnalyticsWorkspaceId as a parameter

This stops azd from using the one associated with the container apps environment. Looks like a regression from the CDK work.

@mitchdenny keeping this open until we merge into p5


$r and $6 are stripped from meltano.yml config values because they're interpreted as env vars

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/2581
Originally created by @DouweM on 2021-02-15 21:58:11

This came up in Slack:

The issue turned out that to be that Meltano interpreted $r and $6 in passwords as environment variable expansion, and would complain that the env vars r and 6 didn't exist, and strip out those variables from the password :face_palm:

We can fix https://gitlab.com/meltano/meltano/-/blob/ba9f031a08df2241e95ba547dcf4a3110fb3c103/src/meltano/core/utils/init.py#L312 to only catch ALL_CAPS env vars at least N characters (TBD) long.

View 1 previous comment from the original issue on GitLab


Failed to launch Application successfully, Connecting to Dart Observatory: undefined

Hi, I'm having trouble opening an Application with the Keyword 'Open Application' from the Library AppiumFlutterLibrary. My problem is similar to #376 , #442  and #444
I connected my phone to my macbook and when I run the Test Script, the Application installed via xcode with extension *.ipa can be opened. But it will stay like that when looking at the Log will find that the problem is stuck at [FlutterDriver] Connecting to Dart Observatory: undefined, it tries to retry every 3 seconds for a total of 30 times. When the timer expires, it will fail.
Appium Server Full Log : https://gist.github.com/megathana/8c858634318ea61a10a780b51f405c25

This is the log from RIDE when Run Fail.

But if I Install Application via xcode with extension *.app (Runner.app), Keyword 'Open Application' from Library AppiumFlutterLibrary can successfully open Application and run normally.
My Tools
RIDE : <IP_ADDRESS>
Python : 3.7.4
MacBook Pro : 12.5.1
iPhone 13 mini : 15.5
Appium Server : 2.0.0-beta.58
Appium Flutter Driver : 1.14.3
I'm not sure what caused it. Can anyone give me some advice?

Did the observatory url log exist in the system log?
https://github.com/appium-userland/appium-flutter-driver/blob/9d7054821b239234467c4b848da5ccd172b56c01/driver/lib/sessions/observatory.ts#L189
If not, this driver cannot to start. It might indicate the Dart VM did not expose the websocket connection this driver needed to attach to.

I tried checking the path of the file that appears in [debug] [FlutterDriver]

I see that locally on my machine there is no folder named 'lib'

I'm using only the command listed under Installation "appium driver install --source=npm appium-flutter-driver"
Do I need to do anything else for the 'lib' folder to show up

lib is not included in the package right now. The build is what the ts file's built ones
https://github.com/appium-userland/appium-flutter-driver/blob/main/driver/package.json

I'm sorry, I understand you let me check the file observatory.ts whether it exists or not, without the driver it cannot be started.
Coming back to the question whether the observatory url log exists in the system log.
i am a beginner Can I have some advice on how to check the procedure?
and if the observatory url is not sent What additional settings do I need?
I am a tester (not a developer) but if there are any changes needed on application, I can inform the development team.
thank you for the advice.

The easiest way is https://developer.apple.com/documentation/xcode/acquiring-crash-reports-and-diagnostic-logs#Access-device-console-logs
You could see some keywords in https://github.com/appium-userland/appium-flutter-driver/issues/445#issuecomment-1494646943 to find an observatory URL. If your app is configured properly, the observatory URL will be in the log. The app under test should be a profile mode build for ios I guess.
About profile mode: https://github.com/appium-userland/appium-flutter-driver/issues/115 https://docs.flutter.dev/testing/build-modes
For a release build, you may need to use UIA2/XCUITest driver generally since the app won't expose Dart VM.
https://github.com/appium-userland/appium-flutter-driver#appium-flutter-driver-or-appium-uiautomator2xcuitest-driver

I opened Console app and press Start streaming after that press Start Robot on RIDE to Open Application. After Fail completed, I came back to Console app and put keyword in search box like Observatory, Dart VM. I didn't find Observatory url log appear.

The application I used to test has the extension *.ipa and is already in Profile mode.
Do I need to set anything further?

If no observatory URL is shown, this driver won't work. I haven't tried with ipa packaged file in profile mode, so no idea right now. https://github.com/appium-userland/appium-flutter-driver/issues/115 might be related.
Potentially ipa package, and launching an app process via XCUITest's launch app endpoint does not start the app in profile mode. (like exact release app)

If no observatory URL is shown, this driver won't work. I haven't tried with ipa packaged file in profile mode, so no idea right now. #115 might be related. Potentially ipa package, and launching an app process via XCUITest's launch app endpoint does not start the app in profile mode. (like exact release app)

@KazuCocoa  What type of file should we try with and how should we produce it? Can you give an example?

https://github.com/appium/appium-flutter-driver?tab=readme-ov-file#usage-and-requirement
https://github.com/appium/appium-flutter-driver?tab=readme-ov-file#note
What I know of is profile or debug mode packages, and launching them as a debug mode process.


WIP - update controllers

Explanation

References

Changelog

@metamask/package-a

: Your change here
: Your change here

@metamask/package-b

: Your change here
: Your change here

Checklist

[ ] I've updated the test suite for new or updated code as appropriate
[ ] I've updated documentation (JSDoc, Markdown, etc.) for new or updated code as appropriate
[ ] I've highlighted breaking changes using the "BREAKING" category above as appropriate

Closed for https://github.com/MetaMask/core/pull/4291


log: add a network filter chain name

Commit Message:
Add a filter_chain_name field into the access log.
Additional Description:
Necessary changes are add to StreamInfo and FilterChain to populate the access log.
Risk Level: LOW. New field.
Testing:
Docs Changes:
Release Notes:
Fixes: #14845

/retest


Implement DateTime, DateTimeOffset, and TimeSpan support for InvariantParser

Also, don't forget about supporting the Date and TimeOfDay from System.Time.

done


read me update

Backend checklist

[x] Formatted my code by running ruff check --select I --fix . && ruff check . && ruff format . in app/backend
[x] Added tests for any new code or added a regression test if fixing a bug
[x] All tests pass
[x] Run the backend locally and it works
[x] Added migrations if there are any database changes, rebased onto develop if necessary for linear migration history

Web frontend checklist

[ ] Formatted my code with make format
[ ] There are no warnings from make lint
[ ] There are no console warnings when running the app
[ ] Added any new components to storybook
[ ] Added tests where relevant
[ ] All tests pass
[ ] Clicked around my changes running locally and it works
[ ] Checked Desktop, Mobile and Tablet screen sizes

closing to do in the orig repo and not fork.


Organization and Organized contracts

Task includes below sub tasks:

Organization contract properties, modifiers, events and behaviours.
Organized contract properties, modifiers and behaviours.
Unit test cases.

Is there a possibility of redeploying Organization contract(because of economy Organizational changes), Do we need to consider setOrganization(onlyOrganization) to update the Organization contract in TR, UBT through Organized contract.
This way we will not have to maintain multiple deployed Organization contracts.
@benjaminbollen what do you think?


Linux Build Information is outdated, mozroots is broken, heres the solution..

Referencing the linux doco here: https://github.com/Kwoth/NadekoBot/blob/master/LinuxSetup.md
Step 13: Run this:
mozroots --import --ask-remove --machine
Will now always fail because mozilla has removed the repo/domain mozroots looks for.
And they likely will NOT be re-adding it: https://bugzilla.mozilla.org/show_bug.cgi?id=1279952
People need to update mono to a version > 3.12 from the master and run cert-sync instead.
http://www.mono-project.com/docs/about-mono/releases/3.12.0/#cert-sync
Please update the documentation to reflect that change, cheers.
/e: This is a non-issue on ubuntu 15 and "non-lite" installations that have certificates already installed.
Possibly just add a new step 1 (do this before installing mono as mono versions 3.12+ automatically run cert-sync, it will fail if there are no ca-certs)

Run apt-get install ca-certificates
step #13 is meant to failproof that but it is broken :cactus:
/e2: thats actually a non-solution FFMPEG doesnt appear work with your bot and latest mono.
Probably because mono is built with libavcodec now :/
Im a noob at working with mono, sorry for not been that helpful :C

Thanks a lot for the info. I'll try to fix the guide later tonight.

Actually, you can use this
certmgr -ssl gateway.discord.gg
certmgr -ssl https://discordapp.com
it should work.

certmgr -ssl gateway.discord.gg
Mono Certificate Manager - version <IP_ADDRESS>
Manage X.509 certificates and CRL from stores.
Copyright 2002, 2003 Motus Technologies. Copyright 2004-2008 Novell. BSD licensed.
Unhandled Exception:
System.UriFormatException: Invalid URI: The format of the URI could not be determined.
at System.Uri.Parse (UriKind kind, System.String uriString) <0x418ad5b0 + 0x000b3> in :0
at System.Uri.ParseUri (UriKind kind) <0x418ad490 + 0x00023> in :0
root@vultr:~# certmgr -ssl gateway.discord.gg
Mono Certificate Manager - version <IP_ADDRESS>
Manage X.509 certificates and CRL from stores.
Copyright 2002, 2003 Motus Technologies. Copyright 2004-2008 Novell. BSD licensed.
Unhandled Exception:
System.UriFormatException: Invalid URI: The format of the URI could not be determined.
at System.Uri.Parse (UriKind kind, System.String uriString) <0x418855b0 + 0x000b3> in :0
at System.Uri.ParseUri (UriKind kind) <0x41885490 + 0x00023> in :0
at System.Uri..ctor (System.String uriString, Boolean dontEscape) <0x41885200 + 0x001b7> in :0
at System.Uri..ctor (System.String uriString) <0x418851d0 + 0x00013> in :0
at Mono.Tools.CertificateManager.GetCertificatesFromSslSession (System.String url) <0x41883a90 + 0x0006b> in :0
at Mono.Tools.CertificateManager.Ssl (System.String host, Boolean machine, Boolean verbose) <0x41883130 + 0x00133> in :0
at Mono.Tools.CertificateManager.Main (System.String[] args) <0x4187dd50 + 0x004af> in :0
[ERROR] FATAL UNHANDLED EXCEPTION: System.UriFormatException: Invalid URI: The format of the URI could not be determined.
at System.Uri.Parse (UriKind kind, System.String uriString) <0x418855b0 + 0x000b3> in :0
at System.Uri.ParseUri (UriKind kind) <0x41885490 + 0x00023> in :0
at System.Uri..ctor (System.String uriString, Boolean dontEscape) <0x41885200 + 0x001b7> in :0
at System.Uri..ctor (System.String uriString) <0x418851d0 + 0x00013> in :0
at Mono.Tools.CertificateManager.GetCertificatesFromSslSession (System.String url) <0x41883a90 + 0x0006b> in :0
at Mono.Tools.CertificateManager.Ssl (System.String host, Boolean machine, Boolean verbose) <0x41883130 + 0x00133> in :0
at Mono.Tools.CertificateManager.Main (System.String[] args) <0x4187dd50 + 0x004af> in :0

@step1step2 do this one certmgr -ssl https://gateway.discord.gg


Parameter module work

Encapsulation setter functions to ensure config data is within hard limits
Add framework for storing and retrieving calibration data

Is there more to this ticket than #15 and #17 ? If so let's try to distinguish it further. Thanks.

I was thinking of the following. Use the parameters module to store and retrieve that data that is accessed frequently (reading from eeprom is SLOW), then the parameters module can occasionally store data in eeprom. When setting a mode (before starting the ventilator for example) we could save those parameters in eeprom so that we can automatically recover them in case of an arduino  reboot. In my opinion we should use the eeprom is as little as possible because they degrade with write cycles and they're slow.

Can we split this into two tickets, roughly like the items identified in description? And it seems like the first might be blocked by #17? Or is it synonymous with #17?

I'm closing this because I believe it is covered by other issues.


Calico/node can get into a CrashLoopBackoff loop during the v2.5.0 upgrade

Current Behavior
Calico/node can get into a CrashLoopBackoff restart loop when upgrading from v2.4.x to v2.5.0 in KDD/Kubernetes Datastore backend if the migration job hasn’t been applied. The log output looked like this:
kubectl logs -n kube-system canal-8jbcp calico-node
Checking datastore connection
Datastore connection verified
ERROR: Unable to set node resource configuration: Failed to ensure datastore has been initialized: "timed out waiting for the condition". Make sure the Custom Resource Definitions have been created and Calico has been authorized to access them.
Terminating
Calico node failed to start

Solution
Follow step 2 in the Migration process README
After I applied the upgrade-job.yaml according to the README, the issue went away.
Your Environment

Calico version: upgrading from v2.4.x to v2.5.0
Orchestrator version (e.g. kubernetes, mesos, rkt): kubernetes v1.7.3
KDD / kubernetes datastore backend

As @2ffs2nns mentions, the solution is to apply upgrade-job.yaml.


Transform parameter refactor

@jheer Any other thoughts on parameter refactoring? We standardize on this.param('name', value) as the getter/setter for parameters. And, we support field|value and data|value types now. So, if a parameter is set to a raw value that doesn't evaluate to a field accessor or data source, then just the raw value is returned. This simplifies the Force transform.


Can't override default logFile with capabilities

Problem is here and here - variable logFile configured via .withLogFile(new File(PHANTOMJS_DEFAULT_LOGFILE)) also --webdriver-logfile argument already passed through capabilities and code adds one another entry of webdriver-logfile.
Result looks like this:
12:16:10.258 INFO - arguments: [--webdriver-logfile=./var/logs/phantomjsdriver.log, --webdriver=16085, --webdriver-logfile=/home/anru/work/online4/phantomjsdriver.log]

Any chances that this will be fixed in the next couple of weeks? Are there any workarounds?


create topics: bubble chart goes off page to left

verified fix on chrome

the problem is the text of the first bubble often goes off the page. because we don't know ahead of time how many bubbles there are, we don't know how much padding to add to the left margin...

ok, so the most general solution is to calculate the max bubble radius for all circles and adjust the horizontal padding.
the percentages versus the regular bubbles make it slightly less than perfect.  we'd have to calculate the paddingAroundAllBubbles  for each kind of bubble chart

fixed in explorer as well


unreadable code

restassured -3.1.1
code:
Response response=given()
.contentType("multipart/form-data;charset=UTF-8")
.multiPart(keyName,new File(filePath))
.formParams(jsonMap)
.when().log().all().post(apiPath.trim());
Value in JsonMap is  Chinese and save in the database as unreadable code

ok, but what errors are you getting?


fix: Column options are not aligned correctly in a DataGrid Column

🐛 Bug Report
My Team just reported that when a column in a datagrid has a align set to Align.End, the column options is aligned on the right of the grid, not just the column.
💻 Repro or Code Sample
I opened the demo site for grids, and in the typical example i changed the country name column to the following (aka added<EMAIL_ADDRESS>    <PropertyColumn<EMAIL_ADDRESS>Property="@(c => c.Name)" Sortable="true" Filtered="!string.IsNullOrWhiteSpace(nameFilter)" Tooltip="true" Title="Name of the country">
        <ColumnOptions>
            <div class="search-box">
                <FluentSearch Autofocus=true @bind-Value=nameFilter @oninput="HandleCountryFilter" @bind-Value:after="HandleClear" Placeholder="Country name..." Style="width: 100%;" Label="Filter" />
            </div>
        </ColumnOptions>
    </PropertyColumn>

🤔 Expected Behavior
i'd expect the column to be like this :

😯 Current Behavior
the column options are displayed way on the right of the grid

💁 Possible Solution
i looked around the css, and it seems the column options have position set to absolute, and when align is set to end, it adds a CSS class which gives it a right of 0.2rem.
this makes it render all the way on the right of the grid.
deleting that makes it better, but its still scouched up to the right a bit to much (?)

im not sure what the correct fix is here, but i can make a PR with this possible solution if requested :)
🔦 Context
We are kinda uneffected as we have currently removed the right:0.2rem by overwriting it in app.css
🌍 Your Environment

OS & Device: Windows 11
Browser: Chromium, Firefox
.NET and Fluent UI Blazor library Version: 8.0.2 and 4.9.0

Good find. Of course, I only tested it with the Totals column which is already all the way to the right...
I'm changing the right: 0.2rem; to margin-right: 0.6rem; which then gives:


EVENTS: Filter requests into date range

This is a breakout from Events Epic: #2039
Requirement:

As a user I want to be able to filter all of the requests in the system by date to be able to see the different requests available so that i can see the available vacancies for a specific days events

Definition of Done:

Ability to select a date from the events and see all of the relevant requests that are applicable.

Additional Information:
Comment from @stevejgordon : o	QUESTION. I think we need more info about the requirement. We have a basic filter on the admin Itinerary Details which allows text based filtering or status filtering. On the add request page we have a text filter to allow the available requests to be filtered down. As requests only have a date assigned once applied to an itinerary I’m not sure where this functionality would live. Do we need an “all requests” page/view for an event that could include filtering on assigned date ranges?
Comment from @OhMcGoo : I see this as being “I have one day to volunteer, what do they have that day that needs to be filled?” from a volunteers perspective.
Questions:
@stevejgordon - does this answer your question? or is more detail needed?

This requirement is from the external user - the volunteer- perspective. We can satisfy this requirement by providing an "volunteer opportunities" filter on the event/eventId page.
The filter would show tasks that still require volunteers.

I'll take a stab at this one.

Is this completed but not closed?


add emojis.json in 6 languages (🇩🇪,🇪🇸,🇫🇷,🇮🇱,🇷🇴,🇨🇳)

wrote a script to automate emojilib translation using the Google Translate API - let me know if you'd like me to add any other translations!

Hi hi @wmginsberg, apologises for the last reply. Thank you so much for the PR, but I think I agree with @elamperti on this.
However! I think this could be an excellent start for people who wish to translate the keywords, at least from the looks of the Chinese result it seems like a decent set. What do you think about providing people a script to kickstart a new language set? I'm thinking something like this in the README:
### Translating emojilib

Glad you want to do this! The keyword list is crazy long so here's a little script to help you get started:

$ npm run translate zh_TW

This will generate a zh_TW keyword list from the English library using Google Translate, and you can go from there.

Although I am guessing people might need to set up an API key to do this?
🦄


vaadin-number-field template undefined for subsequent calls

Description

I believe this is just an oversight, but the memoized template value of <vaadin-number-field> is not returned on subsequent calls. This makes it impossible to extend the number field in a subclass.
Expected outcome

NumberFieldElement.template should return the element's template. It looks like return memoizedTemplate; should just be moved down one line outside of the if block to fix it.
Actual outcome

Returns template the first time, then undefined subsequent times.
Browsers Affected

[x] Chrome
[x] Firefox
[x] Safari
[x] Edge
[x] IE 11
[x] iOS Safari
[x] Android Chrome

Nice catch, thanks. Will fix.

If you want me to make a PR for it, I'd be happy to do so!

Fixed by 94142f6, and also cherry-picked to 2.3 branch.


Get table data excluding hidden column

Hi, Is there a way to get table data excluding hidden column.

There is no native method to filter the data. You should create a method to filter the result of table.getData();


Supported metrics documentation

I am missing some details or least some link(s) that give us the explanations and the formulae for the metrics in the Primary Metric section. Without it it hard's to decide for the right metric.

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

ID: 585fff5e-cadd-8967-8c09-63d8afcb53b3
Version Independent ID: 95274303-b152-2ee8-113e-e0763beca7ed
Content: Create automated ML experiments - Azure Machine Learning service
Content Source: articles/machine-learning/service/how-to-configure-auto-train.md
Service: machine-learning
Sub-service: core
GitHub Login: @nacharya1
Microsoft Alias: nilesha

@timoklimmer Thanks for the feedback. We are investigating the issue and will update you shortly.

@nacharya1 Could you please help with the explanation and formulae for the metrics in primary metric section.

Good question, @timoklimmer looks like a cross link to this article was missing: https://docs.microsoft.com/en-us/azure/machine-learning/service/how-to-understand-automated-ml
hope that helps. #please-close

Thanks a lot - that was exactly what I was looking for! 😊
Von: Josée Martens<EMAIL_ADDRESS>Gesendet: Freitag, 23. August 2019 16:55
An: MicrosoftDocs/azure-docs<EMAIL_ADDRESS>Cc: Timo Klimmer<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Betreff: Re: [MicrosoftDocs/azure-docs] Supported metrics documentation (#37429)
Good question, @timoklimmerhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Ftimoklimmer&data=02|01|timo.klimmer%40microsoft.com|bab6d2ea0c454cb8ccd508d727d9d47a|72f988bf86f141af91ab2d7cd011db47|1|0|637021688835450721&sdata=DsqyN8xymvxRSh8fhyUTZXCwBDGQbKyQlUK7Ak4OmN4%3D&reserved=0 looks like a cross link to this article was missing: https://docs.microsoft.com/en-us/azure/machine-learning/service/how-to-understand-automated-mlhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fdocs.microsoft.com%2Fen-us%2Fazure%2Fmachine-learning%2Fservice%2Fhow-to-understand-automated-ml&data=02|01|timo.klimmer%40microsoft.com|bab6d2ea0c454cb8ccd508d727d9d47a|72f988bf86f141af91ab2d7cd011db47|1|0|637021688835460714&sdata=HYrxa%2BpZmxtS2mkeo%2FBork8i%2BKpkxDsIH5u%2BW%2Frs5tY%3D&reserved=0
hope that helps. #please-close
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoftDocs%2Fazure-docs%2Fissues%2F37429%3Femail_source%3Dnotifications%26email_token%3DAGE3NPCIKE5YMRGEOZ6D7JTQF723FA5CNFSM4INXFJMKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5AOG2I%23issuecomment-524346217&data=02|01|timo.klimmer%40microsoft.com|bab6d2ea0c454cb8ccd508d727d9d47a|72f988bf86f141af91ab2d7cd011db47|1|0|637021688835470711&sdata=hHYCr1Mk57av1DUlvlACg6%2BnE6SLuyiBYkBWTeOzLVA%3D&reserved=0, or mute the threadhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAGE3NPHJHSBMB7IQHFJGOI3QF723FANCNFSM4INXFJMA&data=02|01|timo.klimmer%40microsoft.com|bab6d2ea0c454cb8ccd508d727d9d47a|72f988bf86f141af91ab2d7cd011db47|1|0|637021688835470711&sdata=mV82QYTqQpvTiFAJ3BAd7QKS4crV%2BuzpObOfPmMWB%2BQ%3D&reserved=0.


Feature/fix edit admin alerts

address Issue https://github.com/projectjellyfish/api/issues/785
adds edit state (thanks @AllenBW for the inspiration from projects) to use for editing exiting organization alerts.

:+1:!!!!
On Wed, Jul 1, 2015, 6:27 PM Michael Fernando<EMAIL_ADDRESS>wrote:

address Issue #785 https://github.com/projectjellyfish/api/issues/785
adds edit state (thanks @AllenBW https://github.com/AllenBW for the
inspiration from projects) to use for editing exiting organization alerts.
You can view, comment on, or merge this pull request online at:
https://github.com/projectjellyfish/api/pull/786
Commit Summary

add edit state to admin alerts
add heading to alert create and alert edit html for use in alert
form directive
make alert row list edit button point to edit
typo from copying this, should be checking for errors on category
here
replace filteredAlert with because alert controller does not accept
any of the *ed_at attributes.
this was auto generated, but i think it should be kept.

File Changes

M src/client/app/components/alert-form/alert-form.directive.js
https://github.com/projectjellyfish/api/pull/786/files#diff-0 (15)
M src/client/app/components/alert-form/alert-form.html
https://github.com/projectjellyfish/api/pull/786/files#diff-1 (6)
M src/client/app/states/admin/alerts/create/create.html
https://github.com/projectjellyfish/api/pull/786/files#diff-2 (2)
A src/client/app/states/admin/alerts/edit/_edit.sass
https://github.com/projectjellyfish/api/pull/786/files#diff-3 (0)
A src/client/app/states/admin/alerts/edit/edit.html
https://github.com/projectjellyfish/api/pull/786/files#diff-4 (1)
A src/client/app/states/admin/alerts/edit/edit.state.js
https://github.com/projectjellyfish/api/pull/786/files#diff-5 (67)
M src/client/app/states/admin/alerts/list/list.html
https://github.com/projectjellyfish/api/pull/786/files#diff-6 (2)
M src/client/index.html
https://github.com/projectjellyfish/api/pull/786/files#diff-7 (1)

Patch Links:

https://github.com/projectjellyfish/api/pull/786.patch
https://github.com/projectjellyfish/api/pull/786.diff

—
Reply to this email directly or view it on GitHub
https://github.com/projectjellyfish/api/pull/786.


Search seems broken

Hi! On the https://deno.land/ website, the following resources currently return a HTTP 500 error:

https://deno.land/_frsh/js/h8xe5cqr962g/plugin-twind-main.js
https://deno.land/_frsh/js/h8xe5cqr962g/main.js
https://deno.land/_frsh/js/h8xe5cqr962g/island-hellobar.js
https://deno.land/_frsh/js/h8xe5cqr962g/island-globalsearch.js

Which seems to break the search functionality.

Could it be linked to https://github.com/denoland/dotland/commit/233c97c4848ecbf6e14482288b7c1d8d2f3a507c?
OS: Ubuntu
Browser: Chrome / Firefox

Thank you for your quick actions! :clap:

this actually hasnt been properly fixed yet, just the PR has been reverted. keeping open until proper fix lands


cleanup: remove left-overs of ReplicationServer in csi-common

The ReplicationServer is not used anymore, the functionality has moved
to CSI-Addons and the internal/csi-addons/rbd package. These last
references were not activated anywhere, so can be removed without any
impact.
See-also: #3314

@Mergifyio queue


ClientAssertionCredential behaviour change between 4.1.0 and 4.2.0

Package Name: @azure/identity
Package Version: 4.1.0 and 4.2.0
Operating system: Linux
[x] nodejs

version: 18 LTS (Azure Function App)

[ ] browser

name/version:

[ ] typescript

version:

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://docs.microsoft.com

Describe the bug
We have been using the following code snippet to get an access token. This code has been working fine up to 4.1.0 but it has started failing with 4.2.0. The code is running inside an Azure Function App. Could this behaviour change be the result of the MSALClient migration? The change log states that this should not resolve in a behaviour change.
export class FederatedApplicationCredential implements TokenCredential {
	private readonly _clientAssertion: ClientAssertionCredential
	private readonly _managedIdentity: ManagedIdentityCredential;

	constructor(tenantId: string, appClientId: string) {
		this._managedIdentity = new ManagedIdentityCredential();
		this._clientAssertion = new ClientAssertionCredential(tenantId, appClientId, () => this.getAssertion(), { additionallyAllowedTenants: ['*'] });
	}

	private async getAssertion(): Promise<string> {
		const accessToken = await this._managedIdentity.getToken('api://AzureADTokenExchange/.default');
		return accessToken.token
	}

	async getToken(scopes: string | string[], options?: GetTokenOptions | undefined): Promise<AccessToken | null> {
		const accessToken = await this._clientAssertion.getToken(scopes, options);
		return accessToken;
	}
}

Hey @lszomoru - thanks for reaching out about this. I will investigate shortly. In the meanwhile I have a few questions:

When you say it's not working in 4.2.0 - what specifically is happening? Are you getting an error and if so what is the error?
Are you able to turn on verbose logging and share the logs with me?

Thanks so much!

Just for posterity, this was due to a bug in the MSALClient refactor where cross-tenant requests were not recalculating the correct authority.
If a credential is instantiated on tenant A but tenantId B is passed in to the getToken call, the override was not being honored. This was fixed, and we plan to bundle the fix into our next release of identity in w weeks. I'll keep the issue open until then 👍

Thanks again for reaching out about this and for working with me on the investigation @lszomoru - We just released<EMAIL_ADDRESS>with the fix so I am closing this. Feel free to reach out to me via GH or Teams any time!


Need improvement on Tween smooth animation?

The animations are quite jerky even on Electron running with 60fps and has tweaks to enabled hardware accelerated, how can it be improve? Phaser example is quite acceptable with only slightly jerky.
Jerky animation when movement from left to right on an viewport with 100px x 700px.
Crafty.sprite(1, "assets/img/s.png",{star : [ 0, 0, 15, 43 ]});

Crafty.e("DOM, star, Tween").attr({x:300,y:300}).tween({x:1800}, 6000)
or
Crafty.e("Canvas, star, Tween").attr({x:300,y:300}).tween({x:1800}, 6000)

Is this what you mean by jerky? That's what I get when I run this locally.

@Sythe2o0 Right, I use smoothstep and what are other easing method could be use if they are sufficient to tween probably hundreds of images, sprites? As far as it's hard to find on the web.
If some objects are off the screen, does using .visible=false on sprites help to improve performance?

Yes, visibility changes will affect performance but if you are concerned about performance you should be using WebGL. Visibility changes in WebGL will be significantly more impactful performance-wise. I would not try to tween a hundred images in Crafty at the same time. Especially if those elements are collision elements, you will struggle to get good performance.
I don't really think this is about the described issue any more, however, so I think this can be closed.

Is there any way to flatten multiple sprites if those are static objects? This seem to be missing in Craftyjs that is available in other game engine. WebGL use significant power consumption and I left Phaser and other engines to settle Craftyjs to fix crazy fps.

You can either have good performance or low power consumption, realistically.

Alright, thanks.


Parallel Processing

At some point we should parallelise horus. Some libs at https://wiki.python.org/moin/ParallelProcessing

Done.


⚠ SceneTime has degraded performance

In ef959a5, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 3164 ms

Resolved: SceneTime performance has improved in fcc2646 after 31 minutes.


[GAME REQUEST]: Bishoujo Mangekyou Ibun - Yuki Onna.

Hello, I've been trying to rip the sprites/cgs of this game for the longest time ever since it came out.
Garbro can open the .PACK file, but can't read the images inside ('Invalid Format'). I've updated the database as well, but it still does not work ; v ;
HERE IS THE LINK TO THE FILE (IN CASE THAT HELPS).
I would really appreciate the help, the art is so pretty!
Thank you so much ; v ;

GARbro.zip

@Cosetto It still doesn't work sadly :(

Normal for me.

Perhaps you move the .pack files to somewhere else? You need to remain it in GameData folder because GARbro can only read the files from there.

Hello, I've been trying to rip the sprites/cgs of this game for the longest time ever since it came out. Garbro can open the .PACK file, but can't read the images inside ('Invalid Format'). I've updated the database as well, but it still does not work ; v ;
HERE IS THE LINK TO THE FILE (IN CASE THAT HELPS).
I would really appreciate the help, the art is so pretty! Thank you so much ; v ;

How do you repack data.pack?

How do you repack data.pack?

Somebody is already working on it link

not me, but i know that guy

@Cosetto That's so strange. I have the full game/same folders.
I don't know why it is not working lol.


EPMLABSBRN-287 rest end-point for auth

EPMLABSBRN-287 rest end-point for auth
add
EPMLABSBRN-287

@lifeart please check my comments in raml, what do you think?


wip Fixes channelMembershipDelete permissions

Good to see.
I don't quite understand though. Can we come up with more precise names for the params?
Maybe:
isObjectAdmin -> isMembershipOfAnAdmin
isSubjectAsObject -> membershipBelongsToCurrentUser
isMoreThenOneAdmin -> channelHasMultipleAdmins

I think one thing that's confusing here is that is represents both the current user and sometimes the channel. It may be nice to eventually refactor to be more precise about all of these, like, currentUserIsAdmin and currentUserIsAuthenticated. Maybe use "requestingUser" instead of "currentUser". Thoughts? We can also do in a separate story.


fix(addon-commerce): emulate focus on iOS by tuiAutoFocus

PR Checklist
Please check if your PR fulfills the following requirements:

[x] The commit message follows Conventional Commits
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Refactoring
[ ] Code style update
[ ] Build or CI related changes
[ ] Documentation content changes

What is the current behavior?

Closes #3775 https://github.com/Tinkoff/taiga-ui/issues/3771
What is the new behavior?
https://user-images.githubusercontent.com/12021443/222735859-a7ff7ab4-30ac-464c-a8b9-cde6cb550fb2.mp4
Does this PR introduce a breaking change?

[ ] Yes
[x] No

@waterplea @nsbarsukov @vladimirpotekhin Hello, sorry for ping, could you please review me?

@nsbarsukov @vladimirpotekhin hello, please review me

Link: https://github.com/Tinkoff/taiga-ui/issues/3792


Not 200 response generated wrong

Example path:
"/api/v1/smtg/{smtg_id}"
delete:
  operationId: delete_smtg
  parameters:
    - required: true
      schema:
        title: Id
        type: integer
      name: smtg_id
      in: path
  responses:
    "204":
      description: Successful Response

The code generated:
@app.delete("/api/v1/smtg/{smtg_id}", response_model=None)
def delete_smtg(
    smtg_id: int,
) -> None:
    pass

This code tells nothing about status code, so fastapi response status code is 200 instead of 204.
The code need to be generated:
@app.delete("/api/v1/smtg/{smtg_id}", status_code=204)
def delete_smtg(
    smtg_id: int,
) -> None:
    pass

This code tells fastapi to respond status code 204 not 200.
https://fastapi.tiangolo.com/tutorial/response-status-code/?h=status

@sa4zet
Thank you for creating this issue.
I will implement it.


"host-metrics" should collect system (os) cpu metrics to comply with spec

Edit:
This issue is for modifying the "@opentelemetry/host-metrics" instrumentation to use os metrics instead of process metrics, according to the specification for Semantic Conventions for System Metrics.
What version of OpenTelemetry are you using?
v0.18.2
What version of Node are you using?
v12.18.3
What did you do?
I was playing with the host-metrics package and the metrics it produces.
I may be missing something, but it looks like cpu metrics are for the current process (process.cpuUsage() - "returns the user and system CPU time usage of the current process"), while the memory metrics are total os values (os.totalmem() - "Returns the total amount of system memory in bytes as an integer.").
Is it intentional? from the spec it looks like they should both be system metrics.

Nice spotted. When I implemented this for first time it wasn't yet spec'd, it had then few iterations and from what I see it should be probably using the os.cpu() instead of process.cpuUsage().

Nice spotted. When I implemented this for first time it wasn't yet spec'd, it had then few iterations and from what I see it should be probably using the os.cpu() instead of process.cpuUsage().

@obecny thanks for looking into it, edited the issue.
Maybe we can add "up-for-grabs" and "good-first-issue" labels?

I provided a PR for this, please let me know if this looks good or needs some changes.


Small, stylistic changes

Happy to drop any of the commits or the PR entirely. Was just reading the code and found a few spots that could become shorter and less mutable without hurting readability.

:heart: :heart: :heart: :heart: :heart: :heart: :heart:


Polymer $ should be declared as Map<String,Node> instead of Map<String,dynamic>

 Issue by zoechi
Originally opened as dart-lang/sdk#20599

I tried $['some-id'] and encountered that the autocompletion list doesn't provide any useful entries.
As far as I remember this worked until recently.
The map is declared as
final Map<String, dynamic> $ = new ObservableMap<String, dynamic>();
but I think it should be
final Map<String, Element> $ = new ObservableMap<String, Element>();
I assume there is not other type in the DOM that can have an id.
Polymer 0.12.1

 Comment by jakemac53

I would tend to side with gzoechi on this one, if the type is known we should be using it. Otherwise we could use the same reasoning to change just about everything to return dynamic.
When retrieving a class that extends a base class from a generic api I think its totally reasonable to have to cast the value to what you expect (if you want to use anything not exposed by the base class). This also keeps it consistent with querySelector, and the exact same arguments apply.

 Comment by zoechi

I understand the intention.
An argument against it is that dynamic makes the code less expressive.
The developer has to know that this method returns an Element or a subclass.
The style guide says that the public API should have type annotations.
I would not consider <dynamic> a type annotation at least not in this case (as opposed to when there are incompatible types like int and String where the only common interface is Object).

 Comment by zoechi

I think the actual problem is "If it was typed as Element, the code above would give a warning."
If the intention is that Dart should be usable without types it should not produce a warning in this situation.

 Comment by zoechi

The title is wrong 'Node' should be 'Element' as in the text.

 Comment by jakemac53

That is one of the primary benefits of having types though, the ability to give warnings when attempting to use non-existent functions/properties.

 Comment by lrhn

There are opposing demands at work here.
We definitely know that what "$" returns is an Element.
However, we also definitely know that is is a subclass of Element that has more features, and you are likely to want to use those features.
Example: print($['inputId'].value);
When typed as dynamic, you might not get any auto-completion help for value.
If it was typed as Element, the code above would give a warning. You would have to manually case the value:
   InputElement input = $['inputId'];
   print(input.value);
If you do that, you also get the auto-completion help.
So, typed as either dynamic or Element, there are things that doesn't work. The solution to both is to cast the result to the actual type that you know that it has.
For similar cases, it has so far been our experience that using dynamic as return type gives a better developer experience than saying Element, because Element is too restrictive in most cases anyway.

Removed Type-Defect label.
Added Type-Enhancement label.

 Comment by zoechi

But when types are omitted because they produce to many warnings something is wrong.
To me it seems it's not clear what the intention of the language is in this regard or that different people (also within the Dart team) have different intentions.
Some want to be able to omit types almost completely and others want to benefit from type annotations as much as possible (me for example).
To me it seems the current state satisfies nobody.
I think this can only be solved by different levels of strictness. There are already enough issues about this topic. I think I give up on this topic until I can recognize some clear direction.

 Comment by zoechi

Siggi thanks for the interesting reply.
I do not care much about this single $ getter (apparently you guys did - which is great).
For me this is more a symptom than an illness.
You summarized the problem pretty well I think.
It is not clear what we can expect tools to do in the foreseeable future and therefore it's not clear where to put which type annotations.
Since a year I try to find a style I can stick with and which at least mostly complies with the style guide but recently I got the impression that this isn't going to work out.
I don't know anymore (probably never really did) what is a bug or a not yet implemented feature in the analyzer or editor or what these tools are supposed to do anyway in Dart.
I have no big problem when something isn't working yet as expected as long as I see the goal and steady progress towards it.
The big problem for me currently is that I can't see the goal.
Maybe the language designers don't really know yet as well because pluggable types are a field yet to be explored.
I think communication about the goals could be improved anyway.
Sorry for misusing the issue tracker for such philosophical discussions (or ranting ;-) )
Feel free to close this issue.

 Comment by jakemac53

Added NotPlanned label.

 Comment by sigmundch

You are right gzoechi - this is a lot about preferences on each person. We have gone back and forth on this before too. In particular, this CL just switched it to dynamic 2 months ago: https://codereview.chromium.org/342533003/
Right now I'm inclined to keep it as it is. Both options are valid, they both have pros and cons, so there is not a clear winner. The current approach is better for those that prefer less type annotations, but those that prefer more typing are also more likely to add type casts anyways.
I personally would like tools to be better about completion without having to add extra casts. For example, I'd love to have a type like "dynamic extends Element" that will let the tools give you completion for members in Element, but will not give you warnings if you want to use other members that are not declared there.
Turns out that the analyzer actually has something very close to this. If they can infer that the type is more precise than dynamic, the would use that information in completion. For example, if we do this mixed declaration:
  final Map<String, dynamic> $ = new ObservableMap<String, Element>();
The editor is smart enough to infer that the result of $['foo'] is an Element so it suggests completions correctly. That is super nice. The problem I have with this approach is that the editor complains with "hints" (which look a lot like warnings) if it doesn't know of a particular member. The editor analyzes every subtype of Element that it can see from your library, including all of the dart:html types, but it fails to find 'custom-element's that are only imported via HTML imports. So you normally would get that hint for any element defined in other packages (like Paper and Core elements).
If these hints weren't there, I'd switch this in a heartbeat, but right now to get these hints out you need to add an import and use the type (otherwise you get an unused import hint instead).
While these hints resembles a warning, but they are not standard, so if you are working in team where each member uses a different editor, you'd get a lot of confusion because some people see the hints and want to change the code in one way, some people don't see it and want to change it differently.


port drag and drop fix

This PR fixes #11372

Coverage decreased (-0.005%) to 36.01% when pulling 5943f0230ebb7cd004e86e0c19cf2ddacaa69720 on alanren/port-dragNdrop-fix into 09915fe5574caa739f2e2a5e1dfebd2951762cd4 on release/1.20.


Implement server logging for Jupyter

Whenever the jupyter server doesn't start, user's have no idea where to go to find out what the problem is. This item is to just add an output panel that logs all jupyter output.

Validated


docs: Fix typos

Found via codespell

Thanks so much, I'll review this soon.

@AnonymouX47 Done.

:partying_face: :partying_face: :partying_face: :partying_face: :partying_face:


Drop 005-k8s-min.patch

This patch removes openshift/patches/005-k8s-min.patch.
Since we are disabling min version validation by setting
KUBERNETES_MIN_VERSION env value by https://github.com/openshift-knative/serverless-operator/pull/1816,
the patch is no longer necessary.

/cc @skonto @mgencur
https://github.com/openshift-knative/serving/pull/35 verified this.

/lgtm


Can we skip if a document already exist ? ?

i was loading a 128gb file :) it was all going well, almost around 74% done. it failed..
as i read, default it does is overwrite.. is there any way i can restart loading data and skip the existing documents ?

There is no option for this, which format are you using? You can use head or tail to filter lines

elasticsearch_loader --bulk-size 3000 --index rdns --type dnsrecords json rdns.json --lines
that is the command i used..
this is how my json file format looks like.. it is around 128gb..
{"timestamp":"1573629372","name":"<IP_ADDRESS>","hostname":"node-a19.pool-1-10.dynamic.totinternet.net","type":"ptr"}
{"timestamp":"1573636816","name":"<IP_ADDRESS>","hostname":"node-a1a.pool-1-10.dynamic.totinternet.net","type":"ptr"}
{"timestamp":"1573647966","name":"<IP_ADDRESS>","hostname":"node-a1b.pool-1-10.dynamic.totinternet.net","type":"ptr"}

this tool was the only one which allowed me to import this formatting :smiley:  .. i tried various other tools..
i would like to know if i can resume from the place i got error after skipping the existing records.

Try cat rdns.json| tail -n +1000  | elasticsearch_loader --bulk-size 3000 --index rdns --type dnsrecords json - --lines For skipping first 1000 lines

@ikewzer0 any update?

@moshe i deleted it before i saw your solution, i think that should've fixed it though.. gonna run again.. is there anyway i can add a retry for failing ones ? am gonna run the import again

Add --with-retry


Feature Request: modify actions

Hi, thanks for the great package!
Would you consider adding a function to the configuration for modifying the actions before they are sent?

Yea, sure, but could you provide a use case?

Of course, I'd like to modify portions of state depending on whether broadcastchannel is available and don't have a feasible way to separate those pieces otherwise. I.e. if there will be storage limitations that prevent the sending and receiving via the fallbacks for broadcastchannel.

@aohua I see a few issues that might cause someone to want to modify an action before it's shared. Namely general performance, storage limitations when broadcast channel isn't available, and the structured clone algorithm causing errors if some part of an action is not clonable.

@aohua bump


Add filters

To determine if a give req should be intercepted or not.

I could probably work on this as my first feature.

Great. Filters should be added here:
https://github.com/vinci-proxy/intercept/blob/master/request.go#L174
Also, the response interceptor should be refactored to return a compatible interface method instead of a function to achieve this. See the request implementation as reference.

@groyoh do you need some help with this?

I was thinking of something like:
+var DefaultFilter = func(req *http.Request) bool {
+       if req.Method == "HEAD" || req.Method == "OPTIONS" {
+               return false
+       }
+       return true
+}
+

 // RequestInterceptor interceps a given http.Request using a custom request modifier function.
 type RequestInterceptor struct {
        Modifier ReqModifierFunc
+       Filters  []Filter
 }

 // Request intercepts an HTTP request and passes it to the given request modifier function.
 func Request(h ReqModifierFunc) *RequestInterceptor {
-       return &RequestInterceptor{Modifier: h}
+       filters := []Filter{DefaultFilter}
+       return &RequestInterceptor{Modifier: h, Filters: filters}
+}
+
+// Filter intercepts an HTTP requests if and only if the given filter returns true.
+func (s *RequestInterceptor) Filter(f Filter) {
+       s.Filters = append(s.Filters, f)
 }

 // HandleHTTP handles the middleware call chain, intercepting the request data if possible.
 // This methods implements the middleware layer compatible interface.
 func (s *RequestInterceptor) HandleHTTP(w http.ResponseWriter, r *http.Request, h http.Handler) {
-       if r.Method == "HEAD" || r.Method == "OPTIONS" {
-               h.ServeHTTP(w, r)
-               return
+       if s.filter(r) {
+               req := NewRequestModifier(r)
+               s.Modifier(req)
        }
+       h.ServeHTTP(w, r)
+}

-       req := NewRequestModifier(r)
-       s.Modifier(req)
-       h.ServeHTTP(w, req.Request)
+func (s RequestInterceptor) filter(req *http.Request) bool {
+       for _, filter := range s.Filters {
+               if !filter(req) {
+                       return false
+               }
+       }
+       return true
 }

Let me know if I'm on the right track here. I will create a PR once I have the test passing.

Looking good. Just a few tips:
Filter() could allow variadic arguments (in some cases useful), like this:
+func (s *RequestInterceptor) Filter(f ...Filter) {
+       s.Filters = append(s.Filters, f...)
 }

The HandleHTTP for Request is fine, but for Response, be sure you're only hijacking the ResponseWriter if all the filters passes.

Alright, thanks. Another question, is it common in Go to have method chaining e.g. interceptor.Filter(someFilter).ANewMethod(someDifferentFunction)?

It's not too much common, but I personally tend to implement fluent interfaces for particular cases when the API consumer might need it for convenience. The Go stdlib doesn't support method chaining, but they're not providing that kind of APIs.


Does this library have versions?

I was trying to figure out what version of it I had loaded so I can then tell later if it has changed or not.  Do you have a version number on each build of these files?

Just added a tag/release. Will update the package.json version with subsequent releases as well.


Revisit client docs sections organization

This PR closes https://github.com/0xPolygonMiden/miden-client/issues/282 and changes the client docs sections a bit. The goal is to explain the client design, the client as library and the client's CLI.
The new proposed structure is

Overview:

Overview: index.md
Features: features.md

Design:

Overview: design.md
API docs: api-docs.md

Miden client library: 'library.md'
Miden CLI:

Configuration: 'cli-config.md'
Reference: 'cli-reference.md'

Install and run: install-and-run.md
Examples: 'examples.md'

Whereas api-docs.md links to https://docs.rs/miden-client/latest/miden_client/

@Dominik1999 - let's resolve the merge conflict and merge.


Ensure that macro overloading via availability works

This was already fixed by a previous change (#67698), but this introduces a test case to ensure that we don't regress rdar://113047811.

@swift-ci please smoke test


New test framework

Failing test_Test.py with sqlalchemy error, passing locally with the docker db set up.

FYI linter is still failing


[WIP] Double Exponentially Smoothed Model (Holt linear model)

We implement a holt linear model (additive variant), which allows us to capture trend and level. Estimation of parameters implemented with 2 different optimization methods: BOBYQA (bounded optimization by quadratic optimization, derivative-free), and conjugate gradient descent.
The one point I want to highlight, in case I'm missing something obvious, or we want to leave as "unsupported operation" for this model: removeTimeDependentEffects seems to me to be a bit different here. The smoothed values t are a function of the original values at time t - 1, so given a series of length n, if we apply removeTimeDependentEffects, we can only really arrive at series of length n - 1. In this case, I'm simply filling in the last position with an estimate based off of the trend and levels (the components of Holt). Additionally, I've worked through the algebra various times, but the removeTimeDependentEffects, when solving for original values, seem to be slightly off and that error compounds. I believe I did the algebra correctly (and checked various times), but perhaps an extra set of eyes on this one wouldn't hurt.

Oops accidentally closed

If holtwinter branch is merged. I think I can convert the Triple to Double Exponential. The main problem is in defining the structure. I prefer to create new class like HoltWinter or should I change the existing class to handle both Double and Triple Exponential.


fix(junit): Change the regex to match the parent folder

What does this PR do?
Update the regex on the script that replace the file name to take into account numbers in the file name

Motivation
#incident-28346
Junit upload was silently failing on some jobs (example
It was caused by this line that fails at finding the owners of upgrade-agent6_spec.rb .
This was a consequence of this command that doesn’t handle file names with numbers, and this script is supposed to replace the direct file name with full path that would be matched by the CODEOWNERS file

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

/trigger-ci --variable RUN_ALL_BUILDS=true --variable RUN_KITCHEN_TESTS=true --variable RUN_E2E_TESTS=on --variable RUN_UNIT_TESTS=on --variable RUN_KMT_TESTS=on

/trigger-ci --variable RUN_ALL_BUILDS=true --variable RUN_KITCHEN_TESTS=true --variable RUN_E2E_TESTS=on --variable RUN_UNIT_TESTS=on --variable RUN_KMT_TESTS=on

/merge


[dart mode] add "as" to keywords

import "package:foo/foo.dart" as bar; should highlight "as" as a keyword.

Merged as e477745