in the request URL causing user to redirect to different URL

Hi Hanz,
First of all thanks for such an amazing plugin.
Everything works perfectly fine except one use case i am describing below.
I am protecting an angular app which is registered as an confidential app in keycloak. The typical login flow works as expected which is the user clicks on login, redirected to keycloak auth URL, gets authenticated and redirected to dashboard.
Now i have a use case where i am sending emails to the users which contains a URL to a specific report location and that URL contains a "#" pound sign in the URL. e.g.
http:///app/landing.html#/report/list/1234
Now when a user clicks on this link.

User is redirected to keycloak login page since /app/landing.html is protected in mod-auth conf.
User enters the userid/password
User is redirected to just http:///app/landing.html instead of the original URL which was http:///app/landing.html#/report/list/1234

The URL fragment which contains # (i.e. #/report/list/1234) is lost once the user redirects back to app once he is authenticated on keycloak. I more point to note is, once the user is redirected to auth URL, i see that the keycloak auth URL still contains that URL fragment i.e only #/report/list/1234
I have a strong feeling, its not a bug/issue on mod_auth_openidc. it could be a limitation from a browser agent side as well that doesn't respect if the original URL contains a "#" in it and if there is login redirect occurs.
I am just handling security to an angular app so please pardon to my limited knowledge on angular. In angular world, URL fragments represent routes to the app. but if they becomes the original URL(entrypoint), may be because of a user bookmarked that link or static links send to the user through email. This causes URL redirecting to a different URL instead of where the URL should actually be.
Please help if you have any clue whats happening here. I am just clueless.
platform Ubuntu
mod-auth-version  2.3
Application protected type Angular JS
Auth server  keycloak
Expected behaviour
User should be redirected to the actual URL which a user enters in the browser
Actual behaviour
The part of the URL is trimmed after authentication and user is redirected to the dashboard instead of that specific link.
Thanks.
Abhishek

If you can provide server side debug logs that should show whether it is mod_auth_openidc, Keycloack or the browser stripping the hash from the URL. In general I should mention that the fragment gets stripped by the browser and is never sent to the server which means that mod_auth_openidc would never see it (as opposed to what you seem to suggest). Anyhow, server side debug logs ("LogLevel auth_openidc:debug") should tell you what happens.

Thinking this through a bit more...: by the time your browser hits mod_auth_openidc, it should have already stripped the hash fragment so the scenario you describe cannot work. See e.g.: https://stackoverflow.com/questions/9384373/do-browsers-remove-in-url-automatically

Thanks Hanz, I understood the issue. It’s a browser limitation. I have also verified the mod-auth logs as suggested by you and it simply has a trimmed request URL without the fragment.
So we will have to look for a way to somehow pass this in a different way and then append it back once the authentication is successful.
Thanks for your quick support.

Hi abhishekraghav did you find out the way?


Mod incorrectly uploaded to thunderstore

The mod was not correctly uploaded to thunderstore for 1.5.1 update.
1.5.0 Size: 22.5MB
1.5.1 Size: ~200KB
The only thing included in the mod download is the .dll, manifest.json, readme, and icon.png.

really sorry about that, just uploaded the correct version this time.


TypeError: Cannot read properties of undefined (reading 'label')

Hello, I'm using the new menu header slot with a filter (a button switch) which changes the options model.
But when the option ref is updated then I get this error:
Uncaught (in promise) TypeError: Cannot read properties of undefined (reading 'label')
    at default (vue3-select-component.js?v=e162d33e:130:57)
    at Array.map (<anonymous>)

this is my setup:
<OnSelectV5
    v-model="model"
    @search="onSelectSearch"
    :options="options"
    >
    <template #menu-header>
      <<FILTERS HERE>>
    </template>
</OnSelectV5>

const model = ref();

const options = computed(() =>
  store.data.filter((plant: IPlant) => {
      // filter logic based on the filter ref
    })
    .map((item) => ({
      label: item.name,
      value: item.name,
    }))
);

This also happens if I use a ref for options instead of computed and reassign the items to the options ref from a watch

I have cloned the repo and trying to fix this:
should be related to those lines:
    getOptionLabel: (option: GenericOption) => option.label,
    getMultiValueLabel: (option: GenericOption) => option.label,

it seems to be fixed if I put a ? to option.label:
    getOptionLabel: (option: GenericOption) => option?.label,
    getMultiValueLabel: (option: GenericOption) => option?.label,

If I understand correctly this issue, you are dynamically changing the list of options by creating a filter on the #menu-header template.
Here's what I've done to reproduce the issue:
<script setup lang="ts">
import { computed, ref } from "vue";
import VueSelect, { type Option } from "vue3-select-component";

type UserOption = Option<number> & { username: string };

const activeUsers = ref<number[]>([]);
const usersFilter = ref<"it" | "hr">("it");

const itUsers: UserOption[] = [
  { label: "Alice", value: 1, username: "alice" },
  { label: "Bob", value: 2, username: "bob" },
  { label: "Charlie", value: 3, username: "charlie" },
  { label: "David", value: 4, username: "david" },
];

const hrUsers: UserOption[] = [
  { label: "Eva", value: 5, username: "eva" },
  { label: "Frank", value: 6, username: "frank" },
  { label: "Grace", value: 7, username: "grace" },
  { label: "Helen", value: 8, username: "helen" },
];

const userOptions = computed(() => usersFilter.value === "it" ? [...itUsers] : [...hrUsers]);

const switchFilter = () => {
  usersFilter.value = usersFilter.value === "it" ? "hr" : "it";
};
</script>

<template>
  <VueSelect
    v-model="activeUsers"
    :options="userOptions"
    :is-multi="true"
    placeholder="Pick users"
  >
    <template #menu-header>
      <button type="button" @click="switchFilter">
        Switch filter type
      </button>
    </template>
  </VueSelect>
</template>

To reproduce, open the menu, select a value, switch the filter. Check the console, it should have an error.
The issue arise everywhere getOptionLabel() and getMultiValueLabel() are called, because internally the component uses the :options prop to retrieve the label of the option(s):
{
  getOptionLabel: (option: GenericOption) => option.label,
  getMultiValueLabel: (option: GenericOption) => option.label,
}

However, selected options are not passed anymore since we've changed the filter. There are no work-arounds at the time of writing on vue3-select-component.

displayedOptions (new)

Type: Option[]
Required: false

When passed to the component, only these options will be rendered on the menu. However, make sure you are passing all possible options in the options prop.

Hello, thank you for your investigation of the issue.
How displayedOptions should be used? Is it to be released still?
Do you mean we should have both options with all the available options and displayedOptions with only the options we want to show?

That's right, displayedOptions should be used conjunction with options, where:

options has all the possible options (even the ones you don't want to display on the menu) because this prop is used internally to keep track of selected option(s) and ensure the proper rendering of the component.
displayedOptions when passed to the component, will be used to render its options inside the options menu.

I'm pushing this feature on a branch soon. Would you be down to read the changes?

Thank you I will take a look!

Should be fixed on v0.4.0.
See:

https://vue3-select-component.vercel.app/props.html#displayedoptions
https://vue3-select-component.vercel.app/demo/with-complex-menu-filter.html


Updates for BitBucket release note generation.

Based on issues I experienced using BitBucket (see issue : #71) I have applied the following fixes:

Fixed unit tests to use the string, bool overload  for the Categories creation so that the solution compiled
Added xunit runner nuget package to run the unit tests in Visual Studio 2013
SimpleJson.DeserializeObject to dynamic does not work, I used JsonConvert.DeserializeObject (from http://www.newtonsoft.com/json) instead. This was done in order to keep the use of the dynamic object from the API json response. I'm not sure when this changed, but RestSharp needs a line of code uncommented to use dynamic in SimpleJson (see the comment on line 28 from https://github.com/restsharp/RestSharp/blob/master/RestSharp/SimpleJson.cs).
Added IssueResolved constant and checked for both closed and resolved issues to add to the Release Notes
Used arguments.ConsumerKey and  arguments.ConsumerSecretKey if arguments.Username  or arguments.Password are not supplied for OAth. This may be overkill as the keys need to be supplied for OAuth and not the User and password. Please review this and let me know if this should be changed to use only the keys and not username and password as these do not authenticate correctly using the API in any case as far as I could tell.

These changes now enable BitBucket Release notes using the /ConsumerKey and /ConsumerSecretKey switches.

Are these switches on the help /?

That's correct.  I reused the existing ones.
On 30 Mar 2015 09:52, "Jake Ginnivan"<EMAIL_ADDRESS>wrote:

Are these switches on the help /?
—
Reply to this email directly or view it on GitHub
https://github.com/JakeGinnivan/GitReleaseNotes/pull/72#issuecomment-87582775
.

Sweet, thanks

Hi Jake,
When will this be available on the Chocolatey repo?

I need to do a few releases. Maybe tonight or Thursday night, if not it will be Monday (am away over easter). Got a few things I need to do

No problem, I see there is a backlog on moderating the packages anyway so
it seems it will be a while before it is listed.
On Wed, Apr 1, 2015 at 12:52 PM, Jake Ginnivan<EMAIL_ADDRESS>wrote:

I need to do a few releases. Maybe tonight or Thursday night, if not it
will be Monday (am away over easter). Got a few things I need to do
—
Reply to this email directly or view it on GitHub
https://github.com/JakeGinnivan/GitReleaseNotes/pull/72#issuecomment-88438259
.


How to stream image / not read into memory?

Hey 👋🏻
I'm currently annotating an image with the following code:
const imageBuffer = ...

const client = new vision.ImageAnnotatorClient({ ... })

const [results] = await client.annotateImage({
  image: {
    content: imageBuffer,
  },
  features: [...],
})

After scouring through the documentation, I'm not seeing any way to use a stream instead of a buffer, so that the whole image isn't ready into memory. Am I just missing it? Or is it not possible?
Thanks!

Here's an example of specifying a source URI rather than a content buffer. Does that help?

Hmmm. I guess I could use a stream to upload to Google Cloud Storage (like this example), and then specify the gs URL like in the example above?

Yes, that would be the best approach, as we currently don't support streaming in this library.


chore: set warningAsError in CI only

Don't fail local builds because of this

Also set it in nix, so that fails quickly?

Already there. One more good reason for not using Nix for interactive development in fact :) .


UILocalNotification deprecated in 10.0.1

I wonder if any of you have encountered error like this:
when setting title for local notification in iOS 10.0.1 the app crashes. Seems Apple introduced another way of creating notifications

@evollu this is the error I'm getting on 10.1
https://github.com/evollu/react-native-fcm/issues/218

fixed


Link LZ2 into GoC brokerage system

Describe the issue
Currently, our LZ is not "linked" into the GoC brokerage.  This needs to be in place for billing and other business purposes.
Which Sprint Priority is this issue related to?
The 'Milestone' should clearly list Sprint Priorities - which one is this issue related to?
Additional context
Add any other context, attachments or screenshots
Definition of done
Our existing LZ is confirmed to be connected with GoC.

Nothing to do here. Covered by @scayzer


Unexpected CRK exception when using TestPage with Role Center

Hello,
as part of my test I want to test if my new action was added to role center. Thus I defined TestPage "Order Processor Role Center" in my test, called the Trap function and run the page. Result is:

Code:
        OrderProcessorRCTP.Trap();
        Page.RUN(Page::"Order Processor Role Center");

al-0.16.21713
insider bcsandbox:latest

This seems to be a problem in the client, which occurs also when using C/SIDE.

TestClient does not support role center pages, because role centers cannot have running C/AL code and some of the internal structures required by TestClient are not instantiated.
If you would like to push for added support, please add an entry on Ideas https://aka.ms/bcideas and other partners can vote for it as well.

Closing this issue as was marked as by-design and suggested to log it on BC Ideas.


outage RCA April 24 2022  - API Actions, Sidekiq and Cron jobs

Servers became unresponsive after this commit
This was due to a variety of factors:

Since jobs were running both in sidekiq and cron we ran into the thundering herd problem (see sidekiq's solutions to it here) due to various workloads that were in the initialized state that were now being called every minute
As we fell back on jobs, systems started getting slower and slower until the OS crashed
When systems were booted back up (from forced restart) they functioned until sidekiq started processing jobs and some jobs were being rate limited (causing failed jobs) which then got picked up by cron and the server crashed due to just sheer workload.
systems that had the API plugin Subdomain Events (subdomain/event v1) were hit especially hard since that plugin has a bug that spawns multiple sidekiq jobs for a given event

TLDR; it was a workload bomb
The fix was to either boot the server with the faulty code in place and turn off sidekiq.service or to deploy this hotfix
To ensure fastest time to recovery, the following steps were taken:

Launch fresh instances from the latest stable AMI (faster time to recovery) -- to start serving web traffic (full performance was regained at 9.30AM EST)
after production systems were serving web traffic correctly debug validate assumptions and deploy hotfix to staging (11AM EST)
verify for 1 hour on staging
rotate public key secrets in the deployment pipeline
deploy to production environments (completed by 1PM EST)

Web performance is restored, but job performance keeps regressing and affecting EC2 instance health. Our crontab is as follows:
Last login: Sun Apr 24 18:24:39 2022 from <IP_ADDRESS>
ubuntu@ip-172-31-86-57:~$ crontab -l

# Begin Whenever generated tasks for: violet at: 2022-04-24 17:18:35 +0000
PATH=/var/www/violet/shared/bundle/ruby/2.6.0/bin:/home/ubuntu/.rbenv/versions/2.6.6/bin:/home/ubuntu/.rbenv/libexec:/home/ubuntu/.rbenv/plugins/ruby-build/bin:/home/ubuntu/.rbenv/plugins/rbenv-vars/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin

0 0,12 * * * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake maintenance:clear_old_ahoy_visits --silent >> log/whenever.log 2>&1'

0 0,12 * * * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake external_api_client:drive_cron_jobs CRON_INTERVAL=twelve_hours --silent >> log/whenever.log 2>&1'

* * * * * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake external_api_client:drive_cron_jobs CRON_INTERVAL=every_minute --silent >> log/whenever.log 2>&1'

* * * * * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake api_action:run_initialized_actions --silent >> log/whenever.log 2>&1'

0,5,10,15,20,25,30,35,40,45,50,55 * * * * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake external_api_client:drive_cron_jobs CRON_INTERVAL=five_minutes --silent >> log/whenever.log 2>&1'

0,5,10,15,20,25,30,35,40,45,50,55 * * * * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake api_action:rerun_failed_actions --silent >> log/whenever.log 2>&1'

0,10,20,30,40,50 * * * * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake external_api_client:drive_cron_jobs CRON_INTERVAL=ten_minutes --silent >> log/whenever.log 2>&1'

0,30 * * * * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake external_api_client:drive_cron_jobs CRON_INTERVAL=thirty_minutes --silent >> log/whenever.log 2>&1'

0 * * * * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake external_api_client:drive_cron_jobs CRON_INTERVAL=every_hour --silent >> log/whenever.log 2>&1'

0 0,3,6,9,12,15,18,21 * * * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake external_api_client:drive_cron_jobs CRON_INTERVAL=three_hours --silent >> log/whenever.log 2>&1'

0 0,6,12,18 * * * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake external_api_client:drive_cron_jobs CRON_INTERVAL=six_hours --silent >> log/whenever.log 2>&1'

0 0 * * * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake external_api_client:drive_cron_jobs CRON_INTERVAL=one_day --silent >> log/whenever.log 2>&1'

0 0 * * * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake report:send_analytics_report --silent >> log/whenever.log 2>&1'

0 0 * * * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake maintenance:clear_discarded_api_actions --silent >> log/whenever.log 2>&1'

0 0 1,8,15,22 * * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake external_api_client:drive_cron_jobs CRON_INTERVAL=one_week --silent >> log/whenever.log 2>&1'

0 0 1,15 * * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake external_api_client:drive_cron_jobs CRON_INTERVAL=two_weeks --silent >> log/whenever.log 2>&1'

0 0 1 * * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake external_api_client:drive_cron_jobs CRON_INTERVAL=one_month --silent >> log/whenever.log 2>&1'

0 0 1 1,4,7,10 * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake external_api_client:drive_cron_jobs CRON_INTERVAL=three_months --silent >> log/whenever.log 2>&1'

0 0 1 1,7 * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake external_api_client:drive_cron_jobs CRON_INTERVAL=six_months --silent >> log/whenever.log 2>&1'

0 0 1 1 * /bin/bash -l -c 'export PATH="$HOME/.rbenv/bin:$PATH"; eval "$(rbenv init -)"; cd /var/www/violet/releases/20220424171545 && RAILS_ENV=staging bundle exec rake external_api_client:drive_cron_jobs CRON_INTERVAL=one_year --silent >> log/whenever.log 2>&1'

# End Whenever generated tasks for: violet at: 2022-04-24 17:18:35 +0000
ubuntu@ip-172-31-86-57:~$

Looking at it, the only mission critical jobs are the ones related to running failed api actions + reporting tasks. To debug the issue with jobs falling back and killing EC2 instances, I have temporarily removed the crontabs so while we wont be processing those jobs we will at least have stable web performance to buy us some time.

Update after removing the cron tabs,
Some instances still became unresponsive. From here we have 2 strategies to narrow down the issue:

go back, commit by commit on master until we find the point in time where the issue was introduced
assume the issue is with libraries-- and setup a fresh AMI and test

The following commits on master have been deemed stable for production use:

Branch master (at 5642e8c521a3353bf391efeadeb5bcbddfd018bb) - validated on markedrestaurant.com
Branch master (at 3377efc3fe7cd06493e06ecdc76f42fed43cbfbd) - validated on nikean.org
Branch master (at 11548a1a7cdb136467e81604c5a65a0947d99ecb) - validated on sipshucksip.com
Branch master (at 229cd1feb5fb6f259210acf358ca58c12a5eed1f) - validated on sanjaysinghal.com
Branch master (at 3377efc3fe7cd06493e06ecdc76f42fed43cbfbd) - validated on atoi.to

CC @alis-khadka @Pralish

we are now testing app specific cron jobs. Looks like an externalAPIClient is overloading the system

prevented the same externalApiClient from running at the same time
https://github.com/restarone/violet_rails/pull/496
but instances are still becoming unhealthy. Turning off cron on staging but still running the latest commit here -- lets check back after 24 hours to see if the system is stable with the latest changes and cron turned off


Wrong assembly listed

Text says to add Microsoft.Azure.NotificationHubs but that seems to be a server-side component. The screenshot says WindowsAzure.Messaging.Managed which is the correct assembly.

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

ID: 5b48be73-2f29-6b2a-c837-751d3862bcdb
Version Independent ID: efd00d73-1ff5-8689-8415-fe1630795123
Content: Get started with Azure Notification Hubs for Universal Windows Platform apps
Content Source: articles/notification-hubs/notification-hubs-windows-store-dotnet-get-started-wns-push-notification.md
Service: notification-hubs
GitHub Login: @jwhiteDev
Microsoft Alias: jawh

@amp64 Thanks for the feedback. We are actively investigating and will get back to you soon.

@femsulu please assign to @dimazaid

Updated

@femsulu please close.

#please-close


fix: import hydration utils from core in persistQueryClient

Import hydration utilities directly from the core.
Apparently, it caused issues after https://github.com/tannerlinsley/react-query/pull/2497 got merged.

@TkDodo

@cloudmu can you please verify if this PR would fix your issues?

@TkDodo @DamianOsipiuk   Yes. The PR would fix my issues.  Thank you!

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


add resp3 protocol part 6(remove old protocol code)

Pull Request for issue
#76
Description
remoec unused code
Testing Instructions
Excepted Behavior
Actual Behavior
Additional Comments

Tnx on this awesome work


—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or unsubscribe.￼
[ { @.": "http://schema.org", @.": "EmailMessage", "potentialAction": { @.": "ViewAction", "target": "https://github.com/github/docs/pull/5581#event-4622084240", "url": "https://github.com/github/docs/pull/5581#event-4622084240", "name": "View Pull Request" }, "description": "View this Pull Request on GitHub", "publisher": { @.": "Organization", "name": "GitHub", "url": "https://github.com" } } ]


Set encoding explicitly for zip task

This pr could fix problems such as chinese characters in file name change to garbled characters after using zip type task

Thanks for the pull request.  We'll have someone look at it.
In the meantime, can you sign the CLA online at www.gradleware.com/contributor-agreement?

ok, I've signed it

Hi. A few things this functionality would require:

An integration test covering non-ascii characters
A public way to configure the encoding on a Zip task

Let me know if you want to help out with this.

@bigdaz I'm not sure we'd need to make this configurable. We should look into making this the default. This would be doable if the current value is always a UTF-8 subset. It's probably not possible if the current value is dependent on platform encoding.

@bigdaz I prefer configurable way too, and I will do this next.
@alkemist I think somebody like me will need this feature in some case. For example, the platform encoding of chinese version Windows is GBK by default, and the files with chinese characters in name could not unpack from war archive if not set zip task encoding to UTF-8.

@AlphaHinex we don't know what encoding is currently being used. If the current value is the platform encoding, then we can't change to UTF-8. If it however is hardcoded to US-ASCII, then we can.

@alkemist so we do not hardcode it, we use the platform encoding by default and give user a way to configure it, is it a good idea?

From the docs of ZipOutputStream.setEncoding…

Defaults to the platform's default character encoding.

This makes defaulting/hardcoding to UTF-8 a breaking change, but we might be able to live with it.
Options:

Hardcode this to UTF-8 (i.e. what this PR does)
Add an encoding property to Zip that defaults to platform encoding
Add an encoding property to Zip that defaults to UTF-8

2 is backwards compatible. 3 is a better default, but potentially breaking.
My preference is 3.
@adammurdoch ?

@AlphaHinex Do you mind moving forward on implementing option # 3 above?

OK, I will try

Sorry for the long time, but during these days, I've changed my mind. I just add an encoding property for Zip task, and leave it defaults to platform encoding, option # 2 above.

Thanks @AlphaHinex. I think that you are correct to go with keeping the current default behaviour, and making it possible to override with the encoding option.
Before we can merge, we'd need some test coverage for explicitly setting null encoding, as well as a test for failure with an invalid encoding value.
I think the best way to proceed is to close this PR, and open a new one with just the changes required.


Trace function in a dynamic library

Hi, can I set the module option to a dll and setting addr to one of a function in this dll?
I tested in some program with this settings, which result in a huge size of trace file( and need manually stop it to produce more).

@hexa-synacktiv Thanks for your explanations, I think the fitst one may be the problem, I test with a small funciton and it works.


Conditional image generation problem

Hello, your tutorial is great! I have a question I would like to ask, how do I add conditions in the form of pictures when training the second stage transformer

Hey there, first of all I would suggest maybe not using the second stage as described here anymore as there are much better methods now. You can look into papers for MaskGIT (https://arxiv.org/abs/2202.04200), Paella (https://arxiv.org/abs/2211.07292) or MUSE (https://arxiv.org/abs/2301.00704) as they are more efficient. Secondly to answer the question, there are many different ways to add condition. You could condition on CLIP image embeddings through ModulatedLayernorms or CrossAttention. Or you could condition on images by concatenating the images to the latents that you are training on. Let me know if that helps

Hey there, first of all I would suggest maybe not using the second stage as described here anymore as there are much better methods now. You can look into papers for MaskGIT (https://arxiv.org/abs/2202.04200), Paella (https://arxiv.org/abs/2211.07292) or MUSE (https://arxiv.org/abs/2301.00704) as they are more efficient. Secondly to answer the question, there are many different ways to add condition. You could condition on CLIP image embeddings through ModulatedLayernorms or CrossAttention. Or you could condition on images by concatenating the images to the latents that you are training on. Let me know if that helps

Thanks a lot！！！i will try！


Application Control Panel

Implement an automagically generated Control Panel for each application.

Won't do: for now, each application handle its configuration (if any) by itself.


Version 0.2.1 not published to marketplace

While the repository includes v0.2.1 release, marketplace page states v0.2.0 as latest one, and it's non-functional.

Thanks @vandamm for bringing that up.
I have fixed it.


Typed List preinitialization not working

Trying to set a list item at index postion is not working
from numba import int32, int64, float32, float64, optional
from numba import jit, njit, types, typed, prange

@njit
def foo():
    items = typed.List.empty_list(int64, allocated=1000)
    print(len(items))
    items[0] = 5

foo()

I get and index out of range error..

@rpopovici hi and thank you for reporting this. The allocated parameter is intended to pre-allocate memory under the hood for growing the list later. This may be useful for situations where you know you will add X items to list, e.g. during extend or copy, and you don't want to incur the constant growth of the allocated number of items under the hood. It doesn't work like np.empty in this sense that the length of the list is X and the items are heap trash. However, I deliberately said intended -- since this doesn't quite work properly yet, due to the reallocation strategy of the list, so it's probably best to avoid this for now.

@esc thanks for the clarification

Closing this question as it seems to be resolved. Thanks!


server tasks nesting is borked

I am quoting @at055612

I think so.  Either it is an issue with the display or the task context hierarchy

Seen in dev

Aggregate SQL Statistics should be a child of SQL Stats Database Aggregation


[Debugger] Allow root array in request JSON to be optional

Motivation
The two Debugger endpoints /debugger/v1/input and /debugger/v1/diagnostics both accept an object optionally wrapped in an array. However, the system tests currently expects the array-wrapper. The intent of this PR is to update the system tests to also make the outer array-wrapper optional.
Changes
This is done my modifying the two JSON Schemas governing the two endpoints to use a oneOf to make the containing body either be as-is, or wrapped in an array.
Workflow

⚠ Create your PR as draft ⚠
Work on you PR until the CI passes (if something not related to your task is failing, you can ignore it)
Mark it as ready for review

Test logic is modified? -> Get a review from RFC owner. We're working on refining the codeowners file quickly.
Framework is modified, or non obvious usage of it -> get a review from R&P team

:rocket: Once your PR is reviewed, you can merge it!
🛟 #apm-shared-testing 🛟
Reviewer checklist

[ ] If PR title starts with [<language>], double-check that only <language> is impacted by the change
[ ] No system-tests internal is modified. Otherwise, I have the approval from R&P team
[ ] CI is green, or failing jobs are not related to this change (and you are 100% sure about this statement)
[ ] A docker base image is modified?

[ ] the relevant build-XXX-image label is present

[ ] A scenario is added (or removed)?

[ ] Get a review from R&P team

#3583
#3582
#3581
#3577  👈 (View in Graphite)
main

This stack of pull requests is managed by Graphite. Learn more about stacking.


Allow watching tests with supertest

It is mostly about lines 178, 179 but the rearrangement and return statement won't hurt I guess.

Coverage decreased (-34.2%) to 47.331% when pulling 632e4db0328df3c1072642d2c4fe1ac3ef487766 on hoetmaaiers:master into 75e7b4781f2e85cba0213c367f850b585eb77458 on knor-el-snor:master.

Found no change in tree-house is needed. Just don't fireupEngines when NODE_ENV === 'test'


Unable to configure that on Windows 10

I tried to use that approach by modified my git global config, but nothing seems to be working

@ahmadawais Can you please help me how to configure that & use it ?
Thanks !

I'm super interested in helping you work this out. For me to do that, you have to help understand each step you took. What command you ran how you ran it what error you saw. "Nothing's working" is not helpful information.

@ahmadawais Can you please guide me what's the setup process. I just tried by adding the above snippet in my git global config & on using that command git new getting this information...

I don't have windows so I can't test this but have you tried restarting and reloading the terminal? Looks like your git aliases were not registered.
I recommend using the .bashrc workflow functions
#.# Better Git Logs.
### Using EMOJI-LOG (https://github.com/ahmadawais/Emoji-Log).

# Git Commit, Add all and Push — in one step.
function gcap() {
    git add . && git commit -m "$*" && git push
}

# NEW.
function gnew() {
    gcap "📦 NEW: $@"
}

# IMPROVE.
function gimp() {
    gcap "👌 IMPROVE: $@"
}

# FIX.
function gfix() {
    gcap "🐛 FIX: $@"
}

# RELEASE.
function grlz() {
    gcap "🚀 RELEASE: $@"
}

# DOC.
function gdoc() {
    gcap "📖 DOC: $@"
}

# TEST.
function gtst() {
    gcap "✅ TEST: $@"
}```

@ahmadawais I solved this issue & found something, when we are using command git config --global --edit for editing the config file then on the vim type editor when we copy the snippet so instead of placing the correct image, it's placing the some garbage type values as you can see in the below image
..
After saving the changes, when I tried the command so it returned the correct message but the image wasn't showing, the same type of pattern ~_~ pattern was appearing on vscode logs as stored ...
So, correct approach is to edit config file by the command notepad C:/Users/{name}/.gitconfig, it will open your notepad so that you can easily paste the commands with having image in the right format.
Now, it's working perfect !

What if I just modify the docs with this information through a PR ... :)

Feel free to send in a PR. Does look like a Windows problem but maybe it's not.
Looking forward, peace! ✌


requesting authorization code instead of access token

Is it possible to request authorization code instead of access token as return
value?

My application needs to run a long-running job in the backend server that
should access user's Google Spreadsheet once a day.

In such a case authorization code works better than access token, as the
following (See "Response_type parameter") mentions.
http://code.google.com/apis/accounts/docs/OAuth2Login.html

Thanks!
Hyun

Original issue reported on code.google.com by<EMAIL_ADDRESS>on 15 Feb 2012 at 7:07

Is tbjere a download to walk mee through?


Improve performance of object-to-object code paths

Closes #1773

:herb:

Regarding the original issue raised by @jdalton, I'll see what it takes to incorporate the workarounds into the new helper function (without hurting performance). I'm personally more worried about the Safari bug, which seem to apply to the current browser version (I'll have to test, but the code says Safari 9). IE8 is a lost cause.
If it plays out, I may also fix evolve and other functions to use the new helper. Currently they do not apply any of these workarounds.

Too long abandoned.  If we need this, we should start afresh.


Updated the FAQ to include support for Windows 11.md

Updated the FAQ to include support for Windows 11
Description
Updated the FAQ to include support for Windows 11 users facing ERROR: Could not install packages due to an OSError: [WinError 3] The system cannot find the path specified.
Tests

[X] no new tests required
[ ] new tests added
[ ] existing tests adjusted

Documentation

[X] no documentation changes needed
[ ] documentation added or edited
[ ] example notebook added or updated

Thanks for submitting this PR @DarrenTurchiarelli !
Correct me if I'm wrong but this seems unrelated to PyRIT. Can you perhaps elaborate?


[WIP] model struct tag validator

Where are your tests?

@robbyoconnor This is why i added work in progress in the title. Those need to be added and probably other improvements, after it has been reviewed.

Configure your editor to run goimports on save :)

@robbyoconnor Tests and benchmarks added.


There should be one library_googletest.timestamp and it should be shared accross builds

solved in the library management branch


problems about convmixer

Hello, when I use ConvMixer, I always have the problem of conv2d(): argument 'padding' (position 5) must be tuple of ints, not str

Do you know it？ Thanks a lot

Really? It works fine in my env.
Can you tell me your torch version?

my torch is 1.7.0 in cuda 11.0
---- Replied Message ----
| From | Kentaro @.> |
| Date | 05/30/2022 08:46 |
| To | @.> |
| Cc |<EMAIL_ADDRESS>|
| Subject | Re: [kentaroy47/vision-transformers-cifar10] problems about convmixer (Issue #12) |
Really? It works fine in my env.
Can you tell me your torch version?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

I try it in torch 1.10.0, and it can work now. Thanks!


English corrections in docs

Hey! Thank you for vue-draggable-plus ❤
I made some small corrections in the English translation of the docs.

Thank you for your correction.


go/parser: accepts floating point literals with no decimals after E

package main

const X = 0E

$ go tool compile x.go
x.go:3: malformed floating-point constant exponent
x.go:3: malformed constant: 0E
$ gotype x.go
$

gccgo also rejects this code. gc and gccgo are correct. From the spec's floating-point literal definition: exponent  = ( "e" | "E" ) [ "+" | "-" ] decimals .
cc @mdempsky @griesemer
Found using go-fuzz (@dvyukov).

holy cow!


also run contracts test on latest commits

to be planned...

The contracts are up-to-date and still require patching. The idea is only feasible otherwise.


query parameter support (qp) in dc monitor and ui

DcChecker now checks the sync according to the qp
/proc/dc/syncName: can now parse the sync alias name from /meta/sys/dc/syncName and show the correct lastIndexTime taking into account also the qp

Fixes #138 and #142

@yanpom to test and update testing status to Done once you finish the testing


Add a placeholder gen_tokens.zapt file to src/app/zap-templates in order to avoid build breakage when creating a new zap based accessory

Problem
gen_tokens.h is one of the file required by src/app but that it still missing a template. If someone tries to create a new app from scratch using ZAP it will complains that gen_tokens.h is missing.
This PR introduce a simple template based on the content of gen_tokens.h from what is in the tree. The tokens list is empty because the tokens code has not yet been fixed in src/app and so nobody uses it at the moment.
Most of the changes from this PR comes from syncing the tree with the actual zap templates.

@vivien-apple sorry - conflicts here :(

@saurabhst @jelderton ?


Translate into Vietnamese

Hi man, I am Thuan. Currently I'm a web developer using Rails and Angular. I usually use your gem to build my projects. It's very useful. But not able to Vietnamese customers. So I translated your gem. So please accept my pull request to make your gem more useful for more people, especially with the Vietnamese. Thank you!
I have passed all unit test.

Thanks! I checked some of them with Google Translator and they are ok!


Question about symmetric coloring

For the following matrix, how can I know if I should use the "compressed columns" in the AD context associated to colors[i] or colors[j] to determine the coefficient H[i,j]?
10×10 SparseMatrixCSC{Float64, Int64} with 58 stored entries:
 2.0  1.0  1.0  1.0  1.0  1.0  1.0  1.0  2.0  2.0
 1.0  2.0   ⋅    ⋅    ⋅    ⋅    ⋅    ⋅   1.0  1.0
 1.0   ⋅   2.0   ⋅    ⋅    ⋅    ⋅    ⋅   1.0  1.0
 1.0   ⋅    ⋅   2.0   ⋅    ⋅    ⋅    ⋅   1.0  1.0
 1.0   ⋅    ⋅    ⋅   2.0   ⋅    ⋅    ⋅   1.0  1.0
 1.0   ⋅    ⋅    ⋅    ⋅   2.0   ⋅    ⋅   1.0  1.0
 1.0   ⋅    ⋅    ⋅    ⋅    ⋅   2.0   ⋅   1.0  1.0
 1.0   ⋅    ⋅    ⋅    ⋅    ⋅    ⋅   2.0  1.0  1.0
 2.0  1.0  1.0  1.0  1.0  1.0  1.0  1.0  2.0  1.0
 2.0  1.0  1.0  1.0  1.0  1.0  1.0  1.0  1.0  2.0

julia> colors = symmetric_coloring(H, GreedyColoringAlgorithm())
10-element Vector{Int64}:
 1
 2
 2
 2
 2
 2
 2
 2
 3
 4

It depends which one is the only nonzero coefficient in its group of columns:
help?> ADTypes.symmetric_coloring
  symmetric_coloring(M::AbstractMatrix, ca::ColoringAlgorithm)::AbstractVector{<:Integer}

  Use algorithm ca to construct a symmetrically structurally orthogonal
  partition of the columns (or rows) of the symmetric matrix M.

  The result is a coloring vector c of length size(M, 1) == size(M, 2) such
  that for every non-zero coefficient M[i, j], at least one of the following
  conditions holds:

    •  column j is the only column of its color c[j] with a non-zero
       coefficient in row i;

    •  column i is the only column of its color c[i] with a non-zero
       coefficient in row j.

The star coloring algorithm guarantees that at least one of the conditions will be met.

Thanks for your answer @gdalle!
I found a few examples for symmetric_coloring where the result is not optimal.
It could be useful for you.
I understand that we use an heuristic but is it normal that the coloring to be optimal for a 2x2 matrix?
julia> A = sparse([0.0 1.0; 1.0 0.0])
2×2 SparseMatrixCSC{Float64, Int64} with 2 stored entries:
  ⋅   1.0
 1.0   ⋅

julia> algo = GreedyColoringAlgorithm()
GreedyColoringAlgorithm(SparseMatrixColorings.NaturalOrder())

julia> c2 = symmetric_coloring(A, algo)
2-element Vector{Int64}:
 1
 2

julia> A = [0.0 0.0 1.0; 0.0 1.0 0.0; 1.0 0.0 0.0]
3×3 Matrix{Float64}:
 0.0  0.0  1.0
 0.0  1.0  0.0
 1.0  0.0  0.0

julia> A = sparse(A)
3×3 SparseMatrixCSC{Float64, Int64} with 3 stored entries:
  ⋅    ⋅   1.0
  ⋅   1.0   ⋅
 1.0   ⋅    ⋅

julia> c2 = symmetric_coloring(A, algo)
3-element Vector{Int64}:
 1
 1
 2

julia> A = [0.0 0.0 0.0 1.0; 0.0 0.0 1.0 0.0; 0.0 1.0 0.0 0.0; 1.0 0.0 0.0 0.0]
4×4 Matrix{Float64}:
 0.0  0.0  0.0  1.0
 0.0  0.0  1.0  0.0
 0.0  1.0  0.0  0.0
 1.0  0.0  0.0  0.0

julia> c2 = symmetric_coloring(A, algo)
4-element Vector{Int64}:
 1
 1
 2
 2


Do not add anchor to URL for document (root) titles

We already had a flag for menu entries to mark them as "document root", but it was always set to false. This PR sets it to true when then menu entry references a document root title (in contrast to a sub title within that document).
I've also updated the renderer to not add the anchor when it's a root title. This makes the UX nicer, as you don't need to jump to the document root when you're just navigating to the document start.

I am skeptical about this. Linking to a HTML page instead of the title only works in single rendering mode. Not when all HTML output is rendered in a single page.
Also at least in TYPO3 we treat sections as independent and they can and are moved around. So we frequently link to them by anchor.
While I do agree to appropriately setting the flag I would not remove the anchors. If for some reason a document does not want to render them, that can be decided in the template. However we do need anchors on a documents main section and use them frequently

With https://github.com/phpDocumentor/guides/pull/799 Menu entries for pages and sections are different Node types. You should therefore be able to define different templates for them.

@wouterj closing this, you should be able to solve it with the templates now?


Make FlyingBodySpawner's direction points to the Moon

In the previous iteration, the flying bodies just flight in a straight horizontal line, perpendicular to the FlyingBodySpawner's current vertical position, unless the player changed their route. I friend suggested that it could be a good thing to create a feeling of urgency by making the Comets and ShootingStars always aim the Moon by default.
So instead of this:

Have this:

This creates some interesting emerging gameplays:

Player will have to actively decide to change the flying route both to

Make flying bodies collide with the Moon, creating new Craters that will decrease the increasing the player's survivability
Prevent collisions between flying bodies and the Moon, and potentially the Astromouse as well, increasing the player's survivability

Actions per minute may increase significantly

Cons:

Screen may not have enough room and can potentially cause frustration due to claustrophobic gameplay. Mitigation: decrease Moon and related elements. Consequence: more negative space to fill.
Player will now have to manage asynchronous playstyle and each failed management will pile up to inevitable lost. Mitigation: decrease apm by increasing the spawning interval. Consequence: potential increase in boredom due to apm drop.

Closing this for now until I get some player's feedback


Could you add how to use css-modules with CRA into the readme?

Trying to figure out how to use css-modules with CRA, all I find is a Dan's tweet saying it can now, pointing to a pull request. I tried, but failed.
https://codesandbox.io/s/vyzlpplnzy

It works fine locally, it's a code-sandbox problem.  I've downloaded your code and just did a yarn start.  Also, you seem to be using an older build.  Consider switching to a newer one.
https://github.com/facebook/create-react-app/issues/3815

@bugzpodder Thanks, by older build do you mean my syntax outdated, or codesandbox using an old version of react-scripts?

I meant older release of create-react-app (although css modules still works using it) that was referenced in code-sandbox, ie in your link it was using 2.0.0-next.b2fd8db8.
code-sandbox just doesn't seem to likee it even when I switched to the latest version.

Cool, glad to know it works and the problem is from codesandbox. I'm closing this. Thank you.

This shouldn't be closed as there is no documentation on using CSS modules in create-react-app. Is there plans to document this feature at some point?

Once create-react-app 2.0 is released.  The documentation currently does not include CSS modules so not to confuse users on 1.14


ACOS Client Fix

When testing the SSL stuff, we had issues talking to the AX when we structured this as vport.  I don't know if this is an API version issue or ?  Using 'port' appeared to address those issues.

Closing this PR so it has a clean history when merged.


Some question about eap-md5

I am interested in the auth of h3c, but I am stuck in the md5 part. I have tried md5(hex(id + password + challange)), but the answer is not matched with the eap response I got from wireshark. Can you point out the question?

Remove the hex.

The challenge sent by auth server is in hex mode, but the password is written with string, if I remove the hex, which format I should use? Is it means I should transfer all of them from hex to string? I have tried a lot of method, but I can not solve it.
Thanks for your reply.

The challenge sent by auth server is in hex mode, but the password is written with string, if I remove the hex, which format I should use?

Just append them together! You don't need any "format"!.
https://github.com/zonyitoo/sysuh3c/blob/cpp0x/src/eapauth/eapauth.cpp#L104-L106


Multiple imports from CSV with parent/child skews the output

Repro:
Import the following draw.io CSV file, then import it again with new ID values (e.g., append a zero to each ID and parent ID value).
The result is that imports after the initial import cause the child items inside the container to all be skewed inside the container by the x,y value of the parent item.
## Shapes and their styles
# stylename: shapeType
# label: %name%
# ignore: id
# identity: id
# left: xposn
# top: yposn
# parent: rparent
# parentstyle: shape=rect;whiteSpace=wrap;html=1;horizontal=1;horizontalStack=0;resizeParent=1;fillColor=#F0F8FF;strokeColor=#666666;perimeter=rectanglePerimeter;container=1;recursiveResize=0;collapsible=0;
##*************
id,rparent,name,shapeType,xposn,yposn
a0,1,TopBox,rect,300,200
a1,a0,Name1,rect,50,80
a2,a0,Name2,rect,100,120
a3,a0,Name3,rect,150,80

I am also facing some similar issue, using the following csv file:
## name,parent,egress,ingress,shape
# label: %type%<br>%name%
# style: %shapes%
# identity: name
# parent: parent-application
#
# parentstyle: %shapes%
#
# connect: {"from":"egress", "to":"name", "style":"curved=1;endArrow=blockThin;endFill=1;"}
# connect: {"from":"ingress", "to": "name", "invert": true, "style":"curved=1;"}
#
# namespace: csvimport-
# width: auto
# height: auto
# padding: -12
# ignore: shape,ingress,egress
# nodespacing: 40
# levelspacing: 100
# edgespacing: 40
# layout: verticaltree
## ---- CSV below this line. First line are column names. ----
name,parent-application,egress,ingress,type,shapes
k8s-multi-tenant,,"","",k8s,"fontSize=22;strokeColor=#326ce5;imageBorder=#326ce5;image=https://kubernetes.io/images/favicon.png;style=label;html=1;imageAspect=1;whiteSpace=wrap;fillOpacity=30;container=1;dashed=1;strokeWidth=6;"
istio-system,k8s-multi-tenant,"","",ns,"fontSize=17;strokeColor=#326ce5;imageBorder=#326ce5;style=label;html=1;imageAspect=1;whiteSpace=wrap;fillOpacity=30;container=1;dashed=1;strokeWidth=6;"
istio-prodgateway,istio-system,"","",deploy,"shape=image;fontSize=11;image=https://raw.githubusercontent.com/kubernetes/community/master/icons/svg/resources/labeled/deploy.svg;style=label;html=1;imageAspect=1;aspect=fixed;"
kube-system,k8s-multi-tenant,"","",ns,"fontSize=17;strokeColor=#326ce5;imageBorder=#326ce5;style=label;html=1;imageAspect=1;whiteSpace=wrap;fillOpacity=30;container=1;dashed=1;strokeWidth=6;"

@varunpalekar I don't know whether my code ever got pulled in. If you want to try it, it's at the following link. It's just 3 lines of code, so it's probably easiest to just manually repeat it on the newest code base.
https://github.com/jgraph/drawio/compare/master...GaTechThomas:master

Thanks @GaTechThomas I will try this and let you know 👍

Any luck?

Hi,
I found this issue while searching if there is a way to create a container shape in a csv import. I cannot find any reference to the "parent" configuration statements in the examples (found above)
# parent: parent-application
#
# parentstyle: %shapes%

I would be very interested in knowing how to use these statements, if they are supported (even if a bit buggy).
Is this something that one can experiment with?
Cheers,
Christophe-Marie
By the way: thanks for the csv import! I wrote a small python library to export a python networkx graph in drawio and I will opensource it soon 🙂

In case it helps... I see that the link in my comment earlier in this thread is not displaying any code changes. Here's a link that shows the change... at least it shows the diffs today. 🙂
https://github.com/GaTechThomas/drawio/commit/101a230dac410d2c039cbc27ab2979f8d94755ef?diff=unified

@GaTechThomas this does help tremendously! It looks like this parent feature is in the master branch, so even if not documented, I can try it! Thank you for pointing me to the part of the code which reads the CSV, it helps to understand how it works exactly 🙂

See also https://groups.google.com/g/drawio/c/pDGf3gv5X8w/m/TiNg-oMZBQAJ

This looks fixed. Any issues please reopen new ticket.


Perform structural fallbacks for reliable variance results less liberally

Only performing structural fallbacks of variance-based comparisons if the comparison that actually failed is the unreliable/unmeasurable one allows us to utilize variance results more often, which should be good for performance.
This does not, ultimately, fix #54542 in the case of lodash, unfortunately (I had to narrow the change a bit to fix an issue some tests caught, so I'm still back to looking at other ways to attack the underlying issue of "way too many instantiations required to check an insanely complicated type hierarchy"), but may fix (well, hide, really) other reports.

@typescript-bot test this
@typescript-bot run dt
@typescript-bot test top100
@typescript-bot perf test this


Fix ManagerRefresh::Target spec contexts

The .load context included the .dump, .load_from_db, .id, and .name (all
of the contexts below it.  I suspect this was an accident and these are
indented to be defined at the same level as the .load context, @Ladas?

@agrare Not sure if this needs a rebase or what.

I'll rebase but the test failure looks unrelated, I've been seeing this one on and off https://travis-ci.org/ManageIQ/manageiq/jobs/298928343#L1040-L1046
  1) MiqExpression#reporting_available_fields lists custom attributes in ChargebackVm
     Failure/Error: expect(displayed_columms).to match_array(expected_columns)

       expected collection contained:  ["ChargebackVm-chargeback_rates", "ChargebackVm-cpu_allocated_cost", "ChargebackVm-cpu_allocated_metr...rsion", "ChargebackVm-vm_guid", "ChargebackVm-vm_uid", "Vm-virtual_custom_attribute_my_attribute_1"]
       actual collection contained:    ["ChargebackVm-chargeback_rates", "ChargebackVm-cpu_allocated_cost", "ChargebackVm-cpu_allocated_metr..._cost", "ChargebackVm-vm_guid", "ChargebackVm-vm_uid", "Vm-virtual_custom_attribute_my_attribute_1"]
       the missing elements were:      ["ChargebackVm-virtual_custom_attribute_version"]
     # ./spec/lib/miq_expression_spec.rb:25:in `block (3 levels) in <top (required)>'

@agrare it's a sporadic failure @lpichler is looking on

CI failure fix: https://github.com/ManageIQ/manageiq/pull/16418

Thanks @lpichler !


Possible Bug with eqeqeq and eqnull

Noticed something odd parsing jQuery (http://ajax.googleapis.com/ajax/libs/jquery/1.7.1/jquery.js) at jshint.com.
Yesterday, jshint reported so many eqeqeq errors that it stopped parsing at maxerr.
Today, it seems to be missing most of them:
A grep of the source reports 34 instances of /\s==\s/ (32 are /\s==\snull/), but jshint reports only 2 occurrences.
A grep of the source reports 30 instances of /\s!=\s/ (27 are /\s!=\snull/), but jshint reports only 1 occurrence.
I've also noticed that today the jshint string in the report is '/*jshin */', but yesterday it was something like '/*jshint forin:true, noarg:true, noempty:true, eqeqeq:true, bitwise:true, strict:true, undef:true, curly:true, browser:true, indent:4, maxerr:50 */'.
This is my second day with jshint, so it could be me. Can anyone else confirm this problem?

UPDATE
I built a very simple HTML page to test JSHINT. When I parse the jquery source on my test page, JSHINT again finds so many eqeqeq errors that it stops - just like it did the first day on jshint.com.
Perhaps the problem is actually with the website (jshint.com) rather than the source code (jshint.js). I will continue researching this issue...

Sorry, unintentional issue close.

maxerr by default is set to 50.
Try adding a /*jshint maxerr: 500 */ at the top of the file and you'll get 500 errors instead.

Greetings Josh,
Thanks for the reply. Please let me clarify. I followed your suggestion,
but the problem is not maxerr. It appears jshint.com is detecting only a
few of the dozens of eqeqeq and eqnull errors parsing the jquery source
code. As stated in the issue report, the first day I used jshint.com, it
did detect all the errors (after increasing maxerr). However, the next day
and ever since, it only detects a few regardless of maxerr. If I parse the
source in a standalone page or via a node script, all the eqeqeq and eqnull
errors are detected.
I still looking into this - it's very odd. I find I can't parse a large
source at jshint.com while using firebug with firefox. And also, there
seems to be odd behavior with the option combinations - some combinations
detect most/all the errors; others do not (of course eqeqeq and eqnull are
set). Need to dig further to find something definitive...
Clayton
On Fri, Dec 23, 2011 at 7:41 PM, Josh Perez <
<EMAIL_ADDRESS>
wrote:

maxerr by default is set to 50.
Try adding a /*jshint maxerr: 500 */ at the top of the file and you'll
get 500 errors instead.

Reply to this email directly or view it on GitHub:
https://github.com/jshint/jshint/issues/390#issuecomment-3266242

To assist with confirmation, I created some gists to demonstrate:

https://gist.github.com/1517524 - jquery source file with /*jshint */ header
https://gist.github.com/1517530 - simple web page JSHINT tester
https://gist.github.com/1517533 - node script JSHINT tester

I find both of my testers agree with a simple grep on the source file: '!=' occurs 31 times and '==' occurs 34 times.
jshint.com (in Firefox and Chrome), however reports: '!=' occurs 1 time and '==' occurs 3 times. Also, the report cites an option comment of '/*jshin */', rather than echoing the comment in the source file.
I'll keep researching this. Since I can't seem to get jshint.com and Firebug to co-exist in Firefox, I try using Chrome...

UPDATE
For some reason, http://www.jshint.com/src/js/jshint.js is different from https://github.com/jshint/jshint/blob/master/jshint.js. The version on jshint.com causes the error when forin:true - on the website, my standalone page, and the node script. The Git version works regardless of forin. Why is website's script not up-to-date?

UPDATE
I traced down the area in the jquery source that was causing the error. Again, I must emphasize that the problem only occurs with jshint.js on jshint.com, not the latest jshint.js from Git.
Concise Problem Statement
When forin:true is specified and jshint encounters a for...in statement with and empty block, it gets confused and appears to terminate normal parsing, although it does not appear to throw an exception.
Demonstrative Examples
Running this simplified example at jshint.com with all options unchecked:
/*jshint eqeqeq: true, forin: true */
function test( obj ) {
    var key, temp;
    for ( key in obj ) {}

    return key === undefined || hasOwn.call( obj, key );
}

The report shows a mangled /*jshin */ string and a meaningless 'Line 6: s is undefined' error. However, eliminating the empty block:
/*jshint eqeqeq: true, forin: true */
function test( obj ) {
    var key, temp;
    for ( key in obj ) {temp = key;}

    return key === undefined || hasOwn.call( obj, key );
}

The report correctly identifies a forin error on Line 4.

Moved to jshint/site Issue #36 - Please Update jshint.js.


keyboard-layout-editor.com file

Does anyone have a layout file for keyboard-layout-editor for a Hillside keyboard?

Actually re-opening. If I paste https://raw.githubusercontent.com/wiki/mmccoyd/hillside/hill_46/keyboard-layout-editor.json or https://raw.githubusercontent.com/wiki/mmccoyd/hillside/hill_52/keyboard-layout-editor.json into the </> Raw Data tab here http://www.keyboard-layout-editor.com/#/ I get an empty square.

You need to save locally, then use the Upload JSON a the bottom of that tab.


Sample showing publish status via modulo

Q
A

Bug fix?
no

New sample?
yes

Related issues?
NA

What's in this Pull Request?
This sample shows the use of modulo function % to highlight published page versions

Thanks Paul! Great sample!!


Maybe add Collect task that update manually

There are several options:

Add parameter which block auto updating of prompt
User can manually update prompt

listener


fix(daemon, controller): change nodeName to hostname in disk and BD

earlier node name was used as kubernetes.io/hostname, in disk and blockdevices. But in some cloud providers like AWS, node name and hostname are not same. Therefore, now we fetch the nodename using downward API, and then get the hostname label set on that node.

Codecov Report

Merging #290 into master will decrease coverage by 0.23%.
The diff coverage is 21.73%.

@@            Coverage Diff             @@
##           master     #290      +/-   ##
==========================================
- Coverage   52.17%   51.94%   -0.24%
==========================================
  Files          51       51
  Lines        2386     2395       +9
==========================================
- Hits         1245     1244       -1
- Misses       1042     1052      +10
  Partials       99       99

Impacted Files
Coverage Δ

cmd/ndm_daemonset/controller/controller.go
16.07% <21.73%> (-2.38%)
:arrow_down:

Continue to review full report at Codecov.

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


Support FreeSurfer Point Sets

FreeSurfer allows users to create Point Sets, these are JSON files that share many properties with NiiVue connectome nodes. It would be great to initially read and eventually write this format.

I have added the ability to read and edit these files. I have also added a minimal demo that loads a control points file created by FreeSurfer and allows the user to change the scale and color. The user can also move the crosshairs to a specific control point or add and remove control points.
git clone --branch drawing git@github.com:niivue/niivue.git
cd niivue
npm install
npm run demo

And then open the FreeSurfer point sets demo web page.


API for deletion

I wonder what API we should use for element deletion. E.g. to remove an affiliation or an author. Since we have an id for everything we could do something like doc.delete('aff-2') or affNode.delete() and automagically remove that entry from aff-group.childNodes.
If we want to be more explicit and follow the DOM APi we would rather do:
affGroup.removeChild(affNode)

Or we do it similar to jQuery but with an explicit delete call that takes a selector.
doc.delete('aff[id=aff-1]')

Thoughts?

I think in the local case it make sense to have removeChild()
The latter clashes with our existing doc.delete(nodeId).
What about
doc.findAll('...').forEach((el) => { el.remove() })

Thinking about it, this is a bit more involved. E.g., there is a difference between delete and remove:
Should el.removeChild() also delete the child? To be consistent, appendChild() would then need to create nodes (if not yet created)...
Otherwise, we could separate deletion and removal. The nodes would still exist in the document -- which I think would be more consistent with the DOM -- but in our case we would need some garbage collection after some time.
@michael what is ur opinion?

I think that we shouldn't create/remove children. It's more convenient to
have the same methods which natural DOM have, let's not create a mess. I
would prefare other method which will remove element from children and
delete those nodes.
On Sun, 18 Jun 2017, 12:16 Oliver Buchtala<EMAIL_ADDRESS>wrote:

Thinking about it, this is a bit more involved. E.g., there is a
difference between delete and remove:
Should el.removeChild() also delete the child? To be consistent,
appendChild() would then need to create nodes (if not yet created)...
Otherwise, we could separate deletion and removal. The nodes would still
exist in the document -- which I think would be more consistent with the
DOM -- but in our case we would need some garbage collection after some
time.
@michael https://github.com/michael what is ur opinion?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/substance/texture/issues/145#issuecomment-309265928,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AALG-ol_nz7St1QrcWmxxhtN0Pl2nGo4ks5sFOrTgaJpZM4N9cOm
.

Just a reminder: after this we should look again at removing affiliations/contributors. Currently there are some problems when you try to add new one after deletion.

Hm.. so in the native DOM API you can create elements in memory, which are garbage collected (exist until last reference is deleted). They become part of the DOM only after appendChild, they get removed from the DOM on removeChild. In our case doc.create makes them part of the doc already. However, it becomes only visible when linked to the root (article) node.

I think it's fine to have two separate operations parent.removeChild(node) and doc.delete(nodeId) —  e.g. that's handy when we want to move a node somewhere (oldparent.deleteChild, newParent.appendChild)
in substance every element has an ID, so our API's could support parentNode.deleteChild(nodeId) instead or in addition to parentNode.deleteChild(node)
Think I don't like doc.findAll('...').forEach((el) => { el.remove() }) always found it kind of weird in jQuery to tell an element to delete itself.
Should we allow deletion before removal? E.g. doing doc.delete('aff-1') would mean affGroup has an invalid reference... unless we auto-delete that child or throw an error if you attempt to delete a node that's accessible through the document's root node
should we allow multiple root nodes? ;) E.g. we could have multiple article nodes in one document, and those could reuse the same elements by referencing them.

I think relying on garbage collection is ok. Depending on the underlying persistence implementation, this could happen pretty naturally, e.g. when using XML for long term persistence
On a lower level, I still need to old create/delete API. But I understand, as Daniel said, that this is rather confusing, on the higher level. In a pure XML world, it would be sufficient to have dom.createElement(), el.appendChild(), and el.removeChild(). I need to think, if it was possible to hide the old API.
Multiple parentNodes are problematic, as some of the DOM API such as cssSelect makes use of parentNode.

In this regard, we also need to rethink the DOMChange reflection model. As insertion and removal are now the more relevant operations, as opposed to create and delete.

Added removeChild() and making use of it in: c18fdead5f57aa8763574f91c3b19aa5b8088ab4


ChunkedWriteHandler blocking IO [-> Unified pipe abstraction]

Hi,
the ChunkedWriteHandler lets one transfer huge data sets without getting an OutOfMemoryException or something like that.
But the ChunkedInput has a blocking interface for getting the next chunk. If the ChunkedInput has a slow source, this blocks all channels served by the same event-loop.
Are there any plans for implementing some sort of unifying Pipe-abstraction, like node.js's pipe() ? This would also relate to piping channels to other channels, while respecting flow-control. I know that vert.x does provide something similar, but I think, that this is something that really belongs to the core of netty.
A couple of weeks/months ago I already asked for AsynchronousChannel support in Netty 4+5. If this feature would be re-integrated, it would be very easy to kill two birds with one shot. Because then file access could be implemented as a netty channel and the Pipe-abstraction could be implemented using a ChannelHandler which fetches input from one channel and pushes it into the other one.
What do you think about those kinds of changes ?

Ok, I finally found some time. I think, I'll start with making the ChunkedWriteHandler async, since writing files asynchronously seems to be more useful for now.
We should discuss the best way of implementing it.
I see two options:
Option 1:
Create an abstract class implementing ChunkedInput (lets call it ChunkedAsyncInput), which wraps the mechanics of using a future. An abstract implementation might look like this:
public abstract class ChunkedAsyncInput<B> implements ChunkedInput<B> {

    protected abstract Future<B> readChunkAsync(ChannelHandlerContext channelHandlerContext);

    private Future<B> pendingChunk;

    @Override
    public B readChunk(final ChannelHandlerContext ctx) throws Exception {
        if (isEndOfInput() || (pendingChunk != null && !pendingChunk.isDone())) {
            return null;
        }

        if (pendingChunk != null) {
            Future<B> tmp = pendingChunk;
            pendingChunk = null;
            if(tmp.isSuccess()) {
                return tmp.get();
            } else if(tmp.isCancelled()) {
                throw new CancellationException();
            } else {
                throw new IOException(tmp.cause());
            }
        } else {
            pendingChunk = readChunkAsync(ctx);
            pendingChunk.addListener(new GenericFutureListener<Future<? super B>>() {
                @Override
                public void operationComplete(Future<? super B> future) throws Exception {
                    ((ChunkedWriteHandler) ctx.handler()).resumeTransfer();
                }
            });
            return null;
        }
    }
}
This way, the ChunkedWriteHandler could be left unmodified, which is good, since a lot of testing already went into this class, I guess.

Option 2:
Create an async sibling of ChunkedInput, which operates completely asynchronously:
public interface ChunkedAsyncInput<B> {

    Future<Boolean> isEndOfInput();

    Future<B> readChunk(ChannelHandlerContext ctx);

    // Could be left unmodified, since these methods cannot be made async using stock jdk
    void close() throws Exception;
    long length();
    long progress();
}

In this case, we must extend the ChunkedWriteHandler to support the ChunkedAsyncInput interface.
So some kind of:
            if (pendingMessage instanceof ChunkedInput) { ... }
            else if (pendingMessage instanceof ChunkedAsyncInput) { ... }

Not as trivial as the first option, but probably cleaner.
As some final note: The method isEndOfInput() kind of causes some trouble, since it might read or not, in order to verify the end of a stream.
Any thoughts about this issue ?
Btw: I've setup netty already as a project + all your contribution rules. So making a pull request would be not much work for me ;)

@chrisprobst, did you ever get this implemented? This would be useful for a project I'm working on. How to properly use .resumeTransfer() is not at all clear to me, and it would be a lot nicer if this all worked asynchronously with no intervention.

PS I see a LazyChunkedInput in Play 1.x that might be helpful: https://github.com/playframework/play1/blob/master/framework/src/play/server/PlayHandler.java#L1024

I die über get a response for my proposal. Im not Wirkung with netty anymore so i have no plans to implementiert this currently. Sry.

Hello.
In a POC I tried to wite an IMAP server implementing "Streaming" for the IMAP responses.
Having such an abstraction would be ideal to me.


Support best-effort queries.

All set with #58

Awesome, we can close this issue then @danielmai ?

Added in 65d45150cf338a2126960e71179d6c9c3c6c4d51


Crash when checking options in Project Settings

Operating system or device - Godot version: Ubuntu 16.10, Clang 3.8.1 - Godot Git, compiled from commit https://github.com/godotengine/godot/commit/681575fa7123592897090c6cce44402c4e45baeb
Issue description (what happened, and what was expected):
Godot will crash when checking options in the Project Settings. It occasionally happens, not 100% of the time.
Steps to reproduce: Check some options in the Project Settings, eventually it will crash. GDB reports this as "exiting normally" though, so it doesn't seem to be possible to get a backtrace.

It happens for specific keys when this key is not present in engine.cfg and you change it's value.
The value is not being saved to engine.cfg.
One example I found is Rasterizer.android > fp16_framebuffer:
When clicking "On", it...
[ ] fp16_framebuffer [ ] On > crashes
[X] fp16_framebuffer [ ] On > doesn't crash
[X] fp16_framebuffer [X] On > doesn't crash

And it only occurs on checkboxes, which are not yet stored in the engine.cfg.

Cannot reproduce this on Windows.

Seems to be fixed for now, closing. Reopen if this happens again in the Git master branch.


Nightly CI rviz_default_plugins failed to build on OSX

The nightly repeated job failed to build last night: https://ci.ros2.org/job/nightly_osx_repeated/2339/console
stdout:
01:38:54 [ 38%] Building CXX object CMakeFiles/rviz_default_plugins.dir/src/rviz_default_plugins/displays/marker/marker_common.cpp.o
01:38:54 make[1]: *** [CMakeFiles/rviz_default_plugins.dir/all] Alarm clock: 14
01:38:54 make: *** [all] Error 2

stderr:
--- stderr: rviz_default_plugins
01:38:54 /Users/osrf/jenkins-agent/workspace/nightly_osx_repeated/ws/src/ros2/rviz/rviz_default_plugins/include/rviz_default_plugins/displays/image/get_transport_from_topic.hpp:0: Note: No relevant classes found. No output generated.
01:38:54 /Users/osrf/jenkins-agent/workspace/nightly_osx_repeated/ws/src/ros2/rviz/rviz_default_plugins/include/rviz_default_plugins/displays/image/image_transport_display.hpp:0: Note: No relevant classes found. No output generated.
01:38:54 make[1]: *** [CMakeFiles/rviz_default_plugins.dir/all] Alarm clock: 14
01:38:54 make: *** [all] Error 2
01:38:54 make: INTERNAL: Exiting with 3 jobserver tokens available; should be 4!
01:38:54 ---
01:38:54 Failed   <<< rviz_default_plugins [6min 9s, exited with code 2]

This one is a duplicate of https://github.com/ros2/ros2/issues/1049 (and is a bug in make itself).  I'll close this as a duplicate.


[SL Review]: Update Privacy Statement

User Story
“As a forms owner I want accurate information in the privacy statement so I can ensure I am in compliance with FOIPPA requirements”
Tasks

[x] update FOIPPA statement to read : "This information is being collected under the authority of the Freedom of Information and Protection of Privacy Act (FOIPPA), sections 27(1)(b) pursuant to 33(2)(h)(j), 26(c) and 26(e)".
[x]  hyperlink tagged to this sentence can remain the same

Acceptance Criteria

[x]  FOIPPA Statement now reads:  This information is being collected under the authority of the Freedom of Information and Protection of Privacy Act (FOIPPA), sections 27(1)(b) pursuant to 33(2)(h)(j), 26(c) and 26(e)".

@JodiLittle this has been updated in DEV
https://digital-journeys-dev.apps.silver.devops.gov.bc.ca/form/631f7d7a3d064f5de7a7a03a
please confirm it's all good and I can move it to TEST

All good!

@MissionT could you update this in test?

@MissionT The font seems OK to me. I guess you figured it out, otherwise, let me know to look into it.

Iman showed me how to update the CSS code to keep the text all black. All good now. This ticket can be closed. Chris


volume migration: avoid continuously creating mig objects if the migration fails

When the migration fails, the workload update continues to create new VMI migration object. Now, virt-controller implement the migration backoff mechanism also for workload update migrations
Add a functional test which replicates a migration failure when the destination image is smaller then source one.
Release note

NONE

/cc @awels @mhenriks

@awels right now if we encounter a failure in the migration, then on the VMI has the VolumeChange condition set to false with the error message  "The migration has failed". Would this be enough for you or do you want something more precise? In the VMI status there is also more detail with the reason of the failure, putting that info here as well is a little bit redundant. WDYT?

I am fine with just the migration has failed message. I can look up the additional information if I need it on my end.

@awels @mhenriks @fossedihelm could you please take another look to this. I have changed the implementation. Now, we have a new field in the VMI status which keeps track of the number of retries. I also added the exponential retries suggested by Mike in https://github.com/kubevirt/kubevirt/pull/12336#discussion_r1681145118

@alicefr We already have a backoff mechanism for evacuation migrations.
How about adding an annotation for volume update migrations and extend the behavior of handleMigrationBackoff ?https://github.com/kubevirt/kubevirt/blob/69c1833597ee13f199bcd8aee098f06c853b6ce3/pkg/virt-controller/watch/migration.go#L1267
This would save us a lot of ad hoc code. Have you considered that? Thank you

@alicefr We already have a backoff mechanism for evacuation migrations. How about adding an annotation for volume update migrations and extend the behavior of handleMigrationBackoff ?
https://github.com/kubevirt/kubevirt/blob/69c1833597ee13f199bcd8aee098f06c853b6ce3/pkg/virt-controller/watch/migration.go#L1267
This would save us a lot of ad hoc code, and of course an API change. Have you considered that? Thank you

I didn't know it, I'll look into it, thanks for the suggestion

@fossedihelm would you mind taking another look? It now uses the backoff mechanism

A little performance improvement suggestion. Also, can we add a unit test under the Migration backoff Context? Thank you!

@fossedihelm applied the last comments. PTAL

@alicefr Thank you!
/lgtm
Can you adjust the PR title :)

/remove-label kind/api-change

/approve
Great that you could get this to work with existing backoff machinery

/cherrypick release-1.3


APPS-1173 Apply updated config without service restart

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

Project coverage is 76.31%. Comparing base (656c2ae) to head (d79543f).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #170   +/-   ##
=======================================
  Coverage   76.31%   76.31%
=======================================
  Files          13       13
  Lines         553      553
=======================================
  Hits          422      422
  Misses        109      109
  Partials       22       22

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


WIP: Increase test automation coverage for Emoji picker feature

Summary
Increase test automation coverage for Emoji picker feature as the ticket describes.
Ticket Link
https://github.com/mattermost/mattermost-server/issues/11573
https://mattermost.atlassian.net/browse/MM-16739
Related Pull Requests

Has server changes (please link here)
Has redux changes (please link here)
Has mobile changes (please link here)

@thekiiingbob this is ready to review, please take a look when you have a chance.

@thekiiingbob thanks for the review, I fix the problems you mentioned in the review. please take a look when you have a chance, this is ready to review again.


Add GDPR controls

As requested by Commercial, we are now also mapping GDPR "controls" to Compliant Kubernetes documentation pages.
Adding @simonekman for approval, @llarsson for wordsmithing and @emmasundqvist for awareness.

@llarsson I committed all your suggestions, PTAL.

Nice! :)


feat: changed the design of applicationInfo.

Changed the design and the text of Application info and added new translations for it.

Also one test is failing and you probably have to adjust it: https://github.com/terrestris/shogun-admin/blob/main/src/Component/Modal/ApplicationInfo/ApplicationInfo.spec.tsx#L31


Download broken

The download link offered fails with 403 Forbidden.

Apologies for the broken link. SAAM does not provide the data without password, so I updated the wiki.


XSL 2.0

Some files haven't been updated

@rultor release, tag is 1.9.2

@rultor release, tag is 1.9.2

@yegor256 OK, I will release it now. Please check the progress here

@rultor release, tag is 1.9.2

@yegor256 Done! FYI, the full log is here (took me 13min)

The job is not in WBS, won't close the order


Remove need for "Waiting for secrets.yml"

This step seems kind of clunky. Instead of waiting for the user to create the file, couldn't we just create it with a random sequence for the secret_key?

While this role is clunky, you still do need to take time to ssh into an application and hand-configure somethings like API keys, ect. Until we start using something like ansible-vault to encrypt and store all our keys in version control I say we stick with the "waiting for secrets.yml" thing.
Closing for now.


Swift-cfenv 4.x is not compatible with Swift 3.0.2

It would be nice if Swift-cfenv maintained some level of back-compatibility with Swift. For example, Kitura 1.7 works with Swift 3.0.2 and Swift 3.1.1.
Currently I am seeing the following error when compiling my project on Swift 3.0.2 when using Kitura 1.7, CloudConfiguration 2.x, Swift-cfenv 4.x:
/tmp/041ffec28bc23291ae0b059c3a2123aa393d3991/swiftserver/Packages/CloudFoundryEnv-4.0.2/Sources/CloudFoundryEnv/AppEnv.swift:184:23: error: use of unresolved identifier 'NSRegularExpression'
      let regex = try NSRegularExpression(pattern: spec, options: NSRegularExpression.Options.caseInsensitive)
                      ^~~~~~~~~~~~~~~~~~~
Foundation.RegularExpression:1:12: note: did you mean 'RegularExpression'?
open class RegularExpression : Foundation.NSObject, NSCopying, NSCoding {
           ^
/tmp/041ffec28bc23291ae0b059c3a2123aa393d3991/swiftserver/Packages/CloudFoundryEnv-4.0.2/Sources/CloudFoundryEnv/AppEnv.swift:184:67: error: use of unresolved identifier 'NSRegularExpression'
      let regex = try NSRegularExpression(pattern: spec, options: NSRegularExpression.Options.caseInsensitive)
                                                                  ^~~~~~~~~~~~~~~~~~~
Foundation.RegularExpression:1:12: note: did you mean 'RegularExpression'?
open class RegularExpression : Foundation.NSObject, NSCopying, NSCoding {
           ^
<unknown>:0: error: build had 1 command failures

@tunniclm Let's discuss this during the planning meeting on Monday. We on purpose skipped supporting Swift 3.0.2 in the latest release.

This ship has sailed :)


Binary support

All the files are written at $USER/Library/Application Support/prem.tauri.dev

The Rust code should prevent starting download multiple times if is already in progress.
src/download.rs:210 2023-10-24T15:24:12 [INFO] - downloaded 26%: (/Users/tiero/Library/Application Support/prem.tauri.dev/models/mistral-7b-openorca/mistral-7b-openorca.Q5_K_S.gguf):<PHONE_NUMBER> bytes /<PHONE_NUMBER> bytes
src/controller_binaries.rs:334 2023-10-24T15:24:18 [INFO] - service_id: mistral-7b-openorca
src/download.rs:210 2023-10-24T15:24:24 [INFO] - downloaded 27%: (/Users/tiero/Library/Application Support/prem.tauri.dev/models/mistral-7b-openorca/mistral-7b-openorca.Q5_K_S.gguf):<PHONE_NUMBER> bytes /<PHONE_NUMBER> bytes
src/download.rs:210 2023-10-24T15:24:33 [INFO] - downloaded 19%: (/Users/tiero/Library/Application Support/prem.tauri.dev/models/mistral-7b-openorca/mistral-7b-openorca.Q5_K_S.gguf):  949581107 bytes /<PHONE_NUMBER> bytes
src/download.rs:210 2023-10-24T15:24:38 [INFO] - downloaded 28%: (/Users/tiero/Library/Application Support/prem.tauri.dev/models/mistral-7b-openorca/mistral-7b-openorca.Q5_K_S.gguf):<PHONE_NUMBER> bytes /<PHONE_NUMBER> bytes
src/download.rs:210 2023-10-24T15:24:48 [INFO] - downloaded 29%: (/Users/tiero/Library/Application Support/prem.tauri.dev/models/mistral-7b-openorca/mistral-7b-openorca.Q5_K_S.gguf):<PHONE_NUMBER> bytes /<PHONE_NUMBER> bytes
src/download.rs:210 2023-10-24T15:24:57 [INFO] - downloaded 20%: (/Users/tiero/Library/Application Support/prem.tauri.dev/models/mistral-7b-openorca/mistral-7b-openorca.Q5_K_S.gguf):  999550731 bytes /<PHONE_NUMBER> bytes
src/download.rs:210 2023-10-24T15:25:00 [INFO] - downloaded 30%: (/Users/tiero/Library/Application Support/prem.tauri.dev/models/mistral-7b-openorca/mistral-7b-openorca.Q5_K_S.gguf):<PHONE_NUMBER> bytes /<PHONE_NUMBER> bytes

We should have a guard in download_file
The UI does not shows the right progress when two models are downloaded at the same time

The settings icon is there. It's just covered by the flower when in dev mode.

Message not supported should be
"This models is only available on Prem Server installation"

I would add a checkbox or toggle on the top right that says "Show only available services" that hides all the not usable services. It's ok to start "un-checked" or "un-toggled"

Also as improvement, I would prevent the user to use the playground of the service is the model is not loaded in memory, otherwise an uase that click on the Open and try to chat right away with Mistra will have "Load failed" error


improve Exercise 20.4.5

the explanation for x[x <= 0] is quite good, but I suppose we should go into more details for x[-which(x > 0)] , like
> x <- c(-5:5, Inf, -Inf, NaN, NA)

> x > 0
 [1] FALSE FALSE FALSE FALSE FALSE FALSE  TRUE  TRUE  TRUE  TRUE  TRUE  TRUE FALSE    NA    NA

> which(x > 0)
[1]  7  8  9 10 11 12

> -which(x > 0)
[1]  -7  -8  -9 -10 -11 -12

I think we should tell the story step by step, especially in a book for beginners:

what > 0 does?
what which() does?
what - does?
what [] (with negative integers) does?

Let me think about it. I think there is a fine line between going in depth to answer a question, and reproducing the discussion in R4DS. I think this falls on the scale of okay to extend the depth of R4DS. But for some things may be worth opening an Issue in the R4DS repo if the main text should include a concept or does not describe it clearly.

Do you have an idea of what the explanation should look like? Would you want to open a PR for it?


DELETE support subquery

delete 语句 删除条件 希望支持子查询结果
I hope that DELETE statement support subquery

It's in our plan.
Of course, it will be a grate pleasure if you can summit PRs to solve this feature.


feature: ensure the correct versions of the server-admin-ui and plugin-config is installed

Now thinking maybe we should have some way to indicate that a package is an "internal package" for node server.  @rob42 is going to need a way to ignore these.

Remove the signalk-webapp keyword and mount it explicitly?

Can move to /admin path at the same time?


PDF output PR welcome or should we keep it separated?

We think about creating a PDF output module for this library but I'm not sure if we should keep it as an external repository/module or if you are willed to accept a PR for a default implementation?
We would use FPDF under the hood and the QR codes will be created as simple vector based PDF files. I'm also not sure if (in case for a default implementation) the dependency to FPDF should be hard (in the composer.json) or optional.
The class name for a default implementation would be  chillerlan\QRCode\Output\QRFpdf.
If the created QR codes needs to be reused in dynamic created PDFs they can be reimported by FPDI.
What do you think? Separate repository/module or default implementation? :-)

Hey, this is a great idea - I'd gladly accept a PR to offer it as another built-in output module! I'd keep the dependency optional under require-dev, with additional info in the readme, so that users can decide for themselves whether or not to install it.

Done! It's now released as v4.1.0 (PHP 7.4+) and v3.3.0 (PHP 7.2+). Again, thank you for the contribution!


add requirements.txt

Adds requirements file.
@helrond Not sure if this is the optimal fix, but my guess is that we're getting a ping because the requirements are never installed and CyberX is treating it as anything with the CVE

Closing this. Let's stick with our established pattern.


Update Jobs Marketplace” or “Opportunities Platform” to “Career Marketplace”

Related to issue https://github.com/gctools-outilsgc/gcconnex/issues/1763

Good catch, thanks Steph!


Override Persistent History Clearing

currently, CloudCore assumes ownership of NSPersistentHistoryTracking, clearing out history after its been properly pushed to CloudKit.  But NSPersistentHistoryTracking is also useful for shared containers and app extensions.  Need to refactor such that apps can override default history clearing behavior.

How would you handle the replacement of Core Data?
do {
       try container.persistentStoreCoordinator.replacePersistentStore(at: storeUrl, destinationOptions: nil, withPersistentStoreFrom: inputURL, sourceOptions: nil, ofType: NSSQLiteStoreType)
} catch {
      print("Failed to replace persistent container...NC") }//End do-catch
}

I can successfully load the Core Data backup but get NSPersistentHistoryToken errors:
Fatal error: Unresolved error Error Domain=NSCocoaErrorDomain Code=134501 "(null)" UserInfo={Reason=Store - (
    "4F49429F-3853-4978-9A43-AE28A1B644AE"
) not found in token - <NSPersistentHistoryToken - {
    "4F49429F-3853-4978-9A43-AE28A1B644AE" = 1;
}>}, ["Reason": Store - (
    "4F49429F-3853-4978-9A43-AE28A1B644AE"
)

I'm  having a hard time Manually clearing Persistent History Tokens. Any advice?


Solid app is not enabled when server starts

I added test scripts for Solid. They almost work, just one small issue:
When you run:
./scripts/init-solid.sh
./scripts/testing-solid.sh

And then log in as einstein, you still have to go to the Apps menu, find the Solid app, click 'enable untested app', click 'enable' a second time, and then the Solid app is enabled. It would be nice if we could automate that.
Apparently https://github.com/pondersource/dev-stock/blob/4391b50/docker/scripts/init-nextcloud-solid.sh#L12 is not enough?

It needs https://github.com/pdsinterop/test-suites/blob/main/servers/nextcloud-server/server/init.sh

create something like this one https://github.com/pondersource/dev-stock/blob/main/docker/scripts/init-nextcloud-sciencemesh.sh with data from the repository you have mentioned

It already exists
https://github.com/pondersource/dev-stock/blob/main/docker/scripts/init-nextcloud-solid.sh
my bad :)

if its always Nextcloud, change ${EFSS1} with nc in
https://github.com/pondersource/dev-stock/blob/main/scripts/testing-solid.sh

yes! on it. thanks!


Added WebAssembly features

Resolves mdn/browser-compat-data/issues/5796
Thanks @queengooborg for the new structure, it's a lot easier to make sense of the repo now.
Basic structure:
Added the wasm-feature-detect library, and testWasmFeature to the global bcd object.
testWasmFeature just calls wasm-feature-detect.
All the tests are in custom/wasm.json, where they have a readable name as the key, and the wasm-feature-detect name as the wfd-key value.

Thank you for working on this, @MendyBerger!  This is looking quite wonderful so far!
There are a couple of things that we should do to ensure that the collector will still run in older browser versions:

First, async/await syntax cannot be used, especially in harness.js -- luckily, we can just remove those keywords, as the collector is designed to handle promises!
We should try to avoid adding the library's script directly to tests.ejs, and instead write a script to try and load the script, or set the wasmFeatureDetect variable to false if it couldn't be loaded.

On that note, in the testWasmFeature() function, there should be additional checks in place to make sure that wasmFeatureDetect is defined and is truthy, and that [feature] is defined in that variable

We should also put the testWasmFeature() function below testCSSProperty(), especially as right now, it's defined right below the JSDoc for testCSSProperty()

If you could easily do those things, please do. Otherwise, I'll take another look when I have the time.
Either way, let me know.

@queengooborg how long should it take until these results show up in caniuse.com?

I can't say how long it would take for CanIUse to obtain them because it's a third party service, but for BCD, I'm hoping to get them added as soon as the new collector version is released.  It may be a bit because I'm planning to increase category coverage and have a few other updates I wish to make, but once it's out I'll get a PR going to add this new category into BCD!


[SIL] Remove dependency on SILConstants.h from SILInstruction.h.

Move GraphOperationAttribute to SILConstants.h and forward declare it in
SILInstruction.h.
Change GraphOperationInst to dynamically allocate attributes rather than
tail allocation. This is necessary because GraphOperationAttribute cannot
be used as a template parameter to TrailingObjects.
Move GraphOperationInst::getAttributes and related methods to
SILInstructions.cpp.
Include SILConstants.h in source files that use GraphOperationAttribute.

Quality-of-life improvement requested by @lattner.
Previously, since SILInstruction.h included SILConstants.h, any change to SILConstants.h caused recompilation of all of SIL.

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow

@swift-ci Please test tensorflow

Thanks Chris!
On my Mac Pro, ninja swift takes <30 seconds after making a change to SILConstants.h.


Cloud deployment

Hey guys,
I have a private repository running on the cloud and I would like to use Portus as authentication provider and allow only authenticated parties to access the repository over the Internet. The biggest challenge with the current implementation is that it bundles together auth and UI administration and more importantly, it allows users to register on their own. This is really a big show stopper as I wouldn't want to expose a UI to the world where everyone can register.
By looking at the current configuration, I guess I could configure a reverse proxy exposing /v2/ to the world and disabling the access of anything else unless it comes from a trusted origin. Would that work? Do you have other suggestions?

Hello @boyand !

I have a private repository running on the cloud and I would like to use Portus as authentication provider and allow only authenticated parties to access the repository over the Internet. The biggest challenge with the current implementation is that it bundles together auth and UI administration and more importantly, it allows users to register on their own. This is really a big show stopper as I wouldn't want to expose a UI to the world where everyone can register.

So, if I read it correctly you have two issues here:

Users should not be able to register: this is a known issue, and it's currently on the works. You might want to subscribe to the issue #179.
The authorization service is glued to the UI: this has honestly been done on purpose. Otherwise how would you manage the permissions of each users, create namespaces, etc. ? We believe that a nice web UI on top of all this is easier to manage and more comfortable. However, if you have an idea on this, please don't be shy ;)

By looking at the current configuration, I guess I could configure a reverse proxy exposing /v2/ to the world and disabling the access of anything else unless it comes from a trusted origin. Would that work? Do you have other suggestions?

If you want to check configurations on the matter, you might want to take a look at this NGinx configuration, or most probably, this configuration in which we setup Portus and the registry on the same FQDN. Now, what you propose also sounds good, and it might work, but the devil is in the detail ;) Maybe you might want to provide provide the configuration you're working on so we can help you out :)

Hi @mssola,
Thanks a lot for your quick feedback. You are sport on - the main challenge is the users sign up form being enabled by default. Thanks for pointing out the issue - I am following it now. Once it is resolved, I think the setup I have in mind will work flawlessly.
The configuration I am working on looks like this:

@boyand After that issue is closed, this setup you're trying to implement should be supported.

@boyand note that in the code of current master this has already been implemented. There is one small extra feature that is left to be done, but besides that, now it should work just fine for you ;)

@boyand this is now implemented and documented. You can read about the feature itself here and how to configure it here.